Open
Conversation
…t for pydantics behaviour with None-values.
…ks better accross different platforms
…els files to the package
…splash screen is shown before initializing the app
Contributor
Author
|
The app code is inside the app folder so it is pretty separated. However, all of these (desktop app, API and command like tool) use scripts |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The dev branch contains a lot of changes compared to the main one, it contains api related changes by @MLumme and then it adds the whole desktop app. From the scripts especially the classifier has changed a lot, it now has more config parameters and it works for both keras and tflite models. All the changed have been made so that nothing breaks in the existing scripts, but it may be a good idea to test that the scripts work as before before merging this.