Skip to content

Commit

Permalink
[DEBUG]
Browse files Browse the repository at this point in the history
  • Loading branch information
@s-trinh
s-trinh committed May 7, 2024
1 parent 4a53b5f commit 3362200
Show file tree
Hide file tree
Showing 3 changed files with 82 additions and 83 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/other-arch-isolated.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ jobs:
# make -j$(nproc)
make -j$(nproc) testNPZ
valgrind --leak-check=full --show-leak-kinds=all --track-origins=yes ./modules/core/testNPZ
gdb -ex run -ex quit ./modules/core/testNPZ
valgrind --leak-check=full ./modules/core/testNPZ
ctest -j$(nproc) --output-on-failure
2 changes: 1 addition & 1 deletion .github/workflows/other-arch.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ jobs:
# make -j$(nproc)
make -j$(nproc) testNPZ
valgrind --leak-check=full --show-leak-kinds=all --track-origins=yes ./modules/core/testNPZ
gdb -ex run -ex quit ./modules/core/testNPZ
valgrind --leak-check=full ./modules/core/testNPZ
ctest -j$(nproc) --output-on-failure
161 changes: 80 additions & 81 deletions modules/core/test/tools/io/testNPZ.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -36,17 +36,17 @@
#include <iostream>
#include <visp3/core/vpConfig.h>

#if defined(VISP_HAVE_CATCH2)
#define CATCH_CONFIG_RUNNER
#include <catch.hpp>
#if defined(VISP_HAVE_CATCH2) && \
(defined(_WIN32) || (defined(__unix__) || defined(__unix) || (defined(__APPLE__) && defined(__MACH__))))
// #define CATCH_CONFIG_RUNNER
// #include <catch.hpp>

#include <visp3/core/vpIoTools.h>

namespace
{
std::string createTmpDir()
{
#if defined(_WIN32) || (defined(__unix__) || defined(__unix) || (defined(__APPLE__) && defined(__MACH__)))
std::string username;
vpIoTools::getUserName(username);

Expand All @@ -62,83 +62,82 @@ std::string createTmpDir()
}
}

TEST_CASE("Test visp::cnpy::npy_load/npz_save", "[visp::cnpy I/O]")
{
std::string directory_filename = createTmpDir();
REQUIRE(vpIoTools::checkDirectory(directory_filename));
std::string npz_filename = directory_filename + "/test_npz_read_write.npz";

SECTION("Read/Save string data")
{
const std::string save_string = "Open Source Visual Servoing Platform";
std::vector<char> vec_save_string(save_string.begin(), save_string.end());
// Manually add a null-terminated character
// https://stackoverflow.com/a/8247804
vec_save_string.push_back('\0');
const std::string identifier = "String";
visp::cnpy::npz_save(npz_filename, identifier, &vec_save_string[0], { vec_save_string.size() }, "w");

visp::cnpy::npz_t npz_data = visp::cnpy::npz_load(npz_filename);
visp::cnpy::NpyArray arr_string_data = npz_data[identifier];
const std::string read_string = std::string(arr_string_data.data<char>());
CHECK(save_string == read_string);
}

SECTION("Read/Save multi-dimensional array")
{
size_t height = 5, width = 7, channels = 3;
std::vector<int> save_vec;
save_vec.reserve(height*width*channels);
for (size_t i = 0; i < height*width*channels; i++) {
save_vec.push_back(i);
}

const std::string identifier = "Array";
visp::cnpy::npz_save(npz_filename, identifier, &save_vec[0], { height, width, channels }, "a"); // append
visp::cnpy::npz_t npz_data = visp::cnpy::npz_load(npz_filename);
visp::cnpy::NpyArray arr_vec_data = npz_data[identifier];
std::vector<int> read_vec = arr_vec_data.as_vec<int>();

REQUIRE(save_vec.size() == read_vec.size());
for (size_t i = 0; i < read_vec.size(); i++) {
CHECK(save_vec[i] == read_vec[i]);
}
}

REQUIRE(vpIoTools::remove(directory_filename));
REQUIRE(!vpIoTools::checkDirectory(directory_filename));
#endif
}

