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.

Sign up for GitHub

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

Jump to bottom

Investigate reducing MAXVERTS in 32-bit mode #432

Open
wilfwilson opened this issue Mar 4, 2021 · 1 comment
Open

Investigate reducing MAXVERTS in 32-bit mode #432

wilfwilson opened this issue Mar 4, 2021 · 1 comment
Labels
C language A label for issues or pull requests relating to the kernel module of the package

Comments

@wilfwilson
Copy link
Collaborator

wilfwilson commented Mar 4, 2021 •
edited
Loading

@ChrisJefferson suggested this as a possibly good idea in semigroups/Semigroups#634 (comment). Memory is severely limited on 32-bit mode, and with MAXVERTS = 512, Digraphs can apparently end up being quite demanding on the mrmory. Reducing MAXVERTS should theoretically make Digraphs more stable on 32-bit systems, at the cost of it, unfortunately, being less useful.

Alternatively, or in combination, we might want to be cleverer about how we allocate memory (especially for stabiliser chains); see #433.
@wilfwilson wilfwilson added the C language A label for issues or pull requests relating to the kernel module of the package label Mar 4, 2021
@wilfwilson wilfwilson mentioned this issue Mar 4, 2021
Closed
@digraphs digraphs deleted a comment from github-actions bot Jan 10, 2024
@james-d-mitchell
Copy link
Member

Just to mention #602: we should close this issue if we drop 32-bit support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C language A label for issues or pull requests relating to the kernel module of the package
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@james-d-mitchell @wilfwilson