Repository navigation
GSoC 2026: Clarification on linting tools (ruff/pylint) and coverage expectations #2566
|
Hi, I was looking at the linting setup across repositories and had a few questions.
So is the intention to gradually standardize with ruff as the primary linting tool? In that case, can we remove Also I checked the overall coverage of Moreover, are there particular modules where increasing coverage would be considered more valuable than others or overall coverage increment is the goal? |
Replies: 3 comments 2 replies
|
The general goal should be to have ruff for all formatting and then both ruff and pylint for linting. We started switching to ruff but realized it misses too many pylint checks that are relevant and useful to us so we stopped but didn't get back to adding it to arviz-base again. absolutify-imports can be removed if ruff takes care of its tasks yes (that is, not only warns when linting but automatically fixes things). Checking the coverage manually is quite hard for both arviz-base and arviz-stats due to all the different test environments involved. You can check that on codecov instead: https://app.codecov.io/gh/arviz-devs/arviz-base. It is currently 72% I am not sure what to answer a priori. I think the most important task related to coverage is carefully going through the reports on codecov to see if there are any modules or files with much lower coverage than the others. I would also like to try and take a better look at the hypothesis tests we have for arviz-plots to make sure we are covering all relevant cases. Especially for plots, even if the files have a 100% code coverage, it doesn't mean we are covering all important combinations of input parameters. Tangentially related, the organization of tests files and functions in arviz-stats is also quite a mess; if when going over all this finding where each function is tested turns out to be a challenge it would also be a signal we need to think about their organization. |
|
I wanted to confirm the expected scope for docstring standardization and typing. Also,I reviewed the CI workflow in |
|
Hi, I have prepared a draft of my GSoC proposal and I wanted to get your feedback before finalizing it. Proposal (comments enabled): I would really appreciate any suggestions, especially on timeline, and technical approach. |
The general goal should be to have ruff for all formatting and then both ruff and pylint for linting. We started switching to ruff but realized it misses too many pylint checks that are relevant and useful to us so we stopped but didn't get back to adding it to arviz-base again. absolutify-imports can be removed if ruff takes care of its tasks yes (that is, not only warns when linting but automatically fixes things).
Checking the coverage manually is quite hard for both arviz-base and arviz-stats due to all the different test environments involved. You can check that on codecov instead: https://app.codecov.io/gh/arviz-devs/arviz-base. It is currently 72%
I am not sure what to answer a pri…