jitsi / skynet

AI core services for Jitsi
Apache License 2.0
23 stars 7 forks source link

deps: update llama-cpp-python #77

Closed saghul closed 3 months ago

saghul commented 3 months ago

@quitrk I merged as-is because moving tobool to utils would mean env.py imports utils.py, which in turn imports env.py leading to import errors because the module is not fully initialized.

We can probably reorganize that a bit, but IMHO not worth it for a single function. If things get more hairy we can of course separate things more semantically. For example, app_uuid doesn't really belong in env, but there is no other good place for it, maybe utils? I hate utils modules, them being a mixed bag of everything...