Implement handling of the deprecated size attribute in TAP

Use the value for arraysize to set the size attribute in the TAP data.

python/felis/tap.py

@@ -24,6 +24,7 @@
 from __future__ import annotations
 
 import logging
+import re
 from collections.abc import Iterable, MutableMapping
 from typing import Any
 
@@ -125,7 +126,7 @@ class Tap11Columns(Tap11Base):
         arraysize = Column(String(10))
         xtype = Column(String(SIMPLE_FIELD_LENGTH))
         # Size is deprecated
-        # size = Column(Integer(), quote=True)
+        size = Column("size", Integer(), quote=True)
         description = Column(String(TEXT_FIELD_LENGTH))
         utype = Column(String(SIMPLE_FIELD_LENGTH))
         unit = Column(String(SIMPLE_FIELD_LENGTH))
@@ -410,6 +411,24 @@ def visit_column(self, column_obj: datamodel.Column, table_obj: Table) -> Tap11B
         if (felis_type.is_timestamp or column_obj.datatype == "text") and column.arraysize is None:
             column.arraysize = "*"
 
+        def _is_int(s: str) -> bool:
+            try:
+                int(s)
+                return True
+            except ValueError:
+                return False
+
+        # Handle the deprecated size attribute
+        if column_obj.votable_arraysize is not None and column_obj.votable_arraysize != "":
+            if isinstance(arraysize, int):
+                column.size = arraysize
+            elif _is_int(arraysize):
+                column.size = int(arraysize)
+            elif bool(re.match(r"^[0-9]+\*$", arraysize)):
+                column.size = int(arraysize.replace("*", ""))
+            if column.size is not None:
+                logger.debug(f"Set size to {column.size} for {column.column_name} from arraysize {arraysize}")
+
         column.xtype = column_obj.votable_xtype
         column.description = column_obj.description
         column.utype = column_obj.votable_utype