Skip to content

Added the Artillery Sidewinder X1 and Genius - #8301

Merged
Konstantinos Karmas (konskarm) merged 17 commits into
Ultimaker:masterfrom
cataclism:master
Oct 13, 2020
Merged

Added the Artillery Sidewinder X1 and Genius#8301
Konstantinos Karmas (konskarm) merged 17 commits into
Ultimaker:masterfrom
cataclism:master

Conversation

@cataclism

Copy link
Copy Markdown
Contributor

Added the Artillery Sidewinder X1 and Genius.
Hope everything right.

Added the configuration files for the Artillery Printers
@Ghostkeeper

Copy link
Copy Markdown
Contributor

These files are 5 releases out of date. Please update them to setting_version 15 and make sure that the print quality is the same. In particular, the Maximum Resolution setting has changed meaning since then.

The automated tests are also complaining that the JSON structure is incorrect.

@cataclism

Copy link
Copy Markdown
Contributor Author

Ghostkeeper how do i fix the JSON files? Why the structure is incorrct and how do i make it correct? I'm sorry but i'm a newbie here in git

@kampfwuffi

Copy link
Copy Markdown

Hope this will somehow fixed.
Atm Artillery Sidewinder X1 users are basically stuck to downloading 3DN Settings and Profiles for Cura.
https://3d-nexus.com/resources/file-archives/download/8-artillery-evnovo/32-3dn-artillery-swx1-and-genius-cura-4-6-1-machines-and-profiles-installation-package
They work fine with my Sidewinder X1, as many other users report too.

@Ghostkeeper

Copy link
Copy Markdown
Contributor

Ghostkeeper how do i fix the JSON files? Why the structure is incorrct and how do i make it correct? I'm sorry but i'm a newbie here in git

It's admittedly pretty hard to read through the noise. In the test output it does say why it's failing:

----------------------------- Captured stdout call -----------------------------
[MainThread] UM.Settings.DefinitionContainer._preprocessParsedJson [299]: Unable to override setting machine_head_polygon

The setting machine_head_polygon doesn't exist any more, but your definitions still try to give it a value.

Today we also upped the setting_version for the 4.8 upgrade, which means that setting_version now needs to be 16. Sorry about that. The changes we made are to the unit of the material_shrinkage_factor setting, which I don't think you are using.

@cataclism

Copy link
Copy Markdown
Contributor Author

Ghostkeeper I finally passed the pytest ahahha

@Ghostkeeper

Copy link
Copy Markdown
Contributor

Artillery's printers are the ones I see the most errors from coming in from people who mod it in their own Cura installations. So hopefully this PR would make all of those mods unnecessary ;)

@cataclism

Copy link
Copy Markdown
Contributor Author

Ghostkeeper I hope so too. I hope that everything is correct. How do I know if it will be added in the next cura release?

@konskarm

Konstantinos Karmas (konskarm) commented Sep 28, 2020

Copy link
Copy Markdown
Contributor

Hi cataclism. I just noticed a couple of things.

  • You have put all generic materials in the "exclude_materials" field, even though you have qualities for some of them (e.g. generic_abs and generic_pla).
  • In the generic ABS, PETG, and TPU your 0.4mm nozzle has qualities 0.12-0.28 which makes sense, but then your 0.6mm nozzle has only for 0.2, which seems a bit strange. PLA properly has qualities for layer heights 0.2-0.32mm in the 0.6 nozzle.
  • Both 0.8mm and 1.0mm nozzles have only 0.32mm layer_height quality. What is the difference between them?

Are the above done on purpose?

@cataclism

Copy link
Copy Markdown
Contributor Author

Konstantinos Karmas (@konskarm) I'll add again generic_pla,generic_abs,generic_petg,generic_tpu.

For 0.8mm and 1.0mm nozzle i've left only 0.2 because I thought that they are not often used, and so if someone wants to use it he can change the layer height by himself.

And i'll add more layer heights for 0.6mm nozzle.

If something is wrong or if I have to make changes please let me know

@konskarm

Copy link
Copy Markdown
Contributor

Konstantinos Karmas (@konskarm) I'll add again generic_pla,generic_abs,generic_petg,generic_tpu.

For 0.8mm and 1.0mm nozzle i've left only 0.2 because I thought that they are not often used, and so if someone wants to use it he can change the layer height by himself.

And i'll add more layer heights for 0.6mm nozzle.

If something is wrong or if I have to make changes please let me know

Don't get me wrong, it may be fine for your printers, I was just wondering if my observations were done intentionally on your end. It doesn't mean that they are wrong or they won't work!

@cataclism

cataclism commented Sep 29, 2020

Copy link
Copy Markdown
Contributor Author

Hi Konstantinos Karmas (@konskarm)
Yeah, I did it intentionally. Since most of users use 0.4mm nozzle and everything worked like a charm, I didn't ask myself the problem and I didn't want to change too many things, risking not to make everything work anymore.

And if you swap the nozzle with a different one I thought that you are able to work yourself with the profile.
Tell me if I am wrong.

Anyway, tell me if everything works and if CURA slices correctly (and if it can be merged)
If I have to introduce any modification please let me know. =)

Comment thread resources/definitions/artillery_base.def.json Outdated
Comment thread resources/definitions/artillery_base.def.json Outdated
Comment thread resources/definitions/artillery_base.def.json Outdated
Comment thread resources/quality/artillery/PETG/artillery_0.3_PETG_adaptive.inst.cfg Outdated
Comment thread resources/quality/artillery/PETG/artillery_0.3_PETG_low.inst.cfg Outdated
Comment thread resources/quality/artillery/PETG/artillery_0.2_PETG_super.inst.cfg Outdated
Comment thread resources/quality/artillery/PETG/artillery_0.2_PETG_ultra.inst.cfg Outdated
@cataclism

Copy link
Copy Markdown
Contributor Author

Konstantinos Karmas (@konskarm)
I really hope that everithing now is right. I'm sorry but i'm a newbie and i'm not pratctical here in GitHub.
I hope I don't waste ou time

@kampfwuffi

Copy link
Copy Markdown

Hi, i'm so happy reading this conversation. Thanks for your work!

@cataclism

Copy link
Copy Markdown
Contributor Author

Hi, i'm so happy reading this conversation. Thanks for your work!

pappl Thank you! I really hope everything is right! 🤞

….cfg

Co-authored-by: Konstantinos Karmas <konskarm@gmail.com>
cataclism and others added 3 commits October 12, 2020 10:43
….cfg

Co-authored-by: Konstantinos Karmas <konskarm@gmail.com>
Co-authored-by: Konstantinos Karmas <konskarm@gmail.com>
…nst.cfg

Co-authored-by: Konstantinos Karmas <konskarm@gmail.com>
@konskarm
Konstantinos Karmas (konskarm) merged commit d56fb92 into Ultimaker:master Oct 13, 2020
@ark-

ark- commented Apr 16, 2022

Copy link
Copy Markdown

I wondered if anyone knew the config for the X2? I believe all you need to add is the auto level command to the G code.

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.

5 participants