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

update of interaction types #58

Open
egonw opened this issue Feb 24, 2019 · 3 comments
Open

update of interaction types #58

egonw opened this issue Feb 24, 2019 · 3 comments
Assignees
Labels

Comments

@egonw
Copy link
Member

egonw commented Feb 24, 2019

Can you please have a look at this code and tell me if we can/should update this:

	private static Resource getLineTypeResource(LineType lt) {
		if(lt.equals(LineType.ARROW)) {
			return null;
		} else if (lt.equals(LineType.TBAR)) {
			return Wp.Inhibition;
		} else if (lt.equals(MIMShapes.MIM_CATALYSIS)) {
			return Wp.Catalysis;
		} else if (lt.equals(MIMShapes.MIM_CONVERSION)) {
			return Wp.Conversion;
		} else if (lt.equals(MIMShapes.MIM_INHIBITION)) {
			return Wp.Inhibition;
		} else if (lt.equals(MIMShapes.MIM_STIMULATION)) {
			return Wp.Stimulation;
		} else if (lt.equals(MIMShapes.MIM_TRANSLATION)) {
			return Wp.TranscriptionTranslation;
		} else if (lt.equals(MIMShapes.MIM_NECESSARY_STIMULATION)) {
			System.out.println("TODO: necessary stimulation");
		} else if (lt.equals(MIMShapes.MIM_MODIFICATION)) {
			System.out.println("TODO: modification");
		} else {
			System.out.println("Unknown and unsupported interaction type: " + lt);
		}
		return null;
	}

I think we might have to. This is what I find in the output of the RDF generation

778x "Unknown and unsupported interaction type: mim-binding"
30x "Unknown and unsupported interaction type: mim-cleavage"
9x "Unknown and unsupported interaction type: ReceptorSquare"
6x "Unknown and unsupported interaction type: Receptor"
@egonw
Copy link
Member Author

egonw commented May 13, 2019

@DeniseSl22, @mkutmon, what does your outline of interactions have for these three mim types?

  1. mim-cleavage
  2. MIM_NECESSARY_STIMULATION
  3. MIM_MODIFICATION

Was there a reason why that one was missing in the current WikiPathways Ontology/Vocabulary?

@DeniseSl22
Copy link
Contributor

Hi @egonw , @mkutmon and @RyAMiller ,

I think we should leave the current GPML2RDF conversion for the existing MIM-types in PWs.
We will not have a new interaction panel in the next PV release, because then we would need to do (major) updates to the WPs website as well (and it's better to do this for the new dev site, than for the old one)....

For the unsupported ones: the ReceptorSquare and Receptor should not be used; could you make a unit test for these in Jenkins, so we can find them and change them to whatever it should mean?
For the MIM-binding and cleavage, these do exist and people are allowed to use them.
In the new panel, I'm proposing the following:
Mim-cleavage: becomes "Conversion"
Mim-necessary-stimulation: becomes "Stimulation"
mim-modification: become "Modification" .

And may I draw your attention to the latest addition of the MIM panel, translocation? This was originally mentioned in the paper describing the MIMs in PV, but never implemented. It is part of the code now (and will be part of PV in the next release @mkutmon and me are working on).

Hope this gives @egonw enough info to go in, if not let me know!

@DeniseSl22
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants