CU 3/10 #22
Merged
CU 3/10 #22
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…validate-submission.yaml to invoke a workflow to push data after pr merged, because a PR from forked repo cannot permit using secrets
…ing dir before git diff
…ing dir before git diff
…th modifying origin_date
…gz) with modifying origin_date
…to tkcy-main deleted 2024-03-10-CU-RSV_SVIRS.parquet.gz
|
@tkcy Could you check the file https://github.com/tkcy/rsv-forecast-hub/blob/main/model-output/CU-RSV_SVIRS/2024-03-10-CU-RSV_SVIRS.parquet which I converted the origin_date column if it is okay? |
|
@tkcy I will proceed with it though I did not get an answer, because it seemed no problem would be seen |
|
Oh yes sorry I missed this! Thanks!
Get Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Koji SATO ***@***.***>
Sent: Thursday, March 14, 2024 1:29:01 PM
To: HopkinsIDD/rsv-forecast-hub ***@***.***>
Cc: Yamana, Teresa K. ***@***.***>; Mention ***@***.***>
Subject: [EXTERNAL] Re: [HopkinsIDD/rsv-forecast-hub] CU 3/10 (PR #22)
Merged #22<#22> into main.
—
Reply to this email directly, view it on GitHub<#22 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABSAKVPQY6IGYYCDUTNGACDYYHM53AVCNFSM6AAAAABES66CVKVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGEZDCNZSGQ2TCOA>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
|
No problem, the result can be found: |
|
Thanks, the visualization is helpful! I see I've been misunderstanding the numbering for horizon week, I have been starting at horizon 1 for the first week of projection instead of 0. I'll correct this going forward. |
|
Okay, please issue a new pr after your modification is done. |
kjsato
added a commit
that referenced
this pull request
Mar 26, 2024
modified and deleted the path test code part in ensemble.R
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.