bloop icon indicating copy to clipboard operation
bloop copied to clipboard

Use pre-built `ort`, fix builds on Windows

Open calyptobai opened this issue 2 years ago • 1 comments

This PR adds support for building on Windows. Additionally, we (mostly) revert our changes in ort, and instead rely on pre-built binaries. On Windows, the only platform where onnxruntime has telemetry enabled, we disable it at runtime.

LFS scripts were moved to package.json, and the build process has been simplified slightly. It is now possible to run npm start to run the app.

calyptobai avatar Mar 03 '23 23:03 calyptobai

One question, why has model/ been moved to apps/desktop/src-tauri? We also use the contents of model/ when bleep's run without Tauri.

Mentioned in adb41df, to elaborate: model was moved into the app directory and the symlink was flipped so that we have a root -> apps/desktop/src-tauri/model link. This is functionally equivalent on unix systems, but on Windows it allows us to include model as a Tauri resource. Windows does not support symlinks by default, so we can't rely on the previous setup.

calyptobai avatar Jun 20 '23 16:06 calyptobai

@rsdy

Also, can we integrate the Windows qdrant build into the CI as a separate PR so we have a definitive source of builds like with the other platforms?

We don't build qdrant ourselves here, instead we simply use the official GitHub release builds.

calyptobai avatar Jun 21 '23 22:06 calyptobai