-
-
Notifications
You must be signed in to change notification settings - Fork 193
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
Remove alignment viewer #425
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise LGTM 💃
* your genes or proteins snappily thanks to the underlying WebGL architecture | ||
* powering the component. You can quickly scroll through your long sequence | ||
* with a slider or a heatmap overview. | ||
* Note that the AlignmentChart only returns a chart of the sequence, while |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This docs looks a bit confusing to me.Same as in Alignment Chart which we'll keep
dash-bio/src/lib/components/AlignmentChart.react.js
Lines 17 to 19 in 447ebbe
* Note that the AlignmentChart only returns a chart of the sequence, while | |
* AlignmentViewer has integrated controls for colorscale, heatmaps, and subplots | |
* allowing the user to interactively control their sequences. |
Does this Alignment Viewer
here refers to original react component? I didn't go detailed into the difference between this two components but we could skip mentioning the one we won't be using as it'll be packed into docstring reference.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! Will fix this ASAP :)
4aacecb
to
503382f
Compare
540984e
to
9d80694
Compare
9d80694
to
cb0e4c5
Compare
f311e53
to
842fa53
Compare
842fa53
to
0adc2b7
Compare
0adc2b7
to
42e108e
Compare
Closes #414
About
Description of changes
Before merging