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

Merge Everest into ert #8663

Closed
wants to merge 1 commit into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
2 changes: 1 addition & 1 deletion .github/workflows/build_and_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,7 @@ jobs:

- name: Test docs
run: |
sphinx-build -n -v -E -W ./docs ./tmp/ert_docs
sphinx-build -n -v -E -W ./docs/ert ./tmp/ert_docs

publish:
name: Publish to PyPI
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ jobs:

- name: Install with dependencies
run: |
uv pip install ".[dev]"
uv pip install ".[dev, everest]"

- name: Test GUI
if: matrix.test-type == 'gui-tests'
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/doctest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,13 +36,13 @@ jobs:
run: pip install uv

- run: |
uv pip install -e ".[dev]"
uv pip install -e ".[dev, everest]"

- name: Test doctest
run: |
# dark storage assumes it is a started service so cannot be excluded
# by pytest blindly
pytest --doctest-modules --cov=ert --cov-report=xml:cov.xml src/ --ignore src/ert/dark_storage
pytest --doctest-modules --cov=ert --cov-report=xml:cov.xml src/ --ignore src/ert/dark_storage --ignore src/ieverest

- name: Upload coverage to Codecov
id: codecov1
Expand Down
79 changes: 79 additions & 0 deletions .github/workflows/mac-run-tests-build-docs.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,79 @@
name: macos-run-tests-build-docs

on:
pull_request:
workflow_dispatch:
push:
branches:
- main
tags: "*"

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}

env:
NO_PROJECT_RES: 1

jobs:
tests:
if: github.ref_type == 'tag' # only build for mac when tags
name: "py-${{ matrix.python-version }}_${{ matrix.test_type }}_${{ matrix.os }}"
timeout-minutes: 60
strategy:
fail-fast: false
matrix:
python-version: ['3.8', '3.12']
os: ['macos-13', 'macos-14', 'macos-14-large']
# https://github.com/actions/runner-images/tree/main?tab=readme-ov-file#available-images
exclude:
- os: 'macos-14'
python-version: '3.8'
- os: 'macos-14-large'
python-version: '3.8'
- os: 'macos-13'
python-version: '3.12'

test_type: ['doc', 'test', 'integration-test', 'everest-models-test']
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v4
with:
fetch-depth: 0
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}
cache: "pip"
cache-dependency-path: |
pyproject.toml

- name: Install HDF5 source files
run: brew install hdf5

- name: Install Everest and dependencies
run: |
pip install .[dev, everest]

- name: Run Tests
if: matrix.test_type == 'test'
run: |
python -m pytest tests/everest -n 4 -m "not ui_test and not integration_test and not fails_on_macos_github_workflow" --dist loadgroup

- name: Run Integration Tests
if: matrix.test_type == 'integration-test'
run: |
python -m pytest tests/everest -n 4 -m "integration_test and not fails_on_macos_github_workflow" --dist loadgroup

- name: Build Documentation
if: matrix.test_type == 'doc'
run: |
pip install git+https://github.com/equinor/everest-models.git
mkdir tmp
sphinx-build -n -v -E -W ./docs/everest/source ./tmp/everest_docs

- name: Run tests requiring everest-models
if: matrix.test_type == 'everest-models-test'
run: |
pip install git+https://github.com/equinor/everest-models.git
python -m pytest tests/everest -n 4 -m everest_models_test --dist loadgroup
71 changes: 71 additions & 0 deletions .github/workflows/run-tests-build-docs.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,71 @@
name: run-tests-build-docs

on:
pull_request:
workflow_dispatch:
push:
branches:
- main
tags: "*"

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}

env:
NO_PROJECT_RES: 1

jobs:
tests:
name: "py_${{ matrix.python-version }}_${{ matrix.test_type }}"
timeout-minutes: 60
strategy:
fail-fast: false
matrix:
python-version: ['3.8', '3.11', '3.12']
os: [ubuntu-22.04]
test_type: ['doc', 'ui-test', 'test', 'integration-test', 'everest-models-test']
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v4
with:
fetch-depth: 0
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}
cache: "pip"

- name: Install Everest and dependencies
run: |
pip install ".[dev,everest]"

- name: Run Tests
if: matrix.test_type == 'test'
run: |
pytest tests/everest -n 4 -m "not ui_test and not integration_test" --dist loadgroup -sv

- name: Run Integration Tests
if: matrix.test_type == 'integration-test'
run: |
pytest tests -n 4 -m "integration_test" --dist loadgroup

- name: Run UI Tests
if: matrix.test_type == 'ui-test'
env:
QT_QPA_PLATFORM: 'minimal'
run: |
python -m pytest tests/everest -m "ui_test"

- name: Build Documentation
if: matrix.test_type == 'doc'
run: |
pip install git+https://github.com/equinor/everest-models.git
mkdir tmp
sphinx-build -n -v -E -W ./docs/everest/source ./tmp/everest_docs

- name: Run tests requiring everest-models
if: matrix.test_type == 'everest-models-test'
run: |
pip install git+https://github.com/equinor/everest-models.git
python -m pytest tests -n 4 -m everest_models_test --dist loadgroup
6 changes: 3 additions & 3 deletions .github/workflows/test_ert.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,17 +45,17 @@ jobs:
- name: Test GUI
if: inputs.test-type == 'gui-test'
run: |
pytest tests --junit-xml=junit.xml -v --mpl -m "requires_window_manager" --benchmark-disable
pytest tests --junit-xml=junit.xml -v --mpl -m "requires_window_manager" --benchmark-disable --ignore=tests/everest

- name: Unit Test
if: inputs.test-type == 'unit-tests'
run: |
pytest tests --junit-xml=junit.xml -n logical --show-capture=stderr -v -m "not integration_test and not requires_window_manager" --benchmark-disable --dist loadgroup
pytest tests --junit-xml=junit.xml -n logical --show-capture=stderr -v -m "not integration_test and not requires_window_manager" --benchmark-disable --dist loadgroup --ignore=tests/everest

- name: Integration Test
if: inputs.test-type == 'integration-tests'
run: |
pytest tests --junit-xml=junit.xml -n logical --show-capture=stderr -v -m "integration_test and not requires_window_manager" --benchmark-disable
pytest tests --junit-xml=junit.xml -n logical --show-capture=stderr -v -m "integration_test and not requires_window_manager" --benchmark-disable --ignore=tests/everest

- name: Test for a clean repository
run: |
Expand Down
57 changes: 0 additions & 57 deletions .github/workflows/test_everest.yml

This file was deleted.

1 change: 1 addition & 0 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ repos:
- id: no-commit-to-branch
- id: check-json
- id: check-yaml
exclude: tests/everest/test_data/valid_config_file/invalid_yaml_config.yml # Invalid yml needed for testing
- id: end-of-file-fixer
- id: trailing-whitespace
exclude: test-data/eclipse/parse/ERROR.PRT # exact format is needed for testing
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes
File renamed without changes.
File renamed without changes
File renamed without changes.
File renamed without changes
File renamed without changes
2 changes: 1 addition & 1 deletion docs/conf.py → docs/ert/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
import sys
from importlib import metadata

sys.path.append(os.path.abspath("./_ext"))
sys.path.append(os.path.abspath("_ext"))

# -- Project information -----------------------------------------------------

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ The configuration file needed to use the ``installable_workflow_jobs`` hook must
and specify its arguments.
The built-in internal ``CSV_EXPORT`` workflow job is shown as an example:

.. literalinclude:: ../../../src/ert/resources/workflows/jobs/internal-gui/config/CSV_EXPORT
.. literalinclude:: ../../../../src/ert/resources/workflows/jobs/internal-gui/config/CSV_EXPORT

Implement the hook specification as follows to register the workflow job ``CSV_EXPORT``:

Expand All @@ -149,7 +149,7 @@ Implement the hook specification as follows to register the workflow job ``CSV_E
The second approach does not require creating a workflow job configuration file up-front,
and allows adding documentation.

.. literalinclude:: ../../../src/ert/plugins/hook_specifications/jobs.py
.. literalinclude:: ../../../../src/ert/plugins/hook_specifications/jobs.py
:pyobject: legacy_ertscript_workflow

Minimal example:
Expand Down
File renamed without changes.
File renamed without changes
2 changes: 1 addition & 1 deletion docs/index.rst → docs/ert/index.rst
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
.. image:: ../src/ert/gui/resources/gui/img/ert_icon.svg
.. image:: ../../src/ert/gui/resources/gui/img/ert_icon.svg
:align: center
:scale: 50%

Expand Down
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
<script src="../_static/js/theme.js"></script>
<link rel="index" title="Index" href="../genindex.html" />
<link rel="search" title="Search" href="../search.html" />
<link rel="next" title="&lt;no title&gt;" href="../about/index.html" />
<link rel="next" title="&lt;no title&gt;" href="../ert/about/index.html" />
<link rel="prev" title="Example: Polynomial function" href="Polynomial.html" />
</head>

