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

Initial Hurricane Ian Push #306

Merged
merged 106 commits into from
Jul 17, 2024
Merged

Initial Hurricane Ian Push #306

merged 106 commits into from
Jul 17, 2024

Conversation

WeathermanTrent
Copy link
Collaborator

Why are you creating this Pull Request?

@netlify
Copy link

netlify bot commented Aug 25, 2023

Deploy Preview for visex ready!

Name Link
🔨 Latest commit 5a2f4ac
🔍 Latest deploy log https://app.netlify.com/sites/visex/deploys/6696dafa23ec01000956c1be
😎 Deploy Preview https://deploy-preview-306--visex.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

@moradology moradology left a comment

Choose a reason for hiding this comment

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

lgtm

jerikac and others added 8 commits April 2, 2024 16:02
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
siddharth0248 pushed a commit that referenced this pull request Apr 26, 2024
@jerikac
Copy link
Contributor

jerikac commented Jun 10, 2024

@freitagb - can we get a review of this and then request a PR merge?

@anayeaye
Copy link
Collaborator

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.

@hanbyul-here
Copy link
Collaborator

The thumbnail for Hurricane Ian is too big and long. Can it be replaced?
Screenshot 2024-07-16 at 1 09 26 PM

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

@jerikac
Copy link
Contributor

jerikac commented Jul 16, 2024

@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?

@WeathermanTrent
Copy link
Collaborator Author

@hanbyul-here Hi! I have updated the compressed images in the data story along with the cover photo.

Copy link
Collaborator

@hanbyul-here hanbyul-here left a 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.

Copy link
Collaborator

@hanbyul-here hanbyul-here left a 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 ?

Copy link
Collaborator

@hanbyul-here hanbyul-here left a 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!

@anayeaye
Copy link
Collaborator

I think the production catalog is now also ready for these stac collections
NASA-IMPACT/veda-data#148 (comment)

@sandrahoang686 sandrahoang686 merged commit 6f18fbc into develop Jul 17, 2024
6 checks passed
@sandrahoang686 sandrahoang686 deleted the hurricane-ian branch July 17, 2024 18:02
@hanbyul-here hanbyul-here mentioned this pull request Jul 18, 2024
1 task
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.

None yet

7 participants