IRCaBot 2.1.0
GPLv3 © acetone, 2021-2022
#i2p-dev
/2023/02/04
@eyedeekay
&zzz
+R4SAS
+RN
+T3s|4
+dr|z3d
+hk
+orignal
+postman
+snex
Arch
BravoOreo
Dann
FreefallHeavens_
Irc2PGuest11045
Irc2PGuest3921
Irc2PGuest59134
Irc2PGuest60478
Irc2PGuest62721
Leopold_
Onn4l7h
Onn4|7h
Sleepy_
Soni
T3s|4_
Teeed_
acetone_
aeiou_
aisle
ardu
b3t4f4c3___
b4dab00m
bak83_
bpb
cumlord
dickless
dr4wd3
enoxa
eyedeekay_bnc
hagen_
not_bob_afk
plap
poriori
profetikla
qend-irc2p
radakayot_
rapidash
shiver_
solidx66_
u5657
uop23ip
w8rabbit
weko_
wodencafe2
x74a6h
not_bob zzz: Thank you for fixing the OOM issue and adding a delay in loading torrents with i2psnark!
zzz sure not_bob_afk nothing like an OOM to ruin your day
dr|z3d probably wants to be 60*60*100l
zzz whataboutit?
dr|z3d probably wants to be 60*60*100l
zzz why?
dr|z3d because you changed the stat frequency in ProfileOrganizer?
dr|z3d java.lang.NullPointerException: Cannot invoke "net.i2p.stat.Rate.getCurrentTotalValue()" because "invalidReplyRate" is null
zzz I didnt touch it
zzz ProfileOrganizer.java: Rate invalidReplyRate = invalidReplyRateStat.getRate(30*60*1000l);
dr|z3d ok, my bad, guess your rejigging of the values unturfed something I'd changed, like that rate.
zzz all I did in that commit was move some code around
dr|z3d yeah, not that commit, possibly the one where you junked a bunch of 30m stats.
dr|z3d_ yeah, nevermind then. maybe git.idk.i2p/i2p-hackers/i2p.i2p/-/commit/37beb532454b010eae904c5aca26848d8717c241 did some stuff that indirectly poked at some changes I'd made here.
dr|z3d_ (I removed most of the 30m stats a while back)
dr|z3d_ I see it now.
dr|z3d_ you removed createRates(statGroup);
dr|z3d_ and added: _invalidReplyRate = new RateStat("dbHistory.invalidReplyRate", "How often does this peer give us a bad (nonexistant, forged, etc) peer?", statGroup, new long[] { 30*60*1000l });
dr|z3d_ whereas I'd set that at 60m previously. so no worries.
zzz didnt add it, just moved it
dr|z3d sure, and I missed the change in rate in the merge.
dr|z3d you might disagree, but having a base value of 60s for all graphable stats gives you a common scale for the graphs.
zzz it does, but at a memory/cpu cost
dr|z3d always some tradeoff, but I haven't noticed much in the way of an unreasonable demand on either here, and most of the stats are @ 1m.