Page MenuHomeElementl

[dagit] Fix Gantt chart scale
ClosedPublic

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

Details

Summary

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

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
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.