Skip to content

Props attribute of Species class#278

Merged
connie merged 6 commits intoReactionMechanismGenerator:masterfrom
nickvandewiele:props_species_refactor
Aug 27, 2014
Merged

Props attribute of Species class#278
connie merged 6 commits intoReactionMechanismGenerator:masterfrom
nickvandewiele:props_species_refactor

Conversation

@nickvandewiele
Copy link
Copy Markdown
Contributor

This introduces a generic 'properties' dictionary called props in the Species object.

The attribute is optional and initialized to an empty dict.
The attribute is also cythonized.

RMG minimal, and TDE examples are run without error.

Do not forget to make clean;make before testing this out.

this allows the user to store user-defined, and arbitrary
key-value pairs of his/her choice.

Conflicts:[RESOLVED]

	rmgpy/species.py
…dict

this is more flexible than setting it in the constructor to an empty dict.
This way, one can also transfer a properties dict from elsewhere.

Conflicts:[resolved]

	rmgpy/species.py
testing if props attribute is a dictionary, and if dictionary entries
are properly stored.
@nickvandewiele
Copy link
Copy Markdown
Contributor Author

FYI, the name of the pull request branch has refactor in there, but nothing is really refactored, after some consideration. See #252.

connie added a commit that referenced this pull request Aug 27, 2014
Add props dictionary attribute to the Species class for storing various user-defined properties.
@connie connie merged commit 9dbc4c4 into ReactionMechanismGenerator:master Aug 27, 2014
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