search

econ-ark / DemARK

Demonstrations of how to use material in the Econ-ARK
https://econ-ark.github.io/DemARK/
Apache License 2.0
32 stars 93 forks source link

binder/requirements.txt file not in Repo #205

Closed AMonninger closed 9 months ago

AMonninger commented 1 year ago

In the readme.txt file it says that a user should clone this Repo and install requirements. The binder/requirements.txt file has been removed. I assume the user should install the requirements from the HARK repo.

llorracc commented 1 year ago

Mridul,

I think you (or maybe Cam?) updated Demark to use environment instead of requirements.TXT. Could you update instructions in the read me accordingly?

Sent from Gmail Mobile

On Tue, Sep 12, 2023 at 1:38 PM Adrian Monninger @.***> wrote:

In the readme.txt file it says that a user should clone this Repo and install requirements. The binder/requirements.txt file has been removed. I assume the user should install the requirements from the HARK repo.

— Reply to this email directly, view it on GitHub https://github.com/econ-ark/DemARK/issues/205, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKCK77FSKJSTSLMOVSWX23X2CMXTANCNFSM6AAAAAA4VF6PSE . You are receiving this because you are subscribed to this thread.Message ID: @.***>

llorracc commented 1 year ago

@MridulS, can you do the unticked to-do in Cam's issue, and put somewhere the appropriate corresponding instructions for HARK (which still has a requirements.txt, not an environment.yml, approach to environments)?

Thanks

MridulS commented 9 months ago

This was fixed in https://github.com/econ-ark/DemARK/commit/7d77bdb4defbeca0096226ddb62303ea26e4ad37