Skip to content

alchemy free tier #60

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 28, 2025
Merged

Conversation

ZzzzHui
Copy link
Collaborator

@ZzzzHui ZzzzHui commented May 20, 2025

No description provided.

@miltonjonat
Copy link
Collaborator

Tried it out and I'm still getting the Alchemy errors. Do we need #59 as well to make it work?
Alternatively, maybe we could have some test to check that this is working as intended? @fabiooshiro what do you think?

@ZzzzHui
Copy link
Collaborator Author

ZzzzHui commented May 21, 2025

Do we need #59 as well to make it work?

No we don't.

@miltonjonat
Copy link
Collaborator

Well, then the fix didn't work for me. Were you able to test it?

@fabiooshiro
Copy link
Collaborator

Tried it out and I'm still getting the Alchemy errors. Do we need #59 as well to make it work? Alternatively, maybe we could have some test to check that this is working as intended? @fabiooshiro what do you think?

I think we can go with your idea of having something like BATCH_SIZE or MAX_BLOCK_RANGE.
The env name is the most difficult part...

@miltonjonat
Copy link
Collaborator

Tried it out and I'm still getting the Alchemy errors. Do we need #59 as well to make it work? Alternatively, maybe we could have some test to check that this is working as intended? @fabiooshiro what do you think?

I think we can go with your idea of having something like BATCH_SIZE or MAX_BLOCK_RANGE. The env name is the most difficult part...

The Node is going to create that variable in some upcoming PR. That's not what I meant: I was asking if we could create a test for this specific scenario. As it stands, I understand @ZzzzHui ported the code change but wasn't able to test if it worked.

@miltonjonat
Copy link
Collaborator

@ZzzzHui you should be able to reproduce my test case (on Decaf) with the following parameters:

  • Consensus address: 0xDa3d0547bB1AAa4feBb7599b8ad1871B15dF45bb
  • App address: 0x21EF997AEa1A861c349Bd12d643701143495d158
  • Template: echo-dapp

In the rollups-espresso-reader repo, if I have the docker compose up (pointing at Sepolia+Decaf), I can register the app like so:

docker compose exec cartesi_node_espresso cartesi-rollups-cli app register -v \
    -c 0xDa3d0547bB1AAa4feBb7599b8ad1871B15dF45bb \
    -a 0x21EF997AEa1A861c349Bd12d643701143495d158 \
    -n echo-dapp \
    -t applications/echo-dapp/

Hope that helps.

@fabiooshiro
Copy link
Collaborator

Ok, I will restore our decaf e2e test

@miltonjonat
Copy link
Collaborator

miltonjonat commented May 27, 2025

I tested again and @ZzzzHui is right: in my test scenario, InputBox inputs are being read ok with this PR. The issue is that I was using Node v2.0.0-alpha.4, which doesn't have the fix in its EvmReader, and that leads to Alchemy block range errors when checking for outputs. But the input reading part does seem ok.

@ZzzzHui ZzzzHui force-pushed the feature/alchemy-free-tier branch from f24dc21 to 6624aa8 Compare May 28, 2025 05:30
@ZzzzHui ZzzzHui changed the base branch from main to feature/adapt-node-alpha5 May 28, 2025 05:31
@ZzzzHui ZzzzHui merged commit 19463bd into feature/adapt-node-alpha5 May 28, 2025
0 of 2 checks passed
@ZzzzHui ZzzzHui deleted the feature/alchemy-free-tier branch May 28, 2025 05:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants