-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Fetcher tests failing #6369
Comments
|
API keys: Howto for getting a sandbox key for developers ADs-Fetcher: This is the PR switchting to the new API, isn't it? --> #5501 |
Just got mail that the INSPIRE has now an official API: |
Springerlink is giving m,e 401 locally when I debug |
@DominikVoigt Can you confirm for SpringerLink? 😅 |
If I substitute the API key that is hardcoded into the SpringerLink fetcher with my own API key all tests pass. |
Springer mailed us on 2020-04-29:
Internal stats show "On November 28, your application called the API more than 13,000 times." |
State as of 2020-08-02Source: https://github.com/JabRef/jabref/runs/937189942
|
We should add the Springer key to the build secrets as well |
I will aquire a sepearate key for Springer and enable Springer tests in a separate PR. |
Rate limit of IEEE
Source: Key agreement |
We should execute the fetcher tests maybe once a month on CI and not on every PR |
Since a few months, we DO NOT execute the fetcher tests on each PR. We currently the fetcher tests, when
See https://github.com/JabRef/jabref/blob/master/.github/workflows/tests-fetchers.yml |
The discussion on this move can be found at #5807. |
|
citesser => journaltitle instead of journal |
@DominikVoigt added a conversion layer at #6687. Thus, it should not matter whether journaltitle (BibLaTeX) or journal (BibTeX) is returned. -- the layer converts to the format required by the target database |
@Siedlerchr Thank you for the hint --> @DominikVoigt's layer doesn't help here as the fetcher tests directly check the fetcher result - and not the reformatting by the intermediate layer. |
Refs #3854 (as background information) |
|
Currently there seems to be a latex/unicode conversion issue in the INPIRE test
|
|
We disabled ACM, Google Scholar and JSTOR at #7229. We have different follow-up tickets and PRs. Thus, I can close this issue. |
Some of the fetcher tests are failing (ISBN, Inspire, ...)
https://github.com/JabRef/jabref/actions?query=workflow%3A%22Fetcher+Tests%22
The text was updated successfully, but these errors were encountered: