-
Notifications
You must be signed in to change notification settings - Fork 0
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
Adds verification of TagNode.location_path to the integration tests #87
Conversation
integration-tests/utils.py
Outdated
from typing import Final | ||
|
||
# Neukölln's digest | ||
PROGRESS_INDICATION_CHARCATERS: Final = "✓→🚴✊★☆⯪𓄁𓅯▶️✴️🪇⚒️🧻🚬🗿🎳⏳🌝☕🐑🐞🌼🪱🌸🏵💮️" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the i think fairly standard font i use for code doesn't have a lot of those charcaters...
includes minor cosmetics applied to scripts.
With each run a randomly selected sample of data from the corpora is tested. The inclusion as integration test is motivated by the fact that a representative coverage in the unit tests would take too much time. At the same time, there are no relevant criteria to curate a subset of documents for testing and such curation would have to be repeated for future extensions of the corpora. A future interface for running these tests could include configuration options to set the sample sizes.
4f1026c
to
2239b76
Compare
ACHTUNG! i force-pushed. i'd say we stick with not using |
gonna be afk for a few days |
this is the questionable approach that i mentioned earlier. i think it's justified well enough.