Skip to content

Conversation

rlazo
Copy link
Collaborator

@rlazo rlazo commented Sep 17, 2024

This better matches the backend API.

This better matches the backend API.
Copy link
Contributor

github-actions bot commented Sep 17, 2024

Javadoc Changes:
--- /home/runner/diff/original/firebase-kotlindoc/android/com/google/firebase/vertexai/type/HarmBlockThreshold.html	2024-09-30 16:26:18.882281615 +0000 +++ /home/runner/diff/modified/firebase-kotlindoc/android/com/google/firebase/vertexai/type/HarmBlockThreshold.html	2024-09-30 16:24:10.633729758 +0000 @@ -29,7 +29,7 @@ </table> </div> <hr> - <p>Represents the threshold for some <code><a href="/docs/reference/android/com/google/firebase/vertexai/type/HarmCategory.html">HarmCategory</a></code> that is allowed and blocked by <code><a href="/docs/reference/android/com/google/firebase/vertexai/type/SafetySetting.html">SafetySetting</a></code>.</p> + <p>Represents the threshold for a <code><a href="/docs/reference/android/com/google/firebase/vertexai/type/HarmCategory.html">HarmCategory</a></code> to be allowed by <code><a href="/docs/reference/android/com/google/firebase/vertexai/type/SafetySetting.html">SafetySetting</a></code>.</p> <h2>Summary</h2> <div class="devsite-table-wrapper"> <table class="responsive">
--- /home/runner/diff/original/firebase-kotlindoc/android/com/google/firebase/vertexai/type/package-summary.html	2024-09-30 16:26:18.880281608 +0000 +++ /home/runner/diff/modified/firebase-kotlindoc/android/com/google/firebase/vertexai/type/package-summary.html	2024-09-30 16:24:10.631729750 +0000 @@ -245,7 +245,7 @@ <tr> <td><code><a href="/docs/reference/android/com/google/firebase/vertexai/type/HarmBlockThreshold.html">HarmBlockThreshold</a></code></td> <td> - <p>Represents the threshold for some <code><a href="/docs/reference/android/com/google/firebase/vertexai/type/HarmCategory.html">HarmCategory</a></code> that is allowed and blocked by <code><a href="/docs/reference/android/com/google/firebase/vertexai/type/SafetySetting.html">SafetySetting</a></code>.</p> + <p>Represents the threshold for a <code><a href="/docs/reference/android/com/google/firebase/vertexai/type/HarmCategory.html">HarmCategory</a></code> to be allowed by <code><a href="/docs/reference/android/com/google/firebase/vertexai/type/SafetySetting.html">SafetySetting</a></code>.</p> </td> </tr> <tr>
--- /home/runner/diff/original/firebase-kotlindoc/kotlin/com/google/firebase/vertexai/type/HarmBlockThreshold.html	2024-09-30 16:26:18.874281585 +0000 +++ /home/runner/diff/modified/firebase-kotlindoc/kotlin/com/google/firebase/vertexai/type/HarmBlockThreshold.html	2024-09-30 16:24:10.625729723 +0000 @@ -29,7 +29,7 @@ </table> </div> <hr> - <p>Represents the threshold for some <code><a href="/docs/reference/kotlin/com/google/firebase/vertexai/type/HarmCategory.html">HarmCategory</a></code> that is allowed and blocked by <code><a href="/docs/reference/kotlin/com/google/firebase/vertexai/type/SafetySetting.html">SafetySetting</a></code>.</p> + <p>Represents the threshold for a <code><a href="/docs/reference/kotlin/com/google/firebase/vertexai/type/HarmCategory.html">HarmCategory</a></code> to be allowed by <code><a href="/docs/reference/kotlin/com/google/firebase/vertexai/type/SafetySetting.html">SafetySetting</a></code>.</p> <h2>Summary</h2> <div class="devsite-table-wrapper"> <table class="responsive">
--- /home/runner/diff/original/firebase-kotlindoc/kotlin/com/google/firebase/vertexai/type/package-summary.html	2024-09-30 16:26:18.872281578 +0000 +++ /home/runner/diff/modified/firebase-kotlindoc/kotlin/com/google/firebase/vertexai/type/package-summary.html	2024-09-30 16:24:10.623729715 +0000 @@ -316,7 +316,7 @@ <tr> <td><code><a href="/docs/reference/kotlin/com/google/firebase/vertexai/type/HarmBlockThreshold.html">HarmBlockThreshold</a></code></td> <td> - <p>Represents the threshold for some <code><a href="/docs/reference/kotlin/com/google/firebase/vertexai/type/HarmCategory.html">HarmCategory</a></code> that is allowed and blocked by <code><a href="/docs/reference/kotlin/com/google/firebase/vertexai/type/SafetySetting.html">SafetySetting</a></code>.</p> + <p>Represents the threshold for a <code><a href="/docs/reference/kotlin/com/google/firebase/vertexai/type/HarmCategory.html">HarmCategory</a></code> to be allowed by <code><a href="/docs/reference/kotlin/com/google/firebase/vertexai/type/SafetySetting.html">SafetySetting</a></code>.</p> </td> </tr> <tr>
@google-oss-bot
Copy link
Contributor

google-oss-bot commented Sep 17, 2024

Copy link
Contributor

github-actions bot commented Sep 17, 2024

Unit Test Results

0 files   -   16  0 suites   - 16   0s ⏱️ -13s
0 tests  - 107  0 ✔️  - 107  0 💤 ±0  0 ±0 
0 runs   - 214  0 ✔️  - 214  0 💤 ±0  0 ±0 

Results for commit 7a9130e. ± Comparison against base commit e98a4b6.

♻️ This comment has been updated with latest results.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Sep 17, 2024

Size Report 1

Affected Products

  • firebase-vertexai

    TypeBase (da0eefa)Merge (445343b)Diff
    aar489 kB489 kB+10 B (+0.0%)
    apk (release)9.30 MB9.30 MB+20 B (+0.0%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/OWatz6FcVD.html
@ryanwilson
Copy link
Member

Let's hold off on submitting this until we have a discussion if it makes sense to break - and the strategy for doing so (should we deprecate and introduce a new type, how could that work, etc)

@rlazo rlazo added this to the vertexai-ga milestone Sep 21, 2024
@rlazo
Copy link
Collaborator Author

rlazo commented Sep 21, 2024

@daymxn @emilypgoogle this is OK to review now, thanks!

@google-oss-bot
Copy link
Contributor

1 Warning
⚠️ Did you forget to add a changelog entry? (Add the 'no-changelog' label to the PR to silence this warning.)

Generated by 🚫 Danger

@rlazo rlazo enabled auto-merge (squash) September 30, 2024 16:16
@rlazo rlazo merged commit 877614a into main Sep 30, 2024
26 of 27 checks passed
@rlazo rlazo deleted the rl.rename.harm.threshold branch September 30, 2024 16:27
@firebase firebase locked and limited conversation to collaborators Oct 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

4 participants