-
Notifications
You must be signed in to change notification settings - Fork 3
LLM attribute calculation #287
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.
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
Conversation
alembic/versions/10c48793371d_adds_attribute_additional_config_and_.py
Outdated
Show resolved
Hide resolved
In the LLM playground modal, I would suggest that the 'Use current values for attribute' should be disabled if the 'Test configuartion' is still running (since it takes some time)
|
Should we display this in ![]()
edit @JWittmeyer: |
If i create an attribute from different type (ex. Category) and i change the datatype i don't see any of the configuration for the LLM response (only the provider and the api key), and the code is still the same. We have some logic here that if it's the initial value, so nothing has changes since the creation, the user can still override it, In this case we only need the config fields and the code change.
Edit @JWittmeyer |
Co-authored-by: anmarhindi <153752112+anmarhindi@users.noreply.github.com>
Test configuration and 'Load config template' still possible for usable attributes, i think it should be disabled. Edit Lina: I noticed that most of the fields are enabled, is that wanted?
Edit Jwittmeyer |
This is the Main PR.
Related: