-
Notifications
You must be signed in to change notification settings - Fork 389
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 support of umol/m2-s in IDF Editor for IndoorLivingWall support #10414
Conversation
@@ -22727,9 +22726,9 @@ IndoorLivingWall, | |||
N2 , \field LED Nominal Intensity | |||
\note The value represents photosynthetic photon flux density (PPFD) of LED grow light. | |||
\note PPFD is measured in micro-mole per m2 per second (umol_m2s) which establishes exactly how many photosynthetically active radiation (PAR) photons are landing on a specific area. | |||
\units umol_m2s | |||
\units umol/m2-s |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think is purely an editorial change since \units I think is only used by IDF Edtior
\type real | ||
\ip-units umol_m2s | ||
\ip-units umol/ft2-s |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change to using / instead of _ and using ft2 for the \ip-units should only impact IDF Editor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good here.
VersionCompanyName="GARD Analytics" | ||
VersionLegalCopyright="(c) 2000-2017 GARD Analytics" | ||
VersionLegalCopyright="(c) 2000-2024 GARD Analytics" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😮
Is this still a draft @JasonGlazer ? I trust that you executed IDF Editor to make sure it was happy... |
Because of the minuscule change to the IDD, I wanted to let CI run and clear everything. It is ready to go. |
👍 thanks @JasonGlazer, merging. |
Pull request overview
see #10385 (comment)
Pull Request Author
Add to this list or remove from it as applicable. This is a simple templated set of guidelines.
Reviewer
This will not be exhaustively relevant to every PR.