if a strat eval reverts in take orders, don't revert the whole take orders, instead skip the order

[thedavidmeister]

this one is a bit subtle

consider the case where calculate io does not error, but handle io does error

at this point we've already called store for the kvs from calculate io!

if we don't rollback the transaction we will corrupt the interpreter storage for these orders

otoh we want to gracefully drop orders as much as possible in the loop rather than have an error prevent us taking potentially viable orders further down the list

potentially we have to revert on handle io reverts, but can gracefully drop calculate io reverts (and not update storage!)

[thedavidmeister]

one potential way forward is to simply accept that reverts in handle io are hard reverts, and to treat reverts in calculate io as though they were amount io-ratio: 0 decimal18-max-value(); and ignore any writes