Skip to content

Update the front page in docs - #28

Merged
rahulsavani merged 7 commits into
gambitproject:mainfrom
nataliemes:docs/update-index
Oct 24, 2026
Merged

rahulsavani merged 7 commits into
gambitproject:mainfrom
nataliemes:docs/update-index

Conversation

@nataliemes

@nataliemes nataliemes commented Oct 15, 2026 •

Copy link
Copy Markdown
Collaborator

Expanded the docs front page following the outline in #26.

You can view the updated front page here.

It now contains:

  • short description,
  • main features,
  • two small usage examples (one as a library, another via CLI)
  • installation instructions,
  • licence/citation,
  • links to other pages (only API docs for now).

@nataliemes nataliemes changed the title Update the readthedocs front page Update the front page in docs Oct 20, 2026
@nataliemes
nataliemes marked this pull request as ready for review October 20, 2026 21:32
@rahulsavani
rahulsavani requested a balanced review from Copilot October 21, 2026 04:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The Python doctest fails when run from the documentation directory because its input path is relative to the repository root.

3 open issues
What changed in this PR

Expands the documentation front page to introduce lemke and help users get started.

Changes:

  • Adds features, Python and CLI examples, installation guidance, and citation information.
  • Adds a small LCP example input.
  • Enables Sphinx doctests and mathematical rendering, and preserves full URLs.
File Description
examples/​lcp_2x2 Adds input for the Python example.
doc/​index.rst Expands introductory and getting-started documentation.
doc/​conf.py Enables doctest and MathJax extensions and adjusts URL display.

Review effort: Balanced

Comment thread doc/index.rst Outdated
Comment thread doc/index.rst
Comment thread doc/index.rst Outdated

@rahulsavani rahulsavani left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Basically this looks good to me. Copilots comments seem reasonable to please respond to them -- fine to just commit them if you agree they make sense.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The documentation examples match the existing interfaces and input formats, with no unresolved blocking issues.

No open issues 🎉

3 resolved since last review

Review effort: Balanced

@rahulsavani rahulsavani left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me.

@rahulsavani
rahulsavani merged commit a003e6f into gambitproject:main Oct 24, 2026
9 checks passed
@nataliemes
nataliemes deleted the docs/update-index branch October 24, 2026 13:14
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

Successfully merging this pull request may close these issues.

3 participants