Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Another missing face #175

Closed
ericwa opened this issue Sep 16, 2017 · 2 comments
Closed

Another missing face #175

ericwa opened this issue Sep 16, 2017 · 2 comments

Comments

@ericwa
Copy link
Owner

ericwa commented Sep 16, 2017

Unlike #174 this is a regression from tyrutils v0.15.
Also unlike #174 this bug is not in the CSG phase. The face is there in the exported obj after CSGFaces.

screen shot 2017-09-15 at 11 15 24 pm

spasm0001

{
 "classname" "worldspawn"



 {
  ( -288 -2048 928 ) ( -416 -2048 928 ) ( -288 -1920 928 ) brick2 2048 288 90 1 1 //TX1
  ( -95.99998 -2112 704 ) ( 32.00002 -2112 704 ) ( -95.99998 -1984 704 ) brick2 2112 -96 90 1 -1 //TX1
  ( -32.00013 -2192 672 ) ( -32.00013 -2192 800 ) ( -32.00003 -2064 672 ) brick2mfx2 2192 672 0 1 1 //TX1
  ( -176 -2032 1216 ) ( -176 -2032 1344 ) ( -304 -2032 1216 ) brick2 -176 1216 0 -1 1 //TX1
  ( -96 -1776 704 ) ( -96 -1776 832 ) ( -96 -1904 704 ) brick2 -1776 704 0 -1 1 //TX1
  ( 736 -2112 1184 ) ( 736 -2112 1312 ) ( 864 -2112 1184 ) brick2 -736 1184 0 1 1 //TX1
 }



 {
  ( 104 -1976 704 ) ( 104 -1976 832 ) ( -24 -1976 704 ) archdoor2a 104 704 0 -1 1 //TX1
  ( -208 -2032 896 ) ( -80 -2032 896 ) ( -208 -2160 896 ) brick2 -2032 -208 -90 1 1 //TX1
  ( -144 -2032 704 ) ( -272 -2032 704 ) ( -144 -2160 704 ) brick2 -2032 144 -90 1 -1 //TX1
  ( -96 -1776 704 ) ( -96 -1776 832 ) ( -96 -1904 704 ) archdoor2a -1776 704 0 -1 1 //TX1
  ( -32 -2032 784 ) ( -32 -2032 912 ) ( 96 -2032 784 ) brick2 32 784 0 1 1 //TX1
  ( -32 -2192 672 ) ( -32 -2192 800 ) ( -32 -2064 672 ) brick2mfx2 2192 672 0 1 1 //TX1
 }


 }

// entity 1
{
"classname" "info_player_start"
"origin" "64 -2008 728"
"angle" "180"
}
@ericwa
Copy link
Owner Author

ericwa commented Sep 16, 2017

Introduced by a910dd8
Reverting that fixes the missing face, but reintroduces leaks-through-solids in ad_swampy (and probably shib5 which was the original reason for adding a910dd8)

Probably the best bet is

  • revert that commit.
  • add a flag -new_content_assignment that enables a910dd8 (but causes missing faces)
  • migrate to q2's qbsp?

@ericwa ericwa closed this as completed in d4f3e5d Sep 16, 2017
@ericwa
Copy link
Owner Author

ericwa commented Sep 16, 2017

Fixed by d4f3e5d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant