spacetelescope / cubeviz

Data analysis package for cubes. https://cubeviz.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
15 stars 25 forks source link

User Testing #278

Closed kassin closed 5 years ago

kassin commented 6 years ago

(Apologies in advance if some of these features are not supposed to work yet.)

v0.1.dev534

Region selection doesn't work with the MaNGA cube I've loaded.

Can select a ROI in SpecViz, but cannot Apply to Cube.

Data Processing -> Collapse Cube doesn't seem to do anything.

"a" and "d" hot keys don't change the slice displayed. "f" for freezing doesn't seem to work. same for "p" for pull

When I have a dialog box displayed and I click outside of glue, the box disappears. Whenn I click back in glue, it appears again.

View -> Wavelength Units: When I change the units the spectrum display shows a super large range, including wavelengths where there are no data. Also, this causes the the IVAR and MASK cubes to rescale so that they are both all black. Same thing happens if I choose other options in this menu other than Angstroms.

If I choose mm, the spectrum disappears entirely.

View -> RA - Spectral and DEC - Spectral don't seem to do anything.

Data Processing -> Arithmetic Operations TEST2 = FLUX + 100 However, the spectrum doesn't seem to reflect this change.

The Save Dialog box makes it looks like I will overwrite the currently loaded cube. Phew, I hit "OK" and see this is not the case now. The saved file has no suffix if I don't type one. Perhaps we should automatically append one?

If I save a single smoothed component (as a single componet cube with no ivar or mask) and read it into a fresh glue session, it loads the component in all three viewers. It should only display it in one viewer.

Unable to save session. The error is: "Failed to save session Don't know how to serialize <cubeviz.data_factories.DataConfiguration object at 0x18335e6748> of type <class 'cubeviz.data_factories.DataConfiguration'>"

Smoothing

-"Info: Smoothing previews are displayed on CubeViz's left and single image viewers." Since we will eventually allow the users to move the viewers around, and some users may be unsure about what the single image viewer is, hwo about stating something like this instead: "A preview of the smoothing operation is displayed in Viewer 1."

-We need to state somewhere that the preview smooths the one channel displayed, and clicking on "okay" smooths the entire cube.

Contours

-Perhaps change "Custom Component" to "Other Component" because custom implies that the user can create this component somehow.

Viewer Options

-scaling (ie slider bar next to "color/opacity) in the viewer options panel) shouldn't bee linked across the 3 viewers as a default

-I can't think of a use case where one would want to scale the 3 viewers at the same time, so perhaps remove the "sync" button.

-For playing movie to run through channels, can I change the speed?

-The "limits" in "Plot Options - 2D Image" don't seem useful since they don't correspond to the limits shown on the image.

-If I change the limits, I cannot undo it using the Edit -> Undo option.

@brechmos-stsci @drdavella @astrofrog @robelgeda @javerbukh

brechmos-stsci commented 6 years ago

I'll go through this list and check things.

charlottenosam commented 6 years ago

Here are my testing notes on v0.1.dev540, following the items in #257

UPDATED based on v.558. I crossed out things which work now, thanks :smile: Items in bold are things I think are most important to get cubeviz usable now. The other things are just nice additions. @kassin @brechmos-stsci

hcferguson commented 5 years ago

@kassin can this issue be closed?

kassin commented 5 years ago

Not until Charlotte & I are able to run our use cases through CubeViz.

brechmos-stsci commented 5 years ago

From @kassin's comments above. These are all fixed, worked or I created an issue.

Smoothing

Contours

Viewer Options

brechmos-stsci commented 5 years ago

From @charlottenosam's comments above:

brechmos-stsci commented 5 years ago

Discussed with @hcferguson and we are good to close this.