Skip to content

Conversation

@keegoid-nr
Copy link
Contributor

Updated equations and variables for calculating required CPU cores and job metrics in synthetic monitoring documentation.

  • unify duration variables and specify units as m or s for minutes or seconds
  • clarify rates versus counts with R_ or N_
  • standardizing LaTeX using $\mathrm{...}$ (roman font) for multi-letter variable names like "Parallelism" to distinguish them from single-letter variables.
  • use period instead of frequency
  • add to description for ping runtime scaling

Updated equations and variables for calculating required CPU cores and job metrics in synthetic monitoring documentation.

- unify duration variables and specify units as m or s for minutes or seconds
- clarify rates versus counts with R_ or N_
- standardizing LaTeX using `$\mathrm{...}$` (roman font) for multi-letter variable names like "Parallelism" to distinguish them from single-letter variables.
- use `period` instead of `frequency`
@github-actions
Copy link

Hi @keegoid-nr 👋

Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days.

Please ensure the propsed changes look good by building it first in your local environment. Refer to this contribution guide to get the site up and running in your local.

If you really require a preview url, reach out to one of the writers and they will generate one for you.

@keegoid-nr keegoid-nr marked this pull request as ready for review October 21, 2025 19:01
@Aashirwadjain
Copy link
Contributor

netlify build

@WriteMayur
Copy link
Contributor

netlify build fork

@svc-docs-eng-opensource-bot
Copy link
Contributor

✅ Your PR has been mirrored to our repository as PR #21969.
Commit: 1cf874ce5ed1e96708c39411303ccc4624d54562 (1cf874c)
Our workflows will run in the mirrored PR linked above.
🚀 If the build is successful, a Netlify preview will be available shortly at: https://keegoid-nr-patch-39--docs-website-netlify.netlify.app

@WriteMayur
Copy link
Contributor

Fixed the changes in PR #21975. Hence, closing this PR.

@WriteMayur WriteMayur closed this Oct 24, 2025
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.

4 participants