Skip to content

add immutability test to mixin#724

Draft
atravitz wants to merge 1 commit intomainfrom
tests/add_immutability
Draft

add immutability test to mixin#724
atravitz wants to merge 1 commit intomainfrom
tests/add_immutability

Conversation

@atravitz
Copy link
Contributor

@atravitz atravitz commented Feb 6, 2026

It's my understanding that gufetokenizables should be immutable, and therefore this test should pass. open to discussion though.

Tips

  • Comment "pre-commit.ci autofix" to have pre-commit.ci atomically format your PR.
    Since this will create a commit, it is best to make this comment when you are finished with your work.

Checklist

  • Added a news entry

Developers certificate of origin

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

No API break detected ✅

@IAlibay
Copy link
Member

IAlibay commented Feb 6, 2026

Our GufeTokenizables are unfortunately not all immutable, see SolventComponent:

In [1]: from gufe import SolventComponent

In [2]: a = SolventComponent()

In [3]: a.smiles
Out[3]: 'O'

In [4]: a._smiles = 'C'

In [5]: a.smiles
Out[5]: 'C'

The key is stable until you roundtrip on serialization.

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