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]: Enlsip.jl: A Julia optimization package to solve constrained nonlinear least-squares problems #6226
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:
|
|
Wordcount for |
|
Review checklist for @odunbarConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Review checklist for @tmigotConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@pierre-borie - Our reviewers have created issues in the software directory, have you seen them? Could you please update your status? Please consider changing/adding the suggestions pointed out by our reviewers. Thank you in advance. |
@jbytecode - I thought that the checklist completions were still on progress and did not see the comments associated with it. My apologies for not having responded till today, I am aware of the time and effort put by you. and the reviewers into this. |
I am now removing the |
@pierre-borie - Could you please update your status and inform us how is your set of corrections/additions going? Thank you in advance. |
@jbytecode So far I pushed corrections to the documentation. I also added formal corrections to the paper but I want to include an industrial application of our package before submitting another draft. I am currently working on this and still need to fix what we are allowed to show or not wiht the industrial partner in question. |
@pierre-borie - How is your work going? Could you please update your status and inform us? |
@jbytecode For the past weeks I have encountered difficulties in the process of having access to an industrial application that can be described in the paper. Thoses issues have been fixed so I should be able to submit a new draft next week. Concerning the package, I added and corrected some functionalities following advices from the reviewers. Documentation has been modified accordingly to thoses changes. |
@jbytecode - It took a few more days than I thought it would but I pushed a revised version of the paper on the joss-paper branch. |
@editorialbot generate pdf |
Update on my end, I had another sweep through my checklist: There are a few typos and minor comments outstanding etc. but I do not expect these to take a long time. |
@tmigot - May I kindly ask you to update your review status? Thank you in advance. |
@jbytecode @pierre-borie sorry for the delay, I will update my status in the upcoming days. |
Hi @jbytecode, Thanks! |
@odunbar - You have got still unchecked items in your task list. Are they okay? Could you please finish checking them? |
@odunbar - sorry for pinging. It's my fault. You don't need to take any further actions. |
@jbytecode I pushed another revised version of the paper, correcting (hopefully) the lastest issues. @odunbar Thanks for supporting the submission! The new version does not differ much from the previous one
|
@editorialbot generate pdf |
@pierre-borie - Could you please update your status? |
@jbytecode - I created a pull request covering the last issues about the package. Edit: I merged the above mentionned corrections and released a new version of the package. Also corrected typo in the paper. |
Hi @jbytecode With the last PR now merged, @pierre-borie answered all my comments and I completed the check-list. The nonlinear least squares problem is both important and very frequent in practice, so I think this algorithm will be an interesting addition to the Julia background. My only comment would be that modeling a problem for the solver is not an easy task. I would recommend relying on NLPModels.jl to interact with the model as it would open the door to several modeling and benchmarking tools, but that's a biased opinion and independent of this submission process. Congrats @pierre-borie for the nice work! |
@editorialbot check references |
|
@editorialbot generate pdf |
DOI for JSOSolvers: https://github.com/JuliaSmoothOptimizers/JSOSolvers.jl/blob/main/CITATION.cff |
@pierre-borie - could you please fix the missing DOI issue and ping me again, thank you in advance! |
@jbytecode - I added the DOI for JSOSolvers. For the other two papers, I could not find any. I also checked in papers/books citing them and they are referenced without DOI so I believe they don't have one (perhaps because they are technical reports). |
@tmigot - Thanks for approving the corrections and for your advices! The current modeling interface isn't the best one indeed and switching NLPModels.jl is definitely something I intend to work on. |
@editorialbot check references |
|
@editorialbot generate pdf |
Submitting author: @pierre-borie (Pierre Borie)
Repository: https://github.com/UncertainLab/Enlsip.jl.git
Branch with paper.md (empty if default branch): joss-paper
Version: v0.9.1
Editor: @jbytecode
Reviewers: @tmigot, @odunbar
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
@tmigot & @odunbar, 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 @jbytecode 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 @odunbar
📝 Checklist for @tmigot
The text was updated successfully, but these errors were encountered: