dagster icon indicating copy to clipboard operation
dagster copied to clipboard

Published 20 hours ago • verified publisher icon dagster-io

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

Open OwenKephart opened this issue 9 months ago • 1 comments

Summary & Motivation

Use an AssetSlice object in the internals of this class. Doesn't make a big difference for now, but gives functions that operate over these objects the option to use the AssetSlice API instead of AssetSubset.

Future PRs will change the input type to the create() functions, but that requires larger changes.

How I Tested These Changes

OwenKephart avatar Apr 29 '24 22:04 OwenKephart

  • #21511 Graphite: 2 dependent PRs (#21512 Graphite, #21520 Graphite)
  • #21510 Graphite
  • #21508 Graphite
  • #21507 Graphite
  • #21505 Graphite
  • #21504 Graphite
  • #21503 Graphite 👈
  • #21502 Graphite
  • #21501 Graphite
  • #21500 Graphite
  • #21499 Graphite
  • master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @OwenKephart and the rest of your teammates on Graphite Graphite

OwenKephart avatar Apr 29 '24 22:04 OwenKephart

Merge activity

  • May 3, 5:31 PM EDT: @OwenKephart started a stack merge that includes this pull request via Graphite.
  • May 3, 5:40 PM EDT: Graphite rebased this pull request as part of a merge.
  • May 3, 5:41 PM EDT: @OwenKephart merged this pull request with Graphite.

OwenKephart avatar May 03 '24 21:05 OwenKephart