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

[actions] update to inclusive-heat-sensor v0.1.2 #26443

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

jonathanpeppers
Copy link
Member

Context: https://github.com/jonathanpeppers/inclusive-heat-sensor/releases/tag/v0.1.2
Changes: jonathanpeppers/inclusive-heat-sensor@v0.1.1...v0.1.2

After reviewing comments coming in for the past month, we made a few adjustments to make things more precise. We also stopped recording comment text, and simply a use public link to it instead.

v0.1.2 has these changes.

Let's also configure the bot to auto-hide comments if they score a 9 out of 10 or higher. This seems reasonable as:

  • A comment like "This sucks" is a 7/10. So, it would have to be pretty bad (mean comment!) to get auto-hidden.

  • If the comment triggers a Microsoft/OpenAI "content filter", we ignore the comment entirely as we can't be sure it's offensive at all.

We will continue monitoring the results.

Context: https://github.com/jonathanpeppers/inclusive-heat-sensor/releases/tag/v0.1.2
Changes: jonathanpeppers/inclusive-heat-sensor@v0.1.1...v0.1.2

After reviewing comments coming in for the past month, we made a few
adjustments to make things more precise. We also stopped recording
comment text, and simply a use public link to it instead.

v0.1.2 has these changes.

Let's also configure the bot to auto-hide comments if they score a 9
out of 10 or higher. This seems reasonable as:

* A comment like "This sucks" is a 7/10. So, it would have to be
  pretty bad (mean comment!) to get auto-hidden.

* If the comment triggers a Microsoft/OpenAI "content filter", we
  ignore the comment entirely as we can't be sure it's offensive at all.

We will continue monitoring the results.
@rmarinho rmarinho merged commit 9ab7f18 into dotnet:main Dec 6, 2024
4 checks passed
@jonathanpeppers jonathanpeppers deleted the inclusive-heat-sensor/v0.1.2 branch December 6, 2024 23:03
@samhouts samhouts added the fixed-in-net9.0-nightly This may be available in a nightly release! label Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed-in-net9.0-nightly This may be available in a nightly release!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants