Skip to content

Commit

Permalink
fix: use string instead of boolean in YAML for "YES"
Browse files Browse the repository at this point in the history
`field: YES` will be parsed to boolean in YAML, and
resulting `"field": True` in Python. This makes any
use of "field" as a string problematic in the code.
This commit fixes this bug by quoting it properly.

Signed-off-by: Kai Xu <[email protected]>
  • Loading branch information
xukai92 authored and russellb committed Jul 13, 2024
1 parent d1c5d5b commit 2c52770
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/instructlab/sdg/pipelines/full/knowledge.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ blocks:
type: FilterByValueBlock
config:
filter_column: judgment
filter_value: YES
filter_value: "YES"
operation: eq
drop_columns:
- judgment
Expand Down

0 comments on commit 2c52770

Please sign in to comment.