Skip to content

👽️ PROCESS develop api changes #3636

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

Draft
wants to merge 102 commits into
base: develop
Choose a base branch
from
Draft

Conversation

je-cook
Copy link
Contributor

@je-cook je-cook commented Oct 11, 2024

Description

Updates our PROCESS api for the current develop version of PROCESS.

closes #3650 as of process 3108

Interface Changes

Many....
Still todo:

  • check for any leftover old variable names code wide
  • check process example
  • REVERT private data branch

Checklist

I confirm that I have completed the following checks:

  • Tests run locally and pass pytest tests --reactor
  • Code quality checks run locally and pass pre-commit run --from-ref develop --to-ref HEAD
  • Documentation built locally and checked sphinx-build -W documentation/source documentation/build

@je-cook je-cook added enhancement New feature or request codes Tasks relating to integration of external codes upstream Issues related to upstream dependencies labels Oct 11, 2024
@je-cook je-cook requested a review from a team as a code owner October 11, 2024 15:15
@je-cook je-cook marked this pull request as draft October 11, 2024 15:15
Copy link

codecov bot commented Oct 11, 2024

Codecov Report

Attention: Patch coverage is 84.18079% with 56 lines in your changes missing coverage. Please review.

Project coverage is 77.65%. Comparing base (94d9455) to head (1c8b0b0).
Report is 7 commits behind head on develop.

Files with missing lines Patch % Lines
bluemira/codes/process/_solver.py 12.50% 35 Missing ⚠️
bluemira/codes/process/model_mapping.py 74.32% 19 Missing ⚠️
bluemira/codes/process/_run.py 33.33% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3636      +/-   ##
===========================================
+ Coverage    77.46%   77.65%   +0.19%     
===========================================
  Files          246      246              
  Lines        29191    29380     +189     
===========================================
+ Hits         22612    22816     +204     
+ Misses        6579     6564      -15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@je-cook je-cook force-pushed the je-cook/process_develop branch 2 times, most recently from f4c9acc to 346a6f1 Compare October 22, 2024 12:35
@je-cook je-cook force-pushed the je-cook/process_develop branch 2 times, most recently from c349301 to f61bf02 Compare November 6, 2024 15:15
@je-cook je-cook force-pushed the je-cook/process_develop branch 2 times, most recently from 38c12bd to f635b7c Compare November 18, 2024 12:51
Copy link
Contributor

github-actions bot commented Nov 25, 2024

⚠️ Warning Report

Found 1 new warning, 1 fixed warning.

New warnings (1)

On collect

  • /home/runner/work/bluemira/process/process/io/python_fortran_dicts.py:13: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
All warnings (4)

On collect

  • /home/runner/miniconda3/envs/bluemira/lib/python3.11/site-packages/pkg_resources/__init__.py:3138: DeprecationWarning: Deprecated call to pkg_resources.declare_namespace('sphinxcontrib'). Implementing implicit namespace packages (as specified in PEP 420) is preferred to pkg_resources.declare_namespace. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
  • /home/runner/miniconda3/envs/bluemira/lib/python3.11/site-packages/pkg_resources/__init__.py:3138: DeprecationWarning: Deprecated call to pkg_resources.declare_namespace('cherab'). Implementing implicit namespace packages (as specified in PEP 420) is preferred to pkg_resources.declare_namespace. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
  • /home/runner/work/bluemira/process/process/io/python_fortran_dicts.py:13: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html

On runtest

  • /home/runner/work/bluemira/bluemira/bluemira/codes/process/api.py:150: DeprecationWarning: path is deprecated. Use files() instead. Refer to https://importlib-resources.readthedocs.io/en/latest/using.html#migrating-from-legacy for migration advice.

@je-cook je-cook force-pushed the je-cook/process_develop branch 2 times, most recently from 577b1c2 to 8b60a2e Compare December 2, 2024 11:59
@je-cook je-cook force-pushed the je-cook/process_develop branch from d28b570 to 259f809 Compare December 6, 2024 09:26
@je-cook je-cook force-pushed the je-cook/process_develop branch 2 times, most recently from eeada7e to 1481cee Compare January 8, 2025 09:53
@je-cook je-cook force-pushed the je-cook/process_develop branch from 559f788 to b957845 Compare January 31, 2025 08:19
@@ -48,7 +48,7 @@ jobs:
matrix:
os: ["ubuntu-latest"]
python-version: ["3.10", "3.11"]
with-process: ["v3.1.0", "process_disabled"]
with-process: ["main", "process_disabled"]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change me back

@@ -24,6 +24,7 @@ runs:
repository: "Fusion-Power-Plant-Framework/bluemira-private-data"
ssh-key: ${{ inputs.bm-private-data-key }}
path: "./bluemira-private-data"
ref: je-cook/process_develop
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove me

@je-cook je-cook force-pushed the je-cook/process_develop branch 5 times, most recently from b3f74bc to 89adf0d Compare February 10, 2025 09:40
Copy link

@je-cook je-cook force-pushed the je-cook/process_develop branch from a3271ad to b0fe7af Compare May 28, 2025 09:44
@je-cook je-cook force-pushed the je-cook/process_develop branch from b0fe7af to 1d73992 Compare May 29, 2025 10:21
@je-cook je-cook force-pushed the je-cook/process_develop branch from ff75156 to 6a86f11 Compare June 5, 2025 15:54
@je-cook je-cook force-pushed the je-cook/process_develop branch 2 times, most recently from bc45d82 to 1a737bc Compare June 24, 2025 07:48
@je-cook je-cook force-pushed the je-cook/process_develop branch 2 times, most recently from dc70504 to 722ccf6 Compare June 30, 2025 14:59
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
codes Tasks relating to integration of external codes enhancement New feature or request upstream Issues related to upstream dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

q is not properly put in the MFILE output
4 participants