Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Incorrect needle selection on 270, after starting AYAB with the carriage to the right of the left hall sensor. #667

Open
Adrienne200 opened this issue Jun 26, 2024 · 8 comments
Assignees
Milestone

Comments

@Adrienne200
Copy link

270 only. If you click Cancel, then Knit, while to the right of the left Hall sensor, AYAB appears to start up correctly, but it the "shadows", coping the pattern again when it shouldn't.

AYAB software version: 0.99.0-rc6
Computer/OS: Intel Mac, Big Sur
Knitting machine: KH270
AYAB hardware: Interface

Steps

  1. Make sure that you have AYAB's prefs set to 270, and have flashed the firmware
  2. Use any non-repeating image, sample attached.
  3. First test the image by air-knitting the conventional steps, placing the carriage outside the left turn mark, click Knit, cross the turn mark, wait a moment, continue. This should just work.
  4. Now, stop the carriage anywhere to the right of the left turn mark.
  5. Click Cancel. Then click Knit
  6. Move the carriage from right to left so that both magnets are to the left of the left turn mark
  7. Take the carriage across the turn mark from left to right, wait a moment, continue.

Expected: It should start again at row 1, and just work. (Having ignored all right-left magnet movements)
Actual: It repeats the pattern twice.

abcd vertical 32x64

it's supposed to ignore the right-left movement and

@clholgat
Copy link
Contributor

Oh, I know what this is. There's an early out for the 270 carriage detection because of the two magnets, it's probably causing problems.

@dl1com
Copy link
Contributor

dl1com commented Jul 15, 2024

Oh, I know what this is. There's an early out for the 270 carriage detection because of the two magnets, it's probably causing problems.

Do you think this is a trivial fix and can make it into 1.0 or should be postponed?

@X-sam
Copy link
Member

X-sam commented Jul 29, 2024

Is this related to AllYarnsAreBeautiful/ayab-firmware#179

@jonathanperret
Copy link
Contributor

Is this related to AllYarnsAreBeautiful/ayab-firmware#179

Right, this should probably be tested again.

@dl1com
Copy link
Contributor

dl1com commented Oct 27, 2024

According to @Adrienne200 s report in Discord, still not fixed
(also incorporating with AllYarnsAreBeautiful/ayab-firmware#207 )

@clholgat
Copy link
Contributor

I can repro locally. Digging

@clholgat
Copy link
Contributor

Well, that was easy AllYarnsAreBeautiful/ayab-firmware#208

@jonathanperret
Copy link
Contributor

Fix is included in https://github.com/AllYarnsAreBeautiful/ayab-desktop/releases/tag/1.0.0-beta3 but a round of testing would be nice before closing this.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
Status: Needs Testing
Development

No branches or pull requests

5 participants