geodes-sms / relis Goto Github PK
View Code? Open in Web Editor NEWTool for conducting systematic literature reviews and mapping studies
Home Page: http://relis.iro.umontreal.ca/
License: GNU General Public License v3.0
Tool for conducting systematic literature reviews and mapping studies
Home Page: http://relis.iro.umontreal.ca/
License: GNU General Public License v3.0
Describe the bug
When I go back to a paper that has been previously screened, I get an error on the top of the page. But it does not seem to affect the behavior of the application, so it is just annoying.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
There should be no error.
Screenshots
Here is the error:
A PHP Error was encountered
Severity: Warning
Message: count(): Parameter must be an array or an object that implements Countable
Filename: relis/Manager.php
Line Number: 4182
Backtrace:
File: /local/home/relis/public_html/relis_app/controllers/relis/Manager.php
Line: 4182
Function: _error_handler
File: /local/home/relis/public_html/index.php
Line: 315
Function: require_once
Currently, there is only support for exclusion criteria. We also need to have a way of specifying inclusion criteria. Therefore, when a paper is included, the reviewer must select a criteria as well (same behavior as for excluding a paper).
In the configuration, we should have a way to specify if the inclusion criteria must be selected when including a paper or if it is optional. Show the dropdown list, but allow an empty inclusion criteria to be selected by default.
This enhancement concerns the screening page https://github.com/geodes-sms/relis/blob/master/relis_app/views/relis/screen_paper.php
When screening a paper, the page shows a section (A) with all the paper information (title, pdf link, abstract, etc) and a section (B) with a form to make the decision of including or excluding the paper. Currently, section A is on top of section B. In full screen mode (not in @media), section B can be on the right of section A in order to reduce scrolling by the user.
Anywhere there is a list, e.g., Venues, Authors, Reference Categories, except Papers, it should be possible to merge values into one. For example, we have two authors, Homer Simpson and H Simpson. We want to merge both into Homer Simpson.
Describe the bug
If I create my classification with a list that is not mandatory (no *):
List hello "Hello" [1] = ["Salut", "Ola"]
The classification form does not allow me to leave the value empty. It forces me to choose one of the possible values.
Expected behavior
It should behave like for:
DynamicList hello "Hello" [1] = ["Salut", "Ola"]
which allows me to put an empty value.
Admin users are not shown in projects because they have access to all by default. However, if we want to explicitly have them in a project and assign them papers, then they should appear there in participants and assignments. For example, in project "User interaction and interface in MDE" there should be 2 participants, but I only see one.
There is a problem with the labels and the pages displayed of the reference categories.
In project User interaction and interface in MDE, the configuration had no default values for the hciModel and mdeModel dynamic lists. I see 3 reference categories in the menu: HCI model, hciModel and mdeModel. Should be "HCI model" and "MDE model".
In the configuration file, if I create a subcategory in the classification section, I cannot use it in the report section. We should be able to report any category or sub-category (or sub-sub...-category) defined.
Describe the bug
Multiple values not supported for Simple category in the classification scheme.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
I would expect to put multiple values. They can be separated by semi-colons ";"
Desktop (please complete the following information):
Describe the bug
Deleting a paper does not remove its screening assignments. The assignment is still there for screening. The project owner has to remove the assignments after having removed the papers. This may lead to inconsistent data.
Expected behavior
Deleting a paper should remove its assignments.
Describe the bug
@esyriani was able to create manual assignments as a Project admin in the same project. This right might be assigned to a system role rather than a project role.
Expected behavior
A Project admin should be able to create manual assignments.
Describe the bug
An error appears when I try to add a user to my project.
To Reproduce
Desktop (please complete the following information):
In the configuration file I create any category type (Simple, List, DynamicList) to be multi-valued (so writing [0]). In the classification form, I can only put a single value. How to add multiple values to simple categories? For example if is a string, int, boolean?
Also, if it is a single-valued category [1], the classification form automatically sets the default value to the first one. When it is a multi-value, it keeps it empty by default. It should be empty by default also for single values.
Describe the bug
After a fresh installation and creation of a new user in ReLiS. I logged in with the new User and tried the nav option "Homepage Settings" which lead me to the following error:
A Database Error Occurred
Error Number: 1305
PROCEDURE relisDB_001.get_list_info does not exist
CALL get_list_info(0,0,'' )
Filename: models/DBConnection_mdl.php
Line Number: 971
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Homepage Settings page should open up, or an appropriate message if this action is forbidden for non admin users.
Describe the bug
Accented characters are represented as latex commands in bibtex files. Relis simply strips the braces and backslashes.
To Reproduce
Import a bibtex entry such as this one
@phdthesis{logre2017preserver,
title={Pr{\'e}server la S{\'e}paration des Pr{\'e}occupations durant l'Int{\'e}gration de Domaines H{\'e}t{\'e}rog{\`e}nes dans les Syst{\`e}mes Logiciels.(Preserving Separation of Concern while Integrating Heterogeneous Domains in Software Systems).},
author={Logre, Ivan},
year={2017},
school={University of C{\^o}te d'Azur, France}
}
Expected behavior
The title is displayed as Préserver la Séparation des Préoccupations durant l'Intégration de Domaines Hétéogènes dans les Systèmes Logiciels.
Desktop:
Additional context
This happens on the instance hosted at http://relis.iro.umontreal.ca.
Describe the bug
The visualized ratio is calculated as (screened-conflics)/all
. (In both views: My progress and Overall progress.)
Expected behavior
The visualized ratio should be calculated as screened/all
.
Desktop (please complete the following information):
Describe the bug
I cannot login into the ReLiS instance because of a database error.
To Reproduce
Visit http://relis.iro.umontreal.ca/auth/login.
Expected behavior
The user is presented with the login form.
Screenshots
Describe the bug
Suppose I want to redeploy only the classification of a project. I then write keep_screening in the configuration file. However, when I reinstall the project, it removes all the screening statistics, all screened papers. I only see the imported papers, but it is like the screening phase has been reset.
Expected behavior
keep_screening should not modify anything in the screening phases.
Describe the bug
ReLiS is supposed to support iterative systematic reviews. Therefore, we should be able to change the projection configuration (for example the classification scheme). It should allow me to add new categories, delete exiting categories, or update the configuration of existing categories. However, currently, if I re-generate my configuration and re-deploy it, it erases all previous activities: screening, QA, and classification data. Only the papers, authors and venues remain.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
If we want to "reset" the project, then we can delete all papers (see #19). But modifying the configuration should not delete the previous information. It should try to modify what exists. Especially for the classification scheme. We should allow anything the database allows to modify. So if we are modifying a table that has rows, we let MySQL tell us it is allowed or not.
Describe the bug
Currently, a guest user can modify some entities in a project like:
To Reproduce
Log in a user with Guest role in a project.
Expected behavior
Everything should be read-only. It should be the equivalent of the Demo user but for a specific project.
Describe the bug
When I have a subcategory, the table only shows the top categories. The classification table should show all columns, including those of subcategories (recursively). Solving this should also solve the same problem when exporting the table to csv.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
I should see the following columns:
MDE-based | Tool support | Tool name | Degree of automation |
---|---|---|---|
Yes | Yes | The Tool | Automatic |
Desktop (please complete the following information):
Additional context
This is related to issue #18
Describe the bug
There is an unknown reference category in the classification phase.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
That category should not be there.
Desktop (please complete the following information):
Add the following built-in categories in the configuration editor:
Simple note "Note" [1] : string(500)
Simple year "Year" * [1] : date style("yyyy")
OR int(4) style("[1-2][0-9][0-9][0-9]")
DynamicList venue "Venue" [1] venues
DynamicList author "Author" * [0] authors
If present in the configuration, Year, Venue, and Author should be automatically set for paper classified with the data from the paper. Venue should be a reference to the table of Venues and Author to the table of Authors.
Describe the bug
When we go to a paper to view its history (e.g., log of screening), an error message appears at the top of the page. It is actually a problem on line 4182 in /relis_app/controllers/relis/Manager.php. The code is: if(count($value['val2']>1)){
but count takes a list, not a condition.
To Reproduce
Steps to reproduce the behavior:
Desktop (please complete the following information):
Migrate the production environment to docker. Make sure all existing data are preserved.
Describe the bug
I imported 166 references. I deleted them all. They do not show in the application. Now I try to add a new paper manually by doing
I checked in the database:
select * from paper
select * from paperauthor
select * from author
All the data is actually still there.
Expected behavior
All the papers and authors of deleted papers should be deleted from the database.
Desktop (please complete the following information):
Describe the bug
When we go to assign page to assign paper to a user the redirected page shows error, when executed in localhost.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Assigning paper page to appear.
Desktop (please complete the following information):
Describe the bug
The classification result table does not show the data I entered when I did the classification of a paper.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
It should show all the data I entered. If you click on the paper, you will see the data is there.
Desktop (please complete the following information):
Additional context
Add any other context about the problem here.
Currently, the online database is full of spam users.
Update the empty sections of the readme file.
Describe the bug
Using the installation script currently provided, there is the info table missing. The defintion resides within initial_db.sql but is missing at initial_values.sql (which is the one used by installation script as far as I saw).
Related to #11
Describe the bug
I first assign a paper to a reviewer for screening. Then I remove the paper. The paper still appears in the the list of the reviewer to screen.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
In ReLiS, deleting a paper doesn't actually remove it, but deactivates it. When a paper is deactivated, it should not appear anymore in any list: screening, statistics, classification, authors, venues, etc.
Screenshots
For example, you can go to the project Language Composition. Paper Qiao2020_3 is deactivated but still shows up in Naima's screening.
Actually, if you run the following query:
SELECT count(*)
FROM paper
INNER JOIN screening_paper ON screening_paper.paper_id = paper.id
WHERE paper.paper_active=0 and screening_paper.screening_active=1
You should always get 0, but here you get 22. When paper_active=0, screening_active should also =0. I think you must also have classification.class_active=0. The views in the statistics should also take the active status into consideration.
Describe the bug
My classification scheme as a subcategory:
Simple tool "Tool support" * [1] : bool {
Simple tool_name "Tool name" : string
}
When I classify a paper, I can set the values of the subcategory. But when I save it it is not stored.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
The values of subcategories (recursively) should be saved.
Screenshots
Desktop (please complete the following information):
Additional context
This is related to issue #21
Describe the bug
When screening articles, the link to the PDF of the article is wrongly prefixed by http://relis.iro.umontreal.ca/relis/manager/
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Should go the URL entered in the article link.
Desktop (please complete the following information):
When a paper is pending classification assigned to the currently logged user, it is not possible to reassign the paper to another user via the Edit assignment button. An error is produced. A workaround is to add a new assignment and then delete the current one.
Clicking on "Edit the Assignment" produces the following error:
An uncaught Exception was encountered
Type: ArgumentCountError
Message: Too few arguments to function Op::edit_element(), 1 passed in /local/home/relis/public_html/relis_sys/core/CodeIgniter.php on line 532 and at least 2 expected
Filename: /local/home/relis/public_html/relis_app/controllers/Op.php
Line Number: 2164
Backtrace:
File: /local/home/relis/public_html/index.php
Line: 315
Function: require_once
After a fresh install, I create a new project and generate a config file. The list of config files generated does not appear to install the project. It probably is an access right issue.
Describe the bug
The link to the paper on the screen_paper
view is broken.
The generated HTML is the following
<a href="//http://dx.doi.org/10.1145/3180155.3180250" target="black" style="color:#aa7777;">
<img src="http://relis.iro.umontreal.ca/cside/images/pdf.jpg">
</a>
The issue is that for some reason the URL is prefixed by //
.
Other views like display_paper_screen
do not exhibit this issue.
Expected behavior
The generated URL is http://dx.doi.org/10.1145/3180155.3180250
.
Describe the bug
Any user can delete the picture for other users or any project as long as he has the id of the user or project.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
At least if user b has less permissions then the admin user, it schould not be able to delete the picture and some error should be shown. And clearly it should not be possible to delete the picture.
It is currently possible to create a project with only classification (no screening).
However, it is not possible to classify papers that are not screened.
Currently I need to create a dummy screening phase, include all papers, then assign them to the classification phase.
Describe the bug
I try to enter the a paper manually by BibTeX. It reports an internal error.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Should add the paper with no errors.
Desktop (please complete the following information):
Additional context
Add any other context about the problem here.
Currently, keep_screening
preserves all the screening data when regenerating the project. However it deletes the QA data and classification data. keep_screening
should also preserve the QA data. This is defined when the configuration file is updated.
Describe the bug
If you use the relis php script for setup (install/index2.php), an error occurs after first bootstrap of ReLiS.
A PHP Error was encountered
Severity: Warning
Message: touch(): Unable to create file C:/xampp/htdocs/relis/relis_dev/cside/sessions/relis_sessionbsegq89r7kabj6u6rra975mpk7f15884 because No such file or directory
Filename: drivers/Session_files_driver.php
Line Number: 248
To Reproduce
Steps to reproduce the behavior:
Expected behavior
the directory to store sessions should not be hardcoded and set by install routine automatically.
Screenshots
If applicable, add screenshots to help explain your problem.
Server(please complete the following information):
Additional context
I saw this error while trying to migrate the mysql engine to use mysqli all the time.
An error occurs while creating a new project with a quality assurance step :
A PHP Error was encountered
severity: Notice
Message; Undefined index : qa_action
Filename: controllers/install.php
Line Number: 1697
However the project is created successfully
Steps to reproduce the behavior:
A project must be created without errors
Is your feature request related to a problem? Please describe.
I wanted to provide a collaborator with a quick link to the Conflict's page of our project. The URL, however, does not allow this, because it is missing the project ID. If either of us has more than two projects, this, in principle, cannot be achieved by the current URL structure.
Describe the solution you'd like
Instead of this: http://relis.iro.umontreal.ca/op/entity_list/list_papers_screen_conflict.html, I would like this: http://relis.iro.umontreal.ca/[projectID]/screening/conflicts.html.
A REST-compatible URL would be nice.
ReLiS allows to store the query string and title/abstracts of all uploaded papers. During screening, it would be very beneficial if, on the screening page of each paper, the keywords of the query string are highlighted in the title and abstract. This will greatly improve the time needed to decide including or excluding a paper. Of course, punctuation (e.g., '"' or '( )') and operators (e.g., AND, OR, NOT, NEAR) should be ignored.
Create a documentation of the code.
Given a systematic review that was conducted in ReLiS, the goal is to generate automatically a draft document of the protocol of that review.
An initial prototype of this tool by Lucas Hornung is available here http://www-ens.iro.umontreal.ca/~syriani/files/LucasHornung.zip. But it relies on Eclipse, Ecore, and Java. The idea is to replicate and extend this code in PHP. For code generation (generating the report), we can use https://github.com/divengine/div
Describe the bug
After I classify papers, I have an error when I try to see the results: table or charts.
Expected behavior
I should see the results without errors.
Detailed error
This is the error I get for any new project I create.
A Database Error Occurred
Error Number: 1146
Table 'relis_new_2_hci_mde.view_classification_paper' doesn't exist
CALL get_list_classification(0,0,'' )
Filename: models/DBConnection_mdl.php
Line Number: 951
A declarative, efficient, and flexible JavaScript library for building user interfaces.
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google ❤️ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.