-
Notifications
You must be signed in to change notification settings - Fork 19
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
Mentioning similar work: datastorr, fst, etc. #21
Labels
Comments
@wlandau, happy to do it. Thanks for pointing these out as well, I was not aware of them. I think they can be quite complementary with DataPackageR. |
19 tasks
@wlandau thanks for the review and the suggestions, I look forward to addressing and implementing these changes, I think they'll make the package significantly better. |
gfinak
added a commit
that referenced
this issue
Jul 3, 2018
Glad this is helpful. Your new "Similar work" section is great. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi @gfinak,
I am reviewing
DataPackageR
for ropensci/software-review#230, and I have decided to create separate issues for my comments so we can discuss more easily. Once I have posted all my comments, I will submit my formal review to ropensci/software-review#230, which will cross reference them. I recommend creating a custom label to keep track of these issues.Anyway, as mentioned in this thread from the 2018 Unconference, @noamross has a slide deck in which he describes existing work on data packaging, such as
datastorr
andfst
. Would you consider adding a section to theREADME
that points to this existing work and emphasizes the strengths ofDataPackageR
in this landscape?The text was updated successfully, but these errors were encountered: