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

Restructure documentation and include missing information on installing the storynavigator widget #69

Merged
merged 10 commits into from
Jul 2, 2024

Conversation

ThijsVroegh
Copy link
Collaborator

@ThijsVroegh ThijsVroegh commented Jun 21, 2024

Draft PR work in progress

Issue

Some aspects were missing in the documentation, including how to install the widget as an add-on

Restructure the documentation along:

  • Getting Started
  • Widgets
  • 2-3 example workflows, incl. download availability. These also serve the upcoming workshop in September
  • Rearranged the structure according to Kody's suggestions
  • Known issues
  • Contact information
Description of changes

changed widgets.json and index.rst to account for previously missing pages

Includes
  • Documentation
Previewing documentation page
  • Having installed sphinx, recommonmark en sphinx-rtd-theme, one can see a preview of the updated documentation by following these steps (thanks to Erik):
  • cd doc
  • make html

On OS X

  • open _build/html/index.html

Or

@ThijsVroegh ThijsVroegh self-assigned this Jun 21, 2024
@ThijsVroegh ThijsVroegh added the documentation Improvements or additions to documentation label Jun 25, 2024
@ThijsVroegh ThijsVroegh marked this pull request as ready for review June 25, 2024 09:34
@ThijsVroegh
Copy link
Collaborator Author

@kodymoodley Hi Kody, can you take a look at the updated documentation and structure?

doc/widgets.json Outdated Show resolved Hide resolved
@@ -0,0 +1,6 @@
How to enable StoryNavigator in Orange
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You mean "Usage" right? Not "How to enable..."

doc/index.rst Outdated
@@ -36,7 +36,11 @@ Widgets
widgets/narrativenetwork
widgets/settinganalysis

.. toctree::
:maxdepth: 2
:caption: Contact Information
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought about it a bit more. I would suggest the following structure:

  • Getting Started
    • Installation
    • Usage
    • Tutorials
    • Known Issues
  • Widgets
    • Actors
    • Actions
    • Setting
    • Narrative Network
  • Developers
    • Building from source
    • Contribute
    • Changelog
  • About
    • Contact
    • How to cite Story Navigator
    • In action
    • FAQs

"Usage" will show how to use each widget.
"Tutorials" will provide video tutorials with workflows that connect the widgets in order to answer some predefined research questions.

  • Known issues: runtime issues that users might experience (with workarounds if available) and misconceptions about how SN works or what it is supposed to do
  • "Building from source": contains instructions from Github on how to build SN from source
  • "Contribute": contribution guidelines (styles / conventions we expect from contributors. E.g., "contact developers at developers@storynavigator.com first before submitting PRs. Use markdown for...etc.)
  • "Changelog": list of featured added, removed / amended in each version
  • "How to cite..": citiation.cff file for SN. bibtex entry (copy and paste box)
  • "In action": links to projects, events, papers, blogposts, videos, etc. which mention or make use of SN.
  • FAQs: frequently asked questions about SN with their answers. E.g., where can I find information about how to...?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ThijsVroegh @eriktks what do you think?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @kodymoodley. This looks like a well-defined structure! I updated the structure of the documentation page accordingly, and updated several of the pages already. I'll focus on two example workflows next as part of the 'tutorials' subpage


If you are interested in contributing, reach out at:
- **Email**: [info@esciencecenter.nl](mailto:info@esciencecenter.nl)
- **Github**: [https://github.com/navigating-stories/orange-story-navigator](mailto:https://github.com/navigating-stories/orange-story-navigator)

## Follow Us
**Follow Us**

Stay updated on the projecty by following us on:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: "projecty"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this last line meant to link to something like Twitter? If so, remove it. They can use the contact info about rather.

@ThijsVroegh
Copy link
Collaborator Author

Thansk @kodymoodley for all suggestions! I included them and updated the structure of the page. Just let me know in case any other suggestions pop up

Copy link
Contributor

@kodymoodley kodymoodley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your nice work @ThijsVroegh! The documentation is taking shape very nicely! :) Just some final things to add. If you only address these points and do not add other stuff, I am happy for you to merge directly to master after the changes (without requiring a further review from me).

doc/widgets/changelog.md Outdated Show resolved Hide resolved
doc/widgets/contribute.md Outdated Show resolved Hide resolved
doc/widgets/howtocite.md Outdated Show resolved Hide resolved
doc/widgets/howtocite.md Show resolved Hide resolved
doc/widgets/inaction.md Outdated Show resolved Hide resolved
doc/widgets/installation.md Outdated Show resolved Hide resolved
doc/widgets/contribute.md Outdated Show resolved Hide resolved
@ThijsVroegh ThijsVroegh merged commit e5d3abf into master Jul 2, 2024
3 checks passed
@ThijsVroegh ThijsVroegh deleted the documentation branch July 2, 2024 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants