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

Update config.cfg to include z1 and z2 instead of extruder which is used from toolboard #293

Closed
wants to merge 1 commit into from

Conversation

nickidotdk
Copy link
Contributor

When configuring a VCore3 or 4 the configuration is missing configuration for z1 and z2. In the configurator it states that when using this board a toolboard is required. Going through the steps and choosing the motors for z1 and z2 works fine, but at the last step it fails since there were no configurations defined for z1 and z2. This fixes that issue.

…sed from toolboard

When configuring a VCore3 or 4 the configuration is missing configuration for z1 and z2. In the configurator it states that when using this board a toolboard is required. Going through the steps and choosing the motors for z1 and z2 works fine, but at the last step it fails since there were no configurations defined for z1 and z2. 
This fixes that issue.
@miklschmidt
Copy link
Member

This change would make the board unusable on printers without a toolboard. However i think the actual bug here is that you managed to get through the hardware configuration without assigning motor slots to the undefined axes. Sounds like this isn't an issue anymore though?

In 2.0 we used a separate config-3z.cfg to make 5 driver boards compatible with printers that need 6 steppers.

@nickidotdk
Copy link
Contributor Author

I did assign the motor slots, and then the very last page - the confirmation page, fails due to the missing motor section in the config file that it loads data from when injecting into the config file.

@nickidotdk
Copy link
Contributor Author

But I see your point regarding not working for non toolboard printers. The configurator could use individual confog files for the different printers.

@nickidotdk
Copy link
Contributor Author

I will only use this locally for myself since this introduces unwanted behaviour for non vcore 3.x/4 printers.

@nickidotdk nickidotdk closed this Sep 15, 2024
@miklschmidt
Copy link
Member

miklschmidt commented Sep 15, 2024

@nickidotdk I'm gonna reopen this as there's clearly still a bug somewhere that should be addressed, and we should probably add a config-3z.cfg with the contents of this PR (instead of modifying the original). So what needs to be done is:

  1. Add config-3z.cfg with the contents of this PR's config.cfg.
  2. Add "extruderlessConfig": "config-3z.cfg" to board-definition.json
  3. Before merging, reproduce and fix the issue in the configurator (https://github.com/Rat-OS/RatOS-configurator).

To clarify point 3:

  • When motor slots are assigned, the pins from board-definition.json is used to generate the board_pin alias map, in place of the values from config.cfg (or else it would not work for any includes/configs that use the board pin alias).
  • Fixing this issue involves finding out what is generating the error and why. The user should not be able to proceed from the hardware configuration step without resolving any driver slot conflicts or errors.
  • Provided that all "printer rails" have a motor slot assignment, there should be no error about stepper pins further in the process, regardless of what's in config.cfg or if a extruderlessConfig is present.

@miklschmidt miklschmidt reopened this Sep 15, 2024
calcodk added a commit to calcodk/RatOS-configuration that referenced this pull request Sep 16, 2024
calcodk added a commit to calcodk/RatOS-configuration that referenced this pull request Sep 16, 2024
nickidotdk added a commit to nickidotdk/RatOS-configuration that referenced this pull request Sep 16, 2024
nickidotdk added a commit to nickidotdk/RatOS-configuration that referenced this pull request Sep 16, 2024
This was referenced Sep 16, 2024
@nickidotdk
Copy link
Contributor Author

nickidotdk commented Sep 16, 2024

@miklschmidt I have made the changes. I was not sure how to include both changes in the same PR, so they are seperate.
#295
#294
I don't have a linux box to test this on. Could you help with that?

Also I wanted to delete the PRs made by Calcodk - I was logged into the wrong account when doing the changes. I am not used to working directly in Github :)

Best regards.

@nickidotdk
Copy link
Contributor Author

@miklschmidt It is working as intended. I have tested it by manually adding config-3z.cfg and changing board-definition.json in my RatOS installation on the M5P. Now everything is working. I also changed from CAN to Usb and added a Beacon. Waow what a great job you guys did with this!!!! <3

miklschmidt pushed a commit that referenced this pull request Sep 26, 2024
@nickidotdk nickidotdk closed this Sep 27, 2024
@miklschmidt
Copy link
Member

@nickidotdk Thank you for your contribution and thanks for the kind words!! :)

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