| 14 Jan 2024 |
 nh2 | In reply to @imincik:matrix.org
nh2: there is also pdal upgrade ready in https://github.com/NixOS/nixpkgs/pull/273948 merged | 11:44:18 |
|  Kai joined the room. | 19:21:03 |
| 15 Jan 2024 |
 Ivan Mincik (imincik) | qgis and qgis-ltr are currently both broken in master. | 17:10:30 |
| 19 Jan 2024 |
|  @kidanger:kidanger.net joined the room. | 16:28:06 |
| @kidanger:kidanger.net | Hi all!
Small question, a bit related to the proj-data MR, about pyproj.
Because of this patch https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/python-modules/pyproj/001.proj.patch, it is not possible to override the proj data dir through the env var PROJ_DATA. internal_datadir is checked first (well, after the user data), and with the patch the path is always present on the disk and hardcoded.
On way around that is to call pyproj.datadir.set_data_dir from python, but it is inconvenient to change the code because of nix, and also multiprocessing would make things very ugly, as it always does.
Do you think we could adjust the patch to change the search priorities? or should it be changed upstream? | 18:22:13 |
| Ivan Mincik (imincik) |
Do you think we could adjust the patch to change the search priorities?
Sure, we can change the patch. Do you have some suggestions ?
| 18:29:04 |
| Ivan Mincik (imincik) | *
Do you think we could adjust the patch to change the search priorities?
Sure, we can change the patch if it makes a sense. Do you have some suggestions ?
| 18:29:13 |
| @kidanger:kidanger.net | maybe this https://sebsauvage.net/paste/?c8d8b4d37cf621c1#DsbOMlBf2Y9FApjPiA9rXSSLOc1ntPEn7eJLwKSXVSU=
I've just swapped the conditions in datadir.py (and compared to nixpkgs' patch, I had to update the setup.py "patch context") | 18:39:19 |
| @kidanger:kidanger.net | I'll test this for the project I'm packaging, as soon as I find how to override the patch | 18:40:26 |
| Ivan Mincik (imincik) | You can try something like this
nix-shell -p 'proj.overrideAttrs (_: _: { patches = [(fetchpatch {url = "<PATCH-URL>"; hash = "";}) ]; })'
| 19:02:10 |
| Ivan Mincik (imincik) | Sorry, change proj to python3Packages.pyproj | 19:03:23 |
| @kidanger:kidanger.net | it's all good I have something working (tests in nix, with some srtm tiles and proj data fetched for the tests), after so many hours spent today :( | 19:03:39 |
| Ivan Mincik (imincik) | I am happy to review your PR once you have something working. | 19:04:45 |
| @kidanger:kidanger.net | I can PR the patch above yes; but I'm not 100% confident it fits inside the ecosystem, so experts eyes are welcome! And it would be nice to understand what it means for the proj-data PR too. | 19:06:45 |
| Ivan Mincik (imincik) | BTW, there is related PR introducing proj-data and integrating to proj - https://github.com/NixOS/nixpkgs/pull/280062 | 19:07:28 |
| @kidanger:kidanger.net | yes that's what I was referring to with "the proj-data PR" | 19:12:07 |
| Ivan Mincik (imincik) | Ah, OK. Sorry. Now it makes a sense. | 19:19:38 |
| Ivan Mincik (imincik) | You said proj-data MR instead of PR. I was wondering what does it mean :). | 19:21:22 |
| @kidanger:kidanger.net | yes MR is in gitlab, PR in github :) | 19:27:48 |
| @kidanger:kidanger.net | there it is https://github.com/NixOS/nixpkgs/pull/282139 | 21:07:53 |
| 22 Jan 2024 |
|  maka_77x joined the room. | 03:04:12 |
| 29 Jan 2024 |
| Ivan Mincik (imincik) | https://discourse.nixos.org/t/geospatial-nix-flake-and-geospatial-nix-web-ui/38974 | 11:32:40 |
| 1 Feb 2024 |
|  Matthew Pickering joined the room. | 00:09:52 |
| Matthew Pickering | Good evening, I have just opened a PR to fix an issue with the gdal packaging. https://github.com/NixOS/nixpkgs/pull/285391 | 00:10:23 |
| Matthew Pickering | thank you for your work maintaining the geospatial packages in nix | 00:10:34 |
| Matthew Pickering | I am still having issues with gdal2tiles.py after upgrading but I am not sure if they are nix related or not. I will keep everyone updated. | 00:12:21 |
| Ivan Mincik (imincik) |
Good evening, I have just opened a PR to fix an issue with the gdal packaging. https://github.com/NixOS/nixpkgs/pull/285391
Thanks for opening the PR. I am wondering how tests where passing if we where missing numpy. I'll have a look.
| 06:40:01 |
| Matthew Pickering | It seems that gdal2tiles.py does actually work somehow with/without this change | 10:08:35 |
| Matthew Pickering | I was getting other errors due to some stale files it seemed but it is working now | 10:09:04 |
| Ivan Mincik (imincik) | Yes, but with your PR included, it is not complaining about missing numpy. I was checking wrapped files before and after your PR and before there was no numpy on PYTHONPATH of gdal2tiles.py. So PR might make a sense | 10:11:54 |
