Lorawan-stack: डुप्लिकेट सहसंबंध ID

को निर्मित 10 मई 2019 · 8टिप्पणियाँ · स्रोत: TheThingsNetwork/lorawan-stack

सारांश

कुछ घटनाओं में सहसंबंध आईडी (यानी gs.up.receive और as.up.receive ) डुप्लिकेट हैं।

प्रजनन करने कि प्रक्रिया

घटनाओं की सदस्यता लें, यानी ttn-lw-cli events subscribe --gateway-id ...

अब आप क्या देखते हैं?

उदाहरण के लिए:

{
  "name": "gs.up.receive",
  "time": "2019-05-10T08:48:36.397803Z",
  "identifiers": [
    {
      "gateway_ids": {
        "gateway_id": "tektelic-micro",
        "eui": "647FDAFFFE0059AB"
      }
    }
  ],
  "data": {
    "@type": "type.googleapis.com/ttn.lorawan.v3.UplinkMessage",
    "raw_payload": "QKogASYAkdgBU2NToyoatD/C",
    "settings": {
      "data_rate": {
        "lora": {
          "bandwidth": 125000,
          "spreading_factor": 7
        }
      },
      "coding_rate": "4/5",
      "frequency": "867900000",
      "timestamp": 3375161699
    },
    "rx_metadata": [
      {
        "gateway_ids": {
          "gateway_id": "tektelic-micro",
          "eui": "647FDAFFFE0059AB"
        },
        "timestamp": 3375161699,
        "rssi": 2,
        "snr": 10,
        "uplink_token": "ChwKGgoOdGVrdGVsaWMtbWljcm8SCGR/2v/+AFmrEOPCs8kM"
      }
    ],
    "received_at": "2019-05-10T08:48:36.397329Z",
    "correlation_ids": [
      "gs:conn:01DAGEFV4TBH5K9AVAX86NPDXM",
      "gs:uplink:01DAGEW31DQZQNSCQPJ28KGFMF"
    ],
    "gateway_channel_index": 4
  },
  "correlation_ids": [
    "gs:conn:01DAGEFV4TBH5K9AVAX86NPDXM",
    "gs:uplink:01DAGEW31DQZQNSCQPJ28KGFMF",
    "gs:conn:01DAGEFV4TBH5K9AVAX86NPDXM",
    "gs:uplink:01DAGEW31DQZQNSCQPJ28KGFMF"
  ],
  "origin": "Johans-MacBook-Pro.local"
}

इसके बजाय आप क्या देखना चाहते हैं?

correlation_ids में कोई डुप्लिकेट नहीं

आप इसे कैसे लागू करने का प्रस्ताव देते हैं?

सहसंबंध आईडी शायद दो बार जुड़ जाती हैं। यह उन घटनाओं पैकेज में हो सकता है जहां संदर्भ में होने पर सहसंबंध आईडी जोड़े जाते हैं।

क्या आप इसे स्वयं कर सकते हैं और एक पुल अनुरोध सबमिट कर सकते हैं?

हाँ

bug shared

स्रोत

johanstokking

सबसे उपयोगी टिप्पणी

मैं @rvolosatovs मर्ज एल्गोरिथ्म (मेरी टिप्पणी देखें) का उपयोग करना पसंद करूंगा। हम पहले से ही सहसंबंध आईडी के विलय के लिए एक कुशल कार्यान्वयन करने में समय बिताते हैं, इसलिए यह शर्म की बात होगी यदि हम @ pgalic96 के प्रस्तावित मानचित्र कार्यान्वयन के साथ विलय / कटौती करेंगे जो कि बहुत कम कुशल है और साथ ही

htdvisser 24 मई 2019

👍2

सभी 8 टिप्पणियाँ

@htdvisser समस्या यह है कि घटना पेलोड से सहसंबंध आईडी संदर्भ के उन लोगों के लिए संलग्न हैं, यानी यदि आपके पास यह है;

ctx := events.ContextWithCorrelationID(ctx, fmt.Sprintf("gs:uplink:%s", events.NewCorrelationID()))
msg.CorrelationIDs = append(msg.CorrelationIDs, events.CorrelationIDsFromContext(ctx)...)
registerReceiveUplink(ctx, conn.Gateway(), msg)

सहसंबंध आईडी घटना में डुप्लिकेट हैं क्योंकि वे दोनों ctx और msg ( GetCorrelationIDs() माध्यम से) में हैं।

क्या हमें यहां की विशिष्टता की जांच करनी चाहिए?

https://github.com/TheThingsNetwork/lorawan-stack/blob/master/pkg/events/events.go#L148

johanstokking 10 मई 2019

हाँ, हम में मर्ज एल्गोरिथ्म स्थानांतरित कर सकते हैं pkg/events/correlation_context.go (अपनी ही समारोह में mergeCorrelationIDs ) और उपयोग में pkg/events/events.go :

    if cids := data.GetCorrelationIDs(); len(cids) > 0 {
        cids = append(cids[:0:0], cids...)
        sort.Strings(cids)
        evt.innerEvent.CorrelationIDs = mergeCorrelationIDs(evt.innerEvent.CorrelationIDs, cids)
    }

हमें संभवतः एक नोट जोड़ना चाहिए कि evt.innerEvent.CorrelationIDs को सॉर्ट किया जाना चाहिए (जो यह है, क्योंकि संदर्भ में सहसंबंध आईडी सॉर्ट किए गए हैं)।

htdvisser 23 मई 2019

क्या स्थिति है @ pgalic96 ?

johanstokking 23 मई 2019

सामान के साथ व्यस्त हो गया, मैंने विशिष्टता की जांच की, कुछ इस तरह से:

if data, ok := data.(interface{ GetCorrelationIDs() []string }); ok {
        mapCorrelationIds := make(map[string]struct{})
        for _, correlationID := range evt.innerEvent.CorrelationIDs {
            mapCorrelationIds[correlationID] = struct{}{}
        }
        for _, correlationID := range data.GetCorrelationIDs() {
            if _, ok := mapCorrelationIds[correlationID]; !ok {
                evt.innerEvent.CorrelationIDs = append(evt.innerEvent.CorrelationIDs, correlationID)
            }
        }
    }

अभी तक एक परीक्षण नहीं लिखा है, लेकिन यह TestCorrelationID साथ एक घटना बना रहा है और फिर पेलोड में समान TestCorrelationID की आपूर्ति करेगा, और फिर जाँच करेगा कि संदेश के सहसंबंध की लंबाई क्या है वास्तव में 1 और 2 नहीं।

pgalic96 24 मई 2019

चर नाम के अलावा यह आम तौर पर अच्छा लगता है, हालांकि evt.innerEvent.CorrelationIDs इस तरह से काटे नहीं जाते हैं

johanstokking 24 मई 2019

htdvisser 24 मई 2019

👍2

@ pgalic96 क्या आप स्टेटस अपडेट दे सकते हैं?

johanstokking 28 मई 2019

मैं इस सप्ताह अभी तक इस पर काम नहीं कर रहा हूं, परीक्षा के बाद कल कार्यालय आ रहा हूं। कल इसे अंतिम रूप देने की कोशिश करेंगे।

मैं @htdvisser सुझाव के अनुसार कोड बदल

pgalic96 28 मई 2019

👍1

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स