Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notebook formatting fails with "syntax errors" #34

Open
meono opened this issue Mar 29, 2022 · 5 comments
Open

Notebook formatting fails with "syntax errors" #34

meono opened this issue Mar 29, 2022 · 5 comments

Comments

@meono
Copy link

meono commented Mar 29, 2022

I'm using Blackconnect with dataspell and I'm getting "BlackConnect: Source code contained syntax errors.". As far as I can tell, there is no syntax error, since black notebook.ipynb works just fine. Has anyone encountered this or can help me figure out wha the issue might be?

Thanks

@lensvol
Copy link
Owner

lensvol commented Mar 29, 2022

Hi! Can you please provide an example of such a notebook? I would be trying to replicate that error on my own, but it would significantly reduce the effort necessary.

And please attach a specific sequence that leads to that error.

Thanks!

@meono
Copy link
Author

meono commented Mar 29, 2022

Sorry for not doing that from the start. the original notebook had stuff I can't share. Anyway, I can reproduce the issue on this one. I suspected there might be problems with comments and markdown cells, so there are a few weird notes.

As for sequence, triggering save (new cell or just cmd+S) seems to do it for me.

Please let me know if you need anything else.

Thanks!

notebook.ipynb.zip

@lensvol
Copy link
Owner

lensvol commented Mar 29, 2022

Thank you! I'll take a look as soon as I can.

@lensvol
Copy link
Owner

lensvol commented Mar 29, 2022

So, I've diagnosed the issue and it is definitely Markdown cells. Jupyter support is currently very spotty and the way our plugin interacts with cells treats Markdown cells as code ones. I'll see about fixing that soon if I can find enough inspiration to do that :)

Also, there is a bug with DataSpell that causes "Tools" submenu to disappear on subsequent runs.

@meono
Copy link
Author

meono commented Mar 30, 2022

Thanks a lot :)
Looking forward to the fix (but I understand, it might not be soon).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants