Skip to content
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

Initial VertexAI API surface #1179

Merged
merged 3 commits into from
Feb 10, 2025
Merged

Initial VertexAI API surface #1179

merged 3 commits into from
Feb 10, 2025

Conversation

a-maurice
Copy link
Collaborator

Description

Provide details of the change, and generalize the change in the PR title above.

Stub implementation of the initial planned VertexAI surface


Testing

Describe how you've tested these changes.

https://github.com/firebase/firebase-unity-sdk/actions/runs/13168686833


Type of Change

Place an x the applicable box:

  • Bug fix. Add the issue # below if applicable.
  • New feature. A non-breaking change which adds functionality.
  • Other, such as a build process or documentation change.

public readonly struct UsageMetadata {
public int PromptTokenCount { get; }
public int CandidatesTokenCount { get; }
public int TotalTokenCount { get; }
Copy link
Contributor

Choose a reason for hiding this comment

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

not blocking, maybe consider to add related modality token info similar to this PR as well? firebase/flutterfire#17065

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, I'll need to reorganize what my plans are for some of those based on the recent changes, but I'll wait until I'm doing the implementation of these parts, just to let the other platforms settle in on something for sure.

@a-maurice a-maurice merged commit ba77302 into main Feb 10, 2025
16 checks passed
@a-maurice a-maurice deleted the am-vertexai_api branch February 10, 2025 22:43
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.

2 participants