| 19 Jan 2024 |
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 |
Matthew Pickering | yes I think it makes sense | 10:13:44 |
Ivan Mincik (imincik) | Running nixpkgs-review on #285391 now. | 11:01:01 |
Ivan Mincik (imincik) | Matthew Pickering: thank you very much for your previous work on geospatial packages as well. | 11:12:58 |
| 11 Feb 2024 |
das-g | https://github.com/NixOS/nixpkgs/pull/288107 | 18:14:07 |
Ivan Mincik (imincik) | In reply to @das-g:matrix.org https://github.com/NixOS/nixpkgs/pull/288107 Hi Raphael, from in person discussion I understand the reason why you want to keep your name visible in maintainers list, but I am not sure if that's a good idea. If we do this, your name will be present twice in maintainers list
nix eval --json .#osmium-tool.meta.maintainers
[{"email":"nixpkgs@raphael.dasgupta.ch","github":"das-g","githubId":97746,"name":"Raphael Das Gupta"},{"email":"ivan.mincik@gmail.com","github":"imincik","githubId":476346,"matrix":"@imincik:matrix.org","name":"Ivan Mincik"},{"email":"mail@nh2.me","github":"nh2","githubId":399535,"matrix":"@nh2:matrix.org","name":"Niklas Hambüchen"},{"email":"nikolasovaskainen@gmail.com","github":"nialov","githubId":47318483,"name":"Nikolas Ovaskainen"},{"email":"rtburns@protonmail.com","github":"r-burns","githubId":52847440,"name":"Ryan Burns"},{"email":"sikmir@disroot.org","github":"sikmir","githubId":688044,"keys":[{"fingerprint":"ADF4 C13D 0E36 1240 BD01 9B51 D1DE 6D7F 6936 63A5"}],"name":"Nikolay Korotkiy"},{"github":"willcohen","githubId":5185341,"name":"Will Cohen"},{"email":"nixpkgs@raphael.dasgupta.ch","github":"das-g","githubId":97746,"name":"Raphael Das Gupta"}]
I am not sure if that's OK.
| 18:25:27 |
das-g | I'll probably file another pull request to remove myself from teams.geospatial. I've underestimated how many geospatial packages there are in NixPkgs about which I know nothing and thus can't really contribute, yet receive notifications from GitHub. | 18:27:44 |
das-g | Should I put that removal in the same PR, to avoid me being twice in the maintainers lists of those packages= | 18:28:48 |
das-g | * Of should I put that removal in the same PR, to avoid me being twice in the maintainers lists of those packages? | 18:29:01 |
Ivan Mincik (imincik) | Does it mean that you want to hand over the maintenance to us ? | 18:29:36 |
Ivan Mincik (imincik) | And you don't want to maintain them anymore ? | 18:30:05 |
das-g | No, of the packages I already maintain or co-maintain explicitly, I'm ready to stay a maintainer. | 18:30:22 |
Ivan Mincik (imincik) | In reply to @das-g:matrix.org I'll probably file another pull request to remove myself from teams.geospatial. I've underestimated how many geospatial packages there are in NixPkgs about which I know nothing and thus can't really contribute, yet receive notifications from GitHub. Of course, it is your decision. | 18:31:11 |
das-g | It's just that through teams.geospatial I'm now indirectly treated as a maintainer of too many packages I don't care about, so the signal-to-noise ratio of my GitHub notifications ("signal" as in stuff I sufficiently understand and is actionable for me) kinda got too low for my taste. | 18:32:58 |
Ivan Mincik (imincik) | In reply to @das-g:matrix.org It's just that through teams.geospatial I'm now indirectly treated as a maintainer of too many packages I don't care about, so the signal-to-noise ratio of my GitHub notifications ("signal" as in stuff I sufficiently understand and is actionable for me) kinda got too low for my taste. I am sorry for that. I see you are more after maintenance of OSM software. | 18:36:23 |