Update convert_matrixmarket.cpp

Author: BenBrock

examples/CMakeLists.txt

@@ -6,3 +6,4 @@ endfunction()
 add_example(convert_binsparse)
 add_example(text2hdf5)
 add_example(inspect_binsparse)
+add_example(convert_matrixmarket)

examples/convert_matrixmarket.cpp

@@ -6,13 +6,15 @@
 int main(int argc, char** argv) {
 
   if (argc < 2) {
-    std::cout << "usage: ./inspect_binsparse [input_file.mtx]\n";
+    std::cout << "usage: ./inspect_binsparse [matrix.bsp.h5]\n";
     return 1;
   }
 
   std::string input_file(argv[1]);
 
-  auto metadata = binsparse::inspect(input_file);
+  auto json = binsparse::inspect(input_file);
+
+  auto metadata = json["binsparse"];
 
   std::cout << "Inspecting Binsparse v" << metadata["version"] << " file...\n";
   std::cout << metadata["format"] << " format matrix of dimension "

include/binsparse/type_info.hpp

@@ -116,7 +116,8 @@ template <typename Fn, typename... Args>
 void invoke_visit_fn_impl_(std::vector<std::string> type_labels, Fn&& fn,
                            Args&&... args) {
   if constexpr (sizeof...(Args) <= 3) {
-    if (type_labels.size() == 1) {
+    // The first label we consume is the value label
+    if (type_labels.size() == 3) {
       auto type_label = type_labels.front();
       if (type_label == "uint8") {
         invoke_if_able(std::forward<Fn>(fn), std::uint8_t(),
@@ -155,6 +156,7 @@ void invoke_visit_fn_impl_(std::vector<std::string> type_labels, Fn&& fn,
         assert(false);
       }
     } else {
+      // The next two types are index types, which must be integrals.
       auto type_label = type_labels.back();
       type_labels.pop_back();
       if (type_label == "uint8") {
@@ -181,15 +183,6 @@ void invoke_visit_fn_impl_(std::vector<std::string> type_labels, Fn&& fn,
       } else if (type_label == "int64") {
         invoke_visit_fn_impl_(type_labels, std::forward<Fn>(fn), std::int64_t(),
                               std::forward<Args>(args)...);
-      } else if (type_label == "float32") {
-        invoke_visit_fn_impl_(type_labels, std::forward<Fn>(fn), float(),
-                              std::forward<Args>(args)...);
-      } else if (type_label == "float64") {
-        invoke_visit_fn_impl_(type_labels, std::forward<Fn>(fn), double(),
-                              std::forward<Args>(args)...);
-      } else if (type_label == "bint8") {
-        invoke_visit_fn_impl_(type_labels, std::forward<Fn>(fn), bool(),
-                              std::forward<Args>(args)...);
       } else {
         assert(false);
       }