Production task MATH + FACT: Check theoretical and factual claims

[santiagohermo]

In this paper we will perform a general proofreading of the paper, including both the main text and the appendices.

We will implement tasks MATH and FACT from the gslab/ra-wiki.

Goals

For MATH:

• All nontrivial mathematical claims in the paper are documented.

For FACT:

• Every quantitative/factual claim made in the paper is either autofilled or is supported by (i) a citation, (ii) an entry in a table, (iii) a figure, or (iv) a result shown in a supporting pdf document within the drafts directory (e.g., text.pdf).
  • Log files external to the drafts do not qualify as supporting documents.
  • If a fact is supported by a citation, it is not necessary to check the paper or book we are citing to verify that it contains the fact.
• All content (tables/figures/discussion) in the online appendix or main appendix is referenced at least once in the paper.
• Within each section (tables/figures/discussion) of the online appendix, content appears in the order in which it is referenced in the paper.

Deliverables

• A list of all theoretical claims that are made in the text of the paper that are not supported by proofs in the paper, appendices, or claims.pdf. For example, we may say, "It is easy to show that equations A, B, and C together imply equation D." You should not include claims that are completely obvious. We’re looking for things where if somebody came back and said, "I don’t believe this is true" we would need to go back and do at least a couple of lines of algebra to confirm that we’re right.
  • Call it min_wage_rent_MATH_name.pdf
• A single pdf document with all unsupported claims clearly marked using Adobe Acrobat’s commenting tools and all claims supported externally (e.g. in text.pdf or online.pdf) highlighted with a note stating where they are supported (e.g. “supported in online appendix table 6.”)
  • Call it min_wage_rent_FACT_name.pdf

Relevant version of the paper: min_wage_rent.pdf

[martingallardo23]

One question here @santiagohermo, just to confirm. The deliverable for MATH is a separate .pdf with the derivations of those claims in the math section that are not explicitly derived in the paper, right?

[santiagohermo]

Así es @martingallardo23! La idea es que en el deliverable de MATH marques:

1. errores o cosas poco claras en proofs
2. todas las theoretical claims que no están supported (excepto las que son completely obvious)

Si ya terminaste uno de los dos mandalo separado, no es necesario que mandes los dos a la vez!

[martingallardo23]

Perdón por la demora @santiagohermo! Acá va el factual y en un rato subo el MATH, cuando termine de pasarlo. Respecto a FACT no encontré muchos problemas. Más que nada algunos autofill, (en particular el apéndice completo no tiene casi ningún autofill), y algunas figuras que no aparecen en orden. Igual cuando suba el otro lo voy a volver a revisar.

min_wage_rent_FACT_MG.pdf

[santiagohermo]

Buenísimo @martingallardo23, gracias che! Apenas puedo arranco a revisar.

[martingallardo23]

Mil perdones con la demora acá @santiagohermo. Cayeron los postdocs a BA esta semana y me tuvieron loco, así que recién me puedo poner a liquidar el MATH. Revisando la matemática creo que encontré un par de typos, si te parece los flaggeo en un pdf y los mando a #237

[santiagohermo]

No te hagas drama @martingallardo23! A mi también se me atrasaron los planes esta semana, así que estamos en schedule.

Dale, si te parece que es mejor flaggear los typos de forma separada feel free to post in #237. Gracias por meterle!

(Espero que ande todo lindo con los postdocs!)

[martingallardo23]

Se me había pasado tu comentario en https://github.com/diegogentilepassaro/min_wage_rent/issues/238#issuecomment-1181722439 @santiagohermo! Así que agregué los typos y los otros comentarios en un mismo pdf pero con distintos colores (comentarios amarillos son las partes a documentar y los verdes son los typos). Acá dejo el pdf comentado y en breve mando un overleaf donde estoy pasando las derivaciones que no están explícitas en el paper.

min_wage_rent_MATH_MG.pdf

[santiagohermo]

Gracias @martingallardo23!

Así que agregué los typos y los otros comentarios en un mismo pdf pero con distintos colores (comentarios amarillos son las partes a documentar y los verdes son los typos).

