Skip to content

Conversation

@marta-lokhova
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings January 16, 2026 00:21
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request introduces a new mission MissionPubnetNetworkLimitsBench that benchmarks the Stellar network's performance under specific transaction size limits, specifically targeting SLP-4 (Stellar Ledger Protocol 4) specifications. The mission simulates the full pubnet topology with custom tx and ledger limits to test network behavior under large transaction scenarios.

Changes:

  • Added MissionPubnetNetworkLimitsBench.fs with a new benchmark mission for testing pubnet network limits
  • Exposed utility functions maxDistributionValue and maxOption from MaxTPSTest.fs for reuse
  • Registered the new mission in StellarMission.fs and FSLibrary.fsproj
  • Added comprehensive documentation in missions.md with required parameters and usage examples

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/FSLibrary/MissionPubnetNetworkLimitsBench.fs New mission implementation with SLP-4 specific limits and benchmark logic
src/FSLibrary/MaxTPSTest.fs Made helper functions public for reuse across missions
src/FSLibrary/StellarMission.fs Added module import and registered new mission
src/FSLibrary/FSLibrary.fsproj Included new mission file in build
doc/missions.md Added documentation with required parameters and examples

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@marta-lokhova marta-lokhova force-pushed the simulationTesting branch 2 times, most recently from f58c715 to ea9da61 Compare January 16, 2026 00:52
@marta-lokhova marta-lokhova merged commit 491774b into stellar:main Jan 16, 2026
3 checks passed
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