Page MenuHomeElementl

[dagit] Fix Gantt chart scale

Authored by dish on Feb 5 2021, 8:41 PM.



Resolves #3654.

Use the startedPipelineAt timing for the Gantt chart, to appropriately scale it.

Test Plan

View previous runs, verify that the Gantt looks correct.

Diff Detail

R1 dagster
Lint Not Applicable
Tests Not Applicable

Event Timeline

Also launched a new run, and once it's dequeued, the Gantt looks correct in real time.

dish requested review of this revision.Feb 5 2021, 8:47 PM

Yep I think this will do it ๐Ÿ‘ I think this must have become a problem when we started emitting logs during the "queued" phase and putting a big gap between firstLogAt and startedPipelineAt.

This revision is now accepted and ready to land.Feb 5 2021, 8:50 PM
This revision was automatically updated to reflect the committed changes.