Front end: a resolution choice for the spectra shown

Đang mở
  • Có gì không ổn?
#554 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ó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
48/100
Loại issue
Tính năng
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
data-visualization, frontend

Hướng nghiên cứu

Start with gui/src/solprop_gui/service/reduce.py and trace how the Overview, At this plane, scan spectrum maps, and spatial class maps call solprop's spectral readers and plotting functions. Check how the session stores the floor and spectral axis, and how plots and exports report settings. Done means a persistent resolution control reaches supported readers and the chosen resolution appears in plot readouts and exported figures.

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

Mô tả

area: gui enhancement priority: low

The front end draws spectra at a fixed sampling, and offers no instrument resolution. It would help to choose the resolution of the spectra shown: to compare with a spectrometer's, or to see a broad continuum without its fine fringes.

Today

  • `gui/src/solprop_gui/service/reduce.py`: `Display.n_wavelength = 512` bins the spectrum along z (and `n_time = 2048` the pulse). These are display bins, not a resolution, and they aren't user-settable.
  • solprop's readers already take an instrument resolution: `processing.fields.get_spectral_density(..., resolution=)` (a full width at half maximum, in the abscissa's units), and `solprop.plotting.prop_2d` and `spec_1d` take one too. The front end passes none.

What it should do

  • The control: a Resolution control beside the spectral axis on the pages that draw a spectrum: Overview, At this plane, the scan's spectrum maps, and the spatial classes' spectral maps where their readers support it.
    • Choices: "Full" (as now), or a width in nm (wavelength axis) or THz (frequency axis).
    • The width is passed to solprop's readers as `resolution=`, so the smoothing is solprop's and matches what a script would get.
  • Keeping it: the choice persists per session, like the floor and the spectral axis.
  • Saying it: the plot's readout and the exported figure say the resolution used.
  • Optional: a choice of the number of display bins, for very broad or very fine spectra.

Found while testing the front end for 0.2 (#484).

Ngôn ngữ chính
Python
Star
3
Fork
3
Merge trung bình
8 giờ 37 phút
Pull request đã merge (30 ngày)
159

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 LupoLab/solprop

Tất cả issue của LupoLab/solprop

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.