Skip to content

Commit f5e068b

Browse files
cpsievertclaude
andauthored
fix(pkg-py): Standardize categorical_threshold default to 20 (#125)
Updates the Python package to use categorical_threshold=20 as the default value, matching the R package default. This resolves the inconsistency between the two implementations. Fixes #122 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude <[email protected]>
1 parent 69db396 commit f5e068b

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

pkg-py/src/querychat/_deprecated.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ def system_prompt(
7777
*,
7878
data_description: Optional[str | Path] = None,
7979
extra_instructions: Optional[str | Path] = None,
80-
categorical_threshold: int = 10,
80+
categorical_threshold: int = 20,
8181
prompt_template: Optional[str | Path] = None,
8282
) -> str:
8383
"""

pkg-py/src/querychat/_querychat.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ def __init__(
3434
greeting: Optional[str | Path] = None,
3535
client: Optional[str | chatlas.Chat] = None,
3636
data_description: Optional[str | Path] = None,
37-
categorical_threshold: int = 10,
37+
categorical_threshold: int = 20,
3838
extra_instructions: Optional[str | Path] = None,
3939
prompt_template: Optional[str | Path] = None,
4040
):
@@ -532,7 +532,7 @@ def __init__(
532532
greeting: Optional[str | Path] = None,
533533
client: Optional[str | chatlas.Chat] = None,
534534
data_description: Optional[str | Path] = None,
535-
categorical_threshold: int = 10,
535+
categorical_threshold: int = 20,
536536
extra_instructions: Optional[str | Path] = None,
537537
prompt_template: Optional[str | Path] = None,
538538
enable_bookmarking: Literal["auto", True, False] = "auto",
@@ -697,7 +697,7 @@ def get_system_prompt(
697697
*,
698698
data_description: Optional[str | Path] = None,
699699
extra_instructions: Optional[str | Path] = None,
700-
categorical_threshold: int = 10,
700+
categorical_threshold: int = 20,
701701
prompt_template: Optional[str | Path] = None,
702702
) -> str:
703703
# Read the prompt file

0 commit comments

Comments
 (0)