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

Minimal context in docs #94

Closed
jsheunis opened this issue Dec 3, 2023 · 1 comment
Closed

Minimal context in docs #94

jsheunis opened this issue Dec 3, 2023 · 1 comment

Comments

@jsheunis
Copy link

jsheunis commented Dec 3, 2023

The docs contain a single paragraph explaining the context within which this package operates: https://scienxlab.org/redflag/index.html

I think it would make sense to provide a bit more information to position the software within the context of the ML field so that users can develop a more intuitive understanding of its benefits and use cases. Relatedly, the JOSS checklist states:

A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?

This is covered nicely in the JOSS paper itself, and perhaps some of this content can be worked into the docs. I actually like the current single paragraph intro in the docs index page before jumping into the quickstart, so perhaps a more detailed overview can be provided on a separate page and referenced from the index page.

Overall, the docs are great. I appreciate the attention given to this. Also, +1 on Furo :)


Ping openjournals/joss-reviews#6065

kwinkunks added a commit that referenced this issue Dec 9, 2023
Also tweaked some of the other pages to use more consistent styling.
Fixes #94.
@kwinkunks
Copy link
Member

kwinkunks commented Dec 9, 2023

Thanks for the kind words, appreciated.

Totally agree that more scene-setting was needed -- I often look for this myself in other packages. I've tried to address this with a new 'What is redflag?' page in the docs. I have self-plagiarized it from the draft paper -- I hope that's okay? Happy to re-word if it seems like too much of a shortcut.

Proposed fix in a525174 5e534d8 (forgot to add file, ha)

kwinkunks added a commit that referenced this issue Dec 9, 2023
Also tweaked some of the other pages to use more consistent styling.
Fixes #94.
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

2 participants