Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Missing pixell dependency #205

Closed
sbiquard opened this issue Jan 21, 2025 · 3 comments
Closed

Missing pixell dependency #205

sbiquard opened this issue Jan 21, 2025 · 3 comments

Comments

@sbiquard
Copy link
Contributor

sbiquard commented Jan 21, 2025

Only the "test" optional dependencies contain the pixell==0.26.0 requirement.

But clearly, the package is used in several places in the code, such as in

import pixell.enmap, pixell.curvedsky

which is where I got an ImportError from.

I guess the fix should be as simple as moving the dependency to the main group in pyproject.toml, but maybe I'm missing something.

Let me know if you'd like me to submit the pull request.

@zonca
Copy link
Member

zonca commented Jan 21, 2025

thanks @sbiquard ,

no, we want to be able to run healpix-only with no pixell.
Can you please do a pull request making a conditional import in that file like I have in template.py?

@zonca
Copy link
Member

zonca commented Jan 23, 2025

fixed in #208

@zonca zonca closed this as completed Jan 23, 2025
@zonca
Copy link
Member

zonca commented Jan 23, 2025

Thanks!

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

Successfully merging a pull request may close this issue.

2 participants