Skip to content

Update temporaldata.py#30

Open
omerbe wants to merge 1 commit intoneuro-galaxy:mainfrom
omerbe:patch-1
Open

Update temporaldata.py#30
omerbe wants to merge 1 commit intoneuro-galaxy:mainfrom
omerbe:patch-1

Conversation

@omerbe
Copy link
Copy Markdown

@omerbe omerbe commented Feb 4, 2025

Hi,
I ran into an issue when using the from_hdf5() method. Some of the sessions I loaded from the perich miller dataset have an empty .attrs dictionary which leads to an error. One possible solution I found is to add this line to check if the dictionary is empty.

Hi,
I ran into an issue when using the from_hdf5() method. Some of the sessions I loaded from the perich miller dataset have an empty .attrs dictionary which leads to an error. One possible solution I found is to add this line to check if the dictionary is empty.
@mazabou
Copy link
Copy Markdown
Member

mazabou commented Feb 7, 2025

Thanks for reporting this issue and suggesting a fix! Could you please share the full error log so we can better understand what's going wrong? Is this a particular attribute that is missing in one of these sessions? Additionally, if possible, could you provide details on the specific sessions that are causing this issue, or is it all of them? Thanks!

@mazabou mazabou added the bug Something isn't working label Feb 7, 2025
@omerbe
Copy link
Copy Markdown
Author

omerbe commented Feb 9, 2025 via email

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

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants