search

Keck-DataReductionPipelines / MosfireDRP

http://keck-datareductionpipelines.github.io/MosfireDRP
10 stars 13 forks source link

Plots in interactive wavelength fitting not displaying points for residuals #28

Closed followthesheep closed 9 years ago

followthesheep commented 9 years ago

The interactive wavelength fitting right now looks like this:

figure_1

The bottom plot has no points on it. I think it's because the xlim variable in Wavelength.py is undefined. The plotting is done in draw_found_lines().

lucarizzi commented 9 years ago

Is this true even after you use the "x" key to zoom with the cursor somewhere within the boundaries of the lower plot? If so, this behavior was already there in the previous version

On Jul 7, 2015, at 8:50 AM, Tuan Do notifications@github.com wrote:

The interactive wavelength fitting right now looks like this:

The bottom plot has no points on it. I think it's because the xlim variable in Wavelength.py is undefined. The plotting is done in draw_found_lines().

— Reply to this email directly or view it on GitHub.

followthesheep commented 9 years ago

The points appear after I use the x key.

On Tue, Jul 7, 2015 at 12:03 PM lucarizzi notifications@github.com wrote:

Is this true even after you use the "x" key to zoom with the cursor somewhere within the boundaries of the lower plot? If so, this behavior was already there in the previous version

On Jul 7, 2015, at 8:50 AM, Tuan Do notifications@github.com wrote:

The interactive wavelength fitting right now looks like this:

The bottom plot has no points on it. I think it's because the xlim variable in Wavelength.py is undefined. The plotting is done in draw_found_lines().

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub https://github.com/Keck-DataReductionPipelines/MosfireDRP/issues/28#issuecomment-119304836 .

KeckDRP commented 9 years ago

This error was there in the original version of the pipeline. Fixed today with eea36d6ad6b01520db56abb9eab547865ae365db (for @followthesheep: I cherry-picked this commit to your install-update branch)