Tx evaluation#378
Conversation
Deploying blockfrost-platform with
|
| Latest commit: |
e4991eb
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://96df9d93.blockfrost-platform.pages.dev |
| Branch Preview URL: | https://feat-tx-evaluation-rebased.blockfrost-platform.pages.dev |
michalrus
left a comment
There was a problem hiding this comment.
Hey, I started reviewing, but then noticed that it's a draft! Left a few comments either way :)
c702f03 to
a53a53c
Compare
3bcc5c5 to
e67b1a9
Compare
6980be7 to
f269525
Compare
|
Maybe let's have a simpler merge commit for the current conflicts? 🥺 Unless you're using |
|
I've just merged |
michalrus
left a comment
There was a problem hiding this comment.
I pushed another merge with main, because there were some more conflicts to resolve. I hope I did that correctly!
Now I see that cargo test tests are failing:
❯ cargo nextest run --workspace --lib --verbose
…
Summary [ 76.650s] 650 tests run: 195 passed, 455 failed, 0 skipped
Stuff like:
FAIL [ 1.058s] blockfrost-platform-error-decoder external::tests::test_cbor_0001_ConwayTreasuryValueMismatch_Coin_Coin
Or:
FAIL [ 1.086s] blockfrost-platform-error-decoder tests::specific::test_cbor_0013_ConwayWdrlNotDelegatedToDRep_KeyHash_KeyHash
|
Maybe the tests are failing because my commits updated |
|
Everything is working related to this PR. The CI failure is caused by updated ledger codebase coming from the |
So what do you suggest to be a solution here? Maybe update (I think we shouldn’t be merging "red" ❌ PRs.) |
|
@ginnun wait, but why did I get the same error in:
I don’t have your changes (the updated So the integration tests would also fail on |
|
Look here:
on 2026-02-19T02:55:30,167769000+00:00. |
The plan is to fix/update the code for |
Great! Here's a tracking issue for this new work: |
I suppose… Hmm, WDYT, @vladimirvolek? This temporary fix is in: |
a4a277d to
dde536f
Compare
6642500 to
1db6c61
Compare
…ulate failure_message
Resolves #306
Stacked on #537 - merge that one first.
Context
Implements tx evaluation APIs compatible with the Blockfrost.io version:
/utils/txs/evaluate/utils/txs/evaluate/utxosWhat is added
bf-tx-evaluatorcrate providing native (Rust, viapallas-validate) and external (Haskell, viatestgen-hs) evaluatorsChainConfigWatchinbf-node- a background task that watches the Cardano node and publishes protocol params + genesis config viatokio::sync::watch, refreshed only at epoch boundariesledger,utxo,chain_config_watchWhat is changed
blockfrost/pallaswithpallas-validate(phase2feature) for script evaluationpallas-hardanoformatting failure)secp256k1-sys), make/awk/diffutils (for integration tests)cardano-nodetrace silencing migrated to the newer APICARGO_PROFILE_DEV_DEBUG=0on integration-test jobs to reduce disk pressureWhat is scoped out (now in #537)
testgen-hssubprocess handling extracted fromcrates/node/src/cbor/into dedicatedbf-testgenandbf-error-decodercratestestgen-hsversion bump to10.6.3.1and release binary downloads switched toblockfrost/testgen-hsbuild_utilstarget detection helpers factored out