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

Feature/chains #16

Merged
merged 79 commits into from
Aug 10, 2023
Merged

Feature/chains #16

merged 79 commits into from
Aug 10, 2023

Conversation

alexkuzmik
Copy link
Collaborator

No description provided.

@alexkuzmik
Add tests for chain nodes and prompts
705aa00
@alexkuzmik
Fix some lint errors
c6609eb
@alexkuzmik
Implement chain, move time logic to Timer class, update tests
5215f79
@alexkuzmik
Merge branch 'main' into CM-7071-llm-sdk-allow-users-to-logs-their-ch…
27c3e85 
…ain-execution
@alexkuzmik
Add proper support for category
d2b9e10
@alexkuzmik
Add state.get_new_id()
775ec36
@alexkuzmik
input_ and output_ metadata -> metadata, add more type hints
a9b5c3b
@alexkuzmik
Add some type hints
cd51ba9
@alexkuzmik
Fix lint errors
b2469f3
@alexkuzmik
Add tests for Timer
28770ac
@alexkuzmik
Small refactor
d7790a6
@alexkuzmik
Change error text
29a415f
@alexkuzmik
Rename input -> inputs in class/function signatures
0d1a009
@alexkuzmik
Init test for node_data_to_dict function
f3b775e
@alexkuzmik
Implement convert.node_data_to_dict
15cb92c
@alexkuzmik
Use node_data_to_dict in ChainNode
91a516a
@alexkuzmik
Implement node name logic
12a4221
@alexkuzmik
Merge branch 'main' into CM-7071-llm-sdk-allow-users-to-logs-their-ch…
15c3794 
…ain-execution
@alexkuzmik
Merge branch 'main' into CM-7071-llm-sdk-allow-users-to-logs-their-ch…
ee29cd3 
…ain-execution
@alexkuzmik
Add name to node_data_to_dict
d739bd5
@alexkuzmik
Add name to as_dict logic in prompt and node
42b0e24
@alexkuzmik
Fix lint errors
ca12dbc
@alexkuzmik
Reorganize some code
9be61dd
@alexkuzmik
Remove node data to dict, move this logic to ChainNode.as_dict()
58e5a74
@alexkuzmik
Rename group_stack -> context
962a4e9
@alexkuzmik
Delete modules that are no longer needed. Rename some of the rest
192c702
@alexkuzmik
Fix some lint errors
42d1fe7
@alexkuzmik
Add start() to timer
94dbd27
@alexkuzmik
Fix lint errors
feddd45
@alexkuzmik
Move _timer.start() to __enter__ method
3973ecd
alexkuzmik and others added 3 commits May 31, 2023 20:52
@alexkuzmik
Add deepmerge function
44e3135
@alexkuzmik
Add test for deepmerge
4165b0d
@alexkuzmik
Merge pull request #7 from comet-ml/CM-7071-llm-sdk-allow-users-to-lo…
a7e887b 
…gs-their-chain-execution

[CM-7071] llm sdk allow users to logs their chain execution
@alexkuzmik alexkuzmik self-assigned this Jun 1, 2023
alexkuzmik and others added 24 commits June 1, 2023 17:26
@alexkuzmik
Move Span name to comet_llm namespace
94d2152
@alexkuzmik
Merge branch 'main' into feature/chains
449fbac
@alexkuzmik
Fix merge conflicts
9898627
@alexkuzmik
Merge branch 'main' into feature/chains
3e140f2
@alexkuzmik
Merge branch 'main' into feature/chains
b336807
@alexkuzmik
Merge branch 'main' into feature/chains
55c9146
@alexkuzmik
Merge branch 'main' into feature/chains
018c33d
@alexkuzmik
Rename test
f077042
@alexkuzmik
Add summary logic for chains
3bcd9b0
@alexkuzmik
Fix lint errors
bfc3491
@alexkuzmik
Add category to chain
bd3e8b1
@alexkuzmik
[CM-7509] llm add deepmerge for input and output metadata (#27)
7e1df38 
* Add deepmerge logic to chain set_outputs

* Fix lint errors
@alexkuzmik
Add tags logging to chains. Log chain_duration as a metric in chains (#…
8a85c68 
…28)
@alexkuzmik
Add docstrings to Chain and Span
fe1f2ea
@alexkuzmik
[CM-7772] llm add a proper asset upload error message for on prem bac…
40c0ed7 
…kend users (#34)

* Implement check for on-prem in request exceptions handler

* Fix lint errors

* Refactor

* Refactor request exception wrapper
@sherpan @alexkuzmik
Add files via upload
98812fa
@sherpan @alexkuzmik
added gif to readme
2a890f1
@Lothiraldan @alexkuzmik
Update packaging README with UI Gif
9faf8fc
@alexkuzmik
[CM-7218] introduce new monkey patching to comet llm (#30)
42f7009 
* Add prototype for import_hooks

* Add new entities

* Rename module

* Make it work on psutil example.Hooray

* Fix wrapper

* Add copyrights

* Fix lint errors

* Add docstrings to registry

* Update docstrings

* Fix lint errors

* Add new tests for patching functions

* Add tests

* Add validate.py and tests for it

* Fix lint errors

* Add tests

* Update tests

* Update tests

* Add test for child class method, refactor tests a bit

* Remove currently odd import

* Replace wraps workaround with actual wraps
@omarsar @alexkuzmik
Update README.md (#32)
682564a
@alexkuzmik
[NA] Readme: project_name -> project (#35)
03e4739 
* project_name -> project

* Fix table
@alexkuzmik
Merge branch 'main' into feature/chains
6342796
@Lothiraldan
Update Readme.md to add an example logging chain
21128ad
@alexkuzmik
Cut project_name argument (#37)
3a0c92f
@alexkuzmik alexkuzmik merged commit ed365fb into main Aug 10, 2023
7 checks passed
@andrescrz andrescrz deleted the feature/chains branch September 2, 2024 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@alexkuzmik alexkuzmik

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@alexkuzmik @sherpan @Lothiraldan @omarsar