Skip to content

Add linear encoder test report - #99

Closed
haikew wants to merge 2 commits into
openUC2:masterfrom
haikew:encoder-test-report
Closed

haikew wants to merge 2 commits into
openUC2:masterfrom
haikew:encoder-test-report

Conversation

@haikew

@haikew haikew commented Oct 19, 2026

Copy link
Copy Markdown

The report contains:

  • 24 h long-run test with step-loss recovery
  • Thermal drift over 12 h
  • Backlash at different positions and speeds

Haiyue Li and others added 2 commits October 19, 2026 16:47
@ethanjli

ethanjli commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

CI workflows are currently failing on your PR because the way they're implemented currently doesn't account for people making PRs from forked repos. I'm pretty sure I had previously implemented that logic elsewhere, in other repos, but I hadn't bothered to copy it over to this repo because I didn't think people would be making PRs from forked repos. I'll see if I can find the fix from wherever else I'd already implemented it, and copy it into your PR.

@haikew

haikew commented Oct 20, 2026

Copy link
Copy Markdown
Author

@ethanjli, thanks for explaining! I'll open a new PR from this repo; I forked it a long time ago before I had write access here.

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