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

Mutations table improvements #8711

Open
7 of 13 tasks
leexgh opened this issue Jun 22, 2021 · 2 comments
Open
7 of 13 tasks

Mutations table improvements #8711

leexgh opened this issue Jun 22, 2021 · 2 comments

Comments

@leexgh
Copy link
Member

leexgh commented Jun 22, 2021

Related tickets and mockup

Improvements

1. Annotation column

1.1 Annotation column header

  • Goal:
    image
  • Ticket: Add tooltip for the annotation column in the mutations table Add tooltip for the annotation column in the mutations table #8305
  • Todo list:
    • Add header and tooltips
    • Leave white space for multiple OncoKB icons
  • Status: ready to merge
  • Discussion result: Based on the discussion, for now we only support multiple oncokb icons(leave white space between oncokb and civic)
  • Assignees: Xiang

1.2 Settings for multiple or single oncokb icons

  • Ticket: Settings for multiple vs single OncoKB icon in annotation column Settings for multiple vs single OncoKB icon in annotation column  #8710
  • Discussion result: this will be part of grouped mutation columns(2.4). Once we have the grouped mutation columns, we could either add settings somewhere inside columns menu, or we could have separate columns for single icon oncokb or multiple icons, and we could choose show or hide by adding columns.

1.3 Settings for show or hide annotation sources (superseded by 2.4)

  • Ticket: no (mockup slides 14)
  • Discussion result: this will be part of grouped mutation columns(2.4). Once we have the grouped mutation columns, we could add settings somewhere inside columns menu along with oncokb single vs multiple icon setting.

1.4 Integration oncokb icon

image

  • Ticket: cBioPortal integration icon mockups cBioPortal integration icon mockups oncokb/oncokb#2580
  • Todo list:
    • Finalize design?
  • Discussion result: we probably want to have separate columns for single and multiple icons. And we may need some new headers when having multiple icons (e.g. each column has a icon, different icons to represent Oncogenic, Therapeutic Levels, Diagnostic Levels and Prognostic Levels)
  • Assignees: Hongxin

1.5 Grouped columns for the annotation column (superseded by 2.4)

  • Goal: Figure out a way to have grouped columns for the annotation column (Annotations > (OncoKB, cancerhotsots etc).
  • Ticket: Grouped columns in mutations table Grouped columns in mutations table icebox#189
  • Discussion result: this will be part of grouped mutation columns(2.4).

1.6 Improve sorting (superseded by 2.4)

2. Column menu

2.1 Add Clinical Attributes to Mutations Column Menu

image

2.2 Filtering per column

2.3 Add column number in menu button and tabs

image

  • Ticket: no (mockup slides 24 and 25)
  • Todo list:
  • Status: DONE!
  • Assignees: Rajat

2.4 Grouped mutation columns

  • Ticket: Support grouped columns in LazyMobxTable: Support grouped columns in LazyMobxTable icebox#192
  • Todo list:
    • Add an option to group columns
    • Grouped columns should be rendered together
    • Grouped columns should support having a group header
    • Ideally a group may have other groups, so support for multi level nested groups would be nice.
  • Discussion points:
    • Current design (mockup slides 25) is too redundant on group name, e.g. "Mutation Event" is in every column selection but it's too long to be there
      • some ideas:
      • slide 25: image
      • slide 29: image
  • Discussion result: good to have it! Before having the final UI design we could spend some time on data structure
  • Assignees: Onur

3. Store settings

3.1 Store user settings

Keep track of what columns were selected

3.2 Add "reset to default" button based on app configuration (related to 3.1)

inodb pushed a commit to cBioPortal/cbioportal-frontend that referenced this issue Jul 23, 2021
Fixes cBioPortal/cbioportal#8521, 3.1 Store user settings and 3.2 Add "reset to default" button based on app configuration in cBioPortal/cbioportal#8711, points 3 & 4 in cBioPortal/cbioportal#8716.

Changes:

Remembers active column selection when switching genes in Results View Mutation Table
Adds "Reset Columns" button which resets the visible columns back to their initial configuration (i.e., according to the visible attribute in the column definition). Note, this button is only shown when the current column selection is different from the default configuration. (Feedback on its design is appreciated.)
Fixes bug in cbioportal-frontend-commons where user-selection was prevented when passing columnVisibility prop to LazyMobXTable
inodb pushed a commit to cBioPortal/cbioportal-frontend that referenced this issue Jul 23, 2021
Fix cBioPortal/cbioportal#8713 and `2.2 Filtering per column` in cBioPortal/cbioportal#8711

Changes:

- Added two types of data filters, `NumericalFilter` and `CategoricalFilter`, which are attached to columns in the results view mutation table
- Numerical filters consist of a range slider with two handles and input boxes
- Categorical filters consist of search and multi-select functionality
- Multiple column filters can be active simultaneously, and they work in conjunction with the search filter in the top-right (i.e. only those rows which match ALL active filters will be shown)
- Hover cursor over column in order to show filter icon (if it's applicable for that column)
@stale
Copy link

stale bot commented Nov 6, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Nov 6, 2021
@jjgao jjgao removed the stale label Dec 27, 2021
@stale
Copy link

stale bot commented Apr 17, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Apr 17, 2022
@stale stale bot removed the stale label Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants