Typehints

[ankona]

No description provided.

[codecov]

Codecov Report

Merging #295 (0739ee6) into develop (d4f7df8) will increase coverage by 0.08%.
The diff coverage is 96.52%.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #295      +/-   ##
===========================================
+ Coverage    87.64%   87.72%   +0.08%     
===========================================
  Files           58       59       +1     
  Lines         3319     3390      +71     
===========================================
+ Hits          2909     2974      +65     
- Misses         410      416       +6     

Impacted Files	Coverage Δ
smartsim/entity/entityList.py	92.85% <85.71%> (-7.15%)	⬇️
smartsim/database/orchestrator.py	83.33% <87.09%> (+0.12%)	⬆️
smartsim/entity/dbnode.py	66.66% <88.23%> (+0.62%)	⬆️
smartsim/entity/model.py	96.39% <90.00%> (-0.75%)	⬇️
smartsim/experiment.py	79.85% <95.00%> (+<0.01%)	⬆️
smartsim/_core/control/controller.py	84.66% <100.00%> (+0.21%)	⬆️
smartsim/_core/control/job.py	93.15% <100.00%> (+0.09%)	⬆️
smartsim/_core/control/manifest.py	94.82% <100.00%> (+0.04%)	⬆️
smartsim/_core/generation/generator.py	93.75% <100.00%> (+0.20%)	⬆️
smartsim/_core/launcher/colocated.py	93.25% <100.00%> (+0.15%)	⬆️
... and 24 more

[billschereriii]

A few small questions, but the vast majority of this PR looks really good. Thanks for all the hard work!

[billschereriii]

LGTM. Thanks for all your hard work on this one.
Btw, I really like the recent change of defining type alias names for come of the more complex types. We should replicate that elsewhere as it makes sense to do so to take advantage of the readability gains it provides