cookbook icon indicating copy to clipboard operation
cookbook copied to clipboard
verified publisher icon google-gemini

Migrating `quickstarts/safety` notebook

Open andycandy opened this issue 8 months ago • 4 comments

  • Updated SDK imports
  • lint checked
  • nbfmt

andycandy avatar Apr 01 '25 19:04 andycandy

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

review-notebook-app[bot] avatar Apr 01 '25 19:04 review-notebook-app[bot]

Thank for the great submission as always @andycandy. I just added one minor comment in case we can do better to display the safety output.

Giom-V avatar Apr 02 '25 14:04 Giom-V

@Giom-V Would you mind confirming if using pprint was fine?

andycandy avatar Apr 02 '25 14:04 andycandy

@Giom-V Would you mind confirming if using pprint was fine?

Yes, sure.

Giom-V avatar Apr 02 '25 17:04 Giom-V

@andycandy Can you just fix the format? I think everything else is fine otherwise.

Giom-V avatar Apr 03 '25 14:04 Giom-V

@andycandy Can you just fix the format? I think everything else is fine otherwise.

@Giom-V sure I'll get all of the PRs ready by the end of the day, sorry for being inactive, I got stuck in an unavoidable college meet

andycandy avatar Apr 03 '25 14:04 andycandy

@Giom-V Done!

andycandy avatar Apr 03 '25 17:04 andycandy

Thanks @andycandy. It's approved and merged.

Giom-V avatar Apr 03 '25 19:04 Giom-V