Developer Portal Community

    cancel
    Showing results for 
    Search instead for 
    Did you mean: 
    SOLVED

    MES-Communication OpconTcpDDL <structArrays>! It's a bug or a feature?

    MES-Communication OpconTcpDDL <structArrays>! It's a bug or a feature?

    PKlein
    Established Member

    We had a case this week, which I will try to describe as best as I can:


    So I take the dataDownloadRequired telegram as an example, because we noticed the problem here. However, this problem probably exists with every telegram that uses a <structArrays>.

    Here it is the case that the first item, the item "Counter" defined under <structDef> has also been sent as defined (see point 1 and point 2 in Figure 1).

    The MACRO, perhaps also workflow or whatever in the MES, did not work properly and did not send a "counter" item for the other items of the structure (from two to eight) (see point 3 in Figure 1).

    Pic1.png

    The FB on the PLC for the DDL communication "OpconTcpDDL/DDLV4MessageFB" in version 1.2.2 has now not detected an error here.

    It seems that the FB only checks the first item. If the <counter< item were missing there, the FB would throw an error!

    Furthermore, it is also the case that the FB does not set the data structure on the PLC of the "orderList" array (rDdlData.orderlist) the values ​​of "counter" of items 2 to 8 to the value 0, it was set to the value 14, the value from the first item. (See point 5 and 6 in picture 2)

    Pic2.png

    This means that not only is the error not recognized, but “incorrect” values ​​are also transmitted.

    This could now be a bug within the FB module, but the scheme on the MES side does not recognize this error either, this could also be a desired feature.

    With this option, you could save on communication information if the data remained the same - just leave it out 😊

    Now you can decide for yourself: It's a bug or a feature?

    Maybe the question goes to the guys from BCI!

    11 REPLIES 11

    Düscha
    Long-established Member

    Have you activate AcceptIncompStructArray?

     

    PKlein
    Established Member

    yes!

    Düscha
    Long-established Member

    then it is correct that no error is set.

     

    The second quastion with the value "counter" we must check in the lib what happens if the parameter AcceptIncompStructArray set.

    PKlein
    Established Member

    If I remember correctly, the AcceptIncompStructArray variables were implemented as a request from our department in November 2019. The requirement here was that the variable StructArrayCount does not have to be set when requesting the telegrams and also not validated.

    To explain this requirement more detailed: It can happen that you do not know the number of elements of the "orderList" structure in advance. The number of "OrderList" is only known in the MES and not within the plc. Before this function was implemented , you always had to send two telegrams in order to query the number of elements from the MES with the first and then query the contents with the second telegram.

    This led to a large number of duplicate telegrams for one information and a major impact on the system cycle time.

    At that time, the requirement was not that missing elements (such as "counter") are not validated or that incorrect values ​​are also assigned to them. The definition of the structure must/should be fulfilled for all items. Here is the screenshot from the implementation at that time! The comment "Incomplete data posible" is a bit misleading and "UnknownStructArrayCount" might have been better.

     

    Bild1.jpg

    Unfortunately, the implementation as it is today leads to sensitive MES data being misinterpreted.This has a biginfluence on the quality of the productions and the production. It must not be that wrong values are assumed here.


    And why should one use the value of the last valid entry and not the one before or before that?
    This issue should be fixed!

     

     

    Icon--AD-black-48x48Icon--address-consumer-data-black-48x48Icon--appointment-black-48x48Icon--back-left-black-48x48Icon--calendar-black-48x48Icon--center-alignedIcon--Checkbox-checkIcon--clock-black-48x48Icon--close-black-48x48Icon--compare-black-48x48Icon--confirmation-black-48x48Icon--dealer-details-black-48x48Icon--delete-black-48x48Icon--delivery-black-48x48Icon--down-black-48x48Icon--download-black-48x48Ic-OverlayAlertIcon--externallink-black-48x48Icon-Filledforward-right_adjustedIcon--grid-view-black-48x48IC_gd_Check-Circle170821_Icons_Community170823_Bosch_Icons170823_Bosch_Icons170821_Icons_CommunityIC-logout170821_Icons_Community170825_Bosch_Icons170821_Icons_CommunityIC-shopping-cart2170821_Icons_CommunityIC-upIC_UserIcon--imageIcon--info-i-black-48x48Icon--left-alignedIcon--Less-minimize-black-48x48Icon-FilledIcon--List-Check-grennIcon--List-Check-blackIcon--List-Cross-blackIcon--list-view-mobile-black-48x48Icon--list-view-black-48x48Icon--More-Maximize-black-48x48Icon--my-product-black-48x48Icon--newsletter-black-48x48Icon--payment-black-48x48Icon--print-black-48x48Icon--promotion-black-48x48Icon--registration-black-48x48Icon--Reset-black-48x48Icon--right-alignedshare-circle1Icon--share-black-48x48Icon--shopping-bag-black-48x48Icon-shopping-cartIcon--start-play-black-48x48Icon--store-locator-black-48x48Ic-OverlayAlertIcon--summary-black-48x48tumblrIcon-FilledvineIc-OverlayAlertwhishlist