Review of 'Recipe: Using the OPSIN API in Python'

[jcuadros]

Which page are you reviewing? Recipe: Using the OPSIN API in Python

What is your ORCID https://orcid.org/0000-0001-6513-9140

Review

• A use-case or some context would be helpful to make sense of the connection between the steps in the recipe.
• In my opinion, shorter titles and a descriptive paragraph before the code snippet might work better.
• In Step 2, it may advisable to use the API call ending with '.json' in agreement with the OPSIN documentation. The deletion of the cml field should be commented on.
• In Step 3, it could be helpful to indicate that a vectorial image can also be obtained ('.svg'). Also, more flexibility is available when using the CACTVS Chemical Identifier Resolver at NCI. Check https://cactus.nci.nih.gov/blog/?p=136.

The page works in Binder without any identified issue.

[stuchalk]

Thanks for the review! I will work on your suggestions today.

[stuchalk]

Updates to the opsin.ipynb file have been made based on the review and the changes can be seen in commit 6f60a20. The Cookbook pages have also been updated so this new version is now live. Let me know if if you have any additional comments.