generated from fastai/nbdev_template
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Remove tyro #1176
Merged
Merged
Remove tyro #1176
Changes from all commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
c88e5f6
refactor
vwxyzjn 0ebeb38
Remove tyro in `ppo.py`
vwxyzjn 527d96f
quick update
vwxyzjn 789108a
update default args
vwxyzjn e7f9580
Merge branch 'main' into refactor3
vwxyzjn d7e66c3
quick push
vwxyzjn fa2a4c8
precommit
vwxyzjn 67a8f53
refactor
vwxyzjn fe3f766
quick change
vwxyzjn 34ea7d7
remove tyro
vwxyzjn af724f4
quick change
vwxyzjn 800b9fd
precommit
vwxyzjn c257d85
quick change
vwxyzjn 7960804
fix hello_world
vwxyzjn 782fdd3
remove docstring diffences
vwxyzjn 3c1f3a0
add `module load cuda/12.1`
vwxyzjn da745c8
push changes
vwxyzjn 61506ff
precommit
vwxyzjn 04f5a7b
make dpo runnable
vwxyzjn 83ad452
fix circular import
vwxyzjn 56bea91
quick fix
vwxyzjn 5936fa9
refactor
vwxyzjn c636cfd
quick update
vwxyzjn 0022af6
path change
vwxyzjn cca6940
update plots
vwxyzjn 7b5e7b2
Merge branch 'main' into refactor3
vwxyzjn bfde066
fix docs
vwxyzjn a6addfd
quick change
vwxyzjn 040e32d
Update trl/trainer/model_config.py
vwxyzjn 348faf0
Update trl/trainer/model_config.py
vwxyzjn e646352
Update trl/trainer/utils.py
vwxyzjn 02e3f3c
Update examples/scripts/dpo.py
vwxyzjn f4d0840
Merge branch 'main' into refactor3
vwxyzjn d355e38
address comments. use attn_implementation
vwxyzjn 87fe921
precommit
vwxyzjn bc1a7aa
remove duplicate code
vwxyzjn 161a102
update peft.py
vwxyzjn 884aac1
fix test no op dep
vwxyzjn 5a496f7
Update trl/trainer/utils.py
vwxyzjn bd5ab8c
Apply suggestions from code review
vwxyzjn 455157e
precommit
vwxyzjn 74834d2
add docs
vwxyzjn File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
BENCHMARK_SCRIPT="benchmark/benchmark_level1.sh" \ | ||
BENCHMARK_PLOT_SCRIPT="benchmark/benchmark_level1_plot.sh" \ | ||
bash benchmark/benchmark_and_report.sh |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,16 +1,19 @@ | ||
#!/bin/bash | ||
#SBATCH --job-name=trl | ||
#SBATCH --partition=production-cluster | ||
#SBATCH --partition=hopper-prod | ||
#SBATCH --gpus-per-task={{gpus_per_task}} | ||
#SBATCH --cpus-per-gpu={{cpus_per_gpu}} | ||
#SBATCH --ntasks={{ntasks}} | ||
#SBATCH --output=slurm/logs/%x_%j.out | ||
#SBATCH --array={{array}} | ||
#SBATCH --exclude=ip-26-0-156-239,ip-26-0-148-151,ip-26-0-146-212,ip-26-0-145-137,ip-26-0-146-249,ip-26-0-146-149,ip-26-0-147-233,ip-26-0-145-154,ip-26-0-144-35,ip-26-0-144-189,ip-26-0-146-183,ip-26-0-147-120,ip-26-0-144-95,ip-26-0-145-193 | ||
##SBATCH --exclude=ip-26-0-149-199 | ||
|
||
module load cuda/12.1 | ||
|
||
{{nodes}} | ||
|
||
seeds={{seeds}} | ||
seed=${seeds[$SLURM_ARRAY_TASK_ID % {{len_seeds}}]} | ||
|
||
echo "Running task $SLURM_ARRAY_TASK_ID with seed: $seed" | ||
srun {{command}} --ppo_config.seed $seed | ||
srun {{command}} --seed $seed |
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
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.
Could you add a documentation section here to explain what
ModelConfig
does and how to use it together with the other utility methods that you have exposed on main init? 🙏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.
Done!