Skip to content
GitHub no longer supports this web browser. Learn more about the browsers we support.
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

ObsPy: Software Submission for Review #16

Open
megies opened this issue Dec 12, 2019 · 18 comments
Open

ObsPy: Software Submission for Review #16

megies opened this issue Dec 12, 2019 · 18 comments

Comments

@megies
Copy link

@megies megies commented Dec 12, 2019

Submitting Author: @megies
Package Name: ObsPy
One-Line Description of Package: A Python Toolbox for seismology/seismological observatories.
Repository Link: https://github.com/obspy/obspy
Version submitted: 1.1.1
Editor: @lwasser
Reviewer 1: @canyon289
Reviewer 2: @raoulcollenteur
Archive: TBD
Version accepted: TBD


Description

ObsPy is an open-source project dedicated to provide a Python framework for processing seismological data. It provides parsers for common file formats, clients to access data centers and seismological signal processing routines which allow the manipulation of seismological time series (see Beyreuther et al. 2010, Megies et al. 2011, Krischer et al. 2015).

The goal of the ObsPy project is to facilitate rapid application development for seismology.

ObsPy is licensed under the GNU Lesser General Public License (LGPL) v3.0.

Scope

  • Please indicate which category or categories this package falls under:
    • Data retrieval
    • Data extraction
    • Data munging
    • Data deposition
    • Reproducibility
    • Geospatial
    • Education
    • Data visualization* <-- didn't tick this, because the benefit of a 'pre-submission inquiry' didn't become obvious for me, the issue form seems to be largely the same

* Please fill out a pre-submission inquiry before submitting a data visualization package. For more info, see this section of our guidebook.

  • Explain how the and why the package falls under these categories (briefly, 1-2 sentences):

It falls under these categories, because we retrieve and munge data. Also loads of the functionality is concerning geospatial aspects (can't be avoided in seismology). Being as high-level as we are in functionality, I'd also consider it to be educational, since you can illustrate how to do many processing steps (and what pitfalls there are) in a few lines of code.
If this is more about the core aspects of the software, I guess you could also argue for only retrieval and munging, and the other are rather side aspects.. although it's unclear to me in which bullet item signal processing is included (munging I guess?).

  • Who is the target audience and what are scientific applications of this package?

Mainly scientists/researchers, but we know that obspy is also used in industry (no idea about numbers, though). Scientific applications span most fields of seismology that involve real or synthetic data, basically only excluding purely theoretical, pen and paper seismology.

  • Are there other Python packages that accomplish the same thing? If so, how does yours differ?

There are various packages with I/O of up to a handful file formats at a time, usually concerned only with 1-3 file formats used by the authors. I am not aware of other packages that offer access clients for all important protocols like we do (FDSN web services, SeedLink, Earthworm, Arclink, NRL, syngine, ...). Most important point would be the widespread use in projects built on top of obspy, most likely.

  • If you made a pre-submission enquiry, please paste the link to the corresponding issue, forum post, or other discussion, or @tag the editor you contacted:

no pre-submission enquiry filed.

Technical checks

For details about the pyOpenSci packaging requirements, see our packaging guide. Confirm each of the following by checking the box. This package:

  • does not violate the Terms of Service of any service it interacts with.
    • this feels a bit unclear, but in any case I'm not aware of any ToSs we would violate
  • has an OSI approved license
    • LGPLv3
  • contains a README with instructions for installing the development version.
    • these instructions are in the wiki which is linked in the README to keep it more concise
  • includes documentation with examples for all functions.
    • docs.obspy.org built from source with sphinx
  • contains a vignette with examples of its essential functions and uses.
    • submodule API landing pages (i.e. __init__.pys) hold the submodules' most important functions with explanations and usage examples,
  • has a test suite.
  • has continuous integration, such as Travis CI, AppVeyor, CircleCI, and/or others.
    • Travis for Linux + Mac tests
    • Appveyor for Windows tests
    • Circle for quick linting and Flake8

Publication options

We have citable papers out, so no need for this I think. If anything we might do a "history of obspy" short story?

JOSS Checks
  • The package has an obvious research application according to JOSS's definition in their submission requirements. Be aware that completing the pyOpenSci review process does not guarantee acceptance to JOSS. Be sure to read their submission requirements (linked above) if you are interested in submitting to JOSS.
  • The package is not a "minor utility" as defined by JOSS's submission requirements: "Minor ‘utility’ packages, including ‘thin’ API clients, are not acceptable." pyOpenSci welcomes these packages under "Data Retrieval", but JOSS has slightly different criteria.
  • The package contains a paper.md matching JOSS's requirements with a high-level description in the package root or in inst/.
  • The package is deposited in a long-term repository with the DOI:

Note: Do not submit your package separately to JOSS

Are you OK with Reviewers Submitting Issues and/or pull requests to your Repo Directly?

This option will allow reviewers to open smaller issues that can then be linked to PR's rather than submitting a more dense text based review. It will also allow you to demonstrate addressing the issue via PR links.

  • Yes I am OK with reviewers submitting requested changes as issues to my repo. Reviewers will then link to the issues in their submitted review.

Code of conduct

We're using Contributor Covenant Code of Conduct v1.4

P.S. Have feedback/comments about our review process? Leave a comment here

Editor and Review Templates

Editor and review templates can be found here

@megies

This comment has been minimized.

Copy link
Author

@megies megies commented Dec 12, 2019

Code of conduct was added after last stable release, it's in master and will be part of soon-to-be-released 1.2.0.

@lwasser

This comment has been minimized.

Copy link
Member

@lwasser lwasser commented Dec 12, 2019

awesome! thanks @megies for this submission. we will get back to you soon!! likely after agu :) in the meantime are there people who you could suggest as reviewers? i can also dig into our spreadsheets but if you know of folks, please feel free to suggest here. now that i'm writing this i'm realizing we might consider adding this question to our template as well.

