Closed gabrielfior closed 3 weeks ago
The pull request introduces several modifications across multiple files, primarily focusing on enhancing the DeployablePredictionAgent
and DeployableTraderAgent
classes. Key changes include the addition of an agent_name
property in DeployablePredictionAgent
, updates to the after_process_market
method to include an agent_name
parameter, and modifications to the store_prediction
and store_trades
methods in the AgentMarket
, OmenAgentMarket
, and MetaculusAgentMarket
classes to incorporate this new parameter. Additionally, a new attribute agent_name
is added to the IPFSAgentResult
class, improving the metadata capabilities of the agents.
File Path | Change Summary |
---|---|
prediction_market_agent_tooling/deploy/agent.py |
- Added @property def agent_name(self) -> str in DeployablePredictionAgent class. - Updated after_process_market method to include agent_name parameter in both DeployablePredictionAgent and DeployableTraderAgent classes. - Modified langfuse_update_current_trace for session ID formatting. |
prediction_market_agent_tooling/markets/agent_market.py |
- Updated store_prediction and store_trades method signatures to include agent_name: str . |
prediction_market_agent_tooling/markets/omen/data_models.py |
- Added agent_name: str attribute in IPFSAgentResult class. |
prediction_market_agent_tooling/markets/omen/omen.py | - Updated store_predictionand store_tradesmethod signatures to include agent_name: str. <br> - Streamlined logic to utilize agent_namewhen creating IPFSAgentResult`. |
|
prediction_market_agent_tooling/markets/metaculus/metaculus.py |
- Updated store_prediction method signature to include agent_name: str . |
agent_market.py
to include an agent_name
parameter in the store_prediction
and store_trades
methods are directly related to the updates in the main PR, which also introduces an agent_name
parameter in the after_process_market
methods of DeployablePredictionAgent
and DeployableTraderAgent
.DeployableAgent
class in agent.py
, enhancing the integration with Langfuse, which aligns with the changes in the main PR that improve the langfuse_update_current_trace
method and overall Langfuse integration.process_market
method in DeployableTraderAgent
and the introduction of the ProcessedMarket
class are relevant as they relate to the overall market processing logic, which is a focus of the main PR.DeployableTraderAgent
and the introduction of the DeployablePredictionAgent
class are significant as they relate to the changes made in the main PR regarding the DeployablePredictionAgent
class and its methods.n_markets_to_fetch
variable in DeployablePredictionAgent
and the updates to the get_markets
method are relevant as they enhance the functionality of the agent, similar to the enhancements made in the main PR.get_markets
method as properties in DeployablePredictionAgent
aligns with the overall goal of improving the agent's functionality, which is also a focus of the main PR.Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
-> Adding agent_name to structure being uploaded to IPFS