-
Notifications
You must be signed in to change notification settings - Fork 74
quantities
0.16.0 and newer cannot parse 'angstrom'
#250
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
Comments
quantities
16.0 and newer cannot parse 'angstrom'quantities
0.16.0 and newer cannot parse 'angstrom'
apdavison
added a commit
to apdavison/python-quantities
that referenced
this issue
Dec 3, 2024
… parsing strings as units. Based on https://stackoverflow.com/a/11952618 by https://stackoverflow.com/users/567292/ecatmur Fixes python-quantities#250 Also added test for the original issue: python-quantities#221
apdavison
added a commit
to apdavison/python-quantities
that referenced
this issue
Dec 3, 2024
… parsing strings as units. Based on https://stackoverflow.com/a/11952618 by https://stackoverflow.com/users/567292/ecatmur Fixes python-quantities#250 Also added test for the original issue: python-quantities#221
apdavison
added a commit
to apdavison/python-quantities
that referenced
this issue
Dec 3, 2024
… parsing strings as units. Based on https://stackoverflow.com/a/11952618 by https://stackoverflow.com/users/567292/ecatmur Fixes python-quantities#250 Also added test for the original issue: python-quantities#221
apdavison
added a commit
to apdavison/python-quantities
that referenced
this issue
Dec 3, 2024
… parsing strings as units. Based on https://stackoverflow.com/a/11952618 by https://stackoverflow.com/users/567292/ecatmur Fixes python-quantities#250 Also added test for the original issue: python-quantities#221
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The changes introduced in this PR: #236
Make it impossible to parse 'angstrom', since the built in function
str
partially matches 'angstrom' and is thereforein
that string, see this code snippet fromquantities/registry.py
/example that reproduces the issue:Can
in
be chnaged to==
to avoid partial matches like this?This is somewhat similar to the inability to parse
min
mentioned here: #242The text was updated successfully, but these errors were encountered: