Skip to content

Conversation

dantesab9
Copy link
Contributor

Hi @aaron-kaplan , new attempt here. Thanks for the review, I added the version to be displayed as default this time and seems like there's no conflicts.

@dantesab9 dantesab9 requested a review from aaron-kaplan August 4, 2025 20:02
@dantesab9 dantesab9 self-assigned this Aug 4, 2025
Copy link
Collaborator

@aaron-kaplan aaron-kaplan 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. Ready for me to deploy this to the public site?

@aaron-kaplan
Copy link
Collaborator

BTW there's a guatemala-may config in fbfmaproom-sample.yaml that we never deployed to the public site. Do we still need it?

@dantesab9
Copy link
Contributor Author

Yeah, it is ready for deployment to public. datasets guatemala/prcp-jja-v6 and guatemala/prcp-aso-v4 would need to be synced (I can send a separate email for that.) About the guatemala-may config, we don't need it anymore, should I delete it in a new branch? Btw, is it ok if I delete closed branches I opened?

@aaron-kaplan
Copy link
Collaborator

No need for a separate email. Yes, if you could submit a PR that removes guatemala-may that would be great.

What are we doing with prcp-jja-v5? That's still sitting in fbf-candidate, not synced to fbf.

@aaron-kaplan aaron-kaplan merged commit c963552 into master Aug 4, 2025
1 check passed
@aaron-kaplan aaron-kaplan deleted the guate_new_forecasts_2025 branch August 4, 2025 20:15
@dantesab9
Copy link
Contributor Author

Ok I will remove may. Let's keep prcp-jja-v5 a little longer in candidate in case it is brought back by partners.

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.

2 participants