Skip to content

Graft setup - #28

Merged
theomgdev merged 1 commit into
mainfrom
feature/rectified-flow
Sep 8, 2026
Merged

Graft setup#28
theomgdev merged 1 commit into
mainfrom
feature/rectified-flow

Conversation

@theomgdev

Copy link
Copy Markdown
Owner

Summary

Brief description of what this PR does and why.

Changes

Checklist

All contributions

  • Code follows the project conventions (see CONTRIBUTING.md)
  • Tests pass (python -m pytest tests/or pytest tests/)
  • No sys.path.append hacks — imports use from odyssnet import ... directly

Library changes (odyssnet/)

  • Corresponding test added/updated under tests/
  • Documentation updated in relevant markdown files (LIBRARY.md, CONTRIBUTING.md)

New/modified example scripts (examples/)

  • set_seed(42) is called as the first line of main()
  • Placed in correct folder (examples/ for core validations, examples/advanced/ for complex tasks)
  • Uses OdyssNetTrainer (not a manual training loop)
  • Uses TrainingHistory to record metrics and calls history.plot() at the end
  • File paths are relative to __file__, not hardcoded

Copilot AI lite review requested due to automatic review settings September 8, 2026 20:14
@theomgdev
theomgdev merged commit 25ee5b4 into main Sep 8, 2026

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 new Graft documentation and ignore rules are internally inconsistent (and reference paths not present in-tree), so the intended workflow needs to be clarified and aligned before merging.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Sets up initial repository guidance and ignore rules intended to support a “Graft” context-graph workflow for navigating the codebase.

Changes:

  • Adds a new “Graft — repo context graph” section to AGENTS.md.
  • Introduces a .ignore file intended to influence ripgrep’s search behavior around graft/.
  • Updates .gitignore to ignore various AI/tooling artifacts and the graft/ directory.
File summaries
File Description
AGENTS.md Documents a new Graft-based workflow for repo context lookup.
.ignore Adds ripgrep/tooling ignore overrides for graft/ and its caches.
.gitignore Ignores additional local tooling artifacts and graft/ output.
Review details
  • Files reviewed: 2/3 changed files
  • Comments generated: 2
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .ignore
Comment on lines +1 to +5
# graft's cards are gitignored but should stay greppable: ripgrep reads
# .ignore before .gitignore, so this re-admits the tree to search only.
!graft/
graft/.cache/
graft/.graph/
Comment thread AGENTS.md
Comment on lines +277 to +279
This repo is indexed in `graft/`: small linked markdown nodes that explain each
system and carry exact file:line spans, kept in sync with the code through git.

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