Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[BUG] Improper assignment of steppers on BTT Octopus #22912

Closed
kj4sst opened this issue Oct 9, 2021 · 4 comments
Closed

[BUG] Improper assignment of steppers on BTT Octopus #22912

kj4sst opened this issue Oct 9, 2021 · 4 comments

Comments

@kj4sst
Copy link

kj4sst commented Oct 9, 2021

Did you test the latest bugfix-2.0.x code?

Yes, and the problem still exists.

Bug Description

When building for the Octopus with 3 or more z axis steppers the assignment skips one of the stepper positions, making it so that only 7 out of the 8 drivers can be used. I have tried the latest bugfix and while the issue has been solved, it was solved by heavily modifying "pins_postprocess.h" to check for exceptions. Another, hopefully simpler, way to fix this issue is to bring the stepper assignment in "pins_BTT_OCTOPUS_V1_common.h" into alignment with the other board declarations. The issue is caused due to the fact that in "pins_BTT_OCTOPUS_V1_common.h" both "Z" and "Z2" are defined, if all instances of "Z2" are removed and "E4" is added, for example Z2 -> E0, E0 -> E1, etc... then Marlin will compile without modifying "pins_postprocess.h".

Bug Timeline

Older issue, I found this bug 3-4 weeks ago.

Expected behavior

I expected the stepper drivers to be assigned properly.

Actual behavior

When assigning steppers, specifically with 3 or more z steppers, one of the stepper positions would be skipped leaving the board with 7 functional steppers instead of 8.

Steps to Reproduce

  1. Setup Marlin for BTT Octopus V1
  2. Enable 3 or more z steppers and then fill the rest of the positions with extruder motors (making for a total of 8 steppers enabled in Marlin)
  3. Compile

Version of Marlin Firmware

2.0.x

Printer model

Custom

Electronics

BTT Octopus

Add-ons

No response

Bed Leveling

UBL Bilinear mesh

Your Slicer

No response

Host Software

No response

Additional information & file uploads

ConfigurationFiles.zip

@ellensp
Copy link
Contributor

ellensp commented Oct 9, 2021

Are you sure you tried bugfix? As this very things was fixed in #22771

@kj4sst
Copy link
Author

kj4sst commented Oct 9, 2021

I have tried the bugfix, which did solve the issue. However, the fix which was implemented causes the compiler to throw warnings and introduces complications to "pins_postprocess.h". I was simply suggesting a different, hopefully simpler way of fixing the issue.

@ellensp
Copy link
Contributor

ellensp commented Oct 9, 2021

Please read the comments in the above PR as to why this was done this way and not your way

@ellensp ellensp closed this as completed Oct 9, 2021
@github-actions
Copy link

github-actions bot commented Dec 9, 2021

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Dec 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants