Mert

Results 543 comments of Mert

I'm fine with this being an environmental variable without changing the default behavior. Maybe `MACHINE_LEARNING_KEEPALIVE_TIMEOUT_S`?

Just the trigram index is fine. I'm not sure what the normal index for the filename is used for, but it isn't for this.

This looks good, but there will likely be a tree-structured index as part of the work toward optimizing queries for the new folder view. The ltree index for this will...

I can do some testing today and make a PR for the ltree index.

The relevant change is #12214, so I guess it depends on when that's ready. I'm fine with adding originalPath to the search builder now without the index, but it'd be...

Yes, that seems to be the issue. It might be best to fuse that query with the `getAllForUser` query. `getAllForUser` is in the unfortunate position of being a paginated query...

The version of libjxl in the container is probably too old (v0.7.0) for these JXL files.

Hmm, interesting that it worked with djxl 0.7.0. I think that makes it more likely that it's a libvips issue. It's also possible that libvips is only tested against a...

Other than that, if you'd like to try building immich and libvips with a newer version of libjxl: 1. Clone the [immich](https://github.com/immich-app/immich/tree/main) and [base-images](https://github.com/immich-app/base-images) repos and follow the [setup instructions](https://immich.app/docs/developer/setup#server-and-web-apps)...

Glad to hear it's working! There's no particular stability reason for using 0.7.0 - it's just Debian stable installs. We can change this in the base image.