search

EconForge / dolark.py

2 stars 12 forks source link

dolark still depends on dolo.compiler.language #15

Closed sbenthall closed 3 years ago

sbenthall commented 3 years ago

dolark.model still depends on dolo.compiler.language, though this latter module has been removed from the most recent version of dolo

https://github.com/EconForge/dolark/blob/41deaa026b1943c4829c6b7b17c7b57fdd0aec95/dolark/model.py#L3

albop commented 3 years ago

@normannr should have fixed that very soon.

On Wed, Nov 11, 2020, 7:32 PM Sebastian Benthall notifications@github.com wrote:

dolark.model still depends on dolo.compiler.language, though this latter module has been removed from the most recent version of dolo

https://github.com/EconForge/dolark/blob/41deaa026b1943c4829c6b7b17c7b57fdd0aec95/dolark/model.py#L3

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/EconForge/dolark/issues/15, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACDSKNGACF7PEFJA4JWDH3SPLKCXANCNFSM4TSKKR2Q .

albop commented 3 years ago

in dolo, we can see eval_data is imported from dolang. The line must be replaced by from dolang.language import eval_data