skipped Tests: Chronology

StabiBerlin / sbb-relevance-test

Specs for relevance of stabikat search

MIT License

3 stars 1 forks source link

skipped Tests: Chronology #35

Closed duncdrum closed 9 months ago

duncdrum commented 10 months ago

Andreas Gryphus has many relevant 21st century publication, but the top 20 is 17 of 20 before 1985

annekauf commented 9 months ago

not able to find relevant publications with publication date after 2020 - which is the test assertion (1st hit published after 2020). I'd like to adapt the test assertion.

duncdrum commented 9 months ago

Using chronological sorting in an allFields or Titel seach I see (random selection):

https://stabikat.de/Record/info:doi:10.1017%252Frqx.2022.198
https://stabikat.de/Record/1795232064 (in Title and in Subject) 2020
https://stabikat.de/Record/1747791318
https://stabikat.de/Record/1665420421 (Conference volume about Gryphus) 2020

I m therefore hesitant to change the test assertion, from what i can see these hits should appear in the TOP 20. How could a volume from the 70s or 80s be that much more relevant just based on the ranking config, and without taking something like citation index into account?

annekauf commented 9 months ago

About half of those examples are published in 2020. Could test be amended to "1st hit should be published after 2019"? - Have just seen the exact code: .contains(/202\d/). This means first hit could be published IN 2020, right? Then test doesn't need to be changed.

annekauf commented 9 months ago

Chronology test - therefore test assertion "should contain both primary and secondary literature" for search terms "jugendliteratur mittelalter roman" doesn't compute. Or put it this way: this assertion hasn't been put as a ranking goal. Tbd.

annekauf commented 9 months ago

Similar: assertion "should show relevant titles in translation" for search all fields "griechische mythologie". Tbd, please. :)

duncdrum commented 9 months ago

.contains(/202\d/) means 202[0-9] any digit will work