-
Notifications
You must be signed in to change notification settings - Fork 233
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
Update outdated material model interface description #1839
Update outdated material model interface description #1839
Conversation
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.
OK if you address the minor comments.
doc/manual/manual.tex
Outdated
instead of the evaluate() function discussed above. They are now a deprecated | ||
way of implementing a material model. You can get your old model working | ||
by deriving from InterfaceCompatibility instead of Interface. | ||
instead of the evaluate() function discussed above. This old interface is not longer |
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.
-> is NO longer supported
doc/manual/manual.tex
Outdated
way of implementing a material model. You can get your old model working | ||
by deriving from InterfaceCompatibility instead of Interface. | ||
instead of the evaluate() function discussed above. This old interface is not longer | ||
supported, and you need to restructure your plugin to implement evaluate() |
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 drop ", and you need to ...". This sentence is only true for a short period of time (because everyone will have updated it quickly) and it's not worth keeping in perpetuity.
doc/manual/manual.tex
Outdated
@@ -10316,9 +10302,9 @@ \subsubsection{Material models} | |||
starting point for implementations of new material models. | |||
|
|||
Older versions of \aspect{} used to have individual functions like viscosity() | |||
instead of the evaluate() function discussed above. They are now a deprecated |
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.
While you're there, can you also mark up "viscosity()" and "evaluate()" as code?
43a450a
to
0cbe91b
Compare
Done. |
Found these outdated descriptions during reading a section of the manual.