Page MenuHomeElementl

[crag] Job description

Authored by alangenfeld on Jul 19 2021, 2:49 PM.
Referenced Files
Unknown Object (File)
Sat, Jun 25, 9:45 AM
Unknown Object (File)
Wed, Jun 22, 9:38 AM
Unknown Object (File)
Tue, Jun 21, 4:26 PM
Unknown Object (File)
Sat, Jun 18, 7:05 AM
Unknown Object (File)
Fri, Jun 17, 5:41 PM
Unknown Object (File)
Tue, Jun 14, 3:30 AM
Unknown Object (File)
Sun, Jun 5, 3:41 AM

Diff Detail

R1 dagster
Lint Not Applicable
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Jul 19 2021, 3:30 PM
Harbormaster failed remote builds in B33843: Diff 41799!
alangenfeld added inline comments.

^ looking for feedback here

Do you have thoughts on what the dagit experience should be like? I.e. where does graph description vs. job description show up?

Im not sure. I think the graph page having the graph desc is clear.

I could see the job page showing both (hiding when None) which would imply not inheriting

I could see the job page just showing just job desc which would support inheriting.

Should we start here without and change it to add later based on dagit iteration?


My intuition would be that we would default to the graph's desc, and the desc arg on to_job would act as an override almost.


that behavior does align with name

I think its a reasonable approach to land this and change the defaulting-from-graph behavior later if we decide its the right thing to do

This revision is now accepted and ready to land.Jul 22 2021, 3:47 PM
This revision was automatically updated to reflect the committed changes.