Differential D14 Diff 7886 source/simulation2/helpers/LongPathfinder.h

Changeset View

Standalone View

source/simulation2/helpers/LongPathfinder.h

/* Copyright (C) 2017 Wildfire Games.		/* Copyright (C) 2019 Wildfire Games.
* This file is part of 0 A.D.		* This file is part of 0 A.D.
*		*
* 0 A.D. is free software: you can redistribute it and/or modify		* 0 A.D. is free software: you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by		* it under the terms of the GNU General Public License as published by
* the Free Software Foundation, either version 2 of the License, or		* the Free Software Foundation, either version 2 of the License, or
* (at your option) any later version.		* (at your option) any later version.
*		*
* 0 A.D. is distributed in the hope that it will be useful,		* 0 A.D. is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of		* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the		* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.		* GNU General Public License for more details.
*		*
* You should have received a copy of the GNU General Public License		* You should have received a copy of the GNU General Public License
* along with 0 A.D. If not, see <http://www.gnu.org/licenses/>.		* along with 0 A.D. If not, see <http://www.gnu.org/licenses/>.
*/		*/

#ifndef INCLUDED_LONGPATHFINDER		#ifndef INCLUDED_LONGPATHFINDER
#define INCLUDED_LONGPATHFINDER		#define INCLUDED_LONGPATHFINDER

		#include <mutex>
		vladislavbelovUnsubmitted Not Done Inline Actions Wrong order and empty line. vladislavbelov: Wrong order and empty line.
#include "Pathfinding.h"		#include "Pathfinding.h"
#include "HierarchicalPathfinder.h"		#include "HierarchicalPathfinder.h"

#include "PriorityQueue.h"		#include "PriorityQueue.h"
#include "graphics/Overlay.h"		#include "graphics/Overlay.h"
#include "renderer/Scene.h"		#include "renderer/Scene.h"

/**		/**
▲ Show 20 Lines • Show All 227 Lines • ▼ Show 20 Lines	public:
mutable LongOverlay* m_DebugOverlay;		mutable LongOverlay* m_DebugOverlay;
mutable PathfindTileGrid* m_DebugGrid;		mutable PathfindTileGrid* m_DebugGrid;
mutable u32 m_DebugSteps;		mutable u32 m_DebugSteps;
mutable double m_DebugTime;		mutable double m_DebugTime;
mutable PathGoal m_DebugGoal;		mutable PathGoal m_DebugGoal;
mutable WaypointPath* m_DebugPath;		mutable WaypointPath* m_DebugPath;
mutable pass_class_t m_DebugPassClass;		mutable pass_class_t m_DebugPassClass;

		mutable std::mutex m_DebugMutex;

private:		private:
PathCost CalculateHeuristic(int i, int j, int iGoal, int jGoal) const;		PathCost CalculateHeuristic(int i, int j, int iGoal, int jGoal) const;
void ProcessNeighbour(int pi, int pj, int i, int j, PathCost pg, PathfinderState& state) const;		void ProcessNeighbour(int pi, int pj, int i, int j, PathCost pg, PathfinderState& state) const;

/**		/**
* JPS algorithm helper functions		* JPS algorithm helper functions
* @param detectGoal is not used if m_UseJPSCache is true		* @param detectGoal is not used if m_UseJPSCache is true
*/		*/
Show All 29 Lines	private:
void GenerateSpecialMap(pass_class_t passClass, std::vector<CircularRegion> excludedRegions);		void GenerateSpecialMap(pass_class_t passClass, std::vector<CircularRegion> excludedRegions);

bool m_UseJPSCache;		bool m_UseJPSCache;
// Mutable may be used here as caching does not change the external const-ness of the Long Range pathfinder.		// Mutable may be used here as caching does not change the external const-ness of the Long Range pathfinder.
// This is thread-safe as it is order independent (no change in the output of the function for a given set of params).		// This is thread-safe as it is order independent (no change in the output of the function for a given set of params).
// Obviously, this means that the cache should actually be a cache and not return different results		// Obviously, this means that the cache should actually be a cache and not return different results
// from what would happen if things hadn't been cached.		// from what would happen if things hadn't been cached.
mutable std::map<pass_class_t, shared_ptr<JumpPointCache> > m_JumpPointCache;		mutable std::map<pass_class_t, shared_ptr<JumpPointCache> > m_JumpPointCache;

		kilobugUnsubmitted Not Done Inline Actions Making the cache thread local means that some computation will be done once per thread, instead of once in total. I'm not sure how many of those computations are concerned, and how expensive they are, but with 8 cores some may be done 7x. kilobug: Making the cache thread local means that some computation will be done once per thread, instead…
		StanUnsubmitted Done Inline Actions Same here. Stan: Same here.
		Kuba386Unsubmitted Done Inline Actions Yeah, it's just not trivial to address these problems, I need to get more familiar with both patch's and 0AD's codebase to understand it better. Kuba386: Yeah, it's just not trivial to address these problems, I need to get more familiar with both…
		wraitiiAuthorUnsubmitted Not Done Inline Actions AFAIK the jump point cache is unused, so it should be completely irrelevant. wraitii: AFAIK the jump point cache is unused, so it should be completely irrelevant.
HierarchicalPathfinder m_PathfinderHier;		HierarchicalPathfinder m_PathfinderHier;
};		};

/**		/**
* Terrain overlay for pathfinder debugging.		* Terrain overlay for pathfinder debugging.
* Renders a representation of the most recent pathfinding operation.		* Renders a representation of the most recent pathfinding operation.
*/		*/
class LongOverlay : public TerrainTextureOverlay		class LongOverlay : public TerrainTextureOverlay
▲ Show 20 Lines • Show All 85 Lines • Show Last 20 Lines