tr_model_iqm: rewrite NaN
in model IQM input
#1469
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Rewrite
NaN
in model IQM input.The
prifle_view.iqm
model has someNaN
in the input data:prifle_view.iqm
model has someNaN
in the input data Unvanquished/Unvanquished#3252When such model file is used, it is impossible to get anything useful when trapping for invalid operations because the first call of
R_TBNtoQtangents()
when loading the model will stop the execution.There are other floats in the IQM file, we may want to not hack
BuildTangents()
and sanitize all the floats in the future with a generic function, but this works for now.Here is the IQE text equivalent of the IQM format, there are many float fields like that:
Here is an example of broken input, the current code currently only cares about
vx
.