Skip to content

Conversation

@thofma
Copy link
Collaborator

@thofma thofma commented Feb 4, 2024

Fixes some woes introduced in some of the new type names.

  • Remove redundancy for embedded types by using aliases
  • use compact printing for abelian group elements in arrays

Fixes some woes introduced in some of the new type names.
- Remove redundancy for embedded types by using aliases
- use compact printing for abelian group elements in arrays
@thofma thofma requested a review from fieker February 4, 2024 12:51
@codecov
Copy link

codecov bot commented Feb 4, 2024

Codecov Report

Merging #3311 (d6f3e50) into master (1c92407) will not change coverage.
Report is 1 commits behind head on master.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3311   +/-   ##
=======================================
  Coverage   81.71%   81.71%           
=======================================
  Files         556      556           
  Lines       74141    74141           
=======================================
  Hits        60584    60584           
  Misses      13557    13557           
Files Coverage Δ
src/Modules/ModulesGraded.jl 79.34% <ø> (ø)
src/PolyhedralGeometry/Polyhedron/properties.jl 83.17% <ø> (ø)
...edralGeometry/Polyhedron/standard_constructions.jl 96.24% <ø> (ø)
src/Rings/mpoly-graded.jl 89.71% <ø> (ø)

@lgoettgens
Copy link
Member

The 1.10 failure will be fixed by #3312

@thofma thofma closed this Feb 4, 2024
@thofma thofma reopened this Feb 4, 2024
@thofma thofma enabled auto-merge (squash) February 4, 2024 15:23
@thofma thofma merged commit 250e4f2 into master Feb 4, 2024
@thofma thofma deleted the th/bumphecke branch February 4, 2024 16:14
ooinaruhugh pushed a commit to ooinaruhugh/Oscar.jl that referenced this pull request Feb 15, 2024
Fixes some woes introduced in some of the new type names.
- Remove redundancy for embedded types by using aliases
- use compact printing for abelian group elements in arrays
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants