mbeddr / mbeddr.core

The mbeddr core. An extensible C
Eclipse Public License 2.0
223 stars 77 forks source link

No error in composite component when provided outer port not delegated to any inner instance provided port #897

Closed stephaneberle9 closed 10 years ago

stephaneberle9 commented 10 years ago

One gets only an InvocationTargetException at generation time and then has to do some "advanced guessing"...

bkolb commented 10 years ago

Where do you still see the problem?

stephaneberle9 commented 10 years ago

in com.itemis.smartmeter.ssec.implementation.metrology.MetrologyDataPublisherStack

simply remove

delegate measurementIntervalResultHandler to energyDataPublisherInst.measurementIntervalResultHandler

on

energyDataPublisherInst

bkolb commented 10 years ago

I can see the error. Can you please F5 and check again. Do you run with the powerSave mode enabled?

stephaneberle9 commented 10 years ago

F5 doesn't help. Tested on 2 different machines (Windows 7 and Windows 8.1). Power save is definitely not enabled.

stephaneberle9 commented 10 years ago

There is still no error message in this case. Check e.g.,

com.itemis.embedded.platform.energy.tests.helpers.PowerReadingsComputationTestHarnessImpl

http://localhost:8080/select/com.itemis.smartmeter.main/r:cf7a909d-ccc0-4cbe-884c-d39191aede3d/4904857930387676603/

There is a "provides ISimulatedEnergyData simulatedEnergyData" which is not connected to any inner instance port and no error message complaining about that.

markusvoelter commented 10 years ago

The problem is that the error only appears for client/server interfaces. However, the one discussed here is a S/R interface.

markusvoelter commented 10 years ago

I have fixed the problem. However, it does not yet take into account "direct use" of the S/R interface by the composite component. In the case of the C/S interface we check for triggers. It is not clear what we need to check for in the case of S/R interfaces. Closing for now.