Skip to content

Conversation

@djayatillake
Copy link
Contributor

Check List

  • Tests have been run in packages where changes made if available
  • Linter has been run for changed code
  • Tests for the changes have been added if not covered yet
  • Docs have been added / updated if required

Description of Changes Made (if issue reference is not provided)

Adding docs for Cube Cloud CCU consumption driven by AI API and AI Assistant.

Not sure how to do the links to other docs that the other pricing sections are doing.

@github-actions github-actions bot added the pr:community Contribution from Cube.js community members. label Feb 7, 2025
@igorlukanin igorlukanin added the docs Issues that require a documentation improvement label Feb 11, 2025
@igorlukanin
Copy link
Member

@djayatillake I think it would be better to add these as new types of resources to the table. From this table, you can link feature names to the relevant sections of the docs. Also, since these new types of resources don't consume CCUs per hour but rather consume CCUs per request, I suggest adding a new column to this table for the latter. You can still add an additional section to explain that BYOLLM has different cost implications.

Screenshot 2025-02-11 at 14 21 43

@igorlukanin
Copy link
Member

Also, here's how we usually refer to product tiers throughout the docs:

Screenshot 2025-02-11 at 14 27 43

@djayatillake
Copy link
Contributor Author

@djayatillake I think it would be better to add these as new types of resources to the table. From this table, you can link feature names to the relevant sections of the docs. Also, since these new types of resources don't consume CCUs per hour but rather consume CCUs per request, I suggest adding a new column to this table for the latter. You can still add an additional section to explain that BYOLLM has different cost implications.

Screenshot 2025-02-11 at 14 21 43

But it isn't charged per hour, so it won't really fit in the table.

adding correct reference for enterprise+
@igorlukanin
Copy link
Member

@djayatillake Exactly! That is why I suggested to introduce a new column to the table:

Also, since these new types of resources don't consume CCUs per hour but rather consume CCUs per request, I suggest adding a new column to this table for the latter

Without having these resources in the table, I'm afraid it would be quite hard to discover that we do charge for AI features (too far to scroll).

@djayatillake
Copy link
Contributor Author

@djayatillake Exactly! That is why I suggested to introduce a new column to the table:

Also, since these new types of resources don't consume CCUs per hour but rather consume CCUs per request, I suggest adding a new column to this table for the latter

Without having these resources in the table, I'm afraid it would be quite hard to discover that we do charge for AI features (too far to scroll).

How does it look now?

@igorlukanin igorlukanin merged commit 4a68bcc into cube-js:master Feb 25, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Issues that require a documentation improvement pr:community Contribution from Cube.js community members.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants