search

BU-hammerTeam / PyHammer

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

PyHammer UnboundLocalError #29

Closed JosephKarpinski closed 5 years ago

JosephKarpinski commented 5 years ago

See attachment: UnboundLocalError: local variable 'snVal' referenced before assignment. Workaround, remove last file processed at time of abend from control file and restart.

pyhammer_unboundlocalerror

zephyr5050 commented 5 years ago

Issue should have been fixed in commit 0152715 on developer branch. Check out the lastest of that branch and let me know if this resolves your issue.

JosephKarpinski commented 5 years ago

Just tried the new PyHammer code. Getting UnboundLocalError in two different location:

JosephKarpinski commented 5 years ago

Updated entry on PyHammer issues web site. UnboundLocalError in two locations. See update.

Sent from my iPad

On Oct 11, 2018, at 10:15 PM, zephyr5050 notifications@github.com<mailto:notifications@github.com> wrote:

Issue should have been fixed in commit 0152715https://github.com/BU-hammerTeam/PyHammer/commit/0152715b73aab476799c0fcba53168975ef53c52 on developer branch. Check out the lastest of that branch and let me know if this resolves your issue.

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

zephyr5050 commented 5 years ago

Made more updates. Check out developer branch and give it a go.

JosephKarpinski commented 5 years ago

Looks good! Put it through multiple known NaN abend FITS files with no issues.

Sent from my iPhone

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

Made more updates. Check out developer branch and give it a go.

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