Pyfa
Pyfa copied to clipboard
add prices to market items
#2071 Adds a price column to items in the market view
Note: I am trying to be a good steward of the code and add some tests. However, I am having some trouble with imports. There are no tests for other columns, like price, so it is tough to mimic an existing example.
I think it is a pathing issue, but it is able to import price.py just fine. Any ideas?
ImportError while importing test module '/home/sunder/Pyfa/tests/test_modules/test_gui/test_builtinViewColumnPrice.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/usr/lib/python3.6/importlib/init.py:126: in import_module
return _bootstrap._gcd_import(name[level:], package, level)
tests/test_modules/test_gui/test_builtinViewColumnPrice.py:8: in
eos/db/saveddata/cargo.py:25: in
Hey, regarding your PR - sorry for not replying earlier. Adding price column is easy, and I considered it in the past. The reason why it wasn't added is not to overload price sources with updates. Any kind of search or active browsing will generate requests for thousands of items per minute. While caching policy on client side is decent, I think there has to be more stuff in place to add the price column to default distribution.
- Batch request of multiple prices for all shown items (except for cached entries). This is not trivial thing to do, but it would benefit fitting panel as well.
- Some kind of extra delay after showing items before price fetching requests are sent, like 1s at least - to avoid requesting stuff during searches, but request them only for final results
And those complications are the reasons why it is not in pyfa yet. It is cool end-user feature, but avoiding heavy distributed load on price services is a must in this case imo.
I might have some time into looking how to implement it myself, but no promises. Until that, I am leaving it hanging in here.