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.

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

v0.4.5 #808

Merged
merged 119 commits into from
Jun 6, 2024
Merged

v0.4.5 #808

merged 119 commits into from
Jun 6, 2024

Conversation

CalebCourier
Copy link
Collaborator

No description provided.

nichwch added 30 commits May 7, 2024 12:34
@nichwch
preliminary code and pseudocode
29ee128
@nichwch
add chunk accumulation strategy to Validator base class
66a5333
@nichwch
handle case where llm chunk > validator chunk in validator class
7831466
@nichwch
added some questions
2dbae2e
@nichwch
change stream_runner to handle the result of iterable validate
1e3544d
@nichwch
format
e9084e1
@nichwch
change validator base to use a chunking function instead of specifyin…
1269f68 
…g a chunking strategy
@nichwch
connect streaming all the way down call chain, include validated chun…
b454cd5 
…k in ValidationResult metadata
@nichwch
change execute_validator to handle streaming
b64ab4e
@nichwch
make validate take stream parameter, remove validate_stream in top le…
bf2bd32 
…vel of validator_service
@nichwch
use wyatts sentence splitting strategy
c79e9b2
@nichwch
import nltk
4583cb9
@nichwch
use stream-enabled execute_validator
f1b4a88
@nichwch
format
289745c
@nichwch
fix bug where json_schema was being called with streaming
58d8eed
@nichwch
conditionally use old logic for json_schema to avoid breaking json_sc…
947f476 
…hema parse
@nichwch
validate remainders
8b2c154
@nichwch
merge in main
d6c3739
@nichwch
new chunk span validation schema
0ab245c
@nichwch
field for reason that validation failed for a given span
a320464
@nichwch
add validated_chunk to ValidationResult
93bb781
@nichwch
add helper method to get a list of error spans relative to llm output
1381821
@nichwch
conceptual question
3ccdda1
@nichwch
Merge branch 'main' into nichwch/chunk-accumulation-rewrite
6fdbcd1
@nichwch
Merge branch 'nichwch/chunk-accumulation-rewrite' into nichwch/stream…
f455ae2 
…ing-error-spans
@nichwch
turn chunking_function into class method
74485eb
@nichwch
incomplete tests for streaming chunk accumulation
a39b5af
@nichwch
format
0ae850e
@nichwch
remove print
847dd0a
@nichwch
fix a few bugs uncovered by testing
f0b3030
@CalebCourier CalebCourier marked this pull request as ready for review June 6, 2024 18:38
wylansford
wylansford previously approved these changes Jun 6, 2024
View reviewed changes
Copy link
Contributor

@wylansford wylansford left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM, Async streaming still running as expected locally as well.

@CalebCourier CalebCourier changed the title bump version v0.4.5 Jun 6, 2024
@CalebCourier
Copy link
Collaborator Author

Wrapping up some last minute notebook fixes; mostly missing dependencies.

@zsimjee zsimjee merged commit 3265a1c into main Jun 6, 2024
43 checks passed
@zsimjee zsimjee deleted the 0.4.5-dev branch June 6, 2024 21:43
@CalebCourier CalebCourier mentioned this pull request Jun 25, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zsimjee zsimjee zsimjee approved these changes

@wylansford wylansford Awaiting requested review from wylansford

@nichwch nichwch Awaiting requested review from nichwch

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

Successfully merging this pull request may close these issues.
4 participants
@CalebCourier @zsimjee @wylansford @nichwch