Remove logic blocking PyTorchObserver lines from being parsed

Summary:
Not sure why we require metrics to have "NET" in the name - it's confusing and undocumented.

RE: https://fb.workplace.com/groups/1690415234371429/permalink/8607242839355266/

Reviewed By: Jack-Khuu

Differential Revision: D66785827

fbshipit-source-id: d2c04fc168b14215268bf6853a887edd11e83794

Author: Varun Puri

benchmarking/data_converters/json_converter/json_converter.py

@@ -42,11 +42,7 @@ def collect(self, data, args=None):
                 else:
                     result = json.loads(match[0])
                     if (
-                        (
-                            "type" in result
-                            and result["type"] == "NET"
-                            and "value" in result
-                        )
+                        ("type" in result and "value" in result)
                         or ("NET" in result)
                         or ("custom_output" in result)
                     ):  # for backward compatibility
@@ -70,8 +66,14 @@ def convert(self, data):
                 table_name = d["table_name"] if "table_name" in d else "Custom Output"
                 details["custom_output"][table_name].append(d["custom_output"])
 
-            elif "type" in d and "metric" in d and "unit" in d:
+            if (
+                "type" in d
+                and "metric" in d
+                and "unit" in d
+                and "custom_output" not in d
+            ):
                 # new format
+                getLogger().info("New format")
                 key = d["type"] + " " + d["metric"]
                 if "info_string" in d:
                     if "info_string" in details[key]: