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

Prototype: Satellite #1240

Closed
colinmurphy01 opened this issue May 21, 2024 · 12 comments · Fixed by #1310
Closed

Prototype: Satellite #1240

colinmurphy01 opened this issue May 21, 2024 · 12 comments · Fixed by #1310

Comments

@colinmurphy01
Copy link
Collaborator

colinmurphy01 commented May 21, 2024

@coreypieper coreypieper changed the title Prototype: Satelite Prototype: Satellite Jun 3, 2024
@colinmurphy01
Copy link
Collaborator Author

Assigning to us in product to get this clarified with NESDIS

@colinmurphy01
Copy link
Collaborator Author

Slack thread

  • I think we just need to know which WFOs are serviced by GOES18 and which are GOES16
  • Which products we should start with, based on NESDIS recs

@colinmurphy01
Copy link
Collaborator Author

Email sent

@colinmurphy01
Copy link
Collaborator Author

@greg-does-weather did we meet with NESDIS last week?

@greg-does-weather
Copy link
Collaborator

No. We've got an email thread started with them and an invitation to schedule time, so maybe we can do it next week.

@colinmurphy01
Copy link
Collaborator Author

@greg-does-weather is this unblocked now?

@greg-does-weather
Copy link
Collaborator

I guess we can use the JSON to figure out whether to use GOES16 or 18 which resolves the original blocking issue, but I'm not sure there's anything to prototype. It seems like satellite is ready for research/design/product?

Noting here so I don't lose the URL in the future, those JSON files use a lowercase WFO code, like so for MPX:
https://cdn.star.nesdis.noaa.gov/WFO/catalogs/WFO_02_mpx_catalog.json
I'm not sure which field(s) in there tell us explicitly whether to use 16 or 18, but in the worst case we can figure it out by parsing one of the static image URLs, or we could ask the satellite team to include a link to the gif.

@colinmurphy01
Copy link
Collaborator Author

@partly-igor @shadkeene @coreypieper Greg has a good point that this should be ready to design and build. There isn't much more to learn from an engineering perspective.

My question to you all is, would it be beneficial or an issue if we shipped Greg's prototype to production (updated with the added logic)?

@shadkeene
Copy link
Collaborator

shadkeene commented Jul 29, 2024

heck yes!!! go for it! @colinmurphy01 @coreypieper @partly-igor Sorry for the delayed response.

@greg-does-weather Can we deploy on this release?

@partly-igor
Copy link
Collaborator

@greg-does-weather would you mind deploying it to your env (or design) before we release? I want to take a look from design perspective

@greg-does-weather
Copy link
Collaborator

@partly-igor https://weathergov-greg.app.cloud.gov/point/44.998/-93.156?

Note that it currently only works for the parts of the country served by GOES 16, so if you go to other locations and see nothing, that could be why.

@greg-does-weather
Copy link
Collaborator

@colinmurphy01 @shadkeene From just an engineering perspective, yes, I think we can.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

5 participants