Opened 13 years ago
Closed 12 years ago
#2393 closed enhancement (fixed)
Maze should save state, support diffculty setting
Reported by: | greenfeld | Owned by: | RafaelOrtiz |
---|---|---|---|
Priority: | High | Milestone: | Unspecified |
Component: | Maze | Version: | 0.84.x |
Severity: | Unspecified | Keywords: | easy-hack |
Cc: | Distribution/OS: | OLPC | |
Bug Status: | Unconfirmed |
Description
The Maze activity does not seem to store a Maze's state journal entries, even though this activity shows the "Keep" menu option. Right now when you click on a Maze activity entry in the journal, you start off again with a random easy level.
While it may be hard to save & restart a Maze level in progress, at the very least I would to see if we could support restoring the basic layout of said level, as well as adding the ability to select a relative difficulty, forcing the creation of a new maze. It also seems to be impossible to currently rename a Maze Activity session while it is running without quitting it first.
That way younger users can just run easier mazes, and older users can skip running through lots of simple levels to get to the ones which take more than 10 seconds to solve.
I looked at activities.sugarlabs.org, and it looks like OLPC is using the latest version of this Activity, which I also downloaded to a 0.90 system to check).
Change History (11)
comment:1 Changed 12 years ago by sascha_silbe
- Component changed from untriaged to Maze
- Owner set to RafaelOrtiz
comment:2 Changed 12 years ago by garycmartin
comment:3 Changed 12 years ago by RafaelOrtiz
- Priority changed from Unspecified by Maintainer to High
comment:4 Changed 12 years ago by godiard
- Keywords easy-hack added
comment:5 follow-up: ↓ 6 Changed 12 years ago by humitos
I was trying to solve this ticket but we need this event olpcgames.FILE_READ_REQUEST to resume (integrate with) from the Journal and that event is available on olpcgames 1.6+
Maybe we should update the olpcgames library for Maze (it's included inside the game)
comment:6 in reply to: ↑ 5 Changed 12 years ago by manuq
Replying to humitos:
I was trying to solve this ticket but we need this event olpcgames.FILE_READ_REQUEST to resume (integrate with) from the Journal and that event is available on olpcgames 1.6+
Maybe we should update the olpcgames library for Maze (it's included inside the game)
Sure, if that is needed, provided that the olpcgames update allows the read/write to the Journal, I think is the thing to do, The activity should be tested and the behaviour should not change with the olpcgames update.
comment:7 Changed 12 years ago by humitos
Yes, I think that is the best: update olpcgames in the Maze Activity,
In fact, I've just done that and I fixed this ticket. I'll prepare a PATCH later. Actually, I'm saving the state of the Activity per instance, I mean, every time that you run a new Maze (holding Alt key) you will start a New Maze Game but if you start Maze from the Journal or just clicking in the Home Maze's Icon you will start it from the last state.
comment:8 Changed 12 years ago by humitos
Here is the PATCH that I sent:
comment:9 Changed 12 years ago by AlanJAS
OLPCgames s no longer maintained... Then: Sugargame:
Wiki:
http://wiki.sugarlabs.org/go/Development_Team/Sugargame
comment:10 Changed 12 years ago by RafaelOrtiz
I haven't found substantial differences between sugargames and olpcgames, also Maze use olpcgames and we have been making changes to it, you can say that is unofficially maintained.
I haven't had the chance to publish a tarball or sources though.
comment:11 Changed 12 years ago by RafaelOrtiz
- Resolution set to fixed
- Status changed from new to closed
This patch solves the issue:
http://git.sugarlabs.org/maze/mainline/commit/335ad73456ba3ec8f56811abddcaca4650199db1
Having Maze store some game state would be great, even just the level reached would be a good start. This comes up in conversation from time to time, but most recently Tabitha from the NZ testing team raised it on IRC and in her report to the testing ML:
http://lists.laptop.org/pipermail/testing/2011-October/002423.html