branchrust_reboot/main/naval_update/naval_missionscancel
155 Commits over 153 Days - 0.04cph!
Revert change to MissionPoint type position generator for now due to error
Fix item already added exception in request missions update
Suggested simplifications for performance capture
Cleanup bits from last commit
Add 'profileNextMissionsValidStateWorkQueue' servervar to generate a performance capture containing the next run of the missions work queue if profling is enabled
- Optimise WorldPositionGenerator, this was the most expensive job in the new work queue and is now showing as at least 30x faster at in testing
- Optimise MissionPoint type position generator so it no longer always iterates through all mission points on the server
- Now check when we last revalidated each mission before adding it to a run of the work queue rather than just checking time between runs of the work queue itself
- Move work queue run cooldown to a servervar 'missionValidStateWorkQueueCooldown', and added 'missionPerValidStateCooldown' to configure cooldown between revalidating individual missions
- Add servervars 'resetmissions' and 'resetmissions_alll' to reset missions progress for a target player or all players on a server. Useful if saved missions progress is borked without having to reset entire player state
- Player now requests a mission valid states update when they join a server, so map markers will be ready for when they first open their map
Add some extra guards and server error logs if saved missions objective count is wrong
Remove client error logs regarding mission objectives count being wrong
Will still get logged on the server if it's still a problem. If it's wrong on either the client or server, it'll be wrong on both
Fix pool leak from generating missions protobuf data
- Mission to kill the deep sea patrol boat scientists now also counts PT boat scientists
- Add an option for kills must be in a certain biome to kill objectives
- Disabled positions are exlusive flag from last tutorial mission, was causing it to fail to find a valid position
- Enable "drops loot" on deep sea mission treasure chest so that it actually drops its loot
- Treasure chest spawned in deep sea as part of mission no longer destroys loot as it now no longer has troll loot
- Update mission provider dialogue so it makes better sense now that you pick up something decent
- Make the treasure chest it's own prefab rather than a variant
Fix some weirdness with how missions were first getting loaded from player state on player entered game
Expand log if loaded objective count data is wrong
Fix issues with clearing missions when resetting player state
Fix error when speaking to a NPC to complete a mission if that mission has no rewards
General cleanup of associated code
Reduce default mission valid states work queue budget from 0.3ms to 0.1ms
Add some logs to help figure out why it's possible for an index to be out of bounds when accessing mission objectives
▄ ▊▅▄█▆ ██▋▅▋ █▋▊▅▌▋ ▊█▋▊█▍ ▆▅▅█▉▄▇ ▆▆▋▋ ▇▊▇▌▌▅▇▋ ▋▊▆▄ ▇▇▉█▋ ▌▆█▅ ▊▋▍ ▍▇▌ ▍▇▇ ▌█▋ ▍▆▉█▌▋▆ █▍▊▇▄ ▄▌▄ ▍▉▊ █▄▆▊▋▍█▌▉▇▇▇█▋▋▆▇▄ ▍▄ ▍▄▍▍ ▅▄▇ ▌▌▉▋▍▌▋██▉▇▉ ▊▍▄ ▅▍▅▉ ▅▉ ▇▌█▍
Merge from naval_missions
Fix potential key has already been added exception in Server_RequestValidMissionsUpdate
Also check the length of the static work queue before adding players to the collection of players requesting an update, instead of checking IsInvoking which checks an action only on that player
Codegen
Increase max distance on tutorial mission 8 position generator
- Always check if a player can accept the mission before assigning it
- MissionInstance now saves the provider as type IMissionProvider rather than BaseEntity
- Fixes for registering tutorial pings on SpeakWith objective type
- Don't try to save the dialogue graph if in play mode
Remove garbage allocations from BaseMission.id property
Codegen
- Don't process which missions are available for a player if they have an active mission, or if they are talking to an NPC to complete a mission
- Request a missions update if the deep sea opens/closes whilst our map is open, not only when the deep sea opens
- Other minor fixes
Requests for what missions are valid on the server are now run on a work queue
- Fix some issues with position generators and exclusive mission points
- Fix issues relating to player state missions data and resetting player state
Optimisation for EnvironmentManager.Check
- No longer converts the physics overlap collider buffer to a list
- Returns true as soon as a valid environment volume collider is found rather than always iterating through all colliders
Commit progress on mission "can accept" improvements
- Split up work between client and server. Server checks if missions are valid, this result is the same for all players and does not need to be regenerated at the request of each player. Clients use this result + checking themselves if they could accept this mission for things like mission provider map marker visibility.
- Removed "relative to" position generator type "player" as this no longer works for generating positions in a player agnostic way before missions have been accepted. Replaced each case where this was used in mission position generators, all missions should still be functional as before using alternative position types.
- Modify blocked points so that they work properly with positions being generated in advance of missions being accepted.
- Misc minor mission related optimisations.
TLDR: Restoring mission provider map marker functionality whilst also fixing problems of these being misleading due to discrepancies between the client and server for if missions can be accepted.
Fixes for battery hunt mission
Remove repeated allocations from getting missions from the mission manifest
Dialogue adjustment for Floating Citizen
Dialogue fixes for Lumberjack and Floating Citizen
Remove time limit from boat scientist hunt mission
Fix
138975 2.0
Fix leaky lists on the server relating to missions
