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

Fixed Excel Export #919, #907 #920

Merged
merged 8 commits into from
Apr 7, 2021
Merged

Fixed Excel Export #919, #907 #920

merged 8 commits into from
Apr 7, 2021

Conversation

Onariaginosa
Copy link
Collaborator

@Onariaginosa Onariaginosa commented Apr 7, 2021

Export to Excel works completely for all accepted sheets. See #919 & #907 for further information

Pull Request Checklist

  • Travis C.I. build passes
  • All four Demos look as expected when loaded
  • Reload works as expected
  • Loading file with Error brings up error modal, and it looks as expected
  • Loading file with Warning brings up warnings modal
  • Import works for both SIF and GraphML
  • Graph can be exported to SIF and GraphML
  • Print works as expected
  • Restrict graph to viewport works as expected (check/uncheck)
  • Viewport Size Changing works as expected (small/medium/large/fit)
  • Toggle between Grid Layout and Force Graph Layout works as expected
  • Force Graph Parameter Sliders change the number above them and have an effect on the graph
  • Locking/Unlocking/Resetting/Undo Resetting the force graph parameters works as expected
  • Enabling and disabling node coloring works as expected
  • Node coloring options work as expected (selection top/bottom dataset, averaging values, changing max value)
  • Weighted graph loaded with the "default to black edges" option checked appears as expected
  • Hide/Show Weights works as expected (always/never/upon mouseover)
  • Edge Weight Normalization Factor can be changed (set/reset)
  • Gray Edge Threshold can be changed, slider changes the number and has an effect on the graph
  • Checking Show Gray Edges as Dashed works as expected (check/uncheck)
  • D-pad left/right/top/bottom/center works as expected
  • Zoom slider changes number and has effect on graph (viewport and menu)
  • Right click on a node opens gene page, page is populated with correct data

@dondi
Copy link
Owner

dondi commented Apr 7, 2021

Some checklist notes:

  • Just verifying: “default to black edges” has changed labels to “Enable Edge Coloring” I think?
  • Gene page code may need updating—fewer genes appear to load correctly. e.g., ZAP1 is OK but HMO1 is not—I seem to recall that virtual all genes in our demos used to load something, but now not all of them do

Copy link
Owner

@dondi dondi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants