Log in
Log into community
Find answers from the community
View all posts
Related posts
Was this helpful?
π
π
π
Powered by
Hall
Active
Updated 3 weeks ago
0
Follow
I recently converted NLSQLTableQueryEngine to workflows but it's not showing up on Arize Pheonix properly.
I recently converted NLSQLTableQueryEngine to workflows but it's not showing up on Arize Pheonix properly.
Active
0
Follow
O
Osama Adel
3 weeks ago
Β·
Hi, I recently asked a question regarding the NLSQLTableQueryEngine (
https://docs.llamaindex.ai/en/stable/api_reference/query_engine/NL_SQL_table/
) and how can I implement the row retrieval part in this notebook (
https://docs.llamaindex.ai/en/stable/examples/pipeline/query_pipeline_sql/#2-advanced-capability-2-text-to-sql-with-query-time-row-retrieval-along-with-table-retrieval
) but after converting it to workflows instead, and why it is not showing up on Arize Pheonix properly. Anyway, I have made a new class called NLSQLTableRowQueryEngine that inherits from NLSQLTableQueryEngine and a NLSQLRowRetriever that inherits from NLSQLRetriever with small changes to implement the up-mentioned technique in the notebook. I wanted to know what the process is if I wanted to contribute these changes to code base?
L
O
2 comments
Share
Open in Discord
L
Logan M
3 weeks ago
contributing is just a matter of forking the repo, making the changes in your fork, and proposing a PR π
O
Osama Adel
3 weeks ago
Perfect π
Add a reply
Sign up and join the conversation on Discord
Join on Discord