{
  "version": "https://jsonfeed.org/version/1.1",
  "title": "chem-bla-ics",
  "description": "Chemblaics (pronounced chem-bla-ics) is the science that uses open science and computers to solve problems in chemistry, biochemistry and related fields.",
  "home_page_url": "https://chem-bla-ics.linkedchemistry.info/",
  "feed_url": "https://chem-bla-ics.linkedchemistry.info/2009/06/30/jchempaint-hack-session-at-uppsala.json",
  "icon": "https://chem-bla-ics.linkedchemistry.info/assets/images/chem-bla-ics_logo.png",
  "language": "en",
  "authors": [
    {
      "name": "Egon Willighagen",
      "url": "https://orcid.org/0000-0001-7542-0286",
      "_orcid": "0000-0001-7542-0286"
    }
  ],
  "items": [

    {
      "id": "https://doi.org/10.59350/m0zsx-9ta16",
      "url": "https://chem-bla-ics.linkedchemistry.info/2009/06/30/jchempaint-hack-session-at-uppsala.html",
      "title": "JChemPaint hack session at Uppsala",
      "content_html": "<p>Arvid and I had a meeting on the ControllerHub refactoring, to make it modular to the bone. Actually,\nit is the <a href=\"http://cdk.svn.sourceforge.net/viewvc/cdk/cdk/branches/jchempaint-primary/src/main/org/openscience/cdk/controller/IChemModelRelay.java?view=log\">IChemModelRelay</a>\nthat needs refactoring. This is what we wrote down:</p>\n\n<p><img src=\"/assets/images/jcpHackWeek.JPG\" alt=\"\" /></p>\n\n<p>In this picture you can see our priorities (1,2 for Arvid, and A,B for me).</p>\n\n<p><strong>1</strong> The above mentioned relay will get refactored to have each method a separate class, which at the same time will provide\nthe undo/redo functionality. We might even have undo at a scripting level :)</p>\n\n<p><strong>2</strong> Second item on Arvid’s list is extending the mouse relay to handle key modifiers, an unfortunate omission in that design.\nThis is needed to Ctrl- and Apple’s Command-based selection approach.</p>\n\n<p><strong>A</strong> To easy the footprint reduction of JChemPaint, we are going to split the current <em>render</em> module into <strong>render</strong> and <strong>renderextra</strong>.\nThe second may see future split ups, and both may see name changes, but the first go will split them according to requiring an\nIAtomContainer data model or a IChemModel data model.</p>\n\n<p>This will help us clean up dependencies, by forcing us to have the core functionality not pull in, for example, reaction functionality.\nAdditionally, isotope related rendering will go into <em>renderextra</em> so removing the dependency on the IsotopeFactory and the associated\n800kB-sized <em>isotope.xml</em> data file.</p>\n\n<p>This will not immediately help the applet class partitioning and indexing, but it will help us to keep a sane overview of all the stuff\nwe have around.</p>\n\n<p><strong>B</strong> The goal is to merge this work on JChemPaint back into the CDK libary, so that we again have a CDK version with a\nfully working editing environment as did CDK 1.0. However, that requires to code to be stable, which includes full unit\ntesting, no PMD violations, complete JavaDoc. However, as I wrote this morning to the\n<a href=\"https://lists.sourceforge.net/lists/listinfo/cdk-jchempaint\">cdk-jchempaint mailing list</a>:</p>\n\n<blockquote>\n  <p>But, there is a lot of clean up to do. I counted 497 missing test\nmethods; 326 PMD violations, and saw a lot of missing JavaDoc. This\nmeans, that the current patch is pretty messed up indeed, and we are a\nlong way away from seeing a merge with CDK master :(</p>\n</blockquote>",
      "summary": "Arvid and I had a meeting on the ControllerHub refactoring, to make it modular to the bone. Actually, it is the IChemModelRelay that needs refactoring. This is what we wrote down:",
      "image": "https://chem-bla-ics.linkedchemistry.info/assets/images/jcpHackWeek.JPG",
      "date_published": "2009-06-30T00:00:00+00:00",
      "date_modified": "2009-06-30T00:00:00+00:00",
      "tags": ["cdk","jchempaint"],
      
      
      
      
      
      
        "authors": [ { "name": "Egon Willighagen", "url": "https://orcid.org/0000-0001-7542-0286" } ]
      
    }

  ]
}