Skip to content

Clear cache #2313

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

Merged
merged 9 commits into from
Jan 12, 2021
Merged

Clear cache #2313

merged 9 commits into from
Jan 12, 2021

Conversation

annzimmer
Copy link
Contributor

No description provided.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jan 11, 2021

Coverage Report

Affected SDKs

  • firebase-ml-modeldownloader

    SDK overall coverage changed from 84.31% (3dee255) to 84.25% (fdfcac43) by -0.06%.

    Filename Base (3dee255) Head (fdfcac43) Diff
    CustomModel.java 73.33% 73.02% -0.32%
    CustomModelDownloadService.java 86.41% 85.58% -0.83%
    FirebaseModelDownloader.java 89.62% 87.59% -2.03%
    FirebaseModelDownloaderRegistrar.java 88.57% 94.29% +5.71%
    ModelFileDownloadService.java 91.71% 91.44% -0.26%

Test Logs

Notes

HTML coverage reports can be produced locally with ./gradlew <product>:checkCoverage.
Report files are located at <product-build-dir>/reports/jacoco/.

Head commit (fdfcac43) is created by Prow via merging commits: 3dee255 3176326.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jan 11, 2021

Binary Size Report

Affected SDKs

  • firebase-ml-modeldownloader

    Type Base (3dee255) Head (fdfcac43) Diff
    aar 79.9 kB 81.9 kB +2.06 kB (+2.6%)
    apk (release) 786 kB 786 kB +748 B (+0.1%)

Test Logs

Notes

Head commit (fdfcac43) is created by Prow via merging commits: 3dee255 3176326.

@google-oss-bot
Copy link
Contributor

The public api surface has changed for the subproject firebase-ml-modeldownloader:
error: Method com.google.firebase.ml.modeldownloader.FirebaseModelDownloader.getModel no longer throws exception java.lang.Exception [ChangedThrows]

Please update the api.txt files for the subprojects being affected by this change by running ./gradlew ${subproject}:generateApiTxtFile. Also perform a major/minor bump accordingly.

// start download in back ground, return local model
getCustomModelTask(modelName, conditions, localModel.getModelHash());
return Tasks.forResult(localModel);
// start upload in background, if newer model exists
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it upload?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no download - I fixed this in a bunch of places, where I was using it incorrectly. Thanks for noticing.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jan 12, 2021

@annzimmer: The following test failed, say /retest to rerun them all:

Test name Commit Details Rerun command
copyright-check 3176326 link /test copyright-check

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@annzimmer annzimmer merged commit 8fcd37c into master Jan 12, 2021
@annzimmer annzimmer deleted the clearCache branch January 12, 2021 18:06
andreaowu pushed a commit that referenced this pull request Jan 29, 2021
* Clean up/remove failed or completed download attempts.
Removes old download attempts from the download managers and resets the models in shared preferences.

* fix guarded by issues

* minor updates

* Code to handle weird states caused by cache clearing.

* fix merge error

* remove thrown exception - pass back as task exception.

* remove thrown exception - pass back as task exception.

* fixing incorrect upload word usage.
@firebase firebase locked and limited conversation to collaborators Feb 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants