Notebook rework for smoke tests

matthewcarbone commented 12 months ago

Resolves #53.

codecov-commenter commented 12 months ago

Codecov Report

Merging #60 (80853eb) into main (83c0113) will increase coverage by 0.02%. The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
+ Coverage   95.98%   96.01%   +0.02%     
==========================================
  Files          44       44              
  Lines        3541     3541              
==========================================
+ Hits         3399     3400       +1     
+ Misses        142      141       -1

Flag	Coverage Δ
unittests	`96.01% <ø> (+0.02%)`	:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes

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

matthewcarbone commented 12 months ago

Handled about half of the notebooks. Should decrease the testing time significantly but I could be wrong since I'm running on different hardware locally. Let's see how long it takes now.

matthewcarbone commented 12 months ago

@ziatdinovmax great! Take a quick look at them too when you can too. I did make some QOL changes, made the plots a bit clearer, etc.

ziatdinovmax / gpax

Notebook rework for smoke tests #60

Codecov Report