-
Notifications
You must be signed in to change notification settings - Fork 12
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
Switch to relative coordinates #131
Comments
Just a small comment regarding the attached files - the |
Just to double-check, so it seems like residues still have absolute coordinates, but things like labels now have relative coordinates (relative to the coordinates of the corresponding residue I assume)? To avoid this being a breaking change, would it be possible to include some sort of "flag" property that would specify whether coordinates are to be interpreted as absolute or relative (and then to still interpret coordinates as absolute by default)? |
@davidhoksza or @CaedenM - would you be able to comment on this please?
Having a flag for absolute or relative coordinates sounds like a good idea to me - what does everyone else think? |
@pzhaojohnson Indeed, coordinates of the label and label lines relative to the coordinates of the corresponding residue The flag should be added to the JSON schema, right? No problem with me to include it in the output from Traveler once it's in the schema... |
@CaedenM and @aspetr01, what do you think about adding the flag suggested by Philip to the schema? Using a flag the transition can be made asynchronously. RNAcanvas and XRNA-React could deploy new code on their own schedule and use the flag if it's present or default to the absolute coordinates in its absence for backwards compatibility. |
Would using |
The RNA 2D JSON Schema is switching from absolute to relative coordinates. This means that the code that uses R2DT JSON output needs to be updated or the SVG images that are generated using the new files will be incorrect.
It is necessary to update R2DT without breaking the integration with XRNA-React and RNAcanvas.
There is already a new version of Traveler that implements the change (thanks @davidhoksza!):
https://github.com/cusbg/traveler/tree/relative-labels
Here is the plan:
Here are some example files provided by @davidhoksza that can be used for testing:
Does this plan work? Any ideas or suggestions are very welcome!
cc @aspetr01
The text was updated successfully, but these errors were encountered: