Possible bug in texture lock code #1069

Open
kduske opened this Issue Sep 2, 2015 · 5 comments

Comments

Projects
None yet
3 participants
Owner

kduske commented Sep 2, 2015

Produces nan in one of the face attributes.

kduske added this to the TrenchBroom 2.0.0 milestone Sep 2, 2015

Owner

kduske commented Sep 3, 2015

@totalnewbie here is the bug report for the nan issue in case you find out how to reproduce it.

Ok, thanks. I'll keep trying.

Here's the map file you asked for: https://gist.github.com/totalnewbie/f466c40804db72529570 It's just something I played around with for a couple of hours to try out a few things (i.e. it sucks, but I know it sucks).

As I wrote on Func_msgboard, manually replacing all instances of "nan" with "0" seems to fix things; otherwise TB refuses to open the file and just says

(line 441, column 107)Expected integer or decimal, but got 'nan'

Collaborator

ericwa commented Sep 4, 2015

I see the 'nan' is only happening in the Valve 220 "texture rotation" field.

This is the value whose purpose is to record the rotation knob position in the Hammer texture alignment UI, and is ignored by qbsp.

Owner

kduske commented Sep 4, 2015

I think I was trying to be smart about that field by recomputing it somewhere. Guess I messed up.

Owner

kduske commented Oct 6, 2015

I'm going to push this to 2.1.0 or until someone comes up with a reproducible test case.

kduske removed this from the TrenchBroom 2.1.0 milestone Mar 15, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment