Style/structure questions
See original GitHub issueKymatio deviates afew from PEP8, two things I rather see changed:
- Exceeding 80 characters per line: I have editor on left and console on right, half & half, which is just around 83 characters, so any more and I must scroll horizontally.
- Vertical align, input args: stuff like
compute_params_filterbank(sigma_min, 1, r_psi=r_psi,
alpha=alpha)
the alpha
reads easier when directly under sigma_min
.
Are maintainers okay with having scattering1d/
edited this way?
Issue Analytics
- State:
- Created 2 years ago
- Reactions:1
- Comments:11
Top Results From Across the Web
G11 Masters of Media Pre/Post Quiz: Style/Structure, Concepts Quiz ...
G11 Masters of Media Pre/Post Quiz: Style/Structure, ConceptsDRAFT. 11th - 12th grade. 33 times ... 17 QuestionsShow answers. Question 1. SURVEY. 60 seconds....
Read more >Writing Principles: Style, Structure, and Grammar - MSU Denver
Frequently asked questions ... The student will refresh and refine their writing skills by learning about style, structure, and grammar.
Read more >Deep learning interview questions | Data Science ... - Kaggle
Deep learning interview questions · Why do segmentation CNNs typically have an encoder-decoder style / structure? · What is the significance of Residual...
Read more >Grammar, Style, & Structure: Beyond Grammarly | Wizeprep Blog
The answer to all of these questions includes: content, style, structure, and mechanics. So what are those? These four elements - content, style,...
Read more >Story Style Structure Substance And The Principles Of ...
Recognizing the artifice ways to get this ebook story style structure substance and the principles of screenwriting robert mckee is ...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
p.s. below will spare robos of undue labor via
[skip ci]
in commit title (e.g.added padding [skip ci]
)I’m fully on board with this. That said, we had tentatively agreed on moving to https://black.readthedocs.io/en/stable/ which might follow slightly different conventions. I would be happy with either and would have others weigh in to decide which is used.
The one thing that is important to me is that style changes happen in a separate PR and not entangled with any other contribution.