-
Notifications
You must be signed in to change notification settings - Fork 447
add pricing for new ByteDance node #5481
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.
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
add pricing for new ByteDance node #5481
Conversation
🎭 Playwright Test Results✅ Tests completed successfully! ⏰ Completed at: 09/10/2025, 05:19:00 PM UTC 📊 Test Reports by Browser
🎉 Click on the links above to view detailed test results for each browser configuration. |
|
can we merge this? |
DrJKL
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One question, but otherwise 👍🏻
| if (maxImages === 1) { | ||
| return '$0.03/Run' | ||
| } | ||
| const cost = (0.03 * maxImages).toFixed(2) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Float math. A dangerous game.
Completely optional, but how would you feel about changing it to an int of cents when multiplying then dividing by 100 at the end for formatting?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am fine with changing that, but I am already afk for today. :(
I took this code somewhere from this file
Summary
Pricing for new upcoming API node. Changes should be back-ported for today's release, afaik.
Number of output images can be controlled by prompt, in cases where we do not know the exact price we display maximum price.
Screenshots (if applicable)
simplescreenrecorder-2025-09-10_20.04.44.mp4
┆Issue is synchronized with this Notion page by Unito