uf-mil / SubjuGator

SubjuGator 8 on-board software
31 stars 43 forks source link

sub8_hydrophones: fix to 9b5c452f14491dccbbf6cafba65856efc463d68b which renamed package but didn't fix scripts #367

Closed forrestv closed 5 years ago

RustyBamboo commented 5 years ago

Keep the filenames with .py?

forrestv commented 5 years ago

Why? They're executables, not modules. The .py being removed actually broke something: renaming hydrophones to hydrophones.py made it so import hydrophones in the other scripts ran the hydrophones script rather than importing the hydrophones library.

On Tue, Sep 25, 2018 at 9:36 AM Daniel Volya notifications@github.com wrote:

Keep the filenames with .py?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

forrestv commented 5 years ago

Err, adding the .py broke things.