pombase / canto

The PomBase community curation tool
https://curation.pombase.org
Other
18 stars 7 forks source link

WT allelles #2832

Open ValWood opened 3 weeks ago

ValWood commented 3 weeks ago

We recently made it so that users could add WT alleles. Maybe we should make this 'admin only' because I just had a session with lots of WT[WT le el] alleles included which made them all muli gene phenotypes.

We should monitor if this recurs (there were quite a few in my session >20)

kimrutherford commented 3 weeks ago

We recently made it so that users could add WT alleles.

I thought users could always add WT alleles?

We changed the code recently to allow "Not assayed" for WT alleles: https://github.com/pombase/canto/issues/2818#issuecomment-2017093263

Is that what's happening?

ValWood commented 3 weeks ago

Yes you are correct, I think we have been able to use them since diploids were introduced. We should discuss in which placed WT alleles are appropriate and then add blocks and messaging to lead users to the correct way to curate in these instances.

The one I just removed was where the user had included the WT allele for the assayed gene. There are probably other cases. We will add to this ticket. It isn't super urgent.