Skip to content

0.6.0 dev #1138

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 87 commits into from
Nov 6, 2024
Merged

0.6.0 dev #1138

merged 87 commits into from
Nov 6, 2024

Conversation

zsimjee
Copy link
Collaborator

@zsimjee zsimjee commented Oct 22, 2024

No description provided.

dtam and others added 30 commits September 19, 2024 16:37
@dtam
reapply litellm updates to support only messages llm kwarg
b0baed1
@dtam
tests run and make progress on rewrite, most of unit_tests_passing
5e0eb15
@dtam
migrate more tests partially
a11c4ee
@dtam
Merge remote-tracking branch 'origin/main' into feature/litellm_cleanup
487a06a
@dtam
some progress
b97caa2
@dtam
more progress
e684739
@dtam
fix some more tests
18591b7
@dtam
fix some more tests
a1f7893
@dtam
Merge remote-tracking branch 'origin/main' into feature/litellm_cleanup
816ef5f
@dtam
more progress
67feff4
@dtam
more tests
da2dc63
@dtam
more tests
66a568b
@AlejandroEsquivel
removed nltk dependency
bbef78a
@AlejandroEsquivel
remove nltk import and download from validator base
0298730
@AlejandroEsquivel
remove commented out test referencing nltk
e699397
@AlejandroEsquivel
throwing import error if nltk not available in detect_pii mock
c1f7350
@dtam
tests passing
bc7ff07
@dtam
typing and lint
e8eb7f5
@dtam
lint
f4c4827
@dtam
typing
1228310
@dtam
Merge remote-tracking branch 'origin/main' into feature/litellm_cleanup
1d0490f
@dtam
fix bad merge
4e333b9
@AlejandroEsquivel
Added rule based sentence tokenization from WordTokenizers.jl with mi…
416f290 
…nor modifications
@AlejandroEsquivel
Added new version of split_sentence_str using new rule based sentence…
6743df2 
… tokenization
@AlejandroEsquivel
updates to factor in quotes during sentence splitting
acae737
@AlejandroEsquivel
updated poetry.lock
815300f
@AlejandroEsquivel
replaced split sentence default
1a2546a
@AlejandroEsquivel
testing changes using custom separator in wordtokenizer algo
bd0ab50
@AlejandroEsquivel
fix for counting subs
4ebef18
@AlejandroEsquivel
reverted split sentence in validators base
fb1d2d0
dtam and others added 24 commits October 21, 2024 17:03
@dtam
update validator default on fail behavior from no op to exception
6ae7f75
@dtam
update notebooks
766d9cc
@zsimjee
Merge pull request #1133 from guardrails-ai/default_validator_on_fail…
af9f898 
…_exception

update validator default on fail behavior from no op to exception
@AlejandroEsquivel
Installs from private pypi with validator versioning
595ccf5
@AlejandroEsquivel
chore: Fix typo in pip_process function
6d31efc
@AlejandroEsquivel
wip: updating tests
3b78114
@AlejandroEsquivel
fixed tests
0d1367a
@AlejandroEsquivel
fix f-string syntax error
b8d4d18
@AlejandroEsquivel
fixed uninstall, list validators cli cmds, and tests
5b4d6b8
@AlejandroEsquivel
removed unused methods from cli hub utils
106cc86
@AlejandroEsquivel
Removed writes to org namespaced init files during install, removed u…
f73adf7 
…nused methods
@AlejandroEsquivel
fix tests associated with removal of namespace init file
c0dfcce
@dtam
update for litellm dont pass reask messages to llms
76f6acb
@zsimjee
Merge pull request #1144 from guardrails-ai/dont_pass_gr_args_to_llms
6e8ac73 
update for litellm dont pass reask messages to llms
@zsimjee
Update pyproject.toml
f2a7138
@AlejandroEsquivel
Update guardrails/cli/hub/uninstall.py
a82a412
@AlejandroEsquivel
remove regex pypi package dependency
586cf80
@AlejandroEsquivel
bumped version to 0.6.0-alpha3
cd37159
@AlejandroEsquivel
Merge pull request #1140 from guardrails-ai/feat/validator-versioning
280bb81 
Validator Versioning
@AlejandroEsquivel
bump version to 0.6.0-alpha4
d8fd51a
@AlejandroEsquivel
temp workaround to raise errors on on pip processes
91077a9
@AlejandroEsquivel
raise errors on install failures to avoid placing import statement
943d0c7
@AlejandroEsquivel
Merge pull request #1156 from guardrails-ai/0.6.0-dev-validator-extras
de43a01 
0.6.0 dev validator extras
@AlejandroEsquivel
update version to 0.6.0
1d9f7dd
@zsimjee zsimjee marked this pull request as ready for review November 6, 2024 17:06
AlejandroEsquivel
AlejandroEsquivel approved these changes Nov 6, 2024
View reviewed changes
Copy link
Contributor

@AlejandroEsquivel AlejandroEsquivel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍🏼

@AlejandroEsquivel AlejandroEsquivel merged commit 01c56ec into main Nov 6, 2024
12 checks passed
@AlejandroEsquivel AlejandroEsquivel deleted the 0.6.0-dev branch November 6, 2024 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlejandroEsquivel AlejandroEsquivel AlejandroEsquivel approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zsimjee @AlejandroEsquivel @dtam