deon icon indicating copy to clipboard operation
deon copied to clipboard

LLM-related examples of harm

Open IshaShah27 opened this issue 11 months ago • 1 comments

IshaShah27 avatar Mar 13 '24 15:03 IshaShah27

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.80%. Comparing base (67b2964) to head (a85f210).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #168   +/-   ##
=======================================
  Coverage   96.80%   96.80%           
=======================================
  Files           6        6           
  Lines         188      188           
=======================================
  Hits          182      182           
  Misses          6        6           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Mar 13 '24 15:03 codecov[bot]

@pjbull I made a few more changes to this -- want to take a quick look?

klwetstone avatar Apr 12 '24 18:04 klwetstone

Is there anything else that needs to happen before I merge? Are the steps I took (in the PR description) the right ones to make sure all relevant files are updated?

klwetstone avatar Apr 12 '24 18:04 klwetstone

I think this is good. Pretty sure just changing the yaml is the only required thing. Generated files get overwritten when PR is merged to main: https://github.com/drivendataorg/deon/blob/main/.github/workflows/examples.yml#L20-L29

pjbull avatar Apr 12 '24 20:04 pjbull

Ah got it excellent. Going ahead with the merge!

klwetstone avatar Apr 12 '24 20:04 klwetstone

Is there anything else that needs to happen before I merge? Are the steps I took (in the PR description) the right ones to make sure all relevant files are updated?

For future reference, contributing doc is here: https://github.com/drivendataorg/deon/blob/main/CONTRIBUTING.md

ejm714 avatar Apr 12 '24 20:04 ejm714

^ Contributing doc covers adding examples and adding checklist items, but doesn't explicitly cover changing the main README so wanted to make sure

klwetstone avatar Apr 12 '24 20:04 klwetstone