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

Multiple suspects and assumed #42

Open
nrbiocom opened this issue Feb 11, 2013 · 4 comments
Open

Multiple suspects and assumed #42

nrbiocom opened this issue Feb 11, 2013 · 4 comments

Comments

@nrbiocom
Copy link

I know we have the ability to choose multiple suspects, but did the code that Ken was writing make it in or is this just gratuitous?

We can handle multiple assumed, yes?

@lukeis
Copy link
Contributor

lukeis commented Feb 11, 2013

multiple assumed basically removes the 'Unattributed' from the detected alleles. So, that is working as expected. Multiple suspects compounds them together in 'Suspected' to ken's program. I'm not certain how it handles that or if it needs to change, if it doesn't work as expected right now I can change it so you can only add one suspect.

@nrbiocom
Copy link
Author

Right now the code that derived from Ken does not accommodate multiple suspects. It is the piece he is working on, but we should not try to integrate it this week.

Please disable the ability to input multiple suspects for now. We'll look at integrating that piece of the code after the conference.

On Feb 11, 2013, at 9:38 AM, Luke Inman-Semerau wrote:

multiple assumed basically removes the 'Unattributed' from the detected alleles. So, that is working as expected. Multiple suspects compounds them together in 'Suspected' to ken's program. I'm not certain how it handles that or if it needs to change, if it doesn't work as expected right now I can change it so you can only add one suspect.


Reply to this email directly or view it on GitHub.

@lukeis
Copy link
Contributor

lukeis commented Feb 16, 2013

multiple suspects are disabled for now (1.1.0), one can only add multiple 'assumed'

@nrbiocom
Copy link
Author

implemented for Mac version at least

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

No branches or pull requests

2 participants