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

Migration kernel for the blueprint space-view-related breaking changes #8439

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

teh-cmc
Copy link
Member

@teh-cmc teh-cmc commented Dec 12, 2024

Our first migration kernel! This makes it possible to load <0.21 blueprints into Rerun 0.21, transparently, without user intervention.

More importantly, this marks the first step in exploring these kinds of tools, as part of our general promise of becoming more and more stable over time.

Basically a gloried search and replace.

To test it, save any blueprint from 0.20 (https://app.rerun.io/version/0.20.3/index.html), and load it into 0.21.

@teh-cmc teh-cmc added enhancement New feature or request 🧑‍💻 dev experience developer experience (excluding CI) ⛴ release Related to shipping or publishing 🟦 blueprint The data that defines our UI include in changelog labels Dec 12, 2024
Copy link

github-actions bot commented Dec 12, 2024

Web viewer built successfully. If applicable, you should also test it:

  • I have tested the web viewer
Result Commit Link Manifest
c30d905 https://rerun.io/viewer/pr/8439 +nightly +main

Note: This comment is updated whenever you push a commit.

@teh-cmc
Copy link
Member Author

teh-cmc commented Dec 12, 2024

@rerun-bot full-check

Copy link

@Wumpf Wumpf self-requested a review December 12, 2024 17:04
Copy link
Member

@Wumpf Wumpf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment on lines 64 to 68
if self.id.kind == re_log_types::StoreKind::Blueprint {
chunk = Arc::new(chunk.clone_as_untagged());
let patched = chunk.patched_for_blueprint_021_compat();
let patched = patched.clone_as_untagged();
chunk = Arc::new(patched);
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wish we had a version tag on the store so we're not applying this indefinitely into the future ...

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah we need to pull the StoreInfos all the way in at some point. We can do that later though, that's fine (I don't expect this particular change to be particularly version specific -- SpaceView is not making a come back anytime soon I hope 😛).

@teh-cmc teh-cmc merged commit e85cc13 into main Dec 12, 2024
73 of 74 checks passed
@teh-cmc teh-cmc deleted the cmc/space_view_shenaniganeries branch December 12, 2024 17:47
grtlr pushed a commit that referenced this pull request Dec 13, 2024
#8439)

Our first migration kernel! This makes it possible to load <0.21
blueprints into Rerun 0.21, transparently, without user intervention.

More importantly, this marks the first step in exploring these kinds of
tools, as part of our general promise of becoming more and more stable
over time.

Basically a gloried search and replace.

To test it, save any blueprint from 0.20
(https://app.rerun.io/version/0.20.3/index.html), and load it into 0.21.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🟦 blueprint The data that defines our UI 🧑‍💻 dev experience developer experience (excluding CI) enhancement New feature or request include in changelog ⛴ release Related to shipping or publishing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants