Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[WIP] Enhance slr feature #559

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

[WIP] Enhance slr feature #559

wants to merge 8 commits into from

Conversation

DominikVoigt
Copy link
Collaborator

@DominikVoigt DominikVoigt commented Apr 19, 2022

The goal of this PR is to improve the SLR feature.
Most of it based on: https://discourse.jabref.org/t/systematic-review-tool-feedback/2791

Current todos

  • Remove search date from config file
  • Make use of lower case name for fetchers in config file/repository structure without killing persistence
  • Enable the usage of library-specific query refinements in the config file and model <- Before this, test creating a new SLR and continuing an existing SLR
  • Rename library-specific query refinements to refinements
  • Use raw query string for fetchers (internally)
    • Add ADR: fetcher use raw data (this) OR fetcher use transformers
    • Fetchers get passed the query string from transformers or get the raw query passed through
    • transformers have to be found automatically <- Implement WebFetchers to provide the correct transformer for each fetcher

  • Provide fetchers these refined queries raw
  • Add UI for this extendable queries idea
  • Add modification UI for SLR definition
  • add option in UI to use "lax mode" (allow raw queries)

  • Provide a tutorial for the process and differences between strict and lax mode
    • Extend the pool of translatable query languages

Additional topics

  • Add checking for parseable/transformable queries
  • Add methods for fetching that signal (Mainly used for UI):
    • whether the query could be transformed or was passed plain
    • whether an id fetcher was used instead -> introduce the corresponding layer between fetchers and UI/SLR

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants