Skip to content
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

Mention ReadStatTables.jl in documentation #3252

Merged
merged 4 commits into from
Dec 24, 2022

Conversation

junyuan-chen
Copy link
Contributor

I suggest mentioning ReadStatTables.jl instead of StatFiles.jl in the documentation.

Both packages make use of ReadStat. However, StatFiles.jl does not really work well and freezes for files that are moderately large (>100MB?). It is not updated for several years and lacks features for it to be convenient in actual usage scenarios. ReadStatTables.jl has more convenience features, incorporates the development in the Julia data ecosystem and reads files much faster.

@bkamins
Copy link
Member

bkamins commented Dec 17, 2022

@davidanthoff - what are your plans regarding https://github.com/queryverse/StatFiles.jl maintenance?

We can either mention both in the docs or replace StatFiles.jl with ReadStatTables.jl as @junyuan-chen suggested.

@bkamins bkamins added the doc label Dec 17, 2022
@bkamins bkamins added this to the patch milestone Dec 17, 2022
@davidanthoff
Copy link
Contributor

StatFiles.jl still works, and I do have plans to get back to it at some point, so it is not abandoned. ReadStatTables.jl might well have more features, I haven't really looked into that. I'm indifferent whether both packages are mentioned or just one.

I do have some very ambitious plans for Queryverse in general, and at that point StatFiles.jl will be a key part of that. But these are plans, with no timeline, so it might well be quite a while before I go back to that work, right now I'm trying to finish up Juliaup and some VS Code extension stuff :)

@bkamins
Copy link
Member

bkamins commented Dec 17, 2022

@davidanthoff - I am well aware of your superb work on juliaup and vscode from which the whole community benefits. I will update this PR to reflect the fact that your package is not abandoned.

I have also edited some other relevant entries.

@junyuan-chen - are you OK with the changes?

@junyuan-chen
Copy link
Contributor Author

@bkamins These changes look good. Thank you!

docs/src/index.md Outdated Show resolved Hide resolved
docs/src/index.md Outdated Show resolved Hide resolved
docs/src/index.md Outdated Show resolved Hide resolved
docs/src/index.md Outdated Show resolved Hide resolved
Co-authored-by: Milan Bouchet-Valat <nalimilan@club.fr>
@bkamins bkamins merged commit 3483e1c into JuliaData:main Dec 24, 2022
@bkamins
Copy link
Member

bkamins commented Dec 24, 2022

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants