Small fixes and praat scripts over non-phone annotations

[MichaelGoodale]

This PR just fixes a few bugs here and there as well adding praat scripts over non-phones.

Note, I changed the API for praat scripts a little, but the old API still works, just with a deprecation warning.

[msonderegger]

Great!

On Thu, Jun 27, 2019 at 4:22 PM Michael Goodale notifications@github.com<mailto:notifications@github.com> wrote:

This PR just fixes a few bugs here and there as well adding praat scripts over non-phones.

Note, I changed the API for praat scripts a little, but the old API still works, just with a deprecation warning.

---

You can view, comment on, or merge this pull request online at:

https://github.com/MontrealCorpusTools/PolyglotDB/pull/148

Commit Summary

• Made TextGrid parser skip empty textgrids to fix #145
• Added check for periods in names to fix #128
• Allowed praat scripts to be run over any annotation_type to fix #101
• Fixed typos

File Changes

• M polyglotdb/acoustics/io.pyhttps://github.com/MontrealCorpusTools/PolyglotDB/pull/148/files#diff-0 (10)
• M polyglotdb/acoustics/other.pyhttps://github.com/MontrealCorpusTools/PolyglotDB/pull/148/files#diff-1 (24)
• M polyglotdb/corpus/importable.pyhttps://github.com/MontrealCorpusTools/PolyglotDB/pull/148/files#diff-2 (4)
• M polyglotdb/io/enrichment/helper.pyhttps://github.com/MontrealCorpusTools/PolyglotDB/pull/148/files#diff-3 (3)
• M polyglotdb/io/parsers/textgrid.pyhttps://github.com/MontrealCorpusTools/PolyglotDB/pull/148/files#diff-4 (10)

Patch Links:

• https://github.com/MontrealCorpusTools/PolyglotDB/pull/148.patch
• https://github.com/MontrealCorpusTools/PolyglotDB/pull/148.diff

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/MontrealCorpusTools/PolyglotDB/pull/148?email_source=notifications&email_token=AAGC5UK3DHJ2B3A7FQGMSK3P4TLKHA5CNFSM4H35EXCKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G4DLUOQ, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAGC5UJ7WYVYVHONCP562S3P4TLKHANCNFSM4H35EXCA.

[mmcauliffe]

@MichaelGoodale Looks like it's still failing some tests

[coveralls]

Coverage increased (+0.05%) to 78.554% when pulling 9b8b0aff7af9415cbaf85cc170060cf6c9eb6faf on import-fixes into 36d97af291b7ddc60b5c8c8b974bc9a011ef96ad on master.