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

[FEAT] Add sky-to-Cartesian coordinate transformation to Python interface #16

Closed
MikeSWang opened this issue Jun 25, 2023 · 1 comment
Closed
Assignees
Labels
feature Feature requests python-only Python-specific rejected Rejected

Comments

@MikeSWang
Copy link
Owner

Is the requested feature related to an issue?

No.

Summary

Add sky-to-Cartesian coordinate transformation functions.

Alternatives

N/A

Implementation

Can be implemented in the Python interface through existing astropy dependency.

Additional context

This saves the user from creating separate catalogues with the required Cartesian coordinates.

@MikeSWang MikeSWang added the feature Feature requests label Jun 25, 2023
@MikeSWang MikeSWang self-assigned this Jun 25, 2023
@MikeSWang MikeSWang changed the title [FEAT] <insert-issue-title> [FEAT] Add sky-to-Cartesian coordinate transformation to Python interface Jun 25, 2023
@MikeSWang MikeSWang added the python-only Python-specific label Aug 19, 2024
@MikeSWang
Copy link
Owner Author

Rejected proposal

Since the user can easily convert distances using external packages and pass the catalogue as arrays, and we are only able to support standard-model cosmological models through Astropy, this proposal is not impactful enough for additional development time.

@MikeSWang MikeSWang closed this as not planned Won't fix, can't repro, duplicate, stale Aug 19, 2024
@MikeSWang MikeSWang added the rejected Rejected label Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Feature requests python-only Python-specific rejected Rejected
Projects
None yet
Development

No branches or pull requests

1 participant