Skip to content

[1.1.x] AnyCubic Linear Plus, Update Trigorilla 1.4#11356

Merged
thinkyhead merged 2 commits intoMarlinFirmware:bugfix-1.1.xfrom
Hecatron-Forks:bugfix-1.1.x
Sep 16, 2018
Merged

[1.1.x] AnyCubic Linear Plus, Update Trigorilla 1.4#11356
thinkyhead merged 2 commits intoMarlinFirmware:bugfix-1.1.xfrom
Hecatron-Forks:bugfix-1.1.x

Conversation

@Hecatron
Copy link

Description

  • Added example configurations for the anycubic kossel delta linear plus (with probes)
  • Added a change to pins_TRIGORILLA_13.h so that the fan outputs coincide with the recommended wiring by Anycubic
  • Added example images for both probe types so that the end user can identify which one they have

Note if the changes to pins_TRIGORILLA_13.h are not acceptable then I may need to create a new board header file. I've basically set the fan outputs to how they are normally wired for this type of delta printer
Benefits

Benefits

  • Makes it easier to get started with the latest marlin and the anycubic kossel delta linear plus
  • Should allow other delta's that use the trigorilla board to avoid any hacks to the pins_TRIGORILLA_13.h

@thinkyhead
Copy link
Member

The pins for the Trigorilla boards have been through several iterations and the last time with a lot of feedback from users. So I want to consult those users before merging these changes. Also, can you check to see if the Trigorilla 1.4 pins are closer than the 1.3 ones with your system?

@Hecatron
Copy link
Author

Hecatron commented Jul 26, 2018

It looks like the only current difference between trigorilla 1.3 vs 1.4 according to Marlin
is if IS_RAMPS_13 is set or not (via pins_RAMPS_13.h)
which then influences if SERVO0_PIN is set to 7 or 11 within pins_RAMPS.h
I've not tested that but it's not a pin I'm using

I did also come across this thread now that you mentioned it
#8349

I don't mind using pins_TRIGORILLA_14.h instead if you'd like

for info I've attached an image which matches up against my board
Also the order in which the fans are wired is the one recommended in the Anycubic manual you get with the Kossel Linear Plus model

anycubic_i3_mega_trigorilla_board

@thinkyhead
Copy link
Member

As I understand it the Trigorilla 1.4 has support for up to 2 extruders, while the 1.3 only supports one. Assuming this is correct, if your board has two E stepper plugs, then it's a 1.4 board.

@thinkyhead thinkyhead force-pushed the bugfix-1.1.x branch 2 times, most recently from f559555 to 263beae Compare August 13, 2018 21:55
@Hecatron
Copy link
Author

mine has an E0 and an E1 in that case it must be a 1.4 board
I'll make some changes to the commits so that this is for the 1.4 instead of 1.3 board

@Hecatron
Copy link
Author

Okay the changes should now be switched across to the trigorrila 1.4 board definition

  • I've tried compiling it and that works fine, although I can't test it at the moment since i've recently moved across to a duet board. That being said it worked fine with the previous version so this should also be fine
  • I've disabled the pins for the E1 stepper even though there is a E1 stepper output, I suspect the default pinouts within pins_RAMPS.h are incorrect for E1 for this board. This is due to at least one of them being pin 44 which is the same one as the extruder fan output

@thinkyhead thinkyhead force-pushed the bugfix-1.1.x branch 2 times, most recently from 72f7cd2 to 3ef6cf0 Compare August 17, 2018 02:17
@thinkyhead
Copy link
Member

Will it confuse your workflow if I modify your bugfix-1.1.x branch (the source of this PR) so that it contains bugfix-2.0.x plus these changes instead? (Ideally you should make a copy of a branch before modifying it and submitting it as a PR.)

@Hecatron
Copy link
Author

Hecatron commented Aug 21, 2018

That would be fine
I dont have any plans to make any other changes so feel free to modify

@thinkyhead thinkyhead changed the title Support for Anycubic Linear Plus in the configurations / update to pins_TRIGORILLA_13.h [1.1.x] AnyCubic Linear Plus, Update Trigorilla 1.4 Sep 5, 2018
@thinkyhead
Copy link
Member

Perhaps @ANYCUBIC-3D would like to review this…?

thinkyhead and others added 2 commits September 15, 2018 23:17
Co-Authored-By: grbd <garlicbready@googlemail.com>
Co-Authored-By: grbd <garlicbready@googlemail.com>
@thinkyhead thinkyhead reopened this Sep 16, 2018
@thinkyhead thinkyhead merged commit 52dbd57 into MarlinFirmware:bugfix-1.1.x Sep 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants