Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate is_FiniteField etc., make sage.rings.finite_rings a namespace package #32664

Closed
mkoeppe opened this issue Oct 10, 2021 · 30 comments · Fixed by #35119
Closed

Deprecate is_FiniteField etc., make sage.rings.finite_rings a namespace package #32664

mkoeppe opened this issue Oct 10, 2021 · 30 comments · Fixed by #35119

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Oct 10, 2021

Part of meta-ticket #32414.

Deprecating is_FiniteField, is_PrimeFiniteField, and is_FiniteFieldElement is done for uniformity but is not needed to support modularization. #32432 shows that sage.rings.finite_rings.finite_field_base (which defines the ABC FiniteField) can be imported without pulling in implementation classes.

sage.rings.finite_rings must become a namespace package because element implementations depend on various libraries.

CC: @dimpase @kwankyu @fchapoton @tscrim

Component: refactoring

Author: Matthias Koeppe

Issue created by migration from https://trac.sagemath.org/ticket/32664

@mkoeppe mkoeppe added this to the sage-9.5 milestone Oct 10, 2021
@mkoeppe

This comment has been minimized.

@mkoeppe mkoeppe removed this from the sage-9.5 milestone Oct 22, 2021
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 5, 2022

Changed dependencies from #32612 to none

@mkoeppe

This comment has been minimized.

@mkoeppe mkoeppe changed the title Add sage.rings.abc.FiniteField, deprecate is_FiniteField Deprecate is_FiniteField, make sage.rings.finite_rings a namespace package Dec 5, 2022
@mkoeppe mkoeppe added this to the sage-9.8 milestone Dec 5, 2022
@mkoeppe mkoeppe removed the pending label Dec 5, 2022
@mkoeppe

This comment has been minimized.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 5, 2022

@mkoeppe

This comment has been minimized.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 5, 2022

Commit: 8af57c4

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 5, 2022

New commits:

8af57c4sage.rings.finite_rings: Deprecate is_FiniteField, is_PrimeFiniteField

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 5, 2022

Author: Matthias Koeppe

@mkoeppe

This comment has been minimized.

@mkoeppe mkoeppe changed the title Deprecate is_FiniteField, make sage.rings.finite_rings a namespace package Deprecate is_FiniteField etc., make sage.rings.finite_rings a namespace package Dec 5, 2022
@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 5, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

98b433csage.rings.finite_rings: Make it a namespace package
6323775src/sage/rings/finite_rings/finite_field_constructor.py: Restore re-export of is_FiniteField
473e9d4src/sage/rings/finite_rings/hom_finite_field.pyx: Import ABC as FiniteField_base
b82fb1esage.rings.finite_rings: Deprecate is_FiniteFieldElent

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 5, 2022

Changed commit from 8af57c4 to b82fb1e

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 6, 2022

Changed commit from b82fb1e to 106b22e

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 6, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

106b22eFixups

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 6, 2022

Changed commit from 106b22e to de5cfad

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 6, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

de5cfadFixups

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 6, 2022

Changed commit from de5cfad to 429b372

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 6, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

429b372Fixup: Elements of finite fields are not instances of FieldElement

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 6, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

e633d8fsrc/sage/crypto/mq/rijndael_gf.py: Fix imports

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 6, 2022

Changed commit from 429b372 to e633d8f

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 6, 2022

Changed commit from e633d8f to 886bc1a

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 6, 2022

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

886bc1asrc/sage/crypto/mq/rijndael_gf.py: Fix imports

@dimpase
Copy link
Member

dimpase commented Dec 6, 2022

comment:16

shouldn't this branch be based on #34804 ?

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 18, 2022

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

53ede8csage.rings.finite_rings: Deprecate is_FiniteField, is_PrimeFiniteField
8d4d8e5sage.rings.finite_rings: Make it a namespace package
6ce26bcsrc/sage/rings/finite_rings/finite_field_constructor.py: Restore re-export of is_FiniteField
98a5484src/sage/rings/finite_rings/hom_finite_field.pyx: Import ABC as FiniteField_base
2e6b5efsage.rings.finite_rings: Deprecate is_FiniteFieldElent
3be7c97Fixups
97a363eFixups
0c01403Fixup: Elements of finite fields are not instances of FieldElement
9f4d1c4src/sage/crypto/mq/rijndael_gf.py: Fix imports

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 18, 2022

Changed commit from 886bc1a to 9f4d1c4

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 18, 2022

comment:18

Replying to Dima Pasechnik:

shouldn't this branch be based on #34804 ?

There's no relation. #34804 merges cleanly with this branch

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 7, 2023

Changed commit from 9f4d1c4 to 9ebadcf

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 7, 2023

Branch pushed to git repo; I updated commit sha1. New commits:

9ebadcfMerge tag '9.8.beta6' into t/32664/add_sage_rings_abc_finitefield__deprecate_is_finitefield

@mkoeppe mkoeppe modified the milestones: sage-9.8, sage-9.9 Jan 29, 2023
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 10, 2023

Tests ran successfully in https://github.com/sagemath/sagetrac-mirror/actions/runs/3860089783 (before the migration).

Needs review

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 13, 2023

Removed branch from issue description; replaced by PR #35119

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

Successfully merging a pull request may close this issue.

2 participants