-
Notifications
You must be signed in to change notification settings - Fork 81
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
C4 Equipment Settings #1765
base: master
Are you sure you want to change the base?
C4 Equipment Settings #1765
Conversation
It looks like the initialization of the InnerRadius on the Clientside failed somehow, which is weird since you are using fallback values. |
Thanks for pointing out my mistakes figardo - I'll fix them when I get home tonight. |
Tested with fix and all looks fine. Didn't extensively experiment with the damage scaling but I can confirm it retrieves the value correctly. I'll sort out some translations & changelog now. |
To be close to base ttt c4 you likely also need to decrease default damage to 125 and innerradius to 490 (for me 500 is fine too). Those values are from the base c4. The 0 damage dropoff was at exactly 601.18 aka (490+sqrt(12500)) units, so 600 should be fine |
That's what base TTT uses for its deadly spheredamage, though the blast damage that also happens is 200. Right now, TTT2 uses the same value for both |
Just realising that What I actually meant to say was that TTT2 uses the same damage value for I'm wondering if we want to replicate the TTT damage values by multiplying the damage used for |
to compensate for the loss of LoS damage, cause youd get double damage if you survived one type of damage in the base c4
|
Gotcha. Damage values can be a different issue / PR - this PR helps to customise damage anyway. |
might be out of scope of this pr, but can you add min and max settings for the timer as well |
Personally I think that's already partially built into the C4 itself, as users can set the desired time up to 10 minutes. Granted, 10 minutes is a silly value to set the C4 timer to most of the time. Perhaps it would be a good idea for server admins to reduce user error by adjusting max time. Similarly, may be an argument for admins to be able to adjust the minimum timer amount if 45 seconds is considered too low or too high for the server. |
Placing 10 minute C4 in the middle of an open area is a funny meme though! |
yeah i guess it makes more sense to only be able to change the min timer |
Works towards #1729
Tested on a local server however getting this error that I can't fix without stepping on someone else's toes:@MrXonte Could you advise?