-
woodser[m]
-
woodser[m]
I gather the new `burnt` field in on_money_received() represents the amount burned using the burn bug. since on_money_received() fires for each output, would the burnt amount ever be different from the output amount?
-
selsta
-
selsta
this has some explanation
-
woodser[m]
thx
-
jberman[m]
> would the burnt amount ever be different from the output amount?
-
jberman[m]
Receive output A for 1 XMR in tx A, receive output B with the same output pub key as output A at a later date for 5 XMR in tx B. You receive 5 XMR in tx B, 1 XMR is burnt in tx B, and your reflected increase in balance should be 4 XMR
-
jberman[m]
tagging kayabanerve too to sanity check^
-
kayabanerve[m]
jberman: yep!
-
kayabanerve[m]
I think BP+ proving can be made somewhat faster. There was a couple functions defined solely used in verification, notably sum_of_even_powers. BP+ providing independently calculates even_powers, fine, except it does it twice. It didn't feel as tightly knit as BP proving.
-
kayabanerve[m]
I would also like to say I despise the fact hash_cache_mash always returns a Scalar, which is also always the first arg in the next call, yet for some reason, the BP+ transcript DST is a point. It's never used as a point. It's just a point, relying on rct::key's untyped ambiguity, leading to manually having to impl a Point variation in any re-impls D:.
-
yrt
I am not getting which function is creating pseudo output commitment as given in ztm2 genrctsimple() is doing everything I searched in and i found a keyV Pseudooutput is filled with rv.p.pseudooputputs but i am not getting from where this rv.p is getting that value as Ringsig rv is declared inside the function so no chance it was filled by some
-
yrt
other functions nor do it gives its struct as pointer. So who actually filled value in rv.p.pseudooutputs or which function create pseudo commitments.
-
kayabanerve[m]
yrt: They used to be rv.pseudoOuts, but they were moved to rv.p.pseudoOuts as they're prunable. If you're going through the logical process of it, you can treat them as equivalent.
-
kayabanerve[m]
If you're curious the literal place they're filled in, I believe cryptonote_basic
-
kayabanerve[m]
It does most of the data handling for RCT
-
yrt
But in the function genrctsimple() there was nothing like this
-
yrt
Hlo where are you
-
yrt
Output commitment are created by bulletproof_PROVE() but for pseudo i am not getting which function builds them.
-
yrt
And i got nothing in cryptonote_basic i checked it.
-
kayabanerve[m]
There's a function to create commitments called `genC`
-
yrt
Yes actually i got it
-
yrt
Before only but thanks a lot for giving your timing and finding it.
-
kayabanerve[m]
My comment was that pseudoOuts, which I actually think are serialized with the overall structure on second thought, sorry, would be checked in cryptonote_basic, which also moves a few other pieces of data into RCT.
-
yrt
This community is great.
-
kayabanerve[m]
Sorry, was responding to another few messages :p
-
kayabanerve[m]
I re-opened the TX extra discussion and we've had a few comments there...
-
yrt
🤣
-
yrt
Actually when I gone through from where it is getting it's value i found actually it's getting zeros from rv.p.commitments and genC is wht filling the values in it.
-
yrt
Thanks a lot once again for giving your time.
-
Guest57
Hi! Does monero Bulletproofs transactions still will be supported in monero blockchain after Bulletproofs+ release?
-
rbrunner
Don't think so, no. Having to use BP+ is probably part of consensus and enforced, a tx not using it being malformed and getting rejected
-
luigi1112
there should be roughly a day when both are accepted, which allows tx made around fork time to still go through
-
Lyza
everyone remembers that one hard fork when there were still old style TXs in the mempool at the second fork boundary and nodes that were offline at the time couldn't sync, right :3
-
Lyza
cause let's dont do that again lol
-
selsta
Lyza: we have a patch that avoids that this time
-
Lyza
<3
-
kico
sirs while using newest release
-
kico
on unrestricted rpc I see "target_height": 0,
-
kico
on my nodes
-
kico
is this expected?
-
selsta
yes
-
selsta
assuming your node is synced
-
kico
hopefully it is :D
-
kico
thanks selsta
-
selsta
there should be a synchronized true field
-
kico
yes I see it in the logs
-
kico
though was used to seing smth in target_height before
-
kico
no biggie if all is working :P
-
kico
[P2P2] INFO global src/cryptonote_protocol/cryptonote_protocol_handler.inl:2482 You are now synchronized with the network. You may now start monero-wallet-cli.
-
kico
you mean this ^
-
selsta
no, I meant get_info has a synchronized field
-
kico
duh!
-
» kico hides
-
kico
"synchronized": true,
-
kico
tks
-
moneromooo
target_height is valid if you're syncing. If you're not, its value is... not random, but not relevant.
-
kico
ic
-
kico
thanks guys :)
-
plowsof
semi related but i had some confusion today with the wallet rpc call of 'get_height' which hangs indefinitely / only returns the height when the wallet is fully synced