-
-
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]: VineCopulas: an open-source Python package for vine copula modelling #6728
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: |
|
Dear reviewers, you can start your review by creating your tasklist with the following command:
In that list, there are several tasks. Whenever you perform a task, you can check on the corresponding checkbox. You can also reference the JOSS reviewer guidelines which is linked in first comment in this thread. Since the review process of JOSS is interactive, you can always interact with the author, the other reviewers, and the editor during the process. You can open issues and pull requests in the target repo. Please mention the url of this page in there so that we can keep tracking what is going on. Thank you in advance. |
Review checklist for @haozcbnuConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@couasnonanais let me know if you are able to get started on this soon. Thanks! |
Review checklist for @couasnonanaisConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hello @couasnonanais, here are the things you can ask me to do:
|
Here are several comments/suggestions. Lines 9-10: Here some references can be given to introduce the basic concept of Copula. Lines 15-16: please check the punctuation mark. Lines 19-22: Here the concept of the "curse of dimensionality " can be introduced to demonstrate the challenge in multivariate dependence modeling. Captions of Figure 1: D) What do you mean by " As not every vine copula structure is suitable to generate conditional samples of every variable"? E) Drawing conditional samples of the variable 1 given specific values of variables 2 and 3? This needs to be clear. Lines 59-61: To emphasize the usefulness of this package, the authors can highlight the large number of variables or hazards (>2) in the multi-hazard phenomenon. |
Hello @couasnonanais do you have an update on your review? |
@haozcbnu can you clarify the status regarding these comments? Any updates there? |
@haozcbnu Thank you so much for providing us with your review. @ymzayek I would prefer to incorporate the suggestions after having received @couasnonanais review as well, if possible. |
Hi!
I will send my review on Friday.
Thanks,
Anaïs
Le mar. 28 mai 2024 à 20:44, Yasmin ***@***.***> a écrit :
… Hello @couasnonanais <https://github.com/couasnonanais> do you have an
update on your review?
—
Reply to this email directly, view it on GitHub
<#6728 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AG2TOANYL3BUKA5X2LGSTZLZETGBVAVCNFSM6AAAAABHKVOUQ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZVHA4TMMJWHE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@couasnonanais let me know if you will have time to finish your review soon. If for whatever reason it is not possible for you that is ok but then it might be necessary to look for a new reviewer. Let me know if I can be of any help. Thanks! |
Hi Yasmin,
Apologies for the delay. I have started the review and will upload it
tomorrow.
Anaïs
…On Tue, 11 Jun 2024, 13:57 Yasmin, ***@***.***> wrote:
@couasnonanais <https://github.com/couasnonanais> let me know if you will
have time to finish your review soon. If for whatever reason it is not
possible for you that is ok but then it might be necessary to look for a
new reviewer. Let me know if I can be of any help. Thanks!
—
Reply to this email directly, view it on GitHub
<#6728 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AG2TOAMTDHWY5MLEQQFOMFDZG3Q3ZAVCNFSM6AAAAABHKVOUQ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRQGU3TONZQGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Submitting author: @judithclaassen (Judith Claassen)
Repository: https://github.com/VU-IVM/VineCopulas
Branch with paper.md (empty if default branch):
Version: v0.2.5
Editor: @ymzayek
Reviewers: @haozcbnu, @couasnonanais
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
@haozcbnu & @couasnonanais, 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 @ymzayek 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 @haozcbnu
📝 Checklist for @couasnonanais
The text was updated successfully, but these errors were encountered: