@radar I recall this allowing for an order to be passed as well at some point.

@peterberkenbosch I believe the main use for calculators on shipments are for **compute_package** specially on **spree_active_shipping**, I advice care if you use the different implementations for the same thing which is calling **compute** because of the different overrides we might find already implemented, I know that this happened before when we went from **compute** to **compute_package** on **spree_active_shipping**, what's stopping us from doing ``compute_package(shipment.to_package)`` in here? that might help with this, unless I'm totally missing the point which might be the case :trollface: