Handle blank lines at the end of Karplus data file for jcoupling action

[drroe]

6.27.1.

Fixes behavior where CPPTRAJ would segfault if there were blank lines at the end of the Karplus data file in the jcoupling action.

[drroe]

@hainm any insight into the pytraj error here: https://github.com/Amber-MD/cpptraj/actions/runs/9685730413/job/26726487745?pr=1088

Could it be related to the recent changes? As far as I can tell this change shouldnt affect pytraj at all.

[hainm]

I think it does. (Guess that from reading the title). I will confirm this later (on the way home)).

Hai

On Wed, Jun 26, 2024 at 4:17 PM Daniel R. Roe @.***> wrote:

@hainm https://github.com/hainm any insight into the pytraj error here: https://github.com/Amber-MD/cpptraj/actions/runs/9685730413/job/26726487745?pr=1088

Could it be related to the recent changes? As far as I can tell this change shouldnt affect pytraj at all.

— Reply to this email directly, view it on GitHub https://github.com/Amber-MD/cpptraj/pull/1088#issuecomment-2192552473, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABB645P57KPKC3I3JYPVEHTZJMOXJAVCNFSM6AAAAABJ6Q55BOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJSGU2TENBXGM . You are receiving this because you were mentioned.Message ID: @.***>

[hainm]

I think it does. (Guess that from reading the title). I will confirm this later (on the way home)).

Oh, my mistake. It's because I did code refactoring and the code uses strenum

I will fix this.

In this pytraj's PR, the tests were run with latest python version. That's why tests didn't fail. https://github.com/Amber-MD/pytraj/pull/1657

[hainm]

green now

[drroe]

Thanks Hai!