-
Notifications
You must be signed in to change notification settings - Fork 4
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
make tests run much faster #16
base: master
Are you sure you want to change the base?
Conversation
The issue was that |
Thanks, Jan. I was aware of the issue with the big tuples but don't think there's an easy fix. Like you say, evaluation when needed would be one way. But that would require some major code changes. I'm not sure if fixing it just for the test runs is a good idea. I'll have to understand what you did there, first. |
I guess most people including me don't actually need these high dimensional algebras. But slow tests affect everybody. The tests only instantiate some big algebras, but do nothing else with them. I made instantiation fast. |
I guess I'll try to replace the tuple with something that quacks and walks like a tuple but is really a lazy evaluated expression. Do you happen to know if someone else already has written a library for that? |
I don't know a library that can help here. If you want we could chat about how to do this or code it together. |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
@jw3126 I'd love to do that. Unfortunately I'm knee-deep into a C++ project at the moment and switching to a Julia context would require some effort. But maybe we could meet for a play date some time and discuss the issue when the kids are busy? |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Yeah sounds good! |
Before
After