-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
PhysicalConstants.jl
fails to precompile on master.
#43561
Comments
For the record, this is getting stuck at https://github.com/JuliaPhysics/PhysicalConstants.jl/blob/221b3bee8dd001294ad7f3897bfb2f9e64809b4e/src/PhysicalConstants.jl#L87 for derived physical constants, for example https://github.com/JuliaPhysics/PhysicalConstants.jl/blob/221b3bee8dd001294ad7f3897bfb2f9e64809b4e/src/codata2014.jl#L39. If I comment out all those assertions, I can precompile the package, but doing
Given the files referenced in the stacktrace I presume we should ping @aviatesk? Note that the |
This might be akin to #43542, though the cause and solution are likely quite different |
Bisected to
@vtjnash |
Wow, that's a lot:
It took me 17 seconds to precompile |
Well, I just run |
I just opened a PR to
hang |
Could confirm, and after reverting #43305 the above code is able to finish normally. |
Fixed by #43812 |
On master
using PhysicalConstants
seems run into a dead loop. (CPU usage is not 0)On 1.7 everything seems OK.
The text was updated successfully, but these errors were encountered: