Repository navigation
profile: add OpenAMRobot 2.0 with OpenArm 2.0 block; update lift wording - #81
Conversation
Add a "New: OpenAMRobot 2.0 with OpenArm 2.0" block under the intro with three links (model, roadmap, packages). No image: docs/media/hero.png is absent on openamr-upperbody-sw main (a8c304d). Replace "approved in principle" lift wording with: the combined model including the lift is implemented and verified in simulation; physical hardware build and acceptance are in progress. Link enactic/openarm at the first OpenArm mention. STATE.md records the work. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DZnPEQLNs2EgfGDUYbbXKs Signed-off-by: Alex Reznichenko <info@botshare.ai>
Replace the link line with four links: Explore the 3D model (roadmap robot.html), Combined model (URDF, MoveIt, Gazebo), See the roadmap, and Pre-order and packages. Lift wording and decisions.yaml unchanged. STATE.md updated. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DZnPEQLNs2EgfGDUYbbXKs Signed-off-by: Alex Reznichenko <info@botshare.ai>
panthera-momagdii
left a comment
There was a problem hiding this comment.
Focused COMMENT review on profile PR #81, exact head 952f3ce36e3c4fe6aa770e9233d5afc82d8a4721, base a9e229e (repository openAMRobot/.github, branch claude/bold-franklin-qu0f6i). Expected outcome: check the new public links, model wording and required adoption gates; not approval or publication authorization.
The core wording is appropriately scoped in one respect: merged upperbody #29 provides simulated/mock lift control, the lift and both_arms MoveIt groups, while the physical lift interface and release gates remain separate. The new profile text does not claim physical validation. The in-force LIFT decision still reads "approved in principle"; the proposed follow-up contract/register decision must be handled by its owner, not assumed completed in this Markdown PR.
Two publication/adoption checks remain:
- The new Explore the 3D model link resolves publicly to
https://docs.openamrobot.ai/assets/roadmap/robot.html, but the currently served page still says "Snapshot 24 Oct 2026" and describes the combined model as draft PR #14, with the full simulation work still to be fixed/merged. That public destination is older than the 2 Nov "implemented and verified in simulation" profile claim. Please coordinate a current published page with the docs owner or explicitly label the viewer as an older design/illustration; do not let the CTA imply the 24 Oct text documents current acceptance. The roadmap/pre-order destinations and the#productsanchor were not independently verified. AGENTS.mdsays agent-authored PRs remain draft until the work-package owner writes adopt, adapt or reject in the thread. This AI-authored PR is currently ready/non-draft, and its PR thread contains no owner adoption. Please have the owner record their disposition and preserve that gate; the direct-task statement in the PR body does not replace a recorded adoption.
Evidence: inspected the complete two-file patch, the current profile/STATE, canonical shared rules, LIFT register, merged upperbody #29 description and earlier PR discussion (none). Hosted exact-head repository-quality run 41515862953 inspected, with quality/test reporting 289 executed, 0 skipped, PASS; license/cla status is success. CI passing the static public extract does not validate link destinations or owner adoption. NOT RUN by reviewer: local ROS, Gazebo, Isaac Sim, physical lift, or the author's full local verification command. This is a COMMENT, not approval, merge, register change or a legal attestation.
Signed-off-by: Alex Reznichenko <info@botshare.ai>
|
Owner disposition: adopt, as the project lead and work-package owner for the org profile text. @panthera-momagdii on your two points:
The LIFT register wording stays a separate contract change request, as the PR body says. The STATE.md conflict with main is resolved (both Current work rows kept). |
panthera-momagdii
left a comment
There was a problem hiding this comment.
Re-reviewed #81 at exact head 3ec22b3b7500000a57bf398400a1a943389c5430. Approved for the profile/STATE wording scope.
The two points from my earlier comment are now dispositioned: @BotshareAI recorded adopt as project lead/work-package owner, and the PR explicitly keeps publication order behind docs #68 so the 3D-model link is updated before this profile change is published. The current PR patch against main remains limited to profile/README.md and STATE.md; the text clearly separates simulation verification from physical hardware build/acceptance and leaves the LIFT register wording to its separate contract-change process.
Hosted exact-head Repository quality is green: checker tests, decisions validation, governance/public checks and ROS verifier smoke all succeed; CLA status is success. This approval does not approve the LIFT register change, physical lift, or the current contents of docs #68. Keep #81 ordered after #68 is corrected, merged and deployed as stated in the owner comment.
|
Final check before merge (owner): the four links load. Explore the 3D model and See the roadmap now show the 2 Nov 2026 snapshot (openamrobot-docs #68, #70 and #71 merged and deployed), so the earlier concern about the 24 Oct page is resolved. The combined model repo loads, and the pre-order link reaches the #products section. The adoption decision was recorded above; the LIFT register wording remains a separate contract change request. Merging. |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DZnPERXdK4VNyx7trxz3QR Signed-off-by: Alex Reznichenko <info@botshare.ai>
… power module, receiver gap, flag rod guide) (#85) * decisions: owner decisions of 3 Nov 2026 (K1/K2 GX11CAC decided, rear power module, receiver gap, flag rod guide) SAFETY-PROCUREMENT: K1/K2 option A GX11CAC decided; option B (LC1D80BD, 3RT2045) rejected on fit. BATTERY-PLACEMENT: rear power module per the rear power owner's layout report rev 2; BMS in the right side gap; FB-B SafetyHub 150, FB-A Blue Sea 5046; busbars. CHARGING: 15 mm gap above the receiver, non-metal within 20 mm. LIFT: flag rod second support, open U-ring guide on the static column head. New source: Alex decision 2026-11-03. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DZnPERXdK4VNyx7trxz3QR Signed-off-by: Alex Reznichenko <info@botshare.ai> * STATE: #81 and #84 merged; decisions of 3 Nov 2026 in review Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DZnPERXdK4VNyx7trxz3QR Signed-off-by: Alex Reznichenko <info@botshare.ai> * tests: register test for the decisions of 3 Nov 2026 The 2 Nov test now expects the 3 Nov source for CHARGING, SAFETY-PROCUREMENT, BATTERY-PLACEMENT and LIFT; new test for the 3 Nov values. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DZnPERXdK4VNyx7trxz3QR Signed-off-by: Alex Reznichenko <info@botshare.ai> --------- Signed-off-by: Alex Reznichenko <info@botshare.ai>
Summary
Paths:
profile/README.md,STATE.md.docs/media/hero.pngdoes not exist onopenamr-upperbody-swmain (a8c304d8a6335c479e56f20e32675d2d87144a50, nodocs/media/directory).decisions.yamlis unchanged. The LIFT register wording ("lift approved in principle") goes into a separate contract change request.Work package
None filed: this is a direct task from the project lead on the org profile text.
Integration Gate
No overlapping open PR touches
profile/README.md; #78 touchesdecisions.yamlonly. The block reuses existing link targets (openamr-upperbody-sw, docs.openamrobot.ai, botshare.ai). Rejected: adding an image that does not exist on main.Tests
No behaviour change: this PR edits Markdown text only, so no new test. Existing suites were run unchanged on the current head:
rollout/verify.shexecuted 289 of 289 tests, 0 skipped.Evidence
Base SHA: a9e229e
Head SHA: 952f3ce
Dependencies
None.
Safety impact
None. This PR changes text only. It touches no safety path.
STATE.md
Updated: the Current work row records #81, the four links and the separate contract change request for the LIFT register wording.
Not verified
https://docs.openamrobot.ai/assets/roadmap/robot.html,https://docs.openamrobot.ai/assets/roadmap/andhttps://botshare.ai/openamrobot?utm_source=github&utm_medium=org#products: neither host is in the agent session's allowed network domains, so none of these URLs loaded and the#productsanchor is unchecked.https://github.com/openAMRobot/openamr-upperbody-sw: a public git clone works, but no browser render was checked.AI disclosure
Claude Code (Anthropic) wrote the README and STATE.md edits, ran the checks and wrote this description. Pending review by the work-package owner (adopt, adapt or reject).
Contribution terms
🤖 Generated with Claude Code
https://claude.ai/code/session_01DZnPEQLNs2EgfGDUYbbXKs