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

Submit context menu #414

Merged
merged 13 commits into from
Jul 29, 2023
Merged

Submit context menu #414

merged 13 commits into from
Jul 29, 2023

Conversation

one-some
Copy link

not at main pc so cant make fancy pr with screenshot but right click submit button to see it

  • make context menu more compact
  • add cool context menu on submit button
  • add generation modes for model backend for temp stoppers like in the context menu
  • changes context menu stuff (positioning algorithm, added support for callback in enabled check)

woohooooo back to css
- fixes margins to look better
- moves contents of context menu items 1px down
- fixes context menus near edge wrapping their inner text (ew)
5px was a bit excessive
TODO: studied the context menu in my browser for a bit and noticed that
if it was going to be too close to the bottom, the browser changes the
vertical direction the context menu goes. sounds neat!
I think this would be cool!

Ideas:
    - disable/grey when model doesnt support stopping criteria
    - shortcuts (maybe, this would def be a power user thing)
    - option to generate until EOS token
    - option to generate forever until user manually stops
    - (not super related but pixels away) make retry while generation is
        ongoing cancel generation and retry. same with undo.
too many ideas at once
- adds callback support to `enabledOn` in context menu items
- adds `supported_gen_modes` variable for frontend to check if a gen
  mode is supported
- adds `get_supported_gen_modes` to `InferenceModel` to get supported
  gen modes
- takes advantage of cool enum features for less enum-handling code
This mode enables the EOS token and will generate infinitely until
hitting it.
...without an entry in `context_menu_items`.
@henk717
Copy link
Owner

henk717 commented Jul 24, 2023

Got hit by the other PR's we prioritized.

@one-some
Copy link
Author

fixed

@henk717 henk717 merged commit 21d2085 into henk717:united Jul 29, 2023
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.

2 participants