Skip to content

fix: refactor AI Assistant page with bugs fixes#62

Open
thurstonchen wants to merge 8 commits intofeat/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 intofeat/yem/53-create-webui-with-streamlitfrom
feat/thurston/54-refactor-ai-assistant-page

Conversation

@thurstonchen
Copy link

@thurstonchen thurstonchen commented Sep 21, 2023

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 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.

1 participant