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

Core: Added way to alternate paths - PathSettings #582

Merged
merged 1 commit into from
Apr 21, 2024
Merged

Conversation

Xian55
Copy link
Owner

@Xian55 Xian55 commented Apr 21, 2024

during runtime based on the given Requirements list.

Many requested this feature, #432, #459, #577

Most of the path related settings move to a new class called PathSettings.

I kept the old path related settings in ClassConfiguration for backward compatibility. However from those old values a new PathSettings is created to ensure both ways are supported.

At this stage there are not many Requirement to choose from. So added the followings

  • Seconds
  • Level
  • ExpPerc

For more details please read the Readme file, starting from the Path section.

Note: The Path Editor only works with the first ClassConfiguration.Paths for now.

Media:

Frontend Changes:

Path Selector Changes
For each Path there's a corresponding Path Selector component to override the base path. In this instance there are 3 paths.
image

GoapGoalView Changes
For each path there's a corresponding FollowRouteGoal where the Requirements are visible
image

Class Configuration Tab Changes
For each path there's a corresponding PathSettings, mostly to change PathThereAndBack and PathReduceSteps
image

Raw Values Tab Changes
Session Stats added.
image

Route Component Changes
It shows the last actively used Path, also updates the symbol which is driven by the PathSettings.PathThereAndBack
image

Bug Introduced:

…ng runtime based on the given Requirements list. Most of the path related settings move to a new class called PathSettings. I kept the old path related settings in ClassConfiguration for backward compatibility. However from those old values a new PathSettings is created to ensure both ways are supported. At this stage there are not many Requirement to choose from.
@Xian55 Xian55 added documentation Improvements or additions to documentation refactor This ticket concerns the possible simplification of code/data. enhancement This pull request implements a new feature. labels Apr 21, 2024
@Xian55 Xian55 linked an issue Apr 21, 2024 that may be closed by this pull request
@Xian55 Xian55 merged commit 4d6b6f1 into dev Apr 21, 2024
1 check passed
@Xian55 Xian55 deleted the feature/459 branch April 21, 2024 19:50
@Xian55 Xian55 changed the title Core: Added quick and dirty way to change between multiple paths Core: Added way to change between alternate paths Apr 21, 2024
@Xian55 Xian55 changed the title Core: Added way to change between alternate paths Core: Added way to alternate paths Apr 21, 2024
@Xian55 Xian55 changed the title Core: Added way to alternate paths Core: Added way to alternate paths - PathSettings Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement This pull request implements a new feature. refactor This ticket concerns the possible simplification of code/data.
Projects
None yet
1 participant