8000 General Parameter interface. by chleh · Pull Request #1357 · ufz/ogs · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Merged
merged 24 commits into from
Aug 26, 2016
Merged

General Parameter interface. #1357

merged 24 commits into from
Aug 26, 2016

Conversation

chleh
Copy link
Collaborator
@chleh chleh commented Aug 17, 2016

Predecessor of #1356.

This PR introduces a general Parameter class whose main features are:

  • space-time dependency via double t and class SpatialPosition
  • parameters can have multiple components out of the box.
  • See Parameter
    and SpatialPosition

The SpatialPosition class represents either a node, an element, an integration point or a set of coordinates.

@chleh
Copy link
Collaborator Author
chleh commented Aug 17, 2016

Jenkins, test this please.

TomFischer
< 8000 div data-view-component="true" class="TimelineItem-body"> TomFischer reviewed Aug 19, 2016

// TODO other data types than only double
auto const& property =
mesh.getProperties().template getPropertyVector<double>(field_name);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is template necessary?

Copy link
Collaborator Author

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.

@TomFischer
Copy link
Member

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();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

e -> node_id, n, ...

@endJunction
Copy link
Member

⏩ afterwards...

@chleh chleh force-pushed the param-interface-base branch from e4835a6 to b46d339 Compare August 26, 2016 07:32
@chleh
Copy link
Collaborator Author
chleh commented Aug 26, 2016

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.

@chleh chleh merged commit 7e34c5b into ufz:master Aug 26, 2016
@chleh chleh deleted the param-interface-base branch August 26, 2016 08:24
@ogsbot
Copy link
Member
ogsbot commented Jun 19, 2020

OpenGeoSys development has been moved to GitLab.

See this pull request on GitLab.

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.

4 participants
0