factorio-learning-environment icon indicating copy to clipboard operation
factorio-learning-environment copied to clipboard

Automatically generate an SQLite DB by default

Open JackHopkins opened this issue 7 months ago • 5 comments

Currently, we require a Postgres database set up.

This is a pain for new users.

Instead, we should just create an SQLite DB to use by default, instead of relying on the user setting up a service.

JackHopkins avatar May 28 '25 16:05 JackHopkins

Is anyone working on this? if not i can raise a pr, want to start contributing to this project !

hrshtt avatar May 29 '25 13:05 hrshtt

Nobody has picked this up so far - any help would be great!

JackHopkins avatar May 29 '25 13:05 JackHopkins

I see there's already a partial SQLiteDBClient implementation in db_client.py (about 6 methods need SQLite conversion). I'll complete this implementation to make SQLite fully functional, then we can decide on defaults/migration strategy once it's working?

hrshtt avatar May 29 '25 15:05 hrshtt

There is already a sqllite implementation and support, it's included in the readme as well. We might just need to make it the default db and create a script for setting it up more easily

MartBakler avatar May 29 '25 16:05 MartBakler

okay I see it, my bad! I hadnt fully gone through the readme and was just going off based on the code itself. Let me run everything end to end, tho i did come across an issue which ive raised #214

hrshtt avatar May 29 '25 18:05 hrshtt

closing this issue as @hrshtt's pr has been merged

kiankyars avatar Jun 19 '25 11:06 kiankyars