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] planner.cpp error: 'maxj' was not declared in this scope (bugfix 2.0.x) #16991

Closed
Kokorone opened this issue Feb 26, 2020 · 8 comments
Closed

Comments

@Kokorone
Copy link

Bug Description

Compiling bugfix 2.0.x with todays commits causes this error with a SKR 1.3

image

My Configurations

Required: Please include a ZIP file containing your Configuration.h and Configuration_adv.h files.

Marlin.zip

Steps to Reproduce

  1. Configure bugfix 2.0.x with commits as of 1:23 pm PST on 2/26/2020
  2. Attempt to compile
  3. View errors in terminal

Expected behavior: [What you expect to happen]
Compile as usual with no errors
Actual behavior: [What actually happens]
Fail with several errors related to maxj, axis, and extraxyjerk in planner.cpp

Additional Information

Only noteworthy change is using CLASSIC_JERK

  • Provide pictures or links to videos that clearly demonstrate the issue.
  • See How Can I Contribute for additional guidelines.
@Kokorone
Copy link
Author

Kokorone commented Feb 26, 2020

Recompiled firmware with CLASSIC_JERK disabled and Junction Deviation re-enabled and it compiled with no issues.

Appears to be a bug with how CLASSIC_JERK's hooks into the rest of the firmware

@Kokorone
Copy link
Author

Appears to have been broken with this commit which references the exact lines that are erroring

https://github.com/MarlinFirmware/Marlin/commit/263a7beeb53a619cc319be2e369ef7604f32af4d

@lesstutrey
Copy link

lesstutrey commented Feb 26, 2020

Marlin.zip
having same issue. yup classic jerk enabled. sorry confs attached now. This was for 2.0.4.4 though.

@lesstutrey
Copy link

marlinerror2144

@Kokorone Kokorone mentioned this issue Feb 27, 2020
@mak0t0san
Copy link
Contributor

Getting this as well. The maxj variable was declared as a float variable, but now isn't declared at all. Same with axis. I'm using a delta printer so I can only use classic jerk as junction deviation doesn't work with delta printers.

This was referenced Feb 27, 2020
@ellensp
Copy link
Contributor

ellensp commented Mar 21, 2020

This was fixed three weeks ago. Confirmed that your configuration files currently compile without issue.
Please close this issue.

@boelle
Copy link
Contributor

boelle commented Jun 20, 2020

Lack of Activity
This issue is being closed due to lack of activity. If you have solved the
issue, please let us know how you solved it. If you haven't, please tell us
what else you've tried in the meantime, and possibly this issue will be
reopened.

@boelle boelle closed this as completed Jun 20, 2020
@github-actions
Copy link

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 Aug 21, 2020
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

5 participants