Skip to content

Conversation

@pierre-eliep-met
Copy link
Collaborator

@pierre-eliep-met pierre-eliep-met commented Sep 23, 2022

Signed-off-by: pepmts pierre-elie.personnaz@metroscope.tech

Goal

Set PositiveMassFlowRate min as 0, and NegativeMassFlowRate max to 0, to fix #282. It was not necessary to add this protection for 0 flows, since it is not used by simulation, only by compilation, and only used to show annoying and not relevant warning.

Type of change

  • Bugfix (non-breaking change which fixes a bug/issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change which reorganizes code)
  • Release & Version Update (when cumulative changes justify a release)
  • Documentation Update

Checklist

You can also fill these out after creating the PR, but make sure to check them all before submitting your PR for review.

GitHub checks:

  • I have added the appropriate tags, reviewers, projects and linked issues to this PR
  • I have checked for conflicts with target branch, and merged/rebased in consequence

Model development checks:

  • I have performed a self-review of my own code
  • I have checked that my changes do not bring any retrocompatibility issue comparing to MML3-DTG-v3 version
  • Existing tests pass (including Metroscopia !).
  • I have added/updated tests that prove my development works and does not break anything.

Library checks:

  • I have checked in the file explorer that I am not adding duplicate file/folder/package (due to renaming or moving file/package)
  • I have checked that the packages I upload are stored as separate files

Documentation checks:

  • I have made corresponding changes or additions to the documentation
  • I have added corresponding entries to the Changelog

Signed-off-by: pepmts <pierre-elie.personnaz@metroscope.tech>
@pierre-eliep-met pierre-eliep-met added the 🔢 Declaration Changes related to unit/start/nominal/min/max label Sep 23, 2022
@pierre-eliep-met pierre-eliep-met added this to the MML3 Publication milestone Sep 23, 2022
@pierre-eliep-met pierre-eliep-met self-assigned this Sep 23, 2022
@pierre-eliep-met pierre-eliep-met linked an issue Sep 23, 2022 that may be closed by this pull request
4 tasks
@pierre-eliep-met pierre-eliep-met merged commit 471e48a into mml3-main Sep 23, 2022
@pierre-eliep-met pierre-eliep-met deleted the fix-dp_sensor_warning branch September 23, 2022 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔢 Declaration Changes related to unit/start/nominal/min/max

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] DP sensors have 0 mass flow that create lots of warnings

3 participants