Skip to content

Commit cbb9f77

Browse files
shahor02chiarazampolli
shahor02
authored and
chiarazampolli
committed
Decrease verbosity of CTF encoding/decoding
(cherry picked from commit 79ca607)
1 parent 2c2a532 commit cbb9f77

File tree

7 files changed

+20
-20
lines changed

7 files changed

+20
-20
lines changed

‎DataFormats/Detectors/Common/include/DetectorsCommonDataFormats/EncodedBlocks.h‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1221,7 +1221,7 @@ o2::ctf::CTFIOSize EncodedBlocks<H, N, W>::entropyCodeRANSCompat(const input_IT
12211221

12221222
if (!view.empty()) {
12231223
thisBlock->storeDict(view.size(), view.data());
1224-
LOGP(info, "StoreDict {} bytes, offs: {}:{}", view.size() * sizeof(W), thisBlock->getOffsDict(), thisBlock->getOffsDict() + view.size() * sizeof(W));
1224+
LOGP(debug, "StoreDict {} bytes, offs: {}:{}", view.size() * sizeof(W), thisBlock->getOffsDict(), thisBlock->getOffsDict() + view.size() * sizeof(W));
12251225
}
12261226
// vector of incompressible literal symbols
12271227
std::vector<input_t> literals;
@@ -1233,7 +1233,7 @@ o2::ctf::CTFIOSize EncodedBlocks<H, N, W>::entropyCodeRANSCompat(const input_IT
12331233
dataSize = encodedMessageEnd - thisBlock->getDataPointer();
12341234
thisBlock->setNData(dataSize);
12351235
thisBlock->realignBlock();
1236-
LOGP(info, "StoreData {} bytes, offs: {}:{}", dataSize * sizeof(W), thisBlock->getOffsData(), thisBlock->getOffsData() + dataSize * sizeof(W));
1236+
LOGP(debug, "StoreData {} bytes, offs: {}:{}", dataSize * sizeof(W), thisBlock->getOffsData(), thisBlock->getOffsData() + dataSize * sizeof(W));
12371237
// update the size claimed by encode message directly inside the block
12381238

12391239
// store incompressible symbols if any
@@ -1248,13 +1248,13 @@ o2::ctf::CTFIOSize EncodedBlocks<H, N, W>::entropyCodeRANSCompat(const input_IT
12481248
const size_t nLiteralStorageElems = calculateNDestTElements<input_t, storageBuffer_t>(nSymbols);
12491249
std::tie(thisBlock, thisMetadata) = expandStorage(slot, nLiteralStorageElems, buffer);
12501250
thisBlock->storeLiterals(nLiteralStorageElems, reinterpret_cast<const storageBuffer_t*>(literals.data()));
1251-
LOGP(info, "StoreLiterals {} bytes, offs: {}:{}", nLiteralStorageElems * sizeof(W), thisBlock->getOffsLiterals(), thisBlock->getOffsLiterals() + nLiteralStorageElems * sizeof(W));
1251+
LOGP(debug, "StoreLiterals {} bytes, offs: {}:{}", nLiteralStorageElems * sizeof(W), thisBlock->getOffsLiterals(), thisBlock->getOffsLiterals() + nLiteralStorageElems * sizeof(W));
12521252
return nLiteralStorageElems;
12531253
}
12541254
return size_t(0);
12551255
}();
12561256

1257-
LOGP(info, "Min, {} Max, {}, size, {}, nSamples {}", view.getMin(), view.getMax(), view.size(), frequencyTable.getNumSamples());
1257+
LOGP(debug, "Min, {} Max, {}, size, {}, nSamples {}", view.getMin(), view.getMax(), view.size(), frequencyTable.getNumSamples());
12581258

12591259
*thisMetadata = detail::makeMetadataRansCompat<input_t, ransState_t, ransStream_t>(encoder.getNStreams(),
12601260
messageLength,
@@ -1317,7 +1317,7 @@ CTFIOSize EncodedBlocks<H, N, W>::encodeRANSV1External(const input_IT srcBegin,
13171317
const size_t dataSize = std::distance(thisBlock->getCreateData(), encodedMessageEnd);
13181318
thisBlock->setNData(dataSize);
13191319
thisBlock->realignBlock();
1320-
LOGP(info, "StoreData {} bytes, offs: {}:{}", dataSize * sizeof(storageBuffer_t), thisBlock->getOffsData(), thisBlock->getOffsData() + dataSize * sizeof(storageBuffer_t));
1320+
LOGP(debug, "StoreData {} bytes, offs: {}:{}", dataSize * sizeof(storageBuffer_t), thisBlock->getOffsData(), thisBlock->getOffsData() + dataSize * sizeof(storageBuffer_t));
13211321
// update the size claimed by encoded message directly inside the block
13221322

13231323
// encode literals
@@ -1329,7 +1329,7 @@ CTFIOSize EncodedBlocks<H, N, W>::encodeRANSV1External(const input_IT srcBegin,
13291329
literalsSize = std::distance(thisBlock->getCreateLiterals(), literalsEnd);
13301330
thisBlock->setNLiterals(literalsSize);
13311331
thisBlock->realignBlock();
1332-
LOGP(info, "StoreLiterals {} bytes, offs: {}:{}", literalsSize * sizeof(storageBuffer_t), thisBlock->getOffsLiterals(), thisBlock->getOffsLiterals() + literalsSize * sizeof(storageBuffer_t));
1332+
LOGP(debug, "StoreLiterals {} bytes, offs: {}:{}", literalsSize * sizeof(storageBuffer_t), thisBlock->getOffsLiterals(), thisBlock->getOffsLiterals() + literalsSize * sizeof(storageBuffer_t));
13331333
}
13341334

13351335
// write metadata
@@ -1389,12 +1389,12 @@ CTFIOSize EncodedBlocks<H, N, W>::encodeRANSV1Inplace(const input_IT srcBegin, c
13891389
}
13901390

13911391
const rans::Metrics<input_t>& metrics = encoder.getMetrics();
1392-
1392+
/*
13931393
if constexpr (sizeof(input_t) > 2) {
13941394
const auto& dp = metrics.getDatasetProperties();
13951395
LOGP(info, "Metrics:{{slot: {}, numSamples: {}, min: {}, max: {}, alphabetRangeBits: {}, nUsedAlphabetSymbols: {}, preferPacking: {}}}", slot, dp.numSamples, dp.min, dp.max, dp.alphabetRangeBits, dp.nUsedAlphabetSymbols, metrics.getSizeEstimate().preferPacking());
13961396
}
1397-
1397+
*/
13981398
if (detail::mayPack(opt) && metrics.getSizeEstimate().preferPacking()) {
13991399
if (proxy.isCached()) {
14001400
return pack(proxy.beginCache(), proxy.endCache(), slot, metrics, buffer);
@@ -1417,7 +1417,7 @@ CTFIOSize EncodedBlocks<H, N, W>::encodeRANSV1Inplace(const input_IT srcBegin, c
14171417
const size_t dictSize = std::distance(thisBlock->getCreateDict(), encodedDictEnd);
14181418
thisBlock->setNDict(dictSize);
14191419
thisBlock->realignBlock();
1420-
LOGP(info, "StoreDict {} bytes, offs: {}:{}", dictSize * sizeof(storageBuffer_t), thisBlock->getOffsDict(), thisBlock->getOffsDict() + dictSize * sizeof(storageBuffer_t));
1420+
LOGP(debug, "StoreDict {} bytes, offs: {}:{}", dictSize * sizeof(storageBuffer_t), thisBlock->getOffsDict(), thisBlock->getOffsDict() + dictSize * sizeof(storageBuffer_t));
14211421

14221422
// encode payload
14231423
auto encodedMessageEnd = thisBlock->getCreateData();
@@ -1429,7 +1429,7 @@ CTFIOSize EncodedBlocks<H, N, W>::encodeRANSV1Inplace(const input_IT srcBegin, c
14291429
const size_t dataSize = std::distance(thisBlock->getCreateData(), encodedMessageEnd);
14301430
thisBlock->setNData(dataSize);
14311431
thisBlock->realignBlock();
1432-
LOGP(info, "StoreData {} bytes, offs: {}:{}", dataSize * sizeof(storageBuffer_t), thisBlock->getOffsData(), thisBlock->getOffsData() + dataSize * sizeof(storageBuffer_t));
1432+
LOGP(debug, "StoreData {} bytes, offs: {}:{}", dataSize * sizeof(storageBuffer_t), thisBlock->getOffsData(), thisBlock->getOffsData() + dataSize * sizeof(storageBuffer_t));
14331433
// update the size claimed by encoded message directly inside the block
14341434

14351435
// encode literals
@@ -1439,7 +1439,7 @@ CTFIOSize EncodedBlocks<H, N, W>::encodeRANSV1Inplace(const input_IT srcBegin, c
14391439
literalsSize = std::distance(thisBlock->getCreateLiterals(), literalsEnd);
14401440
thisBlock->setNLiterals(literalsSize);
14411441
thisBlock->realignBlock();
1442-
LOGP(info, "StoreLiterals {} bytes, offs: {}:{}", literalsSize * sizeof(storageBuffer_t), thisBlock->getOffsLiterals(), thisBlock->getOffsLiterals() + literalsSize * sizeof(storageBuffer_t));
1442+
LOGP(debug, "StoreLiterals {} bytes, offs: {}:{}", literalsSize * sizeof(storageBuffer_t), thisBlock->getOffsLiterals(), thisBlock->getOffsLiterals() + literalsSize * sizeof(storageBuffer_t));
14431443
}
14441444

14451445
// write metadata
@@ -1488,7 +1488,7 @@ o2::ctf::CTFIOSize EncodedBlocks<H, N, W>::pack(const input_IT srcBegin, const i
14881488
thisBlock->realignBlock();
14891489
}
14901490

1491-
LOGP(info, "StoreData {} bytes, offs: {}:{}", packedSize * sizeof(storageBuffer_t), thisBlock->getOffsData(), thisBlock->getOffsData() + packedSize * sizeof(storageBuffer_t));
1491+
LOGP(debug, "StoreData {} bytes, offs: {}:{}", packedSize * sizeof(storageBuffer_t), thisBlock->getOffsData(), thisBlock->getOffsData() + packedSize * sizeof(storageBuffer_t));
14921492
return {0, thisMetadata->getUncompressedSize(), thisMetadata->getCompressedSize()};
14931493
};
14941494

‎Detectors/FIT/FDD/reconstruction/src/CTFCoder.cxx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,6 @@ size_t CTFCoder::estimateCompressedSize(const CompressedDigits& cd)
8080
sz += estimateBufferSize(static_cast<int>(CTF::BLC_charge), cd.charge);
8181
sz += estimateBufferSize(static_cast<int>(CTF::BLC_feeBits), cd.feeBits);
8282

83-
LOG(info) << "Estimated output size is " << sz << " bytes";
83+
LOG(debug) << "Estimated output size is " << sz << " bytes";
8484
return sz;
8585
}

‎Detectors/FIT/FT0/reconstruction/src/CTFCoder.cxx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,6 @@ size_t CTFCoder::estimateCompressedSize(const CompressedDigits& cd)
8383
sz += estimateBufferSize(static_cast<int>(CTF::BLC_qtcAmpl), cd.qtcAmpl);
8484
// clang-format on
8585

86-
LOG(info) << "Estimated output size is " << sz << " bytes";
86+
LOG(debug) << "Estimated output size is " << sz << " bytes";
8787
return sz;
8888
}

‎Detectors/FIT/FV0/reconstruction/src/CTFCoder.cxx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,6 @@ size_t CTFCoder::estimateCompressedSize(const CompressedDigits& cd)
8484
sz += estimateBufferSize(static_cast<int>(CTF::BLC_cfdTime), cd.cfdTime);
8585
sz += estimateBufferSize(static_cast<int>(CTF::BLC_qtcAmpl), cd.qtcAmpl);
8686

87-
LOG(info) << "Estimated output size is " << sz << " bytes";
87+
LOG(debug) << "Estimated output size is " << sz << " bytes";
8888
return sz;
8989
};

‎Detectors/ITSMFT/common/reconstruction/src/CTFCoder.cxx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -229,7 +229,7 @@ size_t CTFCoder::estimateCompressedSize(const CompressedClusters& cc)
229229
sz += estimateBufferSize(static_cast<int>(CTF::BLCpattID), cc.pattID);
230230
sz += estimateBufferSize(static_cast<int>(CTF::BLCpattMap), cc.pattMap);
231231
sz *= 2. / 3; // if needed, will be autoexpanded
232-
LOG(info) << "Estimated output size is " << sz << " bytes";
232+
LOG(debug) << "Estimated output size is " << sz << " bytes";
233233
return sz;
234234
}
235235

‎Detectors/TPC/reconstruction/src/CTFCoder.cxx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -211,6 +211,6 @@ size_t CTFCoder::estimateCompressedSize(const CompressedClusters& ccl)
211211
sz += estimateBufferSize(CTF::BLCnSliceRowClusters, ccl.nSliceRowClusters, ccl.nSliceRowClusters + ccl.nSliceRows);
212212

213213
sz *= 2. / 3; // if needed, will be autoexpanded
214-
LOG(info) << "Estimated output size is " << sz << " bytes";
214+
LOG(debug) << "Estimated output size is " << sz << " bytes";
215215
return sz;
216216
}

‎Utilities/rANS/include/rANS/internal/transform/SourceProxy.h‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,11 +105,11 @@ class SourceProxy
105105
if (functor(begin, end)) {
106106
mProxy.template emplace<0>(begin, end);
107107
mIsCached = true;
108-
LOGP(info, "Caching enabled");
108+
LOGP(debug, "Caching enabled");
109109
} else {
110110
mProxy.template emplace<1>(begin, end);
111111
mIsCached = false;
112-
LOGP(info, "Caching disabled");
112+
LOGP(debug, "Caching disabled");
113113
}
114114
}
115115

@@ -148,4 +148,4 @@ class SourceProxy
148148

149149
} // namespace o2::rans
150150

151-
#endif /* RANS_INTERNAL_TRANSFORM_SOURCEPROXY_H_ */
151+
#endif /* RANS_INTERNAL_TRANSFORM_SOURCEPROXY_H_ */

0 commit comments

Comments
 (0)