Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remora for batch creating LTSAs for multiple directories #73

Open
sfregosi opened this issue Aug 4, 2021 · 5 comments
Open

Remora for batch creating LTSAs for multiple directories #73

sfregosi opened this issue Aug 4, 2021 · 5 comments

Comments

@sfregosi
Copy link
Contributor

sfregosi commented Aug 4, 2021

Hi MBARC!

I modified a multiDirLTSA Triton version/code that worked to batch create LTSAs for HARP data into a Remora that could be used with the latest version of Triton and should work for wav files. I am not positive it will work for all, and it could probably use some improvements/clean up, but I wanted to let you all know to see if it was of interest to anyone or if you wanted to try to add it as another branch in the MBARC Triton repository. Let me know!

Selene

@kfrasier
Copy link
Collaborator

kfrasier commented Oct 13, 2021 via email

@marie-r
Copy link
Collaborator

marie-r commented Oct 14, 2021

Dear Selene,

Was this the multiple directory LTSA that I wrote from the old Triton? If it was, it required a modification to the LTSA header. If the LTSA header was modified since that was done (probably over 5 years ago), you'll need to make sure that those modifications were folded into the the new format. In any case, I agree with Kait, it's really nice to see this being added.

All my best,
Marie

@sfregosi
Copy link
Contributor Author

Hi Kait,
First - no worries on the timing and happy and having fun working on this. The good news is @ybarkley has used this and it hasn't broken yet :)

Marie, thanks for checking on this. I built this off multiDirLTSA functions within a Triton version v1.93.20190212 that I got from Ann Allen. I suspect that means the LTSA header modification you are mentioning would not be an issue here, but I'd like to double check. Do you have access to the version you wrote/are talking about? I could maybe compare to what I was working off to be sure.

Thanks!
Selene

@sfregosi
Copy link
Contributor Author

Oh also, yes, will make it a branch like I did the param saving pull request by the end of the week.

sf.

@sfregosi
Copy link
Contributor Author

Created pull request #77

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants