Visitar URL original
Fix cartoon trace normal flips on tight curves by giagitom · Pull Request #1950 · molstar/molstar · GitHub
Skip to content

Fix cartoon trace normal flips on tight curves - #1950

Open
giagitom wants to merge 1 commit into
molstar:masterfrom
giagitom:fix-cartoon-trace-twist
Open

giagitom wants to merge 1 commit into
molstar:masterfrom
giagitom:fix-cartoon-trace-twist

Conversation

@giagitom

Copy link
Copy Markdown
Collaborator

Description

Propagate the ribbon frame with rotation minimizing frames (double reflection method), seeded from the previous segment's end normal, and distribute the residual twist to the chemically preferred normal, taken modulo 180deg since the profiles are 2-fold symmetric, evenly along the segment. Fixes #649

Actions

  • Added description of changes to the [Unreleased] section of CHANGELOG.md
  • Updated headers of modified files
  • Added my name to package.json's contributors
  • (Optional but encouraged) Improved documentation in docs

Propagate the ribbon frame with rotation minimizing frames (double
reflection method), seeded from the previous segment's end normal, and
distribute the residual twist to the chemically preferred normal, taken
modulo 180deg since the profiles are 2-fold symmetric, evenly along the
segment. Fixes molstar#649
@giagitom
giagitom requested a review from arose September 28, 2026 13:58
@arose

arose commented Oct 3, 2026

Copy link
Copy Markdown
Member

lets look at this after v6. we need to keep closer to the given direction normals.

This branch has not been deployed

No deployments
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.

Cartoon trace tilt issue

2 participants