Skip to content

Cleanup MakeSFTDAG

Evan Goetz requested to merge evan-goetz/lalsuite:cleanup-MakeSFTDAG into master

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

Merge request reports