You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There are, at present, a number of places in the codebase where the transforms field of Compose is operated on by code external to Compose. This is suboptimal for the following reasons:
Code duplication
The increasing complexity of executing a list of transforms. In particular, changes being made for 4855 lazy resampling impl -- Compose #5860 must be accurately reflected across everywhere that runs transform execution outside of compose.
The proposed changes do the following:
Add a class method execute that can execute a sequence of transforms with all of the currently identified flags / settings:
a. map_items
a. unpack_items
a. log_stats
Add new parameter to call that are also supported by execute
a. start
a. end
a. threading
Refactor the places where the internals of Compose.call are currently replicated to use call with start, end, threading