-
-
Notifications
You must be signed in to change notification settings - Fork 41
[PRE REVIEW]: MODULO: a python toolbox for data-driven modal decomposition #6208
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
Comments
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:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
|
Wordcount for |
|
Five most similar historical JOSS papers: PySPOD: A Python package for Spectral Proper Orthogonal Decomposition (SPOD) Moead-framework: a modular MOEA/D Python framework EMD: Empirical Mode Decomposition and Hilbert-Huang Spectral Analyses in Python WrightTools: a Python package for multidimensional spectroscopy TLViz: Visualising and analysing tensor decomposition models with Python |
Hello @mendezVKI, thanks for your submission to JOSS. We'll use this pre-review issue to assign an editor and find reviewers; unfortunately, we don't have any editors available to handle your submission right now, so I need to put this on our waitlist until someone can edit it. In the meantime, any reviewer recommendations you can make would be welcome. |
@faroit is this close to your areas of expertise? |
@kyleniemeyer i am unfortunately not, but I can help providing reviewers once we have an editor |
@editorialbot invite @philipcardiff as editor |
Invitation to edit this submission sent! |
@editorialbot assign me as editor |
Assigned! @philipcardiff is now the editor |
@faroit – can you help @philipcardiff with finding some reviewers here? |
Hello (again!) @mendezVKI, Would you mind clarifying the difference between this submission and the toolbox published at https://www.softxjournal.com/article/S2352-7110(20)30335-6/fulltext? This SoftwareX publication also refers to github.com/mendezVKI/MODULO. Thanks |
Hello again Philip! The toolbox published in https://www.softxjournal.com/article/S2352-7110(20)30335-6/fulltext was the first version of the toolbox in this submission. It was written in Matlab and equipped with a GUI and an executable. That version is no longer maintained and has been kept in one of the branches of the MODULO repository. The version we have submitted to JOSS has been entirely rewritten in Python, updated with modern packages from scipy/numpy and considerably extended to have many more decompositions (the SPODs, the DMD, the Kernel POD) and features (non-uniform grids, more sophisticated memory saving, etc.). The main novelties are listed on the doc page: https://modulo.readthedocs.io/en/latest/what_is_new.html . The reason why I kept the original repo and did not move to a new one is that I have promoted it in various published materials (most notably in this chapter: https://www.cambridge.org/core/books/abs/datadriven-fluid-mechanics/generalized-and-multiscale-modal-analysis/F6C342E80A83D68756CEC1492848C6E1 ) and I would like to preserve that. |
Thanks, @mendezVKI, for the details. To avoid any confusion, would it be possible to update the name in the paper to “MODULO v2” or “pyMODULO” or something similar? |
Hi @sichinaga, Would you be interested in reviewing the submission “MODULO: a python toolbox for data-driven modal decomposition ”, which we received at JOSS? The link for the submission is #6208, and the repository is at https://github.com/mendezVKI/MODULO. If you are unfamiliar with JOSS, the review process is unique: it takes place in a GitHub issue, is open, and encourages author-reviewer-editor conversations. You can have a look at our review criteria (https://joss.readthedocs.io/en/latest/review_criteria.html) and review checklist (https://joss.readthedocs.io/en/latest/review_checklist.html) to get an idea of what is expected. |
Hi @modelflows, Would you be interested in reviewing the submission “MODULO: a python toolbox for data-driven modal decomposition ”, which we received at JOSS? The link for the submission is #6208, and the repository is at https://github.com/mendezVKI/MODULO. If you are unfamiliar with JOSS, the review process is unique: it takes place in a GitHub issue, is open, and encourages author-reviewer-editor conversations. You can have a look at our review criteria (https://joss.readthedocs.io/en/latest/review_criteria.html) and review checklist (https://joss.readthedocs.io/en/latest/review_checklist.html) to get an idea of what is expected. |
Indeed, we could do that. We can change to pyMODULO. Our main constraint was keeping the same repository I constantly refer to in the book. I naively thought that having the different branches + readme + docs was enough, keeping in mind that (1) the Matlab version is no longer maintained, (2) there is only one module on PyPI (the Python version). Would it be okay to change the name but keep it in the same repository? (in the master) |
Yes, that sounds fine. |
Hello @mengaldo, would you be interested in reviewing this submission? |
@philipcardiff Yes, happy to review the submission |
@editorialbot add @mengaldo as reviewer |
@mengaldo added to the reviewers list! |
Hi @mtezzele, Would you be interested in reviewing the submission “MODULO: a python toolbox for data-driven modal decomposition ”, which we received at JOSS? The link for the submission is #6208, and the repository is at https://github.com/mendezVKI/MODULO. If you are unfamiliar with JOSS, the review process is unique: it takes place in a GitHub issue, is open, and encourages author-reviewer-editor conversations. You can have a look at our review criteria (https://joss.readthedocs.io/en/latest/review_criteria.html) and review checklist (https://joss.readthedocs.io/en/latest/review_checklist.html) to get an idea of what is expected. |
Hi @AndreWeiner, Would you be interested in reviewing the submission “MODULO: a python toolbox for data-driven modal decomposition ”, which we received at JOSS? The link for the submission is #6208, and the repository is at https://github.com/mendezVKI/MODULO. If you are unfamiliar with JOSS, the review process is unique: it takes place in a GitHub issue, is open, and encourages author-reviewer-editor conversations. You can have a look at our review criteria (https://joss.readthedocs.io/en/latest/review_criteria.html) and review checklist (https://joss.readthedocs.io/en/latest/review_checklist.html) to get an idea of what is expected. |
I'm sorry but I'm not available. Best, |
No problem, thanks, Marco, for your reply. Feel free to suggest a suitable reviewer. |
@philipcardiff happy to review the submission. |
Thanks, Andre. |
@editorialbot add @AndreWeiner as reviewer |
@AndreWeiner added to the reviewers list! |
@editorialbot start review |
OK, I've started the review over in #6753. |
Hi all, we can continue the review over on the review issue #6753. |
Submitting author: @mendezVKI (Miguel Alfonso Mendez)
Repository: https://github.com/mendezVKI/MODULO
Branch with paper.md (empty if default branch):
Version: v2.0
Editor: @philipcardiff
Reviewers: @mengaldo, @AndreWeiner
Managing EiC: Kyle Niemeyer
Status
Status badge code:
Author instructions
Thanks for submitting your paper to JOSS @mendezVKI. Currently, there isn't a JOSS editor assigned to your paper.
@mendezVKI 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:
The text was updated successfully, but these errors were encountered: