Address PR comments
This commit is contained in:
parent
b830a8f942
commit
4929d1297e
@ -66,8 +66,6 @@ class Contractor
|
||||
std::vector<bool> &is_core_node,
|
||||
std::vector<float> &inout_node_levels) const;
|
||||
void WriteCoreNodeMarker(std::vector<bool> &&is_core_node) const;
|
||||
void WriteNodeLevels(std::vector<float> &&node_levels) const;
|
||||
void ReadNodeLevels(std::vector<float> &contraction_order) const;
|
||||
void WriteContractedGraph(unsigned number_of_edge_based_nodes,
|
||||
util::DeallocatingVector<QueryEdge> contracted_edge_list);
|
||||
void FindComponents(unsigned max_edge_id,
|
||||
|
@ -185,7 +185,7 @@ bool Engine<routing_algorithms::corech::Algorithm>::CheckCompability(const Engin
|
||||
|
||||
auto mem = storage::makeSharedMemory(barrier.data().region);
|
||||
auto layout = reinterpret_cast<storage::DataLayout *>(mem->Ptr());
|
||||
return layout->GetBlockSize(storage::DataLayout::CH_CORE_MARKER) > 16;
|
||||
return layout->GetBlockSize(storage::DataLayout::CH_CORE_MARKER) > sizeof(std::uint64_t) + sizeof(util::FingerPrint);
|
||||
}
|
||||
else
|
||||
{
|
||||
|
@ -67,7 +67,7 @@ int Contractor::Run()
|
||||
std::vector<float> node_levels;
|
||||
if (config.use_cached_priority)
|
||||
{
|
||||
ReadNodeLevels(node_levels);
|
||||
files::readLevels(config.level_output_path, node_levels);
|
||||
}
|
||||
|
||||
util::DeallocatingVector<QueryEdge> contracted_edge_list;
|
||||
@ -89,7 +89,9 @@ int Contractor::Run()
|
||||
WriteCoreNodeMarker(std::move(is_core_node));
|
||||
if (!config.use_cached_priority)
|
||||
{
|
||||
WriteNodeLevels(std::move(node_levels));
|
||||
std::vector<float> out_node_levels(std::move(node_levels));
|
||||
|
||||
files::writeLevels(config.level_output_path, node_levels);
|
||||
}
|
||||
|
||||
TIMER_STOP(preparing);
|
||||
@ -101,18 +103,6 @@ int Contractor::Run()
|
||||
return 0;
|
||||
}
|
||||
|
||||
void Contractor::ReadNodeLevels(std::vector<float> &node_levels) const
|
||||
{
|
||||
files::readLevels(config.level_output_path, node_levels);
|
||||
}
|
||||
|
||||
void Contractor::WriteNodeLevels(std::vector<float> &&in_node_levels) const
|
||||
{
|
||||
std::vector<float> node_levels(std::move(in_node_levels));
|
||||
|
||||
files::writeLevels(config.level_output_path, node_levels);
|
||||
}
|
||||
|
||||
void Contractor::WriteCoreNodeMarker(std::vector<bool> &&in_is_core_node) const
|
||||
{
|
||||
std::vector<bool> is_core_node(std::move(in_is_core_node));
|
||||
|
@ -121,7 +121,7 @@ auto mmapFile(const std::string &filename, boost::interprocess::mode_t mode)
|
||||
const file_mapping mapping{filename.c_str(), mode};
|
||||
|
||||
// map region started at an offset of util::FingerPrint size
|
||||
mapped_region region{mapping, mode, 8};
|
||||
mapped_region region{mapping, mode, sizeof(util::FingerPrint)};
|
||||
region.advise(mapped_region::advice_sequential);
|
||||
return region;
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user