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

In the PSM object the score should be plural #68

Closed
ypriverol opened this issue Apr 22, 2020 · 7 comments
Closed

In the PSM object the score should be plural #68

ypriverol opened this issue Apr 22, 2020 · 7 comments
Assignees
Labels
proxi-schemas This issue or PR is schemas related

Comments

@ypriverol
Copy link
Contributor

The searchEngineScore in psm should be plural because it is a list

searchEngineScore -> searchEngineScores

@edeutsch
Copy link
Contributor

I agree it should be plural.

But I'm also wondering if something more general is appropriate. PeptideAtlas does not store actual search engine scores, but rather only probabilities. It seems that in general probabilities or q-values or something like that it more useful than search engine scores anyway (e.g., what does a Mascot score of 35 mean again?) What about renaming this thing to something more like "confidenceScores" or "confidenceMetrics" or something like that? That could hold search engine scores or probabilities or q-values or whatever. What do you think?

@ypriverol
Copy link
Contributor Author

@edeutsch, Im fine with confidenceScores because is more generic. Check here the psm endpoint from PRIDE http://wwwdev.ebi.ac.uk/pride/proxi/archive/v0.1/psms?resultType=full&peptideSequence=YNPTWHCIVGR

@edeutsch
Copy link
Contributor

Looks fantastic!

@ypriverol
Copy link
Contributor Author

if you agree, we move to confidentScores, please give me a 👍

@edeutsch
Copy link
Contributor

Should be confidenceScores instead of confidentScores

@ypriverol
Copy link
Contributor Author

@edeutsch can you take a look to this issue #67

@ypriverol
Copy link
Contributor Author

@edeutsch PR #69 ready to be approved.

@ypriverol ypriverol added the proxi-schemas This issue or PR is schemas related label Apr 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proxi-schemas This issue or PR is schemas related
Projects
None yet
Development

No branches or pull requests

3 participants