search

intercontinental-dictionary-series / keypano

IDS data on Panoan languages coded by Key
Creative Commons Attribution 4.0 International
0 stars 0 forks source link

switch to commandline scripts for the analysis #11

Closed LinguList closed 3 years ago

LinguList commented 3 years ago

@fractaldragonflies, if you check the lexibank/seabor/ scripts, you see that we do all analyses as commandline analyses, so it is much easier to add parsers for cline args.

What you need to do for this is:

I suggest you modify your current analyses in this form, and I also suggest to single out commands, so you do not have one to rule them all, but several scripts for one purpose each

fractaldragonflies commented 3 years ago

Agreed. Sounds very practical. Created simple report that shows the quantity of shared cognates between languages or families. Big difference in thresholds.

John Miller @.***

On Aug 21, 2021, at 4:32 AM, Johann-Mattis List @.***> wrote:

@fractaldragonflies https://github.com/fractaldragonflies, if you check the lexibank/seabor/ scripts, you see that we do all analyses as commandline analyses, so it is much easier to add parsers for cline args.

What you need to do for this is:

modify setup.py add a folder "keypanocommands" add simplified commandline scripts to this folder, where you register the parser, as we do in the seaborcommands folder invoke a script by typing (after installing the keypano package!): cldfbench keypano.command_filename I suggest you modify your current analyses in this form, and I also suggest to single out commands, so you do not have one to rule them all, but several scripts for one purpose each

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/intercontinental-dictionary-series/keypano/issues/11, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIVSLTSW33FNUDAMZOE7ZSDT55XD3ANCNFSM5CRVPQZQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

LinguList commented 3 years ago

Can we merge the branch already? I'd then provide more examples.

fractaldragonflies commented 3 years ago

Branch has been merged. Future improvements have been made and merged as well. While a continuing development effort, the specific issue of using command line scripts has been addressed.