Results 88 comments of Ruslan Dudin

> Right now I have to use LargestFirstMultiasset because RandomImprove is giving `Not enough ADA leftover to include non-ADA assets in a change address` error, @rooooooooob , check it plz...

> I can confirm that the change calculation fails if there are both, native assets AND a minting output in the transaction. Hi, @miguelaeh! Would that be possible for you...

https://github.com/Emurgo/cardano-serialization-lib/pull/520 is fixing how the tx-balance is being asserted during the change-building and makes the lib return proper more informative error messages about the exact cause of the problem. Will...

@hazryder or @AlexDochioiu , would that be possible for you to provide an incorrect transaction hex produced with this? Would help us investigate and reproduce

> That's not possible. The transaction hex/cbor never gets built. UTxO Balance Insufficient is an error thrown when calling txBuilder.add_inputs_from(txUnspentOutputs, 3). @AlexDochioiu , good point. Would that be possible for...

> transaction submit error ShelleyTxValidationError ShelleyBasedEraBabbage (ApplyTxError [UtxowFailure (FromAlonzoUtxowFail (PPViewHashesDontMatch (SJust (SafeHash "cf442fa6f31027353659522764941c71b732702296b6f400c977a4dc5628f872")) (SJust (SafeHash "e6129f50a866d19d95bc9c95ee87b57a9e695c05d92ba2746141b03c15cf5f70"))))]) @AdamMachera , try using the latest available version of the library please, I have...

> const scriptInputIndex = unspentOutput.scriptUtxo.input().index() Irrelevant to the specific error you are getting, but am I understanding correctly that this input index you are getting is the index of the...

> const costModels = TxBuilderConstants.plutus_vasil_cost_models(); > txBuilder.calc_script_data_hash(costModels) > > const txUnspentOutputs = TransactionUnspentOutputs.new(); > const walletOutputs = await cardano.getUtxos(); > walletOutputs.forEach(utxo => txUnspentOutputs.add(utxo)) > txBuilder.add_inputs_from(txUnspentOutputs, 2) Actually, @AdamMachera , I...

Discussed with @lisicky , we gonna try to add an additional validation to fail with an error in case any more inputs are added after the hash already been auto-calculated

> I placed the calculate script hash at the end. Ah, shit,  @AdamMachera, this is my bad, @lisicky is right. My mistake, you need to call the `.calc_script_data_hash` before adding...