Page MenuHomePhabricator

Open Tasks

  • No tasks found.

Active Repositories

Recent Activity

Today

sashank requested review of D6138: (docs-infra-2/n) Write script to pack API docs data.
Mon, Jan 25, 8:42 AM
sashank published D6137: (docs-infra-1/n) Clean up docs directory and move sphinx to subdirectory for review.
Mon, Jan 25, 8:26 AM
yuhan updated subscribers of D5949: Add an example of how to trigger a pipeline by graphQL.

Are we exposing or documenting stuff under dagster.core.host_representation like ManagedGrpcPythonEnvRepositoryLocationOrigin, LoadableTargetOrigin, and PipelineSelector, which have been mentioned in this example?
Should we? Without linking to detailed API docs or further explanation, it could be a bit hard for users to digest what's going on when using these suggested APIs. cc @dgibson: as you may have more context in this corner of the codebase.

Mon, Jan 25, 5:51 AM
yuhan added inline comments to D6048: [Memoized Dev Example 2/2] Memoized development example.
Mon, Jan 25, 4:31 AM
dgibson updated the diff for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.

up

Mon, Jan 25, 2:41 AM
dgibson updated the diff for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.

up

Mon, Jan 25, 1:41 AM

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