Skip to content

Fix issue #61: allow controlled vocabulary reference in linguistic types#62

Closed
dan-ya wants to merge 1 commit into
dopefishh:masterfrom
dan-ya:master
Closed

Fix issue #61: allow controlled vocabulary reference in linguistic types#62
dan-ya wants to merge 1 commit into
dopefishh:masterfrom
dan-ya:master

Conversation

@dan-ya

@dan-ya dan-ya commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

This PR fixes issue #61.

What was wrong:

  • There was no way to reference a controlled vocabulary (CONTROLLED_VOCABULARY_REF)
    when defining a LINGUISTIC_TYPE.

What this PR does:

  • Adds an optional controlled_vocabulary argument to Elan.add_linguistic_type()
  • When provided, the argument is written as CONTROLLED_VOCABULARY_REF in the ELAN file

API change:

  • Elan.add_linguistic_type(..., controlled_vocabulary=None)

Closes #61

@dopefishh

Copy link
Copy Markdown
Owner

Thanks, I fixed the issues and merged your code #63

@dopefishh dopefishh closed this May 21, 2026
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

Successfully merging this pull request may close these issues.

Linguistic type with a reference to controlled vocabulary

2 participants