-
Notifications
You must be signed in to change notification settings - Fork 17
Added agent_id tracking in agent reporter collection #182
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.
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
base: main
Are you sure you want to change the base?
Conversation
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
|
Hi Agrim, thanks for the PR!
Before pushing the next commit, run the test suite locally with pytest and resolve any failures. As a hint, the current failure seems related to the shape of the DataFrame. |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #182 +/- ##
==========================================
+ Coverage 92.54% 92.62% +0.07%
==========================================
Files 14 14
Lines 1717 1748 +31
==========================================
+ Hits 1589 1619 +30
- Misses 128 129 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
for more information, see https://pre-commit.ci
This PR enhances the agent data collection process by ensuring that each agent reporter includes a unique identifier, the agent_id in the collected data.
Also, agent_reporters previously collected only the values of the requested columns (eg- wealth) but now includes the unique agent_id making debugging and analysis easier.
This PR solving the issue #177