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

Search command #150

Open
dnikol1 opened this Issue Dec 17, 2018 · 9 comments

Comments

Projects
None yet
4 participants
@dnikol1

dnikol1 commented Dec 17, 2018

Why is it that we cannot just search molecular formula without putting something in the Compound Name line?

Why is it that the spectrum in the format of m/z vs absolute intensity cannot be searched, but instead requires normalized intensity?

@tsufz tsufz transferred this issue from MassBank/MassBank-data Dec 18, 2018

@tsufz

This comment has been minimized.

Member

tsufz commented Dec 18, 2018

Hi, I transfered this issue, because there was a wrong link on MassBank.

@tsufz

This comment has been minimized.

Member

tsufz commented Dec 18, 2018

Why is it that the spectrum in the format of m/z vs absolute intensity cannot be searched, but instead requires normalized intensity?

I totally agree with the comment. It is not very practical to normalise a peak list before submitting. The online tool should be as comfortable as possible. Usually, the people copy and paste a spectra from their MS viewer or so. We could do the normalisation in the background while handling the query?

@tsufz

This comment has been minimized.

Member

tsufz commented Dec 18, 2018

Why is it that we cannot just search molecular formula without putting something in the Compound Name line?

Thanks for the comment, we will improve the search.

@meier-rene

This comment has been minimized.

Contributor

meier-rene commented Dec 18, 2018

Why is it that we cannot just search molecular formula without putting something in the Compound Name line?

Fixed with 6217299

@tsufz

This comment has been minimized.

Member

tsufz commented Dec 18, 2018

Fixed with 6217299

The search does not handle truncations so far. It is possible to search for C8H10N4O2 and get the results for caffeine. If I truncate like C8H10N*O2, I get empty results.

@meier-rene

This comment has been minimized.

Contributor

meier-rene commented Dec 18, 2018

8f87c92 implements wildcards on chemical formulas. This wildcards work only on string basis without any chemical knowledge. This means C8OP* will result in 291 results while C8PO* will have no results. Implementing chemical knowledge here would be a bigger effort.

@schymane

This comment has been minimized.

Member

schymane commented Dec 18, 2018

@tsufz

This comment has been minimized.

Member

tsufz commented Dec 18, 2018

Would be some easy code: https://stackoverflow.com/questions/2974362/parsing-a-chemical-formula or could be use CDK? Needs to be Java and not R.

@tsufz

This comment has been minimized.

Member

tsufz commented Dec 18, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment