ltdb,fcbond

Would be much more convenient to see the examples with surface forms... Especially when they get longer, having just lemmas in Spanish can be confusing :).

Not showing names in the descriptions

Japanese has names for many types but we are not showing them.

<name lang='ja'>承引用陳述ト目的格助詞

ltdb should only read gold profiles that match the grammar

We can match the Version to the string in the run file for each profile.

Document that you can click on nodes in trees

Host at UPOL and VU along with ITELL

Error handling

Blanket exceptions hide the true problems.

I am printing the exception e (which is otherwise unused here), but ideally I should catch only exceptions that you expect and can handle.

Finally, it's good practice to only put the statements that can trigger the exception (generally just one line) inside the "try" block, so you don't mix up exceptions from different calls.

Thanks to @goodmami for the comment

Use profile data statements to help select examples

requires the profile metadata

Require a newer sqlite and use the JSON interface

https://www.sqlite.org/json1.html

Properly parse docstring when read from tdl file

We can (and should) do this on the fly.

Host some instances of the ltdb and link to them

This repository came up when I was searching GitHub for functionality of PyDelphin that may be changing. The following will break with the v1.0.0 version of PyDelphin, so you can either pin your code to the latest v0.9.x version or port to the new API. This issue is just meant to document the changes that may be necessary when you decide to port; no action is needed just yet.

Old:

[...]
import delphin.mrs
[...]
import delphin.mrs.xmrs
import delphin.mrs.simplemrs
[...]
        profile = itsdb.ItsdbProfile(root)
        [...]
        for  row in profile.read_table('item'):
            items[row['i-id']] = (row['i-input'], row['i-comment'])
        for row in profile.read_table('result'):
            [...]
            try:
                mrs_obj = delphin.mrs.simplemrs.loads(mrs_string, single=True, version=1.1, errors='strict')
                # mrs_obj = delphin.mrs.simplemrs.loads(row['mrs'], single=True, version=1.1, strict=False, errors='warn')
                # mrs_string = row['mrs']  # CHANGING
                mrs_json = delphin.mrs.xmrs.Mrs.to_dict(mrs_obj)
                dmrs_json = delphin.mrs.xmrs.Dmrs.to_dict(mrs_obj)

New:

[...]
from delphin.mrs import simplemrs
from delphin.mrs import mrsjson
from delphin.dmrs import dmrsjson
[...]
        profile = itsdb.TestSuite(root)
        [...]
        for  row in profile['item']:
            items[row['i-id']] = (row['i-input'], row['i-comment'])
        for row in profile['result']:
            [...]
            try:
                mrs_obj = simplemrs.decode(mrs_string)
                mrs_json = mrsjson.to_dict(mrs_obj)
                dmrs_json = dmrsjson.to_dict(mrs_obj)

This is a pretty basic and direct porting to the new API. You may also consider restructuring the loops to use TSQL queries, or at least itsdb.join(), partly and importantly because parse-id cannot be relied on to be the same as i-id.

Also, you use errors='strict' but I was thinking of removing that functionality. Is it useful for you?

Processing /home/bond/git/jacy/tsdb/gold/tc-015
Traceback (most recent call last):
  File "gold2db.py", line 122, in <module>
    deriv_json,
NameError: name 'deriv_json' is not defined