Expand Down Expand Up @@ -54,11 +54,11 @@
</ul>
<p class="caption" role="heading"><span class="caption-text">Getting started</span></p>
<ul>
<li class="toctree-l1"><a class="reference internal" href="../getting_started/setup.html">Setup</a></li>
<li class="toctree-l1"><a class="reference internal" href="../getting_started/key_concepts.html">Key concepts (todo)</a></li>
<li class="toctree-l1"><a class="reference internal" href="../getting_started/configuration/poly_new/guide.html">Configuration Guide</a></li>
<li class="toctree-l1"><a class="reference internal" href="../getting_started/launch.html">Launching ERT</a></li>
<li class="toctree-l1"><a class="reference internal" href="../getting_started/gui/gui.html">Graphical User Interface</a></li>
<li class="toctree-l1"><a class="reference internal" href="../ert/getting_started/setup.html">Setup</a></li>
<li class="toctree-l1"><a class="reference internal" href="../ert/getting_started/key_concepts.html">Key concepts (todo)</a></li>
<li class="toctree-l1"><a class="reference internal" href="../ert/getting_started/configuration/poly_new/guide.html">Configuration Guide</a></li>
<li class="toctree-l1"><a class="reference internal" href="../ert/getting_started/launch.html">Launching ERT</a></li>
<li class="toctree-l1"><a class="reference internal" href="../ert/getting_started/gui/gui.html">Graphical User Interface</a></li>
</ul>
<p class="caption" role="heading"><span class="caption-text">Reference</span></p>
<ul>
Expand All @@ -84,12 +84,12 @@
</ul>
<p class="caption" role="heading"><span class="caption-text">Developer Documentation</span></p>
<ul>
<li class="toctree-l1"><a class="reference internal" href="../developer_documentation/roadmap.html">Roadmap</a></li>
<li class="toctree-l1"><a class="reference internal" href="../developer_documentation/dev-strategy.html">Development strategy</a></li>
<li class="toctree-l1"><a class="reference internal" href="../developer_documentation/milestones.html">Milestones</a></li>
<li class="toctree-l1"><a class="reference internal" href="../developer_documentation/concepts.html">Concepts</a></li>
<li class="toctree-l1"><a class="reference internal" href="../developer_documentation/forward_model.html">The forward model</a></li>
<li class="toctree-l1"><a class="reference internal" href="../developer_documentation/communication_protocol.html">Communication Protocol</a></li>
<li class="toctree-l1"><a class="reference internal" href="../ert/developer_documentation/roadmap.html">Roadmap</a></li>
<li class="toctree-l1"><a class="reference internal" href="../ert/developer_documentation/dev-strategy.html">Development strategy</a></li>
<li class="toctree-l1"><a class="reference internal" href="../ert/developer_documentation/milestones.html">Milestones</a></li>
<li class="toctree-l1"><a class="reference internal" href="../ert/developer_documentation/concepts.html">Concepts</a></li>
<li class="toctree-l1"><a class="reference internal" href="../ert/developer_documentation/forward_model.html">The forward model</a></li>
<li class="toctree-l1"><a class="reference internal" href="../ert/developer_documentation/communication_protocol.html">Communication Protocol</a></li>
</ul>
<p class="caption" role="heading"><span class="caption-text">API Reference</span></p>
<ul>
Expand All @@ -116,7 +116,7 @@
</ul>
<p class="caption" role="heading"><span class="caption-text">About</span></p>
<ul>
<li class="toctree-l1"><a class="reference internal" href="../about/release_notes.html">Release Notes</a></li>
<li class="toctree-l1"><a class="reference internal" href="../ert/about/release_notes.html">Release Notes</a></li>
</ul>

</div>
Expand Down Expand Up @@ -342,7 +342,7 @@ <h1>Example: Oscillator<a class="headerlink" href="#example-oscillator" title="P
</div>
<footer><div class="rst-footer-buttons" role="navigation" aria-label="Footer">
<a href="Polynomial.html" class="btn btn-neutral float-left" title="Example: Polynomial function" accesskey="p" rel="prev"><span class="fa fa-arrow-circle-left" aria-hidden="true"></span> Previous</a>
<a href="../about/index.html" class="btn btn-neutral float-right" title="&lt;no title&gt;" accesskey="n" rel="next">Next <span class="fa fa-arrow-circle-right" aria-hidden="true"></span></a>
<a href="../ert/about/index.html" class="btn btn-neutral float-right" title="&lt;no title&gt;" accesskey="n" rel="next">Next <span class="fa fa-arrow-circle-right" aria-hidden="true"></span></a>
</div>

<hr/>
Expand Down
Loading
Loading