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

[WIP] Enable PDF fulltext search #2264

Closed
wants to merge 6 commits into from

Conversation

koppor
Copy link
Member

@koppor koppor commented Nov 13, 2016

This PR enables a fulltext search of all linked PDFs.

It is an initial implementation skeleton, where work can be started from.

@koppor koppor added the stupro label Nov 13, 2016
@koppor koppor mentioned this pull request Nov 13, 2016
@Braunch
Copy link
Contributor

Braunch commented Nov 15, 2016

Some TODOs I thought about:

  • include index initialization in JabRef
    • evaluate when to create/update index
    • evaluate if update method needs to be written (that doesn't create the whole index from scratch)
  • sort search results
  • use pdf importer from [WIP] Show PDF-Comments in JabRef #1883
  • validate the extracted fields in the search result
  • include in UI
  • Qs stuff, unit/integration tests

@koppor koppor added the on-hold label Dec 8, 2016
@koppor
Copy link
Member Author

koppor commented Dec 8, 2016

There is no person working on it.
We are currently trying to focus on other things. 🔥
We will come back later to this topic, therefore closing adding on-hold label and closing the issue for now.

@koppor koppor closed this Dec 8, 2016
@LinusDietz LinusDietz self-assigned this Mar 27, 2017
@tobiasdiez
Copy link
Member

Some links that are maybe helpful:

@tobiasdiez tobiasdiez self-assigned this Aug 10, 2017
@koppor koppor removed the freeze label Oct 9, 2019
@koppor
Copy link
Member Author

koppor commented Oct 9, 2019

superseded by #2838

@koppor koppor mentioned this pull request Oct 9, 2019
6 tasks
@tobiasdiez tobiasdiez mentioned this pull request Mar 27, 2020
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 this pull request may close these issues.

4 participants