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

Invalid ids in frequencies.csv #77

Merged
merged 2 commits into from
Jan 15, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions .github/workflows/cldf-validation.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:
pip install pytest-cldf
- name: Validate lexicon
run: |
pytest --cldf-metadata=cldf/lexicon-metadata.json test.py
pytest -k test_lexicon

validate-phonemes:
runs-on: ubuntu-latest
Expand All @@ -43,7 +43,7 @@ jobs:
pip install pytest-cldf
- name: Validate phonemes
run: |
pytest --cldf-metadata=cldf/phonemes-metadata.json test.py
pytest -k test_phonemes

validate-phonology:
runs-on: ubuntu-latest
Expand All @@ -62,7 +62,7 @@ jobs:
pip install pytest-cldf
- name: Validate phonology
run: |
pytest --cldf-metadata=cldf/phonology-metadata.json test.py
pytest -k test_phonology

validate-wordlist:
runs-on: ubuntu-latest
Expand All @@ -81,4 +81,4 @@ jobs:
pip install pytest-cldf
- name: Validate wordlist
run: |
pytest --cldf-metadata=cldf/wordlist-metadata.json test.py
pytest -k test_wordlist
2 changes: 1 addition & 1 deletion lexibank_lexibank_analysed.py
Original file line number Diff line number Diff line change
Expand Up @@ -610,7 +610,7 @@ def _add_languages(writer, languages, condition, features, attr_features, collec
))
for lid, freq in sorted(occurrences.items()):
writer.objects['ValueTable'].append(dict(
ID='f{lid}-{clts_id}',
ID=f'{lid}-{clts_id}',
Language_ID=lid,
Parameter_ID=clts_id,
Value=freq,
Expand Down
27 changes: 23 additions & 4 deletions test.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,25 @@
from pycldf import iter_datasets
from pycldf import iter_datasets, Dataset


def test_valid(cldf_dataset, cldf_logger):
for ds in iter_datasets(cldf_dataset.directory):
assert ds.validate(log=cldf_logger)
def test_lexicon(cldf_dataset, cldf_logger):
md = cldf_dataset.directory / 'lexicon-metadata.json'
ds = Dataset.from_metadata(md)
assert ds.validate(log=cldf_logger)


def test_phonemes(cldf_dataset, cldf_logger):
md = cldf_dataset.directory / 'phonemes-metadata.json'
ds = Dataset.from_metadata(md)
assert ds.validate(log=cldf_logger)


def test_phonology(cldf_dataset, cldf_logger):
md = cldf_dataset.directory / 'phonology-metadata.json'
ds = Dataset.from_metadata(md)
assert ds.validate(log=cldf_logger)


def test_wordlist(cldf_dataset, cldf_logger):
md = cldf_dataset.directory / 'wordlist-metadata.json'
ds = Dataset.from_metadata(md)
assert ds.validate(log=cldf_logger)
Loading