-
Notifications
You must be signed in to change notification settings - Fork 30
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
Tensorflow issues #25
Comments
Wait, what code do you use and what TF version do you have? The code in the jupyter notebook uses TF 2.x. There are two versions of the code in the repo - for TF 1.x and TF 2.x, pls make sure that you use the right code with right TF version. PS. I honestly need to get rid of the old code for TF 1.x... |
I've just been using:
python starnet.py transform starrypic.tif
I saw there was a V2 and tried calling:
python starnet_v1_TF2.py transform starrypic.tif
but got no result.
It may well be that I'm using it completely wrong, I'm not really sure
about the jupyter notebooks - I thought they were just documentation of
some kind?
Please can you tell me how I should be calling the new version!
Thanks,
Adrian.
…On Tue, 19 Oct 2021, 16:36 Nikita Misiura, ***@***.***> wrote:
Wait, what code do you use and what TF version do you have? The code in
the jupyter notebook uses TF 2.x. There are two versions of the code in the
repo - for TF 1.x and TF 2.x, pls make sure that you use the right code
with right TF version.
PS. I honestly need to get rid of the old code...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#25 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEHG6X5GGJJX6PKS5RAHWW3UHWGBJANCNFSM5GI22U5Q>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Adrian, Yeah, you are a bit wrong about jupyter notebooks - those are actually a nice way to run python code. You should look it up - long story short you should run 'jupyter notebook' instead of calling python, and then open starnet_v1_TF2_transform.ipynb in the new window that will pop up. It should be clear and straightforward after that! |
Thanks, I see how it works now and I've got it to work. I have to admit
using jupyter is much less convenient than using the command line but using
the python code in the notebooks I think I can see how to modify starnet.py
to use the new code.
I might not get round to it overnight but when I do I'll send it to you for
consideration for the repository, I think it would be really good to have a
command line interface at least for the transform function.
Cheers,
Adrian.
…On Wed, 20 Oct 2021, 20:07 Nikita Misiura, ***@***.***> wrote:
Adrian,
Yeah, you are a bit wrong about jupyter notebooks - those are actually a
nice way to run python code. You should look it up - long story short you
should run 'jupyter notebook' instead of calling python, and then open
starnet_v1_TF2_transform.ipynb in the new window that will pop up. It
should be clear and straightforward after that!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#25 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEHG6XZMBQA22UOL3EGI4K3UH4HP3ANCNFSM5GI22U5Q>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
I edited the python files as you suggested in replying to the other issue I raised and got rid of the toImage errors, unfortunately I now get a variety of tensorflow errors instead that I can't make go away by importing tensorflow.v1.compat and disabling v2 features.
The text was updated successfully, but these errors were encountered: