mjwestgate / revtools

Tools to support research synthesis in R
https://revtools.net
51 stars 26 forks source link

Screening apps reopen on new records #25

Open Stu-dem opened 5 years ago

Stu-dem commented 5 years ago

Thank you for making this package, I'm just getting into it and it is great so far! I just have a quick question on the screen_titles and screen_abstracts function in the revtools package.

Question: Does/should the shiny apps reopen a saved data_frame which already has colour, selected, notes, etc. columns at a new unscreened record? For example, if I screen 10 abstracts and then exit the app, the results save to a data_frame or .csv. If I reopen the app then the app opens up on the first record which has already been screened. Is this just for me? If the app does currently work like this, perhaps when exiting an app session a new order column could be generated that starts the order sequence from the next record with selected == "". The user can then choose to order the records by this column the next time they start the app.

mjwestgate commented 5 years ago

Thanks for getting in touch! This is a good point; I guess the key question here is: Should screen_titles and screen_abstracts display entries that have already been screened? The default has been to show them in different colours, but there is an argument that this wastes time and space.

I think the best solution would be to make this behaviour user-defined, by adding a 'show screened entries' dropdown. The default could be that screened entries are hidden, but the user could display them again if they wished. What do you reckon - is that a useful solution?

jsavinc commented 4 years ago

I've just come across this issue - I set the display order to random, screened 10% of the articles, quit, and then returned to screen more, whereby I now have randomly screened items here and there. Your suggestion of a dropdown for behaviour with previously screened titles/abstracts definitely makes sense!