-
Notifications
You must be signed in to change notification settings - Fork 79
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
added derstandard.at #431
added derstandard.at #431
Conversation
@felixvonberlin Thanks for adding this :) Could you give some more context as to why you could not run Furthermore, before reviewing, could you merge the latest master branch to your branch and run the following
|
Oh, yes, I missed it yesterday; I'm sorry! Running the Click Me
I think, I've installed all dependencies. Calling |
Hey, it seems that pytest runs tests unrelated to Fundus (in this case Do you get the same results if you're running |
I tried in a new environment with the ORF; it still produces a lot of errors: Click Me``` oertzenf@tarvos /tmp$ echo $PYTHONPATHoertzenf@tarvos /tmp$ git clone [email protected]:flairNLP/fundus.git =========================================================== no tests ran in 0.01s ============================================================ =================================================================== ERRORS ===================================================================
|
@felixvonberlin I'm a bit confused by these lines: oertzenf@tarvos /tmp$ python3 -m venv fundus
oertzenf@tarvos /tmp$ cd fundus/
oertzenf@tarvos /tmp/fundus$ source bin/activate Is your venv the same directory you cloned the repository to? Normally it's supposed to be something like this fundus
| venv |
Yes: |
Yes, I'm pretty sure that's whats causing your problems. You should create the git clone ...
cd fundus
python3 -m venv venv
source venv/bin/activate |
Hi,
I've added »der Standard«, an Austrian newspaper to fundus.
I think it looks fine, but I could not test it with
pytest
.Even testing existing news sources failed on my platform.
Best,
Felix