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

T_cycle is unnecessary redundant parameter #1040

Open
sbenthall opened this issue Jul 12, 2021 · 0 comments
Open

T_cycle is unnecessary redundant parameter #1040

sbenthall opened this issue Jul 12, 2021 · 0 comments
Assignees
Milestone

Comments

@sbenthall
Copy link
Contributor

T_cycle is given as a parameter to AgentType.
But the software breaks if this is anything but the length of time-varying parameters, which elsewhere is collected directly.
It can be removed from the parameters list and replaced with an automated calculation in the constructor.

Though this is also a case where possibility of a 'cycle' being a 'lifecycle' i.e. cycles == 1, or a non-cyclic model, is confusing terminology.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants