Maxime Armstrong

Results 40 comments of Maxime Armstrong

* **#21461** * **#21405** 👈 * `master` This stack of pull requests is managed by Graphite. Learn more about stacking. Join @maximearmstrong and the rest of your teammates on Graphite

* **#20013** * **#19697** 👈 * `master` This stack of pull requests is managed by Graphite. Learn more about stacking. Join @maximearmstrong and the rest of your teammates on Graphite

> have you tested this in a toy dagster project and how does that look like in 1) asset metadata which is available across oss and cloud, 2) cloud insights?...

@yuhan @benpankow I updated the code in [ebe9367](https://github.com/dagster-io/dagster/pull/19697/commits/ebe936761952ee7a6aa74d00599f5c7503e74b9e) to match the first reviews. User can now use - `OpenAIResource.get_client(context)` for `AssetExecutionContext` and `OpExecutionContext` - We don't wrap the methods for...

@barbeau Yes. In the Python validator, `feed_start_date` must be in the future, and `feed_end_date` must be in more than 60 days.

* **#21296** 👈 * `master` This stack of pull requests is managed by Graphite. Learn more about stacking. Join @maximearmstrong and the rest of your teammates on Graphite

* **#21240** : 3 dependent PRs ([#21568](https://github.com/dagster-io/dagster/pull/21568) , [#21816](https://github.com/dagster-io/dagster/pull/21816) , [#21817](https://github.com/dagster-io/dagster/pull/21817) ) 👈 * **#21229** : 1 other dependent PR ([#21950](https://github.com/dagster-io/dagster/pull/21950) ) * `master` This stack of pull requests is...

@rexledesma @tacastillo @cmpadden These are very good points. I think using the scaffold would make sense in the quickstart. Maybe we could have a quickstart section and another one for...

The PR has been updated so that - the single file example is at the end of the guide - a call to action to the single file example is...

* **#21568** 👈 * **#21240** : 2 other dependent PRs ([#21816](https://github.com/dagster-io/dagster/pull/21816) , [#21817](https://github.com/dagster-io/dagster/pull/21817) ) * **#21229** : 1 other dependent PR ([#21950](https://github.com/dagster-io/dagster/pull/21950) ) * `master` This stack of pull requests...