-
Notifications
You must be signed in to change notification settings - Fork 69
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
Update of build.py for TensorFlow 2 #10
Comments
To clarify, remove CuDNNLSTM and CuDNNGRU from the "keras.layers import" Attached is the build script as of now. It still has an issue where, when you run the training sequence, you get a "TypeError": '<' not supported between instances of 'function' and 'str'. From what I gathered, it looks like some variable is being declared as a string or not defined as an int. At first I thought this was the "if(bars < window_size):", but even trying to define them as integers and even commenting out the if check didn't work. If I find something I'll update with what I have. |
Hi Sir, ....\AppData\Local\Programs\Python\Python38\lib\site-packages\tensorflow\python\util\nest.py:416 flatten
|
SiriluckLor, did you managed to connect the socketserver to the mql5? |
For my case, I connected from socketserver to mql4.
…----------------------------------------------------------------------------
Asst.Prof. Dr. Siriluck Lorpunmanee, Ph.D. (Computer Science)
Data Science and Analytics/Lecturer
Suan Dusit University
295 Rathchasima Rd. Dusit
Bangkok, 10300, Thailand
Mobile: +66-81-3886877, +66-97-1230067
E-mail: ***@***.***, ***@***.***
URL: HPCNET.info
________________________________
From: QLaHPD ***@***.***>
Sent: Tuesday, July 13, 2021 8:29 AM
To: kartikmadan11/MetaTraderForecast ***@***.***>
Cc: SiriluckLor ***@***.***>; Comment ***@***.***>
Subject: Re: [kartikmadan11/MetaTraderForecast] Update of build.py for TensorFlow 2 (#10)
SiriluckLor, did you managed to connect the socketserver to the mql5?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<#10 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AGBVKDAG3RYNOY5PJYMUC63TXOJJBANCNFSM4XD4N4TQ>.
|
Hey everyone,
I was testing this out and found a few changes that need to be made to the build.py script.
Remove:
Substitute:
Same parameters, just change the syntax.
This is just a quick fix and should be rewritten for the new version of tensorflow.
Great work kartikmadan!
The text was updated successfully, but these errors were encountered: