Episode I Discussion issue.

Bertram25 commented 11 years ago

A place to talk in github for the next part of the episode I. :)

Episode I Release TODO List:

[x] Do not forget to add the dead ally target type for resurrect-based spells.
[x] Add support for scripted animations in battle animation scripts (other than on the battle actor but also on them). #222
[x] Add "soft-edge" character movement support #208
[x] Use Lilou's new horse
[x] Add missing shoulders to existing portraits if possible
[x] Fix menu list refresh in certain cases #248
[x] Finish the Joystick D-Pad support #257
[x] Add Episode ending complete credits. #268
[x] Fix broken parallax effects in the Mt. Elbrus exterior and the Layna Forest caves map scripts.
[x] Spread back status effects to the map mode. I could also add traps in the dungeon poisoning the characters, for instance. #220
[x] Fix first item being listed twice in the trade mode. #283
[x] Add support to change the welcome text in the shop mode through a ShopEvent. #284
[x] Add support to disable the ability to sell in the shop mode. #285
[x] Fix item list handling with one item in battles. #282
[x] Decrease the characters agility at battle start when they are low on Stamina on map. #289
[x] Remove the art/ folder from this repo, and put art/ in its own. #287
[x] Improve a bit the stun particle effect.
[x] Fix Skeletons offset due to Bronann's attack looking weird on them.
[x] Make the Battle item/skill info window speak about seconds, not milliseconds.
[x] Add support to show canceled quests (like when you didn't finish the chicken quest on time.) #288
[x] Fix enemy behaviour #307
[x] Fix last cppcheck warnings #306
[x] Add memory support for shops #298 (Main part done, the next one after release.)
[x] Check Credits completeness
[x] Translation & migration to transifex.
[x] Fix Kalya's skill 'Rain of Arrows' #325
[x] Add second dungeon and boss. #286
[x] Do no forget to add ending credits music. (Used the sad theme)
[x] Episode I RC1 Release! :D
[x] Kill all the issues found.
[x] Episode I Release! :D

Bertram25 commented 11 years ago

@codergreen, @IkarusDowned, @Socapex :

Hi All :)

For future ease of use, map data reusability, get rid of the max script length cap present due to the fact that the scripting data is stored in a string when using the current editor, with the evil secret aim at one day using Tiled as the VT map editor, and with the current urging need to reuse the village center map data without complicating the script even more, I'm going now to split the scripting data from the map data. Might take a few days, but it's feasible. ;)

Regards,

p-groarke commented 11 years ago

Thanks for keeping me posted. I've been super busy with real life (exams and projects etc. think assembly [sigh]) so I haven't had the time to get back to porting. It might be a little while, but at least people have a build that seems to work nicely for now :)

Keep in touch,

Bertram25 commented 11 years ago

@Socapex Hi :)

Thanks for keeping me posted. I've been super busy with real life (exams and projects etc. think assembly [sigh]) so I haven't had the time to get back to porting. It might be a little while, but at least people have a build that seems to work nicely for now :)> Thanks for keeping me posted.

Wow, good luck for your exams and all. Don't worry about the port, as you rightfully said, people have something to play with and it's the main thing. :) We'll talk and fix the boring technical details when you'll have the time.

@ all I'll try to keep you informed about my own progress path in this very thread. Feel free to show whatever you think is necessary or technical here. The issue is here for that :)

Best regards,

IkarusDowned commented 11 years ago

sounds good Bertram, I don't think it will interfere with anything i've pushed so far. One thing that I forgot to ask: The mini-map generation code works relatively quickly on my machine, but admittadly I have a bit of a gaming rig. What are people's feelings as to the current generation time? IE, when you change map / scenes, the minimap / collisionmap gets generated. Is this taking too long for people? Or you guys don't really seem to notice?

Bertram25 commented 11 years ago

The mini-map generation code works relatively quickly on my machine, but admittadly I have a bit of a gaming rig. What are people's feelings as to the current generation time? IE, when you change map / scenes, the minimap / collisionmap gets generated. Is this taking too long for people? Or you guys don't really seem to notice?

I, for one, didn't notice any difference with/without so far, and tried on two laptop-for-office-work like machines. I'm still in the process of seting up an older (and much slower) one atm, but I couldn't get around finishing it yet, though. I'll tell you more when I've been able to deal with that. :)

Btw, I need to take a decision about it since some times now, but I think I don't have an actual choice if I want to go forward tiled support one day:

I do think I'll drop map context support and adapt Bronann's room map consequently. Does someone has a strong and valid point against that?

Bertram25 commented 11 years ago

Hi,

I'm going to change the namespace prefix from hoa to vt now that the project is coming to a decent maturity level. I also removed the defs.h and ask everyone to do declarations within the file needing it from now on.

Don't worry, I didn't plan any other structural changes for the current release ;>

Bertram25 commented 11 years ago

Own todo-list, to not forget about it: (for items not in other issues)

Code:

[x] Remove defs.h
[x] Drop context support
[x] Show the key of key items in the treasure window.
[x] hoa -> vt
[x] Add my own name to source files. (meh!)
Remove the use namespace wherever possible:
[x] map data files.
[x] tilesets are left to do.
[x] Add a mother table to particle effects files, to avoid global variables there.
[x] See what I can import from Socapex's work on my humble own ;)
[x] Remove normal tile display (only smooth shall remain)
[x] Start to test and do advanced Battle Events

Story/Content:

[x] Add the missing boss graphics (map and battle)
[x] Add ambient sound for campfires.
[x] Add missing parents bed missing graphic (before forgetting about this!)
[x] Add the missing Bronann running after victory battle animation (with sheathed sword) The current animation looks weird as he puts his sword out abruptly.
[x] Add missing well in the village center
[x] Add dialogue and fight in the village
[x] Add flee from the village scenes

Won't be done for Ep1 as some other issues need to be sorted out with Tiled upstream:

Readd the tileset images in the map data files, with a path relative to the map data file. This will comply with Tiled specs.

shirishag75 commented 11 years ago

Ah, finally know how much longer I have to wait, thanx @Bertram25 :)

Bertram25 commented 11 years ago

@knitter Thanks for the portuguese translation! :D

Knitter commented 11 years ago

Just have to continue updating it :)

codergreen commented 11 years ago

I'm back and ready to code! I'm willing to start working on the crafting mode unless you can think of something else you need me to do.

Bertram25 commented 11 years ago

Hi @codergreen and welcome back :D

With the trade mode in place, I realize now the crafting one isn't a high priority. That doesn't mean you can't start on it if you really want it done ;)

Here is the list of things to do that would be needed or cool to finally have, from the more or less easiest to the hardest, IMHO:

cheat-sheet for game. (that one is for Shirish) #129
Skills obtained through equipment. #124
Add support to switch the GUI theme from the option menu #156
Add the option menu also in the pause mode (when you press escape) that will permit to change the video/sound, ... from within a game. When in the game, the language selection should be disabled. #155
Add support for text and image from scripts in the help window in order to create a in game manual. linked to #7 and #107
application of non-passive effects from equipment #141
Use a QT renderer in the editor instead of the video manager #157

Your choice! :D

Best regards,

Bertram25 commented 11 years ago

Ah, there is also what's listed in the TODO list in case of interest: http://forum.freegamedev.net/viewtopic.php?f=76&t=4006&p=45799

codergreen commented 11 years ago

@Bertram25 I might start working on: Add support for text and image from scripts in the help window in order to create a in game manual. I found mode_help_window.{cpp, h} and mode_bindings.{cpp, h}. Is that where I start?

Also I found an off-by-one error in the trade mode. In line 551 of shop_trade.cpp, it's: if(obj->GetTradeCount() * obj->GetObject()->GetTradeConditions()[i].second >= GlobalManager->HowManyObjectsInInventory(obj->GetObject()->GetTradeConditions()[i].first)) instead of: if((obj->GetTradeCount()+1) * obj->GetObject()->GetTradeConditions()[i].second > GlobalManager->HowManyObjectsInInventory(obj->GetObject()->GetTradeConditions()[i].first)) This error may exist in other locations too, so I'll look into that also tomorrow.

Bertram25 commented 11 years ago

Hi @codergreen :)

Also I found an off-by-one error in the trade mode. In line 551 of shop_trade.cpp, it's:

Nice catch! Btw, I added an offset here some times ago linked to a crash when trading a key item, just in case: https://github.com/Bertram25/ValyriaTear/blob/master/src/modes/shop/shop_trade.cpp#L184

Add support for text and image from scripts in the help window in order to create a in game manual. I found mode_help_window.{cpp, h} and mode_bindings.{cpp, h}. Is that where I start?

I've added this, if it can help: https://github.com/Bertram25/ValyriaTear/issues/7#issuecomment-17429851 Feel free to comment or to propose something else as you see fit. :)

Bertram25 commented 11 years ago

I'll start a new list in this comment:

Code:

[x] Add skill obtained through equipment #124
[x] Merge Status and elemental effects.
[x] Add support for equipment (passive aka neverending) status effects in battle #141
[x] Add support for elemental based effects.
[x] Optionally add support for poison/healing and other effects applied in battles.

Bertram25 commented 11 years ago

I've started to refactor the skills id for a global game scale and I ended up preparing all the skills skeleton that will be available in game, which is a good thing when you come to think about it.

Let's talk more of this all when Episode II starts, but side-quests will indeed be needed to catch 'spirits' and get certain specials. ;)

Bertram25 commented 11 years ago

@Gallaecio Thanks a lot for the translation and the english fixes. :) Hope you'll stay around for the next updates and that you enjoyed playing the game!

Gallaecio commented 11 years ago

I will stay to maintain the translation as the game evolves, that’s for sure.

While I’ve not played many games of this genre (the closest I’ve played must have been “Golden Sun”), I have to say I really enjoyed it, and it was really sad to get to the end. The game as a whole looks quite polished already, and the story is engaging.

Bertram25 commented 11 years ago

@Gallaecio Thanks a lot for the nice comment :) And I'm quite happy you'll stay around. Just in case, we have a forums at free game dev where there is a translation topic. NaN and BioHazardX are maintaining the German and Italian translations by dropping po files from time to time there.

Bertram25 commented 11 years ago

Due to some own recent code stupidity, the battles are crashing when quitting them after the addition of equipment skill support. I'll fix that asap, but in the meanwhile, I'd recommend you'll stay away from these patches.

Bertram25 commented 11 years ago

Fixed in https://github.com/Bertram25/ValyriaTear/commit/d119ad468abee874fe8e8e7fa408e51af985aefe

Enjoy :)

Bertram25 commented 11 years ago

Thanks a lot to @logzero and @authenticate for the fixes found and/or done!

You rock guys!

Bertram25 commented 11 years ago

While I'm thinking about it, @codergreen I've fixed the off-by-one errors you found in: https://github.com/Bertram25/ValyriaTear/commit/bdca771721fd8319b82bf7a54c938d96704ce589 Have a nice day everyone :)

Bertram25 commented 11 years ago

Hi all,

I've updated the minimalistic hacking conventions here: https://github.com/Bertram25/ValyriaTear/wiki/Hacking-conventions

If you see something that can be improved, feel free to tell. :)

logzero commented 11 years ago

One thing I'd love to see in the Hacking-conventions is a one header/source file per class rule. It should reduce compilation times and, more important, make browsing code much easier. ;)

Bertram25 commented 11 years ago

@logzero I must say you're right and that I didn't do it due to pure laziness. ;) I've added it.

Bertram25 commented 11 years ago

Third update list, added there to not forget stuff.

0.6 Release:

Code:

[x] Readd basic monster movement when not chasing the character and not handling waypoints. The engine will then be able to handle both simple and path based movements for enemies and have a nicer? hybrid solution.
[x] Fix heroes winning animation sometimes going back to idle animation when winning a battle.
[x] Add support for the new triggers type needed for the maps.

Story / Content:

[x] Add pursuit maps (5/5 done)
[x] Add the missing forest minimaps.
[x] Add character XP and stats until level 30 and cap at that level.
[x] Rebalance the Shroom and Dark Soldier enemies.
[x] Add missing jump end sound when Mt Elbrus cave 1.
[x] Find rolling and bump sounds for stones.
[x] Remove the silence at the end of the normal battle music.
[x] Update the credits!
[x] Do some balancing and few bug fixing.
[x] Add back simple enemy movement because of performance issue. #224
[x] Add support for custom enemy AI scripts for deciding the next action. #221
[x] Add missing crystal voice acts. (almost done! arg!)
[x] 0.6.0 RC1 Release!
[x] 0.6.0 Release!

