Marianne Corvellec
Marianne Corvellec
@alexdesiqueira sure, no worries! Your PR title says 'WIP' indeed :)
Hi @eldad-a, > It seems I do not have permissions to push to the [GitLab repo](https://gitlab.com/scikit-image/data). > Who should I contact on this matter? Please proceed with a regular pull...
PR https://gitlab.com/scikit-image/data/-/merge_requests/13 by @eldad-a is now ready :slightly_smiling_face:
Hmm, the [source file](https://github.com/scikit-image/scikit-image/blob/main/doc/source/api_changes.md) hasn't been edited in two years indeed... and it's not entirely clear to me that each version entry would be a summary of the more detailed...
@moi90 you are right. Would you be able to submit a PR removing `api_changes.md` and all references to it (seeing references to `api_changes.txt` as well, although this file doesn't exist)?...
> So I'm +1 on the currently perceived lazy consensus to use `N, M, P` alphabetically (exception for the channel axis). @lagru I guess you meant `M, N, P` :upside_down_face:...
Hey @lagru!! :wink: Is this PR still relevant? The fourth task (out of four) is not relevant anymore; it may be either checked off or removed.
> more benchmarks don't hurt @lagru definitely! Since there hadn't been any activity in a while (:zzz: Dormant :wink:), I was wondering if maybe it was outdated. I guess this...
Hi @fercer, You are right, the CI failure is unrelated to your PR! It's a much broader issue, reported in several places, notably: https://github.com/pyodide/pyodide/issues/2971 /cc @grlee77 @alexdesiqueira @lagru
@fercer meanwhile would you mind fixing the PEP 8 issues (as reported by pep8speaks)? Thanks :pray: