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

Compatibility with TF2 #3

Open
lely475 opened this issue Dec 9, 2020 · 3 comments
Open

Compatibility with TF2 #3

lely475 opened this issue Dec 9, 2020 · 3 comments

Comments

@lely475
Copy link

lely475 commented Dec 9, 2020

Hi,

Is there any update on when / whether this will be made compatible with TF2?
I'd be really interested in using your clahe with TF2.

Greetings,
Lydia

@RealPolitiX
Copy link
Collaborator

@lely475, we currently don't have plans to upgrade to TF2 by ourselves, but you are welcome to do a pull request. There is a numpy version that you can install (see README), which you can call through TF2.

@bhack
Copy link

bhack commented Jan 15, 2021

We have still an upstream ticket if you want to interact upstream tensorflow/tensorflow#24181

@asullivan42
Copy link

One thing I have found that can help with mclahe update to Tensor Flow 2+ is to replace tf.variable_scope with tf.compat.v1.variable_scope on line 43 of core.py

I'm still troubleshooting the rest of the code for making it updated.

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

No branches or pull requests

4 participants