Refactor smart cli into subparsers

[ankona]

Separate creation of the sub-command parsers from parser execution and decouple CLI definition & instantiation.

[codecov]

Codecov Report

Merging #308 (9f2b76c) into develop (a9018e0) will decrease coverage by 0.18%.
The diff coverage is n/a.

❗ Current head 9f2b76c differs from pull request most recent head 8cdcf84. Consider uploading reports for the commit 8cdcf84 to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #308      +/-   ##
===========================================
- Coverage    87.12%   86.95%   -0.18%     
===========================================
  Files           59       59              
  Lines         3518     3518              
===========================================
- Hits          3065     3059       -6     
- Misses         453      459       +6     

see 1 file with indirect coverage changes

[MattToast]

Looks amazing! All of the help statements are now correct, and I love the clean up to remove the (imo unnecessary) classes and shift this to a more procedural/functional style! It should make testing much easier!!

I found a couple small knit picks, the most major being that there seems to be a type error being raised when installing for GPU. Otherwise, all other functionality seems to be working as I would expect it to!!

Thanks for the hard work on this one!!

[MattToast]

LGTM!! Thanks for this refactor, this should be much nicer to work with!!