Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.
Sign upError in DTXSID mapping? #68
Comments
This comment has been minimized.
This comment has been minimized.
That's my issue. I take care of it. |
This comment has been minimized.
This comment has been minimized.
ChemConnector
commented
May 16, 2019
I hope @meier-rene can resolve the issue as it is not obvious at all to me how this would happen. We do have that DTXSID60861331 in our internal production but it is not yet public and certainly is not Acetamiprid. Rene, please let me know whether you can fix it . Thanks |
This comment has been minimized.
This comment has been minimized.
I hope @meier-rene can find the cause but it's worrying that this exists but is not yet in production - it is going to get very confusing if we can access DTXSIDs that are not yet in production via the web services ... we will end up with broken links everywhere and no way to control it? |
This comment has been minimized.
This comment has been minimized.
ChemConnector
commented
May 16, 2019
Yes, I am going to research whether or not the web service is poking back
into our development database rather than our production database. I was
not aware that it was and this is the first indication that it was. I was
just trying to check but one of the Apache security certificates expired
overnight and they are trying to fix it so I cannot do this until
everything is back up and running.
Tony Williams
Phone: +1 (919) 201-1516
Email: tony27587@gmail.com
Skype: tony27587
My Blog: www.chemconnector.com
Twitter: http://twitter.com/ChemConnector
…On Thu, May 16, 2019 at 8:06 AM Emma Schymanski ***@***.***> wrote:
I hope @meier-rene <https://github.com/meier-rene> can find the cause but
it's worrying that this exists but is not yet in production - it is going
to get very confusing if we can access DTXSIDs that are not yet in
production via the web services ... we will end up with broken links
everywhere and no way to control it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#68?email_source=notifications&email_token=AAEYGD35VRM3Q32BL2DX5ALPVVE53A5CNFSM4HNLHA22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVRTEMI#issuecomment-493040177>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAEYGD2D3PUNXVZ67DNQ7TTPVVE53ANCNFSM4HNLHA2Q>
.
|
meier-rene
referenced this issue
May 16, 2019
Open
Add DTXSIDs to all MassBank records with InChIKey match #66
This comment has been minimized.
This comment has been minimized.
Because the InChI key resolver at https://actorws.epa.gov is my only source for DTXSID I have to wait until this service is fixed. |
This comment has been minimized.
This comment has been minimized.
ChemConnector
commented
May 16, 2019
Everything should be back up and running as of about 5pm Thursday. Please
let me know if it works for you...
Tony Williams
Phone: +1 (919) 201-1516
Email: tony27587@gmail.com
Skype: tony27587
My Blog: www.chemconnector.com
Twitter: http://twitter.com/ChemConnector
…On Thu, May 16, 2019 at 8:55 AM René Meier ***@***.***> wrote:
Because the InChI key resolver at https://actorws.epa.gov is my only
source for DTXSID I have to wait until this service is fixed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#68?email_source=notifications&email_token=AAEYGD34MCKAXJZWWXU7CNTPVVKSVA5CNFSM4HNLHA22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVRW4SI#issuecomment-493055561>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAEYGD7UHTVJIQMLX7MKKN3PVVKSVANCNFSM4HNLHA2Q>
.
|
This comment has been minimized.
This comment has been minimized.
The service is back, thank you @ChemConnector. Unfortunately there waits some more work for you. The erroneous record https://massbank.eu/MassBank/RecordDisplay.jsp?id=MSJ01067 contains the InChI key WCXDHFDTOYPNIE-UHFFFAOYSA-N. If I put this in the resolver https://actorws.epa.gov/actorws/chemIdentifier/v01/resolve?identifier=WCXDHFDTOYPNIE-UHFFFAOYSA-N I get DTXSID60861331. This does not resolve to an valid substance https://comptox.epa.gov/dashboard/dsstoxdb/results?search=DTXSID60861331. Please have a look into this. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
ChemConnector
commented
May 17, 2019
I am still researching but I think I know what it is and need to check out with the developer. One comment though is that Acetamiprid is explicit stereo (E-form) for the chemicals. See https://comptox.epa.gov/dashboard/dsstoxdb/results?search=DTXSID0034300 . I have confirmed this will multiple resources so you may wish to update your structure and associated InChIKey to WCXDHFDTOYPNIE-RIYZIHGNSA-N. This resolves correctly. https://actorws.epa.gov/actorws/chemIdentifier/v01/resolve?identifier=WCXDHFDTOYPNIE-RIYZIHGNSA-N This is NOT the cause of the error you are seeing for sure. If my hypothesis is correct it's the fact that one of the synonyms for this chemical https://comptox.epa.gov/dashboard/dsstoxdb/results?search=FAIL%20peptide in the synonym table is "FAIL" and I believe that the service is passing a FAIL message and then resolving to this chemical....it matches the IndigoInChIKey here https://actorws.epa.gov/actorws/chemIdentifier/v01/resolve?identifier=WCXDHFDTOYPNIE-UHFFFAOYSA-N. I am off to go prove it... |
This comment has been minimized.
This comment has been minimized.
@meier-rene can you take care of updating the record, or should I add it to my list along with the CASMI and UFZ ones to resolve (hope to do this next week). Just let me know, thanks! |
meier-rene
pushed a commit
that referenced
this issue
May 17, 2019
This comment has been minimized.
This comment has been minimized.
I changed the chemical information for all records of Acetamiprid. |
This comment has been minimized.
This comment has been minimized.
adelenelai
commented
May 27, 2019
•
Hi Tony @ChemConnector , came across another case of the same Dashboard issue described by Rene: Benzoylecgonine's DTXSID does not seem to surface in the Dashboard... https://comptox.epa.gov/dashboard/dsstoxdb/results?search=DTXSID30859442 ...but does exist: A specific use case which would be affected: if users of RMassBank wanted to manually look up a DTXSID using InChiKey, they must do it via the web service and not via Dashboard MassBank/RMassBank#215 NB: Currently, the Benzoylecgonine that does surface in the Dashboard has specific stereochemistry (whereas in MassBank, no stereochemistry is specified): |
This comment has been minimized.
This comment has been minimized.
ChemConnector
commented
May 29, 2019
I understand what is happening here and causing this issue. The services are hitting the database and retrieving "Level 6" data, that we term as "Incomplete" (non-curated for public release). We have a filter on the dashboard code that does not retrieve Incompletes but this filter was not added to the service. It was developed outside of the core team in order to make it available faster that we would get done otherwise and this filter was missed. It will be added in the a future sprint and fix the issue. The mismatch did highlight that the structure for the chemical does not match the name since, as was pointed out, the chemical is lacking stereochemistry. It would be best to update the InChIKey and other mappings in MassBank too maybe? http://comptox-prod.epa.gov/dashboard/dsstoxdb/results?search=DTXSID7046758 |
This comment has been minimized.
This comment has been minimized.
OK this is an Eawag record ... Tony and I went thru these and all Eawag substances DO have DTXSIDs and we need to figure out a smart way to upgrade all the Eawag records with stereo-specific information. @meier-rene we will need to discuss this, it's been on my todo list for a while but I will need some time to figure out how best to coordinate upgrading the chemical information with you. Pls put this on hold for the moment as it's not just this one ... maybe we need another issue for that. |
This comment has been minimized.
This comment has been minimized.
So, @ChemConnector has provided me with a list of 390 DTXSIDs that are "Level 6" and thus not public that we should remove from our InChIKey - DTXSID mappings. I will email this to @meier-rene then I think we can close this issue, as the main cause was the incorrect retrieval of Level 6 substances from DSSTox due to an error in the CompTox web services [a fix that needs doing their side]. |
This comment has been minimized.
This comment has been minimized.
via @ChemConnector |
This comment has been minimized.
This comment has been minimized.
More clarifications via email conversation with @ChemConnector There are 5 public levels of data: The field qcId in the web services is numbered 1-6 but they do not align with the public Levels.
|
meier-rene
pushed a commit
that referenced
this issue
Jun 19, 2019
This comment has been minimized.
This comment has been minimized.
Also to note: |
schymane commentedMay 16, 2019
•
edited
Bug report from external user:
Follow-up:
Indeed that DTXSID doesn't appear to exist in the public Dashboard, nor do I get a match for that InChIKey. If this is a name match, it's wrong ...
https://massbank.eu/MassBank/RecordDisplay.jsp?id=MSJ01067
https://comptox.epa.gov/dashboard/dsstoxdb/results?search=DTXSID60861331
https://comptox.epa.gov/dashboard/dsstoxdb/results?search=WCXDHFDTOYPNIE-UHFFFAOYSA-N
This is the correct match:
https://comptox.epa.gov/dashboard/dsstoxdb/results?search=DTXSID0034300
and is also found by name:
https://comptox.epa.gov/dashboard/dsstoxdb/results?search=Acetamiprid
Any ideas what went wrong here @meier-rene @ChemConnector ?
PubChem link looks fine
https://pubchem.ncbi.nlm.nih.gov/compound/213021