Periodogram panics on an empty light curve

Open Beginner friendly
  • Is something wrong?
#870 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

@hombit is already working on this.

Since Nov 5, 2026.

  • #880 by @hombit — open

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
68/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Active
Tech stack
python, rust
Domain
data

Research direction

Start at the Python Periodogram entry point and trace how an empty light curve reaches the Rust implementation. Check how fill_value is handled for empty inputs, then add or locate a regression test using the issue’s example. Done when the call returns the configured fill value without raising PanicException.

Written by the indexing model from the issue text.

Description

bug

Title: Periodogram panics on an empty light curve instead of returning fill_value

Periodogram raises a Rust panic for an empty light curve, even when fill_value is set.

import numpy as np                                               
import light_curve as licu                                       
                                                                 
empty = np.array([], dtype=np.float64)                           
licu.Periodogram()(empty, empty, empty, fill_value=np.nan)  #   PanicException
Dominant language
Python
Stars
48
Forks
11
Avg merge
11h 53m
Merged PRs (30d)
7

Getting set up

  • No Dockerfile or Docker Compose file
  • Has a pull request template
  • No contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from light-curve/light-curve-python

All issues in light-curve/light-curve-python

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.