Skip to content
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

windows requires OpalKelly Front-panel to be installed manually #81

Open
hsemwal opened this issue Nov 27, 2024 · 6 comments
Open

windows requires OpalKelly Front-panel to be installed manually #81

hsemwal opened this issue Nov 27, 2024 · 6 comments
Labels
device/stream documentation Improvements or additions to documentation

Comments

@hsemwal
Copy link

hsemwal commented Nov 27, 2024

I set up miniscope-io through github, received the following error messages when I attempted running.

PS C:\Users\Sayum\PycharmProjects\PythonProject.venv\Lib\site-packages> pdm run mio stream capture -c .\miniscope_io\data\config\MSUS.yml
Process _fpga_recv:
Traceback (most recent call last):
File ".\Python313\Lib\multiprocessing\process.py", line 313, in _bootstrap
self.run()
~~~~~~~~^^
File ".\Python\Python313\Lib\multiprocessing\process.py", line 108, in run
self._target(*self._args, **self._kwargs)
~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File ".venv\Lib\site-packages\miniscope_io\stream_daq.py", line 323, in _fpga_recv
dev = self._init_okdev(BIT_FILE)
File ".\venv\Lib\site-packages\miniscope_io\stream_daq.py", line 262, in _init_okdev
dev = okDev()
File ".venv\Lib\site-packages\miniscope_io\devices\opalkelly.py", line 30, in init
raise DeviceOpenError(f"Cannot open device: {serial_id}")
miniscope_io.exceptions.DeviceOpenError: Cannot open device:

However, after I installed Frontpanel from Opalkelly, the FPGA now works. Does this mean that the .dll FPGA drivers did not properly install?

@sneakers-the-rat
Copy link
Collaborator

hmm hmm @MarcelMB / @t-sasatani do you also have to do this when you go to use the FPGA on a new machine?

@sneakers-the-rat
Copy link
Collaborator

sneakers-the-rat commented Dec 3, 2024

We should print the error code there, that would be helpful for debugging lol.

method for GetErrorString - https://github.com/Aharoni-Lab/miniscope-io/blob/87487a1db75d2b59fabfe96dc60653079b0c5f21/miniscope_io/vendor/opalkelly/win/ok.py#L1105

@sneakers-the-rat
Copy link
Collaborator

sneakers-the-rat commented Dec 3, 2024

frontpanel says that windows nees a special driver installed (doesn't have docs for mac or linux, but on mac i can just open frontpanel without installation). https://docs.opalkelly.com/fpsdk/getting-started/

I had just sort of assumed this was working bc it was being used successfully, but if it requires some additional installation we should absolutely vendor that too - we don't want people to have to make an account and download a separate installer and install it, it should work out of the box.

@sneakers-the-rat
Copy link
Collaborator

ok if i extract the windows .exe i find the driver files, but not using windows I don't know how to install them manually. We should

@t-sasatani
Copy link
Collaborator

t-sasatani commented Dec 9, 2024

hmm hmm @MarcelMB / @t-sasatani do you also have to do this when you go to use the FPGA on a new machine?

@MarcelMB works on Mac, and I already had the front panel installed on my Windows PC for different reasons, so not sure if this was necessary. Eventually, it should be boxed, but I think we can just keep this as an issue and say install it from here for now, as we're the only users anyway.

@sneakers-the-rat
Copy link
Collaborator

sneakers-the-rat commented Dec 10, 2024

Agreed - put it in the docs and leave this open as a low priority todo for now

@sneakers-the-rat sneakers-the-rat changed the title Install OpalKelly Front-panel executable to make FPGA work, doesnt work with just mio install windows requires OpalKelly Front-panel to be installed manually Dec 10, 2024
@sneakers-the-rat sneakers-the-rat added device/stream documentation Improvements or additions to documentation labels Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
device/stream documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

3 participants