File extension must be .xls or .xlsx to write to an existing CEQ Master Workbook (requires Stata 13 or newer)

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

File extension must be .xls or .xlsx to write to an existing CEQ Master Workbook (requires Stata 13 or newer) #24

Closed MikhailMatytsin closed 6 years ago

MikhailMatytsin commented 6 years ago

Dear team,

There are few Stata commands that do not work. We have problems with the following ones: • ceqdom that fills E8, • ceqef that fille E9 • ceqefext that fills E14 • ceqdomext that fills E17 When we run those, the Stata says “File extension must be .xls or .xlsx to write to an existing CEQ Master Workbook (requires Stata 13 or newer)” (see the screenshot below). Do you what could be the problem? The Stata version that I have is 14 and the excel files has extension .xlsx.

Thank you very much!

Best, Misha

igutierrezm commented 6 years ago

Hi, @MikhailMatytsin @skhiggins :

This is an old bug that can occur if your filepath has periods elsewhere. The pull request #32 should fix this behavior.

You can find a test's do-file in tests\dofiles\test-issue#24.do.
You can find a log of the test's do-file in tests\logs\test-issue#24.log.

Thank you very much! (for reminding us this bug)

PS:

In order to prevent future related errors, I have designed a new program, "ceq_parse_using.ado":

You can find the ado-file in ceq_parse_using.ado.
You can find a test's do-file in tests\dofiles\test-ceq_parse_using.do.
You can find a log of the test's do-file in tests\logs\test-ceq_parse_using.log.

skhiggins commented 6 years ago

Thanks @igutierrezm, I'm closing this since it solves the issue. Could you also make the code of the other commands with an .xls/.xlsx check more elegant by replacing that code with your ceq_parse_using program?

igutierrezm commented 6 years ago

2018-04-01 23:28 GMT-03:00 Sean Higgins notifications@github.com:

Closed #24 https://github.com/skhiggins/CEQStataPackage/issues/24.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/skhiggins/CEQStataPackage/issues/24#event-1551365233, or mute the thread https://github.com/notifications/unsubscribe-auth/ASp42RUSkCe3u5EC5r6pt1Ue922vEXgaks5tkYy4gaJpZM4S4Du9 .