!QhvgabMQzwEQeWehhZ:lossy.network

NixOS Home Automation

512 Members
Declarative Home Automation and other Sidequests | https://wiki.nixos.org/wiki/Home_Assistant133 Servers

Load older messages


SenderMessageTime
17 Dec 2024
@hexa:lossy.network@hexa:lossy.networkhttps://github.com/NixOS/nixpkgs/pull/36573000:11:10
@hexa:lossy.network@hexa:lossy.networkinteracting with this ecosystem for these kinds of issues is pretty much pointless00:11:27
@ibizaman:matrix.orgibizamanThank you! And sorry for the circumvented way to get to a fix.00:17:46
@hexa:lossy.network@hexa:lossy.network
 - PyChromecast==14.0.4 not satisifed by version 14.0.5
 - pycryptodome==3.21.0 not satisifed by version 3.20.0
 - py-opensonic==5.1.1 not satisifed by version 5.2.1
 - plexapi==4.15.16 not satisifed by version 4.16.0
 - radios==0.3.1 not satisifed by version 0.3.2
 - soco==0.30.5 not satisifed by version 0.30.6
 - defusedxml==0.7.1 not satisifed by version 0.8.0rc2
 - tidalapi==0.8.0 not satisifed by version 0.7.6
 - ytmusicapi==1.8.1 not satisifed by version 1.8.2
 - yt-dlp==2024.10.7 not satisifed by version 2024.12.6
00:22:06
@hexa:lossy.network@hexa:lossy.networkso basically all pins are outdated 🥳00:22:21
@hexa:lossy.network@hexa:lossy.network * so basically all pins are off 🥳00:23:07
@hexa:lossy.network@hexa:lossy.networkso many problems to resolve and only so much time in a day00:23:42
@hexa:lossy.network@hexa:lossy.networkI think we should relax the check in mass and depend only fail on too old versions00:27:17
@hexa:lossy.network@hexa:lossy.network * I think we should relax the check in mass and only fail on too old versions00:29:19
@mjm:midna.devmjmi have this patch for music-assistant in my config: https://git.midna.dev/mjm/nix-config/-/blob/main/packages/music-assistant/ma-no-install.patch?ref_type=heads felt a little too gross for me to want to send to nixpkgs but i wasn't sure a better way03:08:13
@hexa:lossy.network@hexa:lossy.networkthe problem is … i started packaging it because I was curious03:16:23
@hexa:lossy.network@hexa:lossy.networkit didn't work out too well for local files as I had hoped03:16:38
@hexa:lossy.network@hexa:lossy.networksomeone asked for it, so I PRed it03:16:47
@hexa:lossy.network@hexa:lossy.networkand here we are03:16:50
@hexa:lossy.network@hexa:lossy.networkfor home-assistant we ended up making the tests top-level attributes, so that dependency bumps would run them03:22:43
@hexa:lossy.network@hexa:lossy.networkbut music-assistant hardly has any tests for specific providers, just filesystem and jellyfin03:23:02
@hexa:lossy.network@hexa:lossy.networkI'll look into how to best patch out the behavior tomorrow03:24:08
@hexa:lossy.network@hexa:lossy.networkimage.png
Download image.png
03:41:10
@hexa:lossy.network@hexa:lossy.network
diff --git a/music_assistant/server/helpers/util.py b/music_assistant/server/helpers/util.py
index 3d342b73..a395fb46 100644
--- a/music_assistant/server/helpers/util.py
+++ b/music_assistant/server/helpers/util.py
@@ -117,30 +117,11 @@ async def load_provider_module(domain: str, requirements: list[str]) -> Provider
     def _get_provider_module(domain: str) -> ProviderModuleType:
         return importlib.import_module(f".{domain}", "music_assistant.server.providers")
 
-    # ensure module requirements are met
-    for requirement in requirements:
-        if "==" not in requirement:
-            # we should really get rid of unpinned requirements
-            continue
-        package_name, version = requirement.split("==", 1)
-        installed_version = await get_package_version(package_name)
-        if installed_version == "0.0.0":
-            # ignore editable installs
-            continue
-        if installed_version != version:
-            await install_package(requirement)
-
     # try to load the module
     try:
         return await asyncio.to_thread(_get_provider_module, domain)
     except ImportError:
-        # (re)install ALL requirements
-        for requirement in requirements:
-            await install_package(requirement)
-    # try loading the provider again to be safe
-    # this will fail if something else is wrong (as it should)
-    return await asyncio.to_thread(_get_provider_module, domain)
-
+        raise RuntimeError(f"Missing dependencies for provider {domain}.")
 
 def create_tempfile():
     """Return a (named) temporary file."""
03:41:46
@hexa:lossy.network@hexa:lossy.networkso we even get a nice toast 😛03:41:51
@mjm:midna.devmjmyeah that's a little better than what i did :D04:57:08
@mjm:midna.devmjmit was annoying because it seemed like that code was changing a lot when i was trying to make the patch. i was worried it would be hard to maintain but i haven't had to change it so far04:57:43
@hexa:lossy.network@hexa:lossy.networkhttps://www.home-assistant.io/blog/2024/12/16/community-survey-2024/14:03:57
18 Dec 2024
@spacekitteh:matrix.orgspacekittehoh hell yeah, i got the openthread border router building and working with the web frontend finally :D03:32:56
@spacekitteh:matrix.orgspacekittehlmao it's like they know03:35:02
@spacekitteh:matrix.orgspacekittehimage.png
Download image.png
03:35:06
@spacekitteh:matrix.orgspacekitteh i guess i should have read the blog post, they explicitly mention neurodivergency shrugs 03:39:08
@spacekitteh:matrix.orgspacekittehok, a few more tweaks and i think i'll be ready to make a PR for my openthread border router module :305:25:58
@netpleb:matrix.orgnetpleb I am trying to get confirmable notifications working. I have regular notifications working just fine. Do I need to do anything special to enable saving of scripts? I seem to be able to "create" and "save" a script from the confirmable notification blueprint, but then the script does not appear anywhere for me to use. It's like the save button did nothing. Any idea hexa ? 23:44:11
@hexa:lossy.network@hexa:lossy.networkno idea23:53:44

Show newer messages


Back to Room ListRoom Version: 6