pulibrary / bibdata
90%
main: 90%

LAST BUILD BRANCH: dependabot/bundler/net-ldap-0.20.0
DEFAULT BRANCH: main
Repo Added
Files 186
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH pub-citation
branch: pub-citation
CHANGE BRANCH
x

coverage: 89.888% (+0.01%) from 89.874%
9b61276a-eef8-4973-b470-f3a8699b5b69

Pull #3115

circleci

sandbergja 
Re-implement publication_citation_display in Rust

My first attempt was slower than the ruby version,
due to the overhead of re-parsing the MARC record
for every call to Rust.

However, by grouping all the publication data
together as a hash, we only need to experience that
overhead once per record, and we get a nice speedup:

```
require 'benchmark/ips'
record = MARC::XMLReader.new('spec/fixtures/marc_to_solr/9939238033506421.mrx').first
marc_breaker = MarcBreaker.break(record)
Benchmark.ips do |x|
  x.report("before") { set_pub_citation(record) && BibdataRs::Marc.publication_statements(marc_breaker) }
  x.report("after") { BibdataRs::Marc.publication_data(marc_breaker) }
end
```

```
ruby 3.4.4 (2025-05-14 revision a38531fd3f) +YJIT +PRISM [arm64-darwin24]
Warming up --------------------------------------
              before     2.837k i/100ms
Calculating -------------------------------------
              before     27.827k (± 2.0%) i/s   (35.94 μs/i) -    141.850k in   5.099585s
```

```
ruby 3.4.4 (2025-05-14 revision a38531fd3f) +YJIT +PRISM [arm64-darwin24]
Warming up --------------------------------------
               after     4.137k i/100ms
Calculating -------------------------------------
               after     41.399k (± 1.6%) i/s   (24.16 μs/i) -    210.987k in   5.097755s
```
Pull Request #3115: Re-implement publication_citation_display in Rust

80 of 90 new or added lines in 3 files covered. (88.89%)

11 existing lines in 1 file now uncovered.

9751 of 10848 relevant lines covered (89.89%)

299.67 hits per line

Relevant lines Covered
10848 RELEVANT LINES 9751 COVERED LINES
299.67 HITS PER LINE
Source Files on pub-citation

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
9b61276a... pub-citation Re-implement publication_citation_display in Rust My first attempt was slower than the ruby version, due to the overhead of re-parsing the MARC record for every call to Rust. However, by grouping all the publication data together as a hash, we o... Pull #3115 sandbergja circleci
89.89
997b7869... pub-citation Re-implement publication_citation_display in Rust My first attempt was slower than the ruby version, due to the overhead of re-parsing the MARC record for every call to Rust. However, by grouping all the publication data together as a hash, we o... Pull #3115 sandbergja circleci
89.89
See All Builds (3777)