Please Note This forum exists for community support for the Mango product family and the Radix IoT Platform. Although Radix IoT employees participate in this forum from time to time, there is no guarantee of a response to anything posted here, nor can Radix IoT, LLC guarantee the accuracy of any information expressed or conveyed. Specific project questions from customers with active support contracts are asked to send requests to support@radixiot.com.

jeremie

Hi,

In BACNet4J, COV is not allowed for Analog Input, whereas it should.
I think we have to add :

       
    static {
        supportedObjectTypes.add(ObjectType.accessDoor);
...
        **supportedObjectTypes.add(ObjectType.analogInput);**
        
        supportedPropertyIdentifiers.add(PropertyIdentifier.presentValue);
...
    }

in com.serotonin.bacnet4j.obj.ObjectCovSubscription class.

Regards,

Jérémie

jeremie

Hi,

In BACNet4J, COV is not allowed for Analog Input, whereas it should.
I think we have to add :

       
    static {
        supportedObjectTypes.add(ObjectType.accessDoor);
...
        **supportedObjectTypes.add(ObjectType.analogInput);**
        
        supportedPropertyIdentifiers.add(PropertyIdentifier.presentValue);
...
    }

in com.serotonin.bacnet4j.obj.ObjectCovSubscription class.

Regards,

Jérémie

jeremie

Thank for your SimpleSubscriptionClient test file :)
Maybe you can add the unsubscribe code (because COV is done with an indefinite lifetime) :


localDevice.send(d, new SubscribeCOVRequest(new UnsignedInteger(0), oid, null, null));

jeremie

Hi Matthew!,

Playing with services, I have discovered a potential none expected behaviour, with the list of remote devices of a local device.

First, this list can contain remotes devices doubles.
For example, try this :


localDevice.sendBroadcast(0xBAC0, new WhoIsRequest(null, null));                        
localDevice.sendBroadcast(0xBAC0, new WhoIsRequest(null, null));
for (RemoteDevice d : localDevice.getRemoteDevices()) {
      System.out.println("Device found : " + d);
}

Moreover, I do not think it is normal to always have the full remote devices list available, after a call of WhoHasRequest method :


localDevice.sendBroadcast(0xBAC0, new WhoIsRequest(null, null));
localDevice.sendBroadcast(0xBAC0,new WhoHasRequest(null, new ObjectIdentifier(ObjectType.binaryInput, 4097)));

for (RemoteDevice d : localDevice.getRemoteDevices()) {
      System.out.println("Device found : " + d);
}

In conclusion, we always add remote devices in the list.
Therefore, can we imagine to add code for cleaning the remote devices list, before each call of WhoIsRequest, WhoHasRequest, etc. ?

Thank you in advance!

Jérémie

jeremie

Hi Matthew!,

Playing with services, I have discovered a potential none expected behaviour, with the list of remote devices of a local device.

First, this list can contain remotes devices doubles.
For example, try this :


localDevice.sendBroadcast(0xBAC0, new WhoIsRequest(null, null));                        
localDevice.sendBroadcast(0xBAC0, new WhoIsRequest(null, null));
for (RemoteDevice d : localDevice.getRemoteDevices()) {
      System.out.println("Device found : " + d);
}

Moreover, I do not think it is normal to always have the full remote devices list available, after a call of WhoHasRequest method :


localDevice.sendBroadcast(0xBAC0, new WhoIsRequest(null, null));
localDevice.sendBroadcast(0xBAC0,new WhoHasRequest(null, new ObjectIdentifier(ObjectType.binaryInput, 4097)));

for (RemoteDevice d : localDevice.getRemoteDevices()) {
      System.out.println("Device found : " + d);
}

In conclusion, we always add remote devices in the list.
Therefore, can we imagine to add code for cleaning the remote devices list, before each call of WhoIsRequest, WhoHasRequest, etc. ?

Thank you in advance!

Jérémie

jeremie

Hi,

I think I have an idea what Ilyass tries to mean :

when he uses COV on an object with a lifetime greater than 0, he receives correcty events when the PresentValue of the COVing object changes.

But when he tries with a lifetime equals to 0 (no timeout for COV subscription), he catchs only one time the COV event.

For information, in Mango, the point COVing mechanism is configure with a lifetime greather than 0 : you use COV for a certain time, and when this time is finished, you COV again. Have you got a reason to do this, instead of use a lifetime of 0?

jeremie

I will :)

In BACNet, reliability property for AccessDoor Object is always required.
Therefore, I think the declaration is uncorrect in the ObjectProperties.java file (com.serotonin.bacnet4j.obj package) :

