Update citeproc-java to version 2.0.0 by michel-kraemer · Pull Request #5585 · JabRef/jabref

michel-kraemer · 2019-11-10T10:00:06Z

This pull request updates citeproc-java to version 2.0.0. The new version uses GraalVM JavaScript on Java 11 and higher, so it should be significantly faster than the older one. Also, the internal citeproc-js process has been updated to the latest version.

I needed to add a --add-exports argument to make the module com.oracle.truffle.regex export the package com.oracle.truffle.regex to org.graalvm.truffle. Otherwise, the polyglot engine was not able to find the regex language. This might be a bug in GraalVM (i.e. a missing exports statement in the org.graalvm.truffle module). I opened an issue for that (oracle/graal#1822).

I wasn't sure if I should amend the CHANGELOG or not. I can do this if you like me to.

All unit tests run successfully after this change.

Change in CHANGELOG.md described
Tests created for changes
Manually tested changed features in running JabRef
Screenshots added in PR description (for bigger UI changes)
Ensured that the git commit message is a good one
Check documentation status (Issue created for outdated help page at help.jabref.org?)

koppor · 2019-11-10T10:21:42Z

I think, this superseeds #2250. 🎉

@michel-kraemer Could you add some video similar to #2250 (comment) (I used https://www.screentogif.com/)?

tobiasdiez

Cool! Changes look good to me (except that I'm also not sure why commons.lang3 can be removed).

michel-kraemer · 2019-11-10T10:27:49Z

It seems commons.lang3 must be removed. Otherwise, the compiler complains that the module does not exist. I hope this does not affect anything else.

I'll try to create a video but the performance of V8 is still much better of course!

koppor · 2019-11-10T10:44:35Z

@michel-kraemer Thank you for working on this! Do we need to include graalvm in our build process somehow? See https://github.com/JabRef/jabref/blob/master/.github/workflows/deployment.yml

michel-kraemer · 2019-11-10T10:51:28Z

Here are the screen recordings:

citeproc-java 1.0.1 (about 8s for the first call)

citeproc-java 2.0.0 (about 2.5s for the first call)

michel-kraemer · 2019-11-10T10:53:02Z

@michel-kraemer Thank you for working on this! Do we need to include graalvm in our build process somehow? See https://github.com/JabRef/jabref/blob/master/.github/workflows/deployment.yml

@koppor I don't think so. It should work with both OpenJDK and GraalVM.

michel-kraemer · 2019-11-10T10:56:31Z

It's nice to see the videos in comparison. It's actually much faster. I didn't expect such a large difference! 😁

Siedlerchr · 2019-11-10T17:30:28Z

Wohoo! Thanks for the PR! This really looks nice and fast!
I still wonder why the commons lang was not found. Seems odd

4703b5d [Frontiers in ...] Remove formatting for volume (#5580) 3a3eb0c Create journal-of-clinical-and-translational-science.csl (#5588) cf08af8 Create clinical-pulmonary-medicine.csl (#5589) 98985b1 Create cancer-drug-resistance.csl (#5590) 0af101d Create american-journal-of-sociology.csl (#5591) b24e1f5 Create san-francisco-estuary-and-watershed-science.csl (#5592) 6f9323c revise Manchester UP monograph style (note-bib) (#5587) e6f982b Delete harper-adams (#5585) 1eef78f Create international-journal-of-clinical-research.csl (#5584) f28865e OSCOLA.csl: Add publiser info to generic/working paper secondary sources (CSL type "article"). (#5541) 197ceb1 Create system-dynamics-review.csl (#5575) 1d33da2 Create microcirculation.csl (#5577) d06f625 Create iawa-journal.csl (#5576) 4705585 Update for corrections (#5578) c1c7852 Create praxis.csl (#5579) e012054 Create European Review of Agricultural Economics style (#5567) b2fa912 Create biomarkers.csl (#5582) 1b00ec0 Create journal-of-international-studies.csl (#5574) 79a5d9c Switch MLA to 9th edition (#5561) 011fef3 Update for corrections (#5573) ff5490c Create zeitschrift-fur-geschichtsdidaktik.csl (#5569) 9093f2f Create etri-journal.csl (#5570) c9ba757 Create radiation-protection-dosimetry.csl (#5571) 60632bc Create horttechnology.csl (#5572) e63820b Add a new journal (magnetic resonance in medical sciences) (#5542) b656885 Create zeitschrift-fur-papyrologie-und-epigraphik.csl (#5548) 25e1eef Update technische-universitat-dresden-linguistik.csl (#5563) c0c0da4 Create the-american-journal-of-dermatopathology.csl (#5564) 63f7db9 Update journal-of-the-american-association-of-laboratory-animal-scien… (#5566) ba64225 Update institut-français-darcheologie-orientale-en.csl (#5553) 32e9031 Create institut-francais-darcheologie-orientale-domaine-des-etudes-arabes-en.csl (#5551) 23e2cf5 Create institut-francais-darcheologie-orientale-domaine-des-etudes-arabes.csl (#5550) 3e1a3f3 Update institut-français-darcheologie-orientale.csl (#5552) 176997d Update veterinary-pathology.csl (#5554) ee035c1 Update mycologia.csl (#5555) eb4b3a6 Create international-journal-of-automotive-technology.csl (#5557) 3c4e727 Bump addressable from 2.7.0 to 2.8.0 (#5558) ccb81c8 Create szociologiai-szemle.csl (#5547) ed9d740 Update brain.csl to ama.csl dependent (#5546) ab499f2 Create iran-manual-style (#5350) 6e2b13c Update and rename dependent/estuaries-and-coasts.csl to estuaries-and-coasts.csl (#5534) 8441ff9 Create fachhochschule-suedwestfalen.csl (#5459) 8439cf4 Create freie-hochschule-stuttgart.csl (#5466) 076c513 Update midwestern-baptist-theological-seminary.csl (#5420) 350b900 Update veterinary-pathology.csl (#5543) 06261c0 Create United States International Trade Commission.csl (#5508) 7ae505d Add support for blog posts (CSL: post-weblog). (#5536) 93c97ea Update tud bwl marketing style (#5538) 4308791 Create the-chinese-journal-of-international-politics.csl (#5537) 09f3362 Add hanging-indents to infoclio.ch styles (#5540) d4dc679 Further improvements to norsk-henvisningsstandad-for-rettsvitenskapelige-tekster.csl (#5539) git-subtree-dir: buildres/csl/csl-styles git-subtree-split: 4703b5d

* Squashed 'buildres/csl/csl-styles/' changes from ec4a4c0..4703b5d 4703b5d [Frontiers in ...] Remove formatting for volume (#5580) 3a3eb0c Create journal-of-clinical-and-translational-science.csl (#5588) cf08af8 Create clinical-pulmonary-medicine.csl (#5589) 98985b1 Create cancer-drug-resistance.csl (#5590) 0af101d Create american-journal-of-sociology.csl (#5591) b24e1f5 Create san-francisco-estuary-and-watershed-science.csl (#5592) 6f9323c revise Manchester UP monograph style (note-bib) (#5587) e6f982b Delete harper-adams (#5585) 1eef78f Create international-journal-of-clinical-research.csl (#5584) f28865e OSCOLA.csl: Add publiser info to generic/working paper secondary sources (CSL type "article"). (#5541) 197ceb1 Create system-dynamics-review.csl (#5575) 1d33da2 Create microcirculation.csl (#5577) d06f625 Create iawa-journal.csl (#5576) 4705585 Update for corrections (#5578) c1c7852 Create praxis.csl (#5579) e012054 Create European Review of Agricultural Economics style (#5567) b2fa912 Create biomarkers.csl (#5582) 1b00ec0 Create journal-of-international-studies.csl (#5574) 79a5d9c Switch MLA to 9th edition (#5561) 011fef3 Update for corrections (#5573) ff5490c Create zeitschrift-fur-geschichtsdidaktik.csl (#5569) 9093f2f Create etri-journal.csl (#5570) c9ba757 Create radiation-protection-dosimetry.csl (#5571) 60632bc Create horttechnology.csl (#5572) e63820b Add a new journal (magnetic resonance in medical sciences) (#5542) b656885 Create zeitschrift-fur-papyrologie-und-epigraphik.csl (#5548) 25e1eef Update technische-universitat-dresden-linguistik.csl (#5563) c0c0da4 Create the-american-journal-of-dermatopathology.csl (#5564) 63f7db9 Update journal-of-the-american-association-of-laboratory-animal-scien… (#5566) ba64225 Update institut-français-darcheologie-orientale-en.csl (#5553) 32e9031 Create institut-francais-darcheologie-orientale-domaine-des-etudes-arabes-en.csl (#5551) 23e2cf5 Create institut-francais-darcheologie-orientale-domaine-des-etudes-arabes.csl (#5550) 3e1a3f3 Update institut-français-darcheologie-orientale.csl (#5552) 176997d Update veterinary-pathology.csl (#5554) ee035c1 Update mycologia.csl (#5555) eb4b3a6 Create international-journal-of-automotive-technology.csl (#5557) 3c4e727 Bump addressable from 2.7.0 to 2.8.0 (#5558) ccb81c8 Create szociologiai-szemle.csl (#5547) ed9d740 Update brain.csl to ama.csl dependent (#5546) ab499f2 Create iran-manual-style (#5350) 6e2b13c Update and rename dependent/estuaries-and-coasts.csl to estuaries-and-coasts.csl (#5534) 8441ff9 Create fachhochschule-suedwestfalen.csl (#5459) 8439cf4 Create freie-hochschule-stuttgart.csl (#5466) 076c513 Update midwestern-baptist-theological-seminary.csl (#5420) 350b900 Update veterinary-pathology.csl (#5543) 06261c0 Create United States International Trade Commission.csl (#5508) 7ae505d Add support for blog posts (CSL: post-weblog). (#5536) 93c97ea Update tud bwl marketing style (#5538) 4308791 Create the-chinese-journal-of-international-politics.csl (#5537) 09f3362 Add hanging-indents to infoclio.ch styles (#5540) d4dc679 Further improvements to norsk-henvisningsstandad-for-rettsvitenskapelige-tekster.csl (#5539) git-subtree-dir: buildres/csl/csl-styles git-subtree-split: 4703b5d * Squashed 'buildres/csl/csl-locales/' changes from ec6d62a9e7..7a507fc008 7a507fc008 Fix spelling of October in Thai locale git-subtree-dir: buildres/csl/csl-locales git-subtree-split: 7a507fc0084edea8376ee5345879ddc2c99829ba Co-authored-by: github actions <jabrefmail+webfeedback@gmail.com>

2b37392 Create betriebswirtschaftliche-forschung-und-praxis.csl (#5609) a52989c Update universitat-zu-koln-seminar-fur-abwl-und-finanzierungslehre.csl (#5610) 47c21bf Add historioPLUS (#5611) baf11b4 Update urad-rs-za-makroekonomske-analize-in-razvoj.csl (#5607) 636affa Create pain-medicine.csl (#5606) 1d060f9 Update sist02.csl (#5568) 54e8904 Update ucl-university-college-apa.csl (#5597) 9c6308e Creation du style cahiers-du-journalisme.csl (#5595) 7dea54b Update universite-du-quebec-a-montreal.csl (#5596) b110d07 Update journal-of-paleontology.csl (#5594) 9cc3797 Add workaround for software style in ASA (sociology) (#5464) 459dfb7 Update american-journal-of-sociology.csl (#5604) f73d62f Create council-of-science-editors-brackets.csl (#5593) bacf673 Remove extra accent (#5598) 55e1d40 Update system-dynamics-review.csl (#5599) b8262e4 Update independent style for journal-of-cataract-and-refractive-surgery.csl (#5602) 04da306 Create quaderni-ministero-dei-beni-e-delle-attivita-culturali-e-del-t… (#5600) 8144588 Update journal-of-dental-research.csl (#5601) 4703b5d [Frontiers in ...] Remove formatting for volume (#5580) 3a3eb0c Create journal-of-clinical-and-translational-science.csl (#5588) cf08af8 Create clinical-pulmonary-medicine.csl (#5589) 98985b1 Create cancer-drug-resistance.csl (#5590) 0af101d Create american-journal-of-sociology.csl (#5591) b24e1f5 Create san-francisco-estuary-and-watershed-science.csl (#5592) 6f9323c revise Manchester UP monograph style (note-bib) (#5587) e6f982b Delete harper-adams (#5585) 1eef78f Create international-journal-of-clinical-research.csl (#5584) f28865e OSCOLA.csl: Add publiser info to generic/working paper secondary sources (CSL type "article"). (#5541) 197ceb1 Create system-dynamics-review.csl (#5575) 1d33da2 Create microcirculation.csl (#5577) d06f625 Create iawa-journal.csl (#5576) 4705585 Update for corrections (#5578) c1c7852 Create praxis.csl (#5579) e012054 Create European Review of Agricultural Economics style (#5567) b2fa912 Create biomarkers.csl (#5582) 1b00ec0 Create journal-of-international-studies.csl (#5574) 79a5d9c Switch MLA to 9th edition (#5561) 011fef3 Update for corrections (#5573) ff5490c Create zeitschrift-fur-geschichtsdidaktik.csl (#5569) 9093f2f Create etri-journal.csl (#5570) c9ba757 Create radiation-protection-dosimetry.csl (#5571) 60632bc Create horttechnology.csl (#5572) e63820b Add a new journal (magnetic resonance in medical sciences) (#5542) b656885 Create zeitschrift-fur-papyrologie-und-epigraphik.csl (#5548) 25e1eef Update technische-universitat-dresden-linguistik.csl (#5563) c0c0da4 Create the-american-journal-of-dermatopathology.csl (#5564) 63f7db9 Update journal-of-the-american-association-of-laboratory-animal-scien… (#5566) ba64225 Update institut-français-darcheologie-orientale-en.csl (#5553) 32e9031 Create institut-francais-darcheologie-orientale-domaine-des-etudes-arabes-en.csl (#5551) 23e2cf5 Create institut-francais-darcheologie-orientale-domaine-des-etudes-arabes.csl (#5550) 3e1a3f3 Update institut-français-darcheologie-orientale.csl (#5552) 176997d Update veterinary-pathology.csl (#5554) ee035c1 Update mycologia.csl (#5555) eb4b3a6 Create international-journal-of-automotive-technology.csl (#5557) 3c4e727 Bump addressable from 2.7.0 to 2.8.0 (#5558) ccb81c8 Create szociologiai-szemle.csl (#5547) ed9d740 Update brain.csl to ama.csl dependent (#5546) ab499f2 Create iran-manual-style (#5350) 6e2b13c Update and rename dependent/estuaries-and-coasts.csl to estuaries-and-coasts.csl (#5534) 8441ff9 Create fachhochschule-suedwestfalen.csl (#5459) 8439cf4 Create freie-hochschule-stuttgart.csl (#5466) 076c513 Update midwestern-baptist-theological-seminary.csl (#5420) 350b900 Update veterinary-pathology.csl (#5543) 06261c0 Create United States International Trade Commission.csl (#5508) 7ae505d Add support for blog posts (CSL: post-weblog). (#5536) 93c97ea Update tud bwl marketing style (#5538) 4308791 Create the-chinese-journal-of-international-politics.csl (#5537) 09f3362 Add hanging-indents to infoclio.ch styles (#5540) d4dc679 Further improvements to norsk-henvisningsstandad-for-rettsvitenskapelige-tekster.csl (#5539) git-subtree-dir: buildres/csl/csl-styles git-subtree-split: 2b37392

Update citeproc-java to version 2.0.0

09db570

koppor reviewed Nov 10, 2019

View reviewed changes

Comment thread src/main/java/module-info.java

tobiasdiez approved these changes Nov 10, 2019

View reviewed changes

tobiasdiez merged commit 1e48fd1 into JabRef:master Nov 10, 2019

michel-kraemer deleted the citeproc-java-2.0.0 branch November 10, 2019 10:53

This was referenced Nov 16, 2019

Add graalvm export also in eclipse gradle #5619

Merged

Entry preview: Error while generating citation style #5622

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Update citeproc-java to version 2.0.0#5585

Update citeproc-java to version 2.0.0#5585
tobiasdiez merged 1 commit into
JabRef:masterfrom
michel-kraemer:citeproc-java-2.0.0

michel-kraemer commented Nov 10, 2019

Uh oh!

koppor commented Nov 10, 2019 •

edited

Loading

Uh oh!

Uh oh!

tobiasdiez left a comment

Uh oh!

michel-kraemer commented Nov 10, 2019

Uh oh!

koppor commented Nov 10, 2019

Uh oh!

michel-kraemer commented Nov 10, 2019

Uh oh!

michel-kraemer commented Nov 10, 2019

Uh oh!

michel-kraemer commented Nov 10, 2019

Uh oh!

Siedlerchr commented Nov 10, 2019

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

michel-kraemer commented Nov 10, 2019

Uh oh!

koppor commented Nov 10, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

tobiasdiez left a comment

Choose a reason for hiding this comment

Uh oh!

michel-kraemer commented Nov 10, 2019

Uh oh!

koppor commented Nov 10, 2019

Uh oh!

michel-kraemer commented Nov 10, 2019

Uh oh!

michel-kraemer commented Nov 10, 2019

Uh oh!

michel-kraemer commented Nov 10, 2019

Uh oh!

Siedlerchr commented Nov 10, 2019

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

koppor commented Nov 10, 2019 •

edited

Loading