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.

Sign up for GitHub

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

Jump to bottom

updated get_stimuli.py #84

Open

najam-tariq wants to merge 1 commit into nalmadi:main from najam-tariq:main

najam-tariq commented Jan 22, 2023

As discussed in issue #82


          updared get_stimuli.py

b7cc754

As discussed in issue nalmadi#82

Owner

nalmadi commented Jan 22, 2023

It looks like the change broke the tests, can you check the test that failed and possibly update it, if needed?
Here's the failed test output:

=========================== short test summary info ============================
FAILED test/parsers/test_AlMadi.py::test_AlMadi - assert 0 == 16

where 0 = len(array([], dtype=object))
where array([], dtype=object) = <bound method Series.unique of Series([], Name: trial_id, dtype: object)>()

 where <bound method Series.unique of Series([], Name: trial_id, dtype: object)> = Series([], Name: trial_id, dtype: object).unique

============ 1 failed, 8 passed, 571 warnings in 174.29s (0:02:54) =============
Error: Process completed with exit code 1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet