New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactor llm i #353

Open

mschwoer wants to merge 4 commits into simplify_analysis-III from refactor_llm_I

Contributor

mschwoer commented Oct 21, 2024

Note that this is not the final state in terms of UI/UX, but just a step towards fully integrating LLM.

reduce code duplication for LLM part by reusing the information generated in Analysis.
simplify logic on LLM page
remove some keys from session state

mschwoer added 3 commits

October 21, 2024 20:33


          make do_analysis return whole object

03ccb92


          simplify LLM page

ad9f175


          clean up session state

488c7ac

mschwoer requested review from JuliaS92 and boopthesnoot

October 21, 2024 19:28


          make API key messages nicer

92b9040

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

Labels

None yet