Commit Graph

3645 Commits

Author SHA1 Message Date
Jonathan 2d8827e031 Revert "Q3 didn't need chop, we don't either!"
This reverts commit f57ecaf599.

# Conflicts:
#	include/qbsp/brushbsp.hh
2022-08-21 15:04:26 -04:00
Jonathan 282c5ec69f stat printing adjustments 2022-08-21 15:01:45 -04:00
Jonathan 1aaa513368 collapse certain map face issues into loggable warning stats to lessen maps spamming warnings on things that legacy compilers allow
adjust the wording of certain warning messages
don't display face crunching on bmodels/hulls by default
2022-08-20 20:13:27 -04:00
Jonathan 7fad0019c2 set omit properly 2022-08-20 15:09:47 -04:00
Jonathan 7422776f38 move _omitbrushes to the loading phase; this will greatly improve performance of loading a map with a ton of brushes that has most of them omitted. 2022-08-20 09:53:23 -04:00
Jonathan bfe56b0663 fix the Q1 bevel code for testing later 2022-08-20 08:19:46 -04:00
Jonathan 39bbd643d5 Merge branch 'brushbsp' of https://github.com/ericwa/ericw-tools into brushbsp 2022-08-20 08:14:43 -04:00
Jonathan 8a5ad5bc4d simplify Brush_LoadEntity a bit by moving some of the contents setting to mapbrush loading; note that this causes a bit of issue with mirror_inside & clipsametype currently because of Q1 contents code. needs adjusting 2022-08-20 08:14:36 -04:00
Eric Wasylishen d9856230a6 testqbsp: q1_rocks: sky -> solid to simplify test case 2022-08-20 01:24:10 -06:00
Eric Wasylishen 08e3254dae brushbsp: track tree level for debugging 2022-08-20 00:52:24 -06:00
Eric Wasylishen 6b07e0c5f4 brushbsp: return split planes as side_t* for better debuggability 2022-08-20 00:51:52 -06:00
Eric Wasylishen f3a2d99544 testqbsp: adjust q1_rocks test case
avoid potential ambiguity for BrushBSP over whether sky should split wall brushes
2022-08-20 00:22:21 -06:00
Eric Wasylishen 251a6dc7ca testmaps: add a more minimal q1_rocks that's just a cube 2022-08-19 22:55:27 -06:00
Eric Wasylishen 648ffab77b testqbsp: improve q1_rocks test 2022-08-19 22:10:16 -06:00
Eric Wasylishen 10051ceff5 testmaps: remove some accidental brushes in q1_hull_expansion_lip.map
testqbsp: more detailed q1_hull_expansion_lip tests for clipnodes

testqbsp: decompile hull1 of testmaps by default

testqbsp: corrections to the q1_hull_expansion_lip.map expectation

adjust gitignore
2022-08-19 20:49:59 -06:00
Eric Wasylishen c0c146d47c bsputil: move decompiler to common 2022-08-19 20:49:34 -06:00
Jonathan e020cd6be6 comment out the hintskip face removal; this seems to break later processes. qbsp3 doesn't do this.. 2022-08-19 16:09:37 -04:00
Jonathan 84870cf366 since tree_t only exists as a type that only allocates heap memory, we don't need to wrap it in a unique_ptr; this also allows the vectors to keep their memory on the next pass, which may improve performance for huge maps
removed tree_t parameter from functions that don't need it
pass reference to tree_t instead of ptr
use an enum instead of std::optional<bool> for the ternary value to store split type, since it's more explicit and obvious now what the three values do
2022-08-19 15:15:53 -04:00
Jonathan 69aad26097 move & document bevel boolean
don't report invalid winding more than once
ceil the extents, so we don't get weird values like "world extents calculated to 5082.29327318318"
2022-08-19 13:46:37 -04:00
Jonathan 057ae544c4 use references in places where we know it's non-null 2022-08-19 12:11:35 -04:00
Jonathan 39475e3213 use on_epsilon instead of 0.1, to match Q1 2022-08-19 09:44:11 -04:00
Jonathan 0c4888d5d1 remove leftover debug code 2022-08-19 09:42:11 -04:00
Jonathan 9596855c62 Merge branch 'brushbsp' of https://github.com/ericwa/ericw-tools into brushbsp 2022-08-19 07:15:51 -04:00
Jonathan a5557bc87d Revert "remove the Q1 bevel stuff"
This reverts commit f5f80479a3.

# Conflicts:
#	qbsp/map.cc
2022-08-19 07:15:39 -04:00
Eric Wasylishen 832ab0ee94 testqbsp: q1_hull_expansion_lip.map: move detail to world as a temp workaround
for -debugexpand only expanding world brushes
2022-08-19 01:14:38 -06:00
Eric Wasylishen 6eab754ca3 qbsp: reset onnode at the beginning of BrushBSP 2022-08-18 22:26:33 -06:00
Jonathan f57ecaf599 Q3 didn't need chop, we don't either! 2022-08-18 19:47:34 -04:00
Jonathan 48c6b7a370 decouple extra and bounce extra; this allows you to still use extra4 but not have to wait like 16x longer for bounce lights to work 2022-08-18 07:35:06 -04:00
Jonathan 424e3ee110 remove unused portal_t::face, unused
rename and re-type a few things to use twosided
2022-08-17 09:10:58 -04:00
Jonathan 9e764d8035 document some stuff
remove mapface_t::value and mapface_t::flags; they are duplicated by texinfo
add mapface_t::get_texinfo and face_t::get_texinfo so that you can fetch them easier
2022-08-17 07:30:31 -04:00
Jonathan a2d6b63533 Merge branch 'brushbsp' of https://github.com/ericwa/ericw-tools into brushbsp
# Conflicts:
#	qbsp/brush.cc
#	qbsp/map.cc
2022-08-17 03:50:52 -04:00
Jonathan f5f80479a3 remove the Q1 bevel stuff 2022-08-17 03:49:42 -04:00
Eric Wasylishen fe43928802 fix build (unwanted static on TestExpandBrushes) 2022-08-17 01:18:11 -06:00
Eric Wasylishen a2a7b987bc revert last commit, q1 code is still producing better results with q1_rocks.map 2022-08-17 01:17:19 -06:00
Eric Wasylishen 688a033288 qbsp: switch back to qbsp3 brush bevels in q1 mode
now that the bug in the previous commit is fixed
2022-08-16 23:15:09 -06:00
Eric Wasylishen 6b1720e095 qbsp: fix bug from qbsp3's AddBrushBevels where edge bevels are wrongly rejected
fixes bad result in q1_hull_expansion_lip.map
2022-08-16 23:11:43 -06:00
Eric Wasylishen 4801e87b1c testqbsp: don't allow q1_hull_expansion_lip to fail 2022-08-16 23:05:17 -06:00
Jonathan 95c728a5b6 fix legacy bevel bug
pushing up changes for Q1 hulls just in case we ever want these back
2022-08-16 13:52:30 -04:00
Jonathan 0c90a33f28 enable Q1-style hull expansion; these expanded brushes seem to look better. hopefully we can figure out a way to backport & optimize this code for mapbrush_t, but for now it's only used in Q1 hull expansions 2022-08-16 05:33:38 -04:00
Eric Wasylishen f62136e04d qbsp: wip fixing -expand feature 2022-08-16 01:36:09 -06:00
Eric Wasylishen cad5b3ccf3 testqbsp: add failing hull expansion test 2022-08-16 00:43:40 -06:00
Eric Wasylishen 8ce72efba2 testmaps: add q1_hull_expansion_lip.map
which is getting incorrectly expanded
2022-08-16 00:35:47 -06:00
Eric Wasylishen 9ef3de4dbe bsputil: "--decompile-hull n" for decompiling hulls 2022-08-15 23:18:17 -06:00
Eric Wasylishen f5a8bc7b99 bsputils: add a helper struct for checking clipnode identity 2022-08-15 23:06:00 -06:00
Eric Wasylishen 2cc97c2519 testqbsp: start q1_rocks test case 2022-08-15 22:23:52 -06:00
Eric Wasylishen 83cf63e77f revert using detail on hull1/2 for now as it's breaking tests 2022-08-15 21:17:17 -06:00
Eric Wasylishen 41f765376f testqbsp: adjust test expectations now that we're inside filling hull1/2 2022-08-15 21:16:53 -06:00
Jonathan 92fdff1222 FreeTreePortals call missing
allow decompiling hulls (not hooked up to console yet)
2022-08-15 12:23:09 -04:00
Jonathan 027c45d608 Merge branch 'brushbsp' of https://github.com/ericwa/ericw-tools into brushbsp 2022-08-15 06:13:38 -04:00
Jonathan f7b8f85ece use an optional<uint8_t> to store hull number, which gets rid of special -1 collision hull number.
fixed no-hull always chopping even with chop off
2022-08-15 06:13:30 -04:00