Conversation
Save to .pt2 archive format (https://docs.pytorch.org/docs/2.11/user_guide/torch_compiler/export/pt2_archive.html) for better portability.
martham93
left a comment
There was a problem hiding this comment.
This looks good thank you!
I think as a follow up, we should consider expanding and generating .pt2 files to support other modality combinations we think we would want to use? (I can ticket + help work on if we think this would be useful)
These weights will only allow us to run inference with S2 data which is a reasonable starting point but if we wanted to gather other data as well it would fail.
There was a problem hiding this comment.
cool! I didn't know how to separately manage script environments outside of the main project dependencies!
Thanks for including the uv running scripts documentation in the PR description, super helpful
There was a problem hiding this comment.
Yeah, I was aware of PEP 723 scripts for a while, but didn't realize uv was able to generate lockfiles for scripts too! It's also pretty cool that you could add more dependencies using uv add --script *.py 'some-dep' and the lockfile updates accordingly, essentially a mini pyproject.toml+uv.lock kind of situation.
|
Sure, that would be good! Should I maybe rename the weights from |
|
+1 to that naming scheme, but I'll get a ticket up soon so we can discuss there |
What I am changing
How I did it
TODO:
References:
How you can test it
uv run --no-project scripts/export_olmoearth.py --variant tinyscript locallyRelated Issues
Address #32