Skip to content

[core] Adjust desynth success penalty for -1 to -7 range to 5.0f per difficulty#9384

Merged
Xaver-DaRed merged 1 commit into
LandSandBoat:basefrom
Hatberg:desynth-difficulty-penalty
Feb 6, 2026
Merged

[core] Adjust desynth success penalty for -1 to -7 range to 5.0f per difficulty#9384
Xaver-DaRed merged 1 commit into
LandSandBoat:basefrom
Hatberg:desynth-difficulty-penalty

Conversation

@Hatberg
Copy link
Copy Markdown
Contributor

@Hatberg Hatberg commented Feb 6, 2026

I affirm:

  • I understand that if I do not agree to the following points by completing the checkboxes my PR will be ignored.
  • I understand I should leave resolving conversations to the LandSandBoat team so that reviewers won't miss what was said.
  • I have read and understood the Contributing Guide and the Code of Conduct.
  • I have tested my code and the things my code has changed since the last commit in the PR and will test after any later commits.

What does this pull request do?

The Desynthesis successRate penalty was off by two digits in the -1 to -7 range. Changed the penalty per level from 0.05f to 5.0f.

Steps to test these changes

Add print logging to synthutils at the -1/-7 desynthesis range (lines 605, 620) and rebuild

!additem 4100 20 (lightning crystals)
!additem 596 20 (quadav backplates)

Desynth items in game, after setting smithing levels:
!setskill smithing 51 (through 41)
Read out the console logs in map-server

Before:
image

After:
image

Sources: https://www.bluegartr.com/threads/135055-Extensive-Desynthesis-Rate-Research

image

@Xaver-DaRed Xaver-DaRed merged commit ae68462 into LandSandBoat:base Feb 6, 2026
19 of 20 checks passed
@Hatberg Hatberg deleted the desynth-difficulty-penalty branch February 6, 2026 17:59
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.

3 participants