This repository has been archived by the owner on Apr 10, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 657
Upgrade lucid to use tensorflow v2 APIs and make it work on Colab platform #305
Open
ercaronte
wants to merge
38
commits into
tensorflow:master
Choose a base branch
from
ercaronte:feature/tf2compatibility
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…esting are passing. Tested on python 3.7 only.
All tox testing are passing. Tested on python 3.7 only.
All tox testing are passing for python 3.7 and 3.9 One exception with 3.9 is the test_channel_reducer_trivial that fails, but it seems a precision error.
All tox testing are passing for python 3.7 and 3.9 Fixed some small issues with some tests.
…ther minor fixes.
…ontent Adding missing images that are loaded by lucid notebook
…ivation-atlas-collect) that requires the imagenet dataset. Few other minor fixes.
Few other minor fixes in the lucid core python library.
…ontent Bugfix/missing notebook static content
…g off to later on the necessary fixes or refactoring.
Notes: 3D rendering is broken and pyopengl rendering code works only in colab (does not in kaggle).
… the lucid library.
…ate the sprite maps at this stage. Left as future work.
Upgrade lucid to use tensorflow v2 APIs and make it work on Colab platform
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
While this is being handled by the lucid team, I have merged these changes on my fork https://github.com/ercaronte/lucid. For anybody willing to use the lucid library on Colab and Kaggle feel free to use it by replacing the lucid package install with: |
…ecated old functions that are no longer working.
A bit of an old PR but I noticed that the ReviewNB link is perma-loading. These notebook changes are reviewable in GitNotebooks: https://gitnotebooks.com/tensorflow/lucid/pull/305 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is to update the lucid repository so that it is possible to use it in Colab, now that it does not longer support the magic command
%tensorflow_version 1.x
.I have updated the python library python code and most of the notebooks.
Since the lucid gs bucket is down, I have also fixed some of the links to static content, pointing temporarely to my lucid fork. If the PR is accepted, it will be possible to update those links to point to the official repo.
As a future work to maintain this repo: