-
Notifications
You must be signed in to change notification settings - Fork 28
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
Default mask to Trues from FillArray #70
Conversation
Codecov Report
@@ Coverage Diff @@
## master #70 +/- ##
=======================================
Coverage 99.76% 99.76%
=======================================
Files 49 49
Lines 2113 2113
=======================================
Hits 2108 2108
Misses 5 5
Continue to review full report at Codecov.
|
From the description, |
Any benchmark for this? I'm a little doubt about this improvement given that broadcasting is generally doing pretty good. |
@johnnychen94 thanks for telling me about FillArrays. I tried searching for something like this and could not find it. I should have asked you! Do you mind emailing me your email address so I can ask occasional questions like that before investing time repeating existing work? (My email is easy to find on my web page.) There are timing tests in the repo:: |
Sorry for the late response, I was exhausted last week because of the day-to-day seminars. It's I tried to play with the benchmark scripts, but the results are a little obscure to me to understand. My best guess is that these differences are misleading; these fundamental operations might be affected by Julia's optimization on If the memory allocation of |
Thanks for taking a look. I studied it further and was able to figure out how to speed everything up the way I wanted. The timing results are all in the updated |
@johnnychen94 , thanks for the pointer to FillArrays. I contributed a couple of features to it so that it would support what I needed here and now I'm using FillArrays instead. Any comments? |
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.
LGTM
Saves memory for the usual default, and will save time for
.* mask
operations.