TimeSeries

[vhirtham]

No description provided.

[codecov]

Codecov Report

Merging #76 into dev_asdf will decrease coverage by 1.34%.
The diff coverage is 98.53%.

@@             Coverage Diff              @@
##           dev_asdf      #76      +/-   ##
============================================
- Coverage     99.40%   98.06%   -1.35%     
============================================
  Files            59       64       +5     
  Lines          2693     2944     +251     
============================================
+ Hits           2677     2887     +210     
- Misses           16       57      +41     

Impacted Files	Coverage Δ
weldx/asdf/tags/weldx/core/__init__.py	100.00% <ø> (ø)
weldx/utility.py	98.85% <92.00%> (-1.15%)	⬇️
weldx/core.py	99.32% <99.32%> (ø)
weldx/__init__.py	100.00% <100.00%> (ø)
...dx/asdf/tags/weldx/core/mathematical_expression.py	100.00% <100.00%> (+15.15%)	⬆️
weldx/asdf/tags/weldx/core/time_series.py	100.00% <100.00%> (ø)
weldx/asdf/validators.py	78.72% <0.00%> (-14.08%)	⬇️
weldx/transformations.py	100.00% <0.00%> (ø)
weldx/asdf/tags/weldx/core/iso_groove.py	100.00% <0.00%> (ø)
... and 10 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 46d51b4...723561f. Read the comment docs.

[pep8speaks]

Hello @vhirtham! Thanks for updating this PR.

• In the file weldx/asdf/tags/weldx/core/transformations/rotation.py:

Line 72:39: F541 f-string is missing placeholders

Comment last updated at 2020-07-27 09:00:22 UTC

[CagtayFabry]

I will do some testing if pint integration is improved with xarray 0.16.0 in this case

[vhirtham]

TODO

• enum in schema
• data_array accessor

[vhirtham]

Reminder:

• discuss the unit parameter of interp_time
• remove interpolation attribute from the result of interp_time
• isclose utility function for quantities