-
Notifications
You must be signed in to change notification settings - Fork 38
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
Initial Hurricane Ian Push #306
Conversation
✅ Deploy Preview for visex ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Originally uploaded a PNG and need jpg for the cover photo... I'm learning...
Used a jpg instead of png this time..
Updated the format for the key and "other relevant" publications, as requested by the data provider
Added space before report
@freitagb - can we get a review of this and then request a PR merge? |
Head's up we are currently working through the transfer of the assets in this PR from staging to production. Please do not merge until the NASA-IMPACT/veda-data#148 is complete. |
The thumbnail for Hurricane Ian is too big and long. Can it be replaced? Also, multiple images are over 3000px width-wise and > 1 MB. Can you adjust the size of the images and optimize them? You can use tools like https://imageoptim.com/api |
@hanbyul-here I've compressed all the images and am having Trent put them in the story! as for the cover photo - we're going with a new one (likely this image; https://visibleearth.nasa.gov/images/150408/hurricane-ian-reaches-florida ). What size does it need to be? |
@hanbyul-here Hi! I have updated the compressed images in the data story along with the cover photo. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@WeathermanTrent @jerikac Thanks for taking care of the changes. They look great.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My apologies! but I just realized that hurricane-front.jpg has watermarks. can you change it? @jerikac or @WeathermanTrent ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I change the thumbnails for datasets to use the one without watermark. I believe it is good to go!
I think the production catalog is now also ready for these stac collections |
Why are you creating this Pull Request?