-
Notifications
You must be signed in to change notification settings - Fork 3
handle esrun->olmoearth_run naming update #227
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
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.
Seems good to me, thanks for the renames!
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 great. Thanks!
A couple of places that need updating (s/earth-system-run/olmoearth_run/
:
https://github.com/allenai/rslearn_projects/blob/master/requirements-olmoearth_run.txt
https://github.com/allenai/rslearn_projects/blob/master/.github/workflows/build_test.yaml
2ed0c78
to
26e3730
Compare
26e3730
to
4ff3db9
Compare
esrun
is now known asolmoearth_run
.This changeset updates instructions, sample files,
and CLIs to use the new naming scheme.
It does NOT update any individual projects
that may exist, as I'm not sure what the status
of any of these is.
There was also a recent
helios
->olmoearth_pretrain
rename. This does NOT handle that and instead pins to
the commit just prior to the rename.
Updating is not in scope for this PR.