Skip to content

0.17.1#208

Merged
UnHumbleBen merged 44 commits intomainfrom
dev
Dec 20, 2021
Merged

0.17.1#208
UnHumbleBen merged 44 commits intomainfrom
dev

Conversation

@dave-doty
Copy link
Copy Markdown
Member

No description provided.

cesarg707 and others added 30 commits July 30, 2021 12:02
… constructor for ModificationInternal, instead of just frozenset
… delimiter symbols in between each Domain/Loopout
…dnano-export

Refactor cadnano import/export
…it-tests-for-TestAssignDNA

#186 add unit tests for test assign dna
…t_template

Create pull_request_template.md
…xDNA-export-when-multiple-helix-groups-are-present

closes #198: allow oxDNA export when multiple helix groups are present
@dave-doty
Copy link
Copy Markdown
Member Author

Release notes

Several bug fixes and one breaking change in how oxDNA export works.

We have jumped from version 0.16.3 to 0.17.1 (skipping 0.17.0) because of a mistaken update to PyPI with version 0.17.0. That version has been "yanked" (https://pypi.org/project/scadnano/0.17.0/), but the PyPI site will not allow an update to that version, so we are skipping past it directly to version 0.17.1.

Fix Pitch Angle on oxDNA Export

(See also https://github.com/UC-Davis-molecular-computing/scadnano/releases/tag/v0.17.0)

Note: This is a breaking change since it changes how oxDNA export is defined. If you were using oxDNA export before, you will have to change some helix angles (pitch and yaw) to create the same oxDNA output that was previously generated.

Previously, scadnano interpreted the pitch angle as a clockwise rotation in the Y-Z plane, following SVG convention. For example, the following design has a helix group (containing helix 1) with pitch=45 (clockwise, away from the single strand on helix 0):

image

Previously, exporting to oxDNA rotates the helix in the opposite direction (counter-clockwise, towards the single strand on helix 0). Now, exporting to oxDNA rotates the helix in the same direction as in scadnano:

Old New
drawing drawing

@dave-doty dave-doty marked this pull request as ready for review December 19, 2021 17:41
Copy link
Copy Markdown
Collaborator

@UnHumbleBen UnHumbleBen left a comment

Choose a reason for hiding this comment

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

There were a lot of changes in the scadnano.py and scadnano_tests.py file, but I figured that these changes have been reviewed already in earlier PRs so didn't examine too closely. Everything else looks good.

@UnHumbleBen UnHumbleBen merged commit 6aef88f into main Dec 20, 2021
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.

4 participants