search

tensorflow / lucid

A collection of infrastructure and tools for research in neural network interpretability.
Apache License 2.0
4.65k stars 655 forks source link

updated left notebooks #245

Closed abhinavsp0730 closed 3 years ago

abhinavsp0730 commented 4 years ago

Hy, @colah I've completed my rest work. ISSUE: https://github.com/tensorflow/lucid/issues/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[bot] commented 4 years ago

Check out this pull request on  ReviewNB

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

abhinavsp0730 commented 4 years ago

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

ludwigschubert commented 3 years ago

@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.