-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add parquet unit tests. Add static html generator output for parquet front end. #84
Conversation
…l files look good
…and apply to pyproject.toml
Also closes #85 . Note that my set of software requirments is generated from requirements.txt can be freely modified now or in the future. It is meant as a convenience. |
Also resolves #86 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I reviewed the sqlite changes and this works.
My only concern is that the requirements.txt seems to be parquet specific and not DSI. do we want to withhold these? Otherwise we are significantly increasing the number of dependencies for the base DSI code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Issue 89 will address the requirements.txt as a per-example addition to the code
Resolves #81 and #82.