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

Check for allele with same description when entering in Canto #2841

Open
ValWood opened this issue Jun 26, 2024 · 23 comments
Open

Check for allele with same description when entering in Canto #2841

ValWood opened this issue Jun 26, 2024 · 23 comments

Comments

@ValWood
Copy link
Member

ValWood commented Jun 26, 2024

Since we still get new additions into the chado_checks.duplicate_allele_descriptions log

https://curation.pombase.org/dumps/latest_build/logs/log.2024-06-25-21-01-00.chado_checks.duplicate_allele_descriptions

And, since this is going to cause problems once the permanent allele IDs are implemented, we should make a check in Canto so that we cannot add an existing allele with a new name (prompt, did you want to use the existing allele "allele with the same description".

That way, even if we DO need to change a name, we will be alerted before we add it and you try to load it.

@kimrutherford
Copy link
Member

@ValWood ValWood changed the title Check for allee with same descrtiption when entering in Canto Check for allee with same description when entering in Canto Jun 27, 2024
@ValWood ValWood changed the title Check for allee with same description when entering in Canto Check for allele with same description when entering in Canto Jun 27, 2024
@ValWood
Copy link
Member Author

ValWood commented Jul 1, 2024

needed before allele pages

@ValWood
Copy link
Member Author

ValWood commented Jul 2, 2024

This will be implemented as a check to make sure that the allele is not existing with a different name already.

Exampes:
swi5-39

@kimrutherford
Copy link
Member

I've implemented this such that it checks for duplicates as you type. It wasn't any extra effort. The "OK" will be greyed-out if the description is a duplicate.

I've updated the test Canto with this change. Could you do a quick test of the allele dialog when you get a chance? Also, please let me know if the wording of the messages needs improving.

https://curation.pombase.org/test/curs/4666975359de04dd/genotype_manage

no_dup_allele_desc

@ValWood
Copy link
Member Author

ValWood commented Jul 10, 2024

It works, but I think we shoud change the messaging.

In this instance I did a new name, and then a description, and get:

Screenshot 2024-07-10 at 11 40 38

@ValWood

This comment was marked as outdated.

@kimrutherford

This comment was marked as outdated.

@kimrutherford

This comment was marked as outdated.

@ValWood
Copy link
Member Author

ValWood commented Jul 10, 2024

In this scenario, I added the allele description first , which I often do, and got the message before I added a name:
Screenshot 2024-07-10 at 11 41 38

@ValWood

This comment was marked as outdated.

@kimrutherford

This comment was marked as outdated.

@PCarme
Copy link

PCarme commented Jul 12, 2024

The check does not seem to work when renaming an allele in "Edit genotype details".
For example here, I could rename the allele as pcf1-PIP*
image
although there is already an allele with the same description in Canto/PomBase, called pcf1-PIPmut.
image

For context, the allele was called pcf1-PIPmut when I started approving the session, I changed it to pcf1-PIP* to fit the naming used in the paper without realising that the author used an existing allele in his curation.

@ValWood
Copy link
Member Author

ValWood commented Jul 16, 2024

Message:
Add your name as a synonym for existing allele (blah) ? [Add]

@pombase pombase deleted a comment from kimrutherford Jul 16, 2024
@PCarme
Copy link

PCarme commented Jul 16, 2024

The check does not seem to work when renaming an allele in "Edit genotype details". For example here, I could rename the allele as pcf1-PIP*

For context, the allele was called pcf1-PIPmut when I started approving the session, I changed it to pcf1-PIP* to fit the naming used in the paper without realising that the author used an existing allele in his curation.

After checking, this also happened on the test Canto (see the screenshot where I could bypass the check and create two alleles with the same description).
image

@kimrutherford
Copy link
Member

Hi @PCarme

Thanks for testing. I can't work out how to reproduce the bug you found. I'll get you to show me how to do it on the next call.

kimrutherford added a commit that referenced this issue Jul 22, 2024
In the allele dialog, allow the discription to be changed and then
edited back to the starting value.

Refs #2841
@kimrutherford
Copy link
Member

Hi @PCarme

I think I've fixed the bug with copy-and-edit. But I'm not sure about the other problem you demonstrated. When you have time could you try the test Canto again? Thanks.

https://curation.pombase.org/test/curs/4666975359de04dd/genotype_manage

@PCarme
Copy link

PCarme commented Jul 24, 2024

Hi @kimrutherford
The bug with the copy-and-edit is fixed.
The other bug is still there though. To reproduce it, you have to create the existing allele in the session, then you can change its name while still keeping the same description via the "Edit genotype details" action.

@kimrutherford
Copy link
Member

Hi Pascal. I'm having trouble reproducing the name change problem. Could you give me a demo on the next call?

@PCarme
Copy link

PCarme commented Jul 30, 2024

Sure, no problem !

@kimrutherford
Copy link
Member

kimrutherford commented Jul 31, 2024

The cause of this problem might be that we can edit the details of existing alleles when we go back to the Genotype edit page.

good

bad

@PCarme
Copy link

PCarme commented Jul 31, 2024

I think it is good to be able to edit existing alleles, but only to change the new synonyms and expression status.

@kimrutherford
Copy link
Member

I think it is good to be able to edit existing alleles, but only to change the new synonyms and expression status.

Good point. I'll have a think about how that could work.

@kimrutherford
Copy link
Member

The cause of this problem might be that we can edit the details of existing alleles when we go back to the Genotype edit page.

I had forgotten the details about this. The current Canto release has bugs when dealing with existing alleles. These problems are fixed in the new version for the stable allele identifiers project.

When we switch over to the new allele system the bugs should be fixed.

I'll leave this open until the next Zoom to remind us to have a chat about it.

We (I) need to get on with implementing the new system!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

No branches or pull requests

3 participants