// https://en.cppreference.com/w/cpp/types/integer
// https://github.com/catchorg/Catch2/blob/devel/docs/test-cases-and-sections.md#type-parametrised-test-cases
using BasicTypes = std::tuple<uint8_t, int8_t, uint16_t, int16_t, uint32_t, int32_t, uint64_t, int64_t, float, double>;
TEMPLATE_LIST_TEST_CASE("Test visp::cnpy::npy_load/npz_save", "[BasicTypes][list]", BasicTypes)
{
std::string directory_filename = createTmpDir();
REQUIRE(vpIoTools::checkDirectory(directory_filename));
std::string npz_filename = directory_filename + "/test_npz_read_write.npz";

const std::string identifier = "data";
TestType save_data = std::numeric_limits<TestType>::min();
visp::cnpy::npz_save(npz_filename, identifier, &save_data, { 1 }, "w");

visp::cnpy::npz_t npz_data = visp::cnpy::npz_load(npz_filename);
visp::cnpy::NpyArray arr_data = npz_data[identifier];
TestType read_data = *arr_data.data<TestType>();
CHECK(save_data == read_data);

save_data = std::numeric_limits<TestType>::max();
visp::cnpy::npz_save(npz_filename, identifier, &save_data, { 1 }, "a"); // append

npz_data = visp::cnpy::npz_load(npz_filename);
arr_data = npz_data[identifier];
read_data = *arr_data.data<TestType>();
CHECK(save_data == read_data);

REQUIRE(vpIoTools::remove(directory_filename));
REQUIRE(!vpIoTools::checkDirectory(directory_filename));
}
// TEST_CASE("Test visp::cnpy::npy_load/npz_save", "[visp::cnpy I/O]")
// {
// std::string directory_filename = createTmpDir();
// REQUIRE(vpIoTools::checkDirectory(directory_filename));
// std::string npz_filename = directory_filename + "/test_npz_read_write.npz";

// SECTION("Read/Save string data")
// {
// const std::string save_string = "Open Source Visual Servoing Platform";
// std::vector<char> vec_save_string(save_string.begin(), save_string.end());
// // Manually add a null-terminated character
// // https://stackoverflow.com/a/8247804
// vec_save_string.push_back('\0');
// const std::string identifier = "String";
// visp::cnpy::npz_save(npz_filename, identifier, &vec_save_string[0], { vec_save_string.size() }, "w");

// visp::cnpy::npz_t npz_data = visp::cnpy::npz_load(npz_filename);
// visp::cnpy::NpyArray arr_string_data = npz_data[identifier];
// const std::string read_string = std::string(arr_string_data.data<char>());
// CHECK(save_string == read_string);
// }

// SECTION("Read/Save multi-dimensional array")
// {
// size_t height = 5, width = 7, channels = 3;
// std::vector<int> save_vec;
// save_vec.reserve(height*width*channels);
// for (size_t i = 0; i < height*width*channels; i++) {
// save_vec.push_back(i);
// }

// const std::string identifier = "Array";
// visp::cnpy::npz_save(npz_filename, identifier, &save_vec[0], { height, width, channels }, "a"); // append
// visp::cnpy::npz_t npz_data = visp::cnpy::npz_load(npz_filename);
// visp::cnpy::NpyArray arr_vec_data = npz_data[identifier];
// std::vector<int> read_vec = arr_vec_data.as_vec<int>();

// REQUIRE(save_vec.size() == read_vec.size());
// for (size_t i = 0; i < read_vec.size(); i++) {
// CHECK(save_vec[i] == read_vec[i]);
// }
// }

// REQUIRE(vpIoTools::remove(directory_filename));
// REQUIRE(!vpIoTools::checkDirectory(directory_filename));
// }

// // https://en.cppreference.com/w/cpp/types/integer
// // https://github.com/catchorg/Catch2/blob/devel/docs/test-cases-and-sections.md#type-parametrised-test-cases
// using BasicTypes = std::tuple<uint8_t, int8_t, uint16_t, int16_t, uint32_t, int32_t, uint64_t, int64_t, float, double>;
// TEMPLATE_LIST_TEST_CASE("Test visp::cnpy::npy_load/npz_save", "[BasicTypes][list]", BasicTypes)
// {
// std::string directory_filename = createTmpDir();
// REQUIRE(vpIoTools::checkDirectory(directory_filename));
// std::string npz_filename = directory_filename + "/test_npz_read_write.npz";

// const std::string identifier = "data";
// TestType save_data = std::numeric_limits<TestType>::min();
// visp::cnpy::npz_save(npz_filename, identifier, &save_data, { 1 }, "w");

// visp::cnpy::npz_t npz_data = visp::cnpy::npz_load(npz_filename);
// visp::cnpy::NpyArray arr_data = npz_data[identifier];
// TestType read_data = *arr_data.data<TestType>();
// CHECK(save_data == read_data);

// save_data = std::numeric_limits<TestType>::max();
// visp::cnpy::npz_save(npz_filename, identifier, &save_data, { 1 }, "a"); // append

// npz_data = visp::cnpy::npz_load(npz_filename);
// arr_data = npz_data[identifier];
// read_data = *arr_data.data<TestType>();
// CHECK(save_data == read_data);

// REQUIRE(vpIoTools::remove(directory_filename));
// REQUIRE(!vpIoTools::checkDirectory(directory_filename));
// }

int main(int argc, char *argv[])
{
Expand Down

0 comments on commit 3362200

Please sign in to comment.