Skip to content
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

Refs #547 -- Add content about using Pyth oracle to obtain random numbers #627

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jackleeio
Copy link
Contributor

@jackleeio jackleeio commented Jun 24, 2024

Pre-flight checklist


This change is Reviewable

Copy link

vercel bot commented Jun 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
conflux-documentation ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2024 1:45pm

Copy link
Collaborator

@darwintree darwintree left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've followed your tutorial and modified the entropyCallback function to see if the code works(https://evmtestnet.confluxscan.io/address/0x378b8745424aa807edea5c25fe18c7af51c1205d?tab=contract-viewer)

    function entropyCallback(
        uint64 sequenceNumber,
        // If your app uses multiple providers, you can use this argument to
        // distinguish which one is calling the app back.
        address provider,
        bytes32 randomNumber
    ) internal override {
        // Implement your callback logic here.
        latestRandomness = uint256(randomNumber);

        // Use the random number
    }

As there is no project using the pyth entropy, currently pyth entropy is not enabled for eSpace mainnet.

I would know if entropyCallback is triggered from your end? If it works, I think it can be merged into the main barnch as reference for future projects, elsewise it might not be a good idea to merge this pull request at present.

But don't worry, no matter this PR will be merged or not, we would grant you corresponding rewards

@jackleeio
Copy link
Contributor Author

In my test contract, entropyCallback is not triggered either.

@jackleeio
Copy link
Contributor Author

It seems to be a problem with Pyth. I made the tutorial based on their documentation. https://docs.pyth.network/entropy/generate-random-numbers/evm

@darwintree
Copy link
Collaborator

It seems to be a problem with Pyth. I made the tutorial based on their documentation. https://docs.pyth.network/entropy/generate-random-numbers/evm

Thanks. Then I'll check with pyth.

@darwintree darwintree added external contributor This label marks the issues connected to Dework Pending PR Not Active PR labels Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external contributor This label marks the issues connected to Dework Pending PR Not Active PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants