From: SourceForge.net <no...@so...> - 2011-08-04 14:53:50
|
Patches item #3374133, was opened at 2011-07-21 20:58 Message generated for change (Comment added) made by egonw You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3374133&group_id=20024 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: cdk-1.4.x Group: Needs Review Status: Open Resolution: None Priority: 5 Private: No Submitted By: gilleain maclean torrance (gilleain) Assigned to: Egon Willighagen (egonw) Summary: KEGG Atom type patches Initial Comment: 46 element-level patches. Each patch either promotes an existing atom type to a new method, or adds a new method for an element that wasn't already handled. ---------------------------------------------------------------------- >Comment By: Egon Willighagen (egonw) Date: 2011-08-04 16:53 Message: An initial few atom types from this patch already made it into CDK 1.4.1. If you want to help review the patches, please do so. But as soon as you finished one atom type, please send the signed-off patch to me and Rajarshi directly, so that it can immediately be applied. I replied someone earlier today: ------------------------------------------------- > what is the status of Gilleain's reformatted atom type patch. I know you have more than enough to do. Several have been applied already. Last week I had travel days, and this week is catching up with work... See Jul 23: https://github.com/cdk/cdk/commits/cdk-1.4.x Make sure you're local version of the cdk-1.4.x branch is up to date. > I understand that this has been reformatted into individual patches? Yes, see below. > Can we help to expedite the inclusion? Absolutely! (You're the first to ask...) The zip file with patches can be found here: http://sourceforge.net/tracker/?func=detail&aid=3374133&group_id=20024&atid=320024 The patches should be applied to the cdk-1.4.x branch with 'git am -3 --ignore-whitespace' ... skill those that do not immediately work... some will cleanly apply, others need a bit of work. Then check the patch with 'git show' and check if you believe the atom type is reasonable... also check if the number of lone pairs and double bonds are defined (they must have a good excuse not to list those, as we need that for various CDK algorithms). And do a general peer review of the code. Then, make sure nothing gets broken, by running: ant clean dist-all test-dist-all ant -Dmodule=core test-module Particularly focus on the unit test. If you are happy with all, run 'git commit --amend --signoff', and create a new patch with 'git format-patch -1' and send that 0001-*.patch file to Rajarshi *and* me by email. ---------------------------------------------------- ---------------------------------------------------------------------- Comment By: Asad (asadrahman) Date: 2011-07-27 11:54 Message: Hi, I have added a pdf file which contains information about atom types which were fixed/updated. It also contains remark(s) and reference molecules for each atom type for your perusal. https://github.com/downloads/asad/cdk/Curated%20Atom%20type%20patches.pdf Hope this helps. Many thanks Asad ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3374133&group_id=20024 |