Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

PSFEx validation catalogue: ACCEPTED compares unrelated numberings

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#980 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
74/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
data

Hướng nghiên cứu

Bắt đầu trong PSFExInterpolator._write_output_validation trong src/shapepipe/modules/psfex_interp_package/psfex_interp.py, nơi ACCEPTED được điền từ SOURCE_NUMBER, và xác nhận sự không khớp giữa cách đánh số hàng của training và validation. Sau đó đi theo cột ACCEPTED vào các cột tùy chọn của merge_starcat.py và xóa nó khỏi cả hai đường dẫn đầu ra. Hoàn thành nghĩa là các danh mục validation không còn phát ra ACCEPTED và kiểu merge cổ điển không còn điền số không cho nó, với các test bị ảnh hưởng được điều chỉnh.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

The ACCEPTED column of the PSFEx validation catalogues carries no information.
It is not a science bug: rho and tau statistics use the held-out validation stars either way.
But anyone reading ACCEPTED as "PSFEx used this star" is misled.

PSFExInterpolator._write_output_validation (src/shapepipe/modules/psfex_interp_package/psfex_interp.py) sets it like this:

data["ACCEPTED"] = np.ones_like(data["NUMBER"], dtype="int16")
star_used = psfex_cat_dict.pop("SOURCE_NUMBER")
for idx in range(len(data["NUMBER"])):
    if idx + 1 not in star_used:
        data["ACCEPTED"][idx] = 0

SOURCE_NUMBER numbers the rows of the PSFEx training catalogue (.cat).
idx + 1 is the row position in the validation catalogue, which holds the other 20% of stars (setools RAND_SPLIT, RATIO=20).
The two numberings are unrelated, so the test compares a validation row index with training row numbers.

In v2 production, every validation star gets ACCEPTED = 1.
In classic runs, merge_starcat.py zero-fills the column after the pix2world conversion, so v1.4 star catalogues have all zeros.

By construction, a validation star was never offered to PSFEx, so "accepted by PSFEx" is not defined for it.
We propose to drop the column from the validation output and from merge_starcat's optional columns.
If a per-star training flag is wanted, it belongs in the training catalogue, matched on NUMBER.

— Claude (Opus) on behalf of Cail

Ngôn ngữ chính
Python
Star
18
Fork
14
Merge trung bình
4 ngày 14 giờ
Pull request đã merge (30 ngày)
44

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của CosmoStat/shapepipe

Tất cả issue của CosmoStat/shapepipe

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.