Skip to content

feat(SimpleGraph): cycleGraph.cycle is a cycle#34797

Open
vlad902 wants to merge 2 commits intoleanprover-community:masterfrom
vlad902:cycleGraph-eulerianCircuit
Open

feat(SimpleGraph): cycleGraph.cycle is a cycle#34797
vlad902 wants to merge 2 commits intoleanprover-community:masterfrom
vlad902:cycleGraph-eulerianCircuit

Conversation

@vlad902
Copy link
Copy Markdown
Collaborator

@vlad902 vlad902 commented Feb 3, 2026

@vlad902 vlad902 changed the title feat(SimpleGraph): cycleGraph is a cycle and IsContained in every graph with a cycle feat(SimpleGraph): cycleGraph is a cycle and IsContained in every graph with a cycle Feb 3, 2026
@github-actions github-actions bot added the t-combinatorics Combinatorics label Feb 3, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 3, 2026

PR summary db6ec05280

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ cycleGraph.getVert_cycle
+ cycleGraph.getVert_cycleCons
+ cycleGraph.isCycle_cycle
+ cycleGraph.isPath_tail_cycle

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

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

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


No changes to technical debt.

You can run this locally as

./scripts/reporting/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@mathlib-dependent-issues mathlib-dependent-issues bot added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Feb 3, 2026
Comment thread Mathlib/Combinatorics/SimpleGraph/Circulant.lean Outdated
Comment thread Mathlib/Combinatorics/SimpleGraph/Circulant.lean Outdated
Comment thread Mathlib/Combinatorics/SimpleGraph/Circulant.lean Outdated
@mathlib-dependent-issues mathlib-dependent-issues bot removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Feb 9, 2026
@vlad902 vlad902 force-pushed the cycleGraph-eulerianCircuit branch from e3a8af9 to a04966e Compare February 13, 2026 13:55
@vlad902 vlad902 changed the title feat(SimpleGraph): cycleGraph is a cycle and IsContained in every graph with a cycle feat(SimpleGraph): cycleGraph is a cycle Feb 13, 2026
@vlad902 vlad902 changed the title feat(SimpleGraph): cycleGraph is a cycle feat(SimpleGraph): cycleGraph_EulerianCircuit is a cycle Feb 13, 2026
Comment thread Mathlib/Combinatorics/SimpleGraph/Circulant.lean Outdated
@Ruben-VandeVelde Ruben-VandeVelde added the awaiting-author A reviewer has asked the author a question or requested changes. label Feb 14, 2026
vlad902 added a commit to vlad902/mathlib4 that referenced this pull request Feb 15, 2026
….cycle`

Per
[this](leanprover-community#34797 (comment))
review feedback, this definition is inappropriately named with an
underscore and should be renamed.
@vlad902 vlad902 removed the awaiting-author A reviewer has asked the author a question or requested changes. label Mar 8, 2026
@mathlib-dependent-issues mathlib-dependent-issues bot added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Mar 8, 2026
vlad902 added a commit to vlad902/mathlib4 that referenced this pull request Mar 14, 2026
….cycle`

Per
[this](leanprover-community#34797 (comment))
review feedback, this definition is inappropriately named with an
underscore and should be renamed.
@vlad902 vlad902 force-pushed the cycleGraph-eulerianCircuit branch from c9a756e to 21f85c3 Compare March 14, 2026 14:40
@vlad902 vlad902 changed the title feat(SimpleGraph): cycleGraph_EulerianCircuit is a cycle feat(SimpleGraph): cycleGraph.cycle is a cycle Mar 14, 2026
mathlib-bors bot pushed a commit that referenced this pull request Mar 19, 2026
)

Per [this](#34797 (comment)) review feedback, this definition is inappropriately named with an underscore and should be renamed.
(A follow-up PR will show that `cycleGraph.cycle` is also hamiltonian.)
@vlad902 vlad902 force-pushed the cycleGraph-eulerianCircuit branch from 21f85c3 to 1ae94a7 Compare March 19, 2026 17:16
@mathlib-dependent-issues mathlib-dependent-issues bot removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Mar 19, 2026
justus-springer pushed a commit to justus-springer/mathlib4 that referenced this pull request Mar 28, 2026
…nprover-community#35360)

Per [this](leanprover-community#34797 (comment)) review feedback, this definition is inappropriately named with an underscore and should be renamed.
vlad902 added a commit to vlad902/mathlib4 that referenced this pull request Apr 11, 2026
PR leanprover-community#34797 re-defined `cycleGraph` independent of `circulantGraph`. Now
move the definition of `cycleGraph` to its own file so that the
definition can be used without importing the algebra hierarchy.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-combinatorics Combinatorics

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants