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

File can't be processed due to list of TDW #93

Open
nohelix opened this issue Jan 4, 2024 · 2 comments
Open

File can't be processed due to list of TDW #93

nohelix opened this issue Jan 4, 2024 · 2 comments
Labels

Comments

@nohelix
Copy link
Collaborator

nohelix commented Jan 4, 2024

This has only happened in 1 of the thousands of files and is not an expected or desired stim arrangement. Therefore, I don't see the need to fix it at this time

Issue

File has a list of "Nose Out" instead of a single value. This is an unexpected file setting and should not typically happen.

No fix expected as the data would probably be thrown out as I would rule the day invalid as this should not vary.

Reproduce

Attempt to load file and get error:

Error in unique(run_properties$stim_encoding_table["Nose Out TL (s)"]) %>% :
'list' object cannot be coerced to type 'double'

Teal1_4-32kHz_30-90dB_50ms_8s_TR200ms_NGW2.5s_20220201-144644_BOX#003.mat.zip

Possible Fix

To fix, an extra check would need to be added to main when this is forced into a double. If it is a list of >1 then set it to NA_int otherwise force it to a double.

However, this could/will have downstream effects since it should result in odd naming. Hopefully the name would become NAs, but that would be odd. Ideally if it was NA we would also need to add a check at naming to handle this but I have no idea what I would name it and the data would be lost at that point if the earlier fix was implemented.

@nohelix nohelix added wontfix This will not be worked on at: import/main.R labels Jan 4, 2024
@nohelix
Copy link
Collaborator Author

nohelix commented Jan 4, 2024

This has only happened in 1 of the thousands of files and is not an expected or desired stim arrangement. Therefore, I don't see the need to fix it at this time

@nohelix nohelix closed this as completed Jan 4, 2024
@nohelix nohelix reopened this Jan 4, 2024
@nohelix
Copy link
Collaborator Author

nohelix commented Jan 4, 2024

This appears to be a repeated issue for this rat, so may consider fixing in the future. Very low priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant