OwenKephart comments

Results 115 comments of


                                            OwenKephart

[DS][22/n] Convert SchedulingResult to DagsterModel

### Merge activity * **May 3, 5:31 PM EDT**: @OwenKephart started a stack merge that includes this pull request via [Graphite](https://app.graphite.dev/github/pr/dagster-io/dagster/21546). * **May 3, 6:20 PM EDT**: [Graphite](https://app.graphite.dev/github/pr/dagster-io/dagster/21546) rebased this...

[DS][21/n] Rename AssetConditionResult to SchedulingResult

* **#21641** : 2 dependent PRs ([#21670](https://github.com/dagster-io/dagster/pull/21670) , [#21671](https://github.com/dagster-io/dagster/pull/21671) ) * **#21640** * **#21648** * **#21615** * **#21613** * **#21612** * **#21573** * **#21546** * **#21545** 👈 * **#21541** *...

[DS][21/n] Rename AssetConditionResult to SchedulingResult

### Merge activity * **May 3, 5:31 PM EDT**: @OwenKephart started a stack merge that includes this pull request via [Graphite](https://app.graphite.dev/github/pr/dagster-io/dagster/21545). * **May 3, 6:17 PM EDT**: [Graphite](https://app.graphite.dev/github/pr/dagster-io/dagster/21545) rebased this...

[DS][1/n] Create SchedulingConditionEvaluationContext

* **#21511** : 2 dependent PRs ([#21512](https://github.com/dagster-io/dagster/pull/21512) , [#21520](https://github.com/dagster-io/dagster/pull/21520) ) * **#21510** * **#21508** * **#21507** * **#21505** * **#21504** * **#21503** * **#21502** * **#21501** * **#21500** * **#21499**...

[DS][1/n] Create SchedulingConditionEvaluationContext

### Merge activity * **May 3, 5:31 PM EDT**: @OwenKephart started a stack merge that includes this pull request via [Graphite](https://app.graphite.dev/github/pr/dagster-io/dagster/21499). * **May 3, 5:33 PM EDT**: @OwenKephart merged this...

[DS][6/n] Create Any/AllDepsMatchSchedulingConditions

* **#21511** : 2 dependent PRs ([#21512](https://github.com/dagster-io/dagster/pull/21512) , [#21520](https://github.com/dagster-io/dagster/pull/21520) ) * **#21510** * **#21508** * **#21507** * **#21505** * **#21504** 👈 * **#21503** * **#21502** * **#21501** * **#21500** *...

[DS][6/n] Create Any/AllDepsMatchSchedulingConditions

@schrockn I address this AssetCondition/SchedulingCondition weirdness later on in the stack, but essentially `AssetCondition` becomes a subclass of `SchedulingCondition`, and is essentially vestigial. In a future PR, I can completely...

[DS][6/n] Create Any/AllDepsMatchSchedulingConditions

### Merge activity * **May 3, 5:31 PM EDT**: @OwenKephart started a stack merge that includes this pull request via [Graphite](https://app.graphite.dev/github/pr/dagster-io/dagster/21504). * **May 3, 5:42 PM EDT**: [Graphite](https://app.graphite.dev/github/pr/dagster-io/dagster/21504) rebased this...

[DS][5/n] Use AssetSlice in AssetConditionResult

### Merge activity * **May 3, 5:31 PM EDT**: @OwenKephart started a stack merge that includes this pull request via [Graphite](https://app.graphite.dev/github/pr/dagster-io/dagster/21503). * **May 3, 5:40 PM EDT**: [Graphite](https://app.graphite.dev/github/pr/dagster-io/dagster/21503) rebased this...