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

Smooth model switch #166

Merged
merged 11 commits into from
Jul 26, 2024
Merged

Smooth model switch #166

merged 11 commits into from
Jul 26, 2024

Conversation

jmbejar
Copy link
Collaborator

@jmbejar jmbejar commented Jul 23, 2024

No description provided.

@jmbejar jmbejar force-pushed the smooth-model-switch branch from 592bef3 to a352148 Compare July 23, 2024 18:54
@jmbejar jmbejar changed the base branch from main to wrap-up-clicking-issues July 23, 2024 18:59
@jmbejar jmbejar force-pushed the wrap-up-clicking-issues branch from f3636b8 to 7e94837 Compare July 23, 2024 19:03
Base automatically changed from wrap-up-clicking-issues to main July 24, 2024 13:58
@jmbejar jmbejar force-pushed the smooth-model-switch branch from a352148 to 1056e1a Compare July 24, 2024 15:10
@jmbejar jmbejar marked this pull request as ready for review July 24, 2024 15:12
@@ -316,22 +316,15 @@ fn create_wasi(
file: &DownloadedFile,
load_model: &LoadModelOptions,
) -> wasmedge_sdk::WasmEdgeResult<WasiModule> {
let ctx_size = if load_model.n_ctx > 0 {
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Context: This change was originally done here https://github.com/moxin-org/moxin/blob/1056e1a0e2789f99bf7fc5ca40149b462a79c997/moxin-backend/src/backend_impls/api_server.rs#L38-L46, which is the currently active backend.

Making the change here as well since I'm removing those values from the protocol, but it does not have real effect right now.


ANIMATION_SPEED = 1.5;

Bar = <RoundedView> {
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a bunch of repeated logic with other loading widget implemented, but decided to not overcomplicate this PR at this point.

@jmbejar jmbejar force-pushed the smooth-model-switch branch from ed8192d to 3f0832a Compare July 25, 2024 15:27
@noxware noxware self-requested a review July 25, 2024 18:30
@jmbejar jmbejar merged commit 97b08b0 into main Jul 26, 2024
15 checks passed
@jmbejar jmbejar deleted the smooth-model-switch branch July 26, 2024 19:03
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