Since #1124, we use parent assets for dispatch rather than their children. It seems we forgot to update the debug_dispatch_assets.csv file format at the same time though -- it doesn't have a column for group ID. Technically you could probably distinguish the assets in some other way (dispatch can also include candidates, which don't have an asset ID either), but this would make it clearer.
I've also just noticed that the struct we use for serialising data for this file is called ActivityRow, reflecting the old name of the file. There's also a write_activity method on DebugDataWriter. We should rename these to something more obvious and update any comments.
Since #1124, we use parent assets for dispatch rather than their children. It seems we forgot to update the
debug_dispatch_assets.csvfile format at the same time though -- it doesn't have a column for group ID. Technically you could probably distinguish the assets in some other way (dispatch can also include candidates, which don't have an asset ID either), but this would make it clearer.I've also just noticed that the
structwe use for serialising data for this file is calledActivityRow, reflecting the old name of the file. There's also awrite_activitymethod onDebugDataWriter. We should rename these to something more obvious and update any comments.