Camelia, the Perl 6 bug

IRC log for #cdk, 2010-06-11

| Channels | #cdk index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
01:37 CIA-47 cdk: Egon Willighagen cdk-1.2.x * r1523f66 / (build.xml javadoc.xml develjar/.gitignore):
01:37 CIA-47 cdk: Removed use of the proprietary DocCheck utility
01:37 CIA-47 cdk: Signed-off-by: Rajarshi Guha <rajarshi.guha@gmail.com> - http://bit.ly/aiGOSZ
01:37 CIA-47 cdk: Syed Asad Rahman master * rf66a779 / (10 files in 6 dirs):
01:37 CIA-47 cdk: solved cdk-Bugs-3006773 : small JavaDoc errors in the smsd module
01:37 CIA-47 cdk: Signed-off-by: Syed Asad Rahman <s9asad@gmail.com>
01:37 CIA-47 cdk: Signed-off-by: Rajarshi Guha <rajarshi.guha@gmail.com> - http://bit.ly/a3NH9K
01:40 CIA-47 cdk: Egon Willighagen master * rf0c7d0c / develjar/ojdcheck.jar :
01:40 CIA-47 cdk: Update OpenJavaDocCheck to 0.5: fixing a few false positives
01:40 CIA-47 cdk: Signed-off-by: Rajarshi Guha <rajarshi.guha@gmail.com> - http://bit.ly/cxAc6l
04:33 jbrefort joined #cdk
05:05 egonw_ joined #cdk
05:18 sneumann joined #cdk
06:58 Gpox joined #cdk
07:23 sneumann joined #cdk
08:31 sneumann joined #cdk
08:33 sneumann joined #cdk
09:24 egonw_ joined #cdk
09:53 bag joined #cdk
11:33 maclean joined #cdk
11:34 maclean ¥ø
11:39 egonw hi maclean
11:39 maclean hi
11:39 zarah saluton maclean
11:39 egonw maclean: you saw that your unit tests are failing?
11:39 maclean yeah, yeah, hello to you to automaton.
11:39 maclean egonw : no?
11:40 egonw yes, because BasicSceneGenerator does not seem to get registered...
11:40 maclean I was just trying to run ant -Dmodule=renderer test-module
11:40 egonw I am thinking of registrating that by default
11:40 maclean Ah, that was fixed I thought...
11:40 egonw but then we need to have some duplicate detection code in place too
11:40 maclean In one of my patches.
11:40 egonw ok, please check if I applied them all then...
11:41 maclean yeah, it's a rough patch in the design, for sure.
11:41 maclean ok
11:41 egonw I think I did so, but perhaps I missed one
11:42 maclean hmmm. no, it's the opposite problem. it hasn't been pushed. don't know how that is possible :(
11:43 egonw anyway, I will periodically run Nightly -> http://pele.farmbio.uu.se/nightly-jcp/
11:43 zarah egonw's link is also http://tinyurl.com/ms4eer
11:43 egonw so that we can monitor our progress
11:43 maclean http://github.com/gilleain/cdk/commit/d​363ff848eb4f196d6313bf1d64092ad1e30f1ae
11:43 zarah maclean's link is also http://tinyurl.com/262yfgq
11:44 maclean it snuck in before all the patches from cdk-jcp-15.
11:44 maclean I don't know if it is possible to apply just that one out-of-order?
11:45 egonw yeah, no problem
11:45 egonw git cherry-pick
11:45 egonw I only need the commit hash :)
11:45 egonw git++
11:45 egonw already applied, now pushing to pele
11:45 maclean that would have been my first guess, oh yes.
11:48 maclean oh, and egonw: I had a question about @Testclass and @Testmodule
11:48 egonw @TestMethod, yes ...
11:48 maclean Each parameter inner class has a testmethod that tests it - does it make sense to annotate the class with the @Testmethod?
11:49 egonw the class?
11:49 egonw no, don't think you even can
11:50 maclean The IGeneratorParamter class
11:50 egonw I think the @TestMethod annotation is tied to methods...
11:50 egonw tie it to the get and/or set method?
11:50 egonw ah, but that one does not exist...
11:50 egonw becuase it is inherited...
11:50 maclean The getDefault?
11:51 egonw well, it's not testing that one...
11:51 maclean Doesn't really make much sense, though.
11:51 egonw indeed
11:51 maclean No, I know.
11:51 egonw OK, can you file me a CDK bug report, please?
11:51 egonw copy/paste this discussion
11:51 maclean Ok.
11:51 egonw will have to think about that...
11:51 maclean Another problem is that the test package is o.o.cdk.generatorS, not o.o.cdk.generator...
11:51 egonw another limitation is that the system currently only supports *1* test class
11:51 maclean Really?
11:51 egonw yes :(
11:52 maclean I thought I saw several in a list...
11:52 egonw yeah, but then it fails...
11:52 egonw SMSD had that at some point
11:52 egonw but we have to revert that to one class
11:52 egonw have -> had
11:52 maclean oh. InChIGenerator has too
11:52 maclean @TestClass("testGetInchiFromChlorineAtom,testGetIn​chiFromLithiumIontestGetInchiFromChlorine37Atom")
11:53 egonw yes, those packages names need to be synchronized too
11:53 egonw oh...
11:53 egonw that's @TestClass there?
11:53 egonw looks really like a @TestMethod content string
11:53 maclean yup. Inchigenerator line 120
11:53 egonw more than one method in @TestMethod is fine
11:53 egonw @TestClass can only take one class
11:54 egonw if that really says @TestClass, please file a bug report for that too...
11:54 egonw (code review)++
11:54 maclean hmmm. it's actually annotating a method, but it says Testclass...
11:54 egonw maclean++
11:54 egonw for spotting that :)
11:56 maclean I guess it could be another doc-check thing? Checking that methods are not annotated with testclass? (bit rare, I suppose).
12:01 egonw but a useful one...
12:01 egonw or perhaps better at a PMD level...
12:01 egonw both should be easy enough
12:01 egonw ok, file a feature request for that :)
12:02 maclean ok
12:02 egonw http://pele.farmbio.uu.se/​nightly-jcp/junitdiff.html
12:02 zarah egonw's link is also http://tinyurl.com/2w5eo8f
12:03 egonw http://pele.farmbio.uu.se/nightly​-jcp/test/result-renderbasic.html
12:03 zarah egonw's link is also http://tinyurl.com/2vk6usf
12:03 maclean heh. cool, fixed tests? Or deleted (red)?
12:09 egonw could be both, but here it is indeed fixed
12:09 egonw oh, and we need to fix that margin issue
12:10 maclean "ant -Dmodule=renderer test-module" should work, right?
12:10 maclean I get "java.lang.ClassNotFoundException: org.openscience.cdk.modulesuites.MrendererTests"
12:11 egonw module=render
12:11 maclean ahhh
12:14 maclean Hmmm. but then I get : "java.lang.NoClassDefFoundError: org/openscience/cdk/annotations/TestMethod"
12:14 maclean at org.openscience.cdk.coverage.RenderCoverageT​est.testCoverage(RenderCoverageTest.java:38)
12:15 egonw did you update and/or rebase from the latest 13-unsorted ?
12:15 egonw you should have hash:
12:15 egonw 7d3d8e22c3d9c4f77d15186a07785890b4737440
12:17 maclean well I did update from 13-unsorted, but I have no idea if I have that hash or not.
12:18 maclean well, ".git log | grep "7d3d8e22c3d9c4f77"" gives no results.
12:19 maclean ok, did a pull - lots of conflicts :(
12:20 egonw yes
12:20 egonw don't pull
12:20 egonw that indeed does not work when upstream (me) rebases all the time
12:20 egonw git fetch upstream
12:20 egonw and then rebase your patches on top of branch 13-unsorted
12:21 maclean but fetch puts stuff in the index - so how does rebase then work?
12:21 egonw git checkout your_branch
12:22 egonw git checkout -b 13-unsorted-foo upstream/13-unsorted-foo
12:22 egonw git checkout your_branch
12:22 egonw git rebase 13-unsorted-foo
12:22 egonw 13-unsorted-patches
12:22 egonw is the proper name
12:26 maclean where 'upstream' is actually 'pele' which is 'http://pele.farmbio.uu.se/git/cdk-jchempaint.git'
12:26 zarah maclean's link is also http://tinyurl.com/35uojh2
12:31 maclean hmm. doesn't work - but I understand that the checkout -b command is making a new branch with the name '13-unsorted..." that starts at 'pele/13-unsorted..."
12:31 egonw correct
12:32 maclean but I get the message "fatal: git checkout: updating paths is incompatible with switching branches."
12:32 maclean Also : "Did you intend to checkout 'pele/13-unsorted-patches' which can not be resolved as commit?"
12:32 maclean which suggests to me that it can't find 13-unsorted-patches at the pele url.
12:32 egonw try this:
12:33 egonw git branch -r
12:34 maclean yes? this gives me a list of remote branches. it doesn't include pele, and upstream is github's cdk, I think.
12:35 egonw git fetch pele
12:35 egonw then git branch -a
12:36 maclean Ok I'll download all the random branches from pele again. I deleted most of them yesterday, as they were hanging around, untidily :/
12:43 maclean aaaarrhgghghghhhhh.
12:46 maclean this is like trying to understand a particularly bad Sci-Fi tv series whose writers are overly fond of time-travel plots }:-<
12:46 egonw hehehehe
12:47 maclean I've just gone back in time and shot my grandfather.
12:47 egonw yeah, the problem is, that you are current director :)
12:47 egonw -ly
12:48 maclean I think I am trying to rebase on 13-unsorted when I have many of 13-unsorted patches already. Specifically, the controller stuff.
12:48 maclean So it has tons of conflicts.
12:48 egonw mmmm...
12:49 egonw git rebase --abort
12:49 egonw alternatively...
12:49 egonw you can checks the patches in your branch
12:49 egonw branch from 13-unsorted-patches and git cherry-pick
12:50 egonw but git is generally pretty good at detecting the same patches, and not trying to apply them again...
12:50 maclean I aborted already. Will try cherry-picking.
12:50 egonw your current, unrebased branch.... where does it branch from then?
12:50 maclean my master
12:51 maclean It might be that build.props file - I have no idea.
12:51 egonw it will report you what failed
12:51 egonw look into 'git mergetool'
12:52 egonw which is really useful combined with kdiff3
13:05 maclean joined #cdk
13:16 jonalv joined #cdk
13:16 jonalv is there a cdk icon of size say 16x16?
13:16 zarah jonalv: You have new messages. Write '/msg zarah @messages' to read them.
13:16 egonw not that I am aware of
13:17 jonalv hm can I wish for one?
13:17 jonalv Would be nice to use in Bioclipse I think...
13:17 jonalv and probably in other cases too
13:19 egonw there might have been a flavion
13:19 egonw but the logo is not really square
13:19 egonw which makes it difficult
13:20 jonalv hm which one are you refering too?
13:22 jonalv http://www.downloadatoz.com/_imgbank/0​904/Chemistry-Development-Kit.icon.gif
13:22 jonalv that one?
13:22 zarah jonalv's link is also http://tinyurl.com/34el3g7
13:22 egonw that looks 32x32
13:23 egonw and a bit cut off
13:23 jonalv yea, but maybe it's possible to draw a 16x16 like that one?
13:49 Gpox http://pele.farmbio.uu.se/buckminster/cdk16x16.png
13:49 zarah Gpox's link is also http://tinyurl.com/39gjwps
13:52 jonalv Gpox: oh yey Gpox++
13:54 jbrefort joined #cdk
15:07 carsten joined #cdk
16:55 sneumann joined #cdk
18:06 sneumann_ joined #cdk
20:51 bag_ joined #cdk

| Channels | #cdk index | Today | | Search | Google Search | Plain-Text | summary