search

skhiggins / CEQStataPackage

CEQ Stata Package to estimate the impact of taxes and transfers on poverty and inequality following the Commitment to Equity framework. To install directly through Stata: ssc install ceq, replace
5 stars 12 forks source link

ceqassump-problemas con grupos de ingresos #30

Open sanmartineza opened 6 years ago

sanmartineza commented 6 years ago
  1. Los valores de los cuts de los grupos de ingreso están mal (ver screenshot)
  2. El ado file no completa la distribución por grupo de ingresos screenshot 2018-03-28 15 44 19
igutierrezm commented 6 years ago

Hi @sanmartineza , @skhiggins :

Thanks for finding this error. The problem within ceqinfra.ado was quite subtle (a temporary confusion with the row index). The pull request #34 should fix this behavior.

Best regards.

CrisLCarrera commented 6 years ago

Hola Ivan,

Corri el ado file ceqassump y aunque los valores de los cuts de los grupos de ingreso ya aparecen de manera correcta, aun sigue sin completar la distribución por grupo de ingresos.

Saludos y muchas gracias, Cris

igutierrezm commented 6 years ago

Hola: Ah, perdón no leí el issue completo y lo di por resuelto. Resolveré esto a la brevedad. Un abrazo!

El 9 de abril de 2018, 15:03, Cristina notifications@github.com escribió:

Hola Ivan,

Corri el ado file ceqassump y aunque los valores de los cuts de los grupos de ingreso ya aparecen de manera correcta, aun sigue sin completar la distribución por grupo de ingresos.

Saludos y muchas gracias, Cris

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/skhiggins/CEQStataPackage/issues/30#issuecomment-379840981, or mute the thread https://github.com/notifications/unsubscribe-auth/ASp42ZTU28nlu8vlrlHBtrrw2ubxQy4fks5tm6JegaJpZM4S-y1V .

Ruoxi-Rosie commented 6 years ago

Changes:

(https://github.com/skhiggins/CEQStataPackage/blob/436320c5b9f7db751bf97647e1a86ce5cfd96cbc/ceqassump.ado)

New files: * [`test-issue#30.ado`] (https://github.com/skhiggins/CEQStataPackage/blob/436320c5b9f7db751bf97647e1a86ce5cfd96cbc/tests/dofiles/test-issue%2330.do) tests the new ado file with Brazilian data * [`test-issue#30.log`] (https://github.com/skhiggins/CEQStataPackage/pull/49/files#diff-3966607bd5d94df8626e18dcf7d64e74) is a log of the test
skhiggins commented 6 years ago

Thanks Rosie. Please compare the markdown in the example I sent to the markdown here to make sure the comment is formatted correctly. See

https://guides.github.com/features/mastering-markdown/