Skip to content

Add distillation solver metrics and feed tests#1615

Merged
EvenSol merged 24 commits intoequinor:masterfrom
EvenSol:master
Nov 2, 2025
Merged

Add distillation solver metrics and feed tests#1615
EvenSol merged 24 commits intoequinor:masterfrom
EvenSol:master

Conversation

@EvenSol
Copy link
Copy Markdown
Collaborator

@EvenSol EvenSol commented Nov 1, 2025

No description provided.

…yzer-package

Add process safety analyzer and scenario support
…or-extended-limits

Add configurable mechanical design limits and validation
…icaldesign-for-sizing-strategies

Refactor safety valve design to support scenario-based sizing
…-for-safety-metrics

Add process safety reporting builder and integration tests
…n-and-capacity-methods

Add flare disposal network analysis and reporting DTOs
…est-compilation-errors

Enhance process safety analyzer scenario support
…-solver-and-suggest-improvements

Add distillation solver metrics and feed tests
@EvenSol EvenSol marked this pull request as ready for review November 1, 2025 16:00
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 1, 2025

Codecov Report

❌ Patch coverage is 74.13793% with 90 lines in your changes missing coverage. Please review.
✅ Project coverage is 47.44%. Comparing base (4e32fcb) to head (40dd747).
⚠️ Report is 655 commits behind head on master.

Files with missing lines Patch % Lines
...ava/neqsim/process/equipment/EquipmentFactory.java 35.71% 20 Missing and 7 partials ⚠️
...im/process/processmodel/DexpiRoundTripProfile.java 52.00% 12 Missing and 12 partials ⚠️
...ess/equipment/distillation/DistillationColumn.java 85.18% 16 Missing and 4 partials ⚠️
...va/neqsim/process/processmodel/DexpiXmlReader.java 86.20% 8 Missing ⚠️
...ava/neqsim/process/processmodel/DexpiMetadata.java 0.00% 6 Missing ⚠️
...va/neqsim/process/processmodel/DexpiXmlWriter.java 91.48% 3 Missing and 1 partial ⚠️
...ava/neqsim/process/processmodel/ProcessSystem.java 87.50% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1615      +/-   ##
============================================
+ Coverage     47.35%   47.44%   +0.09%     
- Complexity     9114     9157      +43     
============================================
  Files           879      881       +2     
  Lines         86204    86433     +229     
  Branches      10875    10893      +18     
============================================
+ Hits          40818    41007     +189     
- Misses        42311    42337      +26     
- Partials       3075     3089      +14     

☔ 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.

@EvenSol EvenSol merged commit c411bbf into equinor:master Nov 2, 2025
7 checks passed
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