-
Notifications
You must be signed in to change notification settings - Fork 57
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
python 3.4 issue #1
Comments
I have also encountered the same problem while running this script with the Python 3.5, resulting in the following error:
Even the provided tests crash with the same error. Unfortunately, I think that the developer is not supporting this script anymore, which is a shame. |
Hi. I've opened an issue for the kmaknicki error, before reading your item. |
Hello. You can, as for now, use the other Python ELM module which I have adjusted to work under Python 3: https://github.com/kmachnicki/Python-ELM |
Hi Kamil. Thanks a lot. Do you have any suggestion about a primer introduction to the libs? Roberto Da: Kamil Machnicki [mailto:notifications@github.com] Hello. You can, as for now, use the other Python ELM module which I have adjusted to work under Python 3: https://github.com/kmachnicki/Python-ELM — Nessun virus nel messaggio. |
All documentation about the specific lib is always located with its source, and if not, you can always read the source code to check what exactly every class or function is responsible of and what parameters it expects. Tests (if any) along with the code are also a great way to check how to use the lib. |
Hi @kmachnicki |
I have found a couple of error running the example on a python 3.4 platform.
The main difference is that in python 2.7 dictionary.values() gives a list, instead in 3.4 it is returning a view
So I found out that for fix this issue you have to modify:
-api.py file line 205 -> solution = operator.itemgetter(index)(list(f.call_log.keys()))._asdict()
-cma.py file line 82 -> solution = self.centroid = numpy.array(list(centroid))
Can you fix this error please and better check the python 3.x compatibility ??
Best regards,
Sandro
The text was updated successfully, but these errors were encountered: