search

rdadolf / fathom

Reference workloads for modern deep learning methods.
Apache License 2.0
73 stars 21 forks source link

Some prerequisites should be optional #24

Closed z-a-f closed 7 years ago

z-a-f commented 7 years ago

I noticed there are dependencies that are not needed for correct operation of the package. For example, tqdm is just a cool way of visualizing, but is not critical. I would consider making non-critical parts optional.

rdadolf commented 7 years ago

Hi Zafar,

Good seeing you at BARC. You're absolutely right about the package deps. We'd like to remove that one as well as some of the preprocessing dependencies that could probably be integrated without too much effort. If you have a pull request, I'd be happy to test it.

Thanks!

z-a-f commented 7 years ago

I will definitely look into it -- next week (pulling allnighters for some deadlines right now :)