-
Notifications
You must be signed in to change notification settings - Fork 16
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
Control list filters #323
base: dev
Are you sure you want to change the base?
Control list filters #323
Conversation
This reverts commit 6e202f0.
* Add more details about the lemmatizer at the new corpus phase. * Add test for lemmatization services * Add more verbosity because tests hang * Remove various screenshots
|
A faire:
|
|
Les tests ne fonctionnent pas, c'est un premier jet |
Tests selenium qui bug sur github mais pas en local... A retravailler |
* Fixing tests * Revert update of status * Fix an issue where not having a ControlListUser would lead to being unable to apply stuff * Normalize control list
@Juliettejns Je ne suis toujours pas sûr de l'utilisation de ControlListUser plutôt que de ControlList directement. |
app/main/views/corpus.py
Outdated
@main.route('/corpus/new', methods=["POST", "GET"]) | ||
@login_required | ||
def corpus_new(): | ||
""" Register a new corpus | ||
""" | ||
logging.basicConfig(filename='./pyrrha_corpus_creation.log', level=logging.DEBUG, |
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.
La basic config devrait avoir lieu ailleurs je pense... Par exemple, dans create_app
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.
Requests:
- Move ControlListUser to ControlList
- Add combinatory tests for WordToken.is_valid using various regex.
23e2122
to
dcd28a6
Compare
287696f
to
f4e925c
Compare
Issue #180: Allow regular expressions to filter tokens