Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add mvhr and mev #145

Merged
merged 1 commit into from
Dec 21, 2022
Merged

add mvhr and mev #145

merged 1 commit into from
Dec 21, 2022

Conversation

jgunstone
Copy link
Collaborator

pull request for adding MVHR and MEV.

I added MVHR to the "air handling" section, but added MEV to the "fan" section as this felt most appropriate.

Note that the rows are approx. ordered by asset_description but not quite.
Please edit as you see fit.

many thanks

@jgunstone
Copy link
Collaborator Author

#144

@pisuke pisuke self-requested a review December 21, 2022 16:10
Copy link
Collaborator

@pisuke pisuke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great submission, thank you!
LGTM

@RitaLav RitaLav mentioned this pull request Dec 21, 2022
@RitaLav RitaLav merged commit 2e1e220 into theodi:master Dec 21, 2022
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.

3 participants