Edouard Choinière

Console reporting even when saving to json file and other questions

3

I'm trying to integrate slipcover instead of a coverage.py, as coverage.py takes an already too long 4*35 minutes without coverage (test suite split into 4 CI jobs), to about 45-50+...

Paired PR of https://github.com/OSGeo/grass/pull/4234. https://github.com/OSGeo/grass/pull/4234 should be merged, then solving the upcoming merge conflict as both PRs added the pathlib import, then this PR can be considered to merge. I...

grass.temporal.stds_import: Use pathlib Path.read_text and Path.write_text for proj string

3

Similar changes for stds_export are not in this PR by purpose. During the weekend, I had made a mistake in stds_export somewhere and didn't find it, but the mismatch in...

python: Use a context manager for opening files (SIM115) to solve some ResourceWarnings

2

This PR starts fixing some ResourceWarnings. I'm starting small, I am juggling with 4 sets of changes that are tested individually and one on top of the others, plus on...

[Bug] Allow periodic update workflow to create a branch

2

## Describe the bug The periodic update workflow, that runs each month, failed today to create a pull request because of branch naming rules' conventions. ## To reproduce 1. See...

## Describe the bug The code in question is: https://github.com/OSGeo/grass/blob/19539757c088a494366eb6381afc699f1064e2fd/scripts/i.spectral/i.spectral.py#L91-L98 Here it's not related to this issue specifically, but I don't undestand why enumerate is used here, but the count...

style: Fix all flake8-gettext (INT) errors (INT001, INT002, INT003)

9

This fixes all instances in python files where translated strings are formatted before being fetched by gettext. (underscore function) Ruff rules: https://docs.astral.sh/ruff/rules/f-string-in-get-text-func-call/ https://docs.astral.sh/ruff/rules/format-in-get-text-func-call/ https://docs.astral.sh/ruff/rules/printf-in-get-text-func-call/ Some places I changed from percent...

style: Fix raise-without-from-inside-except (B904)

3

This PR I want a little bit more attentive review. Ruff rule: https://docs.astral.sh/ruff/rules/raise-without-from-inside-except/ The flake8 bugbear README also refers to the exception chaining tutorial here: https://docs.python.org/3/tutorial/errors.html#exception-chaining, quite useful to understand...

[Bug] Python doctests must be updated for numpy 2+ or some later versions of numpy 1

1

## Describe the bug Doctest running with the gunittest test suite fail with numpy 2.0.1, as the datatype (like int32) doesn't appear when representing the shape of the array. I...

Status of ruff issues and fixes

4

This is a small issue to track the progress and amount of issues to fix with ruff. --- (0): Initial state for 373800152d4f3b70639e2a7b1942f03b06c2c51b, before my first merged fix in #3948:...