Clemens Brunner

Results 578 comments of Clemens Brunner

Can you upgrade `mne`? The latest version is 1.0.3, and this includes `tqdm` as a dependency.

Thanks for opening your first issue here @larsoner 😆! I'm reluctant to go all Black. Of course this alleviates us from all style-related issues, but on the other hand we...

> you're so European minded Clemens :) I hope this is a compliment 😄? If you really want all or nothing, I'm going to vote against Black. Can we maybe...

@mmagnuski the benefit is that Black auto-formats everything. The issue is that Black auto-formats everything. So there is no in between if for some reason (readability) you would like to...

Black is not a standard, it's an opinionated formatter. Line lengths can be set even in Black (https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html#line-length), so in my opinion we could set *only* a new line length...

> when is it an issue? Everytime. It's just not enough characters, I almost always have to break it up into multiple lines. Most recent example: https://github.com/mne-tools/mne-python/pull/10638/files With 92 characters,...

@drammock we can have a pre-commit on GitHub, but this doesn't mean that you have to use it locally. I think it's not the end of the world if CIs...

> an opinionated formatter that tons and tons of people use I still think there are more people *not* using it. > and that's also officially hosted on the org...

I don't understand the `Info` type here – what does that have to do with `str` and `pathlib.Path` types?

Ah OK! This makes sense!