Autogen latency v2 #31
Open
Conversation
This file contains 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
…AgentOps-AI#479) In this feature we're adding the pop up chat of entelligence inside the agentops repository
* make start_session non blocking * fix build issue * bump version * callback for start session now that it is async * fix callback * unpin dependencies (AgentOps-AI#434) Co-authored-by: Howard Gil <[email protected]> * bump version number * wip * change autogen getting run every time * remove prints * remove more prints * suppress warnings * exponential retry to close data * removed event counter * fix requests mock * remove print * fixed more tests * removed inits from test_agent; does not require a client * create requests fixture * Scope fixtures * black run * remove bad files * add spaces back * revert session threading changes * remove callback * revert session changes * revert session * fix test * update tests * update tox to install local build instead of pypi * replace http client with requests so requests_mock works properly * fixed multiple sessions * fix tool recorder * removed test logs, fixed request count tests * set fixture scopes * Fixed missing async tests failing in tox, updated tox * fixed missing pass in tests * fixed timing * created rc branch --------- Co-authored-by: Shawn Qiu <[email protected]> Co-authored-by: Howard Gil <[email protected]>
* Updated README and assets with new v2 screenshots * updated font size on dashboard banner
Add Hover Badges and Update Badge Style for Social Links
…#382) * add initial files for support * working sync client * stream not working * updated examples notebook for current testing * fix for `delta.content` and cleanup * cleanup again * cleanup and add tool event * structure examples notebook * add contextual answers tracking * cleanup example notebook * create testing file * clean example notebook again * rename examples directory * updated docs page * wrap `chunk.choices[0].delta.content` in `str(...)` * update doc --------- Co-authored-by: reibs <[email protected]>
…Ops-AI#374) * add mistral support * linting * fix typo * add tests * add examples notebook * linting * fix langchain typo in pyproject.toml (updated to 0.2.14) * fix mistralai import and `undo_override` function * add mistral to readme * fix typo * modified self.llm_event to llm_event * refactoring * black * rename examples directory * fix merge * init merge * updated model name so that tokencost will recognize this as a mistral model * black lint --------- Co-authored-by: reibs <[email protected]>
WalkthroughThis update introduces several enhancements and new features to the AgentOps repository. Key changes include setting the default session end state to "Indeterminate" in Changes
Files selected (32)
Files ignored (19)
InstructionsEmoji Descriptions:
Interact with the Bot:
Available Commands:
Tips for Using @Entelligence.AI Effectively:
Need More Help?
|
Comment on lines
24
to
48
| </div> | ||
|
|
||
| <p align="center"> | ||
| <a href="https://twitter.com/agentopsai/">🐦 Twitter</a> | ||
| <span> • </span> | ||
| <a href="https://discord.gg/FagdcwwXRR">📢 Discord</a> | ||
| <span> • </span> | ||
| <a href="https://app.agentops.ai/?ref=gh">🖇️ Dashboard</a> | ||
| <span> • </span> | ||
| <a href="https://docs.agentops.ai/introduction">📙 Documentation</a> | ||
| <span> • </span> | ||
| <a href="https://entelligence.ai/AgentOps-AI&agentops">💬 Chat with Docs</a> | ||
| <a href="https://twitter.com/agentopsai/"> | ||
| <img src="https://img.shields.io/twitter/follow/agentopsai?style=social" alt="Twitter" style="height: 20px;"> | ||
| </a> | ||
| <a href="https://discord.gg/FagdcwwXRR"> | ||
| <img src="https://img.shields.io/badge/discord-7289da.svg?style=flat-square&logo=discord" alt="Discord" style="height: 20px;"> | ||
| </a> | ||
| <a href="https://app.agentops.ai/?ref=gh"> | ||
| <img src="https://img.shields.io/badge/Dashboard-blue.svg?style=flat-square" alt="Dashboard" style="height: 20px;"> | ||
| </a> | ||
| <a href="https://docs.agentops.ai/introduction"> | ||
| <img src="https://img.shields.io/badge/Documentation-orange.svg?style=flat-square" alt="Documentation" style="height: 20px;"> | ||
| </a> | ||
| <a href="https://entelligence.ai/AgentOps-AI&agentops"> | ||
| <img src="https://img.shields.io/badge/Chat%20with%20Docs-green.svg?style=flat-square" alt="Chat with Docs" style="height: 20px;"> | ||
| </a> | ||
| </p> | ||
|
|
||
|
|
||
|
|
||
| <div style="justify-content: center"> | ||
| <img src="docs/images/external/app_screenshots/dashboard_banner.png" alt="Dashboard Banner"> | ||
| <img src="docs/images/external/app_screenshots/dashboard-banner.png" alt="Dashboard Banner"> | ||
| </div> |
There was a problem hiding this comment.
ℹ️ Performance Improvement
Optimize Badge Images for Performance
The update to use badge images for social links in the README.md file enhances the visual appeal and provides a more interactive experience for users. However, ensure that the badge images are optimized for performance to prevent any unnecessary loading delays, especially on slower connections.
Comment on lines
157
to
179
| text-align: center; | ||
| padding: 0; | ||
| margin: 0; | ||
| } | ||
|
|
||
| .hidden { | ||
| display: none; | ||
| } | ||
|
|
||
| @media (min-width: 1024px) { | ||
| .lg\:block { | ||
| display: block; | ||
| } | ||
| .lg\:hidden { | ||
| display: none; | ||
| } | ||
| } | ||
|
|
||
| @media (min-width: 768px) { | ||
| .md\:block { | ||
| display: block; | ||
| } | ||
| } |
There was a problem hiding this comment.
ℹ️ Logic Error
Potential Conflict with Visibility Classes
The CSS class .hidden is defined to set display: none;. Ensure that this class is not applied to elements that should be visible on larger screens, as it might conflict with the .lg:block and .md:block classes. Consider using more specific class names or conditions to avoid unintended hiding of elements.
Comment on lines
47
to
57
|
|
||
|
|
||
| def get_installed_packages(): | ||
|
|
||
| try: | ||
| return { | ||
| # TODO: test | ||
| # TODO: add to opt out | ||
| "Installed Packages": { | ||
| dist.metadata["Name"]: dist.version | ||
| dist.metadata.get("Name"): dist.metadata.get("Version") | ||
| for dist in importlib.metadata.distributions() | ||
| } | ||
| } |
There was a problem hiding this comment.
ℹ️ Logic Error
Ensure Default Values for Metadata Retrieval
The change from dist.metadata["Name"] to dist.metadata.get("Name") and similarly for Version introduces a potential issue where None could be returned if the metadata keys do not exist. This could lead to unexpected behavior when the returned dictionary is used. Consider providing a default value to ensure the dictionary always contains valid strings.
- dist.metadata.get("Name"): dist.metadata.get("Version")
+ dist.metadata.get("Name", "Unknown"): dist.metadata.get("Version", "0.0.0")Commitable Code Suggestion:
Suggested change
| def get_installed_packages(): | |
| try: | |
| return { | |
| # TODO: test | |
| # TODO: add to opt out | |
| "Installed Packages": { | |
| dist.metadata["Name"]: dist.version | |
| dist.metadata.get("Name"): dist.metadata.get("Version") | |
| for dist in importlib.metadata.distributions() | |
| } | |
| } | |
| dist.metadata.get("Name", "Unknown"): dist.metadata.get("Version", "0.0.0") |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
🔍 Review Summary
Purpose:
end_stateto "Indeterminate" to improve session handling.Key Changes:
end_stateset to "Indeterminate" for improved session handling.Impact:
Original Description
No existing description found