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.

Sign up for GitHub

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

Jump to bottom

Mdims: Migrate config to the latest version after fetching from R2 #4239

Closed
sophiamersmann opened this issue Dec 2, 2024 · 1 comment
Closed

Mdims: Migrate config to the latest version after fetching from R2 #4239

sophiamersmann opened this issue Dec 2, 2024 · 1 comment
Labels
needs triage viz

Comments

@sophiamersmann
Copy link
Member

Configs fetched from R2 might be out-of-date. The client should be defensive about that. MDims should also migrate configs after fetch, similar to how it's done in this PR for the Multiembedder.
@sophiamersmann sophiamersmann mentioned this issue Dec 2, 2024
Merged
@danyx23
Copy link
Contributor

danyx23 commented Dec 19, 2024

This is already implemented

@danyx23 danyx23 closed this as completed Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs triage viz
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@danyx23 @sophiamersmann