Skip to content

ir: move basic income to a new container reports#3550

Merged
cthulhu-rider merged 2 commits intomasterfrom
feat/ir-new-estimations
Aug 29, 2025
Merged

ir: move basic income to a new container reports#3550
cthulhu-rider merged 2 commits intomasterfrom
feat/ir-new-estimations

Conversation

@carpawell
Copy link
Member

Transfer assets according to new reports from
nspcc-dev/neofs-contract#507. Simplify processor's logic. Do not use a separate "bank" account, transfer directly from container owner to storage nodes instead. Pay for exact reported values, do not calculate size estimations. Closes #3053.

@codecov
Copy link

codecov bot commented Aug 28, 2025

Codecov Report

❌ Patch coverage is 0% with 127 lines in your changes missing coverage. Please review.
✅ Project coverage is 24.94%. Comparing base (ce013f8) to head (bfdd436).
⚠️ Report is 5 commits behind head on master.

Files with missing lines Patch % Lines
pkg/innerring/processors/settlement/calls.go 0.00% 116 Missing ⚠️
pkg/innerring/innerring.go 0.00% 5 Missing ⚠️
pkg/innerring/processors/settlement/processor.go 0.00% 5 Missing ⚠️
pkg/innerring/processors/settlement/opts.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3550      +/-   ##
==========================================
+ Coverage   24.85%   24.94%   +0.08%     
==========================================
  Files         666      660       -6     
  Lines       49548    49356     -192     
==========================================
- Hits        12315    12310       -5     
+ Misses      36242    36056     -186     
+ Partials      991      990       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@cthulhu-rider cthulhu-rider left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks correct

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
@carpawell carpawell force-pushed the feat/ir-new-estimations branch from b6eeab7 to 057c2e5 Compare August 29, 2025 13:45
Transfer assets according to new reports from
nspcc-dev/neofs-contract#507. Simplify processor's
logic. Do not use a separate "bank" account, transfer directly from container
owner to storage nodes instead. Pay for exact reported values, do not calculate
size estimations. Closes #3053.

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
@carpawell carpawell force-pushed the feat/ir-new-estimations branch from 057c2e5 to bfdd436 Compare August 29, 2025 14:02
@cthulhu-rider cthulhu-rider merged commit 2d36e46 into master Aug 29, 2025
18 of 19 checks passed
@cthulhu-rider cthulhu-rider deleted the feat/ir-new-estimations branch August 29, 2025 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Basic income doesn't follow data distribution

2 participants