Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Get rid of train-taskcluster.sh #579

Closed
bhearsum opened this issue May 8, 2024 · 1 comment
Closed

Get rid of train-taskcluster.sh #579

bhearsum opened this issue May 8, 2024 · 1 comment
Labels
taskcluster Issues related to the Taskcluster implementation of the training pipeline

Comments

@bhearsum
Copy link
Collaborator

bhearsum commented May 8, 2024

This shell script was originally a very simple wrapper for train.sh to handle Taskcluster specific things. It has grown in complexity over time, to the point where it itself is now wrapped in a Python script (#546).

We should remove it, and move its remaining functionality to train_taskcluster.py.
@eu9ene eu9ene added the taskcluster Issues related to the Taskcluster implementation of the training pipeline label May 8, 2024
@eu9ene eu9ene mentioned this issue May 15, 2024
Open
@bhearsum
Copy link
Collaborator Author

Superceded by the more general #607

@eu9ene eu9ene mentioned this issue May 16, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
taskcluster Issues related to the Taskcluster implementation of the training pipeline
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bhearsum @eu9ene