pupil-labs / pupil-invisible-monitor

DEPRECATED - Use https://docs.pupil-labs.com/invisible/how-tos/data-collection-with-the-companion-app/monitor-your-data-collection-in-real-time.html instead.
GNU Lesser General Public License v3.0
10 stars 3 forks source link

Reorder dependency docs and correct windows part #21

Closed pfaion closed 4 years ago

pfaion commented 4 years ago

The Windows section contained outdated links to the old Pupil docs. I copied over the necessary text, so we don't need to link to the Pupil docs anymore.

Additionally, the dependency install instructions came AFTER the install instructions, which normally does not make much sense. So I reordered them.

willpatera commented 4 years ago

@pfaion are there macos setup dependencies included? I would say merge this, but would like to add any macos specific instructions - @papr do you want to contribute these?

pfaion commented 4 years ago

@willpatera good point about the macOS dependencies. @papr looks like you can just install GLFW 3.3 from brew. Can you confirm that this would work for PI Monitor? I can just add a small section with brew install glfw for macOS dependencies.

papr commented 4 years ago

@pfaion As it turns out, pyglfw comes with its own glfw library. Therefore, we can remove the requirements to install glfw via brew

pfaion commented 4 years ago

Looks like we should check again if we need any libglfw then. It looks like the pyglfw package also ships glfw shared libs for Windows and Linux. I'll try this for Windows and Linux and check if we need libglfw at all.

pfaion commented 4 years ago

Note: there's even more to fix! The default script installs pyglui from source, which does not work on Windows. Here we have to install the wheel. Will have to rework the workflow with the requirements.txt.

@papr what was the reason for all of these being editable installs? -> https://github.com/pupil-labs/pupil-invisible-monitor/blob/9324d412f52f82b10fdbd16fca40dbb9615b958c/requirements_custom.txt

pfaion commented 4 years ago

The same ☝️ holds for pyNDSI btw.

papr commented 4 years ago

Editable installs are the only way to add custom dependency urls to your requirements that I know of.

pfaion commented 4 years ago

Closed in favor of #22 which is a lot cleaner.