!rWxyQqNqMUDLECdsIf:blad.is

Poetry2nix

303 Members
https://github.com/nix-community/poetry2nix55 Servers

Load older messages


SenderMessageTime
6 Nov 2023
@matthewcroughan:defenestrate.itmatthewcroughanAnother annoying thing seems to be that the Cargo.lock is available if you get it from pypi, but not from github 18:19:39
@pareto-optimal-dev:matrix.orgpareto-optimal-dev
In reply to @matthewcroughan:defenestrate.it
Another annoying thing seems to be that the Cargo.lock is available if you get it from pypi, but not from github
In the issue above they were less resistant to adding it to pypi releases.
18:20:09
@pareto-optimal-dev:matrix.orgpareto-optimal-dev
In reply to @matthewcroughan:defenestrate.it
                tiktoken = super.tiktoken.overridePythonAttrs
                  (
                    old: {
                      postPatch = ''
                        ln -s ${./nix/tiktoken/Cargo.lock} Cargo.lock
                       '';
                      cargoDeps = pkgs.rustPlatform.importCargoLock {
                        lockFile = ./nix/tiktoken/Cargo.lock;
                      };
                      nativeBuildInputs = (old.nativeBuildInputs or [ ]) ++ [
                        pkgs.rustPlatform.cargoSetupHook
                        pkgs.rustPlatform.maturinBuildHook
                      ];
                    }
                  );

Do you generate /nix/tiktoken/Cargo.lock yourself?
18:21:07
@matthewcroughan:defenestrate.itmatthewcroughanyes 18:21:11
@matthewcroughan:defenestrate.itmatthewcroughan you go to the source code, you type cargo generate-lockfile, then you have the lock file. 18:21:23
@matthewcroughan:defenestrate.itmatthewcroughanevery single time they update the library, you will have to do that.18:21:35
@matthewcroughan:defenestrate.itmatthewcroughanIf you have 5 dependencies that have this problem, you will have to do it 5 times, and you will have to check it in yourself 18:21:50
@matthewcroughan:defenestrate.itmatthewcroughanWe chose Nix, not because it is easy, but because it is hard.18:22:49
@matthewcroughan:defenestrate.itmatthewcroughanAlthough, again, Nix won't be the only tool that has problems dealing with the lack of specification in Python.18:23:28
@pareto-optimal-dev:matrix.orgpareto-optimal-devThis looks interesting for the cargo problem, but then I'll have to learn the flake parts stuff it seems :D https://github.com/yusdacra/nix-cargo-integration18:23:50
@matthewcroughan:defenestrate.itmatthewcroughan It is just information. Is there enough information in the repo to reproduce something/ 18:23:55
@matthewcroughan:defenestrate.itmatthewcroughan * It is just information. Is there enough information in the repo to reproduce something? 18:23:56
@matthewcroughan:defenestrate.itmatthewcroughanIf there is not enough information in the source code, then even a human will have issues discerning how to reproduce what is inside of it.18:24:11
@matthewcroughan:defenestrate.itmatthewcroughanIt's like OCR, it's error prone because of lack of accuracy18:24:40
@matthewcroughan:defenestrate.itmatthewcroughanI have an issue now with llama-index :(18:30:20
@matthewcroughan:defenestrate.itmatthewcroughan
                llama-index = super.llama-index.overridePythonAttrs
                  (
                    old: {
                      propagatedBuildInputs = (old.propagatedBuildInputs or [ ]) ++ [ self.tiktoken ];
                      nativeBuildInputs = (old.nativeBuildInputs or [ ]) ++ [ super.poetry ];
                    }
                  );

18:30:22
@matthewcroughan:defenestrate.itmatthewcroughan
Traceback (most recent call last):
  File "/nix/store/3g04wving6mlz1vxmnaqxy1wfsyh8g50-python3.11-llama-index-0.8.47/lib/python3.11/site-packages/llama_index/utils.py", line 49, in tokenizer
    import tiktoken
ModuleNotFoundError: No module named 'tiktoken'
18:31:00
@pareto-optimal-dev:matrix.orgpareto-optimal-devIt has poetry.lock at least :)18:31:13
@matthewcroughan:defenestrate.itmatthewcroughanhttps://github.com/imartinez/privateGPT18:31:54
@pareto-optimal-dev:matrix.orgpareto-optimal-devis it because it wants version 0.3.3?18:31:54
@matthewcroughan:defenestrate.itmatthewcroughanI'm building this18:31:57
@pareto-optimal-dev:matrix.orgpareto-optimal-devI packaged a similar one, khoj: https://khoj.dev/ I want to compare them though.18:32:38
@pareto-optimal-dev:matrix.orgpareto-optimal-devOh privateGPT has a RAG/memory similar to memgpt and khoj doesn't IIRC.18:33:34
@matthewcroughan:defenestrate.itmatthewcroughan
In reply to @pareto-optimal-dev:matrix.org
is it because it wants version 0.3.3?
maybe, but it should have killed the build if it didn't have the version
18:34:10
@matthewcroughan:defenestrate.itmatthewcroughanllama-index builds fine, it just doesn't run fine 18:34:17
@pareto-optimal-dev:matrix.orgpareto-optimal-devso... the python packages directory it's being run with doesn't contain tiktoken for some reason?18:34:50
@matthewcroughan:defenestrate.itmatthewcroughanyeah, as a result of mkPoetryApplication 18:35:11
@pareto-optimal-dev:matrix.orgpareto-optimal-devI saw a bug about that in poetry2nix18:35:39
@matthewcroughan:defenestrate.itmatthewcroughan it actually results in a lib dir, maybe I've called the wrong function 18:35:48
@matthewcroughan:defenestrate.itmatthewcroughan
result/lib/python3.11/site-packages/private_gpt
components  constants.py  di.py  __init__.py  __main__.py  main.py  open_ai  paths.py  __pycache__  server  settings  ui  utils
18:36:00

Show newer messages


Back to Room ListRoom Version: 6