Skip to content

Conversation

@AmitMY
Copy link
Contributor

@AmitMY AmitMY commented Nov 20, 2022

No description provided.

Copy link
Collaborator

@bricksdont bricksdont left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple comments (I did not test the loader yet, will do later today)

from ..warning import dataset_warning
from ...datasets import SignDatasetConfig

_DESCRIPTION = """
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor: this text is not an ideal description; here is a suggestion:

Ein umfangreiches interaktives Lexikon der drei Gebärdensprachen der Schweiz (Deutschschweizerische Gebärdensprache DSGS, Langue des Signes Française LSF und Lingua Italiana dei Segni LIS). Herausgegeben vom Schweizerischen Gehörlosenbund (SGB-FSS).

@bricksdont
Copy link
Collaborator

I managed to download two thirds of all entries, and then ran into a 500 error:

Found 15491 lexicon items.

Dl Completed...: 68%
10579/15491 [05:30<02:30, 32.72 url/s]

DownloadError: Failed to get url https://signsuisse.sgb-fss.ch/index.php?id=4&tx_issignsuisselexikon_anzeige%5Blanguid%5D=0&tx_issignsuisselexikon_anzeige%5Buid%5D=130937. HTTP code: 500.

Does this perhaps mean that the API is limiting requests?

@AmitMY
Copy link
Contributor Author

AmitMY commented Nov 21, 2022

you should restart the download. it will continue from the file it left off. seems like rate-limit to me, but I did not encounter such.

@bricksdont
Copy link
Collaborator

ok, after restarting I can download everything now; and the dataset entries look correct to me.

Can you bump the version?

@AmitMY AmitMY merged commit 11b3f0f into master Nov 28, 2022
@AmitMY AmitMY deleted the signsuisse branch November 28, 2022 13:26
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.

3 participants