Skip to content

Conversation

@cpsievert
Copy link
Contributor

@cpsievert cpsievert commented Dec 2, 2025

This PR makes the table_name parameter optional when creating a QueryChat object with a data frame. The table name is now automatically inferred from the variable name, simplifying the API for the most common use case.

Before:

qc <- QueryChat$new(mtcars, "mtcars")

After:

qc <- QueryChat$new(mtcars)  # table_name inferred as "mtcars"

The table_name parameter remains required for database connections.

@cpsievert cpsievert requested a review from Copilot December 2, 2025 21:41

This comment was marked as resolved.

@cpsievert cpsievert marked this pull request as ready for review December 2, 2025 21:52
@cpsievert cpsievert requested a review from gadenbuie December 2, 2025 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants