Checklist
App version
8.3.0-beta3
Where did you get the app from
Other
Android version
15
Device model
No response
First occurred
No response
Steps to reproduce
No response
Example post, link, markdown...
|
return new ConnectionPool(0, 1, TimeUnit.NANOSECONDS); |
Saw the following recent change in Continuum and tried the change in it to a Colab build of Infinity.
https://github.com/cygnusx-1-org/continuum/blob/903c07209dee142fa74de3e4946d59245c0d7938/app/src/main/java/ml/docilealligator/infinityforreddit/NetworkModule.java#L98
Did not use the default, but (10, 2, TimeUnit.MINUTES) instead of current value. Should it also be @Singleton?
Not sure if it had much improvement in overall battery usage, but from crude checks on the More Info in Battery Usage, the average CPU & Others sub-category usage for the app went down from 20 to 10%.
A search also showed that this was used in other places too:
https://github.com/search?q=repo%3ADocile-Alligator%2FInfinity-For-Reddit%20TimeUnit.NANOSECONDS&type=code
|
.connectionPool(new ConnectionPool(0, 1, TimeUnit.NANOSECONDS)) |
|
.connectionPool(new ConnectionPool(0, 1, TimeUnit.NANOSECONDS)) |
|
.connectionPool(new ConnectionPool(0, 1, TimeUnit.NANOSECONDS)) |
Not sure how useful the change would be in these 3.
Would this be acceptable as a bug report? Or would the post in discussions be enough?
Expected behaviour
No response
Current behaviour
No response
Logs
No response
Checklist
App version
8.3.0-beta3
Where did you get the app from
Other
Android version
15
Device model
No response
First occurred
No response
Steps to reproduce
No response
Example post, link, markdown...
Infinity-For-Reddit/app/src/main/java/ml/docilealligator/infinityforreddit/NetworkModule.java
Line 92 in 6b6f7bb
Saw the following recent change in Continuum and tried the change in it to a Colab build of Infinity.
https://github.com/cygnusx-1-org/continuum/blob/903c07209dee142fa74de3e4946d59245c0d7938/app/src/main/java/ml/docilealligator/infinityforreddit/NetworkModule.java#L98
Did not use the default, but
(10, 2, TimeUnit.MINUTES)instead of current value. Should it also be@Singleton?Not sure if it had much improvement in overall battery usage, but from crude checks on the More Info in Battery Usage, the average CPU & Others sub-category usage for the app went down from 20 to 10%.
A search also showed that this was used in other places too:
https://github.com/search?q=repo%3ADocile-Alligator%2FInfinity-For-Reddit%20TimeUnit.NANOSECONDS&type=code
Infinity-For-Reddit/app/src/main/java/ml/docilealligator/infinityforreddit/activities/SubscribedThingListingActivity.java
Line 202 in 6b6f7bb
Infinity-For-Reddit/app/src/main/java/ml/docilealligator/infinityforreddit/services/SubmitPostService.java
Line 231 in 6b6f7bb
Infinity-For-Reddit/app/src/main/java/ml/docilealligator/infinityforreddit/activities/CommentActivity.java
Line 526 in 6b6f7bb
Not sure how useful the change would be in these 3.
Would this be acceptable as a bug report? Or would the post in discussions be enough?
Expected behaviour
No response
Current behaviour
No response
Logs
No response