Skip to content

General Parameter interface.#1357

Merged
chleh merged 24 commits intoufz:masterfrom
chleh:param-interface-base
Aug 26, 2016
Merged

General Parameter interface.#1357
chleh merged 24 commits intoufz:masterfrom
chleh:param-interface-base

Conversation

@chleh
Copy link
Copy Markdown
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
Copy Markdown
Collaborator Author

chleh commented Aug 17, 2016

Jenkins, test this please.


// TODO other data types than only double
auto const& property =
mesh.getProperties().template getPropertyVector<double>(field_name);
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member

⏩ afterwards...

@chleh chleh force-pushed the param-interface-base branch from e4835a6 to b46d339 Compare August 26, 2016 07:32
@chleh
Copy link
Copy Markdown
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
Copy Markdown
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