trade_aggregation-rs icon indicating copy to clipboard operation
trade_aggregation-rs copied to clipboard

Generic Agg will take a candle factory rather than rely on Default

Open dam5h opened this issue 4 months ago • 0 comments

This adds more flexibility in the types we can aggregate in case we need runtime parameters as part of that logic.

Changes:

  • 🦚 Feature

References:

Changes proposed by this PR:

Removed the trait bound for Default on the generic aggregators candle type, instead the new(...) method will take a zero argument closure.

📜 Checklist

  • [x] The PR scope is bounded
  • [x] Relevant issues and discussions are referenced
  • [x] Test coverage is excellent and passes
  • [x] Tests test the desired behavior
  • [x] Documentation is thorough

dam5h avatar Aug 25 '25 14:08 dam5h