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]: Fitspy: A Python package for spectral decomposition #5868
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 |
@maurov, @FCMeng, thanks again for reviewing! Please go ahead and create your checklist first using the command |
@phibeck thanks for the reminder. Sorry, I have been very busy these days. I hope to start the review by next week. |
Hello @phibeck, |
@phibeck thanks for the reminder. I apologize for taking so long in reviewing this. From my side, there is nothing special holding me up, simply the fact that I am very busy with my official work. I will try doing this task as soon as possible. |
Review checklist for @maurovConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
I have started reviewing A graphical interface on top of lmfit may be of interest for those users who want to build a quick fitting model and apply it on multiple similar spectra, without prior knowledge of Python programming. I support this idea, but this software should be put in the right context and clearly stated in the paper/documentation. In fact, in addition to what here is called "spectral decomposition", "peak fitting" or "curve fitting" is performed in any field of science. This means that there already many (open source) softwares available and almost every researcher/community has a preferred tool. Clarifying the specific problems In conclusion, the current "Statement of need" is too generic and unclear. I am asking myself if there is "substantial scholarly effort" behind this project. Is this a "yet another peak fitting tool"? |
@phibeck Apologize for the late response. I have been really stacked this semester. I will start the review process as soon as possible. |
Review checklist for @FCMengConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hi, |
@patquem I apologize for not being very reactive on this review, but I am very busy these days. I hope to move forward in the next two weeks. As said in my previous comment, you should clearly give in the "Statement of need" which scientific community this code is targeting and give a real application example. I understand that you want to provide a generic and efficient peak fitting tool, but it is impossible to get all features for all users communities, so the point here is not to demonstrate which software is the best, but just to provide a given tool to a given community. For example, when you refer to 2D maps, are you referring to X-ray fluorescence maps? For such application, one very good open source software that is not in your table is PyMca. |
Thanks @maurov for getting the review started! Have your issues in CEA-MetroCarac/fitspy#9 been addressed? It sounds like there are some clear steps for @patquem to improve the manuscript, too. @FCMeng could you provide an update on the progress of your review? Thank you! |
Hello @maurov, |
@maurov @FCMeng and @phibeck : |
Statement of need (revisited) Numerous open-source tools for spectral fitting exist. However, most of them have been designed for specific application domains, offering a broad range of services beyond mere spectral fitting. Consequently, these tools can prove challenging to use, especially for less experienced individuals. In the vein of generic tools like Fityk or PRISMA, Fitspy is a dedicated tool for spectral fitting—and only spectral fitting—with the following characteristics or functionalities: Agnostic Nature: Fitspy is not tied to any specific physical quantity or database. It is designed to process spectra regardless of their x-support and y-intensity without requiring prior knowledge. Python Implementation: Fitspy is coded in Python. As a result, spectra can be easily processed using Python scripts, catering to individuals with basic knowledge of the language. 2D Maps: Fitspy has been designed to handle spectra derived from 2D acquisitions. Note that beyond "2D", dimensions can encompass time or any other dimension. When dealing with 2D data, an interactive map in the Fitspy GUI allows users to locate and select spectra of interest easily. Multiprocessing Capabilities: Fitspy enables spectral fit processing on multiple processors, enhancing efficiency. Constrained Parameters: Leveraging the Lmfit library, Fitspy empowers users to impose constraints on parameter ranges or establish constraints between parameters using literal expressions. Simple GUI: Fitspy has been designed to be as intuitive and simple to use as possible (subjective criterion). To the author's knowledge, although many open-source software applications are more advanced in certain aspects mentioned, none of them appears to encompass all the functionalities described above. |
@editorialbot set 2024.4 as version |
Done! version is now 2024.4 |
@patquem test mention (checking notifications) |
@phibeck (I noticed that I have not been receiving any notifications since your last two messages from this 'JOSS' page, even though my notification settings are correctly set up (?).) |
@editorialbot generate pdf |
@editorialbot check references |
|
@editorialbot recommend-accept |
|
|
👋 @openjournals/bcm-eics, this paper is ready to be accepted and published. Check final proof 👉📄 Download article If the paper PDF and the deposit XML files look good in openjournals/joss-papers#5203, then you can now move forward with accepting the submission by compiling again with the command |
@patquem as AEiC for JOSS I will now help to process this submission for acceptance in JOSS. I have checked this review, your repository, the archive link, and the paper. Most seems in order, however the below are some points that require your attention:
|
@patquem 👋 |
Hi @Kevin-Mattheus-Moerman, |
@editorialbot generate pdf |
@patquem thanks for making those changes. All looks good now to proceed. |
@editorialbot accept |
|
Ensure proper citation by uploading a plain text CITATION.cff file to the default branch of your repository. If using GitHub, a Cite this repository menu will appear in the About section, containing both APA and BibTeX formats. When exported to Zotero using a browser plugin, Zotero will automatically create an entry using the information contained in the .cff file. You can copy the contents for your CITATION.cff file here: CITATION.cff
If the repository is not hosted on GitHub, a .cff file can still be uploaded to set your preferred citation. Users will be able to manually copy and paste the citation. |
🐘🐘🐘 👉 Toot for this paper 👈 🐘🐘🐘 |
🚨🚨🚨 THIS IS NOT A DRILL, YOU HAVE JUST ACCEPTED A PAPER INTO JOSS! 🚨🚨🚨 Here's what you must now do:
Any issues? Notify your editorial technical team... |
🎉🎉🎉 Congratulations on your paper acceptance! 🎉🎉🎉 If you would like to include a link to your paper from your README use the following code snippets:
This is how it will look in your documentation: We need your help! The Journal of Open Source Software is a community-run journal and relies upon volunteer effort. If you'd like to support us please consider doing either one (or both) of the the following:
|
Thank you all (@maurov , @FCMeng , @phibeck and @Kevin-Mattheus-Moerman ) for reviewing, editing, and publishing the article. |
Submitting author: @patquem (Patrick Quéméré)
Repository: https://github.com/CEA-MetroCarac/fitspy
Branch with paper.md (empty if default branch):
Version: 2024.4
Editor: @phibeck
Reviewers: @maurov, @FCMeng
Archive: 10.5281/zenodo.10812332
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
@maurov & @FCMeng, 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 @phibeck 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 @maurov
📝 Checklist for @FCMeng
The text was updated successfully, but these errors were encountered: