fixes #485
Now the instance names are just the step name and a counting integer. Also, the instance identifier of the steps has now been incorporated into the file names resulting from the disk operator. Also added the df_mode to now be written to the run.yaml file.
Changes
The stepfactory and steps.py file have been adjusted, and the parameters passed to the StepFactory.create() function respectively
Testing
To test:
Add a step and check that the name is correct
Change the method and check whether the name is correct
(obviously while a step of that type is already in the workflow, and also when it is a new step).
Calculate and plot stuff, and check whether the dataframes and plots
a) are written to the disk
b) will be read on restarting protzilla and continueing the workflow
PR checklist
Development
[x] If necessary, I have updated the documentation (README, docstrings, etc.)
[x] If necessary, I have created / updated tests.
Mergeability
[x] main-branch has been merged into local branch to resolve conflicts
[ ] The tests and linter have passed AFTER local merge
[x] The code has been formatted with black
Code review
[x] I have self-reviewed my code.
[ ] At least one other developer reviewed and approved the changes
Description
fixes #485 Now the instance names are just the step name and a counting integer. Also, the instance identifier of the steps has now been incorporated into the file names resulting from the disk operator. Also added the df_mode to now be written to the run.yaml file.
Changes
The stepfactory and steps.py file have been adjusted, and the parameters passed to the StepFactory.create() function respectively
Testing
To test: Add a step and check that the name is correct Change the method and check whether the name is correct (obviously while a step of that type is already in the workflow, and also when it is a new step). Calculate and plot stuff, and check whether the dataframes and plots a) are written to the disk b) will be read on restarting protzilla and continueing the workflow
PR checklist
Development
Mergeability
blackCode review