Dazhong Xia

Results 71 comments of Dazhong Xia

Hi @fitnr ! I'm a coworker of @e-belfer's. Thanks for fixing #8 :) We'd love to avoid to the `fix/8` branch if possible - let me know if we can...

@bendnorman do you think this is done? Or should we make a list of the main themes of the feedback / make a bunch of issues about it before we...

Hmm, for option 3 we can also use something like [`direnv`](https://direnv.net/) and ship some standard environment variables directly with the repo... like setting DAGSTER_HOME to be `./dagster_home` or something. Though...

Aha! Indeed, either setting `SLACK_WEBHOOK_TYPE: INCOMING_WEBHOOK` or using a `SLACK_BOT_TOKEN` instead works - thanks for the quick response! Do you think this warrants an update in `README.md` or was it...

Hi Aurel, I'm glad you took a look but @ggurjar333 has already tackled most of the work for this issue in [this PR](https://github.com/catalyst-cooperative/pudl/pull/3931). I took a quick look at the...

Yeah, that makes sense! I just copied over your list from the original issue but obviously happy to change that 😄

tl;dr: this got hard, I propose: * giving up on "the dbt tests run automatically on dagster asset materialization" * making some QOL changes so "run validation check for asset...

Broke out large portions of this into #4292 and #4293 . What remains here is the dagster DBT connection.

x-posting a comment from #4315 here. > I agree, there's machinery in dagster-dbt to create a mapping between dagster assets + dbt nodes based on the manifest.json - we'd need...

> I want to know more about > > > currently, [`dagster-dbt`] only creates check specs for sources that are relied upon by models. For us this skips the vast...