Skip to content

epic: Essential, advanced parameters are missing #3140

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

Closed
mockinbirdy opened this issue Jul 4, 2024 · 3 comments
Closed

epic: Essential, advanced parameters are missing #3140

mockinbirdy opened this issue Jul 4, 2024 · 3 comments
Labels
P2: nice to have Nice to have feature type: feature request A new feature

Comments

@mockinbirdy
Copy link

It's disappointing to see that essential, advanced parameters like min-p, repetition penalty range, and sample order (etc) can't be adjusted in Jan. Coming from projects like koboldcpp and tavern, where these settings have been implemented for a long time, it feels like a major oversight.

Will these settings be implemented in future updates?

Or am I just not getting it - is Jan supposed to be a super simple, plug-and-play kinda thing, ditching advanced customization for ease of use? I just had an impression it's an open-source soft with nice, clean ui, chat history and customisable settings. Honestly, even for coding, setting parameters precisely is crucial, and the current functionality falls short compared to other solutions.

Maybe you could hide the advanced stuff behind some hidden "Advanced" tab? That way, casual users wouldn't even notice, but pros could still dig in and tweak things.

@mockinbirdy mockinbirdy added the type: epic A major feature or initiative label Jul 4, 2024
@imtuyethan imtuyethan removed their assignment Aug 28, 2024
@imtuyethan imtuyethan added type: feature request A new feature and removed type: epic A major feature or initiative labels Aug 28, 2024
@imtuyethan imtuyethan changed the title epic: [DESCRIPTION] Essential, advanced parameters are missing epic: Essential, advanced parameters are missing Sep 2, 2024
@imtuyethan imtuyethan moved this to Planning in Menlo Sep 2, 2024
@freelerobot
Copy link
Contributor

freelerobot commented Sep 5, 2024

Related:
#3112
#3320

@dan-menlo
Copy link
Contributor

@mockinbirdy: We are starting to work on this in menloresearch/cortex.cpp#1151.

  • Thanks for the honest feedback
  • Jan -is- optimized for "non-technical users" (i.e. as a ChatGPT alternative)
  • However, at this stage of Local AI I fully agree on the need to set parameters
  • We will look into how we can hide this behind an "Advanced" slider, to make Jan useful to both power users and "normal user"

@dan-menlo
Copy link
Contributor

@mockinbirdy I'm merging this issue into #3140 and menloresearch/cortex.cpp#1151 where we are working on this.

ETA: this sprint

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2: nice to have Nice to have feature type: feature request A new feature
Projects
Archived in project
Development

No branches or pull requests

4 participants