You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
A user can enable vep / exomiser via the --tools parameter. We would like to replace this option by 2 separated boolean options (ex: --with-vep --with-exomiser).
We think that it will reduce the complexity of the code, as, with the tools parameter, we need to add extra logic to parse the value. With dedicated boolean parameters, the logic to determine whether or not vep/exomiser applies is more straightforward and easier to read in the code.
Using high priority as, ideally, we would like to do it before qlin deployment since there is a breaking change in the input/output interface. Such change will be harder to do when we have a pipeline deployed in production.
The text was updated successfully, but these errors were encountered:
LysianeBouchard
changed the title
Use separated boolean parameters for vep and exomiser
Use separated boolean parameters for vep and exomiser - MEDIUM PRIORITY
Oct 1, 2024
LysianeBouchard
changed the title
Use separated boolean parameters for vep and exomiser - MEDIUM PRIORITY
Use separated boolean parameters for vep and exomiser - HIGH PRIORITY
Oct 1, 2024
Description of feature
A user can enable vep / exomiser via the --tools parameter. We would like to replace this option by 2 separated boolean options (ex: --with-vep --with-exomiser).
We think that it will reduce the complexity of the code, as, with the tools parameter, we need to add extra logic to parse the value. With dedicated boolean parameters, the logic to determine whether or not vep/exomiser applies is more straightforward and easier to read in the code.
Using high priority as, ideally, we would like to do it before qlin deployment since there is a breaking change in the input/output interface. Such change will be harder to do when we have a pipeline deployed in production.
The text was updated successfully, but these errors were encountered: