-
-
Notifications
You must be signed in to change notification settings - Fork 39
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]: elk: A Python package to elicit latent knowledge from LLMs #6511
Comments
Hello humans, I'm @editorialbot, a robot that can help you with some common editorial tasks. 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:
|
Software report:
Commit count by author:
|
Paper file info: 📄 Wordcount for ✅ The paper includes a |
License info: ✅ License found: |
|
Hi @lauritowal @isdanni @praneethd7 - this is the review thread for the submission. All of our communications will happen here from now on. The title was updated on the pre-review but I'm not sure why it wasn't updated here - I'll try to sort it! Meanwhile, please check the post at the top of the issue for instructions on how to generate your own review checklist. As you go over the submission, please check any items that you feel have been satisfied. There are also links to the JOSS reviewer guidelines. The JOSS review is different from most other journals. Our goal is to work with the authors to help them meet our criteria instead of merely passing judgment on the submission. As such, the reviewers are encouraged to submit issues directly in the software repository. If you do so, please mention this thread so that a link is created (and I can keep an eye on what is happening). Please also feel free to comment and ask questions in this thread. It is often easier to post comments/questions/suggestions as you come across them instead of waiting until you've reviewed the entire package. Please feel free to ping me (@mooniean) if you have any questions or concerns. Thanks! |
@arfon Hello! The title of the paper was updated on the Pre-Review but didn't move properly for the review - is there a way to fix that? Thanks in advance! |
Review checklist for @isdanniConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
👋 @mooniean – just removing the |
Review checklist for @praneethd7Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hello Walter @lauritowal! I just started the review process. I wanted to get some clarity on the following checkbox item:
I am curious that there are more contributors on the GitHub repository than the number of authors listed in the submission. Could you confirm if you checked with all the contributors regarding the authorship? Perhaps, some of their contributions are minor but I wanted to make sure. |
Hi @praneethd7 @isdanni and @lauritowal! Checking in and poking on the review and how things are coming along! |
@mooniean I have asked the corresponding author to provide more clarity on the authorship. Please advise if I should wait for their response to continue my review process since authorship is crucial. |
Thanks for checking in! I finished the initial round of review in the linked issue, waiting for author response. |
👋 @lauritowal - just checking into see if you are able to provide feedback in to the reviewers in the very near future? |
@praneethd7 I can just include the others too |
👋 @mooniean - it looks like @praneethd7 had a question for you here...
|
@mooniean - just pinging you again on this one. Thanks! 👋 @mooniean - it looks like @praneethd7 had a question for you here... @mooniean I have asked the corresponding author to provide more clarity on the authorship. Please advise if I should wait for their response to continue my review process since authorship is crucial. |
Apologies! I thought I had responded - I believe the author has replied to the reviewer
So now is up to the reviewer if that's the best way? |
cc author @lauritowal, just checking in - my initial round of review is in the linked issue waiting for response, let me know if you have any questions! |
@praneethd7 I could not reach all other contributors, but the ones listed in the paper should are the main contributors anyway. |
@isdanni can you provide the exact link? I'm not sure where I can find it? EDIT: nevermind, found it |
Hi everyone @isdanni @praneethd7 @lauritowal ! How's the review going? Do you need anything from me? |
I did get some feedback from one reviewer and still need to incorporate some changes. |
yes, I have provided feedback in the linked thread and will update the checkbox once the changes are done |
👋 @lauritowal, @praneethd7, and @isdanni cc. @mooniean Could you please provide an update in this thread to your progress and an estimate of when you think you may be able to finish your reviews? Thanks! |
@crvernon I'll hope to have all the changes max by the end of August. |
Thank you @lauritowal |
Thanks @lauritowal for the update! I finished the initial round of the review at EleutherAI/ccs#8 and checked off the list items |
@editorialbot generate pdf |
@editorialbot generate pdf |
Let's try now: @editorialbot generate pdf |
@editorialbot generate pdf |
@praneethd7 hello! I can see you have some items on the checklist to be checked! how is the review coming along? |
Hi @mooniean |
Hello @mooniean. I am mostly done with the review. I have a few items that I will check off the list once @lauritowal and their team make changes. Thank you! |
Hi @lauritowal ! How is the process going? Anything I can help with? |
@mooniean Hi - I did not implement all the changes yet. |
Happy new year everyone!! @lauritowal how are the changes coming along? |
Happy New Year! I have set a deadline for myself to complete all the updates by January 20 at the latest. |
Submitting author: @lauritowal (Walter Laurito)
Repository: https://github.com/EleutherAI/ccs
Branch with paper.md (empty if default branch): joss-paper
Version: 0.1.1
Editor: @mooniean
Reviewers: @praneethd7, @isdanni
Archive: Pending
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
@praneethd7 & @isdanni, your review will be checklist based. Each of you will have a separate checklist that you should update when carrying out your review.
First of all you need to run this command in a separate comment to create the checklist:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @mooniean 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 ✨
Checklists
📝 Checklist for @isdanni
📝 Checklist for @praneethd7
The text was updated successfully, but these errors were encountered: