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

Early payout fails when participants submit attestations with the attest_attendees extrinsic #278

Closed
clangenb opened this issue Nov 9, 2022 · 3 comments

Comments

@clangenb
Copy link
Member

clangenb commented Nov 9, 2022

I have currently no idea, what the issue could be. I added the exact setup that fails with my local node and it passes the unit test: #277

According to the below data, everything seems to be ok in the node. And the unit-test uses exactly these parameters, and it passes, which it should. What could be the issue?

Attestation Registry:

AttestationRegistry[2, 1 (5FLSigC9HGRKVhB9FiEo4Y3koPsNmBmLJbpXg2mp1hXcS59Y)] = Some([d43593c715fdd31c61141abd04a99fd6822c8558854ccde39a5684e7a56da27d (5GrwvaEF...), 8eaf04151687736326c9fea17e25fc5287613693c912909cb226aa4794f26a48 (5FHneW46...)])
AttestationRegistry[2, 2 (5FHneW46xGXgs5mUiveU4sbTyGBzmstUspZC92UhjJM694ty)] = Some([d43593c715fdd31c61141abd04a99fd6822c8558854ccde39a5684e7a56da27d (5GrwvaEF...), 90b5ab205c6974c9ea841be688864633dc9ca8a357843eeacf2314649965fe22 (5FLSigC9...)])
AttestationRegistry[2, 3 (5GrwvaEF5zXb26Fz9rcQpDWS57CtERHpNehXCPcNoHGKutQY)] = Some([8eaf04151687736326c9fea17e25fc5287613693c912909cb226aa4794f26a48 (5FHneW46...), 90b5ab205c6974c9ea841be688864633dc9ca8a357843eeacf2314649965fe22 (5FLSigC9...)])

Ceremony-Stats

 ../target/release/encointer-client-notee print-ceremony-stats -c sqm1v79dF6b                                                               
{
  "communityCeremony": [
    {
      "geohash": "0x73716d3176",
      "digest": "0xf08c911c"
    },
    2
  ],
  "assignment": {
    "bootstrappersReputables": {
      "m": 3,
      "s1": 1,
      "s2": 1
    },
    "endorsees": {
      "m": 2,
      "s1": 1,
      "s2": 1
    },
    "newbies": {
      "m": 2,
      "s1": 1,
      "s2": 1
    },
    "locations": {
      "m": 9,
      "s1": 8,
      "s2": 7
    }
  },
  "assignmentCount": {
    "bootstrappers": 3,
    "reputables": 0,
    "endorsees": 0,
    "newbies": 0
  },
  "meetupCount": 1,
  "meetups": [
    {
      "index": 1,
      "location": {
        "lat": "35.2489836657264490327",
        "lon": "18.700103759765625"
      },
      "time": 1668077280000,
      "registrations": [
        [
          "5FLSigC9HGRKVhB9FiEo4Y3koPsNmBmLJbpXg2mp1hXcS59Y",
          {
            "index": 3,
            "registrationType": "Bootstrapper"
          }
        ],
        [
          "5GrwvaEF5zXb26Fz9rcQpDWS57CtERHpNehXCPcNoHGKutQY",
          {
            "index": 1,
            "registrationType": "Bootstrapper"
          }
        ],
        [
          "5FHneW46xGXgs5mUiveU4sbTyGBzmstUspZC92UhjJM694ty",
          {
            "index": 2,
            "registrationType": "Bootstrapper"
          }
        ]
      ]
    }
  ]
}
@clangenb clangenb changed the title Early payout fails when participants submit attestations with the attest_attendees claim Early payout fails when participants submit attestations with the attest_attendees extrinsic Nov 9, 2022
@clangenb
Copy link
Member Author

clangenb commented Nov 9, 2022

The early payout seems to work, if we use the new extrinsic in the encointer-node's CI. This seems to be an issue with the app then.

It is still very mysterious to me.

@brenzi
Copy link
Member

brenzi commented Nov 10, 2022

As reported by @clangenb : there was an error in the app 1.8.9 and 1.8.10:
encointer/encointer-wallet-flutter#866

however, the fact that tests with homogenious version 1.8.9 of the app succeeded shows that there is bug in meetup validation as well: submitting more attestaions than vote -1 should not be accepted: #279

@clangenb
Copy link
Member Author

I will close this one as it was on the app side and could be fixed there: encointer/encointer-wallet-flutter#867

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

No branches or pull requests

2 participants