ecmwf / ecflow

ECMWF's workflow manager
Apache License 2.0
41 stars 15 forks source link

Improve performance of Defs formatting ECFLOW-1928 #54

Closed marcosbento closed 1 year ago

marcosbento commented 1 year ago

Printing N individual space characters to indent Defs contents has very poor performance. Avoiding the loop, by creating a N-length string of space characters, and then performing a single output operation is much quicker.

codecov-commenter commented 1 year ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (80ff9b9) 83.54% compared to head (5e71726) 83.53%. Report is 7 commits behind head on develop.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #54 +/- ## =========================================== - Coverage 83.54% 83.53% -0.02% =========================================== Files 542 542 Lines 52526 52521 -5 =========================================== - Hits 43884 43871 -13 - Misses 8642 8650 +8 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.