Search_object and shortname

[kelle]

Does the search_object method expect a shortname column?

t = db.search_object(
                simbad_names, output_table="Sources", fmt="astropy", fuzzy_search=False
            )

tests/scheduled_checks.py:48: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
../../../../miniforge3/envs/template-dev/lib/python3.13/site-packages/astrodbkit/utils.py:26: in wrapper
    return f(*args, **kwargs)
../../../../miniforge3/envs/template-dev/lib/python3.13/site-packages/astrodbkit/astrodb.py:570: in search_object
    filters = [self.metadata.tables[k].columns[v].ilike(f"{n}") for n in name]
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

self = <sqlalchemy.sql.base.ReadOnlyColumnCollection object at 0x141491940>, key = 'shortname'

    def __getitem__(
        self, key: Union[str, int, slice, Tuple[Union[str, int], ...]]
    ) -> Union[ReadOnlyColumnCollection[_COLKEY, _COL_co], _COL_co]:
        try:
            if isinstance(key, (tuple, slice)):
                if isinstance(key, slice):
                    cols = (
                        (sub_key, col)
                        for (sub_key, col, _) in self._collection[key]
                    )
                else:
                    cols = (self._index[sub_key] for sub_key in key)
    
                return ColumnCollection(cols).as_readonly()
            else:
>               return self._index[key][1]
E               KeyError: 'shortname'

../../../../miniforge3/envs/template-dev/lib/python3.13/site-packages/sqlalchemy/sql/base.py:1605: KeyError

[dr-rodriguez]

The function has a mapping of which tables and which columns it should use for fetching the right names. See https://github.com/astrodbtoolkit/AstrodbKit/blob/main/astrodbkit/astrodb.py#L495

In particular:

table_names={"Sources": ["source", "shortname"], "Names": ["other_name"]},

The user should overwrite that for their schema, but it's probably a good idea to remove shortname since that's less likely.
We should probably also include a print statement that confirms which tables/columns it is using as part of the search.