the stastic figures have been corrected. and the test list has been revised.
And that caravel error was changed.
On Mon, Mar 31, 2008 at 5:01 PM, Alexei Zakharov <[EMAIL PROTECTED]>
wrote:
> Nice! I have only a couple of corrections.
>
> 1. Pay attention to the list of excluded test. Many of tests are
> listed twice, some of them - KeyboardFocusManagerRTest.java for
> example - even more (up to 5 times).
> 2. In scenario number 2 you have "try to move caravan". FYI there are
> no caravans in Colonization, they were in Civilization. :) It was
> written as "caravel" in my original text - i.e. a type of ship.
>
> With Best Regards,
> Alexei
>
> 2008/3/31, Tharindu Mathew <[EMAIL PROTECTED]>:
> > I modified the proposal and made some changes. Tell me if I can improve
> > further.
> >
> > On Sun, Mar 30, 2008 at 7:16 PM, Alexei Fedotov <
> [EMAIL PROTECTED]>
> > wrote:
> >
> >
> > > Let me elaborate that a bit. After getting that list of required
> > > classes for the simple scenario, I think it may be challenging enough
> > > for three months for a student.
> > >
> > > On Sun, Mar 30, 2008 at 5:42 PM, Alexei Zakharov
> > > <[EMAIL PROTECTED]> wrote:
> > > > Hi Tharindu,
> > > >
> > > > Looks good in general. However, about these scenarios. Origianlly
> I
> > > > spent about two minutes creating them. So I don't think you should
> > > > treat it as something final and attach your proposal to it. IMO
> the
> > > > main idea here is just to show you have something to measure your
> > > > success. So I suggest to add a statement that the scenarios are
> not
> > > > finalized and is a subject ot change.
> > > >
> > > > Regards,
> > > > Alexei
> > > >
> > > > 2008/3/30, Alexei Fedotov <[EMAIL PROTECTED]>:
> > > >
> > > >
> > > > > Tharindu,
> > > > > Good job! Though it seems that there are yet things to be
> improved
> > > > > about your application:
> > > > >
> > > > > 1. Deliverables section:
> > > > > "The Harmony will be able to pass the following scenarios". Put
> > > > > scenarios here. Link each scenario to the date when it should
> pass.
> > > > > You may also try running these scenarios on Sun [1] and collect
> a
> > > list
> > > > > of classes required for each scenario to pass. This would allow
> you
> > > > > replacing "first phase" and "second phase" in your text with
> > > something
> > > > > checkable such as a list of enabled cases.
> > > > >
> > > > > 2. Abstract and discussion sections:
> > > > > One important thing about impact is missed: you are improving
> the
> > > > > graphical libraries which hardly pass any tests (name a
> percentage
> > > of
> > > > > excluded unit tests) to the functional state.
> > > > >
> > > > > [1] Here how the estimate could be completed:
> > > > >
> > > > > The list of classes for the first screen (I launched
> application and
> > > > > pressed Quit, then Yes).
> > > > > $ cd freecol
> > > > > $ java -Xmx128m -verbose:class -jar FreeCol.jar>first_screen.txt
> > > > >
> > > > > Replace dot in a class name with
> > > > > $ awk '{ print $2 }' first_screen.txt | sed -e 's/\./\//g'
> > > > > >first_screen_conv.txt
> > > > >
> > > > > Select classes which are not properly tested and may prove a
> > > problem:
> > > > > $ egrep 'awt|swing' /c/temp/freecol/first_screen_conv.txt |
> while
> > > read
> > > > > entry; do fgrep "$entry" modules/awt/make/*
> modules/swing/make/*;
> > > done
> > > > > | tee excludes.txt
> > > > >
> > > > <snip>
> > > >
> > > >
> > > >
> > > > > On Sat, Mar 29, 2008 at 9:49 PM, Tharindu Mathew <
> > > [EMAIL PROTECTED]> wrote:
> > > > > > Hi everyone,
> > > > > >
> > > > > > This is my proposal for the harmony-demo-1 project. It would
> be a
> > > great help
> > > > > > if you could go through it and tell me where I can improve.
> > > > > >
> > > > > >
> > > http://wiki.apache.org/general/TharinduMathew/GSoC2008/harmony-demo-1
> > > > > >
> > > > > > Thanks.
> > > > > >
> > > > > > --
> > > > > > Regards,
> > > > > >
> > > > > > Tharindu
> > > > > >
> > > > > --
> > > > > With best regards,
> > > > >
> > > > > Alexei
> > > > >
> > > >
> > >
> > >
> > >
> > > --
> > > With best regards,
> > > Alexei
> > >
> >
> >
> >
> >
> > --
> > Regards,
> >
> >
> > Tharindu
> >
>
--
Regards,
Tharindu
