Skip to content
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

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rbe051
Copy link
Contributor

@rbe051 rbe051 commented Oct 16, 2019

I'm a new Post-doc at Norce.

@atgeirr
Copy link
Member

atgeirr commented Oct 16, 2019

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
Copy link
Member

atgeirr commented Oct 16, 2019

jenkins build this please

eclipseGrid->exportZCORN( zcorn );
eclipseGrid->exportACTNUM( actnum );

std::vector<double> mapaxes = eclipseGrid->getMAPAXES( );
Copy link
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
Copy link
Member

blattms commented Oct 16, 2019

release or not release, that is the question.

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

atgeirr commented Oct 16, 2019

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.

None yet

4 participants