Skip to content

Renamed functions in opm/grid/polyhedralgrid/grid.hh after changes in previous commit in opm-common#399

Open
rbe051 wants to merge 1 commit into
OPM:masterfrom
rbe051:fix_renaming_polyhedral
Open

Renamed functions in opm/grid/polyhedralgrid/grid.hh after changes in previous commit in opm-common#399
rbe051 wants to merge 1 commit into
OPM:masterfrom
rbe051:fix_renaming_polyhedral

Conversation

@rbe051

@rbe051 rbe051 commented Oct 16, 2019

Copy link
Copy Markdown
Contributor

I'm a new Post-doc at Norce.

@atgeirr

atgeirr commented Oct 16, 2019

Copy link
Copy Markdown
Member

I'm a new Post-doc at Norce.

Welcome to OPM!

I'll run Jenkins on this although I think it will not pick up any mistakes, as there are no tests for this grid (which is why I think this change was not caught earlier).

@atgeirr

atgeirr commented Oct 16, 2019

Copy link
Copy Markdown
Member

jenkins build this please

eclipseGrid->exportZCORN( zcorn );
eclipseGrid->exportACTNUM( actnum );
std::vector<double> mapaxes = eclipseGrid->getMAPAXES( );

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.

these can be const refs afaict.

@blattms

blattms commented Oct 16, 2019

Copy link
Copy Markdown
Member

release or not release, that is the question.

@atgeirr atgeirr added this to the Release 2019.10 milestone Oct 16, 2019
@atgeirr

atgeirr commented Oct 16, 2019

Copy link
Copy Markdown
Member

release or not release, that is the question.

This is a compile failure for the code in question, so I added it to the release milestone.

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