IMMM-SFA / tell

A model to predict Total ELectricity Loads (TELL)
https://immm-sfa.github.io/tell/
BSD 2-Clause "Simplified" License
26 stars 10 forks source link

Merge updates into v1.1 #46

Closed cdburley closed 1 year ago

cdburley commented 1 year ago

Major changes:

Requests for Casey M.:

codecov[bot] commented 1 year ago

Codecov Report

Merging #46 (89217d6) into main (0842c90) will decrease coverage by 0.48%. Report is 11 commits behind head on main. The diff coverage is 41.00%.

@@            Coverage Diff             @@
##             main      #46      +/-   ##
==========================================
- Coverage   48.25%   47.78%   -0.48%     
==========================================
  Files          31       31              
  Lines        1494     1511      +17     
==========================================
+ Hits          721      722       +1     
- Misses        773      789      +16     
Files Changed Coverage Δ
setup.py 0.00% <ø> (ø)
tell/install_forcing_data.py 46.42% <ø> (ø)
tell/install_quickstarter_data.py 46.42% <ø> (ø)
tell/install_raw_data.py 46.42% <ø> (ø)
tell/visualization.py 6.87% <35.71%> (ø)
tell/execute_forward.py 21.12% <41.17%> (-1.33%) :arrow_down:
tell/__init__.py 100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

mcgrathc commented 1 year ago

I have updated the version to v1.1.0 in all the right spots and fixed the build failure. The notebooks look good expect for the tell_runs_exp_group_b.ipynb notebook where you reference your own personal directories. Is this notebook something you are looking to offer to the broader community? For the API references for execute_forward.py I updated the doc strings. Those will update with the github actions Travis built after the push to main but I don't expect any issues after taking a look at it.

cdburley commented 1 year ago

Thanks @mcgrathc. I will move the tell_runs_exp_group_b.ipynb notebook to a different repo since it wasn't meant to be a part of the core TELL package.