15 Jul
2011
15 Jul
'11
1:17 p.m.
It is not enough. Also the lookup should be by the short name.
Cheers,
Doru
On 15 Jul 2011, at 13:14, Andre Hora wrote:
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:
--
www.tudorgirba.com
"Every thing should have the right to be different."
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:
--
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
--
Andre Hora
_______________________________________________
Moose-dev mailing list
Moose-dev(a)iam.unibe.ch
https://www.iam.unibe.ch/mailman/listinfo/moose-dev Hi,
On 14 Jul 2011, at 19:46, Andre Hora wrote:
--
www.tudorgirba.com
"Being happy is a matter of choice."
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."