-
Notifications
You must be signed in to change notification settings - Fork 100
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SL-3389] [Feature] MetricFlow doesn't support DBT_PROJECT_DIR
environment variable
#1591
Comments
Thank you for your bug report! Our team is will be out of the office for Christmas and our Global Week of Rest, from December 25, 2024, through January 3, 2025. We will review your issue as soon as possible after returning. If you are a customer of dbt Cloud, please contact our Customer Support team via the dbt Cloud web interface or email [email protected]. |
DBT_PROJECT_DIR
environment variable
Thanks for reaching out @mercedes-wu-wandb! I'm going to transfer this issue to the |
Transferred from https://github.com/dbt-labs/dbt-core/issues/11179 to #1591. |
DBT_PROJECT_DIR
environment variableDBT_PROJECT_DIR
environment variable
@mercedes-wu-wandb To clarify, |
Yes
|
It should be straightforward to add support for |
Hey @plypaul, so far i have not so I'm happy to change this to issue to a feature if that's helpful! |
Yes, that would be great. Thanks! |
DBT_PROJECT_DIR
environment variableDBT_PROJECT_DIR
environment variable
Changed to "[SL-3389] [Feature] MetricFlow doesn't support |
There was some relevant prior work that has already been done, and I'm aiming to get a PR out this week. |
awesome, thanks a lot for the updates and looking forward to the PR! |
Hi @plypaul ! Just wondering if there were any updates on this feature, thanks a ton for all the help so far! |
To give an update, getting this out for release required a large number of changes to the project's CLI test framework as we want to have good test coverage for this feature. In addition, as I was getting ramped up on the CLI implementation, I saw there were a few CLI bugs (e.g. display of numeric types) that would be beneficial to have in the release. Right now, the feature is in the review pipeline but at the bottom of a stack of changes: https://github.com/dbt-labs/metricflow/pull/1689 The PR for this feature is relatively small, but partly due to the work that was included by prior PRs in the stack. We're looking to get these reviewed and merged, and then I can cut a release. While there a number of changes, I think we can get it out this week. |
thanks a ton for the detailed update! really appreciate the help and looking forward to the release! |
Is this a new bug in dbt-core?
Current Behavior
commands from dbt project level:
mf health-checks
output:
Expected Behavior
Steps To Reproduce
export DBT_PROJECT_DIR=/path/to/dbt/project
export DBT_PROFILES_DIR=/path/to/profiles/directory
mf health-checks
Relevant log output
No response
Environment
Which database adapter are you using with dbt?
bigquery
Additional Context
I am trying to run metricflow on dbt-core
SL-3389
The text was updated successfully, but these errors were encountered: