Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

ENH try parsing code after cell magic, ignore if can't

See original GitHub issue

I haven’t (yet) got to the root of the issue, but the notebook in question is

https://github.com/microsoft/computervision-recipes/blob/1bb489af757fde7c773e16fab87b24305cff4457/scenarios/detection/20_deployment_on_kubernetes.ipynb

I get

$ nbqa black scenarios/detection/20_deployment_on_kubernetes.ipynb
error: cannot format scenarios/detection/20_deployment_on_kubernetes.ipynb: Cannot parse: 50:1: [print(f) for f in Path(data_path()/"odFridgeObjects"/"models").glob('**/*')]
Oh no! 💥 💔 💥
1 file failed to reformat.

Issue Analytics

State:
Created 3 years ago
Comments:11 (5 by maintainers)

Top GitHub Comments

1reaction

MarcoGorellicommented, Oct 20, 2020

We could do, but then there are cases when cells with cell magics contain valid code, e.g. %%time. Maybe we could do something similar to your idea of what to do with inline magics - check if what follows is valid Python code, and if it is, keep it, else ignore it?

0reactions

MarcoGorellicommented, May 3, 2021

closing as I think the current solution of --process-cells is good enough