Skip to content

Upgrade lucid to use tensorflow v2 APIs and make it work on Colab platform #3

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

Merged
merged 26 commits into from
Jan 26, 2023

Conversation

ercaronte
Copy link
Owner

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:

  • upgrade a handful of notebooks in the misc directory,
  • upgrade the opengl code so that is works in kaggle,
  • complete the migration to tensorflow v2 to remove any dependency to the tf.compat.v1 code.

…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.
…ivation-atlas-collect) that requires the imagenet dataset.

Few other minor fixes.
Few other minor fixes in the lucid core python library.
…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).
…ate the sprite maps at this stage. Left as future work.
@ercaronte ercaronte merged commit 22d78c7 into master Jan 26, 2023
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