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

Rename BlockThreshold to HarmBlockThreshold #6262

Merged
merged 5 commits into from
Sep 30, 2024
Merged

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

@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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants