Nächste Runde von Anpassungen #2
Conversation
setup.py
Outdated
| @@ -30,4 +30,6 @@ | |||
| 'console_scripts': [ | |||
| 'ims-nparser = imsnpars.cli:main', | |||
| ], | |||
| }) | |||
| }, | |||
| python_requires=">=3.7" | |||
There was a problem hiding this comment.
Muss das wirklich 3.7 sein? Funktioniert auch 3.6?
There was a problem hiding this comment.
Ich glaube das hat sich aus dem original Repository ergeben und wurde übernommen. Wenn du Tests für ältere Versionen durchlaufen lässt, können wir das gerne abändern.
There was a problem hiding this comment.
Bisher steht im Fork und im Orginal Repo die Version 3.7 nur in der Readme.md
README.md
Outdated
| ### Download training data and serialized model | ||
|
|
||
| ```sh | ||
| dvc pull |
There was a problem hiding this comment.
dvc pull => ERROR: failed to pull data from the cloud - config file error: no remote specified. Create a default remote with dvc remote add -d <remote name> <remote url>
There was a problem hiding this comment.
Wo liegen die Daten? Sind die Daten "public"? Wenn ein Password oder Token benötigt muss in der README.md stehen wer die Kontaktperson ist.
There was a problem hiding this comment.
Welche Daten meinst du genau? Der Parser ist im allgmeinen unabhängig von den Daten - müssen nur im CoNLL Format vorliegen. Für die aktuellen Modelle haben wir die Hamburger Treebank genutzt: https://corpora.uni-hamburg.de/hzsk/de/islandora/object/treebank:hdt
There was a problem hiding this comment.
Warum gibt es den Ordner .dvc? Zum Beispiel in https://github.com/zentrum-lexikographie/IMSnPars/tree/master/data/hdt sind lauter DVC Referenzen auf conllu Dateien, die eigentlich auf einem Remote Server herumliegen müssten
setup.py
Outdated
| 'conllu>=3.1.1', | ||
| 'Click>=7.1.2', | ||
| 'psutil>=5.7.2', | ||
| 'ray>=0.8.7', |
There was a problem hiding this comment.
Gibt es einen großen Unterschied (breaking change) zwischen ray=0.* und ray 1.*
…ture flag to resolve incompatible pkgs
| pip install -r requirements-dev.txt --use-feature=2020-resolver | ||
| ``` | ||
|
|
||
| ### Download training data and serialized model |
There was a problem hiding this comment.
DVC benötigt SSH account.
- Lösung 1: public www ordner auf odo
- Lösung 2: persistent ID (Datenpublikation)
No description provided.