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

EclSum doesn't read network vectors #943

Closed
alifbe opened this issue Jan 3, 2024 · 2 comments · Fixed by #951
Closed

EclSum doesn't read network vectors #943

alifbe opened this issue Jan 3, 2024 · 2 comments · Fixed by #951
Assignees
Labels

Comments

@alifbe
Copy link

alifbe commented Jan 3, 2024

Network vectors from intersect are not imported.

This has been resolved in ResInsight in their local copy

OPM/ResInsight#8990

@sondreso sondreso added this to SCOUT Jan 4, 2024
@sondreso sondreso added the bug label Jan 4, 2024
@eivindjahren eivindjahren moved this to Todo in SCOUT Jan 4, 2024
@eivindjahren
Copy link
Collaborator

The problem is that set_gen_keys does not assign a key to a network var so when you try to look up the var in EclSum by key, then you don't find any network vars.

https://github.com/equinor/ecl/blob/ed744e8b5c28510f41ca021c0318ba0208525e51/lib/ecl/smspec_node.cpp#L663

@eivindjahren
Copy link
Collaborator

@andreas-el andreas-el self-assigned this Jan 29, 2024
@andreas-el andreas-el moved this from Todo to In Progress in SCOUT Jan 29, 2024
@eivindjahren eivindjahren moved this from In Progress to Todo in SCOUT Jan 30, 2024
@andreas-el andreas-el moved this from Todo to In Progress in SCOUT Feb 5, 2024
@andreas-el andreas-el moved this from In Progress to Ready for Review in SCOUT Feb 5, 2024
@github-project-automation github-project-automation bot moved this from Ready for Review to Done in SCOUT Feb 5, 2024
@andreas-el andreas-el moved this from Done to Done-Done in SCOUT Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants