-
Notifications
You must be signed in to change notification settings - Fork 483
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
Added ccache to CI and fixed PyTorch caching #161
Conversation
.circleci/config.yml
Outdated
paths: | ||
- ~/miniconda | ||
- run: | ||
name: ccache stats |
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.
name: ccache stats | |
name: CCache Stats |
.circleci/config.yml
Outdated
@@ -107,6 +107,19 @@ jobs: | |||
- restore_cache: | |||
keys: | |||
- habitat-sim-{{ checksum "./hsim_sha" }} | |||
- restore_cache: | |||
keys: | |||
- ccache-{{ arch }}-{{ .Branch }}-{{ epoch }} |
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.
Won't this not work as {{ epoch }}
is the current time and the cache-key will forever be different? Since this is the API, probably ccache-{{ arch }}-{{ checksum "./hsim_sha" }}
makes sense as the key.
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.
Actually, it should just be ccache-{{ arch }}
as the install of the simulator is already cached.
.circleci/config.yml
Outdated
paths: | ||
- ./habitat-sim | ||
- save_cache: | ||
key: ccache-{{ arch }}-{{ .Branch }}-{{ epoch }} |
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.
Same comment about the key.
8df0996
to
8a2dd4c
Compare
* viewer: access pathfinder only if it's actually loaded. * assets: don't crash on relative house.json paths.
* fixed PyTorch caching * added ccache to CI * broke habitat api step on installation and testing to save extra 5 min on saving cache in parallel * testing time drops from first run to second run from 17.5 minutes to 9 minutes (2x times) for full testing.
The testing time drops from first run to second run from 17.5 minutes to 9 minutes (2x times) for full testing.
How Has This Been Tested
Types of changes
Checklist