humm..yes in fact.
The problem is that, now in the description <MSEProperty:
#numberOfLinesOfCode type: #Number> the first argument must be the name of
the method (and consequently the name of the property in Fame description),
while, for example, in <property: #WLOC longName: 'Number of lines of code'
description:'The number of lines of code in a class'> the first argument may
have a different name, that's it?
I will have a look.
On Fri, Jul 15, 2011 at 12:35 PM, Tudor Girba <tudor(a)tudorgirba.com> wrote:
I added a couple of tests to describe the situation:
VerveineJModelTest>>testCyclomaticComplexity
VerveineJModelTest>>testNumberOfStatements
Cheers,
Doru
On 15 Jul 2011, at 12:23, Tudor Girba wrote:
Hi Andre,
I just saw that the latest Moose build failed because of mismatch between
the
names of metric properties. For example:
FAMIXType>>numberOfLinesOfCode
" <property: #WLOC longName: 'Number of lines of code' description:
'The number of lines of code in a class'>"
<MSEProperty: #numberOfLinesOfCode type: #Number>
<MSEComment: 'The number of lines of code in a class'>
^self
lookUpPropertyNamed: #numberOfLinesOfCode
computedAs: [self methodsGroup sum: #numberOfLinesOfCode]
The name of the metric should be WLOC. This is historical and it is a
metric used
to communicate with external parsers.
So, all in all we need the name of the old properties in the Fame
description.
Could you take care of that?
Cheers,
Doru
On 14 Jul 2011, at 23:05, Tudor Girba wrote:
> Hi,
>
> On 14 Jul 2011, at 19:46, Andre Hora wrote:
>
>> Hello,
>>
>> Yes Stef.
>> Doru thanks for the feedback. You can check the code in (remember to
use
the last Merlin and updated the meta-model):
>>
>> Gofer new
>> squeaksource: 'HoraExp';
>> package: 'FameExportMSE';
>
> I tried this one. The idea is nice, but the problem is that the second
step of
the exporter wizard is too tall and goes out of my screen. We would
need another layout.
>
>> package: 'FameExportProperties';
>
> Very cool. This needs to go in Moose-Finder. The only thing to fix here
is the
label in the first wizard step when we have to choose type of
exported file.
>
>> package: 'Famix-Extensions';
>
> This is cool. I already copied it to the Moose repo. It's that easy :)
>
> Cheers,
> Doru
>
>
>> load.
>>
>> regards,
>>
>> On Thu, Jul 14, 2011 at 5:56 PM, Stéphane Ducasse <
stephane.ducasse(a)inria.fr> wrote:
>> grab me next time I'm around because
I want to have a look at that.
>>
>> Stef
>>
>> On Jul 14, 2011, at 1:51 PM, Andre Hora wrote:
>>
>>> Hello,
>>>
>>> Just returning to this topic.
>>>
>>> As most of the metrics are not described at the Fame level we are
losing this information when we export a model to MSE in Moose. We know that
most of the metrics can be recalculated in the model, but in some cases they
cannot.
>>> For example, when exporting in Moose
a Java (VerveineJ) model we lose
the metrics cyclomaticComplexity (CYCLO) and
numberOfStatements (NOS) and we
cannot recalculate them.
>>>
>>> So, to avoid it, I changed some metrics (some I think are important,
but we can change them if we integrate in Moose) to be described at the Fame
level and like that we can export them into a MSE. In this case, as the we
could have a huge MSE, I implemented a filter and you can select the metrics
to be exported through a wizard. It solves, for example, the lose of
information with VerveineJ if you decide to export CYCLO and NOS.
>>>
>>> To try you can run the script below and the right-click in a
MooseModel and: Import / Export -> Export model to MSE with metrics. Use the
last version of Merlin.
>>>
>>> Gofer new
>>> squeaksource: 'FameImportExportEx';
>>> package: 'Fame-ImportExport-Extensions';
>>> package: 'Fame-ImportExport-Extensions-MSE';
>>> load.
>>>
>>> regards,
>>>
>>> On Thu, Jun 23, 2011 at 3:32 PM, Stéphane Ducasse <
stephane.ducasse(a)inria.fr> wrote:
>>>> At one point metrics were
described in the meta-model but then MSE
files were very large, slow to export
(because you have to compute all
metrics), and slow to load (this was a long time ago). We decided to remove
them from the Smalltalk MSE export (although the MSE importer can still
handle metrics and other data, like for example when importing from
infusion/verveinej).
>>>
>>> but even in Smalltalk you need to keep loc
>>>
>>>> Now is the interface to select what to export/import functional?
>>>
>>> I do not know but this is something you we should definitively do
>>> : select what to save
>>> : select the format
>>> -> tab separated, mse, xml
>>>
>>>
>>>> That would be the way to go. Then describe metrics as part of a Fame
package 'Metrics', then we can select group of data by package. I would
still like to be able to not export metrics in the MSE file.
>>>>
>>>>
>>>> On 22 juin 2011, at 15:27, Stéphane Ducasse wrote:
>>>>
>>>>> Hi guys
>>>>>
>>>>> I could not find metrics in the saved MSE metrics like LOC that
cannot be computed by moose.
>>>>> Are metrics saved? I remember
writing tests to make sure that this
was correct.
>>>>>
>>>>> In addition it would be good to be able to select some metrics to be
exported to other formats.
>>>
>>> Stef
>>> _______________________________________________
>>> Moose-dev mailing list
>>> Moose-dev(a)iam.unibe.ch
>>>
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>>
>> --
>> Simon Denier
>>
>>
>>
>>
>> _______________________________________________
>> Moose-dev mailing list
>> Moose-dev(a)iam.unibe.ch
>>
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>
>
> _______________________________________________
> Moose-dev mailing list
> Moose-dev(a)iam.unibe.ch
>
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>
>
>
> --
> Andre Hora
>
> _______________________________________________
> Moose-dev mailing list
> Moose-dev(a)iam.unibe.ch
>
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
_______________________________________________
Moose-dev mailing list
Moose-dev(a)iam.unibe.ch
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
--
Andre Hora
_______________________________________________
Moose-dev mailing list
Moose-dev(a)iam.unibe.ch
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
--
www.tudorgirba.com
"Beauty is where we see it."
--
www.tudorgirba.com
"Being happy is a matter of choice."
--
www.tudorgirba.com
"Live like you mean it."
_______________________________________________
Moose-dev mailing list
Moose-dev(a)iam.unibe.ch
https://www.iam.unibe.ch/mailman/listinfo/moose-dev