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

[PR] Biologic file with duplicate columns #26

Merged
merged 8 commits into from
May 4, 2019

Conversation

bcolsen
Copy link
Contributor

@bcolsen bcolsen commented Apr 9, 2019

I know this is a large change, but the data_dtype_dict seem made it much easier to implement. It's also a good way to keep the data out of the logic.

If any column is repeated, I append the column number to the name. This isn't overly elegant but it does assure uniqueness.

Fixes #25

@bcolsen
Copy link
Contributor Author

bcolsen commented Apr 10, 2019

Looks like I need to fix the flags.

I will put a test on this with a small test data file too.

@chatcannon
Copy link
Collaborator

Thanks, I was planning to tidy this up in a similar way myself but never got round to it.

@chatcannon
Copy link
Collaborator

I want to make a few tweaks to this but don't have the time right now.

If you could send me a test file (preferably also with the corresponding .mpt exported by EC-Lab) that would be helpful.

@bcolsen
Copy link
Contributor Author

bcolsen commented Apr 12, 2019

I'll make a simple EIS test file with loops next week. I'll use that to write some tests and I'll send it your way.

@chatcannon
Copy link
Collaborator

If you are adding a file to the repo it would be nice if you could try out git-lfs (see #18). When I first set up this repo, git-lfs wasn't supported by GitHub so I stored the files externally on figshare.com. I would need to check whether it would cause any compatibility problems for people who don't have git-lfs installed, though.

@chatcannon
Copy link
Collaborator

Alternatively just add a file to the discussion here and I can add it to figshare.

@chatcannon
Copy link
Collaborator

@bcolsen I have added the changes I wanted to make, do you have any comments?

Also, would you like to join in as a maintainer on the galvani project? Since I'm not actively using it any more, it would be good to have someone on board who has access to the EC-Lab software and instruments.

@bcolsen
Copy link
Contributor Author

bcolsen commented May 3, 2019

@chatcannon I like the cleanup! The comments about the flags are very helpful.

Also, would you like to join in as a maintainer on the galvani project?

Yes, that sounds great. This is actually an important part of our current data analysis pipeline and I have access to 3 different Biologic units and some Arbin units as well.

Thank you posting this project. I know from experience that reading binary files without proper docs is tricky and these ones are particularly obscure.

I need to make some test files for this too.

@chatcannon
Copy link
Collaborator

chatcannon commented May 3, 2019 via email

@chatcannon chatcannon merged commit a41b40c into echemdata:master May 4, 2019
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.

Duplicate columns in the main data biologic
2 participants