BU-hammerTeam / PyHammer

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

nanfunctions.py ValueError: All Nan Slice Encountered #25

Closed JosephKarpinski closed 6 years ago

JosephKarpinski commented 6 years ago

This abend happens to often when processing multiple FITS file. Have to remove the bad FITS file from control file and restart the process. Program should bypass the FITS file and continue processing. Problem program nanfunctions.py Def nanargmin Line 406

0aa8e903-8b5a-4698-9510-f3f9667d7257

zephyr5050 commented 6 years ago

Implemented a potential fix on the Developer Branch. Try it out and let me know if this works for you. If it does, we can get it pushed into master.

JosephKarpinski commented 6 years ago

Is this new?

[cid:CD3E25C3-B1D4-4FA1-8812-EB39CF668EAF]

On Aug 13, 2018, at 10:45 PM, zephyr5050 notifications@github.com<mailto:notifications@github.com> wrote:

Implemented a potential fix on the Developer Branchhttps://github.com/BU-hammerTeam/PyHammer/tree/Developer-Branch. Try it out and let me know if this works for you. If it does, we can get it pushed into master.

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

JosephKarpinski commented 6 years ago

More testing:

[cid:C655FAC0-2E35-4B48-88AF-69A96C2F4EAE]

On Aug 13, 2018, at 10:45 PM, zephyr5050 notifications@github.com<mailto:notifications@github.com> wrote:

Implemented a potential fix on the Developer Branchhttps://github.com/BU-hammerTeam/PyHammer/tree/Developer-Branch. Try it out and let me know if this works for you. If it does, we can get it pushed into master.

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

zephyr5050 commented 6 years ago

I'm sorry, I don't see anything in your comments. Did you intend to attach a picture or something?

JosephKarpinski commented 6 years ago

Still a bends. See attached image.

[image1.jpeg]

Sent from my iPhone

On Aug 18, 2018, at 5:24 PM, zephyr5050 notifications@github.com<mailto:notifications@github.com> wrote:

I'm sorry, I don't see anything in your comments. Did you intend to attach a picture or something?

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

zephyr5050 commented 6 years ago

Your images don't appear to be attached properly and I'm not sure what you mean by saying it is still a bends.

JosephKarpinski commented 6 years ago

Hmm, replies via email do not seem to map attachments to GitHub comments. Will reply directly via the website. Here’s the latest screen print: 494930b1-b246-470f-99d7-000e860bb34f 508390e8-902f-4e47-bd23-266dbf33e985

JosephKarpinski commented 6 years ago

Replied directly on GitHubs website. Attaching previously sent screen prints. Please review. Thanks.

Sent from my iPad

On Aug 18, 2018, at 9:41 PM, zephyr5050 notifications@github.com<mailto:notifications@github.com> wrote:

Your images don't appear to be attached properly and I'm not sure what you mean by saying it is still a bends.

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

zephyr5050 commented 6 years ago

Sorry to take so long to get back to this issue. But should be fixed on the developer branch. Feel free to check it out and let me know if you have any issues. If you don't see anything we can go ahead and push this into master.