search

HEPCloud / decisionengine

HEPCloud Decision Engine framework
Apache License 2.0
6 stars 25 forks source link

Remove unnecessary conversion to Pandas dataframe #535

Closed knoepfel closed 2 years ago

knoepfel commented 2 years ago

The SourceProxy forwards all data products as Pandas dataframes. There is no reason for this restriction, and it has inhibited testing of the SourceProxy. This PR removes such a restriction.

codecov[bot] commented 2 years ago

Codecov Report

Merging #535 (57b76e8) into master (28919b1) will increase coverage by 0.02%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #535      +/-   ##
==========================================
+ Coverage   83.67%   83.70%   +0.02%     
==========================================
  Files          47       47              
  Lines        3081     3080       -1     
  Branches      430      430              
==========================================
  Hits         2578     2578              
  Misses        465      465              
+ Partials       38       37       -1
Flag Coverage Δ
python-3.6 83.00% <100.00%> (-0.01%) :arrow_down:
python-3.9 83.19% <100.00%> (+0.02%) :arrow_up:
python-pypy-3.7 83.26% <100.00%> (+0.05%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...rc/decisionengine/framework/modules/SourceProxy.py 64.70% <100.00%> (-0.52%) :arrow_down:
src/decisionengine/framework/dataspace/maintain.py 99.04% <0.00%> (+0.95%) :arrow_up:

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 28919b1...57b76e8. Read the comment docs.