-
Notifications
You must be signed in to change notification settings - Fork 3
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
[JOSS]: Installation instructions #5
Comments
Ah I see, my bad -- I'm checking the system requirements of the GP3 Analysis software, and saw that it only supports Windows. The installation difficulties I'm experiencing could thus be related to my operating system. @RingoHHuang, you are the other reviewer of this submission, could you check whether the installation of the dependencies works smoother for you, or whether you run into similar dependency conflicts? Please also feel free to close this issue afterwards. Thanks a lot! 🙂 |
I'm on Windows 10 Home Edition using Python 3.7.4. I ran into a bit of a hiccup installing asgi_redis. It seems to replace an existing version of asgiref with a version that is incompatible with some of my other Python packages:
|
(mis-clicked close issue from my phone, sorry about that) @adswa - Yes GP3, unfortunately only works for windows-based platforms. I will gather specific dependency version numbers and add them in this thread. It might be also more useful if we built a build file. My initial intent was to dockerize everything, but unfortunately the python portions of the app need to be installed natively, due to the GP3 restriction to windows and docker not providing an efficient mapping to screenspace objects (like screen resolution the ilk) that the GP3 needs access to. @RingoHHuang - Eyestream is written for Python 2.xx (in our install instructions, we suggest 2.7.9, but it should be forwards compatible with any python 2 release). That probably explains the issue you mentioned regarding asgiref. Supporting Python 3.xx should be theoretically possible, but would require a re-write of the underlying communication mechanisms written in Django Channels (which is generating the error message you are getting) and would require us to re-write some of the dependencies we are using. |
I've added some notes to the readme file in 879c518 to clarify that the GP3 only supports windows 8+ and that eyestream is written for Python 2 environments. I will double check the dependency versions and add those as well. |
Hi @MLHale, Thanks for the clarification! My mistake - didn't see that it was for Python 2. I'm starting the installation steps from the beginning using Python 2.7.9 (Windows 10 Home). I've run into two issues:
|
Hi @RingoHHuang, I just reinstalled everything from scratch to see if I could replicate the issues you identified. I was able to replicate the issue. It seems that the problem lies in making an assumption that future versions of channels were backwards compatible. It seems channels releases 2.x.x and up changed the underlying dependency. For this reason, I will update the installation instructions to change the In terms of installation, you can do the following to retry:
I will submit a commit in a moment to change the instructions in the Below is the log of installation steps after I changed the channels dependency to 1.1.8.
|
Added the details I mentioned in commit d80a3b2 Also to answer @adswa 's comment, here is the
|
Thanks @MLHale. I have now installed all the Python packages without any issues. Can you confirm the output below from
|
@RingoHHuang looks good. |
Great, thanks @MLHale. Will close this issue as it is resolved. |
Hi @MLHale,
I'd appreciate assistance on the installation of the dependencies. I'm testing the installation using Debian 10 in a Python 2.7.16 virtual environment.
pypiwin32
. Is this a requirement for Windows users only, or would I need to have it installed to get your software to work?Could you provide me with the version numbers of the dependencies I would need to install? The output of a
pip freeze
would be helpful for a start.Belongs to this JOSS review.
The text was updated successfully, but these errors were encountered: