Skip to content

Conversation

@samvk12
Copy link

@samvk12 samvk12 commented Nov 20, 2025

Description

Replaced fixed st.selectbox dropdowns with flexible st.text_input fields for the LLM model selection in the Streamlit app. This allows users to enter custom model names.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • 🎨 Code style update (formatting, renaming)
  • ♻️ Refactoring (no functional changes)
  • ⚡ Performance improvement
  • ✅ Test update
  • 🔧 Configuration change
  • 🏗️ Infrastructure/build change

Related Issue(s)

#146
Fixes #146
Related to #146

Changes Made

Replaced st.selectbox with st.text_input: The fixed dropdown lists for model selection of OpenAI and Gemini.
Added help text: A help parameter was added to each text input to suggest common model options to the user.

Testing

I have tested it locally on my laptop only.

Test Configuration

  • Python Version: 3.12.0
  • OS: Windows
  • LLM Provider (if applicable):

Test Cases

  • Unit tests pass locally
  • Manual testing completed
  • Tested with sample datasets

Test Commands

streamlit run src/intugle/streamlit_app/main.py

Screenshots/Examples

image

Checklist

  • My code follows the code style of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or linter errors
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published
  • I have updated the relevant notebooks (if applicable)
  • I have checked my code and corrected any misspellings

Documentation Updates

  • README.md updated
  • Docstrings added/updated
  • Documentation site updated (if needed)
  • Notebook examples updated (if applicable)
  • CHANGELOG updated (if applicable)

Breaking Changes

  • This PR introduces breaking changes
  • Migration guide provided (if applicable)

Performance Impact

  • Performance benchmarks run
  • No significant performance impact
  • Performance improvement:
  • Performance regression:

Additional Context

Deployment Notes


@raphael-intugle
Copy link
Collaborator

raphael-intugle commented Nov 21, 2025

Great work @samvk12 ! Can you remove the txt.py, configs.yaml and requirements.py changes ? I shall merge after that

@raphael-intugle
Copy link
Collaborator

Hey there has been an update in the streamlit app ! There is now a anthropic key as well. Could you please merge from main and apply the same text box for the anthropic key as well ? Appreciate it

@samvk12
Copy link
Author

samvk12 commented Nov 21, 2025

Thanks for the review! I have removed txt.py, configs.yaml, and requirements.py from the PR.
I have also merged the latest changes from main and incorporated the text box for the Anthropic API key into the Streamlit app. The updated changes have now been pushed.

@raphael-intugle
Copy link
Collaborator

File "/home/raphael/intugle/data-tools/src/intugle/streamlit_app/main.py", line 1051
              model = st.text_input(
                                   ^
SyntaxError: '(' was never closed

Hey looks like there are some syntax errors. Can you take a look ? Please run using the intugle-streamlit command and check if the app works

@samvk12
Copy link
Author

samvk12 commented Nov 23, 2025

I’ve resolved the syntax errors and verified the changes by running the app. Everything is working as expected now.
Let me know if you’d like any additional updates.

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.

[GOOD FIRST ISSUE] Allow Custom Model Name Input in Streamlit LLM Settings

2 participants