CenterForTheBuiltEnvironment / pythermalcomfort

Package to calculate several thermal comfort indices (e.g. PMV, PPD, SET, adaptive) and convert physical variables.
https://pythermalcomfort.readthedocs.io/en/latest/
MIT License
144 stars 52 forks source link

First integration of JOS3 model and pythermalcomfort #44

Closed AkihisaNomoto closed 1 year ago

AkihisaNomoto commented 1 year ago

Hello!

This is the first integration of JOS3 model and pythermalcomfort. The JOS-3 model code is basically copied from here (https://github.com/TanabeLab/JOS-3) but I newly added some description about the model and comments to the code for readers to easily understand.

Main changes are as follows;

This is just an initial commit and I will modify the code for JOS3 model as needed.

Thanks, Aki

codecov-commenter commented 1 year ago

Codecov Report

Merging #44 (eb5a80c) into master (645fff6) will decrease coverage by 20.28%. The diff coverage is 17.18%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff             @@
##           master      #44       +/-   ##
===========================================
- Coverage   67.68%   47.41%   -20.28%     
===========================================
  Files          10       14        +4     
  Lines        1597     2668     +1071     
  Branches      257      404      +147     
===========================================
+ Hits         1081     1265      +184     
- Misses        465     1348      +883     
- Partials       51       55        +4     
Impacted Files Coverage Δ
src/pythermalcomfort/shared_functions.py 100.00% <ø> (ø)
...rc/pythermalcomfort/jos3_functions/construction.py 9.44% <9.44%> (ø)
...ythermalcomfort/jos3_functions/thermoregulation.py 11.39% <11.39%> (ø)
src/pythermalcomfort/utilities.py 69.81% <14.28%> (-2.56%) :arrow_down:
src/pythermalcomfort/models.py 58.06% <20.39%> (-30.34%) :arrow_down:
src/pythermalcomfort/jos3_functions/parameters.py 21.05% <21.05%> (ø)
src/pythermalcomfort/jos3_functions/matrix.py 22.56% <22.56%> (ø)
AkihisaNomoto commented 1 year ago

Hi,

Thanks for all your commnets.

Changes are as follows as we discussed yesterday;

New chnages since yesterday is;

Pelase feel free to edit as you like.

Thank you again for adding me to pythermalcomfort co-maintainers. I am happy to internaly work on developing this model.

Akihisa

AkihisaNomoto commented 1 year ago

One more thing - I added a function to get the current version of pythermalcomfort from __init__.py and print it in the output of JOS-3 model to let the users know what version they are using. I think this is important because JOS-3 model can be further developed.

AkihisaNomoto commented 1 year ago

Sorry that I messed up a couple of lines of code. I think I could modify it and now all set.

AkihisaNomoto commented 1 year ago

Thanks for your comment. I modified the code based on your feedback.