re #8840

This was done with an eye towards providing some level
of validation when possible.

The converted text is stored in separate .rst files in the docs directory.
Refs #9562

and added unit test to test ok

Refs #8784

(At least 5 more places that weren't deleting the returned map.)

(At least 5 more places that weren't deleting the returned map.)

1. Continue to cleaning codes by removing some legacy which is not used
now;
2. Removing properties NewInstrument by merging it to InstrumentName

Also re-arranged a bunch of other code. Part of the "clean-up"
that Wenduo had mentioned.

Re #6485.

Touches nearly everything unfortunately but is necessary.

Removed (or commented out) unused variables.
Made use of boost isnan() instead of !=