Skip to content
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

#5283 - Improve usability of interactive recommender sidebar #5284

Conversation

reckart
Copy link
Member

@reckart reckart commented Feb 9, 2025

What's in the PR

  • Set default temperature of LLM recommenders to 0
  • Set default seed of LLM recommenders
  • Execute interactive recommender sidebar by pressing enter
  • Allow changing target layer and feature in interactive recommender sidebar
  • Remember sidebar settings across recommenders and sessions
  • Remember last-used recommender in the sidebar
  • Bit of cleaning up here and there

How to test manually

  • Use interactive recommender sidebar

Automatic testing

  • PR includes unit tests

Documentation

  • PR updates documentation

- Set default temperature of LLM recommenders to 0
- Set default seed of LLM recommenders
- Execute interactive recommender sidebar by pressing enter
- Allow changing target layer and feature in interactive recommender sidebar
- Remember sidebar settings across recommenders and sessions
- Remember last-used recommender in the sidebar
- Bit of cleaning up here and there
@reckart reckart added this to the 36.0 milestone Feb 9, 2025
@reckart reckart self-assigned this Feb 9, 2025
@reckart reckart merged commit 0802f71 into main Feb 9, 2025
3 checks passed
@reckart reckart deleted the feature/5283-Improve-usability-of-interactive-recommender-sidebar branch February 9, 2025 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🍹 Done
Development

Successfully merging this pull request may close these issues.

1 participant