Skip to content
This repository has been archived by the owner on Apr 10, 2024. It is now read-only.

updated left notebooks #245

Closed

Conversation

abhinavsp0730
Copy link
Contributor

@abhinavsp0730 abhinavsp0730 commented Apr 1, 2020

Hy, @colah
I've completed my rest work.
ISSUE: #236

I've done the changes in all of the notebook if by mistake I left any of the notebooks then feel free to mention it here. I'll fix them.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

You'll be able to see Jupyter notebook diff and discuss changes. Powered by ReviewNB.

@abhinavsp0730
Copy link
Contributor Author

Hey, @shancarter would you please review my pull request.

@ludwigschubert ludwigschubert self-assigned this Mar 12, 2021
@ludwigschubert
Copy link
Contributor

@abhinavsp0730 sorry for the delay and thanks so much for your work on this! The diff looks great on reviewNB (https://app.reviewnb.com/tensorflow/lucid/pull/245), so I'd be happy to finally merge these.

Unfortunately it looks like there's a typo in the %tensorflow_version 1.x line; it's missing an 's':
Screen Shot 2021-03-12 at 15 45 07

Did you prepare these changes programmatically and can easily redo them? No worries if not, just let me know and I'll see if I can fix them myself. :-) Thanks again for your willingness to help, and sorry again for not reacting earlier.

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

Successfully merging this pull request may close these issues.

2 participants