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

AER-3128 Animal housing less animal type based #292

Merged

Conversation

BertScholten
Copy link
Member

Basic housing is now based on animal housing category instead of the animal type. This is due to some animal types having multiple basic animal housing categories. Similarly, additional system factors are now based on animal housing category instead of the animal type. This is due to having different reduction factors for some animal housing + additional system combinations. Implementations will have to be adjusted accordingly.

Basic housing is now based on animal housing category instead of the animal type. This is due to some animal types having multiple basic animal housing categories.
Similarly, additional system factors are now based on animal housing category instead of the animal type. This is due to having different reduction factors for some animal housing + additional system combinations.
Implementations will have to be adjusted accordingly.
Copy link
Contributor

@MichielJanssen-DAT MichielJanssen-DAT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Doesn't matter too much as we're not directly using this in test cases, but makes more sense.
@BertScholten BertScholten merged commit b51f072 into aerius:main Aug 7, 2024
1 check passed
@BertScholten BertScholten deleted the AER-3128_animal_housing_change branch August 7, 2024 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants