Opened 10 years ago
Last modified 10 years ago
#13009 confirmed Bug
[Notification Aggregator] Counting _doneWeights is over-complicated — at Version 1
Reported by: | Piotr Jasiun | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | General | Version: | 4.5.0 Beta |
Keywords: | Cc: |
Description (last modified by )
Working on #12955 we agreed that, since counting _doneWeights
and _totalWeights
is pretty simple and can be stored in redundant variables, counting _doneWeights
is over-complicated and can be a reason of the future bugs. Tasks should give its done wights to aggregator and it should could sum of them, when it is needed.
Also with the current implementation it is not possible to update with negative weight, for no apparent reason.
Since this changes may causes some changes in API this ticket should be done before releasing 4.5.0.