Skip to content

[ENH] OWSelectByDataIndex: New widget (move from Prototypes)#3181

Merged
lanzagar merged 4 commits intobiolab:masterfrom
ajdapretnar:select-by-data-index
Aug 3, 2018
Merged

[ENH] OWSelectByDataIndex: New widget (move from Prototypes)#3181
lanzagar merged 4 commits intobiolab:masterfrom
ajdapretnar:select-by-data-index

Conversation

@ajdapretnar
Copy link
Copy Markdown
Contributor

Issue

Select by Data Index widget from Prototypes ready to be moved to core.

Description of changes

Move widget.

Includes
  • Code changes
  • Tests
  • Documentation

@ajdapretnar ajdapretnar requested a review from BlazZupan August 2, 2018 09:41

Now we will select an interesting subset (we could also select the entire data set). If we observe it in a **Data Table**, we can see that the data is transformed. If we would like to see this data with the original features, we will have to retrieve them with **Select by Data Index**.

Connect the original data and the subset from **Scatter Plot** to **Select by Data Index**. The widget will match the indices of the subset with the indices of the reference (original) data and put the matching reference data on the output. A final inspection in another **Data Table** confirms the data on the output is not from the original data space.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

put the matching reference data on the output -> output the matching reference data
output is not from the original data space. -> IS from ...?

@lanzagar lanzagar changed the title OWSelectByDataIndex: move from Prototypes to core [ENH] OWSelectByDataIndex: New widget (move from Prototypes) Aug 2, 2018
@ajdapretnar ajdapretnar force-pushed the select-by-data-index branch from 1f9d7c1 to a8b515e Compare August 2, 2018 10:33
@ajdapretnar ajdapretnar force-pushed the select-by-data-index branch from a8b515e to a8072fb Compare August 2, 2018 12:06
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #3181 into master will increase coverage by 0.01%.
The diff coverage is 89.01%.

@@            Coverage Diff             @@
##           master    #3181      +/-   ##
==========================================
+ Coverage   82.54%   82.55%   +0.01%     
==========================================
  Files         337      339       +2     
  Lines       58431    58522      +91     
==========================================
+ Hits        48229    48310      +81     
- Misses      10202    10212      +10

@lanzagar lanzagar added this to the 3.15 milestone Aug 3, 2018
@lanzagar lanzagar merged commit 3d97116 into biolab:master Aug 3, 2018
@ajdapretnar ajdapretnar deleted the select-by-data-index branch November 30, 2018 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants