Skip to content

Use encoded style ids for AutoRep representations - #297

Open
cpignedoli wants to merge 2 commits into
split/pr-19-auto-representationsfrom
feature/auto-rep-encoded-style-ids
Open

Use encoded style ids for AutoRep representations#297
cpignedoli wants to merge 2 commits into
split/pr-19-auto-representationsfrom
feature/auto-rep-encoded-style-ids

Conversation

@cpignedoli

@cpignedoli cpignedoli commented Jun 24, 2026

Copy link
Copy Markdown
Member

What

PR-S1 for surfaces AutoRep support, stacked on the existing split surfaces chain.

This PR adds AutoRep support for the encoded viewer representation IDs introduced in aiidalab-widgets-base PR-R1 (#768) and keeps imported/stored viewer representation arrays intact when structures change.

AutoRep remains surfaces-only because it uses lowdimfinder/surfaces domain logic. The widget writes molecule atoms as ball+stick and non-molecule atoms as spacefill using the same encoded representation naming convention as widgets-base.

Dependencies

Implemented with Codex.

Validation

  • Manually tested uploaded encoded XYZ representation arrays.
  • Manually tested storing and reloading representation extras through AiiDA nodes.
  • Manually tested switching from stored representation-rich structures to plain structures without stale empty rows corrupting the viewer.
  • Syntax check:
python -m py_compile surfaces_tools/widgets/auto_representations.py

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.

1 participant