Skip to content

chore: Move study configs to logbook artifact directories - #91

Merged
chrisjz merged 1 commit into
mainfrom
chore/move-study-configs-to-artifacts
Mar 31, 2026
Merged

chore: Move study configs to logbook artifact directories#91
chrisjz merged 1 commit into
mainfrom
chore/move-study-configs-to-artifacts

Conversation

@chrisjz

@chrisjz chrisjz commented Mar 31, 2026

Copy link
Copy Markdown
Member

Summary

  • Move configs/studies/health_scaling/artifacts/logbooks/005/health_scaling/ (logbook 005)
  • Move configs/studies/plasticity/artifacts/logbooks/008/plasticity/ (logbook 008)
  • Update all active references in scripts, docs, specs, and gitignore
  • Remove the now-empty configs/studies/ directory

Study configs belonged to specific experiment logbooks but were stored separately under configs/studies/. This aligns them with the established pattern where logbook experiment configs live in artifacts/logbooks/<N>/.

Test plan

  • Pre-commit checks pass
  • All active references to configs/studies/ updated (grep verified)
  • Archived openspec change docs left as-is (historical records)
  • Verify run_plasticity_test.py and run_health_scaling_study.sh work with new paths

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Configuration file paths have been reorganized and updated across documentation and scripts
    • Documentation updated to reflect new artifact locations for experimental configurations

Study configs belonged to specific experiment logbooks but were stored
under configs/studies/. Move them to their respective artifact directories
to match the established pattern for logbook experiment configs:
- health_scaling/ → artifacts/logbooks/005/
- plasticity/ → artifacts/logbooks/008/

Update all active references in scripts, docs, specs, and gitignore.
Remove the now-empty configs/studies/ directory.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Mar 31, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 3a6c487c-1012-413d-9e0e-9181365bc303

📥 Commits

Reviewing files that changed from the base of the PR and between a09879b and fcf84f8.

📒 Files selected for processing (24)
  • .gitignore
  • artifacts/logbooks/005/health_scaling/control_p1.yml
  • artifacts/logbooks/005/health_scaling/control_p10.yml
  • artifacts/logbooks/005/health_scaling/control_p2.yml
  • artifacts/logbooks/005/health_scaling/control_p3.yml
  • artifacts/logbooks/005/health_scaling/control_p5.yml
  • artifacts/logbooks/005/health_scaling/control_p7.yml
  • artifacts/logbooks/005/health_scaling/health_p1.yml
  • artifacts/logbooks/005/health_scaling/health_p10.yml
  • artifacts/logbooks/005/health_scaling/health_p2.yml
  • artifacts/logbooks/005/health_scaling/health_p3.yml
  • artifacts/logbooks/005/health_scaling/health_p5.yml
  • artifacts/logbooks/005/health_scaling/health_p7.yml
  • artifacts/logbooks/008/plasticity/crh_plasticity.yml
  • artifacts/logbooks/008/plasticity/hybridclassical_plasticity.yml
  • artifacts/logbooks/008/plasticity/hybridquantum_plasticity.yml
  • artifacts/logbooks/008/plasticity/mlpppo_plasticity.yml
  • artifacts/logbooks/008/plasticity/qrh_plasticity.yml
  • configs/README.md
  • docs/experiments/logbooks/005-health-system-predator-scaling.md
  • docs/experiments/logbooks/008-quantum-brain-evaluation.md
  • openspec/specs/plasticity-evaluation/spec.md
  • scripts/run_health_scaling_study.sh
  • scripts/run_plasticity_test.py
💤 Files with no reviewable changes (2)
  • .gitignore
  • configs/README.md

📝 Walkthrough

Walkthrough

This PR relocates experiment configuration files from the configs/studies/ directory structure to artifacts/logbooks/ and updates all corresponding references across documentation, scripts, and configuration management files.

Changes

Cohort / File(s) Summary
Configuration Management
.gitignore, configs/README.md
Removed ignore negation pattern and documentation entry for studies/ directory, consolidating study configs under the broader ignore rule and removing outdated directory references.
Experiment Documentation
docs/experiments/logbooks/005-health-system-predator-scaling.md, docs/experiments/logbooks/008-quantum-brain-evaluation.md
Updated config file path references from configs/studies/{health_scaling,plasticity}/... to artifacts/logbooks/{005,008}/{health_scaling,plasticity}/... in both Health-enabled and Control conditions.
Specification & Scripts
openspec/specs/plasticity-evaluation/spec.md, scripts/run_health_scaling_study.sh, scripts/run_plasticity_test.py
Updated CLI invocation paths and configuration directory references to point from configs/studies/ to artifacts/logbooks/{005,008}/ locations in spec examples and script variables.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • feat: Add QSNN type brains #54: Adds and updates logbook artifacts and YAML configuration files that these path updates directly reference.
  • feat: Add health system #38: Introduces health system configurations in the study directory structure that these updates migrate to the new artifacts location.

Poem

🐰 Hoppity-hop, the configs are on the move,
From studies to logbooks, we found our groove,
All paths updated with a careful paw,
References renewed without a flaw,
Artifacts directory—the warren's new home! 🎉

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title accurately describes the main change: moving study configuration files from configs/studies/ to logbook artifact directories, which is the core objective of the entire changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/move-study-configs-to-artifacts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov

codecov Bot commented Mar 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@chrisjz
chrisjz merged commit 3b5a2cc into main Mar 31, 2026
4 checks passed
@chrisjz
chrisjz deleted the chore/move-study-configs-to-artifacts branch March 31, 2026 10:14
@chrisjz chrisjz changed the title Move study configs to logbook artifact directories feat: Move study configs to logbook artifact directories Mar 31, 2026
@chrisjz chrisjz changed the title feat: Move study configs to logbook artifact directories chore: Move study configs to logbook artifact directories Mar 31, 2026
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.

1 participant