Skip to content

Conversation

@tmesh
Copy link
Contributor

@tmesh tmesh commented Mar 26, 2025

updating notebook 4 to be a cutout of the MER image, updated the adql query, updated the author list

df_g_irsa['y_pix']=xy_irsa[1]
```

Due to the large field of view of the MER mosaic, let's cut out a smaller section (30'x30')of the MER mosaic to inspect the image
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not use the astropy cutout?

Copy link
Member

@bsipocz bsipocz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this can still use a lot of cleanup, but the notebook works and renders without an error, so I go ahead and merge it.

Some comments: the object we pull the spectra for has changed, and the new one looks kind of flat. Is that change intended? (the previous one, besides the obvious noisy spikes, looked to have some feature around 1.35 microns.

mpl understands astropy and its units, so I'll add that to the list of cleanups

### Now the data are read in, plot the spectrum

## Now the data are read in, show an image
Divide by 10000 to convert from Angstrom to micron
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, all of these should be dealt with astropy, but I leave that for a future PR.


plt.plot(df_obj_irsa['WAVELENGTH'], df_obj_irsa['SIGNAL'])
```{code-cell} ipython3
plt.plot(df_obj_irsa['WAVELENGTH']/10000., df_obj_irsa['SIGNAL'])
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This new spectra is kind of flat, while the previous one looked like having some signal around 1.35microns. Is that OK?

@bsipocz bsipocz merged commit c51c38b into Caltech-IPAC:main Mar 27, 2025
8 checks passed
@bsipocz
Copy link
Member

bsipocz commented Mar 27, 2025

Thanks!

github-actions bot pushed a commit that referenced this pull request Mar 27, 2025
Updated so MER images plotted as cutouts c51c38b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants