#192 Clean up of Fragmenter class

closed
master (162)
6
2012-10-08
2010-04-29
Rajarshi Guha
No
0 up votes | 0 down votes | 0%
5 comments

This patch cleans up the fragmenter class a little bit by using properly typed containers. Also modifes the ring fragmenter code to return a set of IAtomContainers rather than IRingSet which makes it analogous to the murcko methods

Discussion

  • Rajarshi, can I please update the JavaDoc too for:

    • public List getRingFragments() {

    Since you have a clue what the code is doing...

     
  • Rajarshi Guha
    Rajarshi Guha
    2010-05-04

    Updated with better javadocs and some typing issues which are now fixed

     
  • Rajarshi Guha
    Rajarshi Guha
    2010-05-04

    A bug got introduced. Will upload patches when fixed

     
  • Rajarshi, what is the state of this patch?

     
  • Rajarshi Guha
    Rajarshi Guha
    2010-06-10

    I am in the process of rewriting the entire fragmenter code into a new package. So I will not be working on this specific patch anylonger

     
  • OK, let's close it then, and create a new patch item when that new code is done.