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

[validation] some bugs and improvements #373

Closed
21 of 23 tasks
antonylebechec opened this issue Apr 11, 2022 · 3 comments
Closed
21 of 23 tasks

[validation] some bugs and improvements #373

antonylebechec opened this issue Apr 11, 2022 · 3 comments
Labels
bug Something isn't working core Cutevariant core CRITICAL really dangerous bug plugin related to a plugin

Comments

@antonylebechec
Copy link
Collaborator

antonylebechec commented Apr 11, 2022

Some bugs and improvements:

Traceback (most recent call last):
  File "/Users/lebechea/BIOINFO/git/cutevariant/cutevariant/gui/plugins/validation/widgets.py", line 626, in contextMenuEvent
    self.current_variant["chr"]
KeyError: 'chr'
  • Ficon is not defined
Traceback (most recent call last):
  File "/Users/lebechea/BIOINFO/git/cutevariant/cutevariant/gui/plugins/validation/widgets.py", line 187, in data
    return QIcon(Ficon(0xF139A)).toPixmap(20, 20)
NameError: name 'Ficon' is not defined
  • Ensure icons are clearly defined

    • icons for Lock/Unlock/Rejected sample
    • icon's colors related to validation on variant
  • Ensure than validation is locked for locked samples and no genotype variant

  • List of columns not selected and not visible and not checkable (MacOs)

image

  • Select fields to display close to presets, at the right

  • Add quick selection of samples/genotype in the main window (only on "sample_has_variant" values?)
    As example :

    • selection of samples with the selected variant associated to specific genotypes (to help user to quickly list samples with genotype)
      image

    • selection of samples with the selected variant validated (to help user to quickly list samples with validation)
      image

    • selection of samples with the selected variant tagged with a specific tag (to help user to quickly list tagged validation)
      image

  • Contextual menu > variant validation not locked when no genotype for a variant

image

  • Search samples with pattern
    image image

  • Selection with a list of samples (separated by space or comma, allow both, especially to be able to copy/paste from a spreadsheet)

  • Selection with a SampleSheet (Illumina format)

  • Selection with tags on samples that are not preset (e.g. added during import)

  • Search list with icons (and text?)

image

  • Columns in list of samples more usable

    • Order by on columns
    • Selection of columns (or all columns), such as sex, father, mother, hpo, others...
    image
  • button "Add to selection" and "remove selection(s)" and "clear all" more visible

image

image

  • Columns in selected samples

image

  • Contextual menu to open Edit each samples (either in "All samples" and "X Selected sample(s)" tabs)

  • List of selected samples at the bottom of the window (after the samples' table and before "Cancel" / "OK" button), for each tabs. The list will be just a non editable text field, with samples separated by a space or comma, with a limitation (no more X samples shown, user can list all of them with details in the "X Selected sample(s)" tab)

  • Double click to add samples (one by one)

  • Facilitate source creation and selection (PR 395)

  • Validation of multiple sample_has_variant at the time (issue [validation] Validate a variant for multiple samples at the time impossible #395)

@antonylebechec antonylebechec added bug Something isn't working core Cutevariant core CRITICAL really dangerous bug plugin related to a plugin labels Apr 11, 2022
antonylebechec added a commit that referenced this issue Apr 12, 2022
@dridk
Copy link
Member

dridk commented Apr 20, 2022

  • add all samples button (select all )
  • Add validation source button

@SamuelNicaise
Copy link
Collaborator

Ensure than validation is locked for locked samples and no genotype variant

should be avoided in case a biologist wants to validate a variant that was undetected by callers

@antonylebechec
Copy link
Collaborator Author

Most of items are solved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working core Cutevariant core CRITICAL really dangerous bug plugin related to a plugin
Projects
None yet
Development

No branches or pull requests

3 participants