cytoscape / idmapper

Identifier mapping
2 stars 1 forks source link

Result dialog #16

Closed AlexanderPico closed 8 years ago

AlexanderPico commented 8 years ago

This is looking great, just a minor point. In addition to listing when one-to-many is detected, we should also have a line that reflects the user choice to 'force single' or not. Otherwise, it seems like the selection was ignored and one-to-many results were included in new column.

AdamStuart commented 8 years ago

Do you mean in the dialog box that shows results, or in the header of the new column?

On Sep 15, 2016, at 12:52 PM, Alexander Pico notifications@github.com wrote:

This is looking great, just a minor point. In addition to listing when one-to-many is detected, we should also have a line that reflects the user choice to 'force single' or not. Otherwise, it seems like the selection was ignored and one-to-many results were included in new column.

AlexanderPico commented 8 years ago

the result. This one, for example, makes is seem like I got one-to-many results even though I checked "force single"

screen shot 2016-09-15 at 1 28 36 pm
AdamStuart commented 8 years ago

If "force single" is checked, change text: Not all mappings one-to-one: to Some mappings reduced to first value:

AdamStuart commented 8 years ago

MappingUtil.createMsg:281