Skip to content

Conversation

apparle
Copy link
Contributor

@apparle apparle commented Mar 2, 2025

Description

Sovol introduced a high-speed firmware + profile an year back. That profile was initially merged into Orcaslicer as part of #5673 and #5675 . But it looks like that only updated the process json.
That is incomplete without updated nozzle acceleration settings & filament settings, rendering the High-speed profile broken and no faster than default. Someone else also observed this on #5673

This PR contains rest of the profile changes from Sovol with some minor modifications:

  • Sovol's profile just overwrote the default SV06 nozzle profile. But since not everyone will be on new firmware, I've created a separate nozzle file and left the original one alone as there are significant changes.
  • Minor updates to bring the outdated process files inline with other changes that have happened to all profiles.

Tests

I've tested this with 2.2.0 flatpak locally.

Signed-off-by: Apoorv Parle <19315187+apparle@users.noreply.github.com>
Copy link
Owner

@SoftFever SoftFever left a comment

Choose a reason for hiding this comment

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

Thanks.
Can you take a look at my comment?

@SoftFever
Copy link
Owner

Hey @apparle
A quick heads up, I have merged a PR that fixes wrong default filaments in may profiles.
It might cause some small conflicts

@SoftFever SoftFever closed this Mar 7, 2025
@apparle
Copy link
Contributor Author

apparle commented Mar 7, 2025

@SoftFever not sure you closed this by mistake or assumed I'm not responding.
I did intend to respond, just that my day job is keeping me busy, so haven't had a chance to review the filament profiles properly.
Doing a cursory review if you want to make progress now, I do see many differences like temperature and fan settings between the generic profile and the new profile as part of this PR. It's not the same as far as I can tell.
But maybe I'm just new to profile structure and may be comparing the wrong files - can you point me to the filament file which you're observing as exact same?

Also, I did check with the commit you highlighted and it doesn't have any changes under Sovol directory, so no conflicts.

I'm still interested in getting this merge, so can you please open this again?

@SoftFever
Copy link
Owner

@SoftFever not sure you closed this by mistake or assumed I'm not responding. I did intend to respond, just that my day job is keeping me busy, so haven't had a chance to review the filament profiles properly. Doing a cursory review if you want to make progress now, I do see many differences like temperature and fan settings between the generic profile and the new profile as part of this PR. It's not the same as far as I can tell. But maybe I'm just new to profile structure and may be comparing the wrong files - can you point me to the filament file which you're observing as exact same?

Also, I did check with the commit you highlighted and it doesn't have any changes under Sovol directory, so no conflicts.

I'm still interested in getting this merge, so can you please open this again?

Sure thing, let me re-open it.
You should be able to re-open it yourself as well.

I initially closed it so I wouldn’t keep checking the PR status. But now that I think about it, it’s probably better to convert it to a draft or add a label instead of closing it.

Anyway, feel free to re-open the PR! In the future, it’d be helpful to leave a message to let me know.

@SoftFever SoftFever reopened this Mar 7, 2025
@SoftFever SoftFever added the wait for response waiting for response from PR creator label Mar 7, 2025
@SoftFever
Copy link
Owner

I wasn’t referring to conflicts.
At a glance, the settings in “Sovol SV06 PLA” are the same as those in “Generic PLA @System.”
In such a case, we don’t need to create a new filament profile.

@apparle
Copy link
Contributor Author

apparle commented Mar 8, 2025

I wasn’t referring to conflicts. At a glance, the settings in “Sovol SV06 PLA” are the same as those in “Generic PLA @System.” In such a case, we don’t need to create a new filament profile.

The temperature settings are different between the 2 files are different once I trace through all the "inherits" and look at the final settings.

@SoftFever
Copy link
Owner

I wasn’t referring to conflicts. At a glance, the settings in “Sovol SV06 PLA” are the same as those in “Generic PLA @System.” In such a case, we don’t need to create a new filament profile.

The temperature settings are different between the 2 files are different once I trace through all the "inherits" and look at the final settings.

Thanks for the clarification.
Just a couple of quick follow-up questions:
Is this “Sovol SV06 “brand PLA filament or a generic filament brand?
And was the temperature tested?

@apparle
Copy link
Contributor Author

apparle commented Mar 8, 2025

I wasn’t referring to conflicts. At a glance, the settings in “Sovol SV06 PLA” are the same as those in “Generic PLA @System.” In such a case, we don’t need to create a new filament profile.

The temperature settings are different between the 2 files are different once I trace through all the "inherits" and look at the final settings.

Thanks for the clarification. Just a couple of quick follow-up questions: Is this “Sovol SV06 “brand PLA filament or a generic filament brand? And was the temperature tested?

No, there isn't specific to a filament brand, AFAIK there isn't even a "Sovol SV06" brand filament at all. I'm using these with a Hatchbox PLA filament.

Sovol provided these profiles along with the high-speed firmware here, specifically for the high-speed usage of the SV06 model printer, so they may have studied this. I'm guessing SV06 printer requires higher temperature than usual for a faster flowing filament when machine is using higher acceleration.
I've not myself done an ablation study to check which of these are critical for high-speed; in fact I'm relatively new to 3d printing to understand or carefully study this. I just noticed OrcaSlicer has half-baked and broken high-speed profile for SV06, and I've to copy rest of the files manually to use with my printer. So I'm just trying to fix that :-)

@SoftFever
Copy link
Owner

I wasn’t referring to conflicts. At a glance, the settings in “Sovol SV06 PLA” are the same as those in “Generic PLA @System.” In such a case, we don’t need to create a new filament profile.

The temperature settings are different between the 2 files are different once I trace through all the "inherits" and look at the final settings.

Thanks for the clarification. Just a couple of quick follow-up questions: Is this “Sovol SV06 “brand PLA filament or a generic filament brand? And was the temperature tested?

No, there isn't specific to a filament brand, AFAIK there isn't even a "Sovol SV06" brand filament at all. I'm using these with a Hatchbox PLA filament.

Sovol provided these profiles along with the high-speed firmware here, specifically for the high-speed usage of the SV06 model printer, so they may have studied this. I'm guessing SV06 printer requires higher temperature than usual for a faster flowing filament when machine is using higher acceleration. I've not myself done an ablation study to check which of these are critical for high-speed; in fact I'm relatively new to 3d printing to understand or carefully study this. I just noticed OrcaSlicer has half-baked and broken high-speed profile for SV06, and I've to copy rest of the files manually to use with my printer. So I'm just trying to fix that :-)

Understood, and I appreciate it! This community sharing and caring thing is the core value of OrcaSlicer!
In this case, I would suggest removing this filament profile since, as far as I can see, it didn't really carry many fine-tuned changes for the machine.

… types of filaments.

Signed-off-by: Apoorv Parle <19315187+apparle@users.noreply.github.com>
@apparle
Copy link
Contributor Author

apparle commented Mar 9, 2025

Updated to remove the filament. Also added other Generic filament types.

@apparle apparle requested a review from SoftFever March 10, 2025 04:11
Copy link
Owner

@SoftFever SoftFever left a comment

Choose a reason for hiding this comment

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

LGTM
Thank you.
I will make some adjustments later once this PR is merged.

@SoftFever SoftFever merged commit 9227ebb into SoftFever:main Mar 10, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wait for response waiting for response from PR creator
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants