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

VaryingResolutionSpectralCube.beam? #888

Open
keflavich opened this issue Aug 31, 2023 · 0 comments
Open

VaryingResolutionSpectralCube.beam? #888

keflavich opened this issue Aug 31, 2023 · 0 comments

Comments

@keflavich
Copy link
Contributor

Idea: what if we add a .beam attribute to VaryingResolutionSpectralCube that points to the beams.common_beam()?

I've encountered a lot of cases where I load a bunch of different cubes, and some are VRSCs, while others are just SCs. I then try to loop over them and grab their beams and the code crashes. It would be nice to avoid this situation.

I think the best way to enable this would be with a context manager, since there are some operations that have to be different for the different cube types. But, for example, if you're just going to convolve the cube to a common beam anyway, this might be useful.

I'm not implementing it now because the local workaround will be much quicker at the moment, but this is a spot to brainstorm about it.

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

No branches or pull requests

1 participant