0003972: Have blood flash intensity vary in relation to "player.maxhealth" rather than hard-coded 100

Chat on our Discord Server Get the latest version: 3.2
Source Code

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0003972

Zandronum

[All Projects] Suggestion

public

2022-01-19 00:58

2022-09-29 04:06

Reporter

Trillster

Assigned To

Kaminsky

Priority

normal

Severity

feature

Reproducibility

N/A

Status

resolved

Resolution

fixed

Platform

Windows 11

OS Version

Product Version

3.1

Target Version

3.2

Fixed in Version

3.2

Summary

0003972: Have blood flash intensity vary in relation to "player.maxhealth" rather than hard-coded 100

Description

Blood flash intensity increases off of the damage a player takes, however it does not take into account the player's max health. In a mod where all players have 1000 HP, this can result in rapid screen flashes which requires "blood_fade_scalar" to be lowered to mitigate, however this CVAR just lowers the maximum intensity rather than adjusting the scaling formula.

Best solution would be adjusting the blood flash intensity scaling so that it also takes into account how much max health the player has.

Attached Files

ticket3972.wad [^] (162 bytes) 2022-09-29 03:38

Relationships

Notes
(0022196) Kaminsky (developer) 2022-04-24 20:41	3.2 now has the CVar "blood_fade_usemaxhealth" (disabled by default), which will scale the intensity of the blood flash according to the player's max health when it's not 100. The commit for this can be found here:'http://hg.osdn.net/view/zandronum/zandronum-stable/rev/457cc8aa74be [^]'

(0022412) WaTaKiD (updater) 2022-09-29 03:40	uploaded an example wad where the player has 1k max health tested on windows and linux servers with sv_maxbloodscalar on and off, getting direct hit by rockets fired from a 2nd client in deathmatch, blood_fade_usemaxhealth seemed to work correctly

(0022414) Kaminsky (developer) 2022-09-29 04:06	Thanks for testing, I'll go ahead and mark this ticket as resolved now.

**Issue Community Support**
This issue is already marked as resolved. If you feel that is not the case, please reopen it and explain why.
Supporters:	No one explicitly supports this issue yet.
Opponents:	No one explicitly opposes this issue yet.

Issue History
Date Modified	Username	Field	Change
2022-01-19 00:58	Trillster	New Issue
2022-04-24 20:41	Kaminsky	Note Added: 0022196
2022-04-24 20:41	Kaminsky	Assigned To	=> Kaminsky
2022-04-24 20:41	Kaminsky	Status	new => needs testing
2022-04-24 20:41	Kaminsky	Product Version	3.1-beta => 3.1
2022-04-24 20:41	Kaminsky	Target Version	=> 3.2
2022-09-29 03:38	WaTaKiD	File Added: ticket3972.wad
2022-09-29 03:40	WaTaKiD	Note Added: 0022412
2022-09-29 04:06	Kaminsky	Note Added: 0022414
2022-09-29 04:06	Kaminsky	Status	needs testing => resolved
2022-09-29 04:06	Kaminsky	Fixed in Version	=> 3.2
2022-09-29 04:06	Kaminsky	Resolution	open => fixed

Questions or other issues? Contact Us.

Links

Doom and Doom II are the property of id Software.