Skip to content

Bugfix: fixed error with duplicate property values - #106

Merged
seamm merged 1 commit into
mainfrom
dev
Aug 12, 2025
Merged

Bugfix: fixed error with duplicate property values#106
seamm merged 1 commit into
mainfrom
dev

Conversation

@paulsaxe

Copy link
Copy Markdown
Collaborator
  • Fixed a bug where if a property value was put in the database more than once, it created multiple entries rather than updating the original value. This led to e.g. bad property values in SDF files.
  • Added the 'spin_state' property to Configurations, which returns the spin state, e.g. singlet, triplet, corresponding to the spin multiplicity.

* Fixed a bug where if a property value was put in the database more than once, it
  created multiple entries rather than updating the original value. This led to
  e.g. bad property values in SDF files.
* Added the 'spin_state' property to Configurations, which returns the spin
  state, e.g. singlet, triplet,  corresponding to the spin multiplicity.
@paulsaxe
paulsaxe requested a review from seamm August 12, 2025 15:44
@paulsaxe paulsaxe added bug Something isn't working enhancement New feature or request labels Aug 12, 2025

@seamm seamm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@seamm
seamm merged commit 938384c into main Aug 12, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants