Skip to content

fix: support None custom properties#29

Merged
Ramlaoui merged 2 commits intomainfrom
codex/fix-none-properties
May 10, 2026
Merged

fix: support None custom properties#29
Ramlaoui merged 2 commits intomainfrom
codex/fix-none-properties

Conversation

@Ramlaoui
Copy link
Copy Markdown
Collaborator

Summary

  • add an explicit null property type to the Rust storage model and SOA schema
  • accept and return None in the Python property bridge and database readers
  • preserve None values when importing custom properties from ASE metadata

Testing

  • rtk cargo test -p atompack soa_all_property_value_types
  • rtk uv run --no-sync pytest tests/test_atom_molecule.py::test_molecule_custom_properties tests/test_database.py::test_database_roundtrip -q
  • rtk uv run --no-sync python -c ... (from_ase nullable property smoke check)

Closes #4

@Ramlaoui Ramlaoui merged commit c71760e into main May 10, 2026
5 checks passed
@Ramlaoui Ramlaoui deleted the codex/fix-none-properties branch May 10, 2026 13:31
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.

Support None value for properties

1 participant