Skip to content

Conversation

@curiecrypt
Copy link
Collaborator

@curiecrypt curiecrypt commented Dec 12, 2024

Content

Documentation of parameter setup for the centralized telescope.

This PR includes:

  • Full documentation of parameter setup covering:
    • Overview
    • Parameter selection strategy
    • Internal parameters
    • Protocol
  • The source file src/centralized_telescope/params.rs is updated to include doc links to related sections.

Pre-submit checklist

  • Branch
    • Commit sequence broadly makes sense
    • Key commits have useful messages
  • PR
    • No clippy warnings in the CI
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested
  • Documentation
    • Update README file (if relevant)
    • Update documentation website (if relevant)

Issue(s)

Blocked by #142
Closes #102

@curiecrypt curiecrypt force-pushed the curiecrypt/doc-centralized-param branch from 747148f to 57b2b07 Compare December 16, 2024 15:56
@curiecrypt curiecrypt marked this pull request as ready for review February 3, 2025 16:27
@curiecrypt curiecrypt force-pushed the curiecrypt/doc-centralized-param branch from 0c5f1b6 to e7a719e Compare February 4, 2025 18:45
rrtoledo
rrtoledo previously approved these changes Feb 10, 2025
Copy link
Collaborator

@rrtoledo rrtoledo left a comment

Choose a reason for hiding this comment

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

LGTM

Added a few suggestions to capitalize b for the DFS bounds

Copy link
Collaborator

@rrtoledo rrtoledo left a comment

Choose a reason for hiding this comment

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

LGTM

@curiecrypt curiecrypt merged commit d90a540 into main Apr 2, 2025
1 check passed
@curiecrypt curiecrypt deleted the curiecrypt/doc-centralized-param branch April 2, 2025 08:50
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.

Documentation - Centralized Telescope: Parameter Setup

4 participants