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.

Sign up for GitHub

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

Jump to bottom

enrichment tool tweaks #29

Open
ppavlidis opened this issue Jan 16, 2018 · 3 comments
Open

enrichment tool tweaks #29

ppavlidis opened this issue Jan 16, 2018 · 3 comments
Assignees
@JacobsonMT

Comments

@ppavlidis
Copy link
Collaborator

  • The "run enrichment" button should be disabled while the analysis is running.
  • The CC BP and MF buttons are hard to read - as to whether they are pressed or not.
  • Possibly the default should be BP only?
@ppavlidis ppavlidis assigned JacobsonMT and unassigned JacobsonMT Jan 16, 2018
@ppavlidis
Copy link
Collaborator Author

  • Might need a 'cancel' button if analyses aren't super-fast.

@ppavlidis
Copy link
Collaborator Author

The following screen exposes a few issues:

  • unreasonable precision in numbers
  • Infinities

image

@ppavlidis
Copy link
Collaborator Author

  • The window below loses the context - the title should indicate the time point.
  • Arguably the gene links here should be to the gotrack tracker, not Uniprot. Providing a uniprot link on the gene page is fine, but here we should be encouraging navigation within gotrack.
  • It's difficult to do a comparison of this list to one from a different time point since this window is modal. Perhaps there could be "arrow" buttons to navigate forwards and backwards in time?

image

JacobsonMT added a commit that referenced this issue Mar 16, 2018
@JacobsonMT
Resolve #34: Enrichment iew details needs polish
af8322e 
Polishes enrichment results section in general.

- Removes right clock context menu and replaces
    with row expander.
- Removes double click functionality and replaces
    with graph button near stability score.
- Adds more links to GO Id cell.

Related to Issue #29.
JacobsonMT added a commit that referenced this issue Mar 16, 2018
@JacobsonMT
Simplify and polish enrichment view
82a4fe0 
Removes tabs and makes charts smaller to fit in one screen.
Cleans up lots of little outstanding issues.

Related to Issue #29.
JacobsonMT added a commit that referenced this issue Mar 29, 2018
@JacobsonMT
Resove #29: enrichment tool tweaks
52af450 
- BP is default
- Add links to gene symbols
- Reorganize elements
- Add a genes backing top terms section
- Clicking similarity chart now changes table data
- Fix tooltips and dialog sigfigs in stab. chart
- Add more contextual help to stability chart
- Easier to see when buttons are selected
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JacobsonMT JacobsonMT

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ppavlidis @JacobsonMT