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

Add preferences + popup for usage of GROBID #8000

Closed
koppor opened this issue Aug 18, 2021 · 5 comments · Fixed by #8002
Closed

Add preferences + popup for usage of GROBID #8000

koppor opened this issue Aug 18, 2021 · 5 comments · Fixed by #8002
Assignees
Milestone

Comments

@koppor
Copy link
Member

koppor commented Aug 18, 2021

When the PDF Import is used, the internet service GROBID is used. Default: No usage allowed. If user requests use, they have to confirm.

Preferences:

  • Use GROBID
  • URL of GROBID server
@koppor koppor added this to the v5.4 milestone Aug 18, 2021
@tobiasdiez
Copy link
Member

I wouldn't use the word "Grobid". This is an implementation detail, and users don't know what it is. Some generically "send pdfs to JabRef online service to analyze" is more descriptive in my opinion.

I also don't know if we really need to make the url configurable.

@btut
Copy link
Contributor

btut commented Aug 19, 2021

I wouldn't use the word "Grobid".

I think some credit to Grobid is due here. I would even include a link.

Some generically "send pdfs to JabRef online service to analyze" is more descriptive in my opinion.

I would do both.

I also don't know if we really need to make the url configurable.

I myself am not going to use the service otherwise (actually, I would just point grobid.jabref.org to localhost in my hosts file, but would much rather have a JabRef setting). Setting up a Grobid instance is quite easy. Users concerned about their data can have their own or institutions could provide one for their researchers to use.

@Siedlerchr
Copy link
Member

Add Grobid in braces

@btut
Copy link
Contributor

btut commented Aug 19, 2021

How about this?
image

@btut btut mentioned this issue Aug 19, 2021
3 tasks
@calixtus
Copy link
Member

Fixed by #8002

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 a pull request may close this issue.

5 participants