Closed Mv77 closed 2 years ago
Mateo,
Could we have a Zoom conversation about this? I'm working on it right now and would like to strategize about how to merge the changes that we have both made.
I'm on now at:
https://zoom.us/j/9275945545?pwd=dE0xNDBVbmxRcnRTWGwzUWMzb0Zsdz09
On Thu, Dec 16, 2021 at 6:01 PM Mateo Velásquez-Giraldo < @.***> wrote:
@llorracc https://github.com/llorracc, here is a first finalized version of the Harmenberg Demark.
It is harmonized with the changes you pushed earlier today (around 3pm?).
The things it includes which you have not seen yet are:
- Checks of the Szeidl and Harmenberg convergence conditions.
- A discussion at the bottom about how, even though they integrate to the same thing, m under the PIW measure and m*P under the base measure are completely different things.
I hope you like it!
You can view, comment on, or merge this pull request online at:
https://github.com/econ-ark/DemARK/pull/179 Commit Summary
- 1e40ac5 https://github.com/econ-ark/DemARK/pull/179/commits/1e40ac5dfe2e69f16ad3600d2c718d77645a5d9e Minor edits with Will and Mateo
- 71dfb64 https://github.com/econ-ark/DemARK/pull/179/commits/71dfb64f17ebd91a468c209d31417105346f2448 Fix typo
- 664b2fa https://github.com/econ-ark/DemARK/pull/179/commits/664b2fa2ec24c7684b73ab82415a4f8d7ca0c2d3 Check harmenber and szeidl conditions
- e4c5078 https://github.com/econ-ark/DemARK/pull/179/commits/e4c5078837114d94a6e69f251ff0455d6d6d5739 Merge remote-tracking branch 'upstream/Harmenberg-Aggregation' into Harmenberg-Aggregation
- 5f55156 https://github.com/econ-ark/DemARK/pull/179/commits/5f55156c2f43cc24cc678107005d973f1089f390 Density plot and typo-correction
- 7dba028 https://github.com/econ-ark/DemARK/pull/179/commits/7dba028b36cabe969fe94a2a90f65deabb2fcebd Add Chris to Metadata
File Changes
(4 files https://github.com/econ-ark/DemARK/pull/179/files)
- M notebooks/DCEGM-Upper-Envelope.md https://github.com/econ-ark/DemARK/pull/179/files#diff-2ece4369cf4a602b947c6d02ac31596660fc32c9fb4a1fcd8d6775b80f95ca8a (6)
- M notebooks/Harmenberg-Aggregation.ipynb https://github.com/econ-ark/DemARK/pull/179/files#diff-31c69bc359a2b6ff8520d9bbe37e5ab382c36d6ec4efddc9fc59b64d5f729533 (216)
- M notebooks/Harmenberg-Aggregation.md https://github.com/econ-ark/DemARK/pull/179/files#diff-58b12c41441c0954fb38bc5065588ba61ff86cc8abd83d898c954eaef298405e (8)
- M notebooks/Harmenberg-Aggregation.py https://github.com/econ-ark/DemARK/pull/179/files#diff-93a44ba5ae65c191ef773f93c977dbb9a4dc4a81d60c86850277808aa87f0133 (125)
Patch Links:
— Reply to this email directly, view it on GitHub https://github.com/econ-ark/DemARK/pull/179, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKCK72QJOXGDBB34M63DZDURJVT5ANCNFSM5KHPSMPQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>
--
@llorracc, here is a first finalized version of the Harmenberg Demark.
It is harmonized with the changes you pushed earlier today (around 3pm?).
The things it includes which you have not seen yet are:
I hope you like it!