-
Notifications
You must be signed in to change notification settings - Fork 1
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
base: master
Are you sure you want to change the base?
Conversation
…path for agents and prompts
… serialized content
…U cache for reading local files
- Make local file access more explicit by throwing errors instead of silent fallbacks - Add proper error handling for file not found and IO errors - Improve version/environment handling with clear warnings - Update docstrings to better document behavior This change makes the behavior more predictable when use_local_files=True: - Throws FileNotFoundError/IOError if local file can't be accessed - Warns and uses remote when version_id/environment is specified - No more silent fallbacks to remote API
think this needs a merge with master -- agents has gone in |
pyproject.toml
Outdated
@@ -1,13 +1,18 @@ | |||
[project] | |||
name = "humanloop" | |||
description = "The Humanloop Python Library" | |||
authors = [] | |||
keywords = ["ai", "machine-learning", "llm", "sdk", "humanloop"] |
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?
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
pull
+history
(shows last <=5 sync operations with some metadata) commands. Uses the click library.call()
and.log()
functions to use serialized contents of.prompt
/.agent
local files whenuse_local_files
is set totrue
when initializing the humanloop client.