Page MenuHomeElementl

[dagit] Bucket materializations in asset grid
ClosedPublic

Authored by dish on Apr 19 2021, 6:31 PM.

Details

Summary

Resolves #3951.

A followup to https://dagster.phacility.com/D7487, which used materialization bucketing on the list view. This just does the same on the grid view.

Test Plan

View assets that have previous materializations for certain partitions, and assets that do not. Verify that the extra row appears at appropriate times in both grid and list views.

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dish requested review of this revision.Apr 19 2021, 6:40 PM

Similar to with the list view, it could be clearer what's going on here if we make it clear that the partition is the index, e.g. by making it the top row.

Move bucketing up a level. Make partition name the first row in matrix. Repair sorting by partition for matrix and graph.

This revision is now accepted and ready to land.Apr 19 2021, 9:41 PM
This revision was automatically updated to reflect the committed changes.