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

Remove agent avoidance page #7422

Merged
merged 1 commit into from
May 26, 2023

Conversation

smix8
Copy link
Contributor

@smix8 smix8 commented May 26, 2023

Removes agent avoidance page.

While reworking the pages for the avoidance rework pr I realized that this page now has mostly old or even wrong information, duplicated information from other pages, or information that should be part directly of other pages like the navigation agent or navigation obstacles pages. With all that change the page would be left empty.

I originally had plans to have a page about the NavigationAvoidance as a system. The current agent avoidance page was made at a point where the old avoidance was hard limited on just agents but now we have working obstacles and all kinds of other avoidance features. It does not make sense with the current page name and url so I think it is better to just do some house cleaning and remove it.

Removes agent avoidance page.
@smix8 smix8 force-pushed the remove_agent_avoidance_4.x branch from a609106 to faeb8eb Compare May 26, 2023 10:08
@mhilbrunner mhilbrunner added enhancement content:organization Issues and PRs related to reorganizing the content area:manual Issues and PRs related to the Manual/Tutorials section of the documentation labels May 26, 2023
@mhilbrunner
Copy link
Member

I'm taking from the branch name that this should not be cherrypicked for 4.0?

@YuriSizov
Copy link
Contributor

Well, the rework is only 4.1+, so 4.0 docs should remain as is. We can perhaps add a note if there are some issues with the article that we are not planning to fix.

@smix8
Copy link
Contributor Author

smix8 commented May 26, 2023

This should only be for 4.1.
The old 4.0 version is correct for that version. The old rvo avoidance had many problems that is why that old page also focused so much on those. They are fixed in 4.1 or mentioned in better places where they make contextual more sense.

@YuriSizov YuriSizov added this to the 4.1 milestone May 26, 2023
@mhilbrunner mhilbrunner merged commit b7b25ba into godotengine:master May 26, 2023
@mhilbrunner
Copy link
Member

Thanks! Merged.

@smix8 smix8 deleted the remove_agent_avoidance_4.x branch May 26, 2023 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:manual Issues and PRs related to the Manual/Tutorials section of the documentation content:organization Issues and PRs related to reorganizing the content enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants