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

IndentationError: unexpected indent #29

Open
xgdgsc opened this issue Mar 15, 2017 · 7 comments
Open

IndentationError: unexpected indent #29

xgdgsc opened this issue Mar 15, 2017 · 7 comments

Comments

@xgdgsc
Copy link

xgdgsc commented Mar 15, 2017

When using import ipynb.fs.defs.myipynb, I get an error:

 File "<unknown>", line 81
    alpha_dict[key] = alpha_dict[key].ewm(span=20,adjust=True,min_periods=0,ignore_na=False).mean()
    ^
IndentationError: unexpected indent

The cell is:

for key in alpha_dict:
    alpha_dict[key] = alpha_dict[key].ewm(span=20,adjust=True,min_periods=0,ignore_na=False).mean()

I don' t know why would it try to import a for loop like this.

@AEDWIP
Copy link

AEDWIP commented Apr 14, 2017

I ran into a similar problem. In my case the cell above was markdown and the the last line in the cell was part of an indented list. The first line of the code cell started with 'def'

Adding an extra new line on the bottom of the markdown fixed the problem.

hard to believe its a parsing error given the implementation of ipynb uses the abstract syntax tree.

@alexlouden
Copy link

I had a markdown cell with no trailing newline and the first line of the following cell wasn't being executed - looks like the same issue.

I was able to reproduce here: https://gist.github.com/alexlouden/4d8cae9643b9118e4c3c23d0b44e12d7

You can see that only Line2 is printed by notebook2

@yuvipanda
Copy link
Collaborator

I'm considering getting rid of the 'defs' feature completely, since I am unsure if we can ever implement it in a way that's consistent and safe in all cases...

Would you all be able to use the 'full' import without problems?

@alexlouden
Copy link

alexlouden commented Oct 23, 2017

I was actually using full - see my gist above (using the latest PyPi release - ipynb==0.4)

@alexlouden
Copy link

Just tried version 0.5 from Github and looks like it's fixed - thanks!

Releasing it to PyPi would be great, if you can?

@yuvipanda
Copy link
Collaborator

yuvipanda commented Oct 23, 2017 via email

@alexlouden
Copy link

It's working, thanks!

❯ pip install ipynb
Collecting ipynb
  Downloading ipynb-0.5.1-py3-none-any.whl
Installing collected packages: ipynb
Successfully installed ipynb-0.5.1

tonyfast added a commit to deathbeds/ipynb that referenced this issue Jan 2, 2020
* Create a watchdog trick for modules that is sensitive to notebook paths Fixes ipython#28
* Catch an error when a file is created in the watcher
* Use the watcher while developing importnb
* Rename the unittests to something more canonical.
* Add the github pages deployment to travis.
* Fix the watcher instructions in the readme.
* Add the watchdog requirement for the setup
* Manually install pyyaml from to pass 3.7dev
* Add context to the readme
* Deploy pages on any travis_branch
* Escape the profile read error int he ipython extension
* add pip to the cache on travis
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

4 participants