Add anomaly agent #133
Merged
Add anomaly agent #133
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
…flow - Update .example.env with Langsmith-related environment variables - Simplify requirements.txt to a more concise list of dependencies - Refactor anomaly detection in llmalert.py and llm/agent.py to use AnomalyAgent - Modify anomaly detection to return a DataFrame with consistent column names
|
Your pull request is automatically being deployed to Dagster Cloud.
|
- Remove separate LLM provider modules (anthropic.py, openai.py, detect.py) - Simplify agent.py by renaming detect_anomalies_agent to detect_anomalies - Update import in llmalert.py to reflect function name change - Remove models.py with Pydantic schemas
- Rename `df_anomalies_long` to `df_anomalies` in agent.py - Remove debug logging of detected anomalies in llmalert.py
- Remove OpenAI and Anthropic packages from requirements.txt and requirements.compile - Align with previous refactoring of LLM anomaly detection module
- Refactor LLM Alerts section to focus on anomaly-agent approach - Update description to highlight new agent-based anomaly detection - Remove references to OpenAI and Anthropic direct integrations - Add note about future Ollama support - Adjust formatting and terminology
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.
This pull request includes significant changes to the anomaly detection system, specifically transitioning from using individual LLM providers like OpenAI and Anthropic to a unified
anomaly-agentapproach. Additionally, there are updates to the environment variables and documentation to reflect these changes.Updates to anomaly detection:
openaiandanthropicwithanomaly-agentinanomstack/jobs/llmalert.pyand created a newanomstack/llm/agent.pyfile to handle anomaly detection withanomaly-agent. [1] [2] [3]anomstack/llm/anthropic.py,anomstack/llm/detect.py,anomstack/llm/models.py, andanomstack/llm/openai.py. [1] [2] [3] [4]Environment variables:
LANGSMITHandOPENAI_API_KEYin.example.envfile to support the newanomaly-agentsetup.Documentation updates:
README.mdto reflect the newLLM Agent Alertsterminology and provided links to theanomaly-agentrepository. [1] [2] [3]Dependency changes:
requirements.compileto removeanthropicandopenaidependencies and addanomaly-agent. [1] [2]