| 15 Jul 2024 |
 Ivan Mincik (imincik) | In reply to @autra:trancart.eu
done! https://github.com/NixOS/nixpkgs/pull/327419 Great ! Left one comment and we can merge. | 17:01:52 |
| Ivan Mincik (imincik) | autra: are you interested in onboarding session over video call ? | 17:04:34 |
 autra | Ivan Mincik (imincik): updated | 17:07:06 |
| autra | Ivan Mincik (imincik): good idea! I'll see better how I can help this way | 17:07:45 |
| 17 Jul 2024 |
| autra | Ivan Mincik (imincik): when you have build failure on dependant packages (like for geopandas), do you fix them in the same PR? Or do you open subsequent PRs? | 11:47:08 |
| Ivan Mincik (imincik) | In reply to @autra:trancart.eu
Ivan Mincik (imincik): when you have build failure on dependant packages (like for geopandas), do you fix them in the same PR? Or do you open subsequent PRs? Can be fixed in the same PR or in a new one. Doesn't matter too much. If you personally want to fix some of failures reported in geopandas PR it might be better to open a new PR rather then pushing in PR created by me.
We also ignore failures if we know that they are not caused by current PR. For example, packages like bsuite or plotnine are broken very often because of something else.
| 12:15:26 |
| Ivan Mincik (imincik) | You can try to build a failed package in master branch to see if it is PR breaking it or it was already broken. | 12:17:10 |
| Ivan Mincik (imincik) | I would be happy to see geodatasets and libpysal fixed regardless of what caused the failure. | 12:18:39 |
| 19 Jul 2024 |
| autra | Ivan Mincik (imincik): should we adopt folium and osmnx? | 18:25:29 |
| Ivan Mincik (imincik) | In reply to @autra:trancart.eu
Ivan Mincik (imincik): should we adopt folium and osmnx? Yes, I think so. Adoption should contain some QA - package works correctly and is covered by tests. | 19:09:33 |
| 20 Jul 2024 |
| autra | In reply to @imincik:matrix.org
I would be happy to see geodatasets and libpysal fixed regardless of what caused the failure. Are they broken? They seem ok to me. | 12:26:47 |
| 21 Jul 2024 |
 sikmir | Just in case, maybe interesting for geospatial team https://github.com/NixOS/nixpkgs/pull/328819 | 08:36:39 |
| 22 Jul 2024 |
| autra | indeed! I'd split this MR in several smaller one to ease the review though 😅 | 13:12:48 |
| * autra just reviewed a +5000 - 3000 lines PR from a colleague, that might explain my current feeling! | 13:13:26 |
| autra | nice to see parquet format coming to nixpkgs, it'll be useful :-) | 13:22:45 |
| Ivan Mincik (imincik) | I am on family holiday during this week. I will be less responsive than usual. | 14:01:28 |
| Ivan Mincik (imincik) | Shapely update looks good to go. Please review. Thanks. | 15:12:31 |
| 23 Jul 2024 |
| autra | Ivan Mincik (imincik): hi! when a PR is approved, we can ping the merge bot to have it merged right? | 07:37:53 |
|  Ezzobir Bezziou joined the room. | 08:21:45 |
| Ivan Mincik (imincik) | In reply to @autra:trancart.eu
Ivan Mincik (imincik): hi! when a PR is approved, we can ping the merge bot to have it merged right? You can always ping me for merging. | 11:32:22 |
| Ivan Mincik (imincik) | In reply to @autra:trancart.eu
Ivan Mincik (imincik): hi! when a PR is approved, we can ping the merge bot to have it merged right? Merge bot works only for PRs created by ryantm bot. | 11:35:41 |
| Ivan Mincik (imincik) | * AFAIK, Merge bot works only for PRs created by ryantm bot. | 11:36:08 |
| autra | thanks, it's clearer :-) | 11:51:09 |
| 28 Jul 2024 |
|  biscottymatrix joined the room. | 05:17:44 |
| 29 Jul 2024 |
| Ivan Mincik (imincik) | Back from vacation, catching up on work. | 10:01:19 |
| Ivan Mincik (imincik) left the room. | 10:01:48 |
| Ivan Mincik (imincik) joined the room. | 10:14:00 |
| Ivan Mincik (imincik) | Can I ask for review of https://github.com/NixOS/nixpkgs/pull/326291
This build configuration was discussed with major grass developers during community sprint and is in sync with grass upstream packaging.
| 10:16:14 |
| 30 Jul 2024 |
| Ivan Mincik (imincik) | https://github.com/NixOS/nixpkgs/pull/318245 is ready for review | 11:00:51 |
| Ivan Mincik (imincik) | qgis has started to fail on
> configure: error: *** Unable to locate suitable (>=1.7.1) PDAL library.
in master.
| 11:06:12 |
