-
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
GREOPy: A General Relativistic Emitter-Observer problem Python algorithm #227
Comments
Editor in Chief checksHi there! Thank you for submitting your package for pyOpenSci Please check our Python packaging guide for more information on the elements
Editor commentsThanks for bearing with us as we hibernated over the holidays and assigned out a new EiC. Regarding the package, I'm happy to work on assigning an editor. However, I do want to bring to your attention a few concerns. Specifically, the "How to find initial conditions" page linked on the documentation website is blank. https://greopy.readthedocs.io/en/latest/about/initial_conditions.html Also, the tutorial gallery only one tutorial is present ("Quick Start"). Do you envision additional tutorials being added during the review process? https://greopy.readthedocs.io/en/latest/auto_tutorials/index.html Lastly, the Quickstart example on the reduced grid also took about 15 minutes of compute time on Colab. Would it be possible to include a note on the documentation that computations may be long running? |
Hello! I will begin working on the comments in the next couple of days and update everything accordingly, so thanks again! |
@irideselby Okay, let's hold off on sending it out for the review until the bug regarding light ray calculation is fixed as that may impact what SME reviewers of the software respond with. Sound okay? |
@coatless Thank you for waiting, I fixed the bug and bumped the version accordingly. I also edited the initial message in this thread to reflect this change, so now the submitted version is 0.2.1 instead of 0.2.0, I hope that's alright. |
@irideselby thanks for letting me know. I'll start the process of assigning an editor. |
I have now added the paper.md for JOSS in a paper/ directory and added a tick for the corresponding bullet point in the initial submission message. |
Submitting Author: Jan P. Hackstein (@irideselby)
All current maintainers: (@irideselby)
Package Name: GREOPy
One-Line Description of Package: Calculate relativistic light rays sent by an emitter to a receiver in the presence of a gravitational field.
Repository Link: https://codeberg.org/JPHackstein/GREOPy
Version submitted: v0.2.1
EiC: @coatless
Editor: TBD
Reviewer 1: TBD
Reviewer 2: TBD
Archive: https://zenodo.org/records/14537866
JOSS DOI: TBD
Version accepted: TBD
Date accepted (month/day/year): TBD
Code of Conduct & Commitment to Maintain Package
Description
GREOPy is a Python library for calculating relativistic light rays sent by an emitter to a receiver in the presence of a gravitational field. Finding a light ray connecting two events is sometimes called "Emitter-Observer" problem and is always present when it comes to communication between two observers, e.g. two satellites in orbit. GREOPy allows the emitter and receiver to move along arbitrary curves, making this an initial-value problem to solve from the emitter's perspective, and the gravitational field can be described by a rotating, non-accelerating central mass. Everything is being calculated in the general relativistic framework to include relativistic effects like light bending and the relativistic Doppler effect to be able to quantify their impact on error propagation. While only two spacetimes are implemented at the moment (even though further additions are planned), GREOPy is written in a way to allow the community to easily expand the number of spacetimes to suit their needs.
Scope
Please indicate which category or categories.
Check out our package scope page to learn more about our
scope. (If you are unsure of which category you fit, we suggest you make a pre-submission inquiry):
Domain Specific
Community Partnerships
If your package is associated with an
existing community please check below:
Data processing/munging: GREOPy uses parametrised curves, e.g. orbit data, to simulate communication via relativistic light rays between them. This allows analysis of relativistic effects on light and by extension any corresponding observable in some chosen spacetime, giving insights into fundamental properties of the underlying spacetime.
This package is mainly targeted for scientists working in geodesy; it can be used to simulate satellite-satellite or satellite-ground station communication and from this derive, e.g. how the Earth mass distribution changes over time due to for example climate change.
Not the same thing. There exist of course Python packages that implement General Relativity, e.g. to be able to calculate light rays (lightlike/nulllike geodesics) as one can do with EinsteinPy for example. However there appear to be no packages that implement specifically the Emitter-Observer problem (initial-value problem with a variable target boundary) in terms of General Relativity.
@tag
the editor you contacted:No presubmission inquiry was made
Technical checks
For details about the pyOpenSci packaging requirements, see our packaging guide. Confirm each of the following by checking the box. This package:
Publication Options
JOSS Checks
paper.md
matching JOSS's requirements with a high-level description in the package root or ininst/
.Note: JOSS accepts our review as theirs. You will NOT need to go through another full review. JOSS will only review your paper.md file. Be sure to link to this pyOpenSci issue when a JOSS issue is opened for your package. Also be sure to tell the JOSS editor that this is a pyOpenSci reviewed package once you reach this step.
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.
Confirm each of the following by checking the box.
Please fill out our survey
submission and improve our peer review process. We will also ask our reviewers
and editors to fill this out.
P.S. Have feedback/comments about our review process? Leave a comment here
Editor and Review Templates
The editor template can be found here.
The review template can be found here.
Footnotes
Please fill out a pre-submission inquiry before submitting a data visualization package. ↩
The text was updated successfully, but these errors were encountered: