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

Missing test data and CI #48

Open
ml-evs opened this issue Feb 25, 2024 · 5 comments · Fixed by #51 or #63
Open

Missing test data and CI #48

ml-evs opened this issue Feb 25, 2024 · 5 comments · Fixed by #51 or #63
Assignees
Labels
help wanted Extra attention is needed

Comments

@ml-evs
Copy link

ml-evs commented Feb 25, 2024

Related to #47 and an upcoming PR, there are limited contribution instructions, and it is currently impossible to run the integration tests as the test data is not provided in this repo.

Also, there is no continuous integration testing of new PRs (via e.g., GitHub actions) which can go a long way to prevent contributions and also make expectations clear.

I'm happy to help set this up (it is 5 mins work once you've done it before) if you like, but probably having the test data available is a minimum requirement!

@d-cogswell
Copy link
Collaborator

Hi @ml-evs these are good suggestions. The main issue with finishing off CI is I haven't been able to find public data to use. I run a lot of regression tests, but it's on private company data which I'm not able to share publicly.

Any ideas?

@ml-evs
Copy link
Author

ml-evs commented Feb 26, 2024

That's fair! Well, we have some ndax files that can probably be made public (I can ask the owners). There's also .nda files from the other open source neware reader package, whilst it's open it might be nice to reach out to them and ask (and maybe link to each other's packages in the respective READMEs). I'm not imagining an extensive test set (though it's great you've got more testing happening behind the scenes) but just a single example of each file type would be a good start.

I'm also involved in a more meta initiative https://github.com/marda-alliance/metadata_extractors with a registry of file types and extractors that could join some of these dots, happy to add this repo to the list over there unless you object!

@Grimler91
Copy link
Contributor

Hi, I can check with the company I'm working for if they would be open to sharing some data as well.

As @ml-evs said I guess files for each type would be a good start. Thinking larger, do we want to have data for as many step types, current/voltages ranges and instrument models as possible? (Any other parameters/settings that could be important?)

I guess it would preferred to have many small datafiles, rather than having a large one with many different types of steps?

@d-cogswell
Copy link
Collaborator

I plan to start working on CI next, starting with the file from neware_reader. CI will eventually need a collection of nda/ndax files of different versions, and ideally some with auxiliary data, and some without. Code coverage will identify what additional files are needed.

One idea is to generate public test data from different hardware using commercial batteries (NiMH, alkaline?) so that no sensitive company data is revealed. I will ask for permission to do this.

@d-cogswell d-cogswell self-assigned this Mar 6, 2024
@d-cogswell d-cogswell added the enhancement New feature or request label Mar 6, 2024
@d-cogswell
Copy link
Collaborator

I got a Github workflow running and added a publicly available nda file:
https://github.com/Solid-Energy-Systems/NewareNDA/tree/workflow

@d-cogswell d-cogswell linked a pull request Mar 8, 2024 that will close this issue
@d-cogswell d-cogswell linked a pull request May 20, 2024 that will close this issue
@d-cogswell d-cogswell added help wanted Extra attention is needed and removed enhancement New feature or request labels Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
3 participants