Skip to content
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

Unable to install on fresh julia 1.6/1.7 installs #24

Closed
jmmshn opened this issue Sep 1, 2022 · 5 comments · Fixed by #25
Closed

Unable to install on fresh julia 1.6/1.7 installs #24

jmmshn opened this issue Sep 1, 2022 · 5 comments · Fixed by #25

Comments

@jmmshn
Copy link

jmmshn commented Sep 1, 2022

Dear maintainers,
This looks like a super useful package for science and I'm trying to install it.
I tried a few ways using different versions of Julia but keep running into the same problem:

image

I'm kind of a Julia noob so please let me know if I'm doing something really silly.

Thanks in advance!

@giordano
Copy link
Member

giordano commented Sep 1, 2022

Can you please come back to the normal Julia REPL (press Ctrl + C) and run

using PhysicalConstants

after you get that error, to show the full error message? This is what is suggested in the last line with "load the package" (packages in Julia are loaded with using ...).

@jmmshn
Copy link
Author

jmmshn commented Sep 1, 2022

Oh yeah, I should have probably included that.

Here is it:

image

@giordano
Copy link
Member

giordano commented Sep 1, 2022

Oh, cool, it looks like you tried to install the package right after JuliaPhysics/Measurements.jl#118 was merged and accidentally broke this package, I'll have to fix that

@jmmshn
Copy link
Author

jmmshn commented Sep 1, 2022

OK, I actually look at Measurements.tag_counters and did not notice the missing s.
Thanks for your help!
I'll keep an eye out for the new release.

@giordano
Copy link
Member

giordano commented Sep 1, 2022

Version 0.2.3 is now out. If you update the registry (]up) you should get the latest version of this package, which fixes compatibility with Measurements.jl. Unfortunately this package sneakingly uses some internal structures of Measurements.jl which aren't part of the public API, which however were changed in the PR linked above and broke this package as a side effect. That's on me for using internals of other packages, but at least I maintain both of them 🙂 Also, it doesn't look like other packages are referencing Measurements.tag_counters, so it should be all good now. Thanks for the report!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants