-
Notifications
You must be signed in to change notification settings - Fork 75
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
docs: fix doc & type definitions in the projectors module #1221
base: master
Are you sure you want to change the base?
Conversation
7df2f45
to
3b0246b
Compare
7038fff
to
c8e1271
Compare
d2960a8
to
a7904e5
Compare
c8e1271
to
49400fc
Compare
33465fc
to
7b37709
Compare
49400fc
to
54126d6
Compare
54126d6
to
ebd3829
Compare
3601736
to
9317b7f
Compare
d53f125
to
4a99cb4
Compare
57d74a0
to
584e97f
Compare
4a99cb4
to
dad88a1
Compare
openfisca_core/commons/types.py
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why re-creating types here instead of importing them directly ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm still unsure what is the best way to go.
25cb01f
to
2b24884
Compare
83e521c
to
738652a
Compare
738652a
to
c54d3a5
Compare
@@ -125,7 +141,7 @@ def period(value) -> Period: | |||
Period((<DateUnit.DAY: 'day'>, Instant((2021, 1, 1)), 1)) | |||
|
|||
>>> period(DateUnit.ETERNITY) | |||
Period((<DateUnit.ETERNITY: 'eternity'>, Instant((1, 1, 1)), inf)) | |||
Period((<DateUnit.ETERNITY: 'eternity'>, Instant((1, 1, 1)), 0)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be -1 here ?
@@ -164,15 +180,9 @@ def period(value) -> Period: | |||
return Period((DateUnit.DAY, instant(value), 1)) | |||
|
|||
# We return an "eternity-period", for example | |||
# ``<Period(('eternity', <Instant(1, 1, 1)>, inf))>``. | |||
# ``<Period(('eternity', <Instant(1, 1, 1)>, 0))>``. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be -1 here ?
0da8886
to
0e10d12
Compare
Fixes #1239
Depends on #1224
Depended upon by #1146
Technical changes