Skip to content

Update tests to PoPS Core with rounding#73

Closed
wenzeslaus wants to merge 5 commits intomainfrom
update-to-rounding
Closed

Update tests to PoPS Core with rounding#73
wenzeslaus wants to merge 5 commits intomainfrom
update-to-rounding

Conversation

@wenzeslaus
Copy link
Member

Mortality rate was decreased from 0.5 to 0.4 since more hosts are now dying because of rounding up which in turn causes less spread. The new value keeps the spread on a similar levels, although the values are of course different.

Tests test_sei_treatments_removal_ratio_to_all and test_sei_treatments_pesticide_ratio_to_all have same values for individual outputs at year 2021 because the simple and pesticide treatments are now more similar before one used floor and the other ceil.

Mortality rate was decreased from 0.5 to 0.4 since more hosts are now dying because of rounding up which in turn causes less spread. The new value keeps the spread on a similar levels, although the values are of course different.

Tests test_sei_treatments_removal_ratio_to_all and test_sei_treatments_pesticide_ratio_to_all have same values for individual outputs at year 2021 because the simple and pesticide treatments are now more similar before one used floor and the other ceil.
@wenzeslaus
Copy link
Member Author

This needs to have ncsu-landscape-dynamics/pops-core#218 merged to update the submodule.

@wenzeslaus
Copy link
Member Author

Replaced by PRs for individual changes to track the changes better. The values here are not useful anymore because they don't reflect the finished rounding code, only its initial version.

@wenzeslaus wenzeslaus closed this Jun 19, 2025
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.

1 participant