Closed EmmaRenauld closed 7 months ago
Attention: Patch coverage is 73.39450%
with 29 lines
in your changes are missing coverage. Please review.
Project coverage is 69.46%. Comparing base (
d319ad2
) to head (c32176a
).
Hello @EmmaRenauld, Thank you for updating !
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
Finally I added a bit more changes!
Overall, I thus have mainly small changes, and 4 new functions (created without changing any line of code!). I was not always sure where to put them though.
Quick description
Part 2 in my meticulous verification of reconst scripts. Read the 9 next scripts in our excel list!
Almost all already correct! Changes a few small code style along the way. The only real changes:
args
. Will be easier to create a script.That's it! ...
Type of change
Check the relevant options.
Provide data, screenshots, command line to test (if relevant)
...
Checklist