Skip to content

Update get-issues-and-prs.py script.#1

Merged
ericonr merged 3 commits intomasterfrom
update-script
Jun 3, 2025
Merged

Update get-issues-and-prs.py script.#1
ericonr merged 3 commits intomasterfrom
update-script

Conversation

@ericonr
Copy link
Copy Markdown
Member

@ericonr ericonr commented May 5, 2025

It was missing some repositories, so stop hardcoding those.

Make it capable of listing stale issues and PRs as well.

Sample output:

- areaDetector:
    - PRs:
        - [ ] [Add ADPimega as a submodule (branch: lumentum-v3)](https://github.com/areaDetector/areaDetector/pull/95)
- ADCore:
    - Issues:
        - [ ] [Out of bounds read/type confusion in compressed arrays with NDPluginStdArrays, can cause segmentation fault](https://github.com/areaDetector/ADCore/issues/531)
        - [ ] [Convert NDPluginPva to use PVXS](https://github.com/areaDetector/ADCore/issues/528)
    - PRs:
        - [ ] [Convert NDPluginPva to use PVXS](https://github.com/areaDetector/ADCore/pull/532)
- ADEiger:
    - Issues:
        - [ ] [The lz4 data in Stream2 is different from Stream](https://github.com/areaDetector/ADEiger/issues/69)
        - [ ] [Driver is copying data twice for each frame in Stream mode, not efficient](https://github.com/areaDetector/ADEiger/issues/68)
        - [ ] [Support Stream2 CBOR format and multiple threshold frames](https://github.com/areaDetector/ADEiger/issues/65)
- ADLambda:
    - Issues:
        - [ ] [Probable bug in driver](https://github.com/areaDetector/ADLambda/issues/13)
- ADPICam:
    - PRs:
        - [ ] [Implement pulse parameters and re-populate enums after a parameter is marked as existing](https://github.com/areaDetector/ADPICam/pull/26)
- ADPcoWin:
    - Issues:
        - [ ] [Linux support?](https://github.com/areaDetector/ADPcoWin/issues/24)
- ADPilatus:
    - PRs:
        - [ ] [Check for 7 OK message when waiting for an image to actualy end the acquisition](https://github.com/areaDetector/ADPilatus/pull/17)
- Stale Items:
    - Stale Issues:
        - [ ] [HDF5 plugin feature where datasets with type=constant is broken](https://github.com/areaDetector/ADCore/issues/36)
        - [ ] [Find a way to stop misbehaving plugins from slowing down driver](https://github.com/areaDetector/ADCore/issues/98)
        - [ ] [HDF5 NDAttribute datasets to support "extra" dimensions like detector datasets](https://github.com/areaDetector/ADCore/issues/96)
        - [ ] [Expand CI services to cover more configurations](https://github.com/areaDetector/ADCore/issues/115)
        - [ ] [option to not write NDAttr* attributes in HDF5 dataset](https://github.com/areaDetector/ADCore/issues/145)
        - [ ] [MEDM slider behaves badly on some Windows systems](https://github.com/areaDetector/ADPhotron/issues/6)
        - [ ] [Allow use of the 2nd ethernet port of the SA-Z for data transfer](https://github.com/areaDetector/ADPhotron/issues/7)
    - Stale PRs:
        - [ ] [prosilica.cpp:](https://github.com/areaDetector/ADProsilica/pull/4)
        - [ ] [Added new prosilica resend params and rates for packet counts](https://github.com/areaDetector/ADProsilica/pull/7)
        - [ ] [Added driver support for NDBitsPerPixel and NDBytesPerPixel parameters](https://github.com/areaDetector/ADProsilica/pull/8)
        - [ ] [Added redef of Gain in prosilica.template so we can set driver limits](https://github.com/areaDetector/ADProsilica/pull/5)
        - [ ] [FIX: call start() after NDEdgePlugin object is created for plugin to work for ADCore >= 2.5,  don't process plugin if input NDArray dimension > 2 (i.e. for Color modes)](https://github.com/areaDetector/ADPluginEdge/pull/3)
        - [ ] [Slac edl](https://github.com/areaDetector/ADCore/pull/316)
        - [ ] [Applied bitsPerPixel.patch to bring in bitsPerPixel support from pcds…](https://github.com/areaDetector/ADCore/pull/314)

Copy link
Copy Markdown
Member

@henriquesimoes henriquesimoes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't notice this was a PR before. Just a few suggestions.

Comment thread get-issues-and-prs.py Outdated
Comment thread get-issues-and-prs.py Outdated
Comment thread get-issues-and-prs.py
ericonr added 3 commits May 29, 2025 15:20
7 repositories were missing from the list. Instead of adding them
manually, refactor the script to automatically fetch the list of
repositories from the organization, and add a sort_repos function to
keep the areaDetector, ADCore and ADSupport output on top, since those
are the main repositories.
The stale items are sampled from all areaDetector repositories.

In order to achieve this, it was necessary to increase the amount of
issues and PRs that could be returned from gh, since otherwise they
would be limited to 30, and could miss one of the stale issues. 1000 is
used as a stand-in for "infinite".
@ericonr ericonr requested a review from henriquesimoes May 29, 2025 18:23
@ericonr ericonr merged commit 66b713e into master Jun 3, 2025
@ericonr ericonr deleted the update-script branch June 3, 2025 14:30
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 this pull request may close these issues.

2 participants