Skip to content

Commit

Permalink
AssetGraphComputation inside AssetsDefinition (dagster-io#22554)
Browse files Browse the repository at this point in the history 
## Summary & Motivation

Sprouting off of Nick's comment
[here](dagster-io#22165 (comment)),
this moves all the computation-related properties of `AssetsDefinition`
into their own class.

While in this PR it's restricted to the internals of `AssetsDefinition`,
I think that this object could eventually take on a wider role. It
basically corresponds to the nodes on the right in this diagram I've
been bandying about:

<img width="860" alt="image"
src="https://github.com/dagster-io/dagster/assets/654855/69161dc4-dc23-4a35-a798-73e5548eb216">


## How I Tested These Changes
  • Loading branch information
@sryza @danielgafni
sryza authored and danielgafni committed Jun 18, 2024
1 parent ffdecf5 commit 3d5141f
Show file tree
Hide file tree
Showing 2 changed files with 126 additions and 91 deletions.
Loading

0 comments on commit 3d5141f

Please sign in to comment.