-
-
Notifications
You must be signed in to change notification settings - Fork 64
Fixes for F macro #85
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
Conversation
Memory usage change @ 19fa1b7
Click for full report table
Click for full report CSV
|
I might not understand the approach of this PR entirely, but it seems the idea is to use the This could be fixed by putting just |
Hi @matthijskooijman, yes this is a draft PR for a possible implementation of your option 2. I opened a PR to be able to visualize the memory usage change. |
Ah, good plan (though I think the cost of option 1. is more in terms of runtime than memory usage, but it would be interesting to see how much effect there is on the memory usage between both options indeed). |
Memory usage change @ d5799ab
Click for full report table
Click for full report CSV
|
Closing, for details see #62 (comment). |
This PR is still a draft