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

v0.4 #642

Draft
wants to merge 66 commits into
base: main
Choose a base branch
from
Draft

v0.4 #642

wants to merge 66 commits into from

Conversation

teocns
Copy link
Contributor

@teocns teocns commented Jan 12, 2025

Nothing to see here, yet

teocns added 30 commits January 9, 2025 15:04
@teocns
refactor: reorganize session management into dedicated components
eb072db 
Split session logic into dedicated components for better separation of
concerns
and maintainability:

- Move session code to dedicated session/ module
- Split Session class into:
  - Session: Data container with minimal public API
  - SessionManager: Handles lifecycle and state management
  - SessionApi: Handles API communication
  - SessionTelemetry: Manages event recording and OTEL integration

Key fixes:
- Proper UUID and timestamp serialization in events
- Consistent API key header handling
- Correct token cost formatting in analytics
- Proper session ID inheritance
- Tags conversion and validation
- Event counts type handling

This refactor improves code organization while maintaining backward
compatibility
through the session/__init__.py module.

Signed-off-by: Teo <[email protected]>
@teocns
Merge branch 'feat/session-refactoring' into feat/v0.4
88cdd43
@teocns
feat(session): add README for session package documentation
f9b141f
@teocns
Import agentops/_telemetry from otel/v4 branch
9b6ed9e 
Signed-off-by: Teo <[email protected]>
@teocns
add tests/_telemetry from otel/v4 branch
0fc7701 
Signed-off-by: Teo <[email protected]>
@teocns
test: add __init__ to make tests/ a package
051a85b
@teocns
test: add llm_event_spy fixture for tests
2d97251
@teocns
test: add VCR.py fixture for HTTP interaction recording
4a19dab
@teocns
deps: group integration-testing
51e2da2
@teocns
test: add fixture to mock package availability in tests
0180e2c
@teocns
test: Add integration tests for OpenAI provider and features
73a0110
@teocns
test: add tests for concurrent API requests handling
538bf98
@teocns
Improve vcr.py configuration
d679b93 
Signed-off-by: Teo <[email protected]>
@teocns
ruff
93744ce 
Signed-off-by: Teo <[email protected]>
@teocns
chore(pyproject): update pytest options and loop scope
512c95d
@teocns
chore(tests): update vcr.py ignore_hosts and options
e29d2b2
@teocns
pyproject.toml
8f02961 
Signed-off-by: Teo <[email protected]>
@teocns
centralize teardown in conftest.py (clear singletons, end all sessions)
a012a0f 
Signed-off-by: Teo <[email protected]>
@teocns
change vcr_config scope to session
f51850e 
Signed-off-by: Teo <[email protected]>
@teocns
integration: auto start agentops session
e22513b 
Signed-off-by: Teo <[email protected]>
@teocns
Move unit tests to dedicated folder (tests/unit)
cb014b2 
Signed-off-by: Teo <[email protected]>
@teocns
Isolate vcr_config import into tests/integration
2c3b19d 
Signed-off-by: Teo <[email protected]>
@teocns
configure pytest to run only unit tests by default, and include integ…
6dbe54b 
…ration tests only when explicitly specified.

Signed-off-by: Teo <[email protected]>
@teocns
ci(python-tests): separate job between unit-integration tests
fb2be21
@teocns
set python-tests timeout to 5 minutes
caa08df 
Signed-off-by: Teo <[email protected]>
@teocns
ruff
120c455 
Signed-off-by: Teo <[email protected]>
@teocns
Implement jwt fixture, centralized reusable mock_req into conftest.py
37edbc0 
Signed-off-by: Teo <[email protected]>

reauthorize

Signed-off-by: Teo <[email protected]>
@teocns
ci(python-tests): simplify env management, remove cov from integratio…
6be254a 
…n-teests

Signed-off-by: Teo <[email protected]>
@teocns
ruff
4d0d5a2 
Signed-off-by: Teo <[email protected]>
@teocns
fix: cassette for test_concurrent_api_requests
2a860c8 
Signed-off-by: Teo <[email protected]>
teocns and others added 19 commits January 15, 2025 15:57
@teocns
Introduce API Layer
bcb5ead 
Signed-off-by: Teo <[email protected]>

Make session use new API Layer

Signed-off-by: Teo <[email protected]>
@teocns
v2
9666f20 
Signed-off-by: Teo <[email protected]>
@teocns
base api add post
97f0ec1 
Signed-off-by: Teo <[email protected]>
@teocns
session tests passing
ab059ed 
Signed-off-by: Teo <[email protected]>
@teocns
SessionExporter: make correct use of SessionApiClient
3e84459 
Signed-off-by: Teo <[email protected]>

fix fixture name

Signed-off-by: Teo <[email protected]>
@teocns
Merge branch 'feat/session-refactoring' into feat/v0.4
069d7e7
@the-praxs @teocns
Merge branch 'main' into feat/v0.4
02c9b64
@the-praxs @teocns
fix for autogen
1025af0
@teocns
Merge branch 'feat/optimal-test-suite' into feat/v0.4
88ce5d5
@teocns
move tests/telemetry to tests/unit/telemetry/ to align with #637
ca84cd5 
Signed-off-by: Teo <[email protected]>
@teocns
Merge branch 'feat/optimal-test-suite' into feat/v0.4
b10deca
@teocns
telemetry: add DESIGN.mermaid.md
692e73f 
Signed-off-by: Teo <[email protected]>
@teocns
OTEL Log Capture: allow capturing logging and prints from terminal
e676993 
Includes a test

Signed-off-by: Teo <[email protected]>
@teocns
Merge: main (Test suite v0.4)
bcbe850 
Signed-off-by: Teo <[email protected]>
@teocns
design
36c8cbf 
Signed-off-by: Teo <[email protected]>
@teocns
Merge branch 'main' into feat/v0.4
65c04fc
@teocns
ruff
aa09623 
Signed-off-by: Teo <[email protected]>
@teocns
style(pyproject.toml): allow F403 (import *)
9ecc8f7
@teocns
tests: rename test_event_converter to test_encoders
e94a7fb
teocns added 2 commits January 15, 2025 16:16
@teocns
proposla
ed606e7 
Signed-off-by: Teo <[email protected]>
@teocns
update readme.md
7bb0162 
Signed-off-by: Teo <[email protected]>
@codecov Codecov
Copy link

codecov bot commented Jan 15, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 67.86091% with 305 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
agentops/session/log_capture.py 27.50% 86 Missing and 1 partial ⚠️
agentops/partners/autogen_logger.py 0.00% 46 Missing ⚠️
agentops/session/manager.py 67.76% 20 Missing and 19 partials ⚠️
agentops/telemetry/exporters/event.py 70.83% 24 Missing and 4 partials ⚠️
agentops/telemetry/exporters/session.py 69.51% 18 Missing and 7 partials ⚠️
agentops/session/session.py 73.33% 10 Missing and 10 partials ⚠️
agentops/api/session.py 70.58% 15 Missing ⚠️
agentops/telemetry/manager.py 82.60% 7 Missing and 5 partials ⚠️
agentops/session/telemetry.py 73.17% 6 Missing and 5 partials ⚠️
agentops/telemetry/processors.py 89.36% 1 Missing and 4 partials ⚠️
... and 6 more

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@teocns @the-praxs