Skip to content
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

Feature info schema #34

Closed
wants to merge 2 commits into from

Conversation

hardtke
Copy link
Contributor

@hardtke hardtke commented Jul 12, 2022

ML.FEATURE_INFO seems unstable but this seems to be the latest schema. The documentation on their website does not actually have the dimension field.

This (temporarily) fixes the bug I reported: #33

@stujo
Copy link
Contributor

stujo commented Jul 18, 2022

Thanks @hardtke! Would love to see this issue resolved, did you find a way to run this modified package locally as temporary solution until this PR is accepted and a new version of the package published?

@dmccabe
Copy link

dmccabe commented Aug 10, 2022

@stujo You can do this pretty easily with a fork and tag while the PR is pending.

In your packages.yml:

  - git: [email protected]:<your-org>/dbt_ml.git
    revision: <some-tag>

@rbjerrum rbjerrum closed this Feb 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants