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

Backend-defined package settings #2104

Merged
merged 42 commits into from
Aug 28, 2023
Merged

Backend-defined package settings #2104

merged 42 commits into from
Aug 28, 2023

Conversation

joeyballentine
Copy link
Member

This is a draft of my settings refactor. Package settings are now defined on the backend per-package.

I wanted to open this as a draft to get some feedback on how I approached 1) the settings definitions themselves, 2) how I approached how you get settings for a package during a run, and 3) how I typed the settings (since it's impossible to do typescript-like type magic in python to do it automatically)

I have moved everything to the new settings system, but I have only modified the pytorch nodes to use the new execution options system. I am yet to do it for ncnn and onnx.

joeyballentine and others added 24 commits July 21, 2023 00:18
Add a settings option in the 'Advanced Tab' to allow hardware acceleration.
instead of a toggle for always dark the user can now choose between:

Dark Mode / Light Mode / System
* add delete and select all to the nodes menu

* use proper delete and multiselect keycode on mac
* WIP

* new api

* /features endpoint

* New backend API

* Switch to packages in the UI

* Basic UI support for features

* typo

* UI WIP

* Unified markdown handling

* slight fixes

* Properly throw server errors

* Better error messages

* Basic feature checking

* Added spinner while refreshing
@joeyballentine
Copy link
Member Author

I'd rebase those broken commits out, but that causes a big merge conflict I don't want to deal with at the moment

backend/src/packages/chaiNNer_onnx/__init__.py Outdated Show resolved Hide resolved
backend/src/api.py Outdated Show resolved Hide resolved
backend/src/packages/chaiNNer_pytorch/__init__.py Outdated Show resolved Hide resolved
Co-authored-by: Michael Schmidt <mitchi5000.ms@googlemail.com>
@joeyballentine joeyballentine marked this pull request as ready for review August 19, 2023 22:08
backend/src/nodes/impl/pytorch/utils.py Outdated Show resolved Hide resolved
backend/src/api.py Outdated Show resolved Hide resolved
src/common/common-types.ts Outdated Show resolved Hide resolved
src/common/common-types.ts Outdated Show resolved Hide resolved
src/renderer/contexts/SettingsContext.tsx Show resolved Hide resolved
src/common/common-types.ts Outdated Show resolved Hide resolved
joeyballentine and others added 7 commits August 21, 2023 18:07
* Basic API refactor

* Make tensor cache path nullable

* Minor changes

* Setting components and other stuff

* Only show clear cache when cache is enabled

* Better type

* Get rid of cache object

* linter is always right
@joeyballentine joeyballentine merged commit 1ec090c into main Aug 28, 2023
@joeyballentine joeyballentine deleted the settings-stuff branch August 28, 2023 14:24
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.

4 participants