Skip to content

[Merged by Bors] - chore(NumberTheory/Cyclotomic/CyclotomicCharacter): fix name to camelCase#24521

Closed
acmepjz wants to merge 4 commits intomasterfrom
acmepjz_fix_cycl_char_name
Closed

[Merged by Bors] - chore(NumberTheory/Cyclotomic/CyclotomicCharacter): fix name to camelCase#24521
acmepjz wants to merge 4 commits intomasterfrom
acmepjz_fix_cycl_char_name

Conversation

@acmepjz
Copy link
Copy Markdown
Collaborator

@acmepjz acmepjz commented May 1, 2025

Fix the name of ModularCyclotomicCharacter and CyclotomicCharacter to camelCase, since they are not Type.

Also fix the module docstring.


Open in Gitpod

@acmepjz acmepjz added awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. t-number-theory Number theory (also use t-algebra or t-analysis to specialize) t-algebra Algebra (groups, rings, fields, etc) labels May 1, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented May 1, 2025

PR summary 0b9af5bd97

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ CyclotomicCharacter.toFun_apply
+ CyclotomicCharacter.toFun_spec
+ CyclotomicCharacter.toZModPow_toFun
+ IsPrimitiveRoot.autToPow_eq_modularCyclotomicCharacter
+ ModularCyclotomicCharacter.comp
+ ModularCyclotomicCharacter.id
+ ModularCyclotomicCharacter.spec
+ ModularCyclotomicCharacter.toFun_spec
+ ModularCyclotomicCharacter.toFun_spec'
+ ModularCyclotomicCharacter.toFun_spec''
+ ModularCyclotomicCharacter.toFun_unique
+ ModularCyclotomicCharacter.toFun_unique'
+ ModularCyclotomicCharacter.unique
+ cyclotomicCharacter
+ cyclotomicCharacter.continuous
+ cyclotomicCharacter.spec
+ cyclotomicCharacter.toFun
+ cyclotomicCharacter.toZModPow
+ modularCyclotomicCharacter
+ modularCyclotomicCharacter'
+ modularCyclotomicCharacter'.spec'
+ modularCyclotomicCharacter'.unique'
+ modularCyclotomicCharacter.aux
+ modularCyclotomicCharacter.aux_spec
+ modularCyclotomicCharacter.pow_dvd_aux_pow_sub_aux_pow
+ modularCyclotomicCharacter.toFun

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

@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 May 1, 2025
@grunweg
Copy link
Copy Markdown
Contributor

grunweg commented May 1, 2025

Thanks for doing this! Can you add deprecated aliasses for the renamed definitions, please?
(scripts/add_deprecations.sh should help with that - or you can do so by hand.)

@grunweg grunweg added the awaiting-author A reviewer has asked the author a question or requested changes. label May 1, 2025
@acmepjz
Copy link
Copy Markdown
Collaborator Author

acmepjz commented May 2, 2025

Done. Hopefully I'm doing correctly (there are several).

@acmepjz acmepjz removed the awaiting-author A reviewer has asked the author a question or requested changes. label May 2, 2025
Copy link
Copy Markdown
Contributor

@Ruben-VandeVelde Ruben-VandeVelde left a comment

Choose a reason for hiding this comment

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

Thanks

maintainer merge

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 2, 2025

🚀 Pull request has been placed on the maintainer queue by Ruben-VandeVelde.

@github-actions github-actions bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label May 2, 2025
Copy link
Copy Markdown
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

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

Thanks 🎉

bors merge

@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 May 2, 2025
mathlib-bors bot pushed a commit that referenced this pull request May 2, 2025
…Case (#24521)

Fix the name of `ModularCyclotomicCharacter` and `CyclotomicCharacter` to camelCase, since they are not `Type`.

Also fix the module docstring.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented May 2, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore(NumberTheory/Cyclotomic/CyclotomicCharacter): fix name to camelCase [Merged by Bors] - chore(NumberTheory/Cyclotomic/CyclotomicCharacter): fix name to camelCase May 2, 2025
@mathlib-bors mathlib-bors bot closed this May 2, 2025
@mathlib-bors mathlib-bors bot deleted the acmepjz_fix_cycl_char_name branch May 2, 2025 14:50
pfaffelh pushed a commit that referenced this pull request May 2, 2025
…Case (#24521)

Fix the name of `ModularCyclotomicCharacter` and `CyclotomicCharacter` to camelCase, since they are not `Type`.

Also fix the module docstring.
riccardobrasca pushed a commit that referenced this pull request May 7, 2025
…Case (#24521)

Fix the name of `ModularCyclotomicCharacter` and `CyclotomicCharacter` to camelCase, since they are not `Type`.

Also fix the module docstring.
tannerduve pushed a commit that referenced this pull request May 13, 2025
…Case (#24521)

Fix the name of `ModularCyclotomicCharacter` and `CyclotomicCharacter` to camelCase, since they are not `Type`.

Also fix the module docstring.
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-algebra Algebra (groups, rings, fields, etc) t-number-theory Number theory (also use t-algebra or t-analysis to specialize)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants