Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Rename FunctionField to AbsSimpleFunctionField#2416

Merged
thofma merged 2 commits into
Nemocas:masterNemocas/AbstractAlgebra.jl:masterfrom
alexey-orlov-math:funfield-rename-abssimplealexey-orlov-math/AbstractAlgebra.jl:funfield-rename-abssimpleCopy head branch name to clipboard
Jun 8, 2026
Merged

Rename FunctionField to AbsSimpleFunctionField#2416
thofma merged 2 commits into
Nemocas:masterNemocas/AbstractAlgebra.jl:masterfrom
alexey-orlov-math:funfield-rename-abssimplealexey-orlov-math/AbstractAlgebra.jl:funfield-rename-abssimpleCopy head branch name to clipboard

Conversation

@alexey-orlov-math

Copy link
Copy Markdown
Contributor

Rename both Generic.FunctionField to Generic.AbsSimpleFunctionField, and Generic.FunctionFieldElem to Generic.AbsSimpleFunctionFieldElem; also rename internal cache dict for function fields. Add const aliases inside Generic module. The function_field constructor and file names are unchanged.

Also renamed "Generic.FunctionField.conformance" testset to "Generic.RationalFunctionField.conformance" as it tests conformance of RationalFunctionField exactly

@thofma thofma added enhancement New feature or request release notes: use title For PRs: the title of this PR is suitable for direct use in the release notes labels Jun 8, 2026
@codecov

codecov Bot commented Jun 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 88.11881% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.12%. Comparing base (8ee3eaf) to head (df36c36).

Files with missing lines Patch % Lines
src/generic/FunctionField.jl 87.75% 12 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2416      +/-   ##
==========================================
- Coverage   88.12%   88.12%   -0.01%     
==========================================
  Files         130      130              
  Lines       32972    32972              
==========================================
- Hits        29058    29057       -1     
- Misses       3914     3915       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread src/Deprecations.jl Outdated
@alexey-orlov-math alexey-orlov-math mentioned this pull request Jun 8, 2026
3 tasks
@thofma

thofma commented Jun 8, 2026

Copy link
Copy Markdown
Member

Sorry, there are now merge conflicts because of #2414

alexey-orlov-math and others added 2 commits June 8, 2026 21:17
Rename both Generic.FunctionField to Generic.AbsSimpleFunctionField, and Generic.FunctionFieldElem to Generic.AbsSimpleFunctionFieldElem; also rename internal cache dict for function fields. Add const aliases inside `Generic` module. The `function_field` constructor and file names are unchanged.

Also renamed "Generic.FunctionField.conformance" testset to "Generic.RationalFunctionField.conformance" as it tests conformance of RationalFunctionField exactly
@alexey-orlov-math
alexey-orlov-math force-pushed the funfield-rename-abssimple branch from fe4bb50 to df36c36 Compare June 8, 2026 19:18
@alexey-orlov-math

Copy link
Copy Markdown
Contributor Author

Sorry, there are now merge conflicts because of #2414

I have rebased and force pushed (rewriting history). Should be a clean merge now (but maybe let's wait for non-hecke tests to become green)

@thofma
thofma merged commit f78dec1 into Nemocas:master Jun 8, 2026
19 of 24 checks passed
@alexey-orlov-math
alexey-orlov-math deleted the funfield-rename-abssimple branch June 9, 2026 05:27
thofma referenced this pull request Jun 9, 2026
Co-authored-by: changelog[bot] <changelog[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request release notes: use title For PRs: the title of this PR is suitable for direct use in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.