-
Notifications
You must be signed in to change notification settings - Fork 11
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
Progressive update and cancellation of spatial profiler generation (backend) #1179
Comments
it's not really clear to me why this is needed... shouldn't this be extremely quick? |
it is not. Sometimes it takes quite some time to get a spatial profile. The underlying reason is not clear. @pford do you know where the bottleneck is? |
we normally have the data in memory, but I guess if we're using casacore it tries to load things again. |
I assume this is for region spatial profiles. The default region is 'Active', so if the user creates a region the profile widget clears the cursor profile. Region spatial profiles are only valid for point, line, and polyline, so if the active region is another type the profile is not generated and the widget remains blank. (I think the frontend should not clear and request a spatial profile for a non-valid region type.) If the region is a line, the backend response is the same as for the PV generator, creating box regions along the line and getting stats (the mean) for each box for the current channel. This could take some time if there are many boxes, if the image requires fixed angular spacing, or if the line is a matched region and each box must be converted from the reference image. |
user feedback from v3 beta test
Similar to the progressive update and cancellation features in the spectral profiler widget, the spatial profiler widget should also have the same features as a visual indication of the progress for better UX.
The companion frontend issue is CARTAvis/carta-frontend#1954
The text was updated successfully, but these errors were encountered: