| 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 |
Ivan Mincik (imincik) | What we currently maintain is the list of most important geo software. | 18:37:35 |
Ivan Mincik (imincik) |
I'll probably file another pull request to remove myself from teams.geospatial.
Would you please do this change in separate PR ? Thanks.
| 18:44:57 |
das-g | Sure. | 18:46:43 |
das-g | Should I hold off the current one then, until my removal from the team is merged, so that I'm at no point in the history doubly maintainer of the affected packages? | 18:47:31 |
Ivan Mincik (imincik) | In reply to @das-g:matrix.org Should I hold off the current one then, until my removal from the team is merged, so that I'm at no point in the history doubly maintainer of the affected packages? Yes, we should merge your removal from the team . If you are really sure... | 18:48:25 |
das-g | Or I could change the current one to add the team and remove myself from the individual packages, and than re-add myself when I remove myself from the team. (Though that seems like unnecessary noise in the packages' histories.) | 18:48:53 |
das-g | Alright, here's my self-removal from the team: https://github.com/NixOS/nixpkgs/pull/288118 | 18:57:23 |