search

cooolbros / hud-merger

A C#/WPF application for merging custom HUDs in Team Fortress 2
GNU General Public License v3.0
31 stars 0 forks source link

HUD Animations: Invalid float parsing in some cultures #5

Closed chara22322 closed 5 months ago

chara22322 commented 7 months ago

I got this error when i was merging this: tf2huds.dev/hud/MonoHud with this: tf2huds.dev/hud/DarHUD. If i knew where the default location of the certain file i am merging is, then i can search for it more easily image

cooolbros commented 7 months ago

Thanks for reporting this issue, usually the file that the error occurred in is displayed in the message, but this error actually looks to be a bug in the hud merger itself.

So far I havent been able to reproduce this error myself, because after searching online it seems to be an issue with the number parsing in dotnet itself, but only in certain languages.

Can you tell me what language your windows OS is set to so (for example US english or UK english) I can try to reproduce this error?

cooolbros commented 7 months ago

Update: I found that the french language (fr) parses decimals that resemble 0,8 as 0.8 and fails for decimals used in HUD animations such as 0.8, which matches the behaviour in the initial report

cooolbros commented 5 months ago

Fixed in 2.0.1