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

Implimented arXivId Parsing for PDF with arXivId #12335

Merged
merged 22 commits into from
Jan 20, 2025
Merged
Show file tree
Hide file tree
Changes from 15 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
import org.jabref.model.entry.BibEntry;
import org.jabref.model.entry.LinkedFile;
import org.jabref.model.entry.field.StandardField;
import org.jabref.model.entry.identifier.ArXivIdentifier;
import org.jabref.model.entry.identifier.DOI;
import org.jabref.model.entry.types.EntryType;
import org.jabref.model.entry.types.StandardEntryType;
Expand Down Expand Up @@ -51,6 +52,8 @@ public class PdfContentImporter extends PdfImporter {

private static final Pattern YEAR_EXTRACT_PATTERN = Pattern.compile("\\d{4}");

private static final int ARXIV_PREFIX_LENGTH = "arxiv:".length();

// input lines into several lines
private String[] lines;

Expand Down Expand Up @@ -372,11 +375,13 @@ Optional<BibEntry> getEntryFromPDFContent(String firstpageContents, String lineS
String volume = null;
String number = null;
String pages = null;
String arXivId = null;
// year is a class variable as the method extractYear() uses it;
String publisher = null;

EntryType type = StandardEntryType.InProceedings;
if (curString.length() > 4) {
arXivId = getArXivId(null);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an example why comments are to be avoided as much as possible, this line compiles fine, runs fine but the comment above it is misguiding. Please move comment to its correct location.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Passing null is really bad style. This neesd to be fixed eventually!!

I will approve nevertheless. Too much time spend on this.

// special case: possibly conference as first line on the page
extractYear();
doi = getDoi(null);
Expand All @@ -396,6 +401,7 @@ Optional<BibEntry> getEntryFromPDFContent(String firstpageContents, String lineS
}
}

arXivId = getArXivId(arXivId);
// start: title
fillCurStringWithNonEmptyLines();
title = streamlineTitle(curString);
Expand Down Expand Up @@ -515,6 +521,7 @@ Optional<BibEntry> getEntryFromPDFContent(String firstpageContents, String lineS
}
} else {
doi = getDoi(doi);
arXivId = getArXivId(arXivId);

if ((publisher == null) && curString.contains("IEEE")) {
// IEEE has the conference things at the end
Expand Down Expand Up @@ -565,13 +572,16 @@ Optional<BibEntry> getEntryFromPDFContent(String firstpageContents, String lineS
if (doi != null) {
entry.setField(StandardField.DOI, doi);
}
if (arXivId != null) {
entry.setField(StandardField.EPRINT, arXivId);
koppor marked this conversation as resolved.
Show resolved Hide resolved
}
if (series != null) {
entry.setField(StandardField.SERIES, series);
}
if (volume != null) {
entry.setField(StandardField.VOLUME, volume);
}
if (number != null) {
if (number != null && number.chars().allMatch(Character::isDigit)) {
koppor marked this conversation as resolved.
Show resolved Hide resolved
entry.setField(StandardField.NUMBER, number);
}
if (pages != null) {
Expand Down Expand Up @@ -600,6 +610,26 @@ private String getDoi(String doi) {
return doi;
}

private String getArXivId(String arXivId) {
if (arXivId != null) {
return arXivId;
}

String arXiv = curString.split(" ")[0];
arXivId = ArXivIdentifier.parse(arXiv).map(ArXivIdentifier::asString).orElse(null);

if (arXivId == null || curString.length() < arXivId.length() + ARXIV_PREFIX_LENGTH) {
return arXivId;
}
// The arxiv string also contains the year
curString = curString.substring(arXivId.length() + ARXIV_PREFIX_LENGTH);
extractYear();
curString = "";
proceedToNextNonEmptyLine();

return arXivId;
}

private String getFirstPageContents(PDDocument document) throws IOException {
PDFTextStripper stripper = new PDFTextStripper();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,43 @@ British Journal of Nutrition (2008), 99, 1–11 doi: 10.1017/S0007114507795296
assertEquals(Optional.of(entry), importer.getEntryFromPDFContent(firstPageContent, "\n", Optional.empty()));
}

@Test
void extractArXivFromPage1() {
BibEntry entry = new BibEntry(StandardEntryType.TechReport)
.withField(StandardField.AUTHOR, "Filippo Riccaa and Alessandro Marchettob and Andrea Stoccoc")
.withField(StandardField.TITLE, "A Multi-Year Grey Literature Review on AI-assisted Test Automation")
.withField(StandardField.YEAR, "2408")
.withField(StandardField.EPRINT, "2408.06224v1")
.withField((StandardField.KEYWORDS), "Test Automation Artificial Intelligence AI-assisted Test Automation Grey Literature Automated Test Generation Self-Healing Test Scripts");

String firstPageContent = """
A Multi-Year Grey Literature Review on AI-assisted Test Automation

Filippo Riccaa, Alessandro Marchettob and Andrea Stoccoc

aUniversity of Genoa, Via Balbi 5, Genova, 16126, Italy
bUniversity of Trento, Via Sommarive 9, Trento, 38123, Italy
cTechnical University of Munich, Boltzmannstraße 3, Munich, 85748, Germany
dfortiss GmbH, Guerickestraße 25, Munich, 80805, Germany

Keywords:
Test Automation
Artificial Intelligence
AI-assisted Test Automation
Grey Literature
Automated Test Generation
Self-Healing Test Scripts

*Corresponding author
filippo.ricca@unige.it (F. Ricca)
https://person.dibris.unige.it/ricca-filippo/ (F. Ricca)
ORCID(s): 0000-0002-3928-5408 (F. Ricca); 0000-0002-6833-896X (A. Marchetto); 0000-0001-8956-3894 (A. Stocco)

arXiv:2408.06224v1 [cs.SE] 12 Aug 2024""";

assertEquals(Optional.of(entry), importer.getEntryFromPDFContent(firstPageContent, "\n", Optional.empty()));
}

@ParameterizedTest
@MethodSource("providePdfData")
void pdfTitleExtraction(String expectedTitle, String filePath) throws Exception {
Expand Down
Loading