Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

peer reviews #7

Open
klano opened this issue May 23, 2016 · 3 comments
Open

peer reviews #7

klano opened this issue May 23, 2016 · 3 comments

Comments

@klano
Copy link

klano commented May 23, 2016

Here is my feedback. Evaluation forms will be submitted by my co-author.

Excel solution:

This has clearly produced good results, but it is not clear how it uses transformation technologies, instead of existing non-MT optimisation tools.
In addition, the heuristics in Section 3 seem to require tuning of the algorithm to specific CRA problems. The idea of creating a main class could lead to unbalanced class diagrams. Such diagrams are local but not global optima. The approach does not seem to produce optimal solutions (eg., for C, D, E cases), although the efficiency is very impressive.

Henshin solution:

An interesting investigation of GA application to the problem. It is unclear how Henshin rules are integrated into the GA - is the control of these external? Is the GA and rule control coded in Java? Some example code would be helpful. The population sizes used seem quite small.
Section 4: 'CPA' should be 'CRA'
At some point the best results obtained per model should be given.
Section 4.2: explain the abbreviations RS, JSC, etc.

@maximiliano-vela
Copy link

maximiliano-vela commented May 23, 2016

Thanks for the feedback Kevin, but I don't think you were one of our opponents (Excel solution). Please check so that the other solution is not left unreviewed.

Cheers,

Maxi.

@fikovnik
Copy link

Indeed. Kevin, please have a look at the MDEOptimiser that you were supposed to review. Thanks a lot!

@fikovnik
Copy link

@klano Could you please change the title to "Kevin Lano evaluation of Excel". Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants