Fix outdated string based path concatenation in verify_step

[iksnagreb]

No description provided.

[github-actions]

📋 Docstring Check Report

❌ Docstring check failed!

Missing Docstrings Details:

📄 src/finn/builder/build_dataflow_steps.py:

• Line 368: function 'prepare_loop_ops_fifo_sizing'
• Line 403: function 'prepare_loop_ops_ipgen'
• Line 508: function 'apply_if_relevant'
• Line 1228: function 'verify_mlo'

Total missing docstrings: 4

How to Fix:

Please add docstrings to the missing functions, classes, and modules listed above.

Docstring Guidelines:

• All modules should have a module-level docstring
• All public functions and methods should have docstrings
• All private functions should have docstrings
• All classes should have docstrings
• Use triple quotes (""") for docstrings
• Follow PEP 257 conventions

Raw output from docstring checker

Checking 1 specified Python file(s) (excluding tests folder):
  - src/finn/builder/build_dataflow_steps.py

❌ Missing docstrings found:

📄 src/finn/builder/build_dataflow_steps.py:
  - Line 368: function 'prepare_loop_ops_fifo_sizing'
  - Line 403: function 'prepare_loop_ops_ipgen'
  - Line 508: function 'apply_if_relevant'
  - Line 1228: function 'verify_mlo'

Total missing docstrings: 4

[bwintermann]

Could fail because output_dir is str | Path. For now I'd either wrap everything in Path(...) or remove the str type.