Skip to content

Conversation

@raghavdixit99
Copy link
Owner

@raghavdixit99 raghavdixit99 commented Dec 30, 2024

🔍 Review Summary

Purpose:

  • Enhance session management, user interaction, and system reliability.

Changes:

  • Enhancement: Default session end state set to "Indeterminate"; integrated Entelligence chat pop-up.
  • Refactor: Optimized autogen reload, session start logic, and package retrieval.
  • Test: Updated test cases with automatic request mocking and refined request handling.

Impact:

  • Improved user interaction, adaptability, and system performance.
Original Description

📥 Pull Request

📘 Description
Briefly describe the changes you've made.

🧪 Testing
Describe the tests you performed to validate your changes.

HowieG and others added 15 commits October 30, 2024 13:37
…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 <howardbgil@gmail.com>

* 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 <siyangqiu@gmail.com>
Co-authored-by: Howard Gil <howardbgil@gmail.com>
* 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 <areibman@gmail.com>
…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 <areibman@gmail.com>
@ai-pr-reviewer-tester-app
Copy link

🚀 Code Review Initiated

The review process for this pull request has started. Our system is analyzing the changes for:

  • Code quality, performance, and potential issues
  • Adherence to project guidelines
  • Integration of user-provided learnings

You will receive structured and actionable feedback shortly! ⏳

@ai-pr-reviewer-tester-app
Copy link

🚀 Code Review Initiated

The review process for this pull request has started. Our system is analyzing the changes for:

  • Code quality, performance, and potential issues
  • Adherence to project guidelines
  • Integration of user-provided learnings

You will receive structured and actionable feedback shortly! ⏳

@ai-pr-reviewer-tester-app
Copy link

Walkthrough

This pull request enhances the AgentOps repository by setting the default session end state to "Indeterminate" for better session management. It introduces the Entelligence chat pop-up for improved user interaction and optimizes the autogen reload process. Various test cases have been updated for increased reliability and performance, collectively boosting the system's functionality and maintainability.

Changes

File(s) Summary
agentops/session.py Set default end_state to "Indeterminate" for new sessions.
docs/v1/scripts/entelligence.js, docs/v1/styles/styles.css Integrated Entelligence chat pop-up and updated styles for support.
agentops/__init__.py, agentops/client.py Initialized autogen logger, adjusted session start logic, and refactored session creation.
agentops/helpers.py, agentops/host_env.py Replaced http.client with requests, added error handling, and simplified package retrieval logic.
tests/test_agent.py, tests/test_canary.py, tests/test_events.py, tests/test_pre_init.py, tests/test_record_action.py Enhanced test setups with automatic request mocking, adjusted wait times, and refined request handling.
Entelligence.ai can learn from your feedback. Simply add 👍 / 👎 emojis to teach it your preferences. More shortcuts below

Emoji Descriptions:

  • ⚠️ Potential Issue - May require further investigation.
  • 🔒 Security Vulnerability - Fix to ensure system safety.
  • 💻 Code Improvement - Suggestions to enhance code quality.
  • 🔨 Refactor Suggestion - Recommendations for restructuring code.
  • ℹ️ Others - General comments and information.

Interact with the Bot:

  • Send a message or request using the format:
    @bot + *your message*
Example: @bot Can you suggest improvements for this code?
  • Help the Bot learn by providing feedback on its responses.
    @bot + *feedback*
Example: @bot Do not comment on `save_auth` function !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants