Skip to content

docs(notebook): finalize leksara notebooks#76

Closed
RedEye1605 wants to merge 2 commits into
mainfrom
docs/notebooks
Closed

docs(notebook): finalize leksara notebooks#76
RedEye1605 wants to merge 2 commits into
mainfrom
docs/notebooks

Conversation

@RedEye1605
Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings October 25, 2025 04:41
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR finalizes the Leksara notebook documentation by consolidating two separate notebooks into a single comprehensive quickstart playbook. The updated notebook now provides an end-to-end guide covering installation, CartBoard dashboards, ReviewChain pipelines, benchmarking, logging, cleaning primitives, PII masking, and advanced normalization features—all in one cohesive document with English documentation.

Key changes:

  • Merged feature test content into the main quickstart notebook with improved structure and English descriptions
  • Deleted the redundant leksara_full_feature_tests.ipynb file
  • Enhanced documentation with detailed section headers and usage guidance

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
notebooks/leksara_quickstart.ipynb Expanded from basic quickstart to comprehensive playbook covering all Leksara features with structured sections, installation guide, and closing insights
notebooks/leksara_full_feature_tests.ipynb Removed redundant notebook as content was consolidated into the main quickstart

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

"pip install leksara\n",
"```\n",
"\n",
"Tip: activate a virtual environment (for example `python -m venv .venv` then `.\\.venv\\Scripts\\activate`) to keep project dependencies isolated."
Copy link

Copilot AI Oct 25, 2025

Choose a reason for hiding this comment

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

The virtual environment activation command uses Windows-style path separators (.\\.venv\\Scripts\\activate). Consider providing cross-platform examples or using forward slashes, as the notebook should be platform-agnostic. For example: 'then source .venv/bin/activate on macOS/Linux or .venv\\Scripts\\activate on Windows'.

Suggested change
"Tip: activate a virtual environment (for example `python -m venv .venv` then `.\\.venv\\Scripts\\activate`) to keep project dependencies isolated."
"Tip: activate a virtual environment (for example `python -m venv .venv`, then `source .venv/bin/activate` on macOS/Linux or `.venv\\Scripts\\activate` on Windows) to keep project dependencies isolated."

Copilot uses AI. Check for mistakes.
@RedEye1605 RedEye1605 closed this Oct 25, 2025
@RedEye1605 RedEye1605 deleted the docs/notebooks branch October 25, 2025 04:49
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.

2 participants