-
Notifications
You must be signed in to change notification settings - Fork 0
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
Unable to Load ndx-ophys-devices Namespace to build ndx-microscopy Extension #23
Comments
ndx-ophys-devices does not seem to be available on PyPI. Either it needs to be a Python module that is a dependency of this package (and mentioned in If I clone ndx-ophys-devices and install it locally, then running
so it seems like the namespace is recognized then. |
Ah sorry I forgot to specify that this error occurred running the code in the add_ophys_device branch, not the main branch |
No problem. I guessed that. I see now that you added the GitHub link to ndx-ophys-devices into requirements-min.txt, but it actually needs to be in pyproject.toml or setup.py to be used. requirements-min.txt is only used in a particular GitHub actions workflow that checks whether the tests pass when installing the minimum versions of the packages that you specify in pyproject.toml or setup.py. Both lists need to be manually updated. Sorry that's confusing! I'll try to improve the documentation. Separately you may want to use the latest ndx template for this extension which has a few improvements like having a project.toml which is more standard now. I'm happy to help you migrate this extension over to using that (I do it manually but I've done it enough times for other extensions that it's pretty quick). Let me know. |
But I still have the same issue not finding the namespace |
I am trying to build the ndx-microscopy extension, which requires the namespace of another extension, ndx-ophys-devices. However, I encounter an error indicating that the ndx-ophys-devices namespace cannot be loaded.
Error Details
When running the tests, the following error occurs:
Code
ndx-microscopy/spec/ndx-microscopy.namespace.yaml
Lines 10 to 14 in 65ba204
ndx-microscopy/src/pynwb/ndx_microscopy/__init__.py
Lines 22 to 23 in 65ba204
@rly could you help identify why the ndx-ophys-devices namespace is not being recognized?
After merging this PR I would like to work on the integration with NWBEP003
The text was updated successfully, but these errors were encountered: