Done in <a class="commit-link" data-hovercard-type="commit" data-hovercard-url="https:

Test examples/ocr about deepdive HOT 5 CLOSED

hazyresearch commented on August 22, 2024

Test examples/ocr

from deepdive.

Comments (5)

zifeishan commented on August 22, 2024

Are you sure the example works? OCR example cannot pass on my side.

The issue is that there are TWO variables in ONE table.

Assigining ID will only give ONE id to each ROW, but there are two variables in this row.

How are we dealing with this? I guess one solution is to restrict that every table can only contain one variable, update the example and docs...

02:35:40 [] ERROR SQL execution failed (Reason: ERROR: duplicate key value violates unique constraint "label2_pkey"
  Detail: Key (id)=(620) already exists.):

   UPDATE label2 SET id = nextval('id_sequence')

02:35:40 [PostgresInferenceDataStoreComponent$PostgresInferenceDataStore(akka://deepdive)] ERROR org.postgresql.util.PSQLException: ERROR: duplicate key value violates unique constraint "label2_pkey"
  Detail: Key (id)=(620) already exists.
02:35:40 [PostgresInferenceDataStoreComponent$PostgresInferenceDataStore(akka://deepdive)] INFO  [Error] Please check the SQL cmd!
02:35:40 [inferenceManager] ERROR ERROR: duplicate key value violates unique constraint "label2_pkey"
  Detail: Key (id)=(620) already exists.
org.postgresql.util.PSQLException: ERROR: duplicate key value violates unique constraint "label2_pkey"
  Detail: Key (id)=(620) already exists.

from deepdive.

feiranwang commented on August 22, 2024

Two variables are not in the same table. We have label1, label2. The problem is id is declared as primary key. This can be solved by declaring id as bigint.

    CREATE TABLE label1(
    id    BIGSERIAL PRIMARY KEY, 
    wid   INT, val BOOLEAN);

    CREATE TABLE label2(
    id    BIGSERIAL PRIMARY KEY, 
    wid   INT, val BOOLEAN);

from deepdive.

zifeishan commented on August 22, 2024

Awesome! Would you fix it?

from deepdive.

feiranwang commented on August 22, 2024

Yes, I fixed it locally. I'll push it later on.

from deepdive.

feiranwang commented on August 22, 2024

Done in b0d123a

from deepdive.

Recommend Projects