Hotfixes/missing comma (#20)

* Fixing a missing comma

* Bumping the version

.gitignore

@@ -33,4 +33,4 @@
 
 # Misc
 .vscode
-build
+build*

CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## [2021-02-09 - Version 1.2.2](https://github.com/matajoh/libnpy/releases/tag/v1.2.2)
+
+Improvements:
+- Bug fix for a missing comma on 1d shape
+
 ## [2021-02-08 - Version 1.2.1](https://github.com/matajoh/libnpy/releases/tag/v1.2.1)
 
 Improvements:

RELEASE_NOTES

@@ -1,4 +1,2 @@
 Improvements:
-- Bug fix for scalar tensor reading
-- Bug fix with memstream buffer size at initialization
-- ".npy" will be added to tensor names in NPZ writing if not already present
+- Bug fix for a missing comma on 1d shape

VERSION

@@ -1 +1 @@
-1.2.1
+1.2.2

include/npy/npy.h

@@ -83,6 +83,12 @@ void write_npy_header(std::basic_ostream<CHAR> &output,
             buff << ", ";
         }
     }
+
+    if(shape.size() == 1)
+    {
+        buff << ",";
+    }
+
     buff << "), }";
     std::string dictionary = buff.str();
     auto dict_length = dictionary.size() + 1;

test/libnpy_tests.h

@@ -175,6 +175,15 @@ std::string npy_scalar_stream(npy::endian_t endianness = npy::endian_t::NATIVE)
     return actual_stream.str();
 }
 
+template <typename T>
+std::string npy_array_stream(npy::endian_t endianness = npy::endian_t::NATIVE)
+{
+    std::ostringstream actual_stream;
+    npy::tensor<T> tensor = test_tensor<T>({25});
+    npy::save(actual_stream, tensor, endianness);
+    return actual_stream.str();
+}
+
 template <typename T>
 std::string npy_fortran_stream(npy::endian_t endianness = npy::endian_t::NATIVE)
 {

test/npy_read.cpp

@@ -14,6 +14,7 @@ int test_npy_read()
     test_read<std::int32_t>(result, "int32");
     test_read<std::int32_t>(result, "int32_big");
     test_read_scalar<std::int32_t>(result, "int32_scalar");
+    test_read_array<std::int32_t>(result, "int32_array");
     test_read<std::uint64_t>(result, "uint64");
     test_read<std::int64_t>(result, "int64");
     test_read<float>(result, "float32");

test/npy_read.h

@@ -27,4 +27,12 @@ void test_read_scalar(int &result, const std::string &name)
     test::assert_equal(expected, actual, result, "npy_read_" + name);
 }
 
+template <typename T>
+void test_read_array(int& result, const std::string &name)
+{
+    npy::tensor<T> expected = test::test_tensor<T>({25});
+    npy::tensor<T> actual = npy::load<T, npy::tensor>(test::asset_path(name + ".npy"));
+    test::assert_equal(expected, actual, result, "npy_read_" + name);
+}
+
 #endif

test/npy_write.cpp

@@ -47,6 +47,10 @@ int test_npy_write()
     actual = test::npy_scalar_stream<std::int32_t>(npy::endian_t::LITTLE);
     test::assert_equal(expected, actual, result, "npy_write_int32_scalar");
 
+    expected = test::read_asset("int32_array.npy");
+    actual = test::npy_array_stream<std::int32_t>(npy::endian_t::LITTLE);
+    test::assert_equal(expected, actual, result, "npy_write_int32_array");
+
     expected = test::read_asset("uint64.npy");
     actual = test::npy_stream<std::uint64_t>(npy::endian_t::LITTLE);
     test::assert_equal(expected, actual, result, "npy_write_uint64");