PHI-base / curation

PHI-base curation
0 stars 0 forks source link

PMID:31804478 An effector protein of the wheat stripe rust fungus targets chloroplasts and suppresses chloroplast function. #66

Closed CuzickA closed 1 year ago

CuzickA commented 4 years ago

curation link https://canto.phi-base.org/curs/cc6cf06675cc6e13

CuzickA commented 4 years ago

@ValWood Please can you check this one next thanks.

ValWood commented 4 years ago

I did this. I have a few questions for tomorrow.

CuzickA commented 4 years ago

After today's discussion controls useful for this type of paper AC new term to make Future to discuss conditions/delivery mechanism please continue checking unchecked annotations @ValWood

ValWood commented 4 years ago

checked this.All looks fine to me.

ValWood commented 4 years ago

I removed the discuss label because I think you can do everything else for this one at your end. Let me know if there is any outstanding question.

ValWood commented 4 years ago

I wanted to add occurs_in host chloroplast for MF term in : curs/cc6cf06675cc6e13 Maybe this isn't allowed yet in the PomBase configuration?

CuzickA commented 4 years ago

I wanted to add occurs_in host chloroplast for MF term in : curs/cc6cf06675cc6e13

I have added this now.

CuzickA commented 4 years ago

@ValWood we still need to talk through the conditions required for this paper. I will not be able to approve the session until the appropriate conditions have been added to PHI-ECO and hooked up in PHI-Canto. https://github.com/PHI-base/curation/issues/69

CuzickA commented 4 years ago

Note to self- have added updated conditions. few queries and a final check through to do before approval.

CuzickA commented 4 years ago

Session approved. Closing ticket.

CuzickA commented 4 years ago

This session may need looking at again with reference to the recent reorganisation of the PHI-branch terms eg effector-mediated....

CuzickA commented 3 years ago

Added GFP backgrounds and compared to control AE image

Also added RNA level image

CuzickA commented 3 years ago

Several cases where control metagenotypes cannot be curated, mainly due to being expts with an 'empty vector'.

Only able to add AE disease interaction to a couple of annotations.

Other queries to follow later in this ticket.

CuzickA commented 3 years ago

Hi @ValWood, I have some queries for this session 1) In the past we had a discussion about using 'overexpression' vs 'not assayed' (I think for the atr1/ rpp1 paper). I couldn't quite remember what was decided. This session currently uses 'overexpression' a lot and I'm not sure if it is correct eg for constructs within a Y2H image or constructs delivered via agrobacterium image In these cases the point of the expt is not comparing phenotypes between WT expression vs overexpression, but are methods of producing and delivering the protein within the given assay.

2) image a) are we still happy with making 2 annotations to capture this information eg image image b) would it be clearer to add present / absent to term? effector-mediated suppression of host effector triggered immunity present effector-mediated suppression of host effector triggered immunity absent

Then the following annotation could also have the second part image

3) It is a bit confusing that the host gene is called PetC by UniProt and TaISP in the paper. Currently the genotypes have been renamed with 'TaISP' but the other annotations including the AE assayed_using use 'PetC'. is there a way to make it clear that they are synonyms?

4) image This HIGS metagenotype was a bit tricky to follow and I have made a note in the comments. However, I now think that it is fine as it is so I will delete the note. The end result is a pathogen with a silenced gene. I think it would be good to add this to our Additional tips documentation (renamed from Phenotype tips https://docs.google.com/document/d/1Nm5yAHMwFYfhjRWuatlGA4_X2aSK9l12QQI1ffQXFqs/edit).

ValWood commented 3 years ago

Correction I changed to GO:0140403 | effector-mediated suppression of host innate immune response by symbiont (because this doesn't seem to be affecting the PTI signalling, more the response?)

ValWood commented 3 years ago

I did not underrstand this comment "Should this have output annotation for 'effector-mediated blah... absent'??"

Skype me if you have time to explain. v

ValWood commented 3 years ago

Yes, I think it should be " effector-mediated suppression of host PAMP-triggered immunity present" so I assume the label should change here?

Also with the 2 hybid, because it isn't an in vivo the expression level is not really relevant. The experiment is only checking if the interaction occurs or not with a specific allele, so yes- it would be not assayed

CuzickA commented 3 years ago

Responses to queries above

1) changed overexpression to not assayed 2) term edits NTRs created image 3) pre-fixed allele names with PetC-TaISPblah to make clear 4) fine as is, needs documenting in Additional tips

Waiting for release and loading of PHIPO terms, connect them up and then session ready for approval

CuzickA commented 3 years ago

Updated PHIPO terms. Session approved. Closing ticket.

CuzickA commented 2 years ago

Hi @ValWood I'm working up some examples for the PHI-Canto manuscript. In this annotation I was wondering whether the GO BP term should be 'GO:0052034 effector-mediated suppression of host pattern-triggered immunity' as the effector is suppressing the host callose defense triggered by the addition of a PTI inducer

image image

CuzickA commented 2 years ago

As a follow up question

Is it okay to have different 'effector-mediated' terms as 'part_of' AE for the GO MF? Perhaps we did this as the PTI inducer was not added to these expt.

image

ValWood commented 2 years ago

I was wondering whether the GO BP term should be 'GO:0052034 effector-mediated suppression of host pattern-triggered immunity' as the effector is suppressing the host callose defense triggered by the addition of a PTI inducer

It seems so. It should be a descendant of the term used currently, but this would be more specific.

ValWood commented 2 years ago

I was wondering whether the GO BP term should be 'GO:0052034 effector-mediated suppression of host pattern-triggered immunity' as the effector is suppressing the host callose defense triggered by the addition of a PTI inducer

Maybe this term could be more specific too? I think part of the problem might be that some terms did not exist when we began (but also we can only go as specific as the paper provides evidence for).

This is why in PomBase we only show the most specific annotation by default. But uses can see experimental annotations to less specific parents in the 'detailed view'.

CuzickA commented 2 years ago

I have changed this annotation to the more specific child term, as a PTI inducer was used in the Fig 3a expt image

Should these MF 'part_of' BP terms also be made more specific to be the same as the initial BP annotation? Or should they be left as they are? Do the MF AE BP terms 'have' to be the same as the original BP annotation term? image

CuzickA commented 2 years ago

Multiple annotations have the bug where the compared to control AE is not automatically updated with the edit to the metagenotype. The 'overexpression' needs changing to 'not assayed'. I have manually done this for 7 annotations and changed 'WT product' to 'WT level' for 4 annotations image to image

CuzickA commented 1 year ago

Updated to

image

The BP annotation in the MF part_of BP should be to the same term