Generic Taskcluster task naming #589
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
vrigal
force-pushed
the
generic-tc-task-name
branch
from
da0d6e4
to
f790c3f
Compare
vrigal
force-pushed
the
generic-tc-task-name
branch
from
f790c3f
to
35133ed
Compare
eu9ene
reviewed
May 15, 2024
There was a problem hiding this comment.
@vrigal Do I understand correctly that now a suffix -1 is added to all the runs? If yes, we do not want that. We train multiple models only for the teacher. This -1 makes names longer. To be more explicit, we want:
(for older models)
teacher-base-0
teacher-base-1
teacher-parallel-0
teacher-parallel-1
teacher-all-0
teacher-all-1
finetune-teacher-1
finetune-teacher-2
etc.
(for newer Taskcluster models)
teacher-1
teacher-2
backwards
student
student-finetuned
quantized
teacher-ensemble
Naming of the teacher models will differ for the older experiments but everything else should be exactly the same across all the experiments and publishers. We might want to store a map of renamings.
vrigal
force-pushed
the
generic-tc-task-name
branch
from
469781d
to
f17b963
Compare
vrigal
force-pushed
the
generic-tc-task-name
branch
from
2706f10
to
6c3c411
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Required by #519 (making
build_task_namesupports evaluation tasks labels tags from Taskcluster)Closes #573