Page MenuHomePhabricator

Open Tasks

  • No tasks found.

Active Repositories

Recent Activity

Yesterday

dgibson abandoned D6119: [12/n] Reconstruct ExecutionPlan from the ExecutionPlanSnapshot when it is available.
Sun, Jan 24, 11:29 PM
dgibson abandoned D6095: [10/n]Make StepOutput user-code-free.

squashed into https://dagster.phacility.com/D6133

Sun, Jan 24, 11:28 PM
dgibson abandoned D6094: [9/n] Make StepInputSource use snapshots.

squashed into https://dagster.phacility.com/D6133

Sun, Jan 24, 11:28 PM
dgibson abandoned D6093: [8/n] Make StepInput.dagster_type a snapshot instead (no user code).

squashed into https://dagster.phacility.com/D6133

Sun, Jan 24, 11:28 PM
dgibson abandoned D6086: [5/n] Wait to determine step compute_fn until you're actually in step execution.

squashed into https://dagster.phacility.com/D6133

Sun, Jan 24, 11:28 PM
dgibson abandoned D6083: [3/n] Include pipeline snapshot and execution plan snapshot on system execution context.
Sun, Jan 24, 11:27 PM
dgibson abandoned D6088: [6and7/n] remove solid from ExecutionStep.

squashed into https://dagster.phacility.com/D6133

Sun, Jan 24, 11:27 PM
dgibson retitled D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists from (Squashed) Reconstruct ExecutionPlan from the ExecutionPlanSnapshot to Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.
Sun, Jan 24, 11:26 PM
dgibson updated the diff for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.

up

Sun, Jan 24, 11:21 PM
dgibson updated the diff for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.

up

Sun, Jan 24, 11:07 PM
Harbormaster failed remote builds in B24793: Diff 30192 for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists!
Sun, Jan 24, 9:45 PM
dgibson updated the diff for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.

up

Sun, Jan 24, 9:27 PM
Harbormaster failed remote builds in B24792: Diff 30191 for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists!
Sun, Jan 24, 8:30 PM
Harbormaster failed remote builds in B24791: Diff 30190 for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists!
Sun, Jan 24, 8:23 PM
dgibson updated the diff for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.

up

Sun, Jan 24, 8:13 PM
dgibson updated the diff for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.

up

Sun, Jan 24, 8:04 PM
Harbormaster failed remote builds in B24790: Diff 30189 for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists!
Sun, Jan 24, 7:22 PM
dgibson updated the diff for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.

up

Sun, Jan 24, 7:03 PM
dgibson requested review of D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.
Sun, Jan 24, 4:56 AM
catherinewu closed D6134: Increase UI sensor polling rate.
Sun, Jan 24, 1:02 AM
catherinewu committed R1:f2bd19e1b3d4: Increase UI sensor polling rate (authored by catherinewu).
Increase UI sensor polling rate
Sun, Jan 24, 1:02 AM
prha accepted D6134: Increase UI sensor polling rate.

squirtle

Sun, Jan 24, 12:46 AM
catherinewu published D6134: Increase UI sensor polling rate for review.
Sun, Jan 24, 12:45 AM
schrockn resigned from D6132: UI support for dynamic outputs, Run view state cleanup.

Very excited to see this land. Will defer to dish on the deets of the js.

Sun, Jan 24, 12:12 AM

Sat, Jan 23

schrockn added a comment to D6029: Flip asset materialization UI so newest time/partition is on the left.

adding @sandyryza here. I think this makes a lot of sense and I don't think having the events be left-to-right and the graphs right-to-left is a big problem but open to other's perspective

Sat, Jan 23, 11:58 PM
schrockn added a reviewer for D6029: Flip asset materialization UI so newest time/partition is on the left: sandyryza.
Sat, Jan 23, 11:57 PM
bengotow requested review of D6132: UI support for dynamic outputs, Run view state cleanup.
Sat, Jan 23, 11:50 PM
bengotow updated the diff for D6029: Flip asset materialization UI so newest time/partition is on the left.
  • Instead of flipping, scroll the asset materialization grid to the right by default
  • Fix React warnings in the useViewport component, some useCallback's missing
Sat, Jan 23, 10:34 PM
dgibson added a comment to D6131: Replace dagster api grpc with 'python -m dagster.grpc' in docs?.

"As initialized upon program startup, the first item of this list, path[0], is the directory containing the script that was used to invoke the Python interpreter. If the script directory is not available (e.g. if the interpreter is invoked interactively or if the script is read from standard input), path[0] is the empty string, which directs Python to search modules in the current directory first."

Sat, Jan 23, 7:22 PM
schrockn added a comment to D5115: don't error on solid type annotations that don't resolve to dagster types.

"I'm pretty resistant to directions that involve using type annotations for anything other than annotating the actual Python type that's expected somewhere. It's fairly abusive of a Python language feature, and it causes non-trivial trouble for anyone who's trying to use that language feature in the correct way."

Sat, Jan 23, 7:22 PM
dgibson added a comment to D6131: Replace dagster api grpc with 'python -m dagster.grpc' in docs?.

possibly relevant:

Sat, Jan 23, 7:21 PM
dgibson closed D6081: Remove pipeline_def from top-level SystemExecutionContext, only use in step execution contexts.
Sat, Jan 23, 7:20 PM
dgibson committed R1:ac7dec89ecca: Remove pipeline_def from top-level SystemExecutionContext, only use in step… (authored by dgibson).
Remove pipeline_def from top-level SystemExecutionContext, only use in step…
Sat, Jan 23, 7:20 PM
dgibson published D6131: Replace dagster api grpc with 'python -m dagster.grpc' in docs? for review.
Sat, Jan 23, 7:17 PM
sandyryza added a comment to D5115: don't error on solid type annotations that don't resolve to dagster types.

I like the variants you listed that involve overloading the input_defs arg on solid.  I think that kind of overloading is pretty Pythonic / expected. When reading that code, it's very difficult to misinterpret what's going on.

Sat, Jan 23, 7:07 PM
Harbormaster failed remote builds in B24770: Diff 30165 for D6119: [12/n] Reconstruct ExecutionPlan from the ExecutionPlanSnapshot when it is available!
Sat, Jan 23, 6:38 PM
dgibson updated the diff for D6119: [12/n] Reconstruct ExecutionPlan from the ExecutionPlanSnapshot when it is available.

up

Sat, Jan 23, 6:21 PM
dgibson added inline comments to D6081: Remove pipeline_def from top-level SystemExecutionContext, only use in step execution contexts.
Sat, Jan 23, 6:19 PM
dgibson updated the diff for D6081: Remove pipeline_def from top-level SystemExecutionContext, only use in step execution contexts.

up

Sat, Jan 23, 6:17 PM
dgibson requested review of D6119: [12/n] Reconstruct ExecutionPlan from the ExecutionPlanSnapshot when it is available.
Sat, Jan 23, 4:42 PM