Skip to content

chore: update examples #2461

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
Closed
wants to merge 1 commit into from
Closed

chore: update examples #2461

wants to merge 1 commit into from

Conversation

@NicholasKissel
Copy link
Member

@NicholasKissel NicholasKissel commented May 20, 2025

Changes

@NicholasKissel Graphite App
Copy link
Member Author

NicholasKissel commented May 20, 2025

How to use the Graphite Merge Queue

Add the label merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented May 20, 2025

Deploying rivet with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0ae3f40
Status: ✅  Deploy successful!
Preview URL: https://23ad5076.rivet.pages.dev
Branch Preview URL: https://05-20-chore-update-examples.rivet.pages.dev

View logs

@NicholasKissel NicholasKissel force-pushed the 05-20-chore_update_examples branch from 0ae3f40 to 7e8648f Compare May 20, 2025 21:26
@NicholasKissel NicholasKissel marked this pull request as ready for review May 20, 2025 21:27
@NicholasKissel NicholasKissel force-pushed the 05-20-chore_update_examples branch from 7e8648f to ea6a78e Compare May 20, 2025 21:27
@graphite-app
Copy link
Contributor

graphite-app bot commented May 20, 2025 •
edited
Loading

Merge activity

  • May 20, 9:28 PM UTC: NicholasKissel added this pull request to the Graphite merge queue.
  • May 20, 9:29 PM UTC: CI is running for this pull request on a draft pull request (#2462) due to your merge queue CI optimization settings.
  • May 20, 9:29 PM UTC: Merged by the Graphite merge queue via draft PR: #2462.

greptile-apps[bot]
greptile-apps bot reviewed May 20, 2025
View reviewed changes
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Updated pricing documentation with more granular core allocations and revised cost calculations for different deployment scenarios.

  • Modified core allocations in /site/src/content/docs/pricing.mdx from 1/8th to 1/64th for functions and 1/32nd for actors
  • Updated sample scenarios with more accurate cost calculations (e.g., 1M function calls @ 50ms now costs $0.009)
  • Added clarification that 1/8th core minimum applies specifically to Containers
  • Improved readability of storage pricing examples with clearer formatting

1 file(s) reviewed, 2 comment(s)
Edit PR Review Bot Settings | Greptile

site/src/content/docs/pricing.mdx
Comment on lines +51 to +52
| 1M function calls @ 50ms | Stateless function | 1/64 core | 1M × 50ms = 50M → ÷64 = 0.78125M | $0.009 |
| Agents (Actors) | 500 actors × 30min/day | 1/32 core | 500 × 1800s × 30 = 27B → ÷32 = 843.75M | $9.44 |
Copy link

@greptile-apps greptile-apps bot May 20, 2025

Choose a reason for hiding this comment

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

logic: Core-ms calculation for agents appears incorrect. 500 × 1800s × 30 = 27M (not 27B), which affects the final cost calculation.

Suggested change
| 1M function calls @ 50ms | Stateless function | 1/64 core | 1M × 50ms = 50M → ÷64 = 0.78125M | $0.009 |
| Agents (Actors) | 500 actors × 30min/day | 1/32 core | 500 × 1800s × 30 = 27B → ÷32 = 843.75M | $9.44 |
| 1M function calls @ 50ms | Stateless function | 1/64 core | 1M × 50ms = 50M → ÷64 = 0.78125M | $0.009 |
| Agents (Actors) | 500 actors × 30min/day | 1/32 core | 500 × 1800s × 30 = 27M → ÷32 = 0.84375M | $0.009 |

site/src/content/docs/pricing.mdx
| 1M function calls @ 50ms | Stateless function | 1/64 core | 1M × 50ms = 50M → ÷64 = 0.78125M | $0.009 |
| Agents (Actors) | 500 actors × 30min/day | 1/32 core | 500 × 1800s × 30 = 27B → ÷32 = 843.75M | $9.44 |
| Compute-heavy container (10h) | Full CPU usage | 1 core | 10h = 36,000,000 | $0.40 |
| Storage-heavy app | 10GB storage, 5M reads, 1M writes | - | 10GB × $0.20 = $2.00(5M–1M) reads = $0.80(1M–1M) writes = $0.00 | $2.80 |
Copy link

@greptile-apps greptile-apps bot May 20, 2025

Choose a reason for hiding this comment

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

style: Storage cost calculation formatting needs spaces between components: '$2.00 + (5M–1M) reads = $0.80 + (1M–1M) writes = $0.00'

graphite-app bot pushed a commit that referenced this pull request May 20, 2025
@NicholasKissel
chore: update examples (#2461)
d3916f3 
<!-- Please make sure there is an issue that this PR is correlated to. -->

## Changes

<!-- If there are frontend changes, please include screenshots. -->
@graphite-app graphite-app bot closed this May 20, 2025
@graphite-app graphite-app bot deleted the 05-20-chore_update_examples branch May 20, 2025 21:29
@github-actions github-actions bot mentioned this pull request May 20, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@NicholasKissel