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

various changes about is_field and is_ring #37783

Merged
merged 3 commits into from
Apr 27, 2024

Conversation

fchapoton
Copy link
Contributor

@fchapoton fchapoton commented Apr 10, 2024

just miscellanous changes about not using is_ring, replacing some is_field by in Fields

and having more precise categories in descent algebras

also sorting the imports on the way

📝 Checklist

  • The title is concise and informative.
  • The description explains in detail what this PR is about.

Copy link
Collaborator

@tscrim tscrim left a comment

Choose a reason for hiding this comment

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

LGTM assuming tests pass.

@fchapoton
Copy link
Contributor Author

not yet ready for consumption ; I need to add a default is_field returning False in the category of commutative rings.

@fchapoton
Copy link
Contributor Author

I have decided just to fix the doctests here.

Copy link

Documentation preview for this PR (built with commit b807676; changes) is ready! 🎉

Copy link
Contributor

@mkoeppe mkoeppe left a comment

Choose a reason for hiding this comment

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

LGTM

vbraun pushed a commit to vbraun/sage that referenced this pull request Apr 20, 2024
    
just miscellanous changes about not using `is_ring`, replacing some
`is_field` by `in Fields`

and having more precise categories in descent algebras

also sorting the imports on the way

### 📝 Checklist

- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
    
URL: sagemath#37783
Reported by: Frédéric Chapoton
Reviewer(s): Matthias Köppe, Travis Scrimshaw
@vbraun vbraun merged commit 39fd0cf into sagemath:develop Apr 27, 2024
15 checks passed
@mkoeppe mkoeppe added this to the sage-10.4 milestone Apr 27, 2024
@fchapoton fchapoton deleted the various_about_is_field branch April 27, 2024 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants