up
Today
Today
cdecarolis updated the diff for D9085: [memoization 8/n] make default fs io manager memoizable.
Yesterday
Yesterday
sandyryza committed R1:5b279196a578: add composite solids to graph/job/op migration guide (authored by sandyryza).
add composite solids to graph/job/op migration guide
Harbormaster failed remote builds in B34608: Diff 42806 for D9085: [memoization 8/n] make default fs io manager memoizable!
cdecarolis updated the diff for D9085: [memoization 8/n] make default fs io manager memoizable.
Properly fail if attempting to use mapping keys with versioning.
GitHub <noreply@github.com> committed R1:58ed1e7a9e92: Add definition time error for invalid Mode Definition resource keys (#4445) (authored by Claire Lin <clairelin135@gmail.com>).
Add definition time error for invalid Mode Definition resource keys (#4445)
sandyryza added a comment to D9195: add composite solids to graph/job/op migration guide.
@alangenfeld good point - I updated to give the graphs inputs and outputs
sandyryza added inline comments to D9195: add composite solids to graph/job/op migration guide.
sandyryza updated the diff for D9195: add composite solids to graph/job/op migration guide.
up
Chris DeCarolis <cdguitar817@gmail.com> committed R1:b4ddfb97ba7b: [crag] Add validation step to default config buildup for to_job. (authored by Chris DeCarolis <cdguitar817@gmail.com>).
[crag] Add validation step to default config buildup for to_job.
Chris DeCarolis <cdguitar817@gmail.com> committed R1:97545bbf8f38: [crag] Ignore executor config when using execute_in_process (authored by Chris DeCarolis <cdguitar817@gmail.com>).
[crag] Ignore executor config when using execute_in_process
Harbormaster failed remote builds in B34604: Diff 42800 for D9195: add composite solids to graph/job/op migration guide!
cdecarolis requested review of D9204: [memoization 6/n] Add validation to version strings for memoization.
cdecarolis added inline comments to D9189: [crag] Ignore executor config when using execute_in_process.
sandyryza updated the diff for D9195: add composite solids to graph/job/op migration guide.
up
sandyryza committed R1:4c68172cfcd0: [demo pipelines] remove slack on success hook (authored by sandyryza).
[demo pipelines] remove slack on success hook
dish committed R1:fc435395a8c6: [dagit] Lazy-load partition set info for Schedules (authored by dish).
[dagit] Lazy-load partition set info for Schedules
totally agree with getting rid of this -- it'll also make the dogfooding alert channel much less annoying :)
sandyryza requested review of D9205: [demo pipelines] remove slack on success hook.
dish updated the diff for D9202: [dagit] Lazy-load partition set info for Schedules.
Show coverage
alangenfeld added a comment to D9202: [dagit] Lazy-load partition set info for Schedules.
I think Bens "Show Coverage" suggestion is better the "load" ones
dish added a comment to D9202: [dagit] Lazy-load partition set info for Schedules.
dish updated the diff for D9202: [dagit] Lazy-load partition set info for Schedules.
+alangenfeld
sandyryza added a comment to D9203: WIP: Dockerfile/deployment instructions for deploying hacker-news to k8s.
Thanks a ton for putting this together @dgibson!
Thanks for following up on this one! Does this mean that unlinked inputs would generally make sense to display in an accent color now? (re: https://github.com/dagster-io/dagster/issues/4407)
bengotow added a comment to D9188: [caprisun] use @asset in the story recommender pipeline.
This LGTM! Excited to give this a spin
sandyryza committed R1:56cb2f549956: [crag] add GraphDefinition to the public API (authored by sandyryza).
[crag] add GraphDefinition to the public API
The JS changes here look great. I'd +1 showing the partition set name all the time with a link, and then just offering to "Show Coverage" or something like that to reveal "Missing", etc.
alangenfeld added a comment to D9202: [dagit] Lazy-load partition set info for Schedules.
we can build and show the link to the partitions page with [1] right?
alangenfeld requested changes to D9195: add composite solids to graph/job/op migration guide.
I think its very valuable for this example to include input & output mapping
cdecarolis retitled D9085: [memoization 8/n] make default fs io manager memoizable from [memoization 6/n] make default fs io manager memoizable to [memoization 8/n] make default fs io manager memoizable.
cdecarolis added inline comments to D9085: [memoization 8/n] make default fs io manager memoizable.