demo: getter/setter-in-init temperature attr example

[coretl]

Closes #404

Adds Tutorial 2 (getter/setter) from the demo README ladder: src/fastcs/demo/temperature_attr.py, a small temperature controller (ramp_rate, power) wired attribute-by-attribute in __init__ rather than via shared class-body declarations.

Retargeted onto the temperature sim (per the issue, 2026-07-22 update) rather than a standalone Thorlabs MFF device, so the tutorial ladder keeps to two backends.

Current-API baseline in the intended shape: a single generic TemperatureIO drives every attribute, and each TemperatureIORef carries the command-building callables (read_cmd/write_cmd) sourced from a single TemperatureProtocol class — mirroring fastcs-thorlabs-mff (MFFAttributeIO/MFFAttributeIORef/ThorlabsAPTProtocol). read_cmd/write_cmd are the getter/setter this foreshadows: #392 promotes them onto AttrRW(getter=..., setter=...) and deletes the IO/ref wrapper, while TemperatureProtocol survives unchanged. Response parsing (float()) is inline in TemperatureIO.update, not a response_handler callable. (Contrast the composition example controllers.py, #390, whose shared IO dispatches on a name string.)

Instructions to reviewer on how to test:

1. uv run pytest tests/demo/test_temperature_attr.py -v

Checks for reviewer

• Would the PR title make sense to a user on a set of release notes

Notes

• Verified locally with uv run --locked tox -e pre-commit,type-checking, both green. As on demo: use ControllerVector for temperature ramp sub-controllers #409/demo: cut-down Eiger REST sim + introspectable controller example #410, this sandbox can't run the docs env (no outbound network) or PVA-touching tests (no PVA-capable socket family here). Excluding those, pytest src tests --ignore=tests/benchmarking passes 310/320 with only the same known pre-existing PVA/p4p failures, unrelated to this change. Real CI covers docs and PVA.

---

Generated by Claude Code

Summary by CodeRabbit

• New Features

  • Added a temperature controller demo with readable ramp-rate and power attributes.
  • Added support for updating values from a connected device.
  • Added the ability to set the ramp rate.

• Tests

  • Added coverage for reading ramp rate and power values.
  • Added coverage confirming ramp-rate updates are sent correctly.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 52cf8dba-6ef7-42c7-adad-743a463f94e8

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The PR adds the temperature-simulator example and tests, and it uses the current callback-IO baseline called for in #404.
Out of Scope Changes check	✅ Passed	The changes stay focused on the temperature demo example and its tests, with no unrelated scope evident.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the new temperature attribute demo using getter/setter wiring in init.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch refactor-issue-404

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.25%. Comparing base (3da025f) to head (1051a64).
⚠️ Report is 2 commits behind head on refactor.

Additional details and impacted files

@@            Coverage Diff            @@
##           refactor     #411   +/-   ##
=========================================
  Coverage     91.25%   91.25%           
=========================================
  Files            72       72           
  Lines          2892     2892           
=========================================
  Hits           2639     2639           
  Misses          253      253           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/fastcs/demo/temperature_attr.py`:
- Around line 76-77: Update the async close method to reset the controller’s
_connected state to False after awaiting self.connection.close(), so subsequent
connection status checks correctly report the closed state.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: a64717b0-dfc5-439f-a5e0-0a883dacae7c

📥 Commits

Reviewing files that changed from the base of the PR and between 3da025f and 4dbaf83.

📒 Files selected for processing (2)

• src/fastcs/demo/temperature_attr.py
• tests/demo/test_temperature_attr.py

[shihab-dls]

Just one comment regarding # type: ignore. This is idiomatic fastcs logic, with good tests, and It does show obscurity with passing setters and getters into an AttributeIORef, which I think is a good starting point. Approving.