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.

Sign up for GitHub

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

Jump to bottom

Analytics in prediction mode #365

Merged
merged 38 commits into from
Mar 19, 2021
Merged

Analytics in prediction mode #365

merged 38 commits into from
Mar 19, 2021

Conversation

wouterjdb
Copy link
Contributor

Supersedes "Analytics in prediction mode #349". Closes #349.

Contributor checklist

  • 🎉 This PR closes Enable analytics workflow in prediction mode #335.
  • 📜 I have broken down my PR into the following tasks:
    • Task 1
    • Task 2
  • 🤖 I have added tests, or extended existing tests, to cover any new features or bugs fixed in this PR.
  • 📖 I have considered adding a new entry in CHANGELOG.md.
  • 📚 I have considered updating the documentation.

LonnekevB and others added 30 commits March 3, 2021 16:16
@LonnekevB
Also possible to run with single number additional flownodes but mult…
11ecf7e 
…ple layers
@LonnekevB @wouterjdb
Apply suggestions from code review
13ba234 
Co-authored-by: Wouter J. de Bruin <9119793+wouterjdb@users.noreply.github.com>
@LonnekevB
Added changelog
2253e1a
@LonnekevB
Merge branch 'master' into i335_Analytics_predition_mode
57f883d
@LonnekevB
add analysis to pred config parser
42f5fad
@LonnekevB
Config pars pred reading in ref_sim and analysis metrics
7f853f7
@LonnekevB
Runs again without ref_sim and analysis keywords in config
22f3b4e
@LonnekevB
Analytics enabled for prediction mode
0a50069
@LonnekevB
Pass keyword mode to analytics workflow to determine if running in ah…
feda7a7 
…m or prediction mode
@LonnekevB
update abstract method of get_well_connections to fix pylint bug
aa1ca42
@LonnekevB
Remove perforation argument from FlowData from test
8b62bc2
@LonnekevB
Fix type error
f49cad2
@LonnekevB
Fix type error
12bfa37
@LonnekevB
Move all but abs_path transforamtion function to seperate script
920cde8
@LonnekevB
pylint fix
1d02657
@LonnekevB
update docstring get_well_connections
ae4cc97
@LonnekevB
black reformat
5fcfa04
@LonnekevB
Added Changelog
2c76d91
@LonnekevB
Added Changelog
df60da6
@LonnekevB
Merge branch 'i335_Analytics_predition_mode' of github.com:LonnekevB/…
9bd7220 
…flownet into i335_Analytics_predition_mode
@LonnekevB
Update CHANGELOG.md
4e83eeb
@LonnekevB
Merge branch 'master' into i335_Analytics_predition_mode
e40b81f
@LonnekevB
Merge branch 'master' into i335_Analytics_predition_mode
e95bdf9
@LonnekevB
Merge branch 'i335_Analytics_predition_mode' of github.com:LonnekevB/…
9170635 
…flownet into i335_Analytics_predition_mode
@LonnekevB
Suggested changes from review
9d60ff0
@LonnekevB
fix linting error
2682f4e
@LonnekevB
update description of analysis in config parser
54808e8
@LonnekevB
Clean up of analytics code
4a283ec
@LonnekevB
Restructure of ahm analytics tests
1711ef7
@LonnekevB
Sorting of iteration fixed
59d4c79
@wouterjdb wouterjdb added bug Something isn't working enhancement New feature or request labels Mar 19, 2021
@wouterjdb wouterjdb merged commit d410aa2 into equinor:master Mar 19, 2021
@wouterjdb wouterjdb deleted the i349_pr_fixes branch March 20, 2021 11:58
@edubarrosTNO edubarrosTNO mentioned this pull request Mar 20, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anders-kiaer anders-kiaer Awaiting requested review from anders-kiaer

@olelod olelod Awaiting requested review from olelod olelod is a code owner

Assignees

@LonnekevB LonnekevB

Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Enable analytics workflow in prediction mode
2 participants
@wouterjdb @LonnekevB