py-wdf-reader icon indicating copy to clipboard operation
py-wdf-reader copied to clipboard

New test files

Open alchem0x2A opened this issue 3 years ago • 10 comments

It would be of great help if more test wdf files can be included in the repo to debug different WiRE versions / data types. @expert-git

alchem0x2A avatar Mar 19 '21 20:03 alchem0x2A

sorry, because of security, I can't share file right now, but I will try.

expert-main avatar Mar 19 '21 20:03 expert-main

I'm definitely fine with that. feel free to close the issue if it is infeasible.

alchem0x2A avatar Mar 19 '21 20:03 alchem0x2A

Thanks for understanding.

expert-main avatar Mar 19 '21 20:03 expert-main

It would be good to add a test suite too. I should be able to help with sourcing test file - I am not acquiring the data myself but I can ask someone else to provide these files.

ericpre avatar Apr 24 '21 11:04 ericpre

@ericpre Thanks for your help!

alchem0x2A avatar Apr 24 '21 14:04 alchem0x2A

I am going to provide more example files, but couldn't push because of file size limit.

expert-main avatar Jul 07 '21 18:07 expert-main

@alchem0x2A I created new PR and uploaded some example files, but I see there are issues on this repo and would like to update.

expert-main avatar Jul 08 '21 15:07 expert-main

hi @expert-git thanks for your file uploads! To keep the size of our repo I would prefer to only provide the wdf files as optional binary release assets for downloading https://github.com/alchem0x2A/py-wdf-reader/releases/tag/binary.

Currently I'm busy with other personal things so updating will be delayed, I'll let you know once there I merge your test files with the master branch. If you have example codes to add please feel free to update your PR. thanks again!

alchem0x2A avatar Jul 16 '21 05:07 alchem0x2A

Thanks for response @alchem0x2A I have got two problems.

  • MeasurementType in types.py is not correct. In order to compare with .wdf and .spc data, I tried to upload both of two files. spc file was generated by software from .wdf file, so it will be helpful to catch issues what's wrong.
  • extracting spectra data is not accurate one value is missing, so it's necessary for me to contact and share some knowledges.

expert-main avatar Jul 16 '21 14:07 expert-main

@alchem0x2A I would like to let me know when you are available to connect to discuss about errors. Thanks.

expert-main avatar Jul 29 '21 11:07 expert-main