Editor's Template

Editor checks:

  • Fit: The package meets criteria for fit and overlap.
  • Automated tests: Package has a testing suite and is tested via Travis-CI or another CI service.
  • License: The package has an OSI accepted license
  • Repository: The repository link resolves correctly
  • Archive (JOSS only, may be post-review): The repository DOI resolves correctly
  • Version (JOSS only, may be post-review): Does the release version given match the GitHub release (v1.0.0)?

Editor comments

I asked the submitter below if they were interested in JOSS!


Reviewers: Listed above with a due date of feb 24
Due date:

note: i think it might be good to add the editor checks somewhere in the issue ?? it seems redundant to list reviewers twice and a due date at the bottom.

@megies

This comment has been minimized.

Copy link
Author

@megies megies commented Dec 12, 2019

are there people who you could suggest as reviewers?

That'd be somebody working in seismonology, good in Python and not using obspy? Is that even possible?

👆 just kidding 😜 but yeah, most people I could come up with would be biased 😉

@lwasser

This comment has been minimized.

Copy link
Member

@lwasser lwasser commented Jan 15, 2020

hey there @megies !! i'm just catching up on things with the holidays, and beginning of the semester here... so my apologies for the delay. Our next meeting is this thursday so i'll be sure to get this review going.

I'll put a few calls out to try to find some reviewers. it seems to me that someone who uses obspy could be ok?! but if they are closely related to the development already that would not be ok. let's see if we can use a combo of twitter and our forum to find a few reviewers. more in a bit. i'm hoping to play catch up over the next week and this is a priority for me as it's been sitting for a month. thank you for your patience!!

@megies

This comment has been minimized.

Copy link
Author

@megies megies commented Jan 17, 2020

Like I said, it's all good, we're in no hurry about this at all. 👍

@canyon289

This comment has been minimized.

Copy link

@canyon289 canyon289 commented Jan 29, 2020

Hi, I saw the call for reviewers on twitter, all for open source science, would like to be helpful. I'm not in seismology and don't use ObsPy, but do help maintain ArviZ and PyMC3 which I hope qualifies me enough. Let me know if I meet the bar.

@raoulcollenteur

This comment has been minimized.

Copy link

@raoulcollenteur raoulcollenteur commented Jan 30, 2020

I could help with a review. Not specifically known in seismology but I am a hydrogeologist and (co-) develop some python packages. Let me know if help is needed :)

@lwasser

This comment has been minimized.

Copy link
Member

@lwasser lwasser commented Jan 30, 2020

hi @canyon289 and @raoulcollenteur if you have experience developing packages, and are on the geosciences end of things, I think that could work well for this review. @megies please let me know what you think about two people reviewing who are not from the seismology community explicitly? i suspect this might be challenging to find someone without a conflict of interest and such. I know both are technically qualified based upon the comments above!! Thank you all for your responses.

@megies

This comment has been minimized.

Copy link
Author

@megies megies commented Jan 31, 2020

@megies please let me know what you think about two people reviewing who are not from the seismology community explicitly?

I don't see a problem with that at all. 👍

@lwasser

This comment has been minimized.

Copy link
Member

@lwasser lwasser commented Jan 31, 2020

awesome @raoulcollenteur and @canyon289 if you are still willing, I am going to assign you as reviewers for obspy!! Please plan to have your feedback to @megies by Monday Feb 24th.

You can use our review guide as a basis for your review. https://www.pyopensci.org/dev_guide/peer_review/reviewer_guide.html We actually have some reviews that you can look out now (our guide is dated already!!) which include:

@megies did check they are ok with issues being submitted directly to the obspy repo! so if you'd like you can organize that review with issues providing commentary here and then linking to issues that you open in obspy. how you prefer to work is up to you!

If you have any questions please get in touch here!! Thank you again for volunteering your time to review this package! It is a very widely used package in this domain looking at the package download stats on conda so we are excited to see it submitted here! Have a wonderful weekend!!

@lwasser

This comment has been minimized.

Copy link
Member

@lwasser lwasser commented Jan 31, 2020

@megies please excse me if i already asked you this? are you interested in a JOSS citation? it's not a significant amount of extra work. you write the "paper" which can be quite short. earthpy has one in it's repo, and then joss automatically accepts the review that happens here so it's fast once the paper is complete. You then get a more formal citation through a journal which is really nice and linked to your orcid ID (and any other contributors that youd like to add). this is an option so just let me know if you have any interest in it! i'm excited to get this review going!

@lwasser

This comment has been minimized.

Copy link
Member

@lwasser lwasser commented Jan 31, 2020

one other note -- @raoulcollenteur and @canyon289 the review template is here in case you didn't see it above! https://www.pyopensci.org/dev_guide/appendices/templates.html

@canyon289

This comment has been minimized.

Copy link

@canyon289 canyon289 commented Feb 1, 2020

Great. I'll get this done by the 24th. Also interested in whether this is getting submitted to JOSS as there seem to be extra reviewer items for me to go through if that's the case.

@megies

This comment has been minimized.

Copy link
Author

@megies megies commented Feb 1, 2020

please excse me if i already asked you this? are you interested in a JOSS citation? it's not a significant amount of extra work

No problem. I'm not sure if it would do much though, since we have three citable papers for obspy but if you have a good idea how to theme it or if you think it's still adding something, let me know. :-)

@megies

This comment has been minimized.

Copy link
Author

@megies megies commented Feb 1, 2020

Also interested in whether this is getting submitted to JOSS as there seem to be extra reviewer items for me to go through if that's the case

You can start on those items not affected by JOSS submission :-)

@lwasser

This comment has been minimized.

Copy link
Member

@lwasser lwasser commented Feb 3, 2020

awesome. thanks all!! sure. i'll actually reach out to @arfon on this. Arfon if you happen to see this - does JOSS provide anything new in terms of citation and visibility if a tool has already been published in a few other journals?

@arfon

This comment has been minimized.

Copy link

@arfon arfon commented Feb 3, 2020

I'm not sure if it would do much though, since we have three citable papers for obspy but if you have a good idea how to theme it or if you think it's still adding something, let me know.

👋 If you already have a paper (or multiple papers!) that you like people to cite for this software then I don't think it makes much sense to publish a paper with JOSS.

@lwasser

This comment has been minimized.

Copy link
Member

@lwasser lwasser commented Feb 4, 2020

Thank you @arfon !! that was fast. ok great.. i just emailed you about this given i have a similar question for another review underway!

@lwasser lwasser mentioned this issue Feb 4, 2020
15 of 22 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.