research-software-directory / RSD-as-a-service

This repo contains the new RSD-as-a-service implementation
https://research.software
27 stars 14 forks source link

fix: software mentions counts on homepage #1329

Closed dmijatovic closed 3 weeks ago

dmijatovic commented 4 weeks ago

Improve mention count on homepage

Closes #1326

Changes proposed in this pull request:

How to test:

Example software overview

image

PR Checklist:

jmaassen commented 3 weeks ago

Works as expected for the test cases described above.

I also tested manually adding a mention to software1 that was also found automatically. This isn't counted twice, which is correct.

When also adding this same mention manually to software2 it is counted. This is also correct.

However, if I use the same reference paper for both software 1 and software 2, the citations are only counted once. I don't think this is correct? I agree this is an artificial example though. A more realistic one is the following:

AMBER has 10.1016/j.softx.2020.100549 as one of it's reference papers. This is cited 13 times.

KernelTuner has 10.1016/j.future.2018.08.004 as one of it reference papers. This is cited 64 times.

Because these are different pieces of software, the total citation count should be 13+64 = 77. However, the RSD only reports 75, because some citations are coming from the same source. For example this book chapter 10.1007/978-3-031-69577-3_7 cites both AMBER and KernelTuner. It should count as 2, but is now only counted as 1.

dmijatovic commented 3 weeks ago

@jmaassen I have adjusted RPC to count on unique combination of mention and software id's. Using this approach we count one paper multiple times if it references more than one RSD software entry. Can you test it again, please?

sonarcloud[bot] commented 3 weeks ago

Quality Gate Passed Quality Gate passed for 'rsd-database'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud