TODO

Todo

a module in pyhetdex should not depend on the structure of a class defined in code that uses pyhetdex. Here referring to shot

either pass the illumination_model option as parameters to fplaneToThroughput() or check that it is implemented when initialising ThroughputServer.

original entry

Todo

Improve the documentation.

ifu_centers: plotFocalPlaneQuicklook() and get_image() have a ifu_centers arguments, which is the center of the IFUs in the focal plane. This is confusing as in HETDEX jargon IFUCen refers to the fibers within one IFU.

Write tests.

Use io.StringIO instead of StringIO.StringIO in python2

The CD matrix could be substituted by astropy WCS:

• retrieve_image_DSS(): create it from the retrieved fits header
• retrieve_image_SDSS(): create it by hand using ra, dec, size and scale information

Check if io.StringIO works for python2, if yes, we don’t need it pyhetdex.tools.six

original entry

Todo

filename: is hard coded. Instead of the output directory pass the full output name in the function arguments

Rename the function: probably plot_image would be more appropriate

box, size_cut_out: they should not be hard coded

html size should not be hard code

original entry

Todo

ifu_size is hard coded to 0.012. Move to optional argument

Rename the function: function does not plot!

original entry

Todo

either catch errors from urlopen() in retrieve_image_SDSS() and retrieve_image_DSS() or add the notion that it’s raised and do something about it in get_image()

original entry

Todo

see if it makes sense to use numpy.unique()

original entry