Skip to content

Commit 75b2571

Browse files
committed
Revert "TMP: Debug"
This reverts commit 37daf54.
1 parent ae9a344 commit 75b2571

File tree

2 files changed

+4
-42
lines changed

2 files changed

+4
-42
lines changed

include/openPMD/auxiliary/StringManip.hpp

Lines changed: 4 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -195,44 +195,26 @@ namespace auxiliary
195195
return s.substr(begin - s.begin(), end.base() - begin);
196196
}
197197

198-
template <typename T>
199198
inline std::string
200-
join_generic(std::vector<T> const &vs, std::string const &delimiter)
199+
join(std::vector<std::string> const &vs, std::string const &delimiter)
201200
{
202-
auto as_string = [](T const &t) {
203-
if constexpr (std::is_same_v<T, std::string>)
204-
{
205-
return t;
206-
}
207-
else
208-
{
209-
return std::to_string(t);
210-
}
211-
};
212201
switch (vs.size())
213202
{
214203
case 0:
215204
return "";
216205
case 1:
217-
return as_string(vs[0]);
206+
return vs[0];
218207
default:
219208
std::ostringstream ss;
220-
std::transform(
209+
std::copy(
221210
vs.begin(),
222211
vs.end() - 1,
223-
std::ostream_iterator<std::string>(ss, delimiter.c_str()),
224-
as_string);
212+
std::ostream_iterator<std::string>(ss, delimiter.c_str()));
225213
ss << *(vs.end() - 1);
226214
return ss.str();
227215
}
228216
}
229217

230-
inline std::string
231-
join(std::vector<std::string> const &vs, std::string const &delimiter)
232-
{
233-
return join_generic(vs, delimiter);
234-
}
235-
236218
/**
237219
* @brief Remove surrounding slashes from a string.
238220
*

src/ChunkInfo.cpp

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
#include "openPMD/ChunkInfo_internal.hpp"
2323

2424
#include "openPMD/auxiliary/Mpi.hpp"
25-
#include "openPMD/auxiliary/StringManip.hpp"
2625
#include "openPMD/benchmark/mpi/OneDimensionalBlockSlicer.hpp"
2726

2827
#include <algorithm> // std::sort
@@ -410,17 +409,6 @@ namespace chunk_assignment
410409
PartialAssignment ByHostname::assign(
411410
PartialAssignment res, RankMeta const &in, RankMeta const &out)
412411
{
413-
std::cout << "INRANKS:\n";
414-
for (auto const &[rank, name] : in)
415-
{
416-
std::cout << rank << ":\t" << name << '\n';
417-
}
418-
std::cout << "\nOUTRANKS:\n";
419-
for (auto const &[rank, name] : out)
420-
{
421-
std::cout << rank << ":\t" << name << '\n';
422-
}
423-
std::cout << std::endl;
424412
// collect chunks by hostname
425413
std::map<std::string, ChunkTable> chunkGroups;
426414
ChunkTable &sourceChunks = res.notAssigned;
@@ -484,14 +472,6 @@ namespace chunk_assignment
484472
ranksOnTargetNode);
485473
}
486474
}
487-
std::cout << res.notAssigned.size() << " Chunks unassigned:\n";
488-
for (auto const &chunk : res.notAssigned)
489-
{
490-
std::cout << "\tFROM " << chunk.sourceID << "\t["
491-
<< auxiliary::join_generic(chunk.offset, ",") << "]\t["
492-
<< auxiliary::join_generic(chunk.extent, ",") << "]"
493-
<< std::endl;
494-
}
495475
return res;
496476
}
497477

0 commit comments

Comments
 (0)