Skip to content

added systematics vs galaxy properties notebook#120

Open
zdu863 wants to merge 2 commits into
masterfrom
u/zdu863/issues/118
Open

added systematics vs galaxy properties notebook#120
zdu863 wants to merge 2 commits into
masterfrom
u/zdu863/issues/118

Conversation

@zdu863
Copy link
Copy Markdown

@zdu863 zdu863 commented Aug 22, 2019

Address #118 with systematics maps from Run 1.2i.

@reneehlozek reneehlozek self-requested a review November 28, 2019 20:00
Copy link
Copy Markdown
Contributor

@reneehlozek reneehlozek left a comment

Choose a reason for hiding this comment

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

This notebook relies on healsparse - so perhaps a few installation notes at the beginning would be useful? I'm having a little trouble coordinating the healsparse installation from @esheldon repo with this notebook (most probably just environment issues).

@esheldon
Copy link
Copy Markdown

The repo is here: https://github.com/lsstdesc/healsparse

(@erykoff is the primary author)

it isn't in pypi or conda forge yet, so you need to install from the repo. However the dependencies can all be installed through pip or conda

@reneehlozek
Copy link
Copy Markdown
Contributor

hi @zdu863, would you be able to add a few notes on healsparse at the top of this notebook so that we can close/approve the PR?

@yymao yymao linked an issue Jul 18, 2020 that may be closed by this pull request
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.

Add validation notebook using observing conditions map

3 participants