Skip to content

Conversation

RamesTheGeneric
Copy link
Contributor

App will now load the default model and alert the user when an incorrect file path is specified rather than crashing on startup.

Incremented the version number from 2.0.6 Alpha to 2.1.0 Beta

commit Fix low frame rate by lowering timeout time. has had it's changes reverted, I can't seem to figure out how to remove it from my repo.

Loading an incorrect model path will cause the app to load the default model rather than crash on startup.

Incremented the app version from 2.0.6 Alpha to 2.1.0 Beta.
App will now load the default model and alert the user when an incorrect file path is specified rather than crashing on startup.

Incremented the version number from 2.0.6 Alpha to 2.1.0 Beta
Fix crash on model load and version increment
@dfgHiatus
Copy link
Contributor

Using this, it appears the user is prompted to choose a directory with the new "Browse" button. Is this the intended behavior? Or is the user expected to pick the model file as opposed to the directory

@RamesTheGeneric
Copy link
Contributor Author

The user is intended to pick a model directory just like before, now there is simply a file browser to make picking the model file easier. A model path can still be specified like before.

@dfgHiatus
Copy link
Contributor

"The user is intended to pick a model directory..."
"...now there is ... a file browser to ... [pick a] model file..."

Sorry, which one is this supposed to be here? Presently, if the user is expected to select a model file they are only able to choose a directory.

@RamesTheGeneric
Copy link
Contributor Author

The models we distribute are stored in a specific directory structure. The way we have always specified the model is by the directory. This is no different.

@dfgHiatus
Copy link
Contributor

Did some more testing here. Things are looking pretty good, I'll keep having a look at it, but I have no further comments at this time

@RamesTheGeneric
Copy link
Contributor Author

I believe this is ready to merge then. I just need you to complete the review.

@dfgHiatus dfgHiatus merged commit 8d9d604 into Project-Babble:main Nov 28, 2024
1 check passed
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