svgutils 0.3.2 breaks compose_view (possibly others)

[effigies]

MRIQC release 0.16.0 was held up by a CI failure:

  File "/usr/local/miniconda/lib/python3.7/site-packages/niworkflows/viz/utils.py", line 360, in compose_view
    fig = SVGFigure(width, heights[:nsvgs].sum())
  File "/usr/local/miniconda/lib/python3.7/site-packages/svgutils/transform.py", line 242, in __init__
    self._width = width.value
AttributeError: 'numpy.int64' object has no attribute 'value'

Opened an issue at btel/svg_utils#57 to see what the correct fix is, but the short term fix is to pin svgutils 0.3.1.

[effigies]

Looks like the fix for 0.3.2 is to use Unit objects provided by svgutils: https://github.com/btel/svg_utils/pull/58/files#diff-e8aa6d811244eff36276b6f2042973e5b5b30db429f997288e45bf0ce5be717dR55

[mgxd]

After the fix btel/svg_utils#58, there is still a piece that needs to be updated within our code:

The FigureElement.moveto scale parameter has been extracted into scale_x and scale_y parameters. I think we should be fine just adjusting scale_x and leaving scale_y as None.

EDIT: relevant line

niworkflows/niworkflows/viz/utils.py

Line 396 in b631029

r.moveto(0, yoffset, scale=scales[i])

[effigies]

Agree, though we'll then need to depend on >=0.3.3 or try/except to allow a wider range of versions. Frustrating that they broke their API on a patch release.

[oesteban]

We hit this problem in dMRIPrep already (nipreps/dmriprep#144 (comment)), I don't know what @slimnsour did, whether he reported this or not.

EDIT: I see you guys have all figured out :D