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

Review of submission 52villamizar #20

Closed
neilernst opened this issue Jul 3, 2019 · 5 comments

Comments

@neilernst
Copy link
Collaborator

commented Jul 3, 2019

No description provided.

@artifacts-reviewer

This comment has been minimized.

Copy link
Collaborator

commented Jul 7, 2019

I thank the authors for a well structured artifact submission. The authors provide the materials used in their experiment to validate the proposed approach. It would be great if the authors would consider also sharing the scripts, etc. used to do the natural language processing approach described in the paper.

There are also some minor issues that should be corrected:

  • There is a typo in the Readme file: "inspections.," remove the .
  • The Training_Experiment.pptx file should be translated to English.
  • ~$llowUp_questionnaire_RT_A.docx should be deleted.
  • In each of the task description documents, the defect type boxes appear to be misplaced with text wrapping around the sides of the box or some of the text that should be before the box appearing after the box.

I recommend a badge of Available.

@timm

This comment has been minimized.

Copy link
Contributor

commented Jul 8, 2019

@neilernst : please concur on "Available" (or switch to "SecondReview").

IMHO, while the issues raised by @artifacts-reviewer are things the authors might care to fix (*), those fixes are optional and would not alter the "available-ness" of this artifact.

(*) BTW, I concur that making the NL scripts available would be good, that is neither here nor there about assigning a badge to this artifact.

@hrguarinv

This comment has been minimized.

Copy link
Collaborator

commented Jul 8, 2019

Dear @neilernst , @timm

Thank you so much for your contributions. We will consider all the comments. The repository will be updated tomorrow. Is it ok?

Abs

@neilernst

This comment has been minimized.

Copy link
Collaborator Author

commented Jul 8, 2019

Looks good to me.

@hrguarinv

This comment has been minimized.

Copy link
Collaborator

commented Jul 12, 2019

Dear @neilernst @timm

We have added to our repository the instructions to access our scripts that use NLP. I hope it's not too late for your reviews.

Abs,

@timm timm closed this Jul 19, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.