From 4375c91c1cca2702ffd241f54f9e66fa8d3d1365 Mon Sep 17 00:00:00 2001 From: syedhamidali Date: Wed, 1 Apr 2026 14:07:39 +0530 Subject: [PATCH 1/5] MNT: Improve Contributing Guide --- CONTRIBUTING.md | 64 ++++++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 63 insertions(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index b39034a5..da5bc26c 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -143,8 +143,70 @@ Ready to contribute? Here's how to set up `xradar` for local development. - Verify all checks are passing - Mark your pull request as ready for review +## Roles and Responsibilities + +We want contribution to xradar to have a clear path from first contribution to +shared project stewardship. The project aims to keep responsibility distributed, +make participation transparent, and create an approachable way for contributors +to become more involved over time. + +### Contributors + +Contributors improve xradar through code, documentation, bug reports, testing, +examples, and discussion. This is the starting point for everyone. + +Typical ways contributors help include: + +* opening focused issues with reproducible examples +* submitting pull requests +* improving documentation, examples, or tests +* helping with design discussions and user support + +### Team Members + +Team members are contributors who want to become more involved in the day-to-day +work of the project. In addition to regular contributions, they may help with: + +* issue triage and labeling +* reproducing bug reports +* answering questions and guiding new contributors +* identifying pull requests that need review or follow-up + +This role is a good way to get familiar with the project workflow and where help +is most useful. + +### Maintainers + +Maintainers are active, trusted contributors with broader responsibility for the +health of the project. Maintainers are expected to: + +* review pull requests constructively and in a timely manner +* help guide technical discussions and decisions +* merge contributions when appropriate +* support contributors and encourage sustainable project practices +* help maintain release and documentation quality + +Maintainer status reflects sustained engagement and judgment, not just the +number of contributions. + +## Pathway to Greater Involvement + +There is no strict checklist for moving between roles, but the general pathway is: + +1. Start as a contributor by opening issues or pull requests. +2. Become a regular contributor by showing sustained involvement, reliability, + or expertise in parts of the project. +3. Take on more community-facing work such as triaging issues, reviewing pull + requests, or helping guide discussions. +4. Be invited to join the maintainer group based on continued engagement and + project trust. + +If you contribute regularly and would like to get more involved, reach out in an +issue, pull request, or discussion. We want the process to be welcoming and +transparent. + ## Deploying -A reminder for the maintainers on how to deploy. +A reminder for maintainers on how to deploy. Make sure all your changes are committed (including an entry in [history.md](https://github.com/openradar/xradar/blob/main/docs/history.md)). Then just create a release in the GitHub workflow. GHA will then deploy to PyPI if tests pass. From e5dc94d2b2d1d1b4eced50298803df83eb0d79ca Mon Sep 17 00:00:00 2001 From: syedhamidali Date: Wed, 1 Apr 2026 14:13:07 +0530 Subject: [PATCH 2/5] MNT: Improve Contributing Guide --- docs/history.md | 1 + 1 file changed, 1 insertion(+) diff --git a/docs/history.md b/docs/history.md index 1b46fe51..f88fb906 100644 --- a/docs/history.md +++ b/docs/history.md @@ -3,6 +3,7 @@ ## Development +* MNT: Clarify contributor, team-member, and maintainer roles in the contributing guide, including the pathway to greater project involvement ({issue}`341`) by [@syedhamidali](https://github.com/syedhamidali) * ENH: Move station coordinates (``latitude``, ``longitude``, ``altitude``) to root node as coordinates for DataTree coordinate inheritance ({issue}`331`, {pull}`333`) by [@aladinor](https://github.com/aladinor) * ENH: Add ``optional_groups`` parameter (default ``False``) to all ``open_*_datatree()`` functions to control inclusion of ``/radar_parameters``, ``/georeferencing_correction``, and ``/radar_calibration`` subgroups ({issue}`331`, {pull}`333`) by [@aladinor](https://github.com/aladinor) * ENH: Skip redundant station coordinate reads for subsequent sweeps in DataTree by passing ``site_as_coords=False`` and extracting shared ``_apply_site_as_coords`` helper ({issue}`334`, {pull}`337`) by [@aladinor](https://github.com/aladinor) From 878864c0bce4bdb15c39fdf3d1968f2e2822668b Mon Sep 17 00:00:00 2001 From: syedhamidali Date: Wed, 1 Apr 2026 14:22:47 +0530 Subject: [PATCH 3/5] MNT: Resolve history merge entry --- docs/history.md | 1 + 1 file changed, 1 insertion(+) diff --git a/docs/history.md b/docs/history.md index f88fb906..19d7e7a8 100644 --- a/docs/history.md +++ b/docs/history.md @@ -4,6 +4,7 @@ ## Development * MNT: Clarify contributor, team-member, and maintainer roles in the contributing guide, including the pathway to greater project involvement ({issue}`341`) by [@syedhamidali](https://github.com/syedhamidali) +* ADD: ``open_cfradial2_datatree`` reader with grouped CfRadial2 compatibility normalization for common FM301/CfRadial2 naming differences ({issue}`93`, {issue}`287`) by [@syedhamidali](https://github.com/syedhamidali) * ENH: Move station coordinates (``latitude``, ``longitude``, ``altitude``) to root node as coordinates for DataTree coordinate inheritance ({issue}`331`, {pull}`333`) by [@aladinor](https://github.com/aladinor) * ENH: Add ``optional_groups`` parameter (default ``False``) to all ``open_*_datatree()`` functions to control inclusion of ``/radar_parameters``, ``/georeferencing_correction``, and ``/radar_calibration`` subgroups ({issue}`331`, {pull}`333`) by [@aladinor](https://github.com/aladinor) * ENH: Skip redundant station coordinate reads for subsequent sweeps in DataTree by passing ``site_as_coords=False`` and extracting shared ``_apply_site_as_coords`` helper ({issue}`334`, {pull}`337`) by [@aladinor](https://github.com/aladinor) From e787fb420abd9d16a5f3bdd392f525ec5eb5d061 Mon Sep 17 00:00:00 2001 From: syedhamidali Date: Wed, 1 Apr 2026 14:35:42 +0530 Subject: [PATCH 4/5] MNT: Update history attribution --- docs/history.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/history.md b/docs/history.md index 1beb7c0c..3374e475 100644 --- a/docs/history.md +++ b/docs/history.md @@ -3,7 +3,7 @@ ## Development -* MNT: Clarify contributor, team-member, and maintainer roles in the contributing guide, including the pathway to greater project involvement ({issue}`341`) by [@syedhamidali](https://github.com/syedhamidali) +* MNT: Clarify contributor, team-member, and maintainer roles in the contributing guide, including the pathway to greater project involvement ({issue}`341`) by [@kmuehlbauer](https://github.com/kmuehlbauer), ({pull}`354`) by [@syedhamidali](https://github.com/syedhamidali) * ADD: ``open_cfradial2_datatree`` reader with grouped CfRadial2 compatibility normalization for common FM301/CfRadial2 naming differences ({issue}`93`, {issue}`287`) by [@syedhamidali](https://github.com/syedhamidali) * ENH: Move station coordinates (``latitude``, ``longitude``, ``altitude``) to root node as coordinates for DataTree coordinate inheritance ({issue}`331`, {pull}`333`) by [@aladinor](https://github.com/aladinor) * ENH: Add ``optional_groups`` parameter (default ``False``) to all ``open_*_datatree()`` functions to control inclusion of ``/radar_parameters``, ``/georeferencing_correction``, and ``/radar_calibration`` subgroups ({issue}`331`, {pull}`333`) by [@aladinor](https://github.com/aladinor) From 08f0cd8456c4d6346ae88ca7e78c0cd119389bca Mon Sep 17 00:00:00 2001 From: syedhamidali Date: Wed, 1 Apr 2026 14:42:19 +0530 Subject: [PATCH 5/5] MNT: Add missing history pull reference --- docs/history.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/history.md b/docs/history.md index 3374e475..fc69ad4a 100644 --- a/docs/history.md +++ b/docs/history.md @@ -4,7 +4,7 @@ ## Development * MNT: Clarify contributor, team-member, and maintainer roles in the contributing guide, including the pathway to greater project involvement ({issue}`341`) by [@kmuehlbauer](https://github.com/kmuehlbauer), ({pull}`354`) by [@syedhamidali](https://github.com/syedhamidali) -* ADD: ``open_cfradial2_datatree`` reader with grouped CfRadial2 compatibility normalization for common FM301/CfRadial2 naming differences ({issue}`93`, {issue}`287`) by [@syedhamidali](https://github.com/syedhamidali) +* ADD: ``open_cfradial2_datatree`` reader with grouped CfRadial2 compatibility normalization for common FM301/CfRadial2 naming differences ({issue}`93`, {issue}`287`), ({pull}`349`) by [@syedhamidali](https://github.com/syedhamidali) * ENH: Move station coordinates (``latitude``, ``longitude``, ``altitude``) to root node as coordinates for DataTree coordinate inheritance ({issue}`331`, {pull}`333`) by [@aladinor](https://github.com/aladinor) * ENH: Add ``optional_groups`` parameter (default ``False``) to all ``open_*_datatree()`` functions to control inclusion of ``/radar_parameters``, ``/georeferencing_correction``, and ``/radar_calibration`` subgroups ({issue}`331`, {pull}`333`) by [@aladinor](https://github.com/aladinor) * ENH: Skip redundant station coordinate reads for subsequent sweeps in DataTree by passing ``site_as_coords=False`` and extracting shared ``_apply_site_as_coords`` helper ({issue}`334`, {pull}`337`) by [@aladinor](https://github.com/aladinor)