Brush entity positions are offset

[laurirasanen]

Brushes tied to ents like func_brush, func_illusionary, trigger_teleport get an offset applied to them.

vmf: test.zip

Hammer:

SourceRadiant:

[laurirasanen]

FWIW if I set the entity origin to 0 0 0 in hammer it shows up correctly in SourceRadiant, so it's probably getting applied twice?

[erysdren]

pushed a fix, but it's not foolproof and i'm not sure how well Hammer will take the VMF saved out by SourceRadiant. ideally there would be a flag in the entity that tells other parts of the editor to ignore the origin when setting up the renderable brush geometry, but keeps the "origin" key in the entity keyvalues.

[SirYodaJedi]

i'm not sure how well Hammer will take the VMF saved out by SourceRadiant.

Hammer will re-add the origin key at the center of the brush geometry's AABB, as long as the KV is defined as being the origin type.

ideally there would be a flag in the entity that tells other parts of the editor to ignore the origin when setting up the renderable brush geometry, but keeps the "origin" key in the entity keyvalues.

Or a per-gamepack flag. VBSP treats the origin KV how older Quake family treat origin brushes, offsetting the bmodel's origin. HL1 compilers (which has a gamepack in here now) do actually behave how SR is acting.
I think IdTech 4 handles the origin KV similarly to Source? IDK how that's handled internally in DarkRadiant (see relevant gif on Trenchbroom repo issue).