add(ObjectType.accessDoor, PropertyIdentifier.reliability, Reliability.class, false, **false**, null);

=>

add(ObjectType.accessDoor, PropertyIdentifier.reliability, Reliability.class, false, **true**, null);

jeremie

Many thanks for your help Matthew!

For your information, I have got an OpenSource BACNET Server for Linux (C/C++), embedded in a proprietary equipment.

jeremie

Hi Matthew :)

During the last call to the parseServiceDate() function (before the exception appears), serviceData has this value :

[c,7,80,30,0,1e,29,4b,4e,c4,7,80,30,0,4f,29,4d,4e,75,c,0,4f,54,45,53,20,44,4f,
4f,52,20,31,4f,29,4f,4e,91,1e,4f,29,55,4e,91,0,4f,29,6f,4e,82,4,0,4f,29,24,4e,
91,0,4f,29,67,4e,91,0,4f,29,51,4e,10,4f,29,57,4e,0,0,0,0,0,0,0,0,0,0,0,0,0,0,
0,0,4f,29,68,4e,91,0,4f,29,e7,4e,91,0,4f,29,e9,4e,91,0,4f,29,eb,4e,91,2,4f,29,
e6,4e,1e,4f,29,e3,4e,64,4f,29,e5,4e,7,8,4f,29,1c,4e,75,c,0,4f,54,45,53,20,44,
4f,4f,52,20,31,4f,29,e2,4e,91,0,4f,29,e4,4e,c4,1,0,20,0,4f,29,e8,4e,0,4f,29,ea,
5e,91,2,91,20,5f,1f]

jeremie

Not sure it helps... :

Below the stack trace of the exception :

Exception in thread "main" com.serotonin.bacnet4j.exception.BACnetErrorException: Property: Missing required parameter
	at com.serotonin.bacnet4j.type.Encodable.popEnd(Encodable.java:157)
	at com.serotonin.bacnet4j.type.Encodable.readWrapped(Encodable.java:388)
	at com.serotonin.bacnet4j.type.Encodable.readEncodable(Encodable.java:349)
	at com.serotonin.bacnet4j.type.constructed.ReadAccessResult$Result.<init>(ReadAccessResult.java:126)
	at sun.reflect.GeneratedConstructorAccessor4.newInstance(Unknown Source)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:27)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:513)
	at com.serotonin.bacnet4j.type.Encodable.read(Encodable.java:191)
	at com.serotonin.bacnet4j.type.constructed.SequenceOf.<init>(SequenceOf.java:66)
	at com.serotonin.bacnet4j.type.Encodable.readSequenceOf(Encodable.java:262)
	at com.serotonin.bacnet4j.type.Encodable.readOptionalSequenceOf(Encodable.java:296)
	at com.serotonin.bacnet4j.type.constructed.ReadAccessResult.<init>(ReadAccessResult.java:64)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:39)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:27)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:513)
	at com.serotonin.bacnet4j.type.Encodable.read(Encodable.java:191)
	at com.serotonin.bacnet4j.type.constructed.SequenceOf.<init>(SequenceOf.java:54)
	at com.serotonin.bacnet4j.type.Encodable.readSequenceOf(Encodable.java:251)
	at com.serotonin.bacnet4j.service.acknowledgement.ReadPropertyMultipleAck.<init>(ReadPropertyMultipleAck.java:50)
	at com.serotonin.bacnet4j.service.acknowledgement.AcknowledgementService.createAcknowledgementService(AcknowledgementService.java:48)
	at com.serotonin.bacnet4j.apdu.ComplexACK.parseServiceData(ComplexACK.java:189)
	at com.serotonin.bacnet4j.npdu.ip.IpMessageControl.waitForAck(IpMessageControl.java:696)
	at com.serotonin.bacnet4j.npdu.ip.IpMessageControl.send(IpMessageControl.java:292)
	at com.serotonin.bacnet4j.npdu.ip.IpMessageControl.send(IpMessageControl.java:236)
	at com.serotonin.bacnet4j.LocalDevice.send(LocalDevice.java:430)
	at com.serotonin.bacnet4j.LocalDevice.send(LocalDevice.java:420)
	at com.serotonin.bacnet4j.LocalDevice.send(LocalDevice.java:413)
	at com.serotonin.bacnet4j.LocalDevice.readProperties(LocalDevice.java:869)
	at com.serotonin.bacnet4j.test.TestARD.main(TestARD.java:139)

And here the serviceRequest object who causes this exception :

If it is not enough, can you say me what function/parameters to survey in the source code? Thank you :)

jeremie

@jeremie

Latest posts made by jeremie