Add progress callback so octopus can provide more updates on autoconversion progress
Link to ticket
Solution
Added an optional arguments for a callback function and the number of updates to send to each of the converters. Also added tests.
Type of change
New feature (non-breaking change which adds functionality)
This change requires updated or new tests
Open Questions:
I had difficulties testing for the cellpack converter. I was trying to use the cellpack data from the Jupiter notebook example, since it has multiple ingredients, but I couldn't actually get that example to run without throwing an error. I edited the files down to not include the "snake" ingredient, because the error was thrown in CellpackConverter._unpack_curve(), but that's silly I should go in and figure out what's actually going on here.
Problem
Add progress callback so octopus can provide more updates on autoconversion progress Link to ticket
Solution
Added an optional arguments for a callback function and the number of updates to send to each of the converters. Also added tests.
Type of change
Open Questions:
I had difficulties testing for the cellpack converter. I was trying to use the cellpack data from the Jupiter notebook example, since it has multiple ingredients, but I couldn't actually get that example to run without throwing an error. I edited the files down to not include the "snake" ingredient, because the error was thrown in
CellpackConverter._unpack_curve()
, but that's silly I should go in and figure out what's actually going on here.