Ian Preston

I wasn't able to get the environment working as it is in the repo. This was on Windows 10. Having everything come from conda-forge seems to fix all sorts of...

Migrate from os.path to Pathlib

8

Using os.path to handle files makes the implementation trickier, especially for cross platform support. [Pathlib is great](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/). Time to switch to pathlib

Testing/CI only happens for one version of python

2

Would be cool to use tox or something to check against different versions of python, maybe different library versions too.

See issue #15 HDF5 seemed like a cool storage solution when I was developing this, and it's still pretty handy. I think a SQL backend might be more useful though,...

User experience is not well tested/documented

1

Right now this package is primarily used by the maintainer. This means that new users will likely have trouble using the package. This is a catch all issue for discussion...

Update tables doesn't check for file existence

1

Probably other functions too. At least update_tables doesn't. Noticed it in the StatsCan class, but this should be fixed at the function level

I'm starting to add example code in the docstrings, these should be part of the test suite

There are a few api calls that StatsCan provides that haven't been implemented yet. They have methods created in scwds.py so they're easy to identify, but they just ```pass``` right...

Right now there's a couple print statements (gross) but for the most part the library doesn't provide feedback on what it's doing. This is particularly troubling when downloading a significant...

Some of the API calls have the option to return results in French. Would be good to implement and add as an optional parameter to functions that support it.