Skip to content

Conversation

@pierre-eliep-met
Copy link
Collaborator

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

Goal

⚠️ This PR is a duplicate of @hvernier's modifications not yet in a PR, probably better to wait then
Simply fixes #226

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>
Signed-off-by: pepmts <pierre-elie.personnaz@metroscope.tech>
@pierre-eliep-met pierre-eliep-met added the 🆕 New component For new components added to the library label Sep 21, 2022
@pierre-eliep-met pierre-eliep-met self-assigned this Sep 21, 2022
Signed-off-by: pepmts <pierre-elie.personnaz@metroscope.tech>
@pierre-eliep-met pierre-eliep-met added 🗃️skip-changelog Changes that should be omitted from the release notes and removed 🗃️skip-changelog Changes that should be omitted from the release notes labels Sep 21, 2022
@pierre-eliep-met pierre-eliep-met marked this pull request as ready for review September 21, 2022 15:41
@pierre-eliep-met
Copy link
Collaborator Author

pierre-eliep-met commented Sep 21, 2022

⚠️ This PR is a duplicate of @hvernier's modifications not yet in a PR, probably better to wait then

@pierre-eliep-met pierre-eliep-met added this to the MML3 Publication milestone Sep 22, 2022
@pierre-eliep-met pierre-eliep-met marked this pull request as draft September 23, 2022 08:46
@pierre-eliep-met pierre-eliep-met deleted the add-fluegases-heat-loss-component branch November 28, 2022 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🆕 New component For new components added to the library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] : creation of a fluegases component for heat loss (leak?)

2 participants