Skip to content
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

Importerror fixed #237

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Importerror fixed #237

wants to merge 4 commits into from

Conversation

sleepingcat4
Copy link

I have fixed the import error which occured due to recent keras and tensorflow version changes. Since recent tensorflow and keras changes made them two libraries into one and the version checks were pretty outdated, the code was getting into an error loop.

I made changes in the compat and converter files and fixed them adequately.

@drasmuss
Copy link
Member

drasmuss commented Sep 4, 2024

Hi sleepingcat4. First off, thanks for the PR! I just wanted to drop a note since this PR has been waiting for attention for a while. My time available to work on NengoDL has been very limited recently so I haven't had a chance to look into this (or more recent TF versions in general). But we're hoping to get more maintainers on board for this repo. Thanks for your contributions, and patience!

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

Successfully merging this pull request may close these issues.

2 participants