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

Sortiere Wörter nach Häufigkeit und Autoplay #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

juliapath
Copy link

Hey, zunächst mal sorry, dass das so ein riesen Commit geworden ist. Habe das halt vor allem für mich und ein paar Freunde gemacht und das jetzt alles sauber von einander zu trennen war mir zu viel Aufwand.

Ich nutze die Worthäufigkeit wie sie von duden.de angegeben wird. Die ist zwar nicht sehr genau, hat das offensichtliche Problem, dass Worte speziell relevant für die Gehörlosen-Community eine zu niedrige Häufigkeit haben und zudem ist sie sehr unvollständig. Ich habe versucht dem mit manuellen Ergänzungen entgegenzuwirken.

Das Autoplay hat auch ein Problem, dass es nicht in der Linux Desktop-App funktioniert. Mir egal, da ich nur Ankidroid fürs Lernen nutze, aber sicherlich relevant.

Auch sorry, dass ich so viele off-topic Code-Stil Änderungen gemacht habe. Hatte bei der Entwicklung halt nicht reintegration als erste Priorität im Kopf. Auch habe ich die anderen Decks nicht getestet.

Naja, vielleicht findest du es ja trotzdem nützlich oder interessant oder merged sogar. :)

- Sort fields kinda by word frequency
- Autoplay videos
- Extra fields for personalization
- A lot of code style changes

Sorry for the big commit
@juliapath
Copy link
Author

Achja und ich habe auch die Deck und Model-ID geändert. Denn die sollten doch einzigartig über alle Decks (von allen Personen) sein. Dafür ist doch 7 zu niedrig.

@Sjlver
Copy link

Sjlver commented May 11, 2021

Finde diesen Pull Request fantastisch. Autoplay und eine sinnvolle Häufigkeit der Wörter machen das Lernen viel einfacher und motivierender. Danke, @juliapath !

@juliapath
Copy link
Author

juliapath commented May 11, 2021

@Sjlver Danke. Das freut mich :)

Ein Problem mit dem PR ist mir in der Zwischenzeit noch aufgefallen. Karten von der gleichen Note kommen irgendwie nicht mehr zusammen. Daher hat man am Anfang irgendwie nur eine Rezeptive Karte für jede Vokabel und keine Produktive, obwohl diese im Deck existieren.

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.

2 participants