Bertram25 commented 11 years ago

Hi all, I've basically finished the story content that will be used for the 0.6 release. I still have to fix/add a few things and more importantly add back simple map enemy movement, but hopefully we're getting there. :)

Gallaecio commented 11 years ago

Thanks for the update! Eagerly waiting here :)

Bertram25 commented 11 years ago

Thanks for the update! Eagerly waiting here :)

Thanks to you for all the translation support! When you say you're waiting, it means you don't use the master git source code to play?

Gallaecio commented 11 years ago

It means I did not realize it had been pushed already :D I’ve already played the new stuff now. Quite interesting the cave puzzles, by the way.

Bertram25 commented 11 years ago

Great! :) I still have to balance the mini boss battle but it's getting there. Good thing you enjoyed the puzzles as I used those maps to prototype the new objects so we can use them more extensively later ;)

shirishag75 commented 11 years ago

@Gallaecio - lemme know if you know the answer to the two balls in two holes puzzle. I have been racking my brain to no avail. Answer it in http://forum.freegamedev.net/viewtopic.php?f=76&t=4844 (make sure to hide it ) so people who are desperate (like me) will unhide it, while rest of the gamers will just go on.

Bertram25 commented 11 years ago

@Socapex Hi :) Now the release is out, could you upgrade the OSX dmgs? Best regards,

Bertram25 commented 10 years ago

Moving the latest TODO list in the description.

shirishag75 commented 10 years ago

looking forward to updates on this.

Bertram25 commented 10 years ago

@Socapex : Hi ! :) Still no news about a possible OSX package?

@anholt Hi also! Have you got any news about the debian package state? I thought all the work was done? Is there any help needed there?

Best regards,

p-groarke commented 10 years ago

Oh of course I could release a new package! Do you have a tag you want me to build against?

On 2013-11-25, at 7:15 AM, Yohann Ferreira notifications@github.com wrote:

@Socapex : Hi ! :) Still no news about a possible OSX package?

@anholt Hi also! Have you got any news about the debian package state? I thought all the work was done? Is there any help needed there?

Best regards,

— Reply to this email directly or view it on GitHub.

Bertram25 commented 10 years ago

@Socapex Cool! Thanks a lot! You'll find everything you need here: https://github.com/Bertram25/ValyriaTear/releases/tag/0.6.0

p-groarke commented 10 years ago

@Bertram25 So I have time now :) Settled in this new semester. Should I build RC1 or plain 0.6?

p-groarke commented 10 years ago

New OSX versions available here (rc1):

Lion and higher: https://www.dropbox.com/s/0ja7d0tdbfr3xay/ValyriaTear-lion-0.6.0-rc1.dmg Snow Leopard: https://www.dropbox.com/s/ba8rqmo6v4cswhe/ValyriaTear-snowleopard-0.6.0-rc1.dmg

Bertram25 commented 10 years ago

Hi @Socapex :D

Thanks a lot for your help on it!! Now, unfortunately there have been fixes between the 0.6 RC1 and the 0.6 final release. Could you create packages for the 0.6 tag? https://github.com/Bertram25/ValyriaTear/releases/tag/0.6.0

Best regards,

Bertram25 commented 10 years ago

RC1 incoming!!

Gallaecio commented 10 years ago

:D

Bertram25 commented 10 years ago

RC1 online! Enjoy! \^o^/

Bertram25 commented 10 years ago

Final release incoming this week. ;)

akien-mga commented 10 years ago

A real Coming Soon™ \o/

Bertram25 commented 10 years ago

Episode I released \o/ \o/ \o/

akien-mga commented 10 years ago

Awesome! It's in Mageia Cauldron: http://svnweb.mageia.org/packages/cauldron/valyriatear/current/SPECS/valyriatear.spec?view=markup&pathrev=672174

I want to provide a backport for Mageia 4 too, I'll keep you posted :-)

ValyriaTear / ValyriaTear

Episode I Discussion issue. #126