Realized IceModel had better call PISMHydrology::max_timestep() or else it won't run.  Wrote a helper macro to avoid code duplication when updating time step from submodel's restrictions.