Skip to content

Dark Matters#3756

Open
ViralReaction wants to merge 6 commits into
CombatExtended-Continued:Developmentfrom
ViralReaction:Darkness-Shooting
Open

Dark Matters#3756
ViralReaction wants to merge 6 commits into
CombatExtended-Continued:Developmentfrom
ViralReaction:Darkness-Shooting

Conversation

@ViralReaction
Copy link
Copy Markdown
Contributor

@ViralReaction ViralReaction commented Apr 5, 2025

Changes

Describe adjustments to existing features made in this merge, e.g.

  • Darkness shooting errors only start calculating at more than 50% darkness
  • Use a simple curve to calculate the error as it should get progressively harder to shoot the darker it is
  • Fixes issue with lighting calculation when standing next to buildings

References

Links to the associated issues or other related pull requests, e.g.

Reasoning

Why did you choose to implement things this way, e.g.

  • 50% light is indoor lighting inside RW.
  • Even with a few lights it should not be hard to identify your target

Alternatives

Describe alternative implementations you have considered, e.g.

  • Perfect Shooting

Testing

Check tests you have performed:

  • Compiles without warnings
  • Game runs without errors
  • (For compatibility patches) ...with and without patched mod loaded
  • Playtested a colony (specify how long)

@ViralReaction ViralReaction requested review from a team as code owners April 5, 2025 01:53
@ViralReaction ViralReaction marked this pull request as draft April 5, 2025 01:53
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 5, 2025

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-14277197789.zip

@github-actions github-actions Bot added the Download in Comments This PR has a zipfile download available. label Apr 5, 2025
@ViralReaction ViralReaction changed the title Darkness Matters Dark Matters Apr 5, 2025
@N7Huntsman N7Huntsman added this to [6.4] Apr 25, 2025
@N7Huntsman N7Huntsman moved this to Todo in [6.4] Apr 25, 2025
@N7Huntsman N7Huntsman added the On Hold Being held until after the next release. label Apr 25, 2025
@github-actions
Copy link
Copy Markdown

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-15103026382.zip

@ViralReaction ViralReaction marked this pull request as ready for review May 19, 2025 02:42
@ViralReaction ViralReaction marked this pull request as draft May 21, 2025 00:41
@github-actions
Copy link
Copy Markdown

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-15150824192.zip

@github-actions
Copy link
Copy Markdown

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-15174548710.zip

@Keshash
Copy link
Copy Markdown
Contributor

Keshash commented May 28, 2025

From my testing, this is a good PR, but the added curve makes muzzle flashes way too strong at explosing the shooter. Without the curve, there was a difference between flash sizes, with it there's none.

@ViralReaction ViralReaction marked this pull request as ready for review June 4, 2025 01:33
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2025

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-15431474450.zip

Comment thread Source/CombatExtended/CombatExtended/ShiftVecReport.cs Outdated
@github-actions
Copy link
Copy Markdown

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-15864562978.zip

@github-actions
Copy link
Copy Markdown

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-15864633173.zip

@Breadbox2k19
Copy link
Copy Markdown
Contributor

The accuracy malus at 0% brightness should still be the same, correct?

@Breadbox2k19
Copy link
Copy Markdown
Contributor

From my testing, this is a good PR, but the added curve makes muzzle flashes way too strong at exposing the shooter. Without the curve, there was a difference between flash sizes, with it there's none.

If there's a way to universally halve the lighting effect of muzzle flash, that'd be great.

@Breadbox2k19
Copy link
Copy Markdown
Contributor

Breadbox2k19 commented Jul 2, 2025

Another thing to consider is that it could make having more 50% night vision totally worthless/ have rapidly diminishing return. Until that could be resolved, I think it's best to hold on to this for abit.

@ViralReaction ViralReaction removed the On Hold Being held until after the next release. label Jul 17, 2025
@Breadbox2k19
Copy link
Copy Markdown
Contributor

May I suggest changing the curve to the following, basically the original max darkness penalty but with other issues fixed? (7 being the original max darkness penalty)

  • (0, 0)

  • (0.5, 3.5)

  • (1, 7)

  • I know for a fact that there's not going to be working backwards from big numbers since there's so many variable in CE making controlled testing a pain so barely anyone playtests for balance at all. Considering that 50% of all raids occur at night, simply making ranged weapons considerably worse during 50% of all raids is not a good balance change imo.

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Labels

    Download in Comments This PR has a zipfile download available.

    Projects

    No open projects
    Status: Todo

    Development

    Successfully merging this pull request may close these issues.

    [Bug]: Darkness Accuracy Malus has unexpected behavior

    5 participants