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

[PRE REVIEW]: DistriFS: A Platform and User Agnostic Approach to Dataset Distribution #6565

Closed
editorialbot opened this issue Apr 1, 2024 · 50 comments
Assignees
Labels
Go pre-review Track: 7 (CSISM) Computer science, Information Science, and Mathematics

Comments

@editorialbot
Copy link
Collaborator

editorialbot commented Apr 1, 2024

Submitting author: @JIBSIL (Julian Boesch)
Repository: https://github.com/JIBSIL/DistriFS
Branch with paper.md (empty if default branch):
Version: v0.0.1
Editor: @mahfuz05062
Reviewers: @aparoha, @suriya-ganesh
Managing EiC: Daniel S. Katz

Status

status

Status badge code:

HTML: <a href="https://joss.theoj.org/papers/af09fee13984aa8fc8dc2c5cf062756e"><img src="https://joss.theoj.org/papers/af09fee13984aa8fc8dc2c5cf062756e/status.svg"></a>
Markdown: [![status](https://joss.theoj.org/papers/af09fee13984aa8fc8dc2c5cf062756e/status.svg)](https://joss.theoj.org/papers/af09fee13984aa8fc8dc2c5cf062756e)

Author instructions

Thanks for submitting your paper to JOSS @JIBSIL. Currently, there isn't a JOSS editor assigned to your paper.

@JIBSIL if you have any suggestions for potential reviewers then please mention them here in this thread (without tagging them with an @). You can search the list of people that have already agreed to review and may be suitable for this submission.

Editor instructions

The JOSS submission bot @editorialbot is here to help you find and assign reviewers and start the main review. To find out what @editorialbot can do for you type:

@editorialbot commands
@editorialbot editorialbot added pre-review Track: 7 (CSISM) Computer science, Information Science, and Mathematics labels Apr 1, 2024
@editorialbot
Copy link
Collaborator Author

Hello human, 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:

@editorialbot commands

For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:

@editorialbot generate pdf

@editorialbot
Copy link
Collaborator Author

Checking the BibTeX entries failed with the following error:

No paper file path

@editorialbot
Copy link
Collaborator Author

Software report:

github.com/AlDanial/cloc v 1.90  T=0.02 s (1843.7 files/s, 135681.1 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Go                              28            449            120           1668
Markdown                         3             26              0             69
YAML                             1              1              4             18
-------------------------------------------------------------------------------
SUM:                            32            476            124           1755
-------------------------------------------------------------------------------

Commit count by author:

     5	JIBSIL

@editorialbot
Copy link
Collaborator Author

Paper file info:

⚠️ Failed to find a paper file in https://github.com/JIBSIL/DistriFS

@editorialbot editorialbot added the Go label Apr 1, 2024
@editorialbot
Copy link
Collaborator Author

License info:

✅ License found: MIT License (Valid open source OSI approved license)

@editorialbot
Copy link
Collaborator Author

⚠️ An error happened when generating the pdf. Paper file not found.

@danielskatz
Copy link

@xuanxu - I see PAPER.md in the repo. Do you know why editorialbot doesn't find it?

@danielskatz
Copy link

@JIBSIL - thanks for your submission. I notice that your paper doesn't seem to have any references, and is extremely short. Can you expand it a bit, including some related work perhaps? Please see https://joss.readthedocs.io/en/latest/submitting.html#how-should-my-paper-be-formatted and https://joss.readthedocs.io/en/latest/submitting.html#example-paper-and-bibliography

@JIBSIL
Copy link

JIBSIL commented Apr 1, 2024

@danielskatz - I've added a Current Work section, bibliography and some references from the original paper. Apologies for the lack of references in the original - this is my first time converting a paper into a shorter form. Thanks for checking it out!

@JIBSIL
Copy link

JIBSIL commented Apr 1, 2024

@editorialbot generate pdf

@editorialbot
Copy link
Collaborator Author

⚠️ An error happened when generating the pdf. Paper file not found.

@xuanxu
Copy link
Member

xuanxu commented Apr 1, 2024

editorilabot is case sensitive when looking for the paper source file, it must be named paper.md

@danielskatz
Copy link

Thanks @xuanxu!

@danielskatz
Copy link

@JIBSIL - please change the name of your paper to "paper.md" in lower case, and try to generate the PDF again

@JIBSIL
Copy link

JIBSIL commented Apr 1, 2024

@editorialbot generate pdf

@editorialbot
Copy link
Collaborator Author

👉📄 Download article proof 📄 View article proof on GitHub 📄 👈

@JIBSIL
Copy link

JIBSIL commented Apr 1, 2024

@danielskatz - Looks like it worked!

@danielskatz
Copy link

I don't see the references - Please look at the example paper again to see how this should work. I think you are missing the @s.

@danielskatz
Copy link

@editorialbot check references

@editorialbot
Copy link
Collaborator Author

Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1109/HICSS.2008.436 is OK
- 10.1177/0165551513480107 is OK
- 10.1093/ijlit/ean010 is OK
- 10.1145/269005.266694 is OK
- 10.1145/3488716 is OK

MISSING DOIs

- No DOI given, and none found for title: IPFS - Content addressed, versioned, P2P file syst...
- 10.1109/msr.2017.59 may be a valid DOI for title: Bug characteristics in blockchain systems: A large...

INVALID DOIs

- None

@danielskatz
Copy link

@JIBSIL - also note that one of your references perhaps should have DOI added, as suggested above.

@JIBSIL
Copy link

JIBSIL commented Apr 2, 2024

@editorialbot generate pdf

@editorialbot
Copy link
Collaborator Author

👉📄 Download article proof 📄 View article proof on GitHub 📄 👈

@JIBSIL
Copy link

JIBSIL commented Apr 2, 2024

@editorialbot check references

@editorialbot
Copy link
Collaborator Author

Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1109/HICSS.2008.436 is OK
- 10.1177/0165551513480107 is OK
- 10.1093/ijlit/ean010 is OK
- 10.1145/269005.266694 is OK
- 10.1145/3488716 is OK
- 10.1109/MSR.2017.59 is OK

MISSING DOIs

- No DOI given, and none found for title: IPFS - Content addressed, versioned, P2P file syst...

INVALID DOIs

- None

@JIBSIL
Copy link

JIBSIL commented Apr 2, 2024

@danielskatz - The issues with the references should have been fixed in the latest commit of paper.md

@danielskatz
Copy link

thanks @JIBSIL - I'll next look for an editor, who will then work on the review process

@danielskatz
Copy link

@editorialbot invite @mahfuz05062 as editor

@editorialbot
Copy link
Collaborator Author

Invitation to edit this submission sent!

@danielskatz
Copy link

👋 @mahfuz05062 - do you think you can edit this submission?

@mahfuz05062
Copy link

I am not great at the subject matter, but I do have the bandwidth to edit it.

@danielskatz
Copy link

Thanks! I'll assign it to you, and while the reviewers really should be the subject matter experts, I can try to help if needed, perhaps in finding them, or in any other way.

@danielskatz
Copy link

@editorialbot assign @mahfuz05062 as editor

@editorialbot
Copy link
Collaborator Author

Assigned! @mahfuz05062 is now the editor

@mahfuz05062
Copy link

Hi @JIBSIL , do you have any suggestions for potential reviewers? If so, please mention them here (without tagging them with an @). You can search the list of people that have already agreed to review and may be suitable for this submission.

@JIBSIL
Copy link

JIBSIL commented Apr 14, 2024

Hi @JIBSIL , do you have any suggestions for potential reviewers? If so, please mention them here (without tagging them with an @). You can search the list of people that have already agreed to review and may be suitable for this submission.

Sure! Here would be my suggestions (from the list, but I don't know any of them personally):
aparoha
xiaoyang-sde
achanda
varshaprasad96
suriya-ganesh
imw
joao-lima

Just generally from looking for "distributed systems/networking" and Go from that list. Hope this helps!

@mahfuz05062
Copy link

Hi @aparoha @xiaoyang-sde @achanda @varshaprasad96 @suriya-ganesh are any of you interested in reviewing this paper and have bandwidth?

@aparoha
Copy link

aparoha commented Apr 14, 2024 via email

@JIBSIL
Copy link

JIBSIL commented Apr 15, 2024

@aparoha - Sure! You can view it here (or on the repo here)

@aparoha
Copy link

aparoha commented Apr 15, 2024 via email

@JIBSIL
Copy link

JIBSIL commented Apr 15, 2024

@aparoha - not sure how it works on the reviewer side; perhaps @mahfuz05062 could help?

@mahfuz05062
Copy link

mahfuz05062 commented Apr 15, 2024

Hi @aparoha , all the reviews in JOSS are done through GitHub (so a little different from the traditional peer-review process). I will open up a new review thread and explain the review process once at least two reviewers have agreed to perform the review. This is the guideline for reference: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html.

@suriya-ganesh
Copy link

@mahfuz05062 if the review thread is open, I would love to review this as well.

@mahfuz05062
Copy link

@editorialbot add @aparoha as reviewer

@editorialbot
Copy link
Collaborator Author

@aparoha added to the reviewers list!

@mahfuz05062
Copy link

@editorialbot add @suriya-ganesh as reviewer

@editorialbot
Copy link
Collaborator Author

@suriya-ganesh added to the reviewers list!

@mahfuz05062
Copy link

@editorialbot start review

@editorialbot
Copy link
Collaborator Author

OK, I've started the review over in #6625.

@varshaprasad96
Copy link

varshaprasad96 commented Apr 15, 2024

@mahfuz05062 Sorry, missed the ping! Would be happy to review if some more eyes are helpful!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Go pre-review Track: 7 (CSISM) Computer science, Information Science, and Mathematics
Projects
None yet
Development

No branches or pull requests

8 participants