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.

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

AgentType class attribute AgentCount never initialized #493

Closed
sbenthall opened this issue Feb 7, 2020 · 5 comments · Fixed by #901
Closed

AgentType class attribute AgentCount never initialized #493

sbenthall opened this issue Feb 7, 2020 · 5 comments · Fixed by #901
Assignees
Milestone

Comments

@sbenthall
Copy link
Contributor

Several methods in the class AgentType refer to a class attribute self.AgentCount.

But this attribute is never initialized in the class code, or documented in the API docs.

It is mentioned in some of the other notebooks and examples documentation.
It looks like it plays a role that is not essential to the definition of an agent, but rather is there for convenience for implementing certain simulation methods.

@mnwhite
Copy link
Contributor

mnwhite commented Feb 7, 2020 via email

@sbenthall
Copy link
Contributor Author

Is the simulation functionality more or less distinct from the solver functionality?

@llorracc
Copy link
Collaborator

llorracc commented Feb 8, 2020 via email

@sbenthall
Copy link
Contributor Author

Super.

I had a good meeting with Pablo today and a better idea of how Dolo works.
What I learned from him will definitely be helpful in integrating HARK and Dolo.
I hope to have some demonstrations related to that in DARKolo soon.

I see your point about disentangling the simulation from the solution in HARK. I'll make an issue for it.

@sbenthall
Copy link
Contributor Author

Related to #660

sbenthall added a commit that referenced this issue Jan 7, 2021
makes proper class docstring for AgentType, with AgentCount, fixes #493
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants