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

Feature request: Change rest frame #138

Open
keflavich opened this issue May 10, 2022 · 4 comments
Open

Feature request: Change rest frame #138

keflavich opened this issue May 10, 2022 · 4 comments
Labels
question Further information is requested usability

Comments

@keflavich
Copy link

It would be helpful to be able to change the reference (rest) frequency when showing velocities. If such a feature exists, I haven't found it.

@kswang1029
Copy link

kswang1029 commented May 11, 2022

It is implemented in v3b2. Please check the settings dialog of the image list widget.

In addition, in the upcoming v3b3 release, it is also possible to set a new rest frequency when saving a subimage.

@kswang1029 kswang1029 transferred this issue from CARTAvis/carta-frontend May 11, 2022
@keflavich
Copy link
Author

I see it! Thanks.

Could you add it, or perhaps a link to it, from the z-profile menu?

image

That's where I expected to see it, since that's where I can choose which units to display.

@kswang1029
Copy link

When a new rest frequency is set to a given image cube, all matched cubes are re-matched. Then you can request region analytics on the matched channels, as well as comparing matched channel images. As the image list widget is the key tool we design for image spatial and spectral matching, we think it is logical to put the rest frequency options there.

@keflavich
Copy link
Author

Could you at least add a tooltip or similar in the "Conversion" menu?

While I understand your logic, I always think of the spectrum as the object that has a rest frequency and a velocity, so I expected the Z-axis (spectrum) viewer to have the controls for rest frequency. It doesn't really make sense to enable a velocity view if you cannot control what the velocity is defined with respect to.

@kswang1029 kswang1029 added the question Further information is requested label Aug 1, 2022
@kswang1029 kswang1029 added this to CARTA Dec 6, 2023
@kswang1029 kswang1029 moved this to New issue in CARTA Dec 6, 2023
@kswang1029 kswang1029 removed this from CARTA Dec 6, 2023
@kswang1029 kswang1029 added this to CARTA Dec 6, 2023
@kswang1029 kswang1029 moved this to New issue in CARTA Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested usability
Projects
No open projects
Status: New issues
Development

No branches or pull requests

2 participants