-
Notifications
You must be signed in to change notification settings - Fork 84
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
Fix/one photon series #1660
Merged
Merged
Fix/one photon series #1660
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## dev #1660 +/- ##
=======================================
Coverage 91.99% 91.99%
=======================================
Files 26 26
Lines 2585 2585
Branches 496 496
=======================================
Hits 2378 2378
Misses 132 132
Partials 75 75
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
CodyCBakerPhD
approved these changes
Mar 10, 2023
This appears to be failing due to some sort of h5py issue in Python 3.8 on windows Traceback (most recent call last):
File "test.py", line 115, in __run_example_tests_helper
_import_from_file(script)
File "test.py", line 63, in _import_from_file
return imp.load_source(os.path.basename(script), script)
File "C:\hostedtoolcache\windows\Python\3.8.10\x64\lib\imp.py", line 171, in load_source
module = _load(spec)
File "<frozen importlib._bootstrap>", line 702, in _load
File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
File "<frozen importlib._bootstrap_external>", line 848, in exec_module
File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
File "docs\gallery\domain\brain_observatory.py", line 41, in <module>
dataset = boc.get_ophys_experiment_data(ophys_experiment_id)
File "D:\a\pynwb\pynwb\.tox\gallery-py38-upgraded\lib\site-packages\allensdk\core\brain_observatory_cache.py", line 554, in get_ophys_experiment_data
return BrainObservatoryNwbDataSet(file_name)
File "D:\a\pynwb\pynwb\.tox\gallery-py38-upgraded\lib\site-packages\allensdk\core\brain_observatory_nwb_data_set.py", line 145, in __init__
meta = self.get_metadata()
File "D:\a\pynwb\pynwb\.tox\gallery-py38-upgraded\lib\site-packages\allensdk\core\brain_observatory_nwb_data_set.py", line 739, in get_metadata
with h5py.File(self.nwb_file, 'r') as f:
File "D:\a\pynwb\pynwb\.tox\gallery-py38-upgraded\lib\site-packages\h5py\_hl\files.py", line 567, in __init__
fid = make_fid(name, mode, userblock_size, fapl, fcpl, swmr=swmr)
File "D:\a\pynwb\pynwb\.tox\gallery-py38-upgraded\lib\site-packages\h5py\_hl\files.py", line 231, in make_fid
fid = h5f.open(name, flags, fapl=fapl)
File "h5py\_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
File "h5py\_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
File "h5py\h5f.pyx", line 106, in h5py.h5f.open
OSError: Unable to open file (file signature not found) |
rly
approved these changes
Mar 14, 2023
Closed
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
What was the reasoning behind this change? Please explain the changes briefly.
How to test the behavior?
Checklist
flake8
from the source directory.