Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Change 1: Convert "Case_Data" object to a simple dictionary. This will be much simpler than maintaining a custom object (which seems like overkill in this case), eliminates a ton of redundant code, and avoids the pitfall of accessing internal variables. In making this conversion I discovered an error in the code (which is almost inevitable with so much duplication):
w_d
was assigned the value ofw_0
. This may have implications for already-converted data.Change 2: Make "debug" logging a command-line argument rather than requiring manual editing of code
Change 3: Fix calls to
logging.critical
that had a missing required argument.Change 4: Remove old commented prints/exceptions