Skip to content
This repository has been archived by the owner on Oct 13, 2023. It is now read-only.

6 Local Training #9

Merged
merged 45 commits into from
Oct 13, 2020
Merged

6 Local Training #9

merged 45 commits into from
Oct 13, 2020

Conversation

tsa87
Copy link
Contributor

@tsa87 tsa87 commented Oct 5, 2020

  • Modify CIL options
  • Add local_execution.sh that builds the docker image for local training
  • Change the name of trainer.sh to executor.sh

Feature #5 and #6

Tony Shen added 6 commits October 5, 2020 09:17
trainer.sh takes in github url, oauth token as argument and clones a private repo
Dockerfile is based on Python 3.7 base image, and calls the trainer script
Ubuntu has better compatibility with data scientists
- Modify CIL options
- Add local_execution.sh that builds the docker image for local training
- Change the name of trainer.sh to executor.sh
hydra/cli.py Outdated Show resolved Hide resolved
hydra/cli.py Outdated Show resolved Hide resolved
hydra/utils.py Outdated Show resolved Hide resolved
hydra/cli.py Outdated Show resolved Hide resolved
hydra/cli.py Outdated Show resolved Hide resolved
hydra/cli.py Outdated Show resolved Hide resolved
hydra/utils.py Outdated Show resolved Hide resolved
@coder46
Copy link
Contributor

coder46 commented Oct 8, 2020

@tsa87 Also please add tests for the train command

Comment on lines 1 to 7

DIR="$( dirname "${BASH_SOURCE[0]}" )"
LOG_NAME=$(date +'%Y_%m_%d_%H_%M_%S')

cd $DIR
#TODO Bug fix building hydra image
docker build -t hydra_image .
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

addresses #11

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@coder46 coder46 merged commit 43e53ed into master Oct 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants