Skip to content

Commit

Permalink
Merge pull request #857 from openfisca/refactor-entities
Browse files Browse the repository at this point in the history
Distinguish population-level from model-level abstractions
  • Loading branch information
Morendil authored Apr 12, 2019
2 parents ae1dad2 + f01bff4 commit c64f908
Show file tree
Hide file tree
Showing 23 changed files with 794 additions and 850 deletions.
14 changes: 14 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,19 @@
# Changelog

# 32.0.0 [#857](https://github.com/openfisca/openfisca-core/pull/857)

### Breaking changes

- Split the "Entity" class hierarchy (Entity, PersonEntity, GroupEntity) into two parallel hierarchies, representing respectively the abstract, model-level information (classes named Entity etc.) and the population-level information (classes named Population and GroupPopulation)
- As a result, the first parameter passed to a formula is now a Population instance
- Much more detail (and class diagrams) in the PR description
- Remove support from the syntax `some_entity.SOME_ROLE` to access roles (where `some_entity` is the entity passed to a formula).

### Migration details

- Use the standard SomeEntity.SOME_ROLE instead. (Where SomeEntity is the capitalized entity or instance, Household.PARENT.)
- Code that relied excessively on internal implementation details of Entity may break, and should be updated to access methods of Entity/Population instead.

# 31.0.1 [#840](https://github.com/openfisca/openfisca-core/pull/840)

- Improve usability of Enum values:
Expand Down
Loading

0 comments on commit c64f908

Please sign in to comment.