Refactor vms comment handling, add parser for kratos metadata - Githubissues

FAIRmat-NFDI / pynxtools-xps

A pynxtools reader plugin for X-ray photoelectron spectroscopy (XPS) data

https://fairmat-nfdi.github.io/pynxtools-xps/

Apache License 2.0

2 stars 0 forks source link

Refactor vms comment handling, add parser for kratos metadata #63

Closed lukaspie closed 1 week ago

lukaspie commented 3 weeks ago

[x] Refactor how the comments in vamas files (where most vendors write their metadata) are handled
[x] Add handler for metadata from Kratos
[x] update examples and test data
[x] add test for parsing comments in Vamas files for all supported vendors

coveralls commented 3 weeks ago

Pull Request Test Coverage Report for Build 9560912033

Details

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 9431201784:	0.0%
Covered Lines:	20
Relevant Lines:	20

💛 - Coveralls

coveralls commented 3 weeks ago

Pull Request Test Coverage Report for Build 9561457543

Details

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 9431201784:	0.0%
Covered Lines:	20
Relevant Lines:	20

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9682153985

Details

14 of 14 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 9580060695:	0.0%
Covered Lines:	34
Relevant Lines:	34

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9682153985

Details

14 of 14 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 9580060695:	0.0%
Covered Lines:	34
Relevant Lines:	34

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9695016539

Details

14 of 14 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 9580060695:	0.0%
Covered Lines:	34
Relevant Lines:	34

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9695016539

Details

14 of 14 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 9580060695:	0.0%
Covered Lines:	34
Relevant Lines:	34

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9695016539

Details

14 of 14 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 9580060695:	0.0%
Covered Lines:	34
Relevant Lines:	34

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9695016539

Details

14 of 14 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 9580060695:	0.0%
Covered Lines:	34
Relevant Lines:	34

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9695016539

Details

14 of 14 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 9580060695:	0.0%
Covered Lines:	34
Relevant Lines:	34

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9695122820

Details

14 of 14 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 9580060695:	0.0%
Covered Lines:	34
Relevant Lines:	34

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9695122820

Details

14 of 14 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 9580060695:	0.0%
Covered Lines:	34
Relevant Lines:	34

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9708596154

Details

28 of 28 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 9580060695:	0.0%
Covered Lines:	48
Relevant Lines:	48

💛 - Coveralls

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9708596154

Details

28 of 28 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 9580060695:	0.0%
Covered Lines:	48
Relevant Lines:	48

💛 - Coveralls