SiriusLab / ModelDebugging

Runtime and tools to bring execution and simulation in your domain specific tooling built with Sirius and/or Eclipse Modeling Framework.
Eclipse Public License 1.0
6 stars 10 forks source link

Generated Multidimentional Trace fail #33

Closed fcoulon closed 7 years ago

fcoulon commented 7 years ago

The generation of the trace addon doesn't work anymore.

Step to reproduce

Right click on a xdsml project (org.gemoc.sample.legacyfsm.xsfsm from the sequential example) and then select GEMOC Language > Generated Multidimentional Trace Addon project for language

The cammand fails and we get:

java.lang.NullPointerException
    at org.eclipse.gemoc.trace.metamodel.generator.TraceMMExplorer$3.apply(TraceMMExplorer.java:119)
    at org.eclipse.gemoc.trace.metamodel.generator.TraceMMExplorer$3.apply(TraceMMExplorer.java:1)
    at org.eclipse.xtext.xbase.lib.IteratorExtensions.findFirst(IteratorExtensions.java:102)
    at org.eclipse.xtext.xbase.lib.IterableExtensions.findFirst(IterableExtensions.java:80)
    at org.eclipse.gemoc.trace.metamodel.generator.TraceMMExplorer.<init>(TraceMMExplorer.java:122)
    at org.eclipse.gemoc.trace.metamodel.generator.TraceMMGenerator.<init>(TraceMMGenerator.java:93)
    at org.eclipse.gemoc.trace.gemoc.generator.GenericEngineTraceAddonGenerator.generateTraceMetamodelAndPlugin(GenericEngineTraceAddonGenerator.java:169)
    at org.eclipse.gemoc.trace.gemoc.generator.GenericEngineTraceAddonGenerator.generateCompleteAddon(GenericEngineTraceAddonGenerator.java:161)
    at org.eclipse.gemoc.trace.gemoc.generator.TraceAddonGeneratorIntegration.generateAddon(TraceAddonGeneratorIntegration.java:169)
    at org.eclipse.gemoc.trace.gemoc.generator.TraceAddonGeneratorIntegration.generateAddon(TraceAddonGeneratorIntegration.java:120)
    at org.eclipse.gemoc.trace.gemoc.ui.commands.XDSMLProject2TraceAddonHandler$1.run(XDSMLProject2TraceAddonHandler.java:73)
    at org.eclipse.core.internal.jobs.Worker.run(Worker.java:55)

Cause

This seems happening by a wrong crossref inside org.eclipse.gemoc.trace.metamodel.generator/model/base.ecore (pointing to fr.inria.diverse.trace.commons.model/model/GenericTrace.ecore)

PS: org.eclipse.gemoc.executionframework.reflectivetrace.model/model/GemocExecutionEngineTrace.ecore has also a crossref in fr.inria.diverse

ebousse commented 7 years ago

Indeed, the change from fr.inria.diverse.trace to org.eclipse.gemoc.trace did not go perfectly well. I'll make a pass with a sed script.

It this urgent to fix for some demo/work in particular?

ebousse commented 7 years ago

Fix available in branch fix-more-eclipse-renamings, but it will not be merged until the Eclipse IP review is finished I believe.

fcoulon commented 7 years ago

Nice!

Thanks ;)