Closed whedon closed 1 year ago
@sepandhaghighi moving forward this project will be maintained by Daniel Cahill. He decided to create this own fork. The important bit is that a) the info on the repo is still valid b) the project is considered feature-complete with the 3.0 release and c) the software will always be available as
elephas
package. Thanks for this question, though!
@maxpumperla I think it would be better if the repo url points to the new repo, since new updated will be pushed there. What do you think?
@editorialbot set mp_joss_paper as branch
Done! branch is now mp_joss_paper
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@maxpumperla It seems that
danielenricocahall 101 1855 2424 28.70
contributed to the code as well and is now the new maintainer. Would it make sense to add him as a second author?
yes, thanks @diehlpk. In fact, I've invited @danielenricocahall a couple of times already to add his name to honour his contributions!
@danielenricocahall now would be a good time 👍
Hello @maxpumperla , sorry for the delay! Just added now :) Did I add myself correctly? https://github.com/maxpumperla/elephas/blob/mp_joss_paper/paper.md
@editorialbot generate pdf
:warning: An error happened when generating the pdf.
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@sepandhaghighi moving forward this project will be maintained by Daniel Cahill. He decided to create this own fork. The important bit is that a) the info on the repo is still valid b) the project is considered feature-complete with the 3.0 release and c) the software will always be available as elephas package. Thanks for this question, though!
@maxpumperla I think it would be better if the repo url points to the new repo, since new updated will be pushed there. What do you think?
@danielenricocahall and @maxpumperla any comments?
@sepandhaghighi I think you can proceed with the review.
@sepandhaghighi moving forward this project will be maintained by Daniel Cahall. He decided to create this own fork. The important bit is that a) the info on the repo is still valid b) the project is considered feature-complete with the 3.0 release and c) the software will always be available as elephas package. Thanks for this question, though!
@maxpumperla I think it would be better if the repo url points to the new repo, since new updated will be pushed there. What do you think?
@danielenricocahall and @maxpumperla any comments?
I agree.
@danielenricocahall Could you please recommend some reviewers?
@diehlpk @maxpumperla @danielenricocahall
Result:
A well-known and interesting tool from a prolific team 💯 But there are some concerns about documentation and community guidelines:
Documentation
Community Guidelines
Acknowledgments
sectionBest Regards SH
@maxpumperla @danielenricocahall Please have a look at these comments. All of them are important to finish the review.
Hi @urbanophile do you have time to review that paper?
Hi @nmoran do you have time to review that paper?
@diehlpk happy to address the above comments, but given that we've submitted this ~10 months ago and not much happened in the meantime, I'd prefer to get a preliminary "go" from you first and only then put in more work.
@diehlpk happy to address the above comments, but given that we've submitted this ~10 months ago and not much happened in the meantime, I'd prefer to get a preliminary "go" from you first and only then put in more work.
I think you should address the comments. The software and paper is good and I think it deserves to be published. However, making the software user-friendly will take some effort.
@diehlpk sure, like I said, we definitely intend to! After maintaining this for around 6 years now, I just lean towards being a little conservative when it comes to requests.
@danielenricocahall how does your November look like? Maybe we can discuss in private how to tackle these comments? I think this all very reasonable and should help the project a lot.
@maxpumperla Can you provide some timeline for the changes? So I can pause the review.
Hi @diehlpk , apologies for delay, but was away for a bit. I can review in the coming week if not too late?
Hi @diehlpk , apologies for delay, but was away for a bit. I can review in the coming week if not too late?
Thank you.
@editorialbot add @nmoran as reviewer
@nmoran added to the reviewers list!
@diehlpk sorry, currently down with Covid. Apologies for the delay on my part, I think I might get to this ~ mid December.
@diehlpk sorry, currently down with Covid. Apologies for the delay on my part, I think I might get to this ~ mid December.
Thanks for the heads-up. Sorry to hear and hope you are doing better soon.
@editorialbot generate pdf
:warning: An error happened when generating the pdf.
@danielenricocahall it looks like the mp_joss_paper
branch doesn't exist any longer? You can update the branch @editorialbot uses with the command:
@editorialbot set xxx as branch
Ah! I think because we're now pointing to my fork as the main repo. I can move the branch over there then retry the job later today. Apologies for that!
I am running into some problems getting the software running while following the install instructions provided. I have opened the following two issues to track these.
@danielenricocahall @maxpumperla please have a look when ever you have time.
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
I believe I addressed the issues in the latest release (3.4.7), which required pinning the Tensorflow version to <=2.10. Also, PyPI now has the Github documentation: https://pypi.org/project/elephas/3.4.7/.
The paper and software look good to me and is of use to many users as evidenced by the large number of monthly downloads 💯
I am happy to recommend this be published in JOSS.
In addition I think it could be useful to mention the versions of python that elephas was developed and tested with to save users time in case there are unforeseen incompatibilities with older or newer versions.
@diehlpk LGTM 🚀
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1145/3357223.3362707 is OK
MISSING DOIs
- None
INVALID DOIs
- None
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@maxpumperla or @danielenricocahall please add the city and country to the affiliations, and we can proceed with preparing the publication.
affiliations:
- name: IU Internationale Hochschule, Erfurt, Germany
index: 1
- name: Anyscale Inc, San Francisco, USA
index: 2
@danielenricocahall I don't know which branch and repo we settled on for the PDF by now, but could you please update affiliation info for me like this when you update yours? thank you.
also, many thanks @diehlpk for finally moving this towards the finish line!!!
@editorialbot generate pdf
Submitting author: !--author-handle-->@maxpumperla<!--end-author-handle-- (Max Pumperla) Repository: https://github.com/danielenricocahall/elephas Branch with paper.md (empty if default branch): mp_joss_paper Version: 3.4.7 Editor: !--editor-->@diehlpk<!--end-editor-- Reviewers: @sepandhaghighi, @nmoran Archive: 10.5281/zenodo.7435012
:warning: JOSS reduced service mode :warning:
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
@marksantcroos & @burch-cm, 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 @taless474 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 ✨