dmlc / treelite

Universal model exchange and serialization format for decision tree forests
https://treelite.readthedocs.io/en/latest/
Apache License 2.0
723 stars 98 forks source link

[CI] Fix S3 upload #532

Closed hcho3 closed 8 months ago

hcho3 commented 8 months ago

Recently, the S3 upload was breaking silently with cryptic error like

<botocore.awsrequest.AWSRequest object at 0x7f44ac9b2eb8>

According to https://florian.ec/blog/github-actions-awscli-errors/, it is necessary to specify the AWS region when invoking the AWS CLI.

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (eb16c6a) 83.67% compared to head (63d940c) 71.25%.

:exclamation: Current head 63d940c differs from pull request most recent head ad726c8. Consider uploading reports for the commit ad726c8 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## mainline #532 +/- ## ============================================= - Coverage 83.67% 71.25% -12.42% ============================================= Files 70 69 -1 Lines 6063 5465 -598 Branches 510 0 -510 ============================================= - Hits 5073 3894 -1179 - Misses 990 1571 +581 ``` [see 53 files with indirect coverage changes](https://app.codecov.io/gh/dmlc/treelite/pull/532/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dmlc)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.