Skip to content

Conversation

CalebCourier
Copy link
Collaborator

Pull Request Type

  • Breaking Change
  • Feature
  • Bug Fix

Link to Notion Task or Github Issue

https://www.notion.so/guardrailsai/Merge-main-GaaS-4c764274407e440abacb986ac9bc0e8b?pvs=4

Summary of Feature(s)

  • Adds API Client and serialization methods to allow running select Guards on the Guardrails as a Service API
  • Patch structures to get back and forth between OSS and API to work "as-is"

@CalebCourier
Copy link
Collaborator Author

We'll need to make the api client repo public before we merge/release this

@codecov-commenter
Copy link

codecov-commenter commented Mar 1, 2024

Codecov Report

Attention: Patch coverage is 42.06897% with 84 lines in your changes are missing coverage. Please review.

❗ No coverage uploaded for pull request base (main@33bd89d). Click here to learn what that means.

Files Patch % Lines
guardrails/guard.py 39.43% 43 Missing ⚠️
guardrails/llm_providers.py 46.42% 15 Missing ⚠️
guardrails/rail.py 20.00% 8 Missing ⚠️
guardrails/api_client.py 70.00% 6 Missing ⚠️
guardrails/schema/string_schema.py 16.66% 5 Missing ⚠️
guardrails/datatypes.py 20.00% 4 Missing ⚠️
guardrails/schema/schema.py 33.33% 2 Missing ⚠️
guardrails/prompt/base_prompt.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #606   +/-   ##
=======================================
  Coverage        ?   81.05%           
=======================================
  Files           ?      119           
  Lines           ?     6876           
  Branches        ?        0           
=======================================
  Hits            ?     5573           
  Misses          ?     1303           
  Partials        ?        0           
Flag Coverage Δ
unittests 81.05% <42.06%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@CalebCourier CalebCourier changed the base branch from main to gaas-additions March 4, 2024 16:49
@CalebCourier CalebCourier marked this pull request as ready for review March 4, 2024 16:50
@CalebCourier CalebCourier requested a review from zsimjee March 4, 2024 16:50
@CalebCourier CalebCourier marked this pull request as draft March 4, 2024 16:54
@CalebCourier
Copy link
Collaborator Author

We should merge this one first so we can do a gh release and point to that version: https://github.com/guardrails-ai/guardrails-api-client/pull/6/files

@CalebCourier CalebCourier marked this pull request as ready for review March 4, 2024 17:48
@CalebCourier CalebCourier merged commit 5402bb6 into guardrails-ai:gaas-additions Mar 4, 2024
@ShreyaR ShreyaR deleted the gaas-additions branch March 29, 2024 04:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants