Minor C API changes

Author: olokobayusuf

Changelog.md

@@ -3,6 +3,9 @@
 + Added support for `aac`, `flac`, `opus`, `wav` response formats in `muna.beta.openai.audio.speech.create` function.
 + Fixed `TypeError` when creating a prediction with read-only `bytes` input value.
 + Fixed `AttributeError` when creating a prediction that outputs a binary output value.
++ Removed `Dtype.audio` enumeration member.
++ Removed `Dtype.video` enumeration member.
++ Removed `Dtype.model` enumeration member.
 
 ## 0.0.74
 + Fixed `MunaAPIError` exception class raising exception when being unpickled.

muna/c/map.py

@@ -22,9 +22,12 @@ def __init__(
         if map is None:
             map = c_void_p()
             owner = True
-            status = get_fxnc().FXNValueCreateValueMap(byref(map))
+            status = get_fxnc().FXNValueMapCreate(byref(map))
             if status != FXNStatus.OK:
-                raise RuntimeError(f"Failed to create value map with error: {status_to_error(status)}")
+                raise RuntimeError(
+                    f"Failed to create value map "
+                    f"with error: {status_to_error(status)}"
+                )
         self.__map = map
         self.__owner = owner
 
@@ -39,7 +42,10 @@ def key(self, index: int) -> str:
         if status == FXNStatus.OK:
             return buffer.value.decode("utf-8")
         else:
-            raise RuntimeError(f"Failed to get value map key at index {index} with error: {status_to_error(status)}")
+            raise RuntimeError(
+                f"Failed to get value map key at index {index} "
+                f"with error: {status_to_error(status)}"
+            )
 
     def __getitem__(self, key: str) -> Value | None:
         value = c_void_p()
@@ -48,10 +54,12 @@ def __getitem__(self, key: str) -> Value | None:
             key.encode(),
             byref(value)
         )
-        if status == FXNStatus.OK:
-            return Value(value, owner=False)
-        else:
-            raise RuntimeError(f"Failed to get value map value for key '{key}' with error: {status_to_error(status)}")
+        if status != FXNStatus.OK:
+            raise RuntimeError(
+                f"Failed to get value map value for key '{key}' "
+                f"with error: {status_to_error(status)}"
+            )
+        return Value(value, owner=False)            
 
     def __setitem__(
         self,
@@ -64,7 +72,10 @@ def __setitem__(
             value._Value__value
         )
         if status != FXNStatus.OK:
-            raise RuntimeError(f"Failed to set value map value for key '{key}' with error: {status_to_error(status)}")
+            raise RuntimeError(
+                f"Failed to set value map value for key '{key}' "
+                f"with error: {status_to_error(status)}"
+            )
 
     def __len__(self) -> int:
         count = c_int32()
@@ -82,7 +93,7 @@ def __exit__(self, exc_type, exc_value, traceback):
 
     def __release(self):
         if self.__map and self.__owner:
-            get_fxnc().FXNValueRelease(self.__map)
+            get_fxnc().FXNValueMapRelease(self.__map)
         self.__map = None
 
     @classmethod

muna/types/value.py

@@ -12,30 +12,26 @@ class Dtype(StrEnum):
     """
     Value type.
     """
-    null        = "null"
-    bfloat16    = "bfloat16"
-    float16     = "float16"
-    float32     = "float32"
-    float64     = "float64"
-    int8        = "int8"
-    int16       = "int16"
-    int32       = "int32"
-    int64       = "int64"
-    uint8       = "uint8"
-    uint16      = "uint16"
-    uint32      = "uint32"
-    uint64      = "uint64"
-    bool        = "bool"
-    string      = "string"
-    list        = "list"
-    dict        = "dict"
-    image       = "image"
-    video       = "video"
-    audio       = "audio"
-    model       = "model"
-    binary      = "binary"
-    value_list  = "value_list"
-    value_map   = "value_map"
+    null            = "null"
+    bfloat16        = "bfloat16"
+    float16         = "float16"
+    float32         = "float32"
+    float64         = "float64"
+    int8            = "int8"
+    int16           = "int16"
+    int32           = "int32"
+    int64           = "int64"
+    uint8           = "uint8"
+    uint16          = "uint16"
+    uint32          = "uint32"
+    uint64          = "uint64"
+    bool            = "bool"
+    string          = "string"
+    list            = "list"
+    dict            = "dict"
+    image           = "image"
+    image_sequence  = "image_sequence"
+    binary          = "binary"
 
 Value = (
     None                |