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

add link script for powershell #628

Open
wants to merge 42 commits into
base: changes
Choose a base branch
from

Conversation

getchoo
Copy link
Contributor

@getchoo getchoo commented Feb 17, 2023

Related Issue

n/a

Implementation

this a powershell implementation of link.sh. it allows for an easier development workflow by automatically copying and linking required files from retail TF2 and game_clean, while only needing one parameter (-TF2Dir).

Screenshots

image

Checklist

  • No other PRs implement this idea.
  • This PR does not introduce any regressions.
  • I certify that this PR is my own entirely original work, or I have permission from and have attributed the relevant authors.
  • I have agreed to and signed this project's Contributor License Agreement.
  • This PR only contains changes to the engine and/or core game framework
  • This PR targets the changes branch.
  • This change has been filed as an issue.
  • No other PRs address this.
  • This PR is as minimal as possible.
  • This PR does not introduce any regressions.
  • This PR has been built and locally tested on at least one platform.
  • This PR has been tested on all platforms, on both a dedicated server and on a listen server.

Testing Checklist

Client Server Version
Windows tested tested windows 11
Linux n/a n/a n/a
Mac OS n/a n/a n/a

Caveats

running tc2 after compiling is easier now /s
this requires administrator privileges due to limitations of windows

Alternatives

using git bash with link.sh :trollface:

@mastercoms mastercoms force-pushed the changes branch 11 times, most recently from 801dbb1 to a922702 Compare March 7, 2023 13:19
@getchoo
Copy link
Contributor Author

getchoo commented Mar 10, 2023

rebased onto the new commits from changes and implemented some of the changes suggested in #629

the game_clean and output directories are also nice constants now :)

@mastercoms mastercoms force-pushed the changes branch 11 times, most recently from 6c86864 to b216e17 Compare March 13, 2023 17:05
@mastercoms mastercoms force-pushed the changes branch 2 times, most recently from f8a351c to 9532d33 Compare March 18, 2023 19:03
collision mode COLLISION_MODE_PER_FRAME_PLANESET was running traces
for the next mode due to a missing break, which decreased performance.

these traces are expensive as seen by profiling
allows for much more efficient math optimizations
will be used for optimized SSE routines. I chose this library
because it has a lineage from what Source uses for Xbox 360 math:
xboxmath -> xnamath -> DirectXMath, so porting SIMD operations to PC
is simplified, and there are similar guarantees.

can be retrieved from https://github.com/microsoft/DirectXMath

A blank sal.h also needs to be retrieved from https://github.com/microsoft/omi/blob/master/Unix/common/linux/sal.h
for POSIX (microsoft/DirectXMath#89 (comment))

an edit needs to be made for Inc/DirectXMath.h to include sal.h on Windows
and the above sal.h on POSIX.
with vectorization, more math optimizations, and alignment,
the compiler is more sensitive to some forms of code bugs
mastercoms and others added 2 commits March 22, 2023 18:03
instead of doing a O(n) lookup per frame in some cases, we can just
init the ConVarRef once

hottest function in R_LoadSkys, other ones are just possible during runtime
or called relatively a lot compared to other non-static ConVarRefs
@mastercoms mastercoms force-pushed the changes branch 15 times, most recently from 690e0f5 to 5f4b9e2 Compare April 11, 2023 05:39
@mastercoms mastercoms force-pushed the changes branch 7 times, most recently from aaa6dc4 to 05457f2 Compare April 22, 2023 13:59
@mastercoms mastercoms force-pushed the changes branch 2 times, most recently from d1cf018 to 8963afe Compare April 30, 2023 04:47
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

Successfully merging this pull request may close these issues.

2 participants