Skip to content

[Merged by Bors] - chore: add deprecated modules#29209

Closed
adomani wants to merge 4 commits intoleanprover-community:masterfrom
adomani:module_deprecation
Closed

[Merged by Bors] - chore: add deprecated modules#29209
adomani wants to merge 4 commits intoleanprover-community:masterfrom
adomani:module_deprecation

Conversation

@adomani
Copy link
Copy Markdown
Contributor

@adomani adomani commented Sep 1, 2025

These modules were renamed in #28948 and #28909, and this is semi-autogenerated PR to deprecate the old module names.

These are 9 Data.{Complex, Real}.xxx files that have been renamed to Analysis.{Complex, Real}.xxx.

I had forgotten about running lake exe mk_all after the deprecated modules had been generated.


Open in Gitpod

@github-actions github-actions bot added the t-data Data (lists, quotients, numbers, etc) label Sep 1, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 1, 2025

PR summary 8acd6dfb9e

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference
Mathlib.Data.Complex.Exponential (new file) 1304
Mathlib.Data.Complex.Trigonometric (new file) 1305
Mathlib.Data.Real.Hyperreal (new file) 1444
Mathlib.Data.Real.Cardinality (new file) 1451
Mathlib.Data.Complex.ExponentialBounds (new file) 1884
Mathlib.Data.Real.Pi.Bounds (new file) 1958
Mathlib.Data.Real.Pi.Wallis (new file) 2389
Mathlib.Data.Real.Pi.Leibniz (new file) 2395
Mathlib.Data.Real.Pi.Irrational (new file) 2399

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.


No changes to technical debt.

You can run this locally as

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

@grunweg
Copy link
Copy Markdown
Contributor

grunweg commented Sep 1, 2025

Thank you for doing this! Would you like to also do #28909 and the moving of Data/Matrix files? :-)
maintainer merge

@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 1, 2025

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

@ghost ghost added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Sep 1, 2025
@adomani
Copy link
Copy Markdown
Contributor Author

adomani commented Sep 1, 2025

I added also the deprecations from #28909. This actually uncovered a bug in the process: if a directory no longer exist, the old script would fail. I fixed this now.

@bryangingechen
Copy link
Copy Markdown
Contributor

Thanks!
bors r+

@ghost ghost added ready-to-merge This PR has been sent to bors. and removed maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. labels Sep 2, 2025
mathlib-bors bot pushed a commit that referenced this pull request Sep 2, 2025
These modules were renamed in #28948 and #28909, and this is semi-autogenerated PR to deprecate the old module names.

These are 9 `Data.{Complex, Real}.xxx` files that have been renamed to `Analysis.{Complex, Real}.xxx`.

I had forgotten about running `lake exe mk_all` after the deprecated modules had been generated.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Sep 2, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: add deprecated modules [Merged by Bors] - chore: add deprecated modules Sep 2, 2025
@mathlib-bors mathlib-bors bot closed this Sep 2, 2025
CBirkbeck pushed a commit to CBirkbeck/mathlib4 that referenced this pull request Sep 8, 2025
These modules were renamed in leanprover-community#28948 and leanprover-community#28909, and this is semi-autogenerated PR to deprecate the old module names.

These are 9 `Data.{Complex, Real}.xxx` files that have been renamed to `Analysis.{Complex, Real}.xxx`.

I had forgotten about running `lake exe mk_all` after the deprecated modules had been generated.
yuanyi-350 pushed a commit to yuanyi-350/yuanyi_mathlib4 that referenced this pull request Sep 10, 2025
These modules were renamed in leanprover-community#28948 and leanprover-community#28909, and this is semi-autogenerated PR to deprecate the old module names.

These are 9 `Data.{Complex, Real}.xxx` files that have been renamed to `Analysis.{Complex, Real}.xxx`.

I had forgotten about running `lake exe mk_all` after the deprecated modules had been generated.
robertmaxton42 pushed a commit to robertmaxton42/mathlib4 that referenced this pull request Sep 11, 2025
These modules were renamed in leanprover-community#28948 and leanprover-community#28909, and this is semi-autogenerated PR to deprecate the old module names.

These are 9 `Data.{Complex, Real}.xxx` files that have been renamed to `Analysis.{Complex, Real}.xxx`.

I had forgotten about running `lake exe mk_all` after the deprecated modules had been generated.
mathlib-bors bot pushed a commit that referenced this pull request Sep 17, 2025
Update references of `Data.Real.Pi.*` modules to `Analysis.Real.Pi.*` since they were marked deprecated in #29209.
joelriou pushed a commit to joelriou/mathlib4 that referenced this pull request Oct 2, 2025
These modules were renamed in leanprover-community#28948 and leanprover-community#28909, and this is semi-autogenerated PR to deprecate the old module names.

These are 9 `Data.{Complex, Real}.xxx` files that have been renamed to `Analysis.{Complex, Real}.xxx`.

I had forgotten about running `lake exe mk_all` after the deprecated modules had been generated.
joelriou pushed a commit to joelriou/mathlib4 that referenced this pull request Oct 2, 2025
…mmunity#29653)

Update references of `Data.Real.Pi.*` modules to `Analysis.Real.Pi.*` since they were marked deprecated in leanprover-community#29209.
zhuyizheng pushed a commit to zhuyizheng/mathlib4 that referenced this pull request Oct 2, 2025
…mmunity#29653)

Update references of `Data.Real.Pi.*` modules to `Analysis.Real.Pi.*` since they were marked deprecated in leanprover-community#29209.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors. t-data Data (lists, quotients, numbers, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants