Skip to content

Commit

Permalink
Ignore markdown for pre-commit (#1171)
Browse files Browse the repository at this point in the history
* ignore markdown for pre-commit

* only ignore end of file and trailing whitespace

* Update NeoXArgs docs automatically

---------

Co-authored-by: github-actions <[email protected]>
  • Loading branch information
Quentin-Anthony and github-actions authored Mar 4, 2024
1 parent 7b8187a commit 31cfe52
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
3 changes: 2 additions & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,14 +8,15 @@ repos:
- id: check-yaml
- id: destroyed-symlinks
- id: end-of-file-fixer
exclude: docs/CNAME
exclude: ^(docs/CNAME/|configs/neox_arguments.md)
- id: fix-byte-order-marker
- id: fix-encoding-pragma
args: [--remove]
- id: mixed-line-ending
args: [--fix=lf]
- id: requirements-txt-fixer
- id: trailing-whitespace
exclude: ^(docs/CNAME/|configs/neox_arguments.md)
- repo: https://gitlab.com/daverona/pre-commit/cpp
rev: 0.8.0
hooks:
Expand Down
9 changes: 5 additions & 4 deletions configs/neox_arguments.md
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ Logging Arguments

- **git_hash**: str

Default = 11a2e9b
Default = b804ee8

current git hash of repository

Expand Down Expand Up @@ -1058,7 +1058,7 @@ Text Generation arguments

- **prompt_end**: str

Default =
Default =


a single prompt's end. Defaults to newline
Expand Down Expand Up @@ -1100,7 +1100,7 @@ Text Generation arguments

- **eval_results_prefix**: str

Default =
Default =

prefix to which to save evaluation results - final fp will be {eval_results_prefix}_eval_results_yy-mm-dd-HH-MM.json

Expand Down Expand Up @@ -1844,7 +1844,7 @@ Args for deepspeed config

Default = None





Expand Down Expand Up @@ -2144,3 +2144,4 @@ Args for deepspeed runner (deepspeed.launcher.runner).
Default = None

Adds a `--account` to the DeepSpeed launch command. In DeeperSpeed this is passed on to the SlurmLauncher as well. Sometimes necessary for cluster rules, or so I've heard.

0 comments on commit 31cfe52

Please sign in to comment.