search

dstl / Stone-Soup

A software project to provide the target tracking community with a framework for the development and testing of tracking algorithms.
https://stonesoup.rtfd.io
MIT License
400 stars 131 forks source link

Directly set arguments when instantiating a class #713

Closed sdhiscocks closed 1 year ago

sdhiscocks commented 1 year ago

This avoids the need to first get the signature and avoid more complex bind method which handles things like positional only and keyword only arguments which aren't relevant.

This offers a small performance improvement.

codecov[bot] commented 1 year ago

Codecov Report

Merging #713 (6f35dc9) into main (11a3271) will increase coverage by 0.00%. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #713   +/-   ##
=======================================
  Coverage   94.58%   94.58%           
=======================================
  Files         172      172           
  Lines        8801     8813   +12     
  Branches     1704     1708    +4     
=======================================
+ Hits         8324     8336   +12     
  Misses        344      344           
  Partials      133      133
Flag Coverage Δ
integration 70.23% <50.00%> (-0.04%) :arrow_down:
unittests 90.74% <100.00%> (+0.01%) :arrow_up:

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

Impacted Files Coverage Δ
stonesoup/base.py 95.30% <100.00%> (+0.28%) :arrow_up:

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