BAMWelDX / weldx

The welding data exchange format
https://www.bam.de/weldx
BSD 3-Clause "New" or "Revised" License
19 stars 9 forks source link

update to new Validator interface #863

Closed CagtayFabry closed 1 year ago

CagtayFabry commented 1 year ago

Changes

Related Issues

https://github.com/asdf-format/asdf/issues/1012 https://github.com/asdf-format/asdf/pull/1050

Checks

github-actions[bot] commented 1 year ago

Test Results

2 189 tests  ±0   2 188 :heavy_check_mark: ±0   2m 43s :stopwatch: +2s        1 suites ±0          1 :zzz: ±0         1 files   ±0          0 :x: ±0 

Results for commit efd2615d. ± Comparison against base commit bf5bc70d.

:recycle: This comment has been updated with latest results.

CagtayFabry commented 1 year ago

asdf=2.15.0 is not yet available on conda-forge but the asdf tests pass locally for me

tests passing now with asdf 2.15.0 on conda-forge

codecov[bot] commented 1 year ago

Codecov Report

Merging #863 (efd2615) into master (91140eb) will increase coverage by 0.01%. The diff coverage is 97.77%.

@@            Coverage Diff             @@
##           master     #863      +/-   ##
==========================================
+ Coverage   96.47%   96.48%   +0.01%     
==========================================
  Files          95       94       -1     
  Lines        6238     6230       -8     
==========================================
- Hits         6018     6011       -7     
+ Misses        220      219       -1     
Impacted Files Coverage Δ
weldx/asdf/validators.py 98.24% <97.14%> (+0.10%) :arrow_up:
weldx/asdf/extension.py 100.00% <100.00%> (ø)
weldx/asdf/util.py 90.88% <100.00%> (ø)
weldx/core/math_expression.py 98.59% <100.00%> (+0.08%) :arrow_up:

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.