Skip to content

Support for two energy terms of the same type per State #58

@jakublala

Description

@jakublala

As pointed out by @arnav-cheruku, currently we do not support having two energy terms of the same type on the State, as they will get overwritten in the dictionary that keeps track of the value:

_energy_terms_value: dict[(str, float)] = field(default_factory=lambda: {}, init=False)

We should fix this by either making the dictionary take the actual energy term as the key; or have unique identifiers.

Moreover, we should support custom names for the terms, or have some sort of a clear differentiator of different terms of the same type, so that the logging also works properly. This has not been a problem - for instance - for the pLDDT term given that we also have an OverallPLDDT term that has made this bug hard to see.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions