0
15
0
2
7
2
7
1
11
| ... | ... |
@@ -54,7 +54,6 @@ |
| 54 | 54 |
///This function instantiates a \ref PredMap. |
| 55 | 55 |
///\param g is the digraph, to which we would like to define the |
| 56 | 56 |
///\ref PredMap. |
| 57 |
///\todo The digraph alone may be insufficient to initialize |
|
| 58 | 57 |
static PredMap *createPredMap(const Digraph &g) |
| 59 | 58 |
{
|
| 60 | 59 |
return new PredMap(g); |
| ... | ... |
@@ -64,7 +63,6 @@ |
| 64 | 63 |
|
| 65 | 64 |
///The type of the map that indicates which nodes are processed. |
| 66 | 65 |
///It must meet the \ref concepts::WriteMap "WriteMap" concept. |
| 67 |
///By default it is a NullMap. |
|
| 68 | 66 |
typedef NullMap<typename Digraph::Node,bool> ProcessedMap; |
| 69 | 67 |
///Instantiates a \ref ProcessedMap. |
| 70 | 68 |
|
| ... | ... |
@@ -196,8 +194,7 @@ |
| 196 | 194 |
int _queue_head,_queue_tail,_queue_next_dist; |
| 197 | 195 |
int _curr_dist; |
| 198 | 196 |
|
| 199 |
///Creates the maps if necessary. |
|
| 200 |
///\todo Better memory allocation (instead of new). |
|
| 197 |
//Creates the maps if necessary. |
|
| 201 | 198 |
void create_maps() |
| 202 | 199 |
{
|
| 203 | 200 |
if(!_pred) {
|
| ... | ... |
@@ -848,7 +845,6 @@ |
| 848 | 845 |
///This function instantiates a \ref PredMap. |
| 849 | 846 |
///\param g is the digraph, to which we would like to define the |
| 850 | 847 |
///\ref PredMap. |
| 851 |
///\todo The digraph alone may be insufficient to initialize |
|
| 852 | 848 |
static PredMap *createPredMap(const Digraph &g) |
| 853 | 849 |
{
|
| 854 | 850 |
return new PredMap(g); |
| ... | ... |
@@ -1370,8 +1366,7 @@ |
| 1370 | 1366 |
std::vector<typename Digraph::Node> _list; |
| 1371 | 1367 |
int _list_front, _list_back; |
| 1372 | 1368 |
|
| 1373 |
///Creates the maps if necessary. |
|
| 1374 |
///\todo Better memory allocation (instead of new). |
|
| 1369 |
//Creates the maps if necessary. |
|
| 1375 | 1370 |
void create_maps() {
|
| 1376 | 1371 |
if(!_reached) {
|
| 1377 | 1372 |
local_reached = true; |
| ... | ... |
@@ -105,9 +105,6 @@ |
| 105 | 105 |
|
| 106 | 106 |
/// Returns whether the given directed arc has the same orientation |
| 107 | 107 |
/// as the corresponding edge. |
| 108 |
/// |
|
| 109 |
/// \todo reference to the corresponding point of the undirected digraph |
|
| 110 |
/// concept. "What does the direction of an edge mean?" |
|
| 111 | 108 |
static bool direction(const Arc &a) { return a.forward; }
|
| 112 | 109 |
|
| 113 | 110 |
using Parent::first; |
| ... | ... |
@@ -42,10 +42,9 @@ |
| 42 | 42 |
/// automatically updates the map when a key is added to or erased from |
| 43 | 43 |
/// the map. This map type uses the std::vector to store the values. |
| 44 | 44 |
/// |
| 45 |
/// \tparam |
|
| 45 |
/// \tparam _Graph The graph this map is attached to. |
|
| 46 | 46 |
/// \tparam _Item The item type of the graph items. |
| 47 | 47 |
/// \tparam _Value The value type of the map. |
| 48 |
/// \todo Fix the doc: there is _Graph parameter instead of _Notifier. |
|
| 49 | 48 |
template <typename _Graph, typename _Item, typename _Value> |
| 50 | 49 |
class VectorMap |
| 51 | 50 |
: public ItemSetTraits<_Graph, _Item>::ItemNotifier::ObserverBase {
|
| ... | ... |
@@ -55,7 +55,6 @@ |
| 55 | 55 |
///This function instantiates a \ref PredMap. |
| 56 | 56 |
///\param g is the digraph, to which we would like to define the |
| 57 | 57 |
///\ref PredMap. |
| 58 |
///\todo The digraph alone may be insufficient to initialize |
|
| 59 | 58 |
static PredMap *createPredMap(const Digraph &g) |
| 60 | 59 |
{
|
| 61 | 60 |
return new PredMap(g); |
| ... | ... |
@@ -65,7 +64,6 @@ |
| 65 | 64 |
|
| 66 | 65 |
///The type of the map that indicates which nodes are processed. |
| 67 | 66 |
///It must meet the \ref concepts::WriteMap "WriteMap" concept. |
| 68 |
///By default it is a NullMap. |
|
| 69 | 67 |
typedef NullMap<typename Digraph::Node,bool> ProcessedMap; |
| 70 | 68 |
///Instantiates a \ref ProcessedMap. |
| 71 | 69 |
|
| ... | ... |
@@ -196,8 +194,7 @@ |
| 196 | 194 |
std::vector<typename Digraph::OutArcIt> _stack; |
| 197 | 195 |
int _stack_head; |
| 198 | 196 |
|
| 199 |
///Creates the maps if necessary. |
|
| 200 |
///\todo Better memory allocation (instead of new). |
|
| 197 |
//Creates the maps if necessary. |
|
| 201 | 198 |
void create_maps() |
| 202 | 199 |
{
|
| 203 | 200 |
if(!_pred) {
|
| ... | ... |
@@ -782,7 +779,6 @@ |
| 782 | 779 |
///This function instantiates a \ref PredMap. |
| 783 | 780 |
///\param g is the digraph, to which we would like to define the |
| 784 | 781 |
///\ref PredMap. |
| 785 |
///\todo The digraph alone may be insufficient to initialize |
|
| 786 | 782 |
static PredMap *createPredMap(const Digraph &g) |
| 787 | 783 |
{
|
| 788 | 784 |
return new PredMap(g); |
| ... | ... |
@@ -1317,8 +1313,7 @@ |
| 1317 | 1313 |
std::vector<typename Digraph::Arc> _stack; |
| 1318 | 1314 |
int _stack_head; |
| 1319 | 1315 |
|
| 1320 |
///Creates the maps if necessary. |
|
| 1321 |
///\todo Better memory allocation (instead of new). |
|
| 1316 |
//Creates the maps if necessary. |
|
| 1322 | 1317 |
void create_maps() {
|
| 1323 | 1318 |
if(!_reached) {
|
| 1324 | 1319 |
local_reached = true; |
| ... | ... |
@@ -144,7 +144,6 @@ |
| 144 | 144 |
///This function instantiates a \ref PredMap. |
| 145 | 145 |
///\param g is the digraph, to which we would like to define the |
| 146 | 146 |
///\ref PredMap. |
| 147 |
///\todo The digraph alone may be insufficient for the initialization |
|
| 148 | 147 |
static PredMap *createPredMap(const Digraph &g) |
| 149 | 148 |
{
|
| 150 | 149 |
return new PredMap(g); |
| ... | ... |
@@ -155,8 +154,6 @@ |
| 155 | 154 |
///The type of the map that indicates which nodes are processed. |
| 156 | 155 |
///It must meet the \ref concepts::WriteMap "WriteMap" concept. |
| 157 | 156 |
///By default it is a NullMap. |
| 158 |
///\todo If it is set to a real map, |
|
| 159 |
///Dijkstra::processed() should read this. |
|
| 160 | 157 |
typedef NullMap<typename Digraph::Node,bool> ProcessedMap; |
| 161 | 158 |
///Instantiates a \ref ProcessedMap. |
| 162 | 159 |
|
| ... | ... |
@@ -297,8 +294,7 @@ |
| 297 | 294 |
//Indicates if _heap is locally allocated (true) or not. |
| 298 | 295 |
bool local_heap; |
| 299 | 296 |
|
| 300 |
///Creates the maps if necessary. |
|
| 301 |
///\todo Better memory allocation (instead of new). |
|
| 297 |
//Creates the maps if necessary. |
|
| 302 | 298 |
void create_maps() |
| 303 | 299 |
{
|
| 304 | 300 |
if(!_pred) {
|
| ... | ... |
@@ -958,7 +954,6 @@ |
| 958 | 954 |
///This function instantiates a \ref HeapCrossRef. |
| 959 | 955 |
/// \param g is the digraph, to which we would like to define the |
| 960 | 956 |
/// HeapCrossRef. |
| 961 |
/// \todo The digraph alone may be insufficient for the initialization |
|
| 962 | 957 |
static HeapCrossRef *createHeapCrossRef(const Digraph &g) |
| 963 | 958 |
{
|
| 964 | 959 |
return new HeapCrossRef(g); |
| ... | ... |
@@ -994,7 +989,6 @@ |
| 994 | 989 |
///This function instantiates a \ref PredMap. |
| 995 | 990 |
///\param g is the digraph, to which we would like to define the |
| 996 | 991 |
///\ref PredMap. |
| 997 |
///\todo The digraph alone may be insufficient to initialize |
|
| 998 | 992 |
static PredMap *createPredMap(const Digraph &g) |
| 999 | 993 |
{
|
| 1000 | 994 |
return new PredMap(g); |
| ... | ... |
@@ -1005,9 +999,6 @@ |
| 1005 | 999 |
///The type of the map that indicates which nodes are processed. |
| 1006 | 1000 |
///It must meet the \ref concepts::WriteMap "WriteMap" concept. |
| 1007 | 1001 |
///By default it is a NullMap. |
| 1008 |
///\todo If it is set to a real map, |
|
| 1009 |
///Dijkstra::processed() should read this. |
|
| 1010 |
///\todo named parameter to set this type, function to read and write. |
|
| 1011 | 1002 |
typedef NullMap<typename Digraph::Node,bool> ProcessedMap; |
| 1012 | 1003 |
///Instantiates a \ref ProcessedMap. |
| 1013 | 1004 |
|
| ... | ... |
@@ -1053,7 +1044,6 @@ |
| 1053 | 1044 |
/// as well as the \ref Dijkstra class. |
| 1054 | 1045 |
/// The \ref DijkstraWizardBase is a class to be the default traits of the |
| 1055 | 1046 |
/// \ref DijkstraWizard class. |
| 1056 |
/// \todo More named parameters are required... |
|
| 1057 | 1047 |
template<class GR,class LM> |
| 1058 | 1048 |
class DijkstraWizardBase : public DijkstraWizardDefaultTraits<GR,LM> |
| 1059 | 1049 |
{
|
| ... | ... |
@@ -666,7 +666,6 @@ |
| 666 | 666 |
///this function calls the algorithm itself, i.e. in this case |
| 667 | 667 |
///it draws the graph. |
| 668 | 668 |
void run() {
|
| 669 |
//\todo better 'epsilon' would be nice here. |
|
| 670 | 669 |
const double EPSILON=1e-9; |
| 671 | 670 |
if(dontPrint) return; |
| 672 | 671 |
|
| ... | ... |
@@ -707,7 +706,6 @@ |
| 707 | 706 |
double max_w=0; |
| 708 | 707 |
for(ArcIt e(g);e!=INVALID;++e) |
| 709 | 708 |
max_w=std::max(double(_arcWidths[e]),max_w); |
| 710 |
//\todo better 'epsilon' would be nice here. |
|
| 711 | 709 |
if(max_w>EPSILON) {
|
| 712 | 710 |
_arcWidthScale/=max_w; |
| 713 | 711 |
} |
| ... | ... |
@@ -717,7 +715,6 @@ |
| 717 | 715 |
double max_s=0; |
| 718 | 716 |
for(NodeIt n(g);n!=INVALID;++n) |
| 719 | 717 |
max_s=std::max(double(_nodeSizes[n]),max_s); |
| 720 |
//\todo better 'epsilon' would be nice here. |
|
| 721 | 718 |
if(max_s>EPSILON) {
|
| 722 | 719 |
_nodeScale/=max_s; |
| 723 | 720 |
} |
| ... | ... |
@@ -873,7 +870,6 @@ |
| 873 | 870 |
<< -bb.left() << ' ' << -bb.bottom() << " translate\n"; |
| 874 | 871 |
} |
| 875 | 872 |
else {
|
| 876 |
//\todo Verify centering |
|
| 877 | 873 |
double sc= std::min((A4HEIGHT-2*A4BORDER)/bb.width(), |
| 878 | 874 |
(A4WIDTH-2*A4BORDER)/bb.height()); |
| 879 | 875 |
os << ((A4WIDTH -2*A4BORDER)-sc*bb.height())/2 + A4BORDER << ' ' |
| ... | ... |
@@ -906,7 +902,6 @@ |
| 906 | 902 |
dim2::Point<double> |
| 907 | 903 |
dvec(mycoords[g.target(*i)]-mycoords[g.source(*i)]); |
| 908 | 904 |
double l=std::sqrt(dvec.normSquare()); |
| 909 |
//\todo better 'epsilon' would be nice here. |
|
| 910 | 905 |
dim2::Point<double> d(dvec/std::max(l,EPSILON)); |
| 911 | 906 |
dim2::Point<double> m; |
| 912 | 907 |
// m=dim2::Point<double>(mycoords[g.target(*i)]+ |
| ... | ... |
@@ -501,10 +501,8 @@ |
| 501 | 501 |
///valid. However <tt>InArcIt</tt>s and <tt>OutArcIt</tt>s may |
| 502 | 502 |
///be invalidated. |
| 503 | 503 |
/// |
| 504 |
///\warning This functionality cannot be used |
|
| 504 |
///\warning This functionality cannot be used in conjunction with the |
|
| 505 | 505 |
///Snapshot feature. |
| 506 |
/// |
|
| 507 |
///\todo It could be implemented in a bit faster way. |
|
| 508 | 506 |
Node split(Node n, bool connect = true) {
|
| 509 | 507 |
Node b = addNode(); |
| 510 | 508 |
for(OutArcIt e(*this,n);e!=INVALID;) {
|
| ... | ... |
@@ -484,8 +484,6 @@ |
| 484 | 484 |
/// function. |
| 485 | 485 |
/// |
| 486 | 486 |
/// \sa CombineMap |
| 487 |
/// |
|
| 488 |
/// \todo Check the requirements. |
|
| 489 | 487 |
template <typename M1, typename M2> |
| 490 | 488 |
class ComposeMap : public MapBase<typename M2::Key, typename M1::Value> {
|
| 491 | 489 |
const M1 &_m1; |
| ... | ... |
@@ -540,8 +538,6 @@ |
| 540 | 538 |
/// function. |
| 541 | 539 |
/// |
| 542 | 540 |
/// \sa ComposeMap |
| 543 |
/// |
|
| 544 |
/// \todo Check the requirements. |
|
| 545 | 541 |
template<typename M1, typename M2, typename F, |
| 546 | 542 |
typename V = typename F::result_type> |
| 547 | 543 |
class CombineMap : public MapBase<typename M1::Key, V> {
|
| ... | ... |
@@ -821,7 +821,6 @@ |
| 821 | 821 |
/// Standard Gauss distribution. |
| 822 | 822 |
/// \note The Cartesian form of the Box-Muller |
| 823 | 823 |
/// transformation is used to generate a random normal distribution. |
| 824 |
/// \todo Consider using the "ziggurat" method instead. |
|
| 825 | 824 |
double gauss() |
| 826 | 825 |
{
|
| 827 | 826 |
double V1,V2,S; |
| ... | ... |
@@ -300,7 +300,6 @@ |
| 300 | 300 |
///may be invalidated. |
| 301 | 301 |
///\warning This functionality cannot be used together with the Snapshot |
| 302 | 302 |
///feature. |
| 303 |
///\todo It could be implemented in a bit faster way. |
|
| 304 | 303 |
Node split(Node n, bool connect = true) |
| 305 | 304 |
{
|
| 306 | 305 |
Node b = addNode(); |
| ... | ... |
@@ -292,7 +292,6 @@ |
| 292 | 292 |
///\note If you want to measure the running time of the execution of a certain |
| 293 | 293 |
///function, consider the usage of \ref TimeReport instead. |
| 294 | 294 |
/// |
| 295 |
///\todo This shouldn't be Unix (Linux) specific. |
|
| 296 | 295 |
///\sa TimeReport |
| 297 | 296 |
class Timer |
| 298 | 297 |
{
|
| ... | ... |
@@ -487,7 +486,6 @@ |
| 487 | 486 |
/// |
| 488 | 487 |
///\sa Timer |
| 489 | 488 |
///\sa NoTimeReport |
| 490 |
///\todo There is no test case for this |
|
| 491 | 489 |
class TimeReport : public Timer |
| 492 | 490 |
{
|
| 493 | 491 |
std::string _title; |
0 comments (0 inline)