Comments (15)
Works fine for me. You're either having a very low bandwidth, or packet losses, or you're nailing github until the rate limit kicks in.
That said, I did think about some way to show when the rate limit is blocking the query, but had no nice idea how.
Also ties in with my investigations into GraphQL - which has vastly different rate limits, vastly better bandwidth usage, easier access to correct rate limit info, but also vastly nastier requirements - as in mandatory OAuth.
from unciv.
Problem is, it worked just fine in versions prior to 4.9.19. - filtering menu loads in seconds, whether via phone or PC, using phone packet data or laptop wifi. I don't think it's the rate limit that's causing the issue.
from unciv.
4.10.3 patch 1 on my phone. Loads just fine for me. Just takes some amount of seconds to load. So can't reproduce here
from unciv.
Hmm, I haven't updated it to patch 1, let's see if that fixes things
from unciv.
Is that loading symbol turning? Du you have the "continuous rendering" option on? Should be visual only, but maybe it influences some events/actions processing.. Question valid for Phone + as you say "Problem also exists on PC version". What else do your boxes have in common that may be different from ours?
from unciv.
Just tested again, this time on Android hardware. With and without "continuous rendering", landscape and portrait. The query always finishes, filtering and sorting get enabled just fine. With such repetition of course I hit the rate limit, so the last test needed 40 seconds to finish.
It even logged Unciv RateLimit com.unciv.app D [threadpool-daemon-0] GitHub API Limit reported via http (60) not equal assumed value (10)
- which may be a hint. I don't remember clearly, but I think the ratelimit handler operates on assumptions taken from the documentation at the time. If github changed the rules, the code may not predict properly when the ratelimit will hit, and take longer than maybe needed. But then - that message says the limit is more generous than assumed... But the docs still say 10 requests per minute, which are the values the code uses...
Are you able to do the query normally from a browser?
https://api.github.com/search/repositories?q=%20topic:unciv-mod%20fork:true&sort:stars&per_page=100&page=1
would be the first page. Should return pretty quickly. But as you can see, there's 641 mods therefore 7 pages, meaning if you open mod manager twice and let the queries finish within a minute you're already hitting the documented rate limit.
I still say it's your network stack. Probably a shared external IP so other people contribute to the same rate limit.
from unciv.
Is that loading symbol turning?
Uh... Is it supposed to turn? I don't think I've seen it do so
from unciv.
supposed to turn
Of course it's animated - unless you turned animation off - with this setting.
from unciv.
Looks like it's off by default
from unciv.
Looks like it's off by default
Looks like you're right! I never noticed... The default has been turned off with ... #1996, looong ago - but why?
from unciv.
Is that loading symbol turning? Du you have the "continuous rendering" option on? Should be visual only, but maybe it influences some events/actions processing.. Question valid for Phone + as you say "Problem also exists on PC version". What else do your boxes have in common that may be different from ours?
Loading symbol is not turning on both machines, and it's definitely not a bandwidth throttle since I can play War Thunder and other bandwidth-heavy games on my PC just fine without lags. So that rules it out on PC.
Continuous rendering I haven't touched since installing the game long ago, it's off.
from unciv.
Just tested again, this time on Android hardware. With and without "continuous rendering", landscape and portrait. The query always finishes, filtering and sorting get enabled just fine. With such repetition of course I hit the rate limit, so the last test needed 40 seconds to finish.
It even logged
Unciv RateLimit com.unciv.app D [threadpool-daemon-0] GitHub API Limit reported via http (60) not equal assumed value (10)
- which may be a hint. I don't remember clearly, but I think the ratelimit handler operates on assumptions taken from the documentation at the time. If github changed the rules, the code may not predict properly when the ratelimit will hit, and take longer than maybe needed. But then - that message says the limit is more generous than assumed... But the docs still say 10 requests per minute, which are the values the code uses...Are you able to do the query normally from a browser? https://api.github.com/search/repositories?q=%20topic:unciv-mod%20fork:true&sort:stars&per_page=100&page=1 would be the first page. Should return pretty quickly. But as you can see, there's 641 mods therefore 7 pages, meaning if you open mod manager twice and let the queries finish within a minute you're already hitting the documented rate limit.
I still say it's your network stack. Probably a shared external IP so other people contribute to the same rate limit.
Yeah I can open the page just fine using my phone just now, chrome browser. I'll see if turning continuous rendering on fixes this thing.
from unciv.
Well, the arrow does rotate, but I've waited more than 5 minutes and the filter dropdown didn't appear at all.
The bottom-most mod on the list (sorted by stars iirc) was the Arknights Firewatch mod, as of writing this.
from unciv.
So that rules it out
Not necessarily. Server, provider routing, external firewalls can still make one server slow while another is fast. But let's assume transport to be fine.
turning continuous rendering
Well, we established the animation is purely cosmetic as it should be. If it were frozen while the option is on, that would have been another matter.
So, if we can't reproduce, how should we be able to investigate? The one who can repro can analyze...
Does the console and/or logcat say anything interesting maybe a minute or two into mod manager?
from unciv.
This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 15 days.
from unciv.
Related Issues (20)
- éČăăȘă
- RFC: Telling stories - Ideas and related observations HOT 3
- Units without vision cannot be seen when moved into the fog HOT 2
- Game crash on starting new game after deleting mod HOT 20
- Feature request: Put resource point value for worker tile improvement priority in the settings. HOT 1
- Countable scopes approach decision HOT 6
- "Open terrain" goes wrong HOT 2
- Workers don't build roads. HOT 3
- Unable to decide what to do with a city HOT 9
- I can't connect to multiplayer HOT 3
- Simultaneous determination a tile HOT 3
- Declaration of war bug HOT 8
- Translating site bug HOT 4
- Air units no longer show attack and move range HOT 10
- Feature INQUIRY: AI's predisposition on SAM's HOT 1
- Check mod versions when online-play HOT 4
- Add "Hide this building in city UI"
- Escorted movement crash HOT 2
- City-state condemnation mechanics
- RFC: Fly-by observations (and mini-bugs) concerning Espionage HOT 8
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
đ Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. đđđ
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google â€ïž Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from unciv.