Skip to content
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

Code style #50

Open
AlecThomson opened this issue Apr 22, 2024 · 2 comments · May be fixed by #51
Open

Code style #50

AlecThomson opened this issue Apr 22, 2024 · 2 comments · May be fixed by #51

Comments

@AlecThomson
Copy link
Contributor

Hi @maaijke,

A relatively unimportant issue, but just food for thought. Since this codebase has been around for a while (i.e. during the Python 2->3 era) and has had many contributors over the years, there are a mix of codestyles in place.

IMO, this has hurt some of the code readability and consistency. I'd like to recommend the implementation of a standard formatter / style. My recommendation would be enforcing a tool like ruff. On top of that, a pre-commit hook + CI could be added to enforce / fix codestyle for developers/contributers.

Let me know what you think, and I'm happy to help get these sorts of things implemented.

@maaijke
Copy link
Collaborator

maaijke commented Apr 22, 2024

Hi Alec, I completely agree. I have been thinking about fully rewriting the code for some time now, especially since there are many more advanced updates and it seems many groups are using the code (which originally started as a dump of some of my private scripts on github..). Your help would be very much appreciated, maybe we can continue this discussion in a more direct way. You can contact me via my astron mail address

@twillis449
Copy link
Collaborator

Glad to see that RMextract is being 'modernized' :) So you might want to replace PyEphem interface with SkyField as PyEphem is no longer being supported. Although super-accurate positions are not all that critical for ionosphere modeling. Also I assume that RMextract will now be distinguishing between the pre and post GPS week 2238 (27 Non 2022) names of ionex files.

@AlecThomson AlecThomson linked a pull request May 6, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants