IGS / gEAR

The gEAR Portal was created as a data archive and viewer for gene expression data including microarrays, bulk RNA-Seq, single-cell RNA-Seq and more.
https://umgear.org
GNU Affero General Public License v3.0
12 stars 4 forks source link

Staging gEAR-Comparison tool-X and Y axis not resetting after change of parameters #751

Closed beamilon closed 1 month ago

beamilon commented 1 month ago

@adkinsrs The following happened with several dataset but here is an example with the P30, mouse, snRNA-seq, Stria Vascularis (Hoa). Using the compare tool, I compared Marginal and Intermediate cells and it worked fine. Then I went back to change the parameters, chose to compare Basal and Spindle/Root cells. The X and Y axis do not reset so I can change the cell types. It is locked in Intermediate and Marginal cells. image

adkinsrs commented 1 month ago

@beamilon have you tried unchecking one of those two checkboxes? The validation should reset the checkboxes to be OK. But I do agree that it should be clearer.

Also, if you are choosing "cell types" in X and Y, you do not need to filter them out in the "shared filtered conditions" step. That step is meant for filtering to a subset of data beyond what you choose for X and Y (i.e. look at Intermediate and Marginal cell types only at the 24hr timepoint if "timepoint" was a field in your dataset)

beamilon commented 1 month ago

@adkinsrs Unchecking the checkboxes doesn't work. The choice is still lock on the previous parameters. image

adkinsrs commented 1 month ago

While changing the "series to compare" would cause the checkboxes to re-render, the best solution would be to add triggers on the "filter" selections to disable and enable boxes if a series is rendered. I also changed some of the text to help the user understand that the "shared filters" are meant to filtering subsets of data beyond the main series to compare.

adkinsrs commented 1 month ago

Screenshot 2024-07-22 at 8 38 58 AM

Was chatting with @DanLesperance and I think what I am going to do is swap the "series to compare" section and the "shared filters" section to promote a better flow. First you select the series you want to compare, which will be excluded from the "secondary/shared filters", so there is less chance of running into situations like @beamilon's screenshots where the X/Y conditions are filtered out.

adkinsrs commented 1 month ago

Commit 804b990 address my proposed changes. You select the series first, and this hides that out of the filter categories. Changing the compare series again will reset the state of the selected series groups (x and y) as well as the selected filters.