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

Multiple connectors and group #26

Closed
brice-morin opened this issue Jun 12, 2012 · 6 comments
Closed

Multiple connectors and group #26

brice-morin opened this issue Jun 12, 2012 · 6 comments

Comments

@brice-morin
Copy link
Collaborator

It seems impossible to have multiple connectors targeting the same instance within a group...

@ghost ghost assigned ffleurey Jun 12, 2012
@vassik
Copy link
Contributor

vassik commented Aug 24, 2015

I get the same issue, here is the code snip where i get this:

//alarmpsm.thingml
configuration fragment AlarmPIMMQTTConfig {
    instance mqttalarmpsm : MQTTClientPSM   
    instance alarmpim : AlarmPim

    connector alarmpim.mqttport => mqttalarmpsm.mqttport
}

//alarmpim.thingml
configuration AlarmGatewayTest
  @output_folder "/home/tmp/"
  @add_c_libraries "paho-mqtt3a tmlmqtt cson"
{
    group alarmmqttpsm : AlarmPIMMQTTConfig 
    instance gateway : GatewayImpl

    connector gateway.alarmSirene => alarmmqttpsm.alarmpim.sirene
    connector gateway.alarmAlert => alarmmqttpsm.alarmpim.alert
    connector gateway.alarmMonitor => alarmmqttpsm.alarmpim.monitor
}

for the second and third connector in AlarmGatewayTest, the following error pops up

Description Resource Path Location Type
An expection occured while resolving the proxy for: EMFTEXT_INTERNAL_URI_FRAGMENT_14_alert. (java.lang.NullPointerException) gateway_g2_test_config.thingml /ThingMLSamples/thingmlg2demo line 22 EMFText Problem

@brice-morin brice-morin assigned nharrand and unassigned ffleurey Aug 24, 2015
@brice-morin
Copy link
Collaborator Author

@Lyadis this one you should look into in priority.

@ffleurey
Copy link
Collaborator

Seems like an EMFText issue... do we have good motivations for keeping this group concept or should we re-think it a bit? @Lyadis is going to write a few tests.

@brice-morin
Copy link
Collaborator Author

Well, it is useful e.g in the case of Anatoly, as it makes it easy to include a pre-configured PSM fragment into a PIM model, without polluting the model with the PSM details. Then it is more in the methodology we should advise people to use it wisely.

@nharrand
Copy link
Contributor

After some tests, it appears that the problem happens only when the fragment configuration is in a separated file.
Indeed when both configurations are in the same file, the compiler doesn't complain.

@brice-morin
Copy link
Collaborator Author

Ok. Can you create proper test cases for both cases, so that we can have them included in the test suite.

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

No branches or pull requests

4 participants