Comments (3)
The script only works on listings.csv, whereas the plugin also needs to import listings-live.csv
The script doesn't check to make sure that the listings it's checking to import is newer than the listing it would replace, it just inserts or replaces everything. If the data in the DB is newer than the data from the import file, it should be skipping that item, and it isn't.
What should be done, and hasn't mainly due to lack of time, is to not perform each transaction individually, which is what is currently happening, but to push all the transactions into a list for an executemany call.
The biggest slowdown with the current method is that each transaction is done individually, so the DB gets opened, written, closed, open, written, closed, for every entry in the import file.
It's the constant opening and closing that is the biggest slowdown.
I originally wrote the plugin with literally 0 knowledge of SQLite3 beyond what I was Googling to get the thing to work, I just haven't gone back to redo the importing in a less stupid way.
TBH, every one of the import methods in the plugin really needs to be improved in the same manner.
from trade-dangerous.
Over my head, but there are some examples of executemany in cache.py if that helps.
from trade-dangerous.
This should probably be closed now due to 48686db, no?
from trade-dangerous.
Related Issues (20)
- Fix github actions to not try to publish when none on the main branch HOT 1
- This looks like an FC but --FC N HOT 9
- Add --age functionality to 'trade buy' HOT 1
- Relocate importable files into their own folder HOT 1
- Executing a trade run after a database update gives an error. In my TD\data folder the Tradedangerous.prices file is not showing HOT 6
- Fix the need to generate prices - pretty please with sugar on top. HOT 16
- When I run the command "trade.py import -P eddblink -O all", I get error messages ! HOT 9
- A General List of Potential Issues
- Fix warnings in deploy HOT 4
- Server hosting upgrade HOT 53
- Locale changes break on systems with certain locales, in general unnecessary
- Looks like `--progress` no longer works HOT 1
- Function/command "Cmdr Profile" generates an error HOT 4
- The search for specific goods generates an error ! Function "buy" HOT 10
- Again a problem with the command/function "Cmdr Profile" HOT 5
- The Culling: What's deprecated/discontinued?
- Enable Discussions tab in Github?
- PadSize argument parser is explicitly only allowing one character HOT 3
- Duplicate Rescue Ship "HR 1183/Arc's Faith" HOT 1
- The server seems to be no longer updating. HOT 11
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 trade-dangerous.