search

davidskalinder / mpeds-coder

MPEDS Annotation Interface
MIT License
0 stars 0 forks source link

Figure out why some checkbox entries are in the DB twice #90

Open davidskalinder opened 4 years ago

davidskalinder commented 4 years ago

See #85 (above https://github.com/davidskalinder/mpeds-coder/issues/85#issuecomment-637134966) for initial debug.

davidskalinder commented 4 years ago

So the bug with the checkboxes (which I'm increasingly suspicious is a separate problem) also shows up in the article-level data. It also turns up several times in the testing DB, and even once in the development deployment.

Originally posted by @davidskalinder in https://github.com/davidskalinder/mpeds-coder/issues/85#issuecomment-637093881

So that boldface bit is wrong. The two instances of the variable have different values, which is expected for checkbox batteries.

davidskalinder commented 4 years ago

From https://github.com/davidskalinder/mpeds-coder/issues/85#issuecomment-638464816:

As discussed in #87 (comment) ff., a workaround for this is to simply throw away all but the last value of these fields before analyzing the data. (It looks like this is always the entry that's visible to the coders, and if so will always be the best entry available.)

So I'm going to take this issue out of the epics for the export issues since that workaround should decouple them.

Pretty sure that this applies here too. I'm dropping it from the epic and slightly (informally) lowering the priority of this issue.