Experiment Preview

[juliaputko]

No description provided.

[codecov]

Codecov Report

Attention: 24 lines in your changes are missing coverage. Please review.

Comparison is base (bbce88a) 90.01% compared to head (ca84b72) 90.32%.

❗ Current head ca84b72 differs from pull request most recent head 7d48a1e. Consider uploading reports for the commit 7d48a1e to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           preview     #447      +/-   ##
===========================================
+ Coverage    90.01%   90.32%   +0.31%     
===========================================
  Files           60       61       +1     
  Lines         3826     3866      +40     
===========================================
+ Hits          3444     3492      +48     
+ Misses         382      374       -8     

Files	Coverage Δ
smartsim/_core/__init__.py	100.00% <100.00%> (ø)
smartsim/error/errors.py	100.00% <100.00%> (ø)
smartsim/_core/control/manifest.py	95.91% <66.66%> (-0.61%)	⬇️
smartsim/experiment.py	82.71% <40.00%> (-2.91%)	⬇️
smartsim/_core/control/previewrenderer.py	50.00% <50.00%> (ø)

... and 8 files with indirect coverage changes

[al-rigazzi]

Good starting point, I agree with @ankona's suggestion to use static methods. @MattToast may want to chime in as well.

[ankona]

Do you really want a None here? would "%(message)s" do the job?

I don't even know what happens if you have a null formatter!