search

kylebutts / didimputation

Difference-in-differences Imputation-based Estimator proposed by Borusyak, Jaravel, and Spiess (2021)
Other
46 stars 13 forks source link

adding ARMA_BIT64 in cpp #7

Closed fhollenbach closed 2 years ago

fhollenbach commented 2 years ago

With large datasets the command would throw the following error:

Error in make_V_star((Z data[, zz000weight]), (Z data[, zz000weight])[data$zz000treat == : SpMat::init(): requested size is too large; suggest to enable ARMA_64BIT_WORD

enabled ARMA_64BIT_WORD in code.cpp

kylebutts commented 2 years ago

Did you try this on your code and did it run afterwards?

fhollenbach commented 2 years ago

Hi,

Yes, I tried it on my side, both on a mac and a linux machine. I have not tried it on Windows yet though.

Thanks!

-- Venlig hilsen / Kind regards,

FLORIAN HOLLENBACH Associate Professor, PhD

Department of International Economics, Government and Business COPENHAGEN BUSINESS SCHOOL

Think before you print

On Wed, Aug 17, 2022 at 5:45 PM Kyle F Butts @.***> wrote:

Did you try this on your code and did it run afterwards?

— Reply to this email directly, view it on GitHub https://github.com/kylebutts/didimputation/pull/7#issuecomment-1218196269, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKPDWTQ2BJ36GEEQFTPNATVZUCK3ANCNFSM56Y7J46Q . You are receiving this because you authored the thread.Message ID: @.***>