Use tqdm instead of custom progress bar

econ-ark / DemARK

Demonstrations of how to use material in the Econ-ARK

https://econ-ark.github.io/DemARK/

Apache License 2.0

33 stars 93 forks source link

Use tqdm instead of custom progress bar #74

Closed MridulS closed 5 years ago

MridulS commented 5 years ago

will fix issues like https://github.com/econ-ark/DemARK/issues/52 from happening in the future

llorracc commented 5 years ago

@MridulS could you check whether there is now any reason for the sys.path.insert(0, os.path.abspath('../lib')) code in any of these? I think that was only there for the util thing. If there's no other need for it, please remove it.

MridulS commented 5 years ago

@llorracc you are right, that's not required anymore.

llorracc commented 5 years ago

OK, I'll merge when you do a PR to remove these. (Please check whether any REMARKs do this too).