Cleanup MakeSFTDAG
Description
The point of this is to address #321 (closed). It cleans up the argument parsing, makes the code follow standard python conventions, and adds some comments to help better understand the code. It would be good to add a unit test as part of this merge request, so that will come along soon.
API Changes and Justification
Backwards Compatible Changes
-
This change introduces no API changes -
This change adds new API calls
Backwards Incompatible Changes
-
This change modifies an existing API -
This change removes an existing API
If any of the Backwards Incompatible check boxes are ticked please provide a justification why this change is necessary and why it needs to be done in a backwards incompatible way.
Review Status
This update has not yet been reviewed, but needs to be before merging.
Edited by Evan Goetz