consider containing package about greek-accentuation HOT 7 CLOSED

Yeah, there are a lot of "internal" functions that aren't likely to be useful to most people. I'll think about some key functions that form the "core API" and write up some docs on those which I'll include in this repo but which you can re-use in CLTK.

from greek-accentuation.

Hi James, I think that's a fine name. I like to import my models in the following way. Is this along the lines of what you were thinking about?

>>> from greek_accentuation import characters
>>> from greek_accentuation import syllabify
>>> from greek_accentuation import accentuation

Your project is definitely due for a 1.0 release! I am also thinking about including it by default into the cltk (I don't know if that changes anything for you). @j-duff can chime in, too.

from greek-accentuation.

Yes, you could do

from greek_accentuation import characters

or things like

from greek_accentuation.characters import strip_accents

The other thing I could do if there are no name clashes across the three modules is have greek_accentuation/__init__.py import the modules so things like from greek_accentuation import strip_accents would work.

I'll also take the opportunity to move over some (but not all) of the stuff in https://github.com/jtauber/greek-inflexion/blob/master/accent.py to this package.

from greek-accentuation.

Sounds perfect.

The other thing I could do if there are no name clashes across the three modules is have

I personally disfavor these, as it complicates debugging. However there are times it's called for, I'm sure.

from greek-accentuation.

Done (minus any magic in __init__.py) and released as 1.0.0

from greek-accentuation.

@kylepjohnson I'd be delighted for it to be included in CLTK by default.

from greek-accentuation.

🎉

Let's add it when we push Jack's work!

My request for you is to think about what kind of info we should include in the cltk docs. We should do more than just a link to your site, however we don't need to explain every nuance, either. Let us know (maybe Jack can help) or make a PR yourself.

from greek-accentuation.