Runtime value checking of public API methods

[juliaputko]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 97.14286% with 2 lines in your changes missing coverage. Please review.

Project coverage is 46.86%. Comparing base (cce16e6) to head (228e3f9).
Report is 18 commits behind head on smartsim-refactor.

Files with missing lines	Patch %	Lines
smartsim/_core/utils/helpers.py	85.71%	1 Missing ⚠️
smartsim/entity/application.py	93.33%	1 Missing ⚠️

Additional details and impacted files

@@                  Coverage Diff                  @@
##           smartsim-refactor     #707      +/-   ##
=====================================================
+ Coverage              40.45%   46.86%   +6.40%     
=====================================================
  Files                    110      107       -3     
  Lines                   7326     6512     -814     
=====================================================
+ Hits                    2964     3052      +88     
+ Misses                  4362     3460     -902     

Files with missing lines	Coverage Δ
smartsim/builders/ensemble.py	93.04% <100.00%> (ø)
smartsim/experiment.py	87.74% <100.00%> (+6.18%)	⬆️
smartsim/launchable/job.py	96.00% <100.00%> (+5.52%)	⬆️
smartsim/_core/utils/helpers.py	41.43% <85.71%> (+1.25%)	⬆️
smartsim/entity/application.py	82.35% <93.33%> (ø)

... and 8 files with indirect coverage changes

[MattToast]

A couple of small error message nits, and a couple of requests to change where exactly we look for invalid input, but otherwise looks good so far!

[MattToast]

A couple of style nits and a few minor suggestions for you to consider while you get the tests passing, but nothing big. Otherwise, this is looking about ready to go on my end!

[MattToast]

A couple of last minute inconsistencies. Feel free to lmk what you think. Otherwise, looks about ready to go!

[MattToast]

Two last minute nits, but otherwise looks ready to go on my end!