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

fixed read_csv with sample=False #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

NickMortimer
Copy link

Quick pull request to fix read_csv using sample=False

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@rabernat
Copy link
Member

rabernat commented Jun 1, 2021

Weird, the workflow failed with a binder authorization error:

  File "/opt/hostedtoolcache/Python/3.8.10/x64/lib/python3.8/site-packages/binderbot/binderbot.py", line 111, in start_kernel
    headers = {'Authorization': f'token {self.token}'}
AttributeError: 'BinderUser' object has no attribute 'token'

@yuvipanda - any idea what might be happening here?

@NickMortimer
Copy link
Author

I think it could be to do with me running the notebook locally to test it I think cleared all the outputs and checked it in but I fear it has a memory of something not in binder?

@rabernat
Copy link
Member

rabernat commented Jun 1, 2021

No it has nothing to do with your local environment.

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

Successfully merging this pull request may close these issues.

2 participants