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

Randomspawn #1013

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Randomspawn #1013

wants to merge 7 commits into from

Conversation

Vaqtincha
Copy link
Contributor

No description provided.

@Vaqtincha
Copy link
Contributor Author

@TheKrytyk
Copy link

Can developers ReGameDLL add this to the official version?

@Vaqtincha
Copy link
Contributor Author

@TheKrytyk unlikely. hopes 0.1%

@TheKrytyk
Copy link

Need to add, but how do I edit the .nav file?

@Vaqtincha
Copy link
Contributor Author

@TheKrytyk why would you edit it??

@SergeyShorokhov SergeyShorokhov added Type: 🚀 enhancement Improvement or addition of a new feature. Priority: 🕒 low Low priority tasks that can be postponed for the future. Difficulty: 🟡 medium Task of medium difficulty requiring moderate effort. Status: 🎮 testing Issue in testing stage. Status: 👀 needs review Requires code or changes review. labels Dec 9, 2024
@TheKrytyk
Copy link

TheKrytyk commented Dec 9, 2024

Because players sometimes spawn in strange places ( e.g., cs_office behind the wall, where there is graffiti on the TT side and they can not leave ).
image
Behind this wall resp

@Vaqtincha
Copy link
Contributor Author

@TheKrytyk Yes, the quality of the points directly depends on the quality of the nav zones.

@TheKrytyk
Copy link

TheKrytyk commented Dec 9, 2024

So how do you edit these .nav files? Finally, add it so that there is XYZ distance from the wall

@Vaqtincha

This comment was marked as outdated.

@Vaqtincha

This comment was marked as outdated.

@Vaqtincha
Copy link
Contributor Author

Vaqtincha commented Dec 10, 2024

@TheKrytyk

Youtube Video
https://youtu.be/9UWBJtZaGYk?si=XaQA7SY08u1eKnod
no nav connections = no resp

i use official .nav file with area names
(upload screen with bot_nav_edit 1 please)

@rehlds rehlds deleted a comment from TheKrytyk Dec 11, 2024
Copy link
Member

@SergeyShorokhov SergeyShorokhov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am thankful for the work to bring this new functionality in! I have a request to add the rest of the ConVars from CS:GO so that this idea can work better as it was intended.

https://github.com/perilouswithadollarsign/cstrike15_src/blob/f82112a2388b841d72cb62ca48ab1846dfcc11c8/game/shared/cstrike15/cs_gamerules.cpp#L16960-L16962

ConVar mp_randomspawn("mp_randomspawn", "0", FCVAR_REPLICATED | FCVAR_RELEASE, "Determines whether players are to spawn. 0 = default; 1 = both teams; 2 = Terrorists; 3 = CTs." );
ConVar mp_randomspawn_los("mp_randomspawn_los", "1", FCVAR_REPLICATED | FCVAR_RELEASE, "If using mp_randomspawn, determines whether to test Line of Sight when spawning." );
ConVar mp_randomspawn_dist( "mp_randomspawn_dist", "0", FCVAR_REPLICATED | FCVAR_RELEASE, "If using mp_randomspawn, determines whether to test distance when selecting this spot." );

And please make mp_randomspawn not a boolean, but 0/1/2/3

@TheKrytyk
Copy link

Ok, well @Vaqtincha are you able to include all the reGameDLL fixes since they won't add it officially?

@sweetcarolinahoney
Copy link

Tested it some time ago and it worked just fine. Especially useful when playing csdm with custom waypoints, like A plant only.
111

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Difficulty: 🟡 medium Task of medium difficulty requiring moderate effort. Priority: 🕒 low Low priority tasks that can be postponed for the future. Status: 👀 needs review Requires code or changes review. Status: 🎮 testing Issue in testing stage. Type: 🚀 enhancement Improvement or addition of a new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants