-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Approaching Completion #7
Comments
Seems ok. Will there be a separate flow for initialising a database from YAML? Does the database YAML go in a separate file or in the metafile? |
To load the database from yaml we have a seperate tool. Essentially, when the game runs, we must guarantee that there is a working database. So the dbgen.py file you wrote which is in test should work, and possibly be moved to the root directory. |
What do I know, I'm just the metafile man! |
Very well, I will add those steps to the World.init function. |
I already realised I missed out removal of puppets from the world. Not a good start. |
Okay, I've written it all in, I need Matt to go into World.init and define the cursor value, which is an iterator that will return the ID of every object in the database. If need be, change appearances of "i" to "i[0]". |
Update: As I understand, Matt's doing some shuffling around somewhere in the world retrieval routine and elsewhere to load the whole database serially without need for the whole FOR id IN database GET id, values AND DO ... END cycle, and it'll be replaced with a FOR id, values IN database DO ... END cycle. |
Now that the structure has taken shape and we are now ironing out the kinks, we need to get the world into a workable shape. We need to work out the final part of World.init that will be written to successfully initiate the world, and aquire an initial database to work with. Off the top of my head I can think of the following things, but everyone needs to confirm that this is correct or whether there needs to be any ammendments to account for anything else that needs loading.
The text was updated successfully, but these errors were encountered: