xorbitsai / xoscar

Python actor framework for heterogeneous computing.
https://xoscar.dev
Apache License 2.0
91 stars 21 forks source link

ENH: do not use global variable for profiling #11

Closed qianduoduo0904 closed 1 year ago

qianduoduo0904 commented 1 year ago

What do these changes do?

Related issue number

Fixes #xxxx

Check code requirements

codecov[bot] commented 1 year ago

Codecov Report

Merging #11 (dea9569) into main (0d7e1ac) will decrease coverage by 43.18%. The diff coverage is 75.00%.

@@             Coverage Diff             @@
##             main      #11       +/-   ##
===========================================
- Coverage   91.02%   47.85%   -43.18%     
===========================================
  Files          41       41               
  Lines        3188     3197        +9     
  Branches      638      328      -310     
===========================================
- Hits         2902     1530     -1372     
- Misses        213     1634     +1421     
+ Partials       73       33       -40     
Flag Coverage Δ
unittests 47.85% <75.00%> (-43.02%) :arrow_down:

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

Impacted Files Coverage Δ
python/xoscar/backends/core.py 25.27% <50.00%> (-70.29%) :arrow_down:
python/xoscar/profiling.py 43.02% <83.33%> (-49.62%) :arrow_down:

... and 29 files 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.