ashleychontos / pySYD

automated measurements of global asteroseismic parameters
https://pysyd.readthedocs.io
MIT License
25 stars 13 forks source link

Update target.py #41

Closed saiprathyushamalla closed 2 years ago

saiprathyushamalla commented 2 years ago

name: Update target.py about: pysyd returns a Value error saying "ValueError: All arrays must be of the same length". After looking into the code for the reason, I noticed this happened when the self.get_background() does not converge. So I added a condition to only consider the parameters if get_background() converges. title: '' labels: '' assignees: ''



Relevant Materials

Description

Using the checklist as a guide, please provide more details about the nature of the pull request.

Attachments & Supporting Files

Please provide any relevant files i.e. data if adding a new target to a test or as an example

Other

Any other relevant context i.e. reference/link if directly addressing an existing issue


Summary Checklist:


ashleychontos commented 2 years ago

Many thanks!