Skip to content

proof of concept dynamic pitch angle#11

Draft
halvorgb wants to merge 6 commits intoJelvan1:masterfrom
halvorgb:dynamic-plasma
Draft

proof of concept dynamic pitch angle#11
halvorgb wants to merge 6 commits intoJelvan1:masterfrom
halvorgb:dynamic-plasma

Conversation

@halvorgb
Copy link

@halvorgb halvorgb commented Feb 6, 2022

This enables a dynamic optimal pitch angle for plasma climbing, with some new cvars. This picture has been the reference for these angles:

image

The code and cvars refer to three max-angles: max, submax and badmax.
In the reference picture we can see that the max climb height varies within the range.
submax defaults to orange and is almost as good as max.
max defaults to green.
badmax defaults to red. From the picture it is the 73.4-74.0 -> 1112u one. It is the best angle in those circumstances, but should be avoided if possible (by choosing another wall to climb on).

Video demonstration:
https://streamable.com/fr6eaw

I don't know if the reference picture is correct, I have no reason to doubt it though.

Also, I'm not a C programmer really so you'll probably want to rewrite this if you want it in your project. 😃

seperated color variables into max, submax and badmax.

badmax means that this quadrant/face is just a bad combination and
plasma climbs are affected.

submax is almost as good as the optimal angle.

max is the optimal angle, and different to badmax it is the best we can
do in quake.

Still some gaps between the submax and max pitch angles.
@halvorgb
Copy link
Author

halvorgb commented Feb 9, 2022

Unsure why this fails on macOS 64-bit, the error doesn't give any useful information. Some sort of timeout after 50 minutes?

@Jelvan1
Copy link
Owner

Jelvan1 commented Feb 9, 2022

Thanks for the proof of concept, will have a look at it later.

About the build error, I've seen the same thing happen on Linux and Windows a few months ago, but I could never replicate this on my system. Now it's macOS for some reason (see #10 ). You can ignore this for the moment.

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