System.ArgumentException: Version string portion was too short or too long. about icondiffbot HOT 6 CLOSED

Fixed by tgstation/icon-cutter#11

from icondiffbot.

Still reproduces in a re-ran check, so it's not completely transient: https://github.com/goonstation/goonstation/pull/7226/checks?check_run_id=4877898512

from icondiffbot.

walls_ancient.dmi and walls_cave.dmi (walls_overgrown is fine) as of that commit has metadata that begins like:

# BEGIN DMI
version = 
	width = 32
	height = 32

when it should look like:

# BEGIN DMI
version = 4.0
	width = 32
	height = 32

Probably IconDiffBot should not crash here, and anyways should not be using System.Version to represent this field, which is not "the version number of an assembly, operating system, or the common language runtime".

from icondiffbot.

Interesting, those were both created with Dream Maker, so I blame DM for generating a malformed file.
I'd definitely argue it shouldn't crash and default to 4, especially if the program is liable to omit such information.

from icondiffbot.

I've been running into the same issue on wall-nerds/wallening#83 .
I saw that the /tg/ cutting tool that we were using for this (Which apparently also runs into the same issue with tables as it does with walls!) wasn't adding the version variable as an argument. However, by manually adding that version 4.0 to the file through tweakPNG, it now claims that all of those files are now out of date... AND still can't be read by icondiffbot! Except for the pizza walls, for some reason.
icerock_wall.zip
I don't know how much of it is my end and how much of it is from trying to force a solution, but I found this issue now after working on this for about an hour.

from icondiffbot.

walls_ancient.dmi and walls_cave.dmi (walls_overgrown is fine) as of that commit has metadata that begins like:

# BEGIN DMI
version = 
	width = 32
	height = 32

when it should look like:

# BEGIN DMI
version = 4.0
	width = 32
	height = 32

Probably IconDiffBot should not crash here, and anyways should not be using System.Version to represent this field, which is not "the version number of an assembly, operating system, or the common language runtime".

Ran into an extremely similar issue when doing PR #69147, but tweakpng seems to have fixed it. Unfortunately for Arcane's case, doing so did not fix his.

from icondiffbot.