Adding target to list should warn when coordinates mistyped

[ojf]

When coordinates don't parse correctly Bifrost fails silently when the user tries to "add to list". There should be a warning, maybe a dialog box pops up and says "huh?"

[csayres]

I would like to second this. This silent failure is pretty hard to identify (I've seen a few people get burned by this now in a short time). As an extension, I would suggest that Bifrost not only indicate a failure to parse input coords back to the user, but become more flexible. Eg allow ":" separated coords and " " separated coords. Furthermore the ability to enter coords in decimal degrees (not RA hours!) would be very nice as well...

[dougbrn]

To my knowledge, I believe I incorporated a fail warning when inputting coordinates incorrectly. Looking at the code, the Coordinates.inputcoordsorter includes a validity bool it tracks as it trys to parse input. If no valid format is found, it never flips the validity bool and triggers a pop-up box explaining the vaild coordinate formats. Also, based on that code it should handle ":" separated, " " separated, decimal degrees and hms dms. Its strange that people are getting a silent warning, maybe this function is being invoked incorrectly? I remember it was functional as of the 1.0 release.

[ojf]

I think you did too--but possibly only when using the telescope control tab, not in the Target List tab... Needs more characterization...

[dougbrn]

Hmm, well I see that the inputcoordsorter is invoked in the function bound to the add to list button on the Target list tab, so it should work for the Target List. If anything it wouldn't be implemented in the control tab rather than the Target list tab.