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

feat(alerts): Display the number of proofs in a batch from the created task #1727

Open
wants to merge 5 commits into
base: testnet
Choose a base branch
from

Conversation

PatStiles
Copy link
Contributor

Display the number of proofs in a batch from the created task

Description

Currently the alert sender just sends the number of repetitions of sent proofs. This pr changes this to fetch the number of proof submitters in the batch from the proofSubmitters parameter of the createNewTask function call. This parameters contains the addresses of each submitted proof therefore the number of submitters corresponds to the number of proofs in the batch.

To Test:

  • start a local devnet
  • execute the alert script via ./alerts/sender_with_alert.sh ./alerts/.env
    You should observe that the correct number of proofs in a batch is displayed now and observe that if a second senders proofs are included in the batch the number of proofs are displayed. Adding additional task senders should increase the batch size.
16 Proofs in batch of size 16 submitted and verified. Spent amount: 0.000171417922653 ETH ($ 0.57) [ http://localhost:3000/batches/0x0a8dd28a8e7a38151a22b3f04315d17be3a583fdc5fc925d085084b4c8732b27 ]

Type of change

Please delete options that are not relevant.

  • New feature

Checklist

  • “Hotfix” to testnet, everything else to staging
  • Linked to Github Issue
  • This change depends on code or research by an external entity
    • Acknowledgements were updated to give credit
  • Unit tests added
  • This change requires new documentation.
    • Documentation has been added/updated.
  • This change is an Optimization
    • Benchmarks added/run
  • Has a known issue
  • If your PR changes the Operator compatibility (Ex: Upgrade prover versions)
    • This PR adds compatibility for operator for both versions and do not change batcher/docs/examples
    • This PR updates batcher and docs/examples to the newer version. This requires the operator are already updated to be compatible

alerts/sender_with_alert.sh Outdated Show resolved Hide resolved
alerts/sender_with_alert.sh Outdated Show resolved Hide resolved
alerts/sender_with_alert.sh Outdated Show resolved Hide resolved
alerts/sender_with_alert.sh Outdated Show resolved Hide resolved
alerts/sender_with_alert.sh Outdated Show resolved Hide resolved
alerts/sender_with_alert.sh Outdated Show resolved Hide resolved
alerts/sender_with_alert.sh Outdated Show resolved Hide resolved
alerts/sender_with_alert.sh Outdated Show resolved Hide resolved
@PatStiles PatStiles force-pushed the feat/add_batch_size_to_alert_sender branch 2 times, most recently from 413aada to c8887ea Compare January 10, 2025 15:29
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