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

Fixes for nastruct, ZDNA and parallel DNA #1018

Merged
merged 29 commits into from
Mar 9, 2023

Conversation

drroe
Copy link
Contributor

@drroe drroe commented Mar 9, 2023

Version 6.18.0.

  • ZDNA (left handed) is now properly detected and parameters have the correct sign.
  • Improved base pair detection, especially for parallel strands and/or Hoogsteen base pairing. Renders guessbp obsolete.
  • Parameters for parallel DNA can now be calculated using 3DNA-type conventions (new default) or Babcock et al. conventions (previous behavior). Can switch with new keyword, bpmode.
  • Improved Zp parameter calculation; fixed for parallel DNA.
  • Add new data set, <name>[nxyz] for printing the base pair axes normal (Z) vector.

drroe added 29 commits March 3, 2023 09:36
…e same direction. If so this was a chi flip and we also need to flip X. If not that is an error bc a flip around X should not be possible.
…rs, ensure axes are properly flipped before looking at cutoffs etc
… 'guessbp'; was always a hack to make up for the bad bp detection of non wc pairs which should now be fixed.
…3' to 5'. Flipx XZ when ZDNA in base pair determination
@drroe drroe added bugfix New keywords New keywords for existing commands. labels Mar 9, 2023
@drroe drroe self-assigned this Mar 9, 2023
@drroe
Copy link
Contributor Author

drroe commented Mar 9, 2023

@hainm I added a new data set to nastruct with aspect [nxyz] that is a DataSet_Vector (x y z, ox oy oz). How can I modify nucleic_acid_analysis.py in pytraj so it understands this? Right now the test is failing because everything else is a 1D array.

@drroe
Copy link
Contributor Author

drroe commented Mar 9, 2023

Since I have people waiting on this functionality I'm going to merge, but I'll open an issue over at pytraj so as not to forget.

@drroe drroe merged commit 8af5492 into Amber-MD:master Mar 9, 2023
@drroe drroe deleted the debug.nastruct.z branch March 9, 2023 21:19
@hainm
Copy link
Contributor

hainm commented Mar 9, 2023

but I'll open an issue over at pytraj so as not to forget.

yeah please open a case. Please post the error too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix enhancement New keywords New keywords for existing commands.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants