Move key-binding descriptions from class docstring to pypeit_show_1dspec description

[kbwestfall]

As titled. This is primarily a documentation PR that is directly to release, and it includes some changelog comments related to #2079.

I'll tag and release version 2.0.1, assuming this passes tests.

[kbwestfall]

@ejeschke Somewhat related to this, the connect_to_ginga function in pypeit/display/display.py will call

subprocess.Popen(['ginga', f'--rcport={port}',  '--modules=RC,SlitWavelength'])

If it can't connect to an already open ginga window. Should this instead be:

subprocess.Popen(['ginga', f'--rcport={port}',  '--modules=RC,SlitWavelength,Spec1dView'])

?

[debora-pe]

Thanks @kbwestfall!

[debora-pe]

Thank you for this @kbwestfall!
One thing that I keep forgetting when I use pypeit_show_1dspec is how to show the help screen on the side with all the relevant bindings.
Do you mind adding here that, after entering the mode, by typing h in the plot window ginga will show the help on the side?

[debora-pe]

Should this be:

        if args.obj is not None:
            exten = np.where(sobjs.NAME == args.obj)[0][0]
        else:
            exten = args.exten-1 # 1-index in FITS file
        if exten < 0:
            raise PypeItError(f"Bad input extension/object name: {args.obj}")

[ejeschke]

@kbwestfall , I don't think you should remove this help docstring. It is used by Ginga to show help for that pypeit special mode when a person types 'h'. I suppose we could add a special command to fetch the help from somewhere else, but I would be a little concerned that we bypassed the internal help system.

[ejeschke]

If it can't connect to an already open ginga window. Should this instead be:

subprocess.Popen(['ginga', f'--rcport={port}', '--modules=RC,SlitWavelength,Spec1dView'])

?

No, doesn't need to because Spec1dView registers itself via PypeIt as a plugin, and the plugin is available and invoked by pypeit_show_1dspec. I think SlitWavelength could probably be treated the same way, if we modify the relevant pypeit "show_" scripts. The RC plugin needs to be passed with --modules because it is not set to auto-start by default. That also very likely could be automated in the "show" scripts; I could take a look at that.