-
8000
-
Notifications
You must be signed in to change notification settings - Fork 239
General Parameter interface. #1357
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
Conversation
Jenkins, test this please. |
|
||
// TODO other data types than only double | ||
auto const& property = | ||
mesh.getProperties().template getPropertyVector<double>(field_name); |
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.
Why is template necessary?
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.
Removed. GCC eats it without it.
Code looks okay to me. After resolving conflicts: ⏩ |
std::vector<T> const& getTuple(double const /*t*/, | ||
SpatialPosition const& pos) const override | ||
{ | ||
auto const e = pos.getNodeID(); |
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.
e -> node_id, n, ...
⏩ afterwards... |
e4835a6
to
b46d339
Compare
I won't squash the remaining fixup commits since I have a file rename commit in between and I'm too lazy to untangle this. |
OpenGeoSys development has been moved to GitLab. |
Predecessor of #1356.
This PR introduces a general Parameter class whose main features are:
double t
andclass SpatialPosition
and SpatialPosition
The
SpatialPosition
class represents either a node, an element, an integration point or a set of coordinates.