Skip to content

Fix MCV, PCV (#260)#287

Open
Nicsda wants to merge 5 commits intoDLR-SR:mainfrom
Nicsda:main
Open

Fix MCV, PCV (#260)#287
Nicsda wants to merge 5 commits intoDLR-SR:mainfrom
Nicsda:main

Conversation

@Nicsda
Copy link

@Nicsda Nicsda commented Feb 16, 2026

Solved the issue nr. 260 with all suggested changes plus added the same indicator as for the MCV in the PCV as well. Updated each test case as well, because they were very messy, therefore the regression testing yielded 'no validation' for both of them. However, the test HeatExchangers>Tests>Recuperator was validated including at least the PCV.

@Nicsda Nicsda added enhancement New feature or request layer::icon Improvements or additions to model/class icon p::FlowControl Concerns package FlowControl or Undirected.FlowControl labels Feb 16, 2026
@RaphaelGebhart RaphaelGebhart assigned Nicsda and unassigned Nicsda Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request layer::icon Improvements or additions to model/class icon p::FlowControl Concerns package FlowControl or Undirected.FlowControl

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant