-
Notifications
You must be signed in to change notification settings - Fork 399
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
Move tests/assets/video
over to git-lfs
#8936
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks! but you missed a lot of all ;) places that try to invoke download_test_assets
there's more! rust_checks.py for instance and some ymls |
for the yml you need to make sure that those jobs pull lfs now |
Web viewer built successfully. If applicable, you should also test it:
Note: This comment is updated whenever you push a commit. |
Latest documentation preview deployed successfully.
Note: This comment is updated whenever you push a commit. |
@rerun-bot full-check |
Started a full build: https://github.com/rerun-io/rerun/actions/runs/13161672076 |
@rerun-bot full-check |
Started a full build: https://github.com/rerun-io/rerun/actions/runs/13161954049 |
(still one last thing to fix but im on another fire right now) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks about right now! good to go after main build is confirmed working
nice to not have these steps anymore
458e41f
to
07b7ed2
Compare
Passed ✨ |
It's a bit of an odd one because of the way the dataloader works, but I would still feel much better with an odd test than no tests at all. I've uploaded a heavily sampled down `apple_storage` dataset to LFS in order to make this work. * DNM: requires #8936
Title.
git-lfs
to our CI container #8946