visualization improvement - #2
Merged
Merged
Conversation
Three complementary improvements:
1. Helix: render as longitudinal zigzag stripes instead of cross-sectional rings
- Each of 16 stripes (up from 8) runs along the cylinder axis at a fixed angle;
alternating stripes go forward/backward so transitions are short end-cap arcs
rather than long diagonal cuts across the surface → clean tube appearance
- Axial slices scale with helix length (one per residue, was min(8, len))
so longer helices are no longer under-sampled
2. Sheet: process the full strand segment at once with a consistent perpendicular
- Was: pair-by-pair, each pair independently computing its own normal (could flip)
- Now: overall segment direction determines a single stable n1 vector; ribbon
drawn as zigzag parallel stripes across the full strand → uniform flat ribbon
- Wider ribbon: width 4→6 steps, offset 0.05→0.07 per step (total ±0.42 units)
3. Backbone: Catmull-Rom spline interpolation in all three project() paths
- 4 sub-steps per Cα–Cα bond using the four surrounding control points
- Turns every straight segment into a smooth curve; visually most significant
for loop/coil regions and helix spirals without -s flag
https://claude.ai/code/session_01LaPVogs2SPnVqeSSbd2ux6
Convenience patch for applying the secondary structure commit on top of a main branch that already has the Braille rendering commit merged. Apply with: git am structty-improvements.patch https://claude.ai/code/session_01LaPVogs2SPnVqeSSbd2ux6
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
visualization improvement