Skip to content
This repository has been archived by the owner on Oct 14, 2023. It is now read-only.

Unit conversion error when using newer astroquery #1229

Closed
astrojuanlu opened this issue May 16, 2021 · 5 comments
Closed

Unit conversion error when using newer astroquery #1229

astrojuanlu opened this issue May 16, 2021 · 5 comments

Comments

@astrojuanlu
Copy link
Member

I think we broke the docs after I merged some of the recent pull requests: either #1184 or #1190, both by @Yash-10.

Here's the latest failed build:

https://readthedocs.org/projects/poliastro/builds/13776407/

UnitConversionError: Can only apply 'add' function to quantities with compatible dimensions

there's a units problem in the NEOs notebook.

@astrojuanlu
Copy link
Member Author

Both Read the Docs and our py38_online_test check have failed, see for example https://github.com/poliastro/poliastro/runs/2594411686.

@jtegedor
Copy link

I found the same while doing something else, I do not think it is related specifically to those PR.

After doing the changes here #1228, docs and pytest work again.

@astrojuanlu
Copy link
Member Author

Oh, now I know what's happening! The from_sbdb old code looked weird to me, but I remembered it had to do with some weird astroquery thing. I checked their release notes and saw that astropy/astroquery#2011 has been released, hence the jplsbdb service now returns astropy quantities. It makes total sense to merge #1228, doing it now.

@astrojuanlu
Copy link
Member Author

Closed in #1228 🚀

@astrojuanlu astrojuanlu changed the title Docs are broken Unit conversion error when using newer astroquery May 30, 2021
@astrojuanlu
Copy link
Member Author

For future readers: Until we release poliastro 0.15.1 with a fix for this, a solution is to downgrade astroquery:

pip install "astroquery<0.4.2"

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

No branches or pull requests

2 participants