Suena bien!

Acá dejo el pdf comentado y en breve mando un overleaf donde estoy pasando las derivaciones que no están explícitas en el paper.

Podés agregar esas derivaciones en paper_slides/paper/claims.lyx?

[martingallardo23]

La verdad no había notado la existencia de ese archivo @santiagohermo. Parece que varios de los que marqué ya están ahí! Ahora agrego los que faltan. Avisame si te parece que hay algo más que valga la pena agregar

[santiagohermo]

About FACT @martingallardo23. Thanks for the careful review! My replies are in min_wage_rent_FACT_MG_SH.pdf

I implemented a few changes in the branch for #237. I think it makes sense to keep everything together.

Let me know when you finished with MATH and I will take a look at it.

[santiagohermo]

Review of FACT in PR #242. fyi @martingallardo23

[martingallardo23]

Gracias @santiagohermo! Creo que no veo tus comentarios en el pdf, puede ser?

[santiagohermo]

Bajé el PDF en https://github.com/diegogentilepassaro/min_wage_rent/issues/238#issuecomment-1186725334 y me aparecen normalmente:

Qué te aparece cuando bajas el pdf y lo abrís @martingallardo23? Podés mandar un screenshot?

[martingallardo23]

Ahora sí @santiagohermo! Lo había abierto con el chrome y por algún motivo no me aparecía, pero ahí lo abrí con el Adobe y se ven. Ahí lo miro!

[santiagohermo]

About MATH @martingallardo23. Thanks for the useful comments!

I implemented fixes in https://github.com/diegogentilepassaro/min_wage_rent/commit/db8827d287f63be2d4c06a2c6f11abe59056d52a, and my replies are in min_wage_rent_MATH_MG_SH.pdf. There were a few things that I had missed.

Can you confirm that all the comments you raised have been resolved? Once you are done, maybe you can review the claims note if necessary and close this issue (we can add it to the PR #242)

[martingallardo23]

Ahí van algunos comentarios de FACT @santiagohermo. Los comentarios verdes son los resueltos, quedaron 2 para revisar un poco más

min_wage_rent_FACT_MG_SH_MG.pdf

[santiagohermo]

Gracias @martingallardo23! Mis respuestas en min_wage_rent_FACT_MG_SH_MG_SH.pdf, con edits al paper en https://github.com/diegogentilepassaro/min_wage_rent/commit/e43845146c9423fb2aa3da338e88b592f97b2aa2

Si estás contento con las respuestas creo que la parte de FACT estaría lista.

[martingallardo23]

All good con la parte de FACT @santiagohermo! Respecto al percent de population y zips en el baseline leí rápido y estaba haciendo la cuenta sobre el total y no sobre los urban. Mala mía! Respecto al otro comentario creo que ya estamos!

Ahora reviso el MATH

[martingallardo23]

Ahí va el MATH @santiagohermo. Quedó una cosa para rever nada más.

min_wage_rent_MATH_MG_SH_MG.pdf

[santiagohermo]

Gracias @martingallardo23! Respuestas en min_wage_rent_MATH_MG_SH_MG_SH.pdf con un pequeño cambio en https://github.com/diegogentilepassaro/min_wage_rent/commit/588669f81404dcf3d731c70fa5d5b8b1eb0d139f.

Ahí vi tu punto y lo arreglé :) Si estás de acuerdo ya estamos acá!

[martingallardo23]

Ahí vi tu punto y lo arreglé :) Si estás de acuerdo ya estamos acá!

Sounds good @santiagohermo!

[santiagohermo]

Continues in #242

[santiagohermo]

Summary: In this issue we conducted production task PROOF. Final versions of annotated PDFs:

• FACT: https://github.com/diegogentilepassaro/min_wage_rent/issues/238#issuecomment-1189564085
• MATH: https://github.com/diegogentilepassaro/min_wage_rent/issues/238#issuecomment-1190307494

Task was reviewed jointly with #237.

Changes merged to master in https://github.com/diegogentilepassaro/min_wage_rent/commit/b86d3754a60574c6d73af6020108bef1c4a4f125