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

ASA compatible .elc importer. #87

Merged
merged 1 commit into from
Nov 29, 2019
Merged

ASA compatible .elc importer. #87

merged 1 commit into from
Nov 29, 2019

Conversation

sjeung
Copy link
Contributor

@sjeung sjeung commented Nov 29, 2019

Imports both EETrack and asa format eloc files.

Imports both EETrack and asa format eloc files.
@arnodelorme arnodelorme merged commit 3070f3c into sccn:develop Nov 29, 2019
@arnodelorme
Copy link
Collaborator

Actually I had to undo your changes because some files would not import anymore. See the attached file. It should import 63 channels but with your function it only imports 4.

mgh60.elc.zip

@sjeung
Copy link
Contributor Author

sjeung commented Dec 2, 2019

Hello Arno, thank you for your prompt reply. It is because the .elc files I have tested the function with had Labels on one row, not column. I have now implemented a change to first take all elements from rows after and including the row of 'Labels' index and then remove empty cells so that it works for both cases.

@arnodelorme
Copy link
Collaborator

arnodelorme commented Dec 2, 2019 via email

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

Successfully merging this pull request may close these issues.

2 participants