researchart / re19

control repo for the re19 artifacts evaluation committee
BSD 2-Clause "Simplified" License
3 stars 2 forks source link

Review of submission 17tizard #18

Closed neilernst closed 5 years ago

timm commented 5 years ago

@jtiz003 : I take it the install instructions for this are

please confirm.

BuddyWasisname2019 commented 5 years ago

The data is as described with the attached README.md, containing three tables (POS, labelled_sentences, verbs). The description of table labelled_sentences is consistent with the actual dataset. However, the POS and verbs table are lacking documentation (they contain more than simply an ID column), especially with respect to column names, which are missing in the README.md. In addition, it might be beneficial to provide a more verbose name for the POS table. Of course, some of these details might be covered in the paper itself (of which I do not yet have access to).

In addition, perhaps an entity relationship diagram would help with the documentation of the tables, since they are relational -- this should include the primary/foreign keys for each table.

Overall, the data appear to be interesting; although the data could benefit from being normalized (industry standard for relational databases); mainly for consistency: eg: dispraise application vs
dispraise for application and problem resolution vs
Problem resolution

Finally, I do not think the availability aspect of the the artifacts is met, as it is only available on THIS GitHub repository (as opposed to something like zenodo). So if the authors upload the artifacts to a publicly available location then perhaps we can consider upgrading the badge to available.

For now, provided that the documentation is updated to be "very carefully documented" as per my aforementioned comments, I recommend the badge of reusable.

neilernst commented 5 years ago

Hi @jtiz003 - we will be finalizing badges this weekend - can you please respond to the review?

RESubmission17 commented 5 years ago

Sorry for the slow reply.

I have updated the readme on the original repo with the following additions (https://github.com/RESubmission17/RE_Submission_17/blob/master/README.md)

We have made the data available on Zenodo with a link provided in the paper (https://zenodo.org/record/3315707#%23.XSZ8_-gzZPYz) The readme hasn't been updated on Zenodo yet, but I will do this once the documentation is finalized.

Feedback classifications: We propose a new set of classification in our paper, so the data follows these.

Please let me know if anything additional is needed.

@neilernst @BuddyWasisname2019 @timm

timm commented 5 years ago

@BuddyWasisname2019 : please note our current decision is reusbale unless you elect to fault the revised doc. nbow no doco is perfect but is the new doco better?

jtiz003 commented 5 years ago

@timm @neilernst Are we missing something for available? The data is available on zenodo

BuddyWasisname2019 commented 5 years ago

Based on the updated documentation and the added Zenodo link, I recommend the badge of available.

timm commented 5 years ago

ok, @neilernst : I think we should rebadge this "available" (not reusable). do u concur?