Skip to content

Commit

Permalink
Add docblock for SensorResult.asset_events (#16705)
Browse files Browse the repository at this point in the history
## Summary & Motivation

Since we are putting an experimental param on a publicly exporting
class, we should document it.

## How I Tested These Changes

BK

---------

Co-authored-by: Yuhan Luo <[email protected]>
  • Loading branch information
schrockn and yuhan authored Sep 26, 2023
1 parent ce270a8 commit ad95c8a
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 3 deletions.
2 changes: 1 addition & 1 deletion docs/content/api/modules.json

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion docs/content/api/searchindex.json

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion docs/content/api/sections.json

Large diffs are not rendered by default.

Binary file modified docs/next/public/objects.inv
Binary file not shown.
Original file line number Diff line number Diff line change
Expand Up @@ -387,6 +387,12 @@ class SensorResult(
AddDynamicPartitionsRequest]]]): A list of dynamic partition requests to request dynamic
partition addition and deletion. Run requests will be evaluated using the state of the
partitions with these changes applied.
asset_events (Optional[Sequence[Union[AssetObservation, AssetMaterialization, AssetCheckEvaluation]]]): (Experimental) A
list of materializations, observations, and asset check evaluations that the system
will persist on your behalf at the end of sensor evaluation. These events will be not
be associated with any particular run, but will be queryable and viewable in the asset catalog.
"""

def __new__(
Expand Down

1 comment on commit ad95c8a

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deploy preview for dagster-docs ready!

✅ Preview
https://dagster-docs-jme90q2yi-elementl.vercel.app
https://master.dagster.dagster-docs.io

Built with commit ad95c8a.
This pull request is being automatically deployed with vercel-action

Please sign in to comment.