@davidorme suggested that multi-grid occupancy might work better without the AnimalCommunity class. I have spent some time refactoring the code to see if this is the case. The current version is not finished and probably has a number of errors (many tests are now broken) but this gives a strong sense of the resulting format.
Pros: I got to ditch a lot of hacky nonsense that I was doing to avoid circular imports and related issues.
Cons: I am aesthetically vexed by having all of this content crammed into AnimalModel.
So it does seem like this format is probably better.
I would appreciate some feedback on this format for multi-grid and the future of the Animal Model in comparison to my previous multi-grid attempt here #530.
Fixes # (issue)
Type of change
[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)
Key checklist
[x] Make sure you've run the pre-commit checks: $ pre-commit run -a
[ ] All tests pass: $ poetry run pytest
Further checks
[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works
Description
@davidorme suggested that multi-grid occupancy might work better without the AnimalCommunity class. I have spent some time refactoring the code to see if this is the case. The current version is not finished and probably has a number of errors (many tests are now broken) but this gives a strong sense of the resulting format.
Pros: I got to ditch a lot of hacky nonsense that I was doing to avoid circular imports and related issues. Cons: I am aesthetically vexed by having all of this content crammed into AnimalModel.
So it does seem like this format is probably better.
I would appreciate some feedback on this format for multi-grid and the future of the Animal Model in comparison to my previous multi-grid attempt here #530.
Fixes # (issue)
Type of change
Key checklist
pre-commitchecks:$ pre-commit run -a$ poetry run pytestFurther checks