Add compatibility with new pclayer release

Upcoming pclayer package has a few minor visible changes for us:

• DataDispatcher.trainStride becomes pipeline.dispatcher.trainStride
• DataAggregator is now DataAggregatorXtdf for XTDF inputs

@hammerd @silenzi @haufs

Nice!

This will not be backwards compatible right?

It's not, but there is no need. We aim to keep up to date with whatever shiny new features you throw at us.

Thanks for the vote of confidence!

If we need to pull the parachute I guess it is easy to revert

Low grade - LGTM

Changes LGTM. I agree that we needn't keep backwards compatibility. Or rather, rolling back a deployed calng version in case we have to is very easy, so adding complexity for compatibility doesn't seem worth it. Is new pcLayer release expected to be running for next run?

if by expected you mean "we will give our best shot so that it does", yes

does this change create the right widget in the scene?

Well spotted, thanks!

Gerne!

added 1 commit

• 8ebdbe08 - (fixup) Fix trainStride property name on manager scenes

Compare with previous version

changed milestone to %2023 winter shutdown

mentioned in merge request !85 (closed)

marked this merge request as ready

merged

mentioned in commit 18a03cb6