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

@otu missing where required in treebase conversion #126

Open
jar398 opened this issue Jul 15, 2015 · 0 comments
Open

@otu missing where required in treebase conversion #126

jar398 opened this issue Jul 15, 2015 · 0 comments
Assignees

Comments

@jar398
Copy link
Member

jar398 commented Jul 15, 2015

ot_31 fails the validator. It comes from the treebase converter, so the converter probably needs to change. Also if it fails to validate I don't know how it got into the repo. Spawned from OpenTreeOfLife/oti#34 .

wget --no-check-certificate -O ot_31.json "https://api.opentreeoflife.org/phylesystem/v1/study/ot_31?output_nexml2json=1.2.1"
... [manually snip off the "data" wrapper ...] ...
./scripts/nexson/validate_ot_nexson.py ot_31.json
{
  "errors": {
    "MISSING_MANDATORY_KEY": {
      "data": [
        "@otu"
      ], 
      "refersTo": {
        "@idref": "Tn295377", 
        "@nodeID": "Tn295377", 
        "@top": "trees", 
        "@treeID": "Tr4206", 
        "@treesID": "Tb6619"
      }
    }
  }, 
...
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

2 participants