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

Update 10_nlp.ipynbto say 'spaces' #613

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

cjon256
Copy link

@cjon256 cjon256 commented Aug 15, 2023

In the videos it is quite clear spaces is what in meant here.

In the videos it is quite clear spaces is what in meant here.
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@cjon256
Copy link
Author

cjon256 commented Aug 24, 2023

I'm kind of confused as to your process, this was apparently used in commit 3982885, but that is not part of any branch in this repository? Whot?

When mean() takes a tuple argument it treats it as a list of axes to reduce, not as a range of axes.
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.

1 participant