-
Notifications
You must be signed in to change notification settings - Fork 1
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
Ckpt docs #61
Conversation
the checkpoints don't have to be copied there. We just need to refer to wherever it was saved in the results config under |
maybe we can update the |
|
||
## Steps to run benchmarking analysis | ||
|
||
1. To compute benchmarking features from the embeddings and trained models, run | ||
|
||
``` | ||
```bash | ||
python src/br/analysis/run_features.py --save_path "/outputs_cellpack/" --embeddings_path "./morphology_appropriate_representation_learning/model_embeddings/cellpack" --sdf False --dataset_name "cellpack" --debug False |
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.
we can make a similar matrix/table for both run_features and run_analysis
…ing/model_checkpoints
Updated with instructions to download to |
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.
LGTM!
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.
LGTM!
Issue
We need to explain how to download and use the checkpoints.
Closes #6
Changes
configs/experiment/cellpack/ckpts
.run_embeddings.py
with a table of all 6 options.pre-commit
Reviewing
configs/experiment/cellpack/ckpts
without further changes? I know that's where the checkpoints are output, but I don't know if they read from there too.