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

Updating all the colab notebooks #236

Closed
abhinavsp0730 opened this issue Mar 28, 2020 · 8 comments · Fixed by #239, #240, #241, #242 or #243
Closed

Updating all the colab notebooks #236

abhinavsp0730 opened this issue Mar 28, 2020 · 8 comments · Fixed by #239, #240, #241, #242 or #243
Assignees

Comments

@abhinavsp0730
Copy link
Contributor

Problem

As of now, colab is having TensorFlow 2.x by default but Tensorflow Lucid works on Tensorflow 1.x .
But in the colab notebooks, there is no installation of TensorFlow 1.x. So, it might causes errors and It's a big issue.

Solution

I'll love to address this issue by proper documentation and code which implies the installation of TensorFlow 1.x and the Tensorflow Lucid as of now supports only Tensorflow 1.x.
Please assign me this task.

@abhinavsp0730
Copy link
Contributor Author

Hey, @colah would you please assign me this task

@colah
Copy link
Contributor

colah commented Mar 28, 2020

@abhinavsp0730 : A PR on this would be very appreciated!

(1) %tensorflow_version 1.x can be used to get colab to use TF version 1.x
(2) While you are at it, it would be great if notebooks could also be ported to use the most recent version of lucid!

@abhinavsp0730
Copy link
Contributor Author

abhinavsp0730 commented Mar 28, 2020

@colah I'll love to do this task.
(1) I'll do the first task
(2) Would you please tell me how to update all the notebooks for the latest version of lucid.
!pip install --quiet lucid will these do the task?

@colah
Copy link
Contributor

colah commented Mar 28, 2020

Maybe !pip install --quiet --upgrade-strategy=only-if-needed lucid?

There have been some API changes here and there, so some notebooks may break once you do this, and need to be ported to the new API.

@abhinavsp0730
Copy link
Contributor Author

abhinavsp0730 commented Mar 28, 2020

Okay then lemme do the task 1 first .
And for task 2 I'll add this !pip install --quiet --upgrade-strategy=only-if-needed lucid and then I'll run all the cell if it works fine I'll send PR for that notebook.
what are your thoughts in this?

@colah
Copy link
Contributor

colah commented Mar 28, 2020

Sounds great!

@abhinavsp0730
Copy link
Contributor Author

Okay, I'll start doing work in this.

abhinavsp0730 added a commit to abhinavsp0730/lucid that referenced this issue Mar 29, 2020
abhinavsp0730 added a commit to abhinavsp0730/lucid that referenced this issue Mar 29, 2020
abhinavsp0730 added a commit to abhinavsp0730/lucid that referenced this issue Mar 29, 2020
@Abderrahmenee
Copy link

This issue is still not fixed. Colab does not support Tensorflow1.x. So %tensorflow_version 1.x returns error

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.