-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Added BIQU machine base files as well as BIQU B1 machine files. #8276
Conversation
Is there a specific reason that you added an inbetween base profile? You also don't need to exclude the 2.85 materials in the machine definition. Those are automatically excluded. |
BIQU have many printers that are similar in design and will therefore share mostly the same settings with just a few variations per machine. While I've only added one machine in this PR there will be others to come and having a base definition made the most sense to me since I would therefore only need to edit the deltas between machines. Please let me know if I have misunderstood this point. Regarding the materials I'll just allow anything through and leave Cura to filter out the 2.85 batch. Thanks for the tip. New commit incoming. |
You didn't, but it's a mistake I've seen other people make before, hence me mentioning it. |
…nts. Adjusted materials in quality profiles to refer to the 175 profiles directly.
I've fixed the author in the variants and the comments in the quality profiles. I also adjusted the quality profiles to refer directly to the 175 generic materials instead of to the 2.85 generic materials. Please let me know if you see any issue with this adjustment. I know that Cura automatically makes the change but I figured that there would be no harm in referencing the correct file directly. |
… of infill for the base profile.
Hi @Ghostkeeper and @nallath. I just want to check up on this to see whether there is anything left that is expected of me before this can be merged. Thanks gents. |
I was reviewing the CR6SE PR and I saw that you recommended a settings version of 16 for 4.8 merge candidates. I'll bump these and push the changes shortly. |
…n anticipation of release 4.8
Since you made your PR before we bumped up the settings version, I would normally do the version upgrade myself upon merging. It's really our fault for being slow then. I think you're okay. It's just us being slow. |
Would be awesome to get this merged; let me know if I can help! |
I'm sure @Ghostkeeper and @nallath will get around to it. The 4.8 release must just be keeping them super tied up. They did approve my other PR to the FDM materials so there has been some movement 👍💪😀 |
Hey @looxonline. I noticed that you made changes to this PR up until recently. Is it finalized now? |
Hey there. Yup, I've just been tweaking and finessing but none of it was essential. It's good to be merged now. Thanks as always guys. |
Both machines slice fine. Also the generic_pla_175 is modified to override some material settings for these printers. As a result, I am accepting this PR and the printers will become available from Cura 4.8 onwards. |
Hey @konskarm. Any chance to see a nightly build of the master or Cura 4.8 (beta) containing this pull? 🙃 |
Y'all are the best, thanks for landing this for 4.8 <3 |
If you guys are really that eager for it then join the B1 facebook group and you will find a post there where I have all the files and instructions on how to integrate them into the current Cura for testing purposes. |
We found a few more problems with these profiles, after the fact:
I've fixed the IDs here: a73cd96 |
Thanks for picking up on these points. The profiles were strongly based on the Creality methods and original profiles due to the machine similarities which explains the quality overlap. I figured that the internal definition name change would be enough to allow the parser to differentiate. I didn't realize that the filename could also not be duplicated. Thanks again. |
looxonlineIs there any way to re-edit this? The Biqu B1 printer is set in its base profile to a support angle of 45º and this is preventing Cura from calculating different overhang angles... it simply does not change the angle as other printers allow, since Cura now does this automatically and on Biqu printers this is not possible to be done automatically by Cura. |
This PR adds a base profile for BIQU printers and on top of that it adds the first printer, the B1. More will follow.
There is also a change required to the PLA 175 materials file which will be committed shortly. I'll reference this PR in that one.