-
-
Notifications
You must be signed in to change notification settings - Fork 36
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]: Reiz: Structural Source Code Search at Scale #3296
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @lutzhamel, @yuhc it looks like you're currently assigned to review this paper 🎉. Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post. ⭐ Important ⭐ If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-reviews) repository. As a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews 😿 To fix this do the following two things:
For a list of things I can do to help you, just type:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
|
|
Hi @mjsottile, I've tried to run the software within the latest docker on Ubuntu 20.04, but it requires a subtle change and doesn't return search results as expected. Shall I contact the author to resolve this issue? (The document doesn't state enough about the environment settings.) |
Please let me know about any problems you experience to build / run the Reiz on the bug tracker @yuhc. Thanks! |
Hi @isidentical, I'm running Ubuntu 20.04, Docker 19.03.13, docker-compose 1.26.2. The build instruction is usually expected to contain the system environment. I didn't see Could you explain more about what the dataset is built on top of ("~75 files from 10 different projects") and help check what's going wrong here? |
Interesting, I can successfully spin the instances with 3.9 though will definitely investigate (would you mind opening an issue on the tracker).
You should wait the API to start before accessing, since without it the web ui will just wait and timeout eventually. Would you mind sending me the logs (btw it would be better if you could create an issue on the repo itself!) Thanks! |
Created reizio/reiz.io#51.
Created reizio/reiz.io#52. |
Hi @isidentical, perhaps I missed it, but I don't see any instructions in the repo on how to run this software. Could you point me to the spot where it tells me how to install and run the software? Thanks. |
@isidentical, never mind just found it under the docs link.... |
@isidentical, docker-compose does not run on the given files...
|
What docker version are you using @lutzhamel? Please ensure you are using a newer one, something like |
@isidentical, here is what I am using:
|
Hi @isidentical , I just uploaded a screenshot of the 8000 page to reizio/reiz.io#52, and hope it would be helpful for your debugging. While I don't think the review is going in the right direction and I think I should state it clearly in case you may not know:
|
Thanks for your comment @yuhc! It is a bit of a block box situation for myself too, since everything is simply reproducible on my environment. I'll try to get everything setup on an AWS machine this weekend and let you both know about the exact environment. Thanks for your patience. |
@isidentical, have you considered setting up a working instance on a web based virtual machine like https://replit.com/ eliminating set up issues all together? @yuhc, thanks for your comments. I agree, as reviewers we should not be wrangling software, we should be just verifying that it works as advertised. I too will be waiting for a working instance before continuing the review. |
That is a great idea! i'll create an open instance as well as the instructions on a clear aws machine. Sorry for all the inconvenience i caused! |
@yuhc @lutzhamel I've deployed a public instance on the web address: |
👋 @yuhc, please update us on how your review is going (this is an automated reminder). |
👋 @lutzhamel, please update us on how your review is going (this is an automated reminder). |
Thanks for setting up reiz.io. It works for me. BTW, registering an AWS account should just take a few minutes. I guess something went wrong, and you may need to contact the support. |
@mjsottile all points are checked out
https://zenodo.org/record/5029255 (with the DOI
The
The list is correct. |
@whedon set 10.5281/zenodo.5029255 as archive |
OK. 10.5281/zenodo.5029255 is the archive. |
@whedon recommend-accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#2416 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#2416, then you can now move forward with accepting the submission by compiling again with the flag
|
Hi @isidentical, I am doing some final checks for publishing your article. It looks like most/all of your in-text citations are not using the appropriate citation command. Instead of using
I also see some issues with the references at the end to GitHub packages. Could you add the authors for those? It also looks like the "Github code search" reference does not have the correct URL, since it is the same as for "Grep.app". |
Cool, good work @isidentical ! BTW I'd recommend you to read "Cross-Language Code Search using Static and Dynamic Analyses" (which appears to be the latest work in the AST-based code search area). You'll get some knowledge of the state of the art from it and its citations. |
@whedon generate pdf |
@whedon generate pdf |
@whedon check references |
|
Thanks @kyleniemeyer, all references should have been fixed now. |
@whedon recommend-accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#2421 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#2421, then you can now move forward with accepting the submission by compiling again with the flag
|
@whedon accept deposit=true |
|
🐦🐦🐦 👉 Tweet for this paper 👈 🐦🐦🐦 |
🚨🚨🚨 THIS IS NOT A DRILL, YOU HAVE JUST ACCEPTED A PAPER INTO JOSS! 🚨🚨🚨 Here's what you must now do:
Any issues? Notify your editorial technical team... |
@lutzhamel, @yuhc – many thanks for your reviews here and to @mjsottile for editing this submission! JOSS relies upon the volunteer effort of people like you and we simply wouldn't be able to do this without you ✨ @isidentical – your paper is now accepted and published in JOSS ⚡🚀💥 |
🎉🎉🎉 Congratulations on your paper acceptance! 🎉🎉🎉 If you would like to include a link to your paper from your README use the following code snippets:
This is how it will look in your documentation: We need your help! Journal of Open Source Software is a community-run journal and relies upon volunteer effort. If you'd like to support us please consider doing either one (or both) of the the following:
|
Submitting author: @isidentical (Batuhan Taskaya)
Repository: https://github.com/reizio/reiz.io
Version: v1.0.0
Editor: @mjsottile
Reviewer: @lutzhamel, @yuhc
Archive: 10.5281/zenodo.5029255
Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.
Status
Status badge code:
Reviewers and authors:
Please avoid lengthy details of difficulties in the review thread. Instead, please create a new issue in the target repository and link to those issues (especially acceptance-blockers) by leaving comments in the review thread below. (For completists: if the target issue tracker is also on GitHub, linking the review thread in the issue or vice versa will create corresponding breadcrumb trails in the link target.)
Reviewer instructions & questions
@lutzhamel & @yuhc, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @mjsottile know.
✨ Please start on your review when you are able, and be sure to complete your review in the next six weeks, at the very latest ✨
Review checklist for @lutzhamel
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @yuhc
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
The text was updated successfully, but these errors were encountered: