Skip to content

Partitioning docs #3855

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.

Sign up for GitHub

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

Jump to bottom
Open
Parent: Add partitioning logic
wants to merge 13 commits into
base: 3548-3-partition-functionality
Choose a base branch
from
Open

Partitioning docs #3855

wants to merge 13 commits into from

Conversation

@franciszekjob
Copy link
Contributor

@franciszekjob franciszekjob commented Oct 22, 2025 •
edited
Loading

Closes #3548

Stack:

Introduced changes

Checklist

  • Linked relevant issue
  • Updated relevant documentation
  • Added relevant tests
  • Performed self-review of the code
  • Added changes to CHANGELOG.md

This was referenced Oct 22, 2025
Open
Open
Open
franciszekjob and others added 8 commits October 22, 2025 19:58
franciszekjob
franciszekjob commented Oct 22, 2025
View reviewed changes
docs/book.toml

[preprocessor.variables.variables]
snforge_std_version = "0.51.0"
matrix.partition = "{{ matrix.partition }}"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

note: mdbook recognizes variables inside {{ variable }}, so having {{ matrix.partition }} in example yml workflow will produce en empty string there. Adding this variable is a hacky workaround for it 😅 .

franciszekjob
franciszekjob commented Oct 22, 2025
View reviewed changes
docs/src/snforge-advanced-features/tests-partitioning.md
Copy link
Contributor Author

@franciszekjob franciszekjob Oct 22, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

note: Potentially we can explain more details about partitioning model, that currently only execution is optimized. Lmk what you think.

@franciszekjob franciszekjob marked this pull request as ready for review October 22, 2025 18:32
@franciszekjob franciszekjob requested a review from a team as a code owner October 22, 2025 18:32
@franciszekjob franciszekjob requested review from MKowalski8 and ddoktorski and removed request for a team October 22, 2025 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ddoktorski ddoktorski Awaiting requested review from ddoktorski ddoktorski is a code owner automatically assigned from foundry-rs/starknet-foundry

@MKowalski8 MKowalski8 Awaiting requested review from MKowalski8 MKowalski8 is a code owner automatically assigned from foundry-rs/starknet-foundry

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@franciszekjob