Skip to content

chore: rename continuous{,On,At,Within}_const to ContinuousFoo.const#31607

Open
grunweg wants to merge 5 commits intoleanprover-community:masterfrom
grunweg:rename-continuousconst
Open

chore: rename continuous{,On,At,Within}_const to ContinuousFoo.const#31607
grunweg wants to merge 5 commits intoleanprover-community:masterfrom
grunweg:rename-continuousconst

Conversation

@grunweg
Copy link
Copy Markdown
Contributor

@grunweg grunweg commented Nov 13, 2025

@grunweg grunweg added the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Nov 13, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 13, 2025

PR summary 56e100aabd

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ Continuous.const
+ ContinuousAt.const
+ ContinuousOn.const
+ ContinuousWithinAt.const

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).

Copy link
Copy Markdown
Contributor

@j-loreaux j-loreaux left a comment

Choose a reason for hiding this comment

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

I don't think any of these are required to be protected, because we don't have const in the root namespace. If you think it's better to do so, that's fine.

bors d+

((hff' x hx).const_mul (g b - g a)).sub ((hgg' x hx).const_mul (f b - f a))
have hhc : ContinuousOn h (Icc a b) :=
(continuousOn_const.mul hfc).sub (continuousOn_const.mul hgc)
(ContinuousOn.const.mul hfc).sub (ContinuousOn.const.mul hgc)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not necessarily saying you should do this, but I expect this would work:

Suggested change
(ContinuousOn.const.mul hfc).sub (ContinuousOn.const.mul hgc)
.sub (.mul .const hfc) (.mul .const hgc)

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Nov 15, 2025

✌️ grunweg can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@ghost ghost added the delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). label Nov 15, 2025
@grunweg grunweg changed the title chore: rename continuous{,On,At,Within}_const to ContinuousXXX.const` chore: rename continuous{,On,At,Within}_const to ContinuousFoo.const` Nov 15, 2025
@mathlib4-merge-conflict-bot
Copy link
Copy Markdown
Collaborator

This pull request has conflicts, please merge master and resolve them.

@mathlib4-merge-conflict-bot mathlib4-merge-conflict-bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jan 7, 2026
@grunweg grunweg removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jan 26, 2026
grunweg added a commit to grunweg/mathlib4 that referenced this pull request Jan 26, 2026
The goal of this PR is to remove many explicit references to continuous(On)_const;
to reduce the diff in leanprover-community#31607
grunweg added a commit to grunweg/mathlib4 that referenced this pull request Jan 26, 2026
The goal of this PR is to remove many explicit references to continuous(On)_const;
to reduce the diff in leanprover-community#31607
@github-actions github-actions bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jan 27, 2026
mathlib-bors bot pushed a commit that referenced this pull request Jan 30, 2026
The goal of this PR is to remove many explicit references to continuous(On)_const; to reduce the diff in #31607.
@github-actions github-actions bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jan 30, 2026
@mathlib4-merge-conflict-bot mathlib4-merge-conflict-bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Feb 1, 2026
@mathlib4-merge-conflict-bot
Copy link
Copy Markdown
Collaborator

This pull request has conflicts, please merge master and resolve them.

@grunweg grunweg force-pushed the rename-continuousconst branch from 86fc9bd to 8501fa8 Compare February 1, 2026 16:13
@github-actions github-actions bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Feb 1, 2026
grunweg added a commit to grunweg/mathlib4 that referenced this pull request Feb 1, 2026
Use fun_prop more, and make indentation match mathlib style better.
Extracted from leanprover-community#31607.
@mathlib4-dependent-issues-bot mathlib4-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 1, 2026
mathlib-bors bot pushed a commit that referenced this pull request Feb 1, 2026
Use fun_prop more, and make indentation match mathlib style better. Extracted from #31607.
@mathlib4-dependent-issues-bot mathlib4-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 2, 2026
@mathlib4-dependent-issues-bot
Copy link
Copy Markdown
Collaborator

@mathlib4-merge-conflict-bot mathlib4-merge-conflict-bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Feb 2, 2026
@mathlib4-merge-conflict-bot
Copy link
Copy Markdown
Collaborator

This pull request has conflicts, please merge master and resolve them.

@github-actions github-actions bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Feb 2, 2026
@Ruben-VandeVelde Ruben-VandeVelde changed the title chore: rename continuous{,On,At,Within}_const to ContinuousFoo.const` chore: rename continuous{,On,At,Within}_const to ContinuousFoo.const Feb 2, 2026
YellPika pushed a commit to YellPika/mathlib4 that referenced this pull request Feb 3, 2026
The goal of this PR is to remove many explicit references to continuous(On)_const; to reduce the diff in leanprover-community#31607.
@github-actions github-actions bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Feb 15, 2026
michaellee94 pushed a commit to michaellee94/mathlib4 that referenced this pull request Feb 15, 2026
The goal of this PR is to remove many explicit references to continuous(On)_const; to reduce the diff in leanprover-community#31607.
michaellee94 pushed a commit to michaellee94/mathlib4 that referenced this pull request Feb 15, 2026
Use fun_prop more, and make indentation match mathlib style better. Extracted from leanprover-community#31607.
@mathlib-merge-conflicts
Copy link
Copy Markdown

This pull request has conflicts, please merge master and resolve them.

@mathlib-merge-conflicts mathlib-merge-conflicts bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Feb 17, 2026
Maldooor pushed a commit to Maldooor/mathlib4 that referenced this pull request Feb 25, 2026
The goal of this PR is to remove many explicit references to continuous(On)_const; to reduce the diff in leanprover-community#31607.
Maldooor pushed a commit to Maldooor/mathlib4 that referenced this pull request Feb 25, 2026
Use fun_prop more, and make indentation match mathlib style better. Extracted from leanprover-community#31607.
pfaffelh pushed a commit to pfaffelh/mathlib4 that referenced this pull request Mar 2, 2026
The goal of this PR is to remove many explicit references to continuous(On)_const; to reduce the diff in leanprover-community#31607.
pfaffelh pushed a commit to pfaffelh/mathlib4 that referenced this pull request Mar 2, 2026
Use fun_prop more, and make indentation match mathlib style better. Extracted from leanprover-community#31607.
@github-actions github-actions bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 7, 2026
@mathlib-merge-conflicts mathlib-merge-conflicts bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 8, 2026
@mathlib-merge-conflicts
Copy link
Copy Markdown

This pull request has conflicts, please merge master and resolve them.

@github-actions github-actions bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 13, 2026
@mathlib-merge-conflicts mathlib-merge-conflicts bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 13, 2026
@mathlib-merge-conflicts
Copy link
Copy Markdown

This pull request has conflicts, please merge master and resolve them.

grunweg added a commit to grunweg/mathlib4 that referenced this pull request Mar 18, 2026
This issue was pre-existing, but has been exposed through leanprover-community#31607 where
it caused test failures.
@github-actions github-actions bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 18, 2026
@grunweg grunweg force-pushed the rename-continuousconst branch from 1553504 to e7a1acf Compare March 18, 2026 14:31
@github-actions github-actions bot removed the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Mar 18, 2026
@mathlib-merge-conflicts mathlib-merge-conflicts bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 20, 2026
@mathlib-merge-conflicts
Copy link
Copy Markdown

This pull request has conflicts, please merge master and resolve them.

@github-actions github-actions bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 21, 2026
@mathlib-merge-conflicts mathlib-merge-conflicts bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 23, 2026
@mathlib-merge-conflicts
Copy link
Copy Markdown

This pull request has conflicts, please merge master and resolve them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants