Skip to content
This repository was archived by the owner on Jun 15, 2026. It is now read-only.

fix: refactor AI Assistant page with bugs fixes#62

Open
thurstonchen wants to merge 8 commits into
feat/yem/53-create-webui-with-streamlitfrom
feat/thurston/54-refactor-ai-assistant-page
Open

fix: refactor AI Assistant page with bugs fixes#62
thurstonchen wants to merge 8 commits into
feat/yem/53-create-webui-with-streamlitfrom
feat/thurston/54-refactor-ai-assistant-page

Conversation

@thurstonchen

@thurstonchen thurstonchen commented Sep 21, 2023

Copy link
Copy Markdown

Major changes of this PR:

  • Update Standardization SQL and Aggregation SQL tabs of AI Assistant page, to make them have same UI of Editor page, so that users could run and test the AI-generated SQL in the same page.
  • Fixed the table_name as session state key TODO issue, now the checkbox would show up only after sample data has been generated.
  • Fixed the aggregation SQL generation issue when standardization dataset has been involved (a bug found by Gary).
  • Added dependency/reference checking logic to stg/std/srv tables, therefore tables referenced by others cannot be renamed or deleted, a st.info() message will show up to remind end users.
    Exp:
  • Refactored the table generation logic, to make it generate one table at a time/request.
    Exp:
  • Added retry as well as exception handling for OpenAI requests.
  • Downgrade numpy version to 1.23.5, since we notice an incompatible issue from pyspark while running Spark SQL during testing; details please refer to the below expired-deprecations (the last bullet point) since version of 1.24.0.
    https://numpy.org/devdocs/release/1.24.0-notes.html#expired-deprecations

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant