Skip to content

update serialization#1798

Merged
EvenSol merged 14 commits intomasterfrom
serialization-issues
Jan 2, 2026
Merged

update serialization#1798
EvenSol merged 14 commits intomasterfrom
serialization-issues

Conversation

@EvenSol
Copy link
Copy Markdown
Collaborator

@EvenSol EvenSol commented Jan 1, 2026

Pull Request

Thank you for contributing to NeqSim! Please complete the following checklist before requesting review:

  • Confirm ./mvnw test runs successfully
  • Verify code formatting using Checkstyle
  • Update any relevant docs/README
  • Link to an associated issue or discussion

See CONTRIBUTING.md for the full contribution process.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 1, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 58.12274% with 348 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.40%. Comparing base (108fb8e) to head (294ba42).

Files with missing lines Patch % Lines
...java/neqsim/process/processmodel/ProcessModel.java 54.05% 45 Missing and 6 partials ⚠️
...n/java/neqsim/process/equipment/util/Adjuster.java 35.08% 22 Missing and 15 partials ⚠️
...thermodynamicoperations/flashops/VUflashQfunc.java 0.00% 32 Missing ⚠️
...ermodynamicoperations/ThermodynamicOperations.java 23.07% 30 Missing ⚠️
...va/neqsim/process/processmodel/DexpiXmlWriter.java 0.00% 23 Missing ⚠️
...qsim/thermodynamicoperations/flashops/TUflash.java 65.51% 13 Missing and 7 partials ⚠️
...qsim/thermodynamicoperations/flashops/THflash.java 66.07% 12 Missing and 7 partials ⚠️
...qsim/thermodynamicoperations/flashops/PVflash.java 68.00% 9 Missing and 7 partials ⚠️
...ava/neqsim/process/processmodel/ProcessSystem.java 68.75% 7 Missing and 8 partials ⚠️
...qsim/thermodynamicoperations/flashops/TSFlash.java 54.54% 11 Missing and 4 partials ⚠️
... and 23 more
Additional details and impacted files 
@@             Coverage Diff              @@
##             master    #1798      +/-   ##
============================================
+ Coverage     51.16%   51.40%   +0.24%     
- Complexity    16119    16265     +146     
============================================
  Files          1216     1223       +7     
  Lines        138638   139317     +679     
  Branches      17790    17908     +118     
============================================
+ Hits          70931    71619     +688     
+ Misses        61932    61856      -76     
- Partials       5775     5842      +67

☔ 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 c8f29bd into master Jan 2, 2026
12 checks passed
@EvenSol EvenSol deleted the serialization-issues branch January 2, 2026 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@EvenSol @codecov-commenter