qpl icon indicating copy to clipboard operation
qpl copied to clipboard

replace for loops with std algorithm usage (C++ modernization)

Open fwph opened this issue 1 year ago • 1 comments

Description

The code changes in this PR were generated automatically by the Permanence AI Coder and reviewed by myself. This code improvement PR updates several C++ files to replace conventional for loops with uses of std algorithms (transform, fill_n, and copy_n).

Checklist

Test run summary

[----------] Global test environment tear-down
[==========] 388 tests from 125 test suites ran. (417845 ms total)
[  PASSED  ] 334 tests.
[  SKIPPED ] 48 tests, listed below:
. . .
[  FAILED  ] 6 tests, listed below:
[  FAILED  ] ta_c_api_deflate_stateful.dynamic_default_verify
[  FAILED  ] ta_c_api_deflate_stateful.dynamic_high_verify
[  FAILED  ] ta_c_api_deflate_stateful.fixed_default_verify
[  FAILED  ] ta_c_api_deflate_stateful.fixed_high_verify
[  FAILED  ] ta_c_api_deflate_stateful.static_default_verify
[  FAILED  ] ta_c_api_deflate_stateful.static_high_verify

 6 FAILED TESTS
 YOU HAVE 22 DISABLED TESTS

on-behalf-of: @permanence-ai [email protected]

fwph avatar Jun 28 '24 21:06 fwph

Unclear on the value and on the performance impact of these changes. We could take a look at some point and do an extensive performance testing on our side, since it is not provided in the PR, but it would be a lowest priority and I couldn't comment on the ETA right now.

mzhukova avatar Jul 01 '24 17:07 mzhukova