Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update the front page of website #163

Merged
merged 9 commits into from
Dec 5, 2018
Merged

Conversation

yymao
Copy link
Member

@yymao yymao commented Dec 3, 2018

As discussed in #160, the current front page is very outdated. Before we come up with a more permanent solution, we should put something more informative on the front page. This PR makes a quick update to the front page.

You can preview this PR here: https://portal.nersc.gov/project/lsst/descqa/v2-new-home/

rmandelb
rmandelb previously approved these changes Dec 4, 2018
Copy link

@rmandelb rmandelb left a comment

Choose a reason for hiding this comment

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

This is definitely a better placeholder than the current version! I like the links to specific protoDC2/cosmoDC2 versions and the indication of the corresponding image simulation versions. Thanks, @yymao - I think that people who are looking at the extragalactic catalogs or image sims and want to see the validation tests for those catalogs will find this a more useful landing page.

@katrinheitmann
Copy link

Thanks Yao! May you could put a link somewhere to the definitions of the catalogs (the confluence page) -- e.g. there are no DESCQA results for the whole cosmoDC2 catalogs, just for e.g. "small". (small is not very descriptive, so if there is the link that people can then look up to understand what it means that will be helpful). Thanks!

@yymao
Copy link
Member Author

yymao commented Dec 4, 2018

@rmandelb @katrinheitmann thanks for the feedback. I've added some links for reference per @katrinheitmann's suggestion. Can you take a look to see if that's good? Thank you!

@evevkovacs
Copy link
Contributor

@yymao @katrinheitmann @rmandelb The "official" CS page describing the catalogs is here ( https://confluence.slac.stanford.edu/display/LSSTDESC/ProtoDC2+and+CosmoDC2+Information )
and I think it would be useful to link that page. BTW, the "small" area catalogs are 17 healpixels, not 16, as stated here (https://github.com/LSSTDESC/gcr-catalogs#available-catalogs)

@katrinheitmann
Copy link

katrinheitmann commented Dec 4, 2018 via email

@evevkovacs
Copy link
Contributor

Yes it's true there is some overlap. The "ExtraGalactic-Catalog" page is older, but has some additional information (including Buzzard) and a lot more information about accessing the catalog and contributing to DESCQA. I don't want to clutter the summary page (protoDC2 and cosmoDC2 information) with all the information in the EGC page. Let me think about how to break things up. Maybe the EGC page could be broken into a couple of child pages...

@yymao
Copy link
Member Author

yymao commented Dec 4, 2018

I think we should consolidate the confluence pages on extragalactic catalogs and link the consolidated page from the data product overview page.

In the meantime, should we try to merge this PR first so that the DESCQA website can be updated?

Also, @evevkovacs, can you open a PR in gcr-catalogs to fix the typo in the README? Thanks!

@rmandelb
Copy link

rmandelb commented Dec 5, 2018

@yymao - the links you added look like good additions to me.

@yymao
Copy link
Member Author

yymao commented Dec 5, 2018

Thanks @rmandelb.

The PR hasn't gotten a new approval since the last update yet. @katrinheitmann @rmandelb Let me know if you want to hold off of merging this, or if not, can one of you approve the PR? Thank you!

Copy link

@katrinheitmann katrinheitmann left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks!

@yymao yymao merged commit e913357 into master Dec 5, 2018
@yymao yymao deleted the issues/160/update-web-front-page branch December 5, 2018 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants