-
azy
is there a blockchain.raw for stagenet?
-
azy
is there a regular pruned blockchain.raw anywhere?
-
dEBRUYNE
No and no as far as I know
-
dEBRUYNE
^ azy
-
rating89us-
hi
-
rating89us-
im redesigning the progress bars in monero gui
-
rating89us-
I see that sometimes the blocksToSync can be different from targetBlock - currentBlock
-
rating89us-
is it common?
-
rating89us-
can the difference be large enough?
-
rating89us-
Im asking this because I'd like to leave the 100% width of the progress bar as just targetBlock - currentBlock
-
niocbrrrrrr
rating89us-: there is also #monero-gui
-
niocbrrrrrr
not sure where you should post as I am just a clueless person
-
niocbrrrrrr
truth not troll
-
moneromooo
If targetBlock refers to target height from the daemon, that's a sync mode value. If not in sync node, it's not used. I don't know if it refers to it or not, I just note it has a similar name so it might.
-
rating89us-
nevermind i found a solution, thanks
-
rbrunner
rating89us: Could be that we work accidentally on the very same thing right now :)
-
rbrunner
I finally came around to implement this here, from January:
monero-project/monero-gui #2713
-
rbrunner
Commit on my own repo is here, no PR yet:
rbrunner7/monero-core 2f5095e
-
rbrunner
Do we indeed conflict here?
-
selsta
AFAIK rating89us- is working more on design changes.
-
selsta
rbrunner: your change looks pretty simple and can get merged quickly, design changes will take more time
-
rbrunner
Thanks for the info. Let's see whether rating89us comes back to comment as well ...
-
rbrunner
I was indeed surprised how easy my proposal was to implement. In hindsight should have done that a long time ago
-
selsta
fwiw QML is quite easy to get into
-
rbrunner
Yeah, and the logic with the different 100% for the progress bar really was not that hard
-
rbrunner
And it turned out the control is also quite forgiving all on itself. Feeding it -50 and 200 did not result in anything bad :)
-
rbrunner
(The Qt control itself)
-
rating89us-
I'm doing some changes to both daemon and wallet bar
-
rating89us-
adding the height mark in both progress bars
-
rating89us-
i'll open a pr to show it better
-
rating89us-
ops, issue
-
rbrunner
Alright, will have a look tomorrow to check how it compares with my changes.
-
rating89us-
ok, i'm almost finishing it
-
rating89us-
i guess tomorrow it will be ready
-
rbrunner
Funny how nothing changes with those bars for months and then 2 people get to work at the same time :)
-
moneromooo
woodser: did you bisect that block weight test fail yet ?
-
woodser
no I've been focused on the issue with only account 0 info returning
-
woodser
was wondering if you posted another patch for that while I was disconnected?
-
moneromooo
I posted another based on master, not sure whether yo uwere disconnected at the time. Let me find it...
-
woodser
excellent I was disconnected
-
moneromooo
-
moneromooo
But you said: < woodser_> got it, thanks
-
moneromooo
So I guess this one ain't good either ?
-
woodser
your first patch was not based on master, if I recall. didn't get a second
-
moneromooo
Can you pretend this one is maybe based off master and try it ? :)
-
woodser
yes
-
moneromooo
Or maybe I posted the wrong URL, lemme check quick...
-
woodser
the patch I tested resulted in an out of memory access
-
woodser
trying this one
-
moneromooo
Ah. Either you had not mentioned this, or I missed it.
-
moneromooo
What do you do which I could do here to see if it works ?
-
woodser
all I do is open a wallet that has balances in multiple accounts, and call balance_all (there are other methods too), and it returns only account 0 balance
-
moneromooo
Alright, manual, then I'll do that sometime later.
-
woodser
yeah this is the same patch as yesterday which accessed memory out of bounds
-
woodser
it can also be observed through monero-wallet-rpc, e.g. get_balance