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

Who is Tim Matsumoto... and we should probably update the README #94

Closed
davibarreira opened this issue Jun 4, 2021 · 5 comments
Closed

Comments

@davibarreira
Copy link
Member

This issue is to deal with the README. I see that our documentation is already very far ahead from the README. So we should probably update it. I've been thinking about doing it, but I was wondering what you guys think it should have (usage examples with figures? list of functionalities? a bit of Optimal Transport theory?).

P.S: Who is Tim Matsumoto?

@devmotion
Copy link
Member

devmotion commented Jun 4, 2021

I would suggest reducing redundant information and pointing users mainly to the documentation (which could be improved and extended, of course). Probably it is still useful to quickly summarize the package in the README (without explicitly listing individual functions) and to keep the list of related packages and instructions for new contributors. A very short example of code (without (almost) any explanations) and/or a plot could be nice but I think anything more detailed should be added to the documentation.

@davibarreira
Copy link
Member Author

Cool. So we should then get rid of that "Overview of supported functionality" and replace by the quick summary and the example case. I believe your suggestion would be something similar to your repo here.

@devmotion
Copy link
Member

No, that's an example of a package where the README is the only and full documentation. I had something in mind like https://github.com/TuringLang/MCMCChains.jl, https://github.com/JuliaGaussianProcesses/KernelFunctions.jl, or https://github.com/JuliaGaussianProcesses/AbstractGPs.jl

@zsteve
Copy link
Member

zsteve commented Jun 4, 2021

Tim Matsumoto implemented the current quadratic regularisation solver (which also needs to be refactored)

I agree about really needing to update the README. I agree with @devmotion, we should have probably a brief overview of the supported functionality and a short 'quick start' guide for the most basic tasks. Don't need to explain the theory (this might be good for the docs though!)

@davibarreira
Copy link
Member Author

Thanks @zsteve. I was wondering who he was since he does not appear as a github contributor to the package.

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

No branches or pull requests

3 participants