-
Notifications
You must be signed in to change notification settings - Fork 808
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
Numba warnings #252
Comments
These are, indeed, non-critical -- numba has just gotten a little more verbose about it's warnings, and umap continues to be somewhat optimistic about how well numba can compile things. In this case numba is just letting you know that umap was a little over-optimistic and not as much could be as completely compiled as one might have hoped. I'll try to fix this up in an upcoming release. Thanks for the heads up, it is appreciated. |
I was getting the same warnings. I added the following code to my notebooks and it solved it.
This was based upon the StackExchange Post |
This is actually fine -- the latest version of numba now issues warning for routines it couldn't completely compile. There will be a release of umap-learn soon that will remove the ineffective jit instructions and thus remove the warnings. |
Could you not do |
That might we work yes. Right now it is mostly just a matter of having the time to sit down and go through all the options and the code and make changes. I was very busy last month, and am on vacation right now and trying not to look at code. Hopefully I can make some time in a couple of weeks. |
Enjoy your vacation :) To help you resolve the warnings quickly I compiled a list of all numba errors. A list of warnings I currently see:
|
Changed in: afb819e |
Should be resolved in EDIT: This is 100% my bad. I thought @lmcinnes added this to 0.3.10 but I think it only made it into the upcoming release of 0.4. The PR for the changes is here #287 |
The keyword argument 'parallel=True' was specified but no transformation for To find out why, try turning on parallel diagnostics, see I am having this problem too.. /usr/local/lib/python3.6/dist-packages/umap/spectral.py:229: UserWarning: Embedding a total of 178 separate connected components using meta-embedding (experimental) When I press a key, I started to get another message. Would not I run my script and obtain reduced dimensions? |
Sorry @asfix ; these are just warnings and don't represent a problem beyond the fact that various things have been done in the processing that a user may wish to be aware of. It should all run without problem, but there may be some warning messages flashing by. |
Hey,
|
It has been addressed in 0.4dev. Could you try with that branch, please. And report back any issues. |
Is the 0.4dev only available for python 3.5? |
If I remember correctly there was a PR that included the @ operator which is only available in Python 3.5. |
I believe future releases will be targeting 3.5 and upwards. So things may work for now, but i a version or so you'll likely need a newer python if you haven't gotten onto 3.5 or newer. |
Yeah there was build problems on python 2.7 due to the @ but we've updated to 3.7.3 and everything is smooth now! Thank you very much |
I'm seeking clarity on: I'm trying to do some benchmarking and would like to know if this means that Just found issue #317 from November 2019 so I'm guessing parallelization is still not available out-of-the-box? |
This can just mean that a single function within UMAP could not be optimized to be run in parallel by Numba. It does not mean that everything else, that can be, wasn't executed in parallel. This should not happen with the current dev branch. If so, please report. |
I use the following to ignore the numba warning without affecting other warnings for now.
|
This solution works for me! |
Thank you for numba! This warning: "The keyword argument 'parallel=True' was specified but no transformation for parallel execution was possible." is always occurring only on the 1st execution of the jited function. Every subsequent execution runs smoothly and and several times faster. Is there a way to avoid this warning entirely? Many computationally-heavy functions are called only once and thus it would be a great improvement if it could be avoided. |
First I want to thank you for sharing this great work. It is very useful for me.
The first time I execute UMAP fit and transform I get some numba warnings. They do not seem to be critical, but they are somewhat disturbing.
numba.__version__
'0.44.0'
sys.version
'3.7.3 (default, Mar 27 2019, 22:11:17) \n[GCC 7.3.0]'
The text was updated successfully, but these errors were encountered: