-
Notifications
You must be signed in to change notification settings - Fork 51
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
Upgrade RGL and support allowOverlap
#1046
base: develop
Are you sure you want to change the base?
Conversation
allowOverlap
for React Grid Layout
allowOverlap
for React Grid LayoutallowOverlap
for React Grid Layout
allowOverlap
for React Grid LayoutallowOverlap
for React Grid Layout
allowOverlap
for React Grid LayoutallowOverlap
allowOverlap
allowOverlap
Hi Sehi, I think updating rgl is great! I'm not so sure about having overlapping views. We don't allow transparency so one matrix would just occlude the other. What's the use case? Also AFAICT zooming doesn't work because all of the events are caught by the topmost view. If you wanted to enable this in gosling, why not just create two separate higlass components and overlay them? |
Hi Pete, I guess it may not be useful for now, but I was thinking about potential use cases with additional encoding functionality integrated (e.g., left-bottom half of the diagonal shows chr1 and the other side shows chr2). But, I am not sure if this kind of cross-region comparison makes sense at all in the HiC analysis... Yes, the interaction does not work, and it is something that I wanted to somehow address as well in the future. I previously tried with multiple HiGlass components, but since using one component showed better performance with multiple zoom and location locks with multiple views, I was trying to use only one component as much as possible. |
I think we do support transparency. At least back in 2019 at Scipy we overlayed a heatmap onto a worldmap. I understand the basic idea: compare two regions through superimposition, however, I do share the concern with Pete that it's not clear at all how interactions are suppose to work in that case. I wonder if this special case could also be handles through a meta-track like scalable-insets did. Other than that I appreciate the update of RGL 🎉 |
Thank you both for reviewing the PR! @pkerpedjiev @flekschas I think it was not a good idea to add documentation already without proper interactions supported. Regarding interactions, I was thinking of using location locks (with offsets) and zoom locks so that navigation on the top view affects the other, or allowing users to somehow interactively change the z-index of superposed visualizations. But, I would need to think about this more with some potential use cases. If you think it does not hurt to have |
If we include it, it should be with the documentation. If it’s useful to
you we can include it and have it be off by default.
Be careful with future work with event handling. That code is complex and
potentially fragile.
…On Tue, Aug 31, 2021 at 7:03 AM sehilyi ***@***.***> wrote:
Thank you both for reviewing the PR! @pkerpedjiev
<https://github.com/pkerpedjiev> @flekschas <https://github.com/flekschas>
I think it was not a good idea to add documentation already without proper
interactions supported. Regarding interactions, I was thinking of using
location locks (with offsets) and zoom locks so that navigation on the top
view affects the other, or allowing users to somehow interactively change
the z-index of superposed visualizations. But, I would need to think about
this more with some potential use cases.
If you think it does not hurt to have allowOverlap and can be potentially
useful with proper interactions integrated, do you think we can remove the
documentation for now but still have the allowOverlap as a root-level
property?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1046 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAQLLDKNXIR7DH7FPZQITHLT7TOJ3ANCNFSM5CU7UC7A>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Description
This PR adds a root-level option of viewConfig, called
allowOverlap
:This uses react-grid-layout's recently released option,
allowOverlap
.When a user sets
allowOverlap
totrue
(defaultfalse
), higlass views can be overlapped. While this functionality may not be needed for most cases, this can be useful if a user wants to overlay multiple visualizations that show different genomic intervals (e.g., superpose a matrix that shows a part of chr1 and another matrix that shows a part of chr2). In Gosling, we would like to create for example multiple circular ideograms of different chromosomes at once (gosling-lang/gosling.js#422).I've upgrade RGL, from
0.16.6
to1.3.0
, for this update. To make HiGlass compatible with the new version of RGL, I also had to edit one part of the code inHiGlassComponent.js
:Other than this issue, I think the new version is compatible by judging from the CHANGELOG.md, especially the "Breaking Updates" parts.
@pkerpedjiev, do you think we can upgrade RGL?
Checklist