Skip to content

[Merged by Bors] - chore(Computability/ContextFreeGrammar): change the definition `Langu…#16268

Closed
madvorak wants to merge 1 commit intomasterfrom
madvorak-cf
Closed

[Merged by Bors] - chore(Computability/ContextFreeGrammar): change the definition `Langu…#16268
madvorak wants to merge 1 commit intomasterfrom
madvorak-cf

Conversation

@madvorak
Copy link
Copy Markdown
Collaborator

…age.IsContextFree` for consistency with #13554


Open in Gitpod

@madvorak madvorak added the easy < 20s of review time. See the lifecycle page for guidelines. label Aug 29, 2024
@github-actions
Copy link
Copy Markdown

PR summary 9ab38f61d1

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

No declarations were harmed in the making of this PR! 🐙

You can run this locally as follows
## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>

The doc-module for script/declarations_diff.sh contains some details about this script.

@madvorak madvorak added the t-computability Computability theory (TMs, DFAs, languages, grammars, etc) label Aug 29, 2024
Copy link
Copy Markdown
Contributor

@YaelDillies YaelDillies left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maintainer merge

@github-actions
Copy link
Copy Markdown

🚀 Pull request has been placed on the maintainer queue by YaelDillies.

@github-actions github-actions bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Aug 30, 2024
∃ g : ContextFreeGrammar.{0} T, g.language = L

proof_wanted Language.isContextFree_iff {L : Language T} :
L.IsContextFree ↔ ∃ g : ContextFreeGrammar.{uN} T, g.language = L
Copy link
Copy Markdown
Member

@urkud urkud Aug 31, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this true? Can't you show that some simple language in Type 3 can't be IsContextFree in the sense of the new definition by cardinality reasons? Or do we require [Fintype _] somewhere along the path? UPD: ignore this, we can't reach more than countably many words, because we use a list of rules and a unique initial element.

@urkud
Copy link
Copy Markdown
Member

urkud commented Aug 31, 2024

Thanks! 🎉
bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Aug 31, 2024
mathlib-bors bot pushed a commit that referenced this pull request Aug 31, 2024
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Aug 31, 2024

This PR was included in a batch that was canceled, it will be automatically retried

mathlib-bors bot pushed a commit that referenced this pull request Aug 31, 2024
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Aug 31, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore(Computability/ContextFreeGrammar): change the definition `Langu… [Merged by Bors] - chore(Computability/ContextFreeGrammar): change the definition `Langu… Aug 31, 2024
@mathlib-bors mathlib-bors bot closed this Aug 31, 2024
@mathlib-bors mathlib-bors bot deleted the madvorak-cf branch August 31, 2024 14:49
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 9, 2024
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 9, 2024
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

easy < 20s of review time. See the lifecycle page for guidelines. maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. ready-to-merge This PR has been sent to bors. t-computability Computability theory (TMs, DFAs, languages, grammars, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants