Skip to content

Conversation

@roed314
Copy link
Member

@roed314 roed314 commented May 2, 2025

This is an alternative for #6366. It does the following:

  • Move the Group invariants section from the bottom to the top (with the exception of the Character Table, which becomes its own section).
  • Change heading from "Group" to "Abstract Group," make the latex into an abstract group knowl, and have the label of the abstract group now shown implicitly in the knowl, rather than on a separate line
  • Remove "Transitive number," since it's not a mathematical invariant and is already included in the label
  • Fix a bug for code placement of magma: NilpotencyClass(G);
  • Add closing </td> tags and adjust some spacing.

@jwj61
Copy link
Member

jwj61 commented May 3, 2025

I think the transitive number should be put back in. It isn't a counter made up by the LMFDB; it is a recognized standard identifier for the action.

@roed314
Copy link
Member Author

roed314 commented May 3, 2025

Ok; I put it back in.

@jwj61 jwj61 merged commit b3019f8 into LMFDB:main May 5, 2025
13 checks passed
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.

2 participants