Skip to content
New issue

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

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

Already on GitHub? Sign in 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants