From: SourceForge.net <no...@so...> - 2010-08-29 11:41:23
|
Patches item #3055420, was opened at 2010-08-29 13:41 Message generated for change (Tracker Item Submitted) made by egonw You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3055420&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: master Group: Needs Review Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: Use PeriodicTable to detect two-char symbols in SMILES Initial Comment: Patch depends on this patch to be applied first: 3054454 Adds two char elem symbols missing in the SMILESParser Aimed at master. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3055420&group_id=20024 |
From: SourceForge.net <no...@so...> - 2010-08-29 19:15:06
|
Patches item #3055420, was opened at 2010-08-29 07:41 Message generated for change (Comment added) made by rajarshi You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3055420&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: master Group: Needs Review Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: Use PeriodicTable to detect two-char symbols in SMILES Initial Comment: Patch depends on this patch to be applied first: 3054454 Adds two char elem symbols missing in the SMILESParser Aimed at master. ---------------------------------------------------------------------- >Comment By: Rajarshi Guha (rajarshi) Date: 2010-08-29 15:15 Message: doesn't apply to master ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3055420&group_id=20024 |
From: SourceForge.net <no...@so...> - 2010-08-29 19:16:12
|
Patches item #3055420, was opened at 2010-08-29 13:41 Message generated for change (Comment added) made by egonw You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3055420&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: master Group: Needs Review Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: Use PeriodicTable to detect two-char symbols in SMILES Initial Comment: Patch depends on this patch to be applied first: 3054454 Adds two char elem symbols missing in the SMILESParser Aimed at master. ---------------------------------------------------------------------- >Comment By: Egon Willighagen (egonw) Date: 2010-08-29 21:16 Message: Did you apply the other patches first? ---------------------------------------------------------------------- Comment By: Rajarshi Guha (rajarshi) Date: 2010-08-29 21:15 Message: doesn't apply to master ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3055420&group_id=20024 |
From: SourceForge.net <no...@so...> - 2010-08-29 19:16:56
|
Patches item #3055420, was opened at 2010-08-29 13:41 Message generated for change (Comment added) made by egonw You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3055420&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: master Group: Needs Review Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: Use PeriodicTable to detect two-char symbols in SMILES Initial Comment: Patch depends on this patch to be applied first: 3054454 Adds two char elem symbols missing in the SMILESParser Aimed at master. ---------------------------------------------------------------------- >Comment By: Egon Willighagen (egonw) Date: 2010-08-29 21:16 Message: Ah, but then only the 0001 in that link... ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2010-08-29 21:16 Message: Did you apply the other patches first? ---------------------------------------------------------------------- Comment By: Rajarshi Guha (rajarshi) Date: 2010-08-29 21:15 Message: doesn't apply to master ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3055420&group_id=20024 |
From: SourceForge.net <no...@so...> - 2010-08-29 19:24:52
|
Patches item #3055420, was opened at 2010-08-29 13:41 Message generated for change (Comment added) made by egonw You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3055420&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: master Group: Needs Review Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: Use PeriodicTable to detect two-char symbols in SMILES Initial Comment: Patch depends on this patch to be applied first: 3054454 Adds two char elem symbols missing in the SMILESParser Aimed at master. ---------------------------------------------------------------------- >Comment By: Egon Willighagen (egonw) Date: 2010-08-29 21:24 Message: OK, yeah, my bad... don't mix cdk-1.2.x with master patches. New proper patch attached. ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2010-08-29 21:16 Message: Ah, but then only the 0001 in that link... ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2010-08-29 21:16 Message: Did you apply the other patches first? ---------------------------------------------------------------------- Comment By: Rajarshi Guha (rajarshi) Date: 2010-08-29 21:15 Message: doesn't apply to master ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3055420&group_id=20024 |
From: SourceForge.net <no...@so...> - 2010-08-29 19:27:06
|
Patches item #3055420, was opened at 2010-08-29 07:41 Message generated for change (Comment added) made by rajarshi You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3055420&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: master >Group: Accepted >Status: Closed >Resolution: Accepted Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: Use PeriodicTable to detect two-char symbols in SMILES Initial Comment: Patch depends on this patch to be applied first: 3054454 Adds two char elem symbols missing in the SMILESParser Aimed at master. ---------------------------------------------------------------------- >Comment By: Rajarshi Guha (rajarshi) Date: 2010-08-29 15:27 Message: applied and pushed ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2010-08-29 15:24 Message: OK, yeah, my bad... don't mix cdk-1.2.x with master patches. New proper patch attached. ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2010-08-29 15:16 Message: Ah, but then only the 0001 in that link... ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2010-08-29 15:16 Message: Did you apply the other patches first? ---------------------------------------------------------------------- Comment By: Rajarshi Guha (rajarshi) Date: 2010-08-29 15:15 Message: doesn't apply to master ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3055420&group_id=20024 |