Different recommendations for "maxuploadtarget" in docs vs. 13.0rc3 debug.log

[ghost]

Using 13.0rc3
Here
https://github.com/bitcoin/bitcoin/blob/master/doc/reduce-traffic.md
is written, regarding -maxuploadtarget: "The recommended minimum is 144 blocks per day (max. 144MB per day)".

Using -maxuploadtarget=200 leads to the following message in debug.log:
"Max outbound target is very small (209715200 bytes) and will be overshot. Recommended minimum is 576000000 bytes."

The different recommendations should be harmonized.

I would also suggest to consistentley use "MiB". E.g. no "MB".

Instead of "Recommended minimum is 576000000 bytes." I would suggest to write "Recommended minimum is 550 MiB." (rounded up)

Instead of "Max outbound target is very small (209715200 bytes) ..." should be written "Max outbound target is very small (200 MiB) ..."

[jonasschnelli]

Thanks for the report. I'll have a look at that.

[maflcko]

This was missed in 2b1f6f9, should be easy to fix.

[sipa]

Fix for 0.13.0?

[jonasschnelli]

Its just a warning, the miscalculation is a little bit ugly, but I think a fix in 0.13.1 is okay.

[laanwj]

Closed by #8712