Skip to content

Remove some dicts#4387

Open
timothy-nunn wants to merge 4 commits into
ukaea:mainfrom
timothy-nunn:remove-dicts
Open

Remove some dicts#4387
timothy-nunn wants to merge 4 commits into
ukaea:mainfrom
timothy-nunn:remove-dicts

Conversation

@timothy-nunn

Copy link
Copy Markdown
Collaborator

Removes some of the dicts that are not too intertwined with bits of PROCESS. The remaining dicts should probably wait until we replace the input/output format because they cannot be easily substituted.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 17.64706% with 42 lines in your changes missing coverage. Please review.
✅ Project coverage is 48.81%. Comparing base (ddcba6c) to head (81deab0).

Files with missing lines Patch % Lines
process/core/io/vary_run/tools.py 8.33% 33 Missing ⚠️
process/core/input.py 12.50% 7 Missing ⚠️
process/core/io/in_dat/base.py 71.42% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4387      +/-   ##
==========================================
- Coverage   48.85%   48.81%   -0.04%     
==========================================
  Files         151      151              
  Lines       29385    29356      -29     
==========================================
- Hits        14356    14331      -25     
+ Misses      15029    15025       -4     

☔ View full report in Codecov by Harness.
📢 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.

@timothy-nunn timothy-nunn marked this pull request as ready for review June 26, 2026 10:02
@timothy-nunn timothy-nunn requested a review from a team as a code owner June 26, 2026 10:02
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.

2 participants