Skip to content

Vaihi exp #8964

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
Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Vaihi exp #8964

wants to merge 14 commits into from

Conversation

gsiddh
Copy link

@gsiddh gsiddh commented Apr 22, 2025

Hey there! So you want to contribute to a Firebase SDK?
Before you file this pull request, please read these guidelines:

Discussion

  • Read the contribution guidelines (CONTRIBUTING.md).
  • If this has been discussed in an issue, make sure to link to the issue here.
    If not, go file an issue about this before creating a pull request to discuss.

Testing

  • Make sure all existing tests in the repository pass after your change.
  • If you fixed a bug or added a feature, add a new test to cover your code.

API Changes

  • At this time we cannot accept changes that affect the public API. If you'd like to help
    us make Firebase APIs better, please propose your change in an issue so that we
    can discuss it together.

gsiddh and others added 13 commits April 22, 2025 14:15
@gsiddh
Vaihi add langmodel types. (#8927)
d84d8e8 
* Adding LanguageModel types. These are based off https://github.com/webmachinelearning/prompt-api?tab=readme-ov-file#full-api-surface-in-web-idl

* Adding LanguageModel types.

* Remove bunch of exports

* yarn formatted

* after lint
@gsiddh @erikeldridge
Define HybridParams (#8935)
94bd803 
Co-authored-by: Erik Eldridge <[email protected]>
@gsiddh @erikeldridge
Adding smoke test for new hybrid params (#8937)
8971cc6 
* Adding smoke test for new hybrid params

* Use the existing name of the model params input

---------

Co-authored-by: Erik Eldridge <[email protected]>
@gsiddh @erikeldridge
Moving to in-cloud naming (#8938)
5a4fb81 
Co-authored-by: Erik Eldridge <[email protected]>
@gsiddh
Moving to string type for the inference mode (#8941)
eeeab4e
@gsiddh @erikeldridge
Define ChromeAdapter class (#8942)
b7a3d24 
Co-authored-by: Erik Eldridge <[email protected]>
@erikeldridge @gsiddh
VinF Hybrid Inference: Implement ChromeAdapter (rebased) (#8943)
57c8841
@gsiddh
Adding count token impl (#8950)
5d4c7a0
@erikeldridge @gsiddh
VinF Hybrid Inference #4: ChromeAdapter in stream methods (rebased) (#…
58d6d1f 
…8949)
@erikeldridge @gsiddh
Define values for Availability enum (#8951)
ee81092
@erikeldridge @gsiddh
VinF Hybrid Inference: narrow Chrome input type (#8953)
eb1dcb4
@gsiddh
Add image inference support (#8954)
2a503a9 
* Adding image based input for inference

* adding image as input to create language model object
@gsiddh
disable count tokens api for on-device inference (#8962)
5fa83b3
@gsiddh gsiddh requested review from a team as code owners April 22, 2025 21:20
@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Apr 22, 2025 •
edited
Loading

⚠️ No Changeset found

Latest commit: e839743

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions GitHub Actions
Copy link
Contributor

Vertex AI Mock Responses Check ⚠️

A newer major version of the mock responses for Vertex AI unit tests is available. update_vertexai_responses.sh should be updated to clone the latest version of the responses: v10.0

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Apr 22, 2025 •
edited
Loading

Size Report 1

Affected Products

  • @firebase/vertexai

    TypeBase (1363ecc)Merge (b02a5ea)Diff
    browser34.7 kB39.4 kB+4.70 kB (+13.5%)
    main35.7 kB40.4 kB+4.70 kB (+13.2%)
    module34.7 kB39.4 kB+4.70 kB (+13.5%)

  • firebase

    TypeBase (1363ecc)Merge (b02a5ea)Diff
    firebase-vertexai.js28.3 kB31.6 kB+3.37 kB (+11.9%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/jBkhgQmKi1.html

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Apr 22, 2025 •
edited
Loading

Size Analysis Report 1

This report is too large (68,271 characters) to be displayed here in a GitHub comment. Please use the below link to see the full report on Google Cloud Storage.

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/O6VaGFpREv.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gsiddh @google-oss-bot @erikeldridge