BU-hammerTeam / PyHammer

A Python Spectral Typing Suite
MIT License
53 stars 14 forks source link

Doc Mismatch: 'The Template Spectra' documentation versus as seen in Spectrum.py #32

Closed JosephKarpinski closed 5 years ago

JosephKarpinski commented 5 years ago

Anyone using the new feature in PyHammer to generate a sprectralIndices.csv may look for definitions of the spectra line indices. Those found in the paper "An Empirical Template Library of Stellar Spectra for a Wide Range of Spectral Classes, Luminosity Classes, and Metallicities Using SDSS BOSS Spectra" link "The Template Spectra" differ from what actually is used in spectrum.py by labels and wavelength ranges used in creating each spectra indices. Possibly update the paper link with matching label and wavelength values, to clarify any confusion or concerns?

http://iopscience.iop.org/0067-0049/230/2/16/suppdata/apjsaa656dt2_mrt.txt

template spectra doc mismatch

zephyr5050 commented 5 years ago

The paper is a published paper and cannot be amended. Unfortunately, it cannot keep up with any changes to PyHammer. However, we can change the wiki documentation on this GitHub page to keep up to date. I will close this issue for now though.

JosephKarpinski commented 5 years ago

Maybe an update on the GitHub site when the new spectra line feature in PyHammer is generally released?

Sent from my iPhone

On Oct 13, 2018, at 9:56 AM, zephyr5050 notifications@github.com<mailto:notifications@github.com> wrote:

Closed #32https://github.com/BU-hammerTeam/PyHammer/issues/32.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/BU-hammerTeam/PyHammer/issues/32#event-1902174443, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AjZ2ILZ2WfiRW2JgPzwRydCUmFkjRYCfks5ukfEdgaJpZM4XanH2.

zephyr5050 commented 5 years ago

The wiki page has been updated with the changes. All recent changes have been merged into the master branch and tagged as a new release.