fix pep 8 for all previously-excluded notebooks

[mingxin-zheng]

Signed-off-by: Mingxin Zheng [email protected]

Fixes #1136 .

Description

The PR expands the scope of PEP 8 checks to cover some notebooks there were previously excluded

Checks

• Notebook runs automatically ./runner --no-run

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[wyli]

please be careful when fixing the new pep8 issues, there's no automated tests to verify the code changes...

[mingxin-zheng]

Hi @wyli , I start to do some manual checks on the notebooks that I touched. The coverage will be limited, and I can see some notebook outside the coverage of our CI/CD is not running correctly, like this one #1139 .

Between fixing the bug in this PR or in another PR, I am inclined to the latter if it is not caused by my changes, so that we can make the PR as atomic as possible.

[wyli]

sure, this looks good to me, if @Nic-Ma doesn't have any other comments let's merge it first.

[Nic-Ma]

Hi @binliunls @mingxin-zheng ,

Will this PR conflict with #1100?
Just a reminder.

Thanks.

[mingxin-zheng]

@binliunls my edits here rearrange some code blocks in video_seg.ipynb to pass PEP 8 checks. If your PR can be merged soon, maybe I can wait and merge yours to mine. The other way also works (merge mine then yours) for me.

[binliunls]

@binliunls my edits here rearrange some code blocks in video_seg.ipynb to pass PEP 8 checks. If your PR can be merged soon, maybe I can wait and merge yours to mine. The other way also works (merge mine then yours) for me.

You can merge first. I will then merge your update to my repo.

[mingxin-zheng]

Thanks @binliunls , I have just merged mine to the main.