Bump DSI Version #1876
Merged
Bump DSI Version #1876
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide. |
theyostalservice
force-pushed
the
patricky/bump-dsi-version-hooray
branch
2 times, most recently
from
ec37377 to
b1f9453
Compare
theyostalservice
changed the title
theyostalservice
requested review from
courtneyholcomb and
plypaul
and removed request for
a team
theyostalservice
force-pushed
the
patricky/bump-dsi-version-hooray
branch
from
bdcdd56 to
0fd70d9
Compare
tests_metricflow/snapshots/test_mf_engine.py/list/test_list_metrics__result0.txt
Outdated
Show resolved
Hide resolved
courtneyholcomb
approved these changes
Oct 3, 2025
Contributor
courtneyholcomb
left a comment
courtneyholcomb
left a comment
There was a problem hiding this comment.
One more thing - I do think we should add an under the hood changelog entry for this!
theyostalservice
added a commit
that referenced
this pull request
Oct 3, 2025
Depends on #1876 Upstream PR bumps the dsi version, implicitly updating tests to using the default ruleset from dsi. We can't do that for production MF rn because MF uses an extra transformation internally, so this PR adds all the newer built in PRs from dsi to metricflow-semantics/metricflow_semantics/model/dbt_manifest_parser.py to be used in real circumstances. (If time was less limited before coalesce, I'd prefer to move the rule upstream if possible, but I don't want to pick up extra work before code freeze.) Edit: There's also a rule (AddInputMeasuresToMetrics) that only works in dsi. It probably merits a refactor/rewrite later to clean up the usage code here, but I'd rather prioritize getting this updated here for now.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Version BUUUUUUUMP
This PR is just here to pull in the new transformations and such from dsi. Early commits in this PR will be pinned to commits while I'm working on feeding dbt-labs/dbt-semantic-interfaces#436 to the merge-asaurus.
Snapshot updates reflect the fact that now(*) we produce more metrics to replace measures, reflecting the changes in structure for the new user YAML.
(*) "now" is true, but ONLY IN TESTS. Because of a transformation rule that exists only in MF right now, we don't use the list of transformations directly from dsi. See upstack #1877 for the PR that brings these changes to prod. OTH, tests DO run on the default list of metric transformations from dsi, so the snapshots needed more immediate updating.