diff --git a/dhp-shade-package/pom.xml b/dhp-shade-package/pom.xml
index d8e17ed465..c4f9b262e9 100644
--- a/dhp-shade-package/pom.xml
+++ b/dhp-shade-package/pom.xml
@@ -26,16 +26,16 @@
-
- eu.dnetlib.dhp
- dhp-actionmanager
- ${project.version}
-
-
+
+
+ eu.dnetlib.dhp
+ dhp-aggregation
+ ${project.version}
+
@@ -56,61 +56,61 @@
-
- eu.dnetlib.dhp
- dhp-graph-mapper
- ${project.version}
-
-
- eu.dnetlib.dhp
- dhp-graph-provision
- ${project.version}
-
-
- eu.dnetlib.dhp
- dhp-impact-indicators
- ${project.version}
-
-
- eu.dnetlib.dhp
- dhp-stats-actionsets
- ${project.version}
-
-
- eu.dnetlib.dhp
- dhp-stats-hist-snaps
- ${project.version}
-
-
- eu.dnetlib.dhp
- dhp-stats-monitor-irish
- ${project.version}
-
-
- eu.dnetlib.dhp
- dhp-stats-promote
- ${project.version}
-
-
- eu.dnetlib.dhp
- dhp-stats-update
- ${project.version}
-
-
- eu.dnetlib.dhp
- dhp-swh
- ${project.version}
-
-
- eu.dnetlib.dhp
- dhp-usage-raw-data-update
- ${project.version}
-
-
- eu.dnetlib.dhp
- dhp-usage-stats-build
- ${project.version}
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/actionmanager/personentity/ExtractPerson.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/actionmanager/personentity/ExtractPerson.java
index 6976def4c8..05f0837402 100644
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/actionmanager/personentity/ExtractPerson.java
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/actionmanager/personentity/ExtractPerson.java
@@ -15,7 +15,7 @@ import java.util.stream.Collectors;
import org.apache.commons.cli.ParseException;
import org.apache.commons.io.IOUtils;
-import org.apache.commons.lang.StringUtils;
+import org.apache.commons.lang3.StringUtils;
import org.apache.hadoop.conf.Configuration;
import org.apache.hadoop.fs.FSDataOutputStream;
import org.apache.hadoop.fs.FileSystem;
@@ -30,7 +30,6 @@ import org.apache.spark.sql.Dataset;
import org.jetbrains.annotations.NotNull;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import org.spark_project.jetty.util.StringUtil;
import com.fasterxml.jackson.databind.ObjectMapper;
@@ -207,7 +206,7 @@ public class ExtractPerson implements Serializable {
null);
relation.setValidated(true);
- if (StringUtil.isNotBlank(role)) {
+ if (StringUtils.isNotBlank(role)) {
KeyValue kv = new KeyValue();
kv.setKey("role");
kv.setValue(role);
@@ -347,15 +346,19 @@ public class ExtractPerson implements Serializable {
.structuredProperty(
op.getOrcid(), ModelConstants.ORCID, ModelConstants.ORCID_CLASSNAME,
ModelConstants.DNET_PID_TYPES, ModelConstants.DNET_PID_TYPES,
- OafMapperUtils.dataInfo(false,
- null,
- false,
- false,
- OafMapperUtils.qualifier(ModelConstants.SYSIMPORT_CROSSWALK_ENTITYREGISTRY,
- ModelConstants.SYSIMPORT_CROSSWALK_ENTITYREGISTRY,
- ModelConstants.DNET_PID_TYPES,
- ModelConstants.DNET_PID_TYPES),
- "0.91")));
+ OafMapperUtils
+ .dataInfo(
+ false,
+ null,
+ false,
+ false,
+ OafMapperUtils
+ .qualifier(
+ ModelConstants.SYSIMPORT_CROSSWALK_ENTITYREGISTRY,
+ ModelConstants.SYSIMPORT_CROSSWALK_ENTITYREGISTRY,
+ ModelConstants.DNET_PID_TYPES,
+ ModelConstants.DNET_PID_TYPES),
+ "0.91")));
person.setDateofcollection(op.getLastModifiedDate());
person.setOriginalId(Arrays.asList(op.getOrcid()));
person.setDataInfo(ORCIDDATAINFO);
@@ -449,13 +452,13 @@ public class ExtractPerson implements Serializable {
null);
relation.setValidated(true);
- if (Optional.ofNullable(row.getStartDate()).isPresent() && StringUtil.isNotBlank(row.getStartDate())) {
+ if (Optional.ofNullable(row.getStartDate()).isPresent() && StringUtils.isNotBlank(row.getStartDate())) {
KeyValue kv = new KeyValue();
kv.setKey("startDate");
kv.setValue(row.getStartDate());
properties.add(kv);
}
- if (Optional.ofNullable(row.getEndDate()).isPresent() && StringUtil.isNotBlank(row.getEndDate())) {
+ if (Optional.ofNullable(row.getEndDate()).isPresent() && StringUtils.isNotBlank(row.getEndDate())) {
KeyValue kv = new KeyValue();
kv.setKey("endDate");
kv.setValue(row.getEndDate());
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/plugin/gtr2/Gtr2PublicationsIterator.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/plugin/gtr2/Gtr2PublicationsIterator.java
index 5b8f48680a..1b1ff8db43 100644
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/plugin/gtr2/Gtr2PublicationsIterator.java
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/plugin/gtr2/Gtr2PublicationsIterator.java
@@ -1,6 +1,9 @@
package eu.dnetlib.dhp.collection.plugin.gtr2;
+import java.nio.charset.StandardCharsets;
+import java.time.LocalDate;
+import java.time.format.DateTimeFormatter;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.Iterator;
@@ -16,9 +19,6 @@ import org.dom4j.Document;
import org.dom4j.DocumentException;
import org.dom4j.DocumentHelper;
import org.dom4j.Element;
-import org.joda.time.DateTime;
-import org.joda.time.format.DateTimeFormat;
-import org.joda.time.format.DateTimeFormatter;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -33,7 +33,7 @@ public class Gtr2PublicationsIterator implements Iterator {
private static final Logger log = LoggerFactory.getLogger(Gtr2PublicationsIterator.class);
private final HttpConnector2 connector;
- private static final DateTimeFormatter simpleDateTimeFormatter = DateTimeFormat.forPattern("yyyy-MM-dd");
+ private static final DateTimeFormatter simpleDateTimeFormatter = DateTimeFormatter.ofPattern("yyyy-MM-dd");
private static final int MAX_ATTEMPTS = 10;
@@ -41,7 +41,7 @@ public class Gtr2PublicationsIterator implements Iterator {
private int currPage;
private int endPage;
private boolean incremental = false;
- private DateTime fromDate;
+ private LocalDate fromDate;
private final Map cache = new HashMap<>();
@@ -188,28 +188,28 @@ public class Gtr2PublicationsIterator implements Iterator {
private Document loadURL(final String cleanUrl, final int attempt) {
try {
- log.debug(" * Downloading Url: " + cleanUrl);
- final byte[] bytes = this.connector.getInputSource(cleanUrl).getBytes("UTF-8");
+ log.debug(" * Downloading Url: {}", cleanUrl);
+ final byte[] bytes = this.connector.getInputSource(cleanUrl).getBytes(StandardCharsets.UTF_8);
return DocumentHelper.parseText(new String(bytes));
} catch (final Throwable e) {
- log.error("Error dowloading url: " + cleanUrl + ", attempt = " + attempt, e);
+ log.error("Error dowloading url: {}, attempt = {}", cleanUrl, attempt, e);
if (attempt >= MAX_ATTEMPTS) {
- throw new RuntimeException("Error dowloading url: " + cleanUrl, e);
+ throw new RuntimeException("Error downloading url: " + cleanUrl, e);
}
try {
Thread.sleep(60000); // I wait for a minute
} catch (final InterruptedException e1) {
- throw new RuntimeException("Error dowloading url: " + cleanUrl, e);
+ throw new RuntimeException("Error downloading url: " + cleanUrl, e);
}
return loadURL(cleanUrl, attempt + 1);
}
}
- private DateTime parseDate(final String s) {
- return DateTime.parse(s.contains("T") ? s.substring(0, s.indexOf("T")) : s, simpleDateTimeFormatter);
+ private LocalDate parseDate(final String s) {
+ return LocalDate.parse(s.contains("T") ? s.substring(0, s.indexOf("T")) : s, simpleDateTimeFormatter);
}
- private boolean isAfter(final String d, final DateTime fromDate) {
+ private boolean isAfter(final String d, final LocalDate fromDate) {
return StringUtils.isNotBlank(d) && parseDate(d).isAfter(fromDate);
}
}
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/sx/bio/pubmed/PMAffiliation.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/sx/bio/pubmed/PMAffiliation.java
new file mode 100644
index 0000000000..54aba87151
--- /dev/null
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/sx/bio/pubmed/PMAffiliation.java
@@ -0,0 +1,39 @@
+package eu.dnetlib.dhp.sx.bio.pubmed;
+
+/**
+ * The type Pubmed Affiliation.
+ *
+ * @author Sandro La Bruzzo
+ */
+public class PMAffiliation {
+
+ private String name;
+
+ private PMIdentifier identifier;
+
+ public PMAffiliation() {
+
+ }
+ public PMAffiliation(String name, PMIdentifier identifier) {
+ this.name = name;
+ this.identifier = identifier;
+ }
+
+ public String getName() {
+ return name;
+ }
+
+ public PMAffiliation setName(String name) {
+ this.name = name;
+ return this;
+ }
+
+ public PMIdentifier getIdentifier() {
+ return identifier;
+ }
+
+ public PMAffiliation setIdentifier(PMIdentifier identifier) {
+ this.identifier = identifier;
+ return this;
+ }
+}
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/sx/bio/pubmed/PMArticle.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/sx/bio/pubmed/PMArticle.java
index 3fb814606a..6191f6446b 100644
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/sx/bio/pubmed/PMArticle.java
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/sx/bio/pubmed/PMArticle.java
@@ -8,259 +8,115 @@ import java.util.List;
/**
* This class represent an instance of Pubmed Article extracted from the native XML
*
- * @author Sandro La Bruzzo
*/
-
public class PMArticle implements Serializable {
- /**
- * the Pubmed Identifier
- */
private String pmid;
-
private String pmcId;
-
- /**
- * the DOI
- */
private String doi;
- /**
- * the Pubmed Date extracted from Specifies a date significant to either the article's history or the citation's processing.
- * All dates will have a , , and elements. Some may have an , , and element(s).
- */
private String date;
- /**
- * This is an 'envelop' element that contains various elements describing the journal cited; i.e., ISSN, Volume, Issue, and PubDate and author name(s), however, it does not contain data itself.
- */
private PMJournal journal;
- /**
- * The full journal title (taken from NLM cataloging data following NLM rules for how to compile a serial name) is exported in this element. Some characters that are not part of the NLM MEDLINE/PubMed Character Set reside in a relatively small number of full journal titles. The NLM journal title abbreviation is exported in the element.
- */
private String title;
- /**
- * English-language abstracts are taken directly from the published article.
- * If the article does not have a published abstract, the National Library of Medicine does not create one,
- * thus the record lacks the and elements. However, in the absence of a formally
- * labeled abstract in the published article, text from a substantive "summary", "summary and conclusions" or "conclusions and summary" may be used.
- */
private String description;
- /**
- * the language in which an article was published is recorded in .
- * All entries are three letter abbreviations stored in lower case, such as eng, fre, ger, jpn, etc. When a single
- * record contains more than one language value the XML export program extracts the languages in alphabetic order by the 3-letter language value.
- * Some records provided by collaborating data producers may contain the value und to identify articles whose language is undetermined.
- */
private String language;
-
- /**
- * NLM controlled vocabulary, Medical Subject Headings (MeSH®), is used to characterize the content of the articles represented by MEDLINE citations. *
- */
- private final List subjects = new ArrayList<>();
- /**
- * This element is used to identify the type of article indexed for MEDLINE;
- * it characterizes the nature of the information or the manner in which it is conveyed as well as the type of
- * research support received (e.g., Review, Letter, Retracted Publication, Clinical Conference, Research Support, N.I.H., Extramural).
- */
- private final List publicationTypes = new ArrayList<>();
- /**
- * Personal and collective (corporate) author names published with the article are found in .
- */
+ private List subjects;
+ private List publicationTypes = new ArrayList<>();
private List authors = new ArrayList<>();
+ private List grants = new ArrayList<>();
- /**
- * contains the research grant or contract number (or both) that designates financial support by any agency of the United States Public Health Service
- * or any institute of the National Institutes of Health. Additionally, beginning in late 2005, grant numbers are included for many other US and non-US funding agencies and organizations.
- */
- private final List grants = new ArrayList<>();
-
- /**
- * get the DOI
- * @return a DOI
- */
- public String getDoi() {
- return doi;
- }
-
- /**
- * Set the DOI
- * @param doi a DOI
- */
- public void setDoi(String doi) {
- this.doi = doi;
- }
-
- /**
- * get the Pubmed Identifier
- * @return the PMID
- */
public String getPmid() {
return pmid;
}
- /**
- * set the Pubmed Identifier
- * @param pmid the Pubmed Identifier
- */
public void setPmid(String pmid) {
this.pmid = pmid;
}
- /**
- * the Pubmed Date extracted from Specifies a date significant to either the article's history or the citation's processing.
- * All dates will have a , , and elements. Some may have an , , and element(s).
- *
- * @return the Pubmed Date
- */
- public String getDate() {
- return date;
- }
-
- /**
- * Set the pubmed Date
- * @param date
- */
- public void setDate(String date) {
- this.date = date;
- }
-
- /**
- * The full journal title (taken from NLM cataloging data following NLM rules for how to compile a serial name) is exported in this element.
- * Some characters that are not part of the NLM MEDLINE/PubMed Character Set reside in a relatively small number of full journal titles.
- * The NLM journal title abbreviation is exported in the element.
- *
- * @return the pubmed Journal Extracted
- */
- public PMJournal getJournal() {
- return journal;
- }
-
- /**
- * Set the mapped pubmed Journal
- * @param journal
- */
- public void setJournal(PMJournal journal) {
- this.journal = journal;
- }
-
- /**
- * contains the entire title of the journal article. is always in English;
- * those titles originally published in a non-English language and translated for are enclosed in square brackets.
- * All titles end with a period unless another punctuation mark such as a question mark or bracket is present.
- * Explanatory information about the title itself is enclosed in parentheses, e.g.: (author's transl).
- * Corporate/collective authors may appear at the end of for citations up to about the year 2000.
- *
- * @return the extracted pubmed Title
- */
- public String getTitle() {
- return title;
- }
-
- /**
- * set the pubmed title
- * @param title
- */
- public void setTitle(String title) {
- this.title = title;
- }
-
- /**
- * English-language abstracts are taken directly from the published article.
- * If the article does not have a published abstract, the National Library of Medicine does not create one,
- * thus the record lacks the and elements. However, in the absence of a formally
- * labeled abstract in the published article, text from a substantive "summary", "summary and conclusions" or "conclusions and summary" may be used.
- *
- * @return the Mapped Pubmed Article Abstracts
- */
- public String getDescription() {
- return description;
- }
-
- /**
- * Set the Mapped Pubmed Article Abstracts
- * @param description
- */
- public void setDescription(String description) {
- this.description = description;
- }
-
- /**
- * Personal and collective (corporate) author names published with the article are found in .
- *
- * @return get the Mapped Authors lists
- */
- public List getAuthors() {
- return authors;
- }
-
- /**
- * Set the Mapped Authors lists
- * @param authors
- */
- public void setAuthors(List authors) {
- this.authors = authors;
- }
-
- /**
- * This element is used to identify the type of article indexed for MEDLINE;
- * it characterizes the nature of the information or the manner in which it is conveyed as well as the type of
- * research support received (e.g., Review, Letter, Retracted Publication, Clinical Conference, Research Support, N.I.H., Extramural).
- *
- * @return the mapped Subjects
- */
- public List getSubjects() {
- return subjects;
- }
-
- /**
- *
- * the language in which an article was published is recorded in .
- * All entries are three letter abbreviations stored in lower case, such as eng, fre, ger, jpn, etc. When a single
- * record contains more than one language value the XML export program extracts the languages in alphabetic order by the 3-letter language value.
- * Some records provided by collaborating data producers may contain the value und to identify articles whose language is undetermined.
- *
- * @return The mapped Language
- */
- public String getLanguage() {
- return language;
- }
-
- /**
- *
- * Set The mapped Language
- *
- * @param language the mapped Language
- */
- public void setLanguage(String language) {
- this.language = language;
- }
-
- /**
- * This element is used to identify the type of article indexed for MEDLINE;
- * it characterizes the nature of the information or the manner in which it is conveyed as well as the type of
- * research support received (e.g., Review, Letter, Retracted Publication, Clinical Conference, Research Support, N.I.H., Extramural).
- *
- * @return the mapped Publication Type
- */
- public List getPublicationTypes() {
- return publicationTypes;
- }
-
- /**
- * contains the research grant or contract number (or both) that designates financial support by any agency of the United States Public Health Service
- * or any institute of the National Institutes of Health. Additionally, beginning in late 2005, grant numbers are included for many other US and non-US funding agencies and organizations.
- * @return the mapped grants
- */
-
- public List getGrants() {
- return grants;
- }
-
public String getPmcId() {
return pmcId;
}
- public PMArticle setPmcId(String pmcId) {
+ public void setPmcId(String pmcId) {
this.pmcId = pmcId;
- return this;
+ }
+
+ public String getDoi() {
+ return doi;
+ }
+
+ public void setDoi(String doi) {
+ this.doi = doi;
+ }
+
+ public String getDate() {
+ return date;
+ }
+
+ public void setDate(String date) {
+ this.date = date;
+ }
+
+ public PMJournal getJournal() {
+ return journal;
+ }
+
+ public void setJournal(PMJournal journal) {
+ this.journal = journal;
+ }
+
+ public String getTitle() {
+ return title;
+ }
+
+ public void setTitle(String title) {
+ this.title = title;
+ }
+
+ public String getDescription() {
+ return description;
+ }
+
+ public void setDescription(String description) {
+ this.description = description;
+ }
+
+ public String getLanguage() {
+ return language;
+ }
+
+ public void setLanguage(String language) {
+ this.language = language;
+ }
+
+ public List getSubjects() {
+ return subjects;
+ }
+
+ public void setSubjects(List subjects) {
+ this.subjects = subjects;
+ }
+
+ public List getPublicationTypes() {
+ return publicationTypes;
+ }
+
+ public void setPublicationTypes(List publicationTypes) {
+ this.publicationTypes = publicationTypes;
+ }
+
+ public List getAuthors() {
+ return authors;
+ }
+
+ public void setAuthors(List authors) {
+ this.authors = authors;
+ }
+
+ public List getGrants() {
+ return grants;
+ }
+
+ public void setGrants(List grants) {
+ this.grants = grants;
}
}
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/sx/bio/pubmed/PMAuthor.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/sx/bio/pubmed/PMAuthor.java
index 68ef6459e0..b0df256634 100644
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/sx/bio/pubmed/PMAuthor.java
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/sx/bio/pubmed/PMAuthor.java
@@ -12,6 +12,8 @@ public class PMAuthor implements Serializable {
private String lastName;
private String foreName;
+ private PMIdentifier identifier;
+ private PMAffiliation affiliation;
/**
* Gets last name.
@@ -59,4 +61,41 @@ public class PMAuthor implements Serializable {
.format("%s, %s", this.foreName != null ? this.foreName : "", this.lastName != null ? this.lastName : "");
}
+ /**
+ * Gets identifier.
+ *
+ * @return the identifier
+ */
+ public PMIdentifier getIdentifier() {
+ return identifier;
+ }
+
+ /**
+ * Sets identifier.
+ *
+ * @param identifier the identifier
+ */
+ public void setIdentifier(PMIdentifier identifier) {
+ this.identifier = identifier;
+ }
+
+ /**
+ * Gets affiliation.
+ *
+ * @return the affiliation
+ */
+ public PMAffiliation getAffiliation() {
+ return affiliation;
+ }
+
+ /**
+ * Sets affiliation.
+ *
+ * @param affiliation the affiliation
+ */
+ public void setAffiliation(PMAffiliation affiliation) {
+ this.affiliation = affiliation;
+ }
+
+
}
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/sx/bio/pubmed/PMIdentifier.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/sx/bio/pubmed/PMIdentifier.java
new file mode 100644
index 0000000000..0c8c55e40e
--- /dev/null
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/sx/bio/pubmed/PMIdentifier.java
@@ -0,0 +1,53 @@
+package eu.dnetlib.dhp.sx.bio.pubmed;
+
+public class PMIdentifier {
+
+ private String pid;
+ private String type;
+
+
+ public PMIdentifier(String pid, String type) {
+ this.pid = cleanPid(pid);
+ this.type = type;
+ }
+
+ public PMIdentifier() {
+
+ }
+
+ private String cleanPid(String pid) {
+
+ if (pid == null) {
+ return null;
+ }
+
+ // clean ORCID ID in the form 0000000163025705 to 0000-0001-6302-5705
+ if (pid.matches("[0-9]{15}[0-9X]")) {
+ return pid.replaceAll("(.{4})(.{4})(.{4})(.{4})", "$1-$2-$3-$4");
+ }
+
+ // clean ORCID in the form http://orcid.org/0000-0001-8567-3543 to 0000-0001-8567-3543
+ if (pid.matches("http://orcid.org/[0-9]{4}-[0-9]{4}-[0-9]{4}-[0-9]{4}")) {
+ return pid.replaceAll("http://orcid.org/", "");
+ }
+ return pid;
+ }
+
+ public String getPid() {
+ return pid;
+ }
+
+ public PMIdentifier setPid(String pid) {
+ this.pid = cleanPid(pid);
+ return this;
+ }
+
+ public String getType() {
+ return type;
+ }
+
+ public PMIdentifier setType(String type) {
+ this.type = type;
+ return this;
+ }
+}
diff --git a/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/sx/bio/ebi/baseline_to_oaf_params.json b/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/sx/bio/ebi/baseline_to_oaf_params.json
index 3ba83764df..0fcc03266e 100644
--- a/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/sx/bio/ebi/baseline_to_oaf_params.json
+++ b/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/sx/bio/ebi/baseline_to_oaf_params.json
@@ -1,8 +1,7 @@
[
- {"paramName":"mt", "paramLongName":"master", "paramDescription": "should be local or yarn", "paramRequired": true},
- {"paramName":"i", "paramLongName":"isLookupUrl", "paramDescription": "isLookupUrl", "paramRequired": true},
- {"paramName":"w", "paramLongName":"workingPath", "paramDescription": "the path of the sequencial file to read", "paramRequired": true},
- {"paramName":"mo", "paramLongName":"mdstoreOutputVersion", "paramDescription": "the oaf path ", "paramRequired": true},
- {"paramName":"s", "paramLongName":"skipUpdate", "paramDescription": "skip update ", "paramRequired": false},
- {"paramName":"h", "paramLongName":"hdfsServerUri", "paramDescription": "the working path ", "paramRequired": true}
+ {"paramName":"mt", "paramLongName":"master", "paramDescription": "should be local or yarn", "paramRequired": true},
+ {"paramName":"i", "paramLongName":"isLookupUrl", "paramDescription": "isLookupUrl", "paramRequired": true},
+ {"paramName":"s", "paramLongName":"sourcePath", "paramDescription": "the baseline path", "paramRequired": true},
+ {"paramName":"mo", "paramLongName":"mdstoreOutputVersion", "paramDescription": "the mdstore path to save", "paramRequired": true}
+
]
\ No newline at end of file
diff --git a/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/sx/bio/pubmed/oozie_app/workflow.xml b/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/sx/bio/pubmed/oozie_app/workflow.xml
index 30eb414698..2a51b40509 100644
--- a/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/sx/bio/pubmed/oozie_app/workflow.xml
+++ b/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/sx/bio/pubmed/oozie_app/workflow.xml
@@ -1,4 +1,4 @@
-
+
baselineWorkingPath
@@ -16,11 +16,6 @@
mdStoreManagerURI
the path of the cleaned mdstore
-
- skipUpdate
- false
- The request block size
-
@@ -44,16 +39,16 @@
--mdStoreManagerURI${mdStoreManagerURI}
-
+
-
+
yarn
cluster
- Convert Baseline to OAF Dataset
- eu.dnetlib.dhp.sx.bio.ebi.SparkCreateBaselineDataFrame
+ Convert Baseline Pubmed to OAF Dataset
+ eu.dnetlib.dhp.sx.bio.ebi.SparkCreatePubmedDump
dhp-aggregation-${projectVersion}.jar
--executor-memory=${sparkExecutorMemory}
@@ -65,12 +60,10 @@
--conf spark.yarn.historyServer.address=${spark2YarnHistoryServerAddress}
--conf spark.eventLog.dir=${nameNode}${spark2EventLogDir}
- --workingPath${baselineWorkingPath}
+ --sourcePath${baselineWorkingPath}
--mdstoreOutputVersion${wf:actionData('StartTransaction')['mdStoreVersion']}
--masteryarn
--isLookupUrl${isLookupUrl}
- --hdfsServerUri${nameNode}
- --skipUpdate${skipUpdate}
diff --git a/dhp-workflows/dhp-aggregation/src/main/scala/eu/dnetlib/dhp/sx/bio/ebi/SparkCreatePubmedDump.scala b/dhp-workflows/dhp-aggregation/src/main/scala/eu/dnetlib/dhp/sx/bio/ebi/SparkCreatePubmedDump.scala
new file mode 100644
index 0000000000..1bdd2a4bcc
--- /dev/null
+++ b/dhp-workflows/dhp-aggregation/src/main/scala/eu/dnetlib/dhp/sx/bio/ebi/SparkCreatePubmedDump.scala
@@ -0,0 +1,97 @@
+package eu.dnetlib.dhp.sx.bio.ebi
+
+import com.fasterxml.jackson.databind.ObjectMapper
+import eu.dnetlib.dhp.application.AbstractScalaApplication
+import eu.dnetlib.dhp.common.vocabulary.VocabularyGroup
+import eu.dnetlib.dhp.schema.mdstore.MDStoreVersion
+import eu.dnetlib.dhp.sx.bio.pubmed.{PMArticle, PMParser2, PubMedToOaf}
+import eu.dnetlib.dhp.utils.ISLookupClientFactory
+import org.apache.spark.sql.{Encoder, Encoders, SparkSession}
+import org.slf4j.{Logger, LoggerFactory}
+
+class SparkCreatePubmedDump(propertyPath: String, args: Array[String], log: Logger)
+ extends AbstractScalaApplication(propertyPath, args, log: Logger) {
+
+ /** Here all the spark applications runs this method
+ * where the whole logic of the spark node is defined
+ */
+ override def run(): Unit = {
+ val isLookupUrl: String = parser.get("isLookupUrl")
+ log.info("isLookupUrl: {}", isLookupUrl)
+ val sourcePath = parser.get("sourcePath")
+ log.info(s"SourcePath is '$sourcePath'")
+ val mdstoreOutputVersion = parser.get("mdstoreOutputVersion")
+ log.info(s"mdstoreOutputVersion is '$mdstoreOutputVersion'")
+ val mapper = new ObjectMapper()
+ val cleanedMdStoreVersion = mapper.readValue(mdstoreOutputVersion, classOf[MDStoreVersion])
+ val outputBasePath = cleanedMdStoreVersion.getHdfsPath
+ log.info(s"outputBasePath is '$outputBasePath'")
+
+ val isLookupService = ISLookupClientFactory.getLookUpService(isLookupUrl)
+ val vocabularies = VocabularyGroup.loadVocsFromIS(isLookupService)
+
+ createPubmedDump(spark, sourcePath, outputBasePath, vocabularies)
+
+ }
+
+ /** This method creates a dump of the pubmed articles
+ * @param spark the spark session
+ * @param sourcePath the path of the source file
+ * @param targetPath the path of the target file
+ * @param vocabularies the vocabularies
+ */
+ def createPubmedDump(
+ spark: SparkSession,
+ sourcePath: String,
+ targetPath: String,
+ vocabularies: VocabularyGroup
+ ): Unit = {
+ require(spark != null)
+
+ implicit val PMEncoder: Encoder[PMArticle] = Encoders.bean(classOf[PMArticle])
+
+ import spark.implicits._
+ val df = spark.read.option("lineSep", "").text(sourcePath)
+ val mapper = new ObjectMapper()
+ df.as[String]
+ .map(s => {
+ val id = s.indexOf("")
+ if (id >= 0) s"${s.substring(id)}" else null
+ })
+ .filter(s => s != null)
+ .map { i =>
+ //remove try catch
+ try {
+ new PMParser2().parse(i)
+ } catch {
+ case _: Exception => {
+ throw new RuntimeException(s"Error parsing article: $i")
+ }
+ }
+ }
+ .dropDuplicates("pmid")
+ .map { a =>
+ val oaf = PubMedToOaf.convert(a, vocabularies)
+ if (oaf != null)
+ mapper.writeValueAsString(oaf)
+ else
+ null
+ }
+ .as[String]
+ .filter(s => s != null)
+ .write
+ .option("compression", "gzip")
+ .mode("overwrite")
+ .text(targetPath)
+ }
+}
+
+object SparkCreatePubmedDump {
+
+ def main(args: Array[String]): Unit = {
+ val log: Logger = LoggerFactory.getLogger(getClass)
+
+ new SparkCreatePubmedDump("/eu/dnetlib/dhp/sx/bio/ebi/baseline_to_oaf_params.json", args, log).initialize().run()
+
+ }
+}
diff --git a/dhp-workflows/dhp-aggregation/src/main/scala/eu/dnetlib/dhp/sx/bio/pubmed/PMParser2.scala b/dhp-workflows/dhp-aggregation/src/main/scala/eu/dnetlib/dhp/sx/bio/pubmed/PMParser2.scala
new file mode 100644
index 0000000000..2eb4bea65c
--- /dev/null
+++ b/dhp-workflows/dhp-aggregation/src/main/scala/eu/dnetlib/dhp/sx/bio/pubmed/PMParser2.scala
@@ -0,0 +1,276 @@
+package eu.dnetlib.dhp.sx.bio.pubmed
+
+import org.apache.commons.lang3.StringUtils
+
+import javax.xml.stream.XMLEventReader
+import scala.collection.JavaConverters._
+import scala.xml.{MetaData, NodeSeq}
+import scala.xml.pull.{EvElemEnd, EvElemStart, EvText}
+
+class PMParser2 {
+
+ /** Extracts the value of an attribute from a MetaData object.
+ * @param attrs the MetaData object
+ * @param key the key of the attribute
+ * @return the value of the attribute or null if the attribute is not found
+ */
+ private def extractAttributes(attrs: MetaData, key: String): String = {
+
+ val res = attrs.get(key)
+ if (res.isDefined) {
+ val s = res.get
+ if (s != null && s.nonEmpty)
+ s.head.text
+ else
+ null
+ } else null
+ }
+
+ /** Validates and formats a date given the year, month, and day as strings.
+ *
+ * @param year the year as a string
+ * @param month the month as a string
+ * @param day the day as a string
+ * @return the formatted date as "YYYY-MM-DD" or null if the date is invalid
+ */
+ private def validate_Date(year: String, month: String, day: String): String = {
+ try {
+ f"${year.toInt}-${month.toInt}%02d-${day.toInt}%02d"
+
+ } catch {
+ case _: Throwable => null
+ }
+ }
+
+ /** Extracts the grant information from a NodeSeq object.
+ *
+ * @param gNode the NodeSeq object
+ * @return the grant information or an empty list if the grant information is not found
+ */
+ private def extractGrant(gNode: NodeSeq): List[PMGrant] = {
+ gNode
+ .map(node => {
+ val grantId = (node \ "GrantID").text
+ val agency = (node \ "Agency").text
+ val country = (node \ "Country").text
+ new PMGrant(grantId, agency, country)
+ })
+ .toList
+ }
+
+ /** Extracts the journal information from a NodeSeq object.
+ *
+ * @param jNode the NodeSeq object
+ * @return the journal information or null if the journal information is not found
+ */
+ private def extractJournal(jNode: NodeSeq): PMJournal = {
+ val journal = new PMJournal
+ journal.setTitle((jNode \ "Title").text)
+ journal.setIssn((jNode \ "ISSN").text)
+ journal.setVolume((jNode \ "JournalIssue" \ "Volume").text)
+ journal.setIssue((jNode \ "JournalIssue" \ "Issue").text)
+ if (journal.getTitle != null && StringUtils.isNotEmpty(journal.getTitle))
+ journal
+ else
+ null
+ }
+
+ private def extractAuthors(aNode: NodeSeq): List[PMAuthor] = {
+ aNode
+ .map(author => {
+ val a = new PMAuthor
+ a.setLastName((author \ "LastName").text)
+ a.setForeName((author \ "ForeName").text)
+ val id = (author \ "Identifier").text
+ val idType =(author \ "Identifier" \ "@Source").text
+
+ if(id != null && id.nonEmpty && idType != null && idType.nonEmpty) {
+ a.setIdentifier(new PMIdentifier(id, idType))
+ }
+
+
+ val affiliation = (author \ "AffiliationInfo" \ "Affiliation").text
+ val affiliationId = (author \ "AffiliationInfo" \ "Identifier").text
+ val affiliationIdType = (author \ "AffiliationInfo" \ "Identifier" \ "@Source").text
+
+ if(affiliation != null && affiliation.nonEmpty) {
+ val aff = new PMAffiliation()
+ aff.setName(affiliation)
+ if(affiliationId != null && affiliationId.nonEmpty && affiliationIdType != null && affiliationIdType.nonEmpty) {
+ aff.setIdentifier(new PMIdentifier(affiliationId, affiliationIdType))
+ }
+ a.setAffiliation(aff)
+ }
+ a
+ })
+ .toList
+ }
+
+ def parse(input: String): PMArticle = {
+ val xml = scala.xml.XML.loadString(input)
+ val article = new PMArticle
+
+ val grantNodes = xml \ "MedlineCitation" \\ "Grant"
+ article.setGrants(extractGrant(grantNodes).asJava)
+
+ val journal = xml \ "MedlineCitation" \ "Article" \ "Journal"
+ article.setJournal(extractJournal(journal))
+
+ val authors = xml \ "MedlineCitation" \ "Article" \ "AuthorList" \ "Author"
+
+ article.setAuthors(
+ extractAuthors(authors).asJava
+ )
+
+ val pmId = xml \ "MedlineCitation" \ "PMID"
+
+ val articleIds = xml \ "PubmedData" \ "ArticleIdList" \ "ArticleId"
+ articleIds.foreach(articleId => {
+ val idType = (articleId \ "@IdType").text
+ val id = articleId.text
+ if ("doi".equalsIgnoreCase(idType)) article.setDoi(id)
+ if ("pmc".equalsIgnoreCase(idType)) article.setPmcId(id)
+ })
+ article.setPmid(pmId.text)
+
+ val pubMedPubDate = xml \ "MedlineCitation" \ "DateCompleted"
+ val currentDate =
+ validate_Date((pubMedPubDate \ "Year").text, (pubMedPubDate \ "Month").text, (pubMedPubDate \ "Day").text)
+ if (currentDate != null) article.setDate(currentDate)
+
+ val articleTitle = xml \ "MedlineCitation" \ "Article" \ "ArticleTitle"
+ article.setTitle(articleTitle.text)
+
+ val abstractText = xml \ "MedlineCitation" \ "Article" \ "Abstract" \ "AbstractText"
+ if (abstractText != null && abstractText.text != null && abstractText.text.nonEmpty)
+ article.setDescription(abstractText.text.split("\n").map(s => s.trim).mkString(" ").trim)
+
+ val language = xml \ "MedlineCitation" \ "Article" \ "Language"
+ article.setLanguage(language.text)
+
+ val subjects = xml \ "MedlineCitation" \ "MeshHeadingList" \ "MeshHeading"
+ article.setSubjects(
+ subjects
+ .take(20)
+ .map(subject => {
+ val descriptorName = (subject \ "DescriptorName").text
+ val ui = (subject \ "DescriptorName" \ "@UI").text
+ val s = new PMSubject
+ s.setValue(descriptorName)
+ s.setMeshId(ui)
+ s
+ })
+ .toList
+ .asJava
+ )
+ val publicationTypes = xml \ "MedlineCitation" \ "Article" \ "PublicationTypeList" \ "PublicationType"
+ article.setPublicationTypes(
+ publicationTypes
+ .map(pt => {
+ val s = new PMSubject
+ s.setValue(pt.text)
+ s
+ })
+ .toList
+ .asJava
+ )
+
+ article
+ }
+
+ def parse2(xml: XMLEventReader): PMArticle = {
+ var currentArticle: PMArticle = null
+ var currentSubject: PMSubject = null
+ var currentAuthor: PMAuthor = null
+ var currentJournal: PMJournal = null
+ var currentGrant: PMGrant = null
+ var currNode: String = null
+ var currentYear = "0"
+ var currentMonth = "01"
+ var currentDay = "01"
+ var currentArticleType: String = null
+
+ while (xml.hasNext) {
+ val ne = xml.next
+ ne match {
+ case EvElemStart(_, label, attrs, _) =>
+ currNode = label
+
+ label match {
+ case "PubmedArticle" => currentArticle = new PMArticle
+ case "Author" => currentAuthor = new PMAuthor
+ case "Journal" => currentJournal = new PMJournal
+ case "Grant" => currentGrant = new PMGrant
+ case "PublicationType" | "DescriptorName" =>
+ currentSubject = new PMSubject
+ currentSubject.setMeshId(extractAttributes(attrs, "UI"))
+ case "ArticleId" => currentArticleType = extractAttributes(attrs, "IdType")
+ case _ =>
+ }
+ case EvElemEnd(_, label) =>
+ label match {
+ case "PubmedArticle" => return currentArticle
+ case "Author" => currentArticle.getAuthors.add(currentAuthor)
+ case "Journal" => currentArticle.setJournal(currentJournal)
+ case "Grant" => currentArticle.getGrants.add(currentGrant)
+ case "PubMedPubDate" =>
+ if (currentArticle.getDate == null)
+ currentArticle.setDate(validate_Date(currentYear, currentMonth, currentDay))
+ case "PubDate" => currentJournal.setDate(s"$currentYear-$currentMonth-$currentDay")
+ case "DescriptorName" => currentArticle.getSubjects.add(currentSubject)
+ case "PublicationType" => currentArticle.getPublicationTypes.add(currentSubject)
+ case _ =>
+ }
+ case EvText(text) =>
+ if (currNode != null && text.trim.nonEmpty)
+ currNode match {
+ case "ArticleTitle" => {
+ if (currentArticle.getTitle == null)
+ currentArticle.setTitle(text.trim)
+ else
+ currentArticle.setTitle(currentArticle.getTitle + text.trim)
+ }
+ case "AbstractText" => {
+ if (currentArticle.getDescription == null)
+ currentArticle.setDescription(text.trim)
+ else
+ currentArticle.setDescription(currentArticle.getDescription + text.trim)
+ }
+ case "PMID" => currentArticle.setPmid(text.trim)
+ case "ArticleId" =>
+ if ("doi".equalsIgnoreCase(currentArticleType)) currentArticle.setDoi(text.trim)
+ if ("pmc".equalsIgnoreCase(currentArticleType)) currentArticle.setPmcId(text.trim)
+ case "Language" => currentArticle.setLanguage(text.trim)
+ case "ISSN" => currentJournal.setIssn(text.trim)
+ case "GrantID" => currentGrant.setGrantID(text.trim)
+ case "Agency" => currentGrant.setAgency(text.trim)
+ case "Country" => if (currentGrant != null) currentGrant.setCountry(text.trim)
+ case "Year" => currentYear = text.trim
+ case "Month" => currentMonth = text.trim
+ case "Day" => currentDay = text.trim
+ case "Volume" => currentJournal.setVolume(text.trim)
+ case "Issue" => currentJournal.setIssue(text.trim)
+ case "PublicationType" | "DescriptorName" => currentSubject.setValue(text.trim)
+ case "LastName" => {
+ if (currentAuthor != null)
+ currentAuthor.setLastName(text.trim)
+ }
+ case "ForeName" =>
+ if (currentAuthor != null)
+ currentAuthor.setForeName(text.trim)
+ case "Title" =>
+ if (currentJournal.getTitle == null)
+ currentJournal.setTitle(text.trim)
+ else
+ currentJournal.setTitle(currentJournal.getTitle + text.trim)
+ case _ =>
+
+ }
+ case _ =>
+ }
+
+ }
+ null
+ }
+
+}
diff --git a/dhp-workflows/dhp-aggregation/src/main/scala/eu/dnetlib/dhp/sx/bio/pubmed/PubMedToOaf.scala b/dhp-workflows/dhp-aggregation/src/main/scala/eu/dnetlib/dhp/sx/bio/pubmed/PubMedToOaf.scala
index d59d73bd05..5e14c731a6 100644
--- a/dhp-workflows/dhp-aggregation/src/main/scala/eu/dnetlib/dhp/sx/bio/pubmed/PubMedToOaf.scala
+++ b/dhp-workflows/dhp-aggregation/src/main/scala/eu/dnetlib/dhp/sx/bio/pubmed/PubMedToOaf.scala
@@ -294,6 +294,12 @@ object PubMedToOaf {
author.setName(a.getForeName)
author.setSurname(a.getLastName)
author.setFullname(a.getFullName)
+ if(a.getIdentifier != null) {
+ author.setPid(List(OafMapperUtils.structuredProperty(a.getIdentifier.getPid,
+ OafMapperUtils.qualifier(a.getIdentifier.getType,a.getIdentifier.getType,ModelConstants.DNET_PID_TYPES, ModelConstants.DNET_PID_TYPES), dataInfo)).asJava)
+ }
+ if (a.getAffiliation!= null)
+ author.setRawAffiliationString(List(a.getAffiliation.getName).asJava)
author.setRank(index + 1)
author
}(collection.breakOut)
diff --git a/dhp-workflows/dhp-aggregation/src/test/resources/eu/dnetlib/dhp/sx/graph/bio/single_pubmed.xml b/dhp-workflows/dhp-aggregation/src/test/resources/eu/dnetlib/dhp/sx/graph/bio/single_pubmed.xml
new file mode 100644
index 0000000000..c2e503f57e
--- /dev/null
+++ b/dhp-workflows/dhp-aggregation/src/test/resources/eu/dnetlib/dhp/sx/graph/bio/single_pubmed.xml
@@ -0,0 +1,157 @@
+
+
+ 37318999
+
+ 2024
+ 02
+ 09
+
+
+ 2024
+ 02
+ 09
+
+
+
+ 1522-1229
+
+ 47
+ 3
+
+ 2023
+ Sep
+ 01
+
+
+ Advances in physiology education
+ Adv Physiol Educ
+
+ Providing the choice of in-person or videoconference attendance in a clinical physiology course may harm learning outcomes for the entire cohort.
+
+ 548-556
+
+ 10.1152/advan.00160.2022
+
+ Clinical Physiology 1 and 2 are flipped classes in which students watch prerecorded videos before class. During the 3-h class, students take practice assessments, work in groups on critical thinking exercises, work through case studies, and engage in drawing exercises. Due to the COVID pandemic, these courses were transitioned from in-person classes to online classes. Despite the university's return-to-class policy, some students were reluctant to return to in-person classes; therefore during the 2021-2022 academic year, Clinical Physiology 1 and 2 were offered as flipped, hybrid courses. In a hybrid format, students either attended the synchronous class in person or online. Here we evaluate the learning outcomes and the perceptions of the learning experience for students who attended Clinical Physiology 1 and 2 either online (2020-2021) or in a hybrid format (2021-2022). In addition to exam scores, in-class surveys and end of course evaluations were compiled to describe the student experience in the flipped hybrid setting. Retrospective linear mixed-model regression analysis of exam scores revealed that a hybrid modality (2021-2022) was associated with lower exam scores when controlling for sex, graduate/undergraduate status, delivery method, and the order in which the courses were taken (F test: F = 8.65, df1 = 2, df2 = 179.28, P = 0.0003). In addition, being a Black Indigenous Person of Color (BIPOC) student is associated with a lower exam score, controlling for the same previous factors (F test: F = 4.23, df1 = 1, df2 = 130.28, P = 0.04), albeit with lower confidence; the BIPOC representation in this sample is small (BIPOC: n = 144; total: n = 504). There is no significant interaction between the hybrid modality and race, meaning that BIPOC and White students are both negatively affected in a hybrid flipped course. Instructors should consider carefully about offering hybrid courses and build in extra student support.NEW & NOTEWORTHY The transition from online to in-person teaching has been as challenging as the original transition to remote teaching with the onset of the pandemic. Since not all students were ready to return to the classroom, students could choose to take this course in person or online. This arrangement provided flexibility and opportunities for innovative class activities for students but introduced tradeoffs in lower test scores from the hybrid modality than fully online or fully in-person modalities.
+
+
+
+ Anderson
+ Lisa Carney
+ LC
+ 0000-0003-2261-1921
+
+ Department of Integrative Biology and Physiology, University of Minnesota, Minneapolis, Minnesota, United States.
+ https://ror.org/017zqws13
+
+
+
+ Jacobson
+ Tate
+ T
+
+ Department of Statistics, University of Minnesota, Minneapolis, Minnesota, United States.
+
+
+
+ eng
+
+ Journal Article
+
+
+ 2023
+ 06
+ 15
+
+
+
+ United States
+ Adv Physiol Educ
+ 100913944
+ 1043-4046
+
+ IM
+
+
+ Physiology
+ education
+
+
+ Retrospective Studies
+
+
+ Learning
+
+
+ Pandemics
+
+
+ COVID-19
+
+
+ Regression Analysis
+
+
+ Students
+
+
+ Humans
+
+
+ Male
+
+
+ Female
+
+
+ White People
+
+
+ Black People
+
+
+ Education, Distance
+
+
+ Curriculum
+
+
+
+ flipped teaching
+ hybrid teaching
+ inequity
+ learning outcomes
+ responsive teaching
+
+
+
+
+
+ 2023
+ 7
+ 21
+ 6
+ 44
+
+
+ 2023
+ 6
+ 15
+ 19
+ 14
+
+
+ 2023
+ 6
+ 15
+ 12
+ 53
+
+
+ ppublish
+
+ 37318999
+ 10.1152/advan.00160.2022
+
+
+
diff --git a/dhp-workflows/dhp-aggregation/src/test/scala/eu/dnetlib/dhp/sx/bio/BioScholixTest.scala b/dhp-workflows/dhp-aggregation/src/test/scala/eu/dnetlib/dhp/sx/bio/BioScholixTest.scala
index c4af14c409..4a926df015 100644
--- a/dhp-workflows/dhp-aggregation/src/test/scala/eu/dnetlib/dhp/sx/bio/BioScholixTest.scala
+++ b/dhp-workflows/dhp-aggregation/src/test/scala/eu/dnetlib/dhp/sx/bio/BioScholixTest.scala
@@ -5,7 +5,10 @@ import eu.dnetlib.dhp.aggregation.AbstractVocabularyTest
import eu.dnetlib.dhp.schema.oaf.utils.PidType
import eu.dnetlib.dhp.schema.oaf.{Oaf, Publication, Relation, Result}
import eu.dnetlib.dhp.sx.bio.BioDBToOAF.ScholixResolved
-import eu.dnetlib.dhp.sx.bio.pubmed.{PMArticle, PMParser, PMSubject, PubMedToOaf}
+import eu.dnetlib.dhp.sx.bio.ebi.SparkCreatePubmedDump
+import eu.dnetlib.dhp.sx.bio.pubmed._
+import org.apache.commons.io.IOUtils
+import org.apache.spark.sql.SparkSession
import org.json4s.DefaultFormats
import org.json4s.JsonAST.{JField, JObject, JString}
import org.json4s.jackson.JsonMethods.parse
@@ -13,14 +16,16 @@ import org.junit.jupiter.api.Assertions._
import org.junit.jupiter.api.extension.ExtendWith
import org.junit.jupiter.api.{BeforeEach, Test}
import org.mockito.junit.jupiter.MockitoExtension
+import org.slf4j.LoggerFactory
import java.io.{BufferedReader, InputStream, InputStreamReader}
+import java.util.regex.Pattern
import java.util.zip.GZIPInputStream
import javax.xml.stream.XMLInputFactory
import scala.collection.JavaConverters._
+import scala.collection.mutable
import scala.collection.mutable.ListBuffer
import scala.io.Source
-import scala.xml.pull.XMLEventReader
@ExtendWith(Array(classOf[MockitoExtension]))
class BioScholixTest extends AbstractVocabularyTest {
@@ -48,6 +53,77 @@ class BioScholixTest extends AbstractVocabularyTest {
}
}
+ @Test
+ def testPid(): Unit = {
+ val pids = List(
+ "0000000163025705",
+ "000000018494732X",
+ "0000000308873343",
+ "0000000335964515",
+ "0000000333457333",
+ "0000000335964515",
+ "0000000302921949",
+
+ "http://orcid.org/0000-0001-8567-3543",
+ "http://orcid.org/0000-0001-7868-8528",
+ "0000-0001-9189-1440",
+ "0000-0003-3727-9247",
+ "0000-0001-7246-1058",
+ "000000033962389X",
+ "0000000330371470",
+ "0000000171236123",
+ "0000000272569752",
+ "0000000293231371",
+ "http://orcid.org/0000-0003-3345-7333",
+ "0000000340145688",
+ "http://orcid.org/0000-0003-4894-1689"
+ )
+
+ pids.foreach(pid => {
+ val pidCleaned = new PMIdentifier(pid, "ORCID").getPid
+ // assert pid is in the format of ORCID
+ println(pidCleaned)
+ assertTrue(pidCleaned.matches("[0-9]{4}-[0-9]{4}-[0-9]{4}-[0-9]{3}[0-9X]"))
+ })
+ }
+
+ def extractAffiliation(s: String): List[String] = {
+ val regex: String = "(.*)<\\/Affiliation>"
+ val pattern = Pattern.compile(regex, Pattern.MULTILINE)
+ val matcher = pattern.matcher(s)
+ val l: mutable.ListBuffer[String] = mutable.ListBuffer()
+ while (matcher.find()) {
+ l += matcher.group(1)
+ }
+ l.toList
+ }
+
+ case class AuthorPID(pidType: String, pid: String) {}
+
+ def extractAuthorIdentifier(s: String): List[AuthorPID] = {
+ val regex: String = "(.*)<\\/Identifier>"
+ val pattern = Pattern.compile(regex, Pattern.MULTILINE)
+ val matcher = pattern.matcher(s)
+ val l: mutable.ListBuffer[AuthorPID] = mutable.ListBuffer()
+ while (matcher.find()) {
+ l += AuthorPID(pidType = matcher.group(1), pid = matcher.group(2))
+ }
+ l.toList
+ }
+
+ @Test
+ def testParsingPubmed2(): Unit = {
+ val mapper = new ObjectMapper()
+ val xml = IOUtils.toString(getClass.getResourceAsStream("/eu/dnetlib/dhp/sx/graph/bio/single_pubmed.xml"))
+ val parser = new PMParser2()
+ val article = parser.parse(xml)
+
+// println(mapper.writerWithDefaultPrettyPrinter().writeValueAsString(article))
+
+ println(mapper.writerWithDefaultPrettyPrinter().writeValueAsString(PubMedToOaf.convert(article, vocabularies)))
+
+ }
+
@Test
def testEBIData() = {
val inputFactory = XMLInputFactory.newInstance
@@ -124,6 +200,14 @@ class BioScholixTest extends AbstractVocabularyTest {
}
}
+ def testPubmedSplitting(): Unit = {
+
+ val spark: SparkSession = SparkSession.builder().appName("test").master("local").getOrCreate()
+ new SparkCreatePubmedDump("", Array.empty, LoggerFactory.getLogger(getClass))
+ .createPubmedDump(spark, "/home/sandro/Downloads/pubmed", "/home/sandro/Downloads/pubmed_mapped", vocabularies)
+
+ }
+
@Test
def testPubmedOriginalID(): Unit = {
val article: PMArticle = new PMArticle