Skip to content

Adding the MINGDA D2 - #8872

Merged
Konstantinos Karmas (konskarm) merged 29 commits into
Ultimaker:masterfrom
cataclism:master
Jan 11, 2021
Merged

Adding the MINGDA D2#8872
Konstantinos Karmas (konskarm) merged 29 commits into
Ultimaker:masterfrom
cataclism:master

Conversation

@cataclism

Copy link
Copy Markdown
Contributor

Hi! Cataclism here! I've worked in order to add the MINGDA D2 to Cura. The base is currently missing but i'll add it soon =)

cataclism and others added 24 commits September 2, 2020 11:08
Added the configuration files for the Artillery Printers
Co-authored-by: Konstantinos Karmas <konskarm@gmail.com>
Co-authored-by: Konstantinos Karmas <konskarm@gmail.com>
Co-authored-by: Konstantinos Karmas <konskarm@gmail.com>
….cfg

Co-authored-by: Konstantinos Karmas <konskarm@gmail.com>
….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>
Inserted artillery genius plate for the moment
@nallath

Copy link
Copy Markdown
Contributor

I'm a bit confused by the commit history of this PR. Why does it make all kinds of changes to the artillery_base?

@cataclism

Copy link
Copy Markdown
Contributor Author

Jaime van Kessel (@nallath) i don't know why all the history of changes I made in the past are attacched to the PR. Only the last point of the list is the one of interest, all the previous are from my work for the Artillery printers and it has already been approved

@cataclism

Copy link
Copy Markdown
Contributor Author

I don't know how to remove it, if you can tell me how to, i'll do it Jaime van Kessel (@nallath)

@Ghostkeeper

Copy link
Copy Markdown
Contributor

We can squash the commits when merging this.

You can also squash the commits yourself if you'd like to help. There are some instructions online. The basic idea is to do git rebase -i master while on your branch, which comes up with a file editor where you can say which commits need to be squashed into the previous commit. Then afterwards you'd need to force-push.

@Ghostkeeper

Copy link
Copy Markdown
Contributor

If the profiles for these printers are the same as the Artillery printers, I'd like to propose a change to set the following metadata on all of the new printers:

"quality_definition": "artillery_base"

This tells Cura that you're using the Artillery's profiles. This has three consequences:

  • If the Artillery profiles get updated, Mingda profiles will get updated too. This could be good (you get improvements made by the Artillery users as well) or bad (those improvements don't necessarily apply to your printer in some cases, although it might apply in many cases).
  • You can then delete these quality profiles. Cura won't need to read these extra ~80 files at every launch, which will save an extra I/O block or two to read from disk.
  • Material profiles that apply to Artillery printers will also apply to Mingda. This is something that is currently missing from your PR I think.

You can still override settings in the definition files and nozzle files, but you'd share the quality and material profiles with Artillery then.

@cataclism

Copy link
Copy Markdown
Contributor Author

I'll try to squash, hope i can do it!
I'd like to keep them separate since they are different machines. I'll work on it

Updated machine dimension
@Ghostkeeper

Copy link
Copy Markdown
Contributor

I'd like to keep them separate since they are different machines.

I'd like to deduplicate them if their profiles are the same. If they need to be different that's fine, but if they might be different in the future we can always split the profiles then and deduplicate it now.

@cataclism

Copy link
Copy Markdown
Contributor Author

Ok Ghostkeeper , got it. I'm working with the community in order to make the transition to the profiles as smooth as possible. So changhes are coming in the next days

@cataclism

Copy link
Copy Markdown
Contributor Author

Ghostkeeper first of all happy new year!!
I've talked with the community and the profiles must be separated.
So If there aren't any problems, you can merge.

Comment thread resources/definitions/mingda_d2.def.json Outdated
Comment thread resources/quality/mingda/PETG/mingda_0.3_PETG_standard.inst.cfg Outdated
Comment thread resources/quality/mingda/PETG/mingda_0.3_PETG_super.inst.cfg Outdated
Comment thread resources/quality/mingda/PETG/mingda_0.4_PETG_adaptive.inst.cfg Outdated
Comment thread resources/quality/mingda/PETG/mingda_0.4_PETG_low.inst.cfg Outdated
Comment thread resources/quality/mingda/PETG/mingda_0.8_PETG_draft.inst.cfg Outdated
Comment thread resources/quality/mingda/PETG/mingda_1.0_PETG_draft.inst.cfg Outdated
Comment thread resources/definitions/mingda_base.def.json Outdated
Comment thread resources/definitions/mingda_base.def.json Outdated
Comment thread resources/definitions/mingda_base.def.json Outdated
cataclism and others added 3 commits January 8, 2021 15:15
Co-authored-by: Konstantinos Karmas <konskarm@gmail.com>
Co-authored-by: Konstantinos Karmas <konskarm@gmail.com>
Co-authored-by: Konstantinos Karmas <konskarm@gmail.com>

@cataclism cataclism left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hope I did everything right! If there are any problems tell me Konstantinos Karmas (@konskarm)

@konskarm

Copy link
Copy Markdown
Contributor

Alright, since it is decided that the MINGDA definition should be completely separate from the Artillery ones, I will accept this PR. Just make sure to re-use the mingda_base, or else we are adding 88 files for a single printer, which is a bit excessive.

@konskarm
Konstantinos Karmas (konskarm) merged commit a2e5ce6 into Ultimaker:master Jan 11, 2021
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.

4 participants