AST astGetFits<X> functions (astGetFitsS, astGetFitsF, etc.) take a name to specify which FITS card to read; if the name (a char *) is NULL then the current card is read. astshim does not yet support the ability to read the current card, and it is very useful; in particular it would be a big help for
DM-10765. I propose to add this functionality as follows:
- if name is "" then read the current card
- name will default to ""
- In the pybind11 wrapper allow name=None, which will be the default
In addition, the most recent version of AST enhances astTestFits to support name=NULL for the current card. Add similar support to astshim, as above.
An obvious alternative is to add an overload for each FitsChan.getFIts<X> that has no name argument. That is arguably a bit more C++-like, but in my opinion it adds too many new member functions to be worthwhile. Plus overloads make the Python interface a bit harder to understand.