-
Notifications
You must be signed in to change notification settings - Fork 2
p_sync Milestone 1 (Pull operation + CLI + call and load overloads) #61
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.
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
Closed
Closed
Changes from 31 commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
76a728b
Release 0.8.36
fern-api[bot] 3f3e7d2
set up initial test for sync operation
AlePouroullis 0834825
fix type error in test
AlePouroullis 4ff9180
basic sync functionality
AlePouroullis c685887
fix type error and formatting
AlePouroullis 67f17c8
refactor sync utils into SyncClient class
AlePouroullis 0b12a30
add client overloads for call method to use local files at specified …
AlePouroullis 612046e
infer file type from client passed into call overload
AlePouroullis 48262a1
simplify sync client to use updated list files endpoint that includes…
AlePouroullis 2eb8ec1
update test fixture to use staging
AlePouroullis 152fbfa
overload log operation to use local files when relevant
AlePouroullis c453a6c
Release 0.8.36
fern-api[bot] 6318918
merge auto-generated sdk with new serialize response type
AlePouroullis d703478
refactor overloads
AlePouroullis f72fda0
add tests for overloading call and log operations on agents and prompts
AlePouroullis 881f6e2
add environment and path filter to pull operation
AlePouroullis 2745020
Release 0.8.36
fern-api[bot] 6c01255
added include_content query param for retrieving file by path
AlePouroullis 816eaff
add path filter for pulling
AlePouroullis 39662df
refactor overload, moving functionality to sync client + implement LR…
AlePouroullis db5e37a
add DEFAULT_CACHE_SIZE constant
AlePouroullis b371348
remove unused imports in sync client
AlePouroullis 693262f
Release 0.8.36
fern-api[bot] 3d478a3
Merge branch 'master' into p_sync/sync
AlePouroullis 941a52e
feat(sync): Add metadata logging for sync operations
AlePouroullis 0063a45
create humanloop cli and support pull operation
AlePouroullis de5c767
add oneline option for displaying sync history
AlePouroullis d5e2de1
feat: improve local file handling in overload
AlePouroullis 7f89e45
improve error handling and cli message formatting
AlePouroullis ddeef13
fix version in pyproject.toml
AlePouroullis fe18a9c
add readme to pyproject.toml
AlePouroullis c6539ed
improve docstrings and help for cli
AlePouroullis 158f90c
Merge branch 'master' of github.com:humanloop/humanloop-python into p…
AlePouroullis 318aa6f
clean up pyproject.toml
AlePouroullis 63cc8af
add comments for clarity and remove keywords from pyproject.toml
AlePouroullis 9f5a3f2
improve docstrings and comments in overload.py
AlePouroullis 9afd2fc
Release 0.8.36
fern-api[bot] d30cc33
Replace with auto-generated SDK
AlePouroullis 0414fe6
docs: capitalize Prompt and Agent in doc strings
AlePouroullis 550fb44
Improve overloading and warnings; content -> raw file content
AlePouroullis 010d82e
Improve error message formatting in sync client
AlePouroullis e0a9d53
Improve logging in sync client and CLI
AlePouroullis e0a4ae9
Add static type hints to MetadataHandler
AlePouroullis 40ffd1e
Make duration_ms in log_operation required
AlePouroullis 01f2ba5
Merge branch 'p_sync/base' into p_sync/sync
AlePouroullis File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't think machine-learning is appropriate. where is this shown?
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These keywords aren't shown, only used for indexing and increasing discoverability.

https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#keywords
But perhaps machine learning would not be a good idea. Can just remove this altogether. The main thing was adding the description so it doesn't just say None below the package name