Skip to content

metainsuranceorg: Abstract out obligation into a class#121

Open
rht wants to merge 1 commit intoINET-Complexity:masterfrom
rht:obligations
Open

metainsuranceorg: Abstract out obligation into a class#121
rht wants to merge 1 commit intoINET-Complexity:masterfrom
rht:obligations

Conversation

@rht
Copy link
Contributor

@rht rht commented Dec 9, 2018

This has 2 benefits:

  1. An object with __slots__ predefined is faster to access than a dict entry, and also it consumes far less memory.
  2. There is already an Obligation class defined in https://github.com/rht/py-distilledESL/blob/master/economicsl/obligations.py, although it has more information than is needed with the insurance model. There are also other useful functions elsewhere in the library that can be reused in refreshing the obligation list for each time.

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