-
Notifications
You must be signed in to change notification settings - Fork 55
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
Very slow @astable
macro outside a function
#363
Comments
Thank you for your bug report! It is true that That said, I can't replicate your issue on version 1.9. I get roughly comparable timings.
Can you double check your measurements and give me your version info? |
So I double checked the results and found something potentially interesting! In the REPL I get a similar timing just as you, but the above timings came from running inside a Pluto notebook. Seems like the issue is using Here is my version info
|
That's pretty frustrating. Can you post an issue with Pluto.jl? (Or their internal tracker or whatever they use) and I can cross-link with this issue? |
Bumping this @mbataillou did you ever file an issue with Pluto.jl? |
Here is the experiment.
Given the dataframe and functions
f0, f1
belowWe get an improvement in performance in
f1
, which is what one would expect given it does not need to create columnsb, c
.However, if one uses this code outside a function (see below) it becomes 46 times slower! Making it unusable for datasets of a larger size.
Thanks for the great work :)
The text was updated successfully, but these errors were encountered: