Skip to content

docs: #978 - [M3-P4] Update Documentation and Create M4 Follow-up Issue#982

Merged
Gorkowski merged 3 commits intouncscode:mainfrom
Gorkowski:issue-978-adw-2b58aa9a
Jan 30, 2026
Merged

docs: #978 - [M3-P4] Update Documentation and Create M4 Follow-up Issue#982
Gorkowski merged 3 commits intouncscode:mainfrom
Gorkowski:issue-978-adw-2b58aa9a

Conversation

@Gorkowski
Copy link
Collaborator

Target Branch: main

Fixes #978 | Workflow: 2b58aa9a

Summary

Completes Phase 4 (final phase) of the M3 Jupytext Pilot Migration by updating documentation to reflect the Jupytext paired sync workflow. This adds notebook editing guidance to AGENTS.md, marks the M3 maintenance plan as "Shipped", and creates the M4 follow-up issue (#979) for migrating the remaining ~35 notebooks.

What Changed

Modified Components

How It Works

This is a documentation-only change with no code modifications. The workflow documented in AGENTS.md follows this pattern:

┌─────────────┐     ┌─────────────┐     ┌─────────────┐     ┌─────────────┐     ┌─────────────┐
│  Edit .py   │────▶│  Lint .py   │────▶│    Sync     │────▶│   Execute   │────▶│   Commit    │
│   file      │     │   (ruff)    │     │  (jupytext) │     │  (validate) │     │ both files  │
└─────────────┘     └─────────────┘     └─────────────┘     └─────────────┘     └─────────────┘
        │                  │                   │                   │                   │
        ▼                  ▼                   ▼                   ▼                   ▼
   percent format     catch syntax      update .ipynb       generate outputs     keep paired
   with # %% cells      errors           from .py            for website          .py + .ipynb

Key workflow principles:

  • Always edit .py files, not .ipynb directly
  • Lint before sync to catch syntax errors early
  • Execute after sync to validate code and generate website outputs
  • Commit both files to keep them paired

Implementation Notes

  • M4 Follow-up Issue: Created issue fix: #975 - [M3-P1] Configure Jupytext Paired Sync for Pilot Notebooks #979 for migrating remaining ~35 notebooks outside the pilot scope (Activity/ and Gas_Phase/)
  • AGENTS.md placement: New section placed between "Quick Checks" and "Getting Help" sections for logical flow
  • Documentation reference: Full Jupytext documentation exists in adw-docs/documentation_guide.md (lines 341-534)

Testing

  • Manual verification: AGENTS.md markdown renders correctly
  • Link validation: All references point to valid files and sections
  • No code changes: This is documentation-only, no test suite impact

References

M3 Pilot Migration:

Follow-up Work:

Add Jupytext workflow guidance to AGENTS.md and finalize M3 maintenance plan.

AGENTS.md updates:
- Add "Notebook Editing (Jupytext Paired Sync)" section with 5-step workflow
- Update Last Updated date to 2026-01-30

M3 maintenance plan updates:
- Change status from "Planning" to "Shipped"
- Fill in issue numbers for all 4 phases (uncscode#975, uncscode#976, uncscode#977, uncscode#978)
- Mark all phase tasks as complete
- Add completion entry to Change Log
- Reference M4 follow-up issue #175 in Future Work section

Closes uncscode#978

ADW-ID: 2b58aa9a
Copilot AI review requested due to automatic review settings January 30, 2026 21:45
@Gorkowski Gorkowski added agent Created or managed by ADW automation blocked Blocked - review required before ADW can process labels Jan 30, 2026
Copy link
Contributor

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 completes Phase 4 (final phase) of the M3 Jupytext Pilot Migration by updating documentation to reflect the Jupytext paired sync workflow. It adds notebook editing guidance to AGENTS.md, marks the M3 maintenance plan as "Shipped", and documents the creation of M4 follow-up issue #979 for migrating the remaining notebooks.

Changes:

  • Added "Notebook Editing (Jupytext Paired Sync)" section to AGENTS.md with a 5-step workflow guide for LLM agents
  • Updated M3 maintenance plan status from "Planning" to "Shipped", filled in all phase issue numbers, and marked all tasks complete
  • Added completion entry to the M3 plan's Change Log

Reviewed changes

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

File Description
AGENTS.md Added new notebook editing workflow section after "Quick Checks" and updated the "Last Updated" date to 2026-01-30
adw-docs/dev-plans/maintenance/M3-jupytext-notebook-sync.md Changed status to "Shipped", filled in issue numbers for all phases, marked all tasks complete, updated Change Log, and referenced M4 follow-up

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

Gorkowski and others added 2 commits January 30, 2026 14:50
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Gorkowski Gorkowski merged commit 9598ed4 into uncscode:main Jan 30, 2026
7 checks passed
@Gorkowski Gorkowski deleted the issue-978-adw-2b58aa9a branch January 30, 2026 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent Created or managed by ADW automation blocked Blocked - review required before ADW can process

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[M3-P4] Update Documentation and Create M4 Follow-up Issue

1 participant