Skip to content

Add atom bonds cache and benchmark for SARS-CoV-2#71

Merged
unkcpz merged 5 commits into
rs4rse:mainfrom
syzer:feature/bounds-cache
Mar 4, 2026
Merged

Add atom bonds cache and benchmark for SARS-CoV-2#71
unkcpz merged 5 commits into
rs4rse:mainfrom
syzer:feature/bounds-cache

Conversation

@syzer

@syzer syzer commented Mar 3, 2026

Copy link
Copy Markdown
Collaborator

inferred atom bonds are cached. add benchmark for 6VXX inference. first inference took ~0.026s for 6VXX (23,694 atoms, 24,297 bonds), with cache 200,000x loop timing effectively: 0ms.

  • bond_tolerance_controls
  • apply_bond_tolerance_debounce
  • mark_bond_cache_dirty (in Running)
  • update_color_mode_availability (after mark_bond_cache_dirty)
  • update_atom_hover_cache (after update_color_mode_availability)
  • update_scene (after mark_bond_cache_dirty)

And update_scene still feeds the render-related follow-ups:

  • sync_atom_selection_highlight after update_scene
  • update_bond_order_legend after update_scene
  • update_atom_hover_label after update_scene
Cache

@syzer syzer force-pushed the feature/bounds-cache branch from 71a8e60 to edf8446 Compare March 4, 2026 11:07
@syzer syzer changed the title Add benchmark recipe Add benchmark and performance fix for SARS-CoV-2 Mar 4, 2026
@syzer syzer marked this pull request as ready for review March 4, 2026 15:34
@syzer syzer changed the title Add benchmark and performance fix for SARS-CoV-2 Add atom bonds cache and benchmark for SARS-CoV-2 Mar 4, 2026

@unkcpz unkcpz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

purely well done. Thanks (again) @syzer

Comment thread vizmat-core/benches/bond_cache.rs
@unkcpz unkcpz merged commit 1d62bd9 into rs4rse:main Mar 4, 2026
4 checks passed
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.

2 participants