Skip to content

Fix is_irreducible for number field order elements#4424

Merged
thofma merged 1 commit intomasterfrom
th/isirred
Jan 8, 2025
Merged

Fix is_irreducible for number field order elements#4424
thofma merged 1 commit intomasterfrom
th/isirred

Conversation

@thofma
Copy link
Copy Markdown
Collaborator

@thofma thofma commented Jan 7, 2025

No description provided.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.36%. Comparing base (264490b) to head (9e6a6b4).
Report is 74 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4424      +/-   ##
==========================================
+ Coverage   84.34%   84.36%   +0.02%     
==========================================
  Files         663      663              
  Lines       87788    87788              
==========================================
+ Hits        74042    74060      +18     
+ Misses      13746    13728      -18     
Files with missing lines Coverage Δ
src/NumberTheory/NmbThy.jl 88.08% <100.00%> (+9.32%) ⬆️

@thofma thofma requested a review from fieker January 7, 2025 19:18
Copy link
Copy Markdown
Contributor

@fieker fieker left a comment

Choose a reason for hiding this comment

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

I wonder how many solve/ kernel right left problems we're still having?

@thofma thofma merged commit 91f076a into master Jan 8, 2025
@thofma thofma deleted the th/isirred branch January 8, 2025 10:53
@aaruni96 aaruni96 added the release notes: use title For PRs: the title of this PR is suitable for direct use in the release notes label Jan 30, 2025
@fingolfin fingolfin changed the title fix: is_irreducible for number field order elements Fix is_irreducible for number field order elements Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release notes: use title For PRs: the title of this PR is suitable for direct use in the release notes topic: number theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants