-
Notifications
You must be signed in to change notification settings - Fork 49
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
fixing discrete topologies #1323
Conversation
@zstone1 |
The problem I'm having is that it takes 3 minutes to run that one command. Since I know what structure i want it to saturate for, is there anyway for me to improve its performance? |
Yes, but in the next HB release. math-comp/hierarchy-builder#414 |
All the reshuffling of |
67536af
to
3800c53
Compare
4957f38
to
5f47a66
Compare
5f47a66
to
2e448c5
Compare
I've rebased, fix the changelog (but not checked it yet), and recycled documentation for the file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This simplifies and clarifies the code. The doc seems complete as well as the changelog.
Adding discrete topologies, uniformities, and metrics to the hierarchy. Cleans up a bunch of pre-HB issues, and now we can use
bool
directly instead of through a bunch of aliases. We can now just writecantor_space = prod_topology (fun=> bool)
andUniform.on cantor_space
. Pretty nice!However, I encountered a lot of weird HB errors. I put in tickets math-comp/hierarchy-builder#447 and math-comp/hierarchy-builder#446
Furthermore, I could only get this to compile with a call to
HB.saturate
. This is fine, except that it the commands takes several minutes to run. And is broken on master with errorI am skeptical if I get a build that works on both master and the current release
Checklist
CHANGELOG_UNRELEASED.md
Reference: How to document
Reminder to reviewers