<div dir="ltr"><div dir="ltr"><br></div><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 12 May 2020 at 01:03, Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com" target="_blank">laurent.pinchart@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello,<br>
<br>
This patch series fixes most of the coding style issues reported by<br>
pycodestyle in the ctt tool. I've left out two remaining issues as they<br>
could be more controversial:<br>
<br>
- Spaces around operators (E226 and 227)<br>
- Usage of docstrings for comments (flagged by E302)<br>
<br>
I can easily fix them if desired.<br>
<br>
Laurent Pinchart (24):<br>
utils: raspberrypi: ctt: Fix pycodestyle E231<br>
utils: raspberrypi: ctt: Fix pycodestyle E201 and E202<br>
utils: raspberrypi: ctt: Fix pycodestyle E203<br>
utils: raspberrypi: ctt: Fix pycodestyle E241<br>
utils: raspberrypi: ctt: Fix pycodestyle E211<br>
utils: raspberrypi: ctt: Fix pycodestyle E251<br>
utils: raspberrypi: ctt: Fix pycodestyle E128<br>
utils: raspberrypi: ctt: Fix pycodestyle E225<br>
utils: raspberrypi: ctt: Fix pycodestyle E228<br>
utils: raspberrypi: ctt: Fix pycodestyle E701<br>
utils: raspberrypi: ctt: Fix pycodestyle E303<br>
utils: raspberrypi: ctt: Fix pycodestyle E261 and E262<br>
utils: raspberrypi: ctt: Fix pycodestyle W605<br>
utils: raspberrypi: ctt: Fix pycodestyle E222<br>
utils: raspberrypi: ctt: Fix pycodestyle E711 and E712<br>
utils: raspberrypi: ctt: Fix pycodestyle E123 and E126<br>
utils: raspberrypi: ctt: Fix pycodestyle E116 and E117<br>
utils: raspberrypi: ctt: Fix pycodestyle E713<br>
utils: raspberrypi: ctt: Fix pycodestyle E721<br>
utils: raspberrypi: ctt: Fix pycodestyle E722<br>
utils: raspberrypi: ctt: Fix pycodestyle W504<br>
utils: raspberrypi: ctt: Fix pycodestyle E741<br>
utils: raspberrypi: ctt: Fix pycodestyle E305<br>
utils: raspberrypi: ctt: Fix pycodestyle E302<br>
<br>
utils/raspberrypi/ctt/ctt.py | 257 ++++++++--------<br>
utils/raspberrypi/ctt/ctt_alsc.py | 146 ++++-----<br>
utils/raspberrypi/ctt/ctt_awb.py | 140 ++++-----<br>
utils/raspberrypi/ctt/ctt_ccm.py | 67 ++--<br>
utils/raspberrypi/ctt/ctt_geq.py | 50 +--<br>
utils/raspberrypi/ctt/ctt_image_load.py | 125 ++++----<br>
utils/raspberrypi/ctt/ctt_lux.py | 19 +-<br>
utils/raspberrypi/ctt/ctt_macbeth_locator.py | 288 +++++++++---------<br>
utils/raspberrypi/ctt/ctt_noise.py | 30 +-<br>
.../raspberrypi/ctt/ctt_pretty_print_json.py | 9 +-<br>
utils/raspberrypi/ctt/ctt_ransac.py | 56 ++--<br>
utils/raspberrypi/ctt/ctt_tools.py | 64 ++--<br>
12 files changed, 641 insertions(+), 610 deletions(-)<br>
<br></blockquote><div><br></div><div>Reviewed-by: David Plowman <<a href="mailto:david.plowman@raspberrypi.com" target="_blank">david.plowman@raspberrypi.com</a>> </div><div><br></div><div>(Can I reply just to this message to cover all 24 individual patches? Thanks...!) <br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
-- <br>
Regards,<br>
<br>
Laurent Pinchart<br>
<br>
</blockquote></div></div>
</div>