-
Notifications
You must be signed in to change notification settings - Fork 20
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
CEPH Round 02 #31
CEPH Round 02 #31
Conversation
Hi @paulocv , Thank you for the submission, it looks like there is an issue with your submission file. I try to load it, and arrow package returns an error: Would it be possible to verify it, please? Please let me know if any questions or issues, |
Thank you for checking. Have you tried to decompress the file first? Since the flat I was able to reproduce your error message in
opens without problems. |
Hi @paulocv , Thank you for the answer, and the information. For the compression file, would it be possible to compress it with pyarrow so I can open it with arrow, please? Arrow can normally open compress files. (https://arrow.apache.org/docs/python/parquet.html#compression-encoding-and-file-compatibility)
The loading process of all the data use arrow packages, so it's important to be able to read it with it. Does that make sense? Please let me know if any questions or issues, |
Thank you for the instructions, I was able to create the I will clone the repo and submit using git, which I hadn't done so far to avoid cloning the entire repo to my computer. |
Thank you for the update, Best, Lucie |
Run validation on files: 2024-07-28-CEPH-MetaRSV.gz.parquet Required values:No missing required value found Columns:❌ Error 102: The data frame should contains 9 columns, not 10. Please verify if one or multiple columns have been added. Scenarios:No errors or warnings found on scenario name and scenario id columns Origin Date Column:No errors or warnings found on the column 'origin_date' Value and Type Columns:🟡 Warning 5043: All values associated with output type 'sample' should have a maximum of 1 decimal place Target Columns:🟡 Warning 602: No value found associated with the targets: inc hosp (optional), cum hosp (optional), inc inf (optional), cum inf (optional); output_type: quantile. Locations:No errors or warnings found on Location Sample:Column Pairing information:Run grouping pairing: Quantiles:No errors or warnings found on quantiles values and format Age Group:No errors or warnings found on Age_group |
Run validation on files: 2024-07-28-CEPH-MetaRSV.gz.parquet Required values:No missing required value found Columns:No errors or warnings found on the column names and numbers Scenarios:No errors or warnings found on scenario name and scenario id columns Origin Date Column:No errors or warnings found on the column 'origin_date' Value and Type Columns:🟡 Warning 5043: All values associated with output type 'sample' should have a maximum of 1 decimal place Target Columns:🟡 Warning 602: No value found associated with the targets: inc hosp (optional), cum hosp (optional), inc inf (optional), cum inf (optional); output_type: quantile. Locations:No errors or warnings found on Location Sample:Column Pairing information:Run grouping pairing: Quantiles:No errors or warnings found on quantiles values and format Age Group:No errors or warnings found on Age_group |
model-metadata/CEPH-MetaRSV.yaml
Outdated
@@ -11,7 +11,7 @@ model_contributors: [ | |||
{ | |||
"name": "Shreeya Mhade", | |||
"affiliation": "Indiana University Bloomington", | |||
"email": "[email protected]" | |||
"email": "[email protected]", |
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.
"email": "[email protected]", | |
"email": "[email protected]" |
The additional comma is not necessary here, please feel free to remove it.
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.
Thanks for the submission, I just made a small comment on the metadata file, but it does not block the PR.
Please feel free to let me know if any questions or issues,
Best, Lucie
Thank you @LucieContamin . I removed any trailing commas from the YAML elements. |
Here is our submission for the SMH round 2, including minor changes to the metadata.
Feel free to use the comments to let us know about any issues with the files.