New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Particle property Solid Comp #1692
Conversation
…e depletion. This can be used to track a simplistic petrological evolution of material in the model.
Property renamed as requested @gassmoeller |
/run-tests |
…nged name from peridotite to composition.
Changes made. Now tracks all composition fields. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just two stylistic comments. Otherwise good to go. Thanks for suffering my requests for changes 👍
doc/modules/changes/20170515_schools
Outdated
if there is a compositonal field named "porosity". | ||
if there is a compositonal field named "porosity". A second new particle property | ||
is also added which indicates the peridotite depletion value. This can be used | ||
to track the petrological evolution of material. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update the entry.
{ | ||
std::ostringstream field_name; | ||
field_name << this->introspection().name_for_compositional_index(i); | ||
property_information.push_back(std::make_pair(field_name.str(),1)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can simplify these 3 lines into the following:
const std::string field_name = this->introspection().name_for_compositional_index(i);
property_information.push_back(std::make_pair(field_name,1));
Requested changes made. Thank you for suffering through my ignorance and thank you for all your help over the last ten days. |
The last formality. You will need to download the
Make a new commit and push your branch. Then the tester should pass, and this is ready to go. |
Patch applied |
Thanks! Ready to go. |
I added a new particle property solid comp which records the peridotite depletion. I believe this is currently the best way in aspect to track the petrological evolution of the the model and in conjunction with pT path, can be used to compare to MELTS or other thermodynamic software/databases.