--- Log opened Thu May 16 00:00:31 2013
20130516 00:03:42-!- jetrel2 [~jetrel2@64.208.23.76] has quit [Ping timeout: 264 seconds]
20130516 00:05:22-!- flix [~flix@178.77.129.78] has quit [Quit: Leaving.]
20130516 00:06:17-!- shadowm_desktop [ignacio@wesnoth/developer/shadowmaster] has quit [Ping timeout: 246 seconds]
20130516 00:08:14-!- shadowm_desktop [ignacio@wesnoth/developer/shadowmaster] has joined #wesnoth-dev
20130516 00:14:55-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has quit [Read error: Connection reset by peer]
20130516 00:17:38-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has joined #wesnoth-dev
20130516 00:18:09-!- Ivanovic [~ivanovic@wesnoth/developer/ivanovic] has quit [Ping timeout: 248 seconds]
20130516 00:20:04-!- LordBob_ [~chatzilla@2a01:e34:ee82:47e0:21e:c2ff:fe01:261f] has quit [Ping timeout: 245 seconds]
20130516 00:33:37-!- vultraz [~chatzilla@wesnoth/developer/vultraz] has quit [Read error: Connection reset by peer]
20130516 00:33:47-!- ancestral [~ancestral@mobile-198-228-235-071.mycingular.net] has joined #wesnoth-dev
20130516 00:34:02-!- vultraz [~chatzilla@wesnoth/developer/vultraz] has joined #wesnoth-dev
20130516 00:36:17-!- boucman [~rosen@wesnoth/developer/boucman] has quit [Remote host closed the connection]
20130516 00:37:01-!- Ivanovic [~ivanovic@x2f41160.dyn.telefonica.de] has joined #wesnoth-dev
20130516 00:46:15< irker867> wesnoth: fendrin wesnoth-old:master 4e6826 / src/hotkeys.cpp: Update the tooltips of the unit and village tool. http://git.io/ZdKnpA
20130516 00:46:18-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has quit [Ping timeout: 264 seconds]
20130516 00:54:16-!- TheMonster [~TheMonste@41.69.197.0] has joined #wesnoth-dev
20130516 00:59:06-!- mjs-de [~mjs-de@g224176216.adsl.alicedsl.de] has quit [Remote host closed the connection]
20130516 01:00:48-!- ancestral [~ancestral@mobile-198-228-235-071.mycingular.net] has quit [Quit: Smell ya later!]
20130516 01:01:37-!- Nephro [~neph@80.233.231.12] has quit [Ping timeout: 256 seconds]
20130516 01:32:12-!- H-Hour_ [~H-Hour@cpc7-sgyl35-2-0-cust428.18-2.cable.virginmedia.com] has quit [Quit: Leaving]
20130516 01:35:07-!- loonybot [~loonybot@wesnoth/bot/loonybot] has quit [Remote host closed the connection]
20130516 01:39:06< mattsc> jleldridge: are you back?
20130516 01:46:06< mattsc> jleldridge: for when you come back, your pull request looks good.  I have 2 very minor (formatting) comments and one that about the kind of data returned.
20130516 01:48:25< mattsc> 1. location_set.lua seems to use tabs, not spaces, so let's do the same for your code.
20130516 01:48:34< mattsc> 2. It's good to have the changes to location_set.lua and the HS MAI in separate commits.  Please make sure that the final version doesn't mix those.
20130516 01:51:19< mattsc> 3. Now that I see it, I don't really like that the output format of the data is different depending on whether hexes are specified by [1]/[2] or x/y.  We should make it consistent one way or the other.  I'm not 100% decided on what's best, but I'm currently favoring using what you do for x/y for everything.
20130516 01:52:01< mattsc> jleldridge: that's all, the rest looks good to me (but I've not had time to test it yet)
20130516 02:04:56-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Quit: Konversation terminated!]
20130516 02:07:18-!- mattsc [~mattsc@fw.hia.nrc.ca] has quit [Ping timeout: 264 seconds]
20130516 02:59:35-!- noy [~Noy@wesnoth/developer/noy] has joined #wesnoth-dev
20130516 03:05:22< jleldridge> mattsc: The pull request has been updated with the changes you mentioned.
20130516 03:12:25-!- TheMonster [~TheMonste@41.69.197.0] has quit [Quit: Later]
20130516 03:19:40-!- leo42 [~leantros@cpe-384193.ip.primehome.com] has quit [Ping timeout: 240 seconds]
20130516 03:38:18-!- mattsc [~mattsc@d154-20-34-165.bchsia.telus.net] has joined #wesnoth-dev
20130516 03:43:27< mattsc> jleldridge: Thanks.  I'll do a couple tests later to see if this works as I expect it for some special cases.  I'll try to do that tonight, but can't promise that yet...
20130516 03:44:23-!- rsyh93 [~Young@c-76-26-142-24.hsd1.md.comcast.net] has joined #wesnoth-dev
20130516 03:44:55< jleldridge> mattsc: ok :) I'm changing one more small formatting thing, a blank line snuck into my code
20130516 03:45:09< jleldridge> but not changing any real code
20130516 03:45:10< mattsc> I saw that, would have mentioned it later  :)
20130516 03:46:07< mattsc> TBH, what I am wondering at the moment is whether it is worth avoiding all that almost duplication of code...
20130516 03:46:50< mattsc> You can do the whole thing in (almost) half the number of lines, but it would probably lose some readability.  So I'm not sure if that should be done or not.
20130516 03:47:48< mattsc> As in, by defining an 'index' for each case, and setting value_table=true in the case there's no table content; then just assign everything at the very end in a single line.
20130516 03:48:22< mattsc> As I said, I'm not sure which way I'd prefer, so we should probably just leave it as it is.
20130516 03:49:08< jleldridge> But when you build the value table you either have to avoid "x" and "y" or 1 and 2
20130516 03:49:35< jleldridge> which you could do by defining them as variables, but that's just moving the conditional elsewhere, not really eliminating it
20130516 03:51:42< mattsc> Yeah, I know.
20130516 03:52:11-!- perennate [~wizardus@MACGREGOR-THREE-TWENTY-EIGHT.MIT.EDU] has left #wesnoth-dev []
20130516 03:52:13< mattsc> As I said, I think it's good to leave it this way.  Everything else would require additional variables or something...
20130516 03:52:58< mattsc> I'll test it sometime later, but it looks good for what I can tell.
20130516 03:53:05< jleldridge> Ok :)
20130516 03:58:08< mattsc> jleldridge: oh, the wiki needs to be changed too, of course.  So if you're bored, you could get started on that.  ;)
20130516 03:59:01< mattsc> jleldridge: what's the next thing on your list after this?
20130516 03:59:23< jleldridge> mattsc: Sure! I can do that.
20130516 04:00:13< jleldridge> mattsc: I was going to spend a bit of time working on my summer of code proposal then maybe read the logs to see what else you and anon had mentioned last night.
20130516 04:01:03< mattsc> jleldridge: ok, sounds good.  I just want to make sure that I don't accidentally work on something myself that you want to do.
20130516 04:02:07< jleldridge> mattsc: Haha, understood, let me take a look really quick.
20130516 04:03:24< vultraz_laptop> fabi: around?
20130516 04:05:23< jleldridge> mattsc: actually, you guys ended up boiling it down to just of_pairs that needed to be changed, is there still a reason to change to_pairs()?
20130516 04:07:18< fabi> vultraz_laptop: hi
20130516 04:08:06< mattsc> jleldridge: not sure.  Maybe do the same as with with of_pairs?  Come up with a way of adding other keys if value is not just 'true'?
20130516 04:08:21< mattsc> I'd have to think about whether that makes sense or not.
20130516 04:08:50< mattsc> There were other tasks though, if you're looking for something else.
20130516 04:09:38< fabi> vultraz_laptop: I saw you complaining about your own usefulness regarding the editor artwork. You should not underestimate your doings. You provided me with stubs I could work with and start the project. The original artwork of Wesnoth is completely replaced now, still the original author did important work to get the project started. 
20130516 04:09:55< vultraz_laptop> :)
20130516 04:10:11< vultraz_laptop> anyway um, I got the repo and it's working
20130516 04:10:22< vultraz_laptop> building wesnoth in xcode, but got an error
20130516 04:10:30< vultraz_laptop> src/text.cpp:84: error: 'pango_font_map_create_context' was not declared in this scope
20130516 04:10:43< mattsc> vultraz_laptop: there's an easy fix for that error.
20130516 04:11:14< vultraz_laptop> declare it in the scope?
20130516 04:11:22< jleldridge> mattsc: I would, can you remember anything specific? I could start working on the aggression/[filter] problem.
20130516 04:11:42< mattsc> It's caused by the pango libraries not being updated for Xcode yet (something Alarantalara has to do, I do not know how)
20130516 04:12:01< vultraz_laptop> :/
20130516 04:12:14< mattsc> vultraz_laptop: so what you do is put the test for pango version back in.  Hold on, I'll look up the link...
20130516 04:12:52< vultraz_laptop> has there been a maccompilestuff update recently?
20130516 04:13:08< mattsc> not that I'm aware of
20130516 04:13:46< mattsc> vultraz_laptop: undo the change to lines 89-94 here for your local copy: https://github.com/wesnoth/wesnoth-old/commit/2b939cebc7a4eea6ec6adae40c8396f836122940
20130516 04:13:52< mattsc> You can leave all the rest as it is
20130516 04:14:12< mattsc> jleldridge: yes, that would be my first suggestion.
20130516 04:14:43< mattsc> jleldridge: moving a couple of the ai_helper location_set related functions into location_set.lua would be another.
20130516 04:14:43< jleldridge> mattsc: Ok, I'll work on that then.
20130516 04:14:52< mattsc> After that, I have others, if you want  :)
20130516 04:17:52< jleldridge> mattsc: Sure. I'm going to take it one at a time for now though :P I'll let you know if I manage to make any progress on the aggression/[filter] problem
20130516 04:18:16-!- EliDupree2 [~eli@dhip-029.rrw.residences.colby.edu] has quit [Ping timeout: 246 seconds]
20130516 04:18:19< mattsc> jleldridge: sounds good
20130516 04:19:11< vultraz_laptop> mattsc: ok, so basically alarantalara needs to upload a new maccompilestuff?
20130516 04:23:58< mattsc> vultraz_laptop: I think so (but am not entirely sure if that's all that's needed; probably the Xcode project file(s) need to be modified as well)
20130516 04:25:58-!- vultraz_laptop [~chatzilla@wesnoth/developer/vultraz] has quit [Ping timeout: 245 seconds]
20130516 04:40:13-!- Yukiria [~chatzilla@124.109.10.167] has joined #wesnoth-dev
20130516 04:40:58-!- vultraz [~chatzilla@wesnoth/developer/vultraz] has quit [Ping timeout: 245 seconds]
20130516 04:41:22-!- Yukiria is now known as vultraz
20130516 04:48:47-!- vultraz [~chatzilla@124.109.10.167] has quit [Changing host]
20130516 04:48:47-!- vultraz [~chatzilla@wesnoth/developer/vultraz] has joined #wesnoth-dev
20130516 04:56:51-!- Ivanovic_ [~ivanovic@x2f3f1a2.dyn.telefonica.de] has joined #wesnoth-dev
20130516 05:00:11-!- Ivanovic [~ivanovic@x2f41160.dyn.telefonica.de] has quit [Ping timeout: 256 seconds]
20130516 05:00:44-!- noy [~Noy@wesnoth/developer/noy] has quit [Quit: noy]
20130516 05:00:44-!- Ivanovic_ is now known as Ivanovic
20130516 05:02:17-!- ancestral [~ancestral@50-78-227-230-static.hfc.comcastbusiness.net] has joined #wesnoth-dev
20130516 05:06:45-!- ancestral [~ancestral@50-78-227-230-static.hfc.comcastbusiness.net] has quit [Ping timeout: 248 seconds]
20130516 05:07:55-!- ancestral [~ancestral@mobile-198-228-235-071.mycingular.net] has joined #wesnoth-dev
20130516 05:09:49-!- shikadibot_ [~shikadi@wesnoth/umc-dev/bot/shikadibot] has joined #wesnoth-dev
20130516 05:11:53-!- Gambit [~gambit@wesnoth/developer/grickit] has quit [Remote host closed the connection]
20130516 05:13:32-!- DHost_ [~Pcy@vps.inux.fr] has joined #wesnoth-dev
20130516 05:13:55-!- shikadibot [~shikadi@wesnoth/umc-dev/bot/shikadibot] has quit [Ping timeout: 245 seconds]
20130516 05:13:58-!- balrog [~balrog@discferret/developer/balrog] has quit [Ping timeout: 245 seconds]
20130516 05:13:59-!- DHost [~Pcy@vps.inux.fr] has quit [Ping timeout: 245 seconds]
20130516 05:14:04-!- balrog_ [~balrog@discferret/developer/balrog] has joined #wesnoth-dev
20130516 05:14:04-!- balrog_ is now known as balrog
20130516 05:14:30-!- ancestral [~ancestral@mobile-198-228-235-071.mycingular.net] has quit [Quit: Smell ya later!]
20130516 05:21:05-!- shikadibot_ is now known as shikadibot
20130516 05:26:52-!- ancestral [~ancestral@mobile-198-228-235-071.mycingular.net] has joined #wesnoth-dev
20130516 05:30:02< irker867> wesnoth: fendrin wesnoth-old:master 54b007 / data/themes/editor.cfg: Tooltip for the time schedule menu button. http://git.io/6RO0Dg
20130516 05:32:58< irker867> wesnoth: fendrin wesnoth-old:master 08a22a / src/editor/action/mouse/mouse_action_unit.cpp: Editor: Added labels to the unit tooltip. http://git.io/e38CBg
20130516 05:35:08-!- vultraz [~chatzilla@wesnoth/developer/vultraz] has quit [Ping timeout: 245 seconds]
20130516 05:36:16-!- vultraz [~chatzilla@wesnoth/developer/vultraz] has joined #wesnoth-dev
20130516 05:41:23< mattsc> jleldridge: still around?
20130516 05:42:27-!- love1cat [~Adium@75-130-92-96.dhcp.wlmn.ct.charter.com] has left #wesnoth-dev []
20130516 05:45:17< mattsc> jleldridge: minor problem:  http://pastebin.com/8mWnfVu5
20130516 05:46:41< mattsc> That's what I get when testing this in my test scenario.  The fix is easy, just a minor oversight in the code.  (But this is why it is so important to make sure you do extensive testing of any change you make.)
20130516 05:51:43-!- ancestral [~ancestral@mobile-198-228-235-071.mycingular.net] has quit [Quit: Smell ya later!]
20130516 05:51:51-!- ancestral [~ancestral@65-128-236-198.mpls.qwest.net] has joined #wesnoth-dev
20130516 06:03:45-!- shadowm_desktop [ignacio@wesnoth/developer/shadowmaster] has quit [Ping timeout: 240 seconds]
20130516 06:05:43-!- shadowm_desktop [ignacio@wesnoth/developer/shadowmaster] has joined #wesnoth-dev
20130516 06:08:18-!- vultraz_laptop [~chatzilla@wesnoth/developer/vultraz] has joined #wesnoth-dev
20130516 06:08:33< jleldridge> mattsc: I'm around, I'll get it fixed.
20130516 06:09:06< mattsc> jleldridge: cool.  Don't spend too much time on it.  All you need to do is move one line around.  :)
20130516 06:09:24< mattsc> As in, tell me if you just want me to tell you.
20130516 06:09:57< vultraz_laptop> holy crap holy crap I have a working build
20130516 06:10:04< vultraz_laptop> it actually compiles and runs and doesn't crash at the titlescreen
20130516 06:10:07< mattsc> Once I do that, everything checks out, both for the LS functions and for the change to the HS MAI
20130516 06:10:23< mattsc> vultraz_laptop: yay!
20130516 06:11:45< vultraz_laptop> fabi: the editor looks beautiful :D
20130516 06:12:07< vultraz_laptop> and LordBob's buttons look etremely good
20130516 06:15:36< fabi> vultraz_laptop: :-)
20130516 06:15:42< fabi> vultraz_laptop: Does it work?
20130516 06:15:58< fabi> vultraz_laptop: Have you discovered how to put units on the map?
20130516 06:16:19< jleldridge> mattsc: Oh I see, value_table needs to be declared in the scope of the loop
20130516 06:16:41< mattsc> jleldridge: yep, that's all.
20130516 06:16:42< jleldridge> otherwise we're using the same table for every index
20130516 06:23:14< jleldridge> mattsc: Ok, that's done and pushed. I'm so good at rewriting git history now :P
20130516 06:25:45< irker867> wesnoth: jleldridge wesnoth-old:master 0d8bd9 / data/lua/location_set.lua: Changed location_set.of_pairs() to work with different formats of x,y coordinate http://git.io/dnCcEA
20130516 06:25:53< irker867> wesnoth: jleldridge wesnoth-old:master 8665d0 / data/ai/micro_ais/ais/mai_healer_support_engine.lua: Changed healer support mai to use new location_set.of_pairs() method to create a http://git.io/uRcOgg
20130516 06:26:02< irker867> wesnoth: mattsc wesnoth-old:master 9c25c0 / data/ (ai/micro_ais/ais/mai_healer_support_engine.lua lua/location_set.lua): Merge pull request #11 from jleldridge/master http://git.io/OeKnWg
20130516 06:26:46< mattsc> jleldridge: ^ well done.  Again.
20130516 06:26:59< mattsc> I'm going to sign off now.
20130516 06:27:07< fabi> mattsc: bye
20130516 06:27:19< jleldridge> mattsc: Ok, thanks again for the help, talk to you later :)
20130516 06:27:34< mattsc> fabi: Ciao
20130516 06:28:27-!- mattsc [~mattsc@d154-20-34-165.bchsia.telus.net] has quit [Quit: bye]
20130516 06:28:56< vultraz> fabi: I'll test more later, I was just seeing that it launched w/o problems
20130516 06:32:44< vultraz> important thing is I have the repo and a working build :)
20130516 06:42:04< fabi> vultraz: Do you know how to code c++?
20130516 06:42:23< vultraz> no, not really
20130516 06:42:43< fabi> vultraz: But you know how programming works in general?
20130516 06:42:47< vultraz> and even if I did, I'm not familiar with how the codebase is set up
20130516 06:42:50< vultraz> fabi: yes
20130516 06:43:25< fabi> vultraz: Well, c++ is just another language. Maybe not the easiest one but not black magic either.
20130516 06:43:58< vultraz> lua is basically the only think I know
20130516 06:44:09< vultraz> and even that not very wel
20130516 06:44:10< vultraz> l
20130516 06:55:01-!- Nephro [~neph@80.233.231.12] has joined #wesnoth-dev
20130516 07:04:27-!- [Relic] [~relic@99-58-54-211.lightspeed.milwwi.sbcglobal.net] has quit [Quit: Leaving]
20130516 07:08:25-!- trademark [~ptalbot@mne69-1-82-67-17-201.fbx.proxad.net] has joined #wesnoth-dev
20130516 07:08:35-!- noy [~Noy@wesnoth/developer/noy] has joined #wesnoth-dev
20130516 07:21:30-!- jleldridge [~jleldridg@cpe-098-026-227-236.triad.res.rr.com] has quit [Quit: Leaving]
20130516 07:28:33-!- Nephro [~neph@80.233.231.12] has quit [Ping timeout: 248 seconds]
20130516 07:32:34-!- zookeeper [~lmsnie@wesnoth/developer/zookeeper] has joined #wesnoth-dev
20130516 07:34:44-!- noy [~Noy@wesnoth/developer/noy] has quit [Quit: noy]
20130516 07:46:17-!- noy [~Noy@wesnoth/developer/noy] has joined #wesnoth-dev
20130516 07:49:50-!- Ivanovic [~ivanovic@x2f3f1a2.dyn.telefonica.de] has quit [Changing host]
20130516 07:49:50-!- Ivanovic [~ivanovic@wesnoth/developer/ivanovic] has joined #wesnoth-dev
20130516 07:57:38-!- just4fun [dea7ea83@gateway/web/freenode/ip.222.167.234.131] has joined #wesnoth-dev
20130516 08:01:18-!- Necrosporus [~Necrospor@unaffiliated/necrosporus] has quit [Ping timeout: 252 seconds]
20130516 08:01:59-!- Blueblaze [~Blueblaze@adsl-99-158-45-47.dsl.hstntx.sbcglobal.net] has quit [Quit: Blueblaze]
20130516 08:04:09-!- Necrosporus [~Necrospor@unaffiliated/necrosporus] has joined #wesnoth-dev
20130516 08:14:40-!- trademark [~ptalbot@mne69-1-82-67-17-201.fbx.proxad.net] has quit [Remote host closed the connection]
20130516 08:15:20-!- noy [~Noy@wesnoth/developer/noy] has quit [Quit: noy]
20130516 08:24:01-!- Necrosporus [~Necrospor@unaffiliated/necrosporus] has quit [Remote host closed the connection]
20130516 08:26:52-!- LordBob_ [~chatzilla@2a01:e34:ee82:47e0:21e:c2ff:fe01:261f] has joined #wesnoth-dev
20130516 08:29:10-!- rsyh93 [~Young@c-76-26-142-24.hsd1.md.comcast.net] has left #wesnoth-dev []
20130516 08:30:21-!- ancestral [~ancestral@65-128-236-198.mpls.qwest.net] has quit [Quit: i go nstuf kthxbai]
20130516 08:37:36-!- Ivanovic_ [~ivanovic@x2f4faba.dyn.telefonica.de] has joined #wesnoth-dev
20130516 08:40:03-!- Ivanovic [~ivanovic@wesnoth/developer/ivanovic] has quit [Ping timeout: 256 seconds]
20130516 08:41:30-!- Ivanovic_ is now known as Ivanovic
20130516 08:47:53-!- boucman_work [~rosen@wesnoth/developer/boucman] has joined #wesnoth-dev
20130516 09:02:44-!- Rhonda [~rhonda@anguilla.debian.or.at] has quit [Remote host closed the connection]
20130516 09:05:33-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has joined #wesnoth-dev
20130516 09:07:01-!- just4fun [dea7ea83@gateway/web/freenode/ip.222.167.234.131] has quit [Ping timeout: 250 seconds]
20130516 09:18:39-!- Rhonda [~rhonda@anguilla.debian.or.at] has joined #wesnoth-dev
20130516 09:32:44-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has left #wesnoth-dev []
20130516 09:43:34-!- Ivanovic [~ivanovic@x2f4faba.dyn.telefonica.de] has quit [Remote host closed the connection]
20130516 09:43:51-!- Ivanovic [~ivanovic@x2f4faba.dyn.telefonica.de] has joined #wesnoth-dev
20130516 09:49:18-!- vultraz_laptop [~chatzilla@wesnoth/developer/vultraz] has quit [Ping timeout: 245 seconds]
20130516 09:50:04-!- vultraz [~chatzilla@wesnoth/developer/vultraz] has quit [Ping timeout: 246 seconds]
20130516 09:50:38-!- vultraz_laptop [~chatzilla@wesnoth/developer/vultraz] has joined #wesnoth-dev
20130516 09:56:12-!- vultraz [~chatzilla@wesnoth/developer/vultraz] has joined #wesnoth-dev
20130516 10:10:01-!- lipkab [~lipk@2001:738:5404:192:ecde:8253:b6f4:a9fb] has joined #wesnoth-dev
20130516 10:24:38-!- mjs-de [~mjs-de@g224180190.adsl.alicedsl.de] has joined #wesnoth-dev
20130516 10:39:40-!- noy [~Noy@wesnoth/developer/noy] has joined #wesnoth-dev
20130516 10:51:45-!- TheMonster [~TheMonste@41.69.219.30] has joined #wesnoth-dev
20130516 11:04:26-!- exciton [chuck-the-@89.208.169.104] has quit [Ping timeout: 252 seconds]
20130516 11:05:20-!- exciton [chuck-the-@89.208.169.104] has joined #wesnoth-dev
20130516 11:05:24-!- Crendgrim [~quassel@g224212009.adsl.alicedsl.de] has quit [Ping timeout: 256 seconds]
20130516 11:07:38-!- Crendgrim [~quassel@f050094009.adsl.alicedsl.de] has joined #wesnoth-dev
20130516 11:17:14-!- TheMonster [~TheMonste@41.69.219.30] has quit [Read error: Connection reset by peer]
20130516 11:18:11< LordBob_> fabi: ping ?
20130516 11:25:17-!- Crab_ [Crab_@wesnoth/developer/crab] has joined #wesnoth-dev
20130516 11:29:09-!- shadowm_desktop [ignacio@wesnoth/developer/shadowmaster] has quit [Ping timeout: 248 seconds]
20130516 11:56:32-!- irker867 [~irker@ai0867.net] has quit [Quit: transmission timeout]
20130516 12:08:18-!- lipkab [~lipk@2001:738:5404:192:ecde:8253:b6f4:a9fb] has quit [Quit: Konversation terminated!]
20130516 12:11:05-!- EliDupree2 [~eli@dhip-029.rrw.residences.colby.edu] has joined #wesnoth-dev
20130516 12:12:13-!- Kexoth [~kex@89.205.70.185] has joined #wesnoth-dev
20130516 12:16:22-!- Kexoth [~kex@89.205.70.185] has quit [Remote host closed the connection]
20130516 12:24:59-!- Kexoth [~kex@89.205.70.185] has joined #wesnoth-dev
20130516 12:33:05-!- DHost_ [~Pcy@vps.inux.fr] has quit [Quit: another kernel upgrade]
20130516 12:36:15-!- Kexoth [~kex@89.205.70.185] has quit [Remote host closed the connection]
20130516 12:40:57-!- Kexoth [~kex@89.205.70.185] has joined #wesnoth-dev
20130516 12:43:26< vultraz> LordBob_: I got a working build, and I just gotta say, your art looks great :D
20130516 12:43:57< LordBob_> Haha, thanks, that's a good reward for the long hours spent on it :)
20130516 12:44:51-!- DHost [~Pcy@vps.inux.fr] has joined #wesnoth-dev
20130516 12:45:08< vultraz> Especially in dialogs were GUI2 is used, it makes everything look 10x neater
20130516 12:46:00< LordBob_> Yeah, the new buttons do a fine job. And wait 'till we upgrade the backgrounds for these dialogs... 8)
20130516 12:46:43< LordBob_> Vultraz: btw, I'm testing new status icons: which ones do you prefer, the monochromes or the coloured ones ? http://imagebin.org/257819
20130516 12:46:57< vultraz> holy crap, you're going the backgrounds too?
20130516 12:47:45< LordBob_> Well, sure, except maybe the transparent ones that already look fine (maybe they'll get a shiner border, and that's it) 
20130516 12:50:48-!- crimson_penguin [~crimson_p@wesnoth/developer/crimsonpenguin] has quit [Excess Flood]
20130516 12:51:22< vultraz> LordBob_: colored ones for sure
20130516 12:51:28-!- crimson_penguin [~crimson_p@wesnoth/developer/crimsonpenguin] has joined #wesnoth-dev
20130516 12:52:13< vultraz> LordBob_: is the topbar bg also going to get an upgrade?
20130516 12:53:33< LordBob_> vultraz: There will likely be a rearrangement of its content once we turn our efforts to the game UI
20130516 12:53:51< LordBob_> but for the time being, I'm just doing icons
20130516 12:54:19< vultraz> alright
20130516 12:54:26< vultraz> can't wait until you get to the UI :)
20130516 12:54:27< LordBob_> as for the background, it's highly likely that it will receive the same polish as the editor panels
20130516 12:56:53< vultraz> sweet :D
20130516 13:03:37-!- thunderstruck [~thunderst@cpc5-sgyl29-2-0-cust174.sgyl.cable.virginmedia.com] has joined #wesnoth-dev
20130516 13:12:56-!- PL_kolek [~PL_kolek@dynamic-78-8-156-131.ssp.dialog.net.pl] has joined #wesnoth-dev
20130516 13:36:40-!- loonybot [~loonybot@ppp91-78-87-169.pppoe.mtu-net.ru] has joined #wesnoth-dev
20130516 13:36:41-!- loonybot [~loonybot@ppp91-78-87-169.pppoe.mtu-net.ru] has quit [Changing host]
20130516 13:36:41-!- loonybot [~loonybot@wesnoth/bot/loonybot] has joined #wesnoth-dev
20130516 13:48:03-!- loonybot [~loonybot@wesnoth/bot/loonybot] has quit [Remote host closed the connection]
20130516 13:55:43-!- knotwork [~markm@unaffiliated/knotwork] has quit [Ping timeout: 240 seconds]
20130516 14:21:45< LordBob_> Vultraz: new unit groups icons; they're slightly larger than the previous attempt (30 pix), therefore easier to recognize I hope http://imagebin.org/257821
20130516 14:28:23-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has joined #wesnoth-dev
20130516 14:32:04-!- knotwork [~markm@142.68.210.209] has joined #wesnoth-dev
20130516 14:32:04-!- knotwork [~markm@142.68.210.209] has quit [Changing host]
20130516 14:32:04-!- knotwork [~markm@unaffiliated/knotwork] has joined #wesnoth-dev
20130516 14:39:13-!- irker832 [~irker@ai0867.net] has joined #wesnoth-dev
20130516 14:39:13< irker832> wesnoth: Emilien Rotival wesnoth-old:master 4f5873 / images/icons/status/ (status_16x16_units.png status_16x16_villages.png): removing misplaced icon files http://git.io/EdE5PQ
20130516 14:39:13< irker832> wesnoth: Emilien Rotival wesnoth-old:master 54b2f5 / data/core/images/themes/ (17 files): New artwork for status icons http://git.io/wINlQQ
20130516 14:39:21< irker832> wesnoth: Emilien Rotival wesnoth-old:master ad46a0 / images/icons/unit-groups/ (37 files): New artwork for the editor palette: unit groups icons http://git.io/nxJFig
20130516 14:39:44-!- dragos [~quassel@5-15-212-194.residential.rdsnet.ro] has joined #wesnoth-dev
20130516 14:41:00< zookeeper> uh, why are there so _many_ of them?
20130516 14:45:37< vultraz> LordBob_: definitely an improvement :)
20130516 14:45:41< LordBob_> zookeeper: there's a group icon for each race and faction of the default era, plus 10 generic ones for UMC eras that would not have their own artwork
20130516 14:46:00-!- PL_kolek_ [~PL_kolek@dynamic-78-8-156-131.ssp.dialog.net.pl] has joined #wesnoth-dev
20130516 14:47:26< vultraz> LordBob_: so those are for custom factions?
20130516 14:48:01< vultraz> LordBob_: BTW, are there icons for custom terrain groups?
20130516 14:48:07< vultraz> my build earlier didn't  show any
20130516 14:48:54-!- loonybot [~loonybot@ppp91-78-87-169.pppoe.mtu-net.ru] has joined #wesnoth-dev
20130516 14:48:54-!- loonybot [~loonybot@ppp91-78-87-169.pppoe.mtu-net.ru] has quit [Changing host]
20130516 14:48:54-!- loonybot [~loonybot@wesnoth/bot/loonybot] has joined #wesnoth-dev
20130516 14:49:12< LordBob_> Yes to both. There are 4 custom terrain icons, but maybe wiring them requires some coding
20130516 14:49:40< vultraz> Maybe fabi just forgot or something
20130516 14:49:41< bumbadadabum> LordBob_: Oh cool
20130516 14:50:34< LordBob_> Yeah, he'll know what needs to/can be done. I'm clueless when it comes to code
20130516 14:53:31< LordBob_> zookeeper: altogether, I'm producing a lot more icons than the strict required minimum (some in anticipation of high res displays, some redundant when I'm not sure which one fits better for a given use, etc.). Once we're done with the editor facelift, I'll do a bit of cleaning up
20130516 14:53:54< zookeeper> okay
20130516 15:05:20-!- Zazweda [c130e7e4@gateway/web/freenode/ip.193.48.231.228] has joined #wesnoth-dev
20130516 15:06:09< Zazweda> hey
20130516 15:08:01-!- Zazweda_ [c130f380@gateway/web/freenode/ip.193.48.243.128] has joined #wesnoth-dev
20130516 15:08:04-!- loonybot [~loonybot@wesnoth/bot/loonybot] has quit [Remote host closed the connection]
20130516 15:10:35-!- Zazweda [c130e7e4@gateway/web/freenode/ip.193.48.231.228] has quit [Ping timeout: 250 seconds]
20130516 15:19:56-!- Blueblaze [~Blueblaze@adsl-99-158-45-47.dsl.hstntx.sbcglobal.net] has joined #wesnoth-dev
20130516 15:25:09-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has left #wesnoth-dev []
20130516 15:27:47-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has joined #wesnoth-dev
20130516 15:33:40-!- artisticdude [~artisticd@vsat-148-64-186-230.c005.g4.mrt.starband.net] has joined #wesnoth-dev
20130516 15:34:58< Zazweda_> Crab_: I am looking at the wiki right now,  but I can't find a description of how lua is linked to C++
20130516 15:35:09< Zazweda_> I guess the LUA state contains all the LUA information
20130516 15:35:57< Crab_> there's the lua manual at http://www.lua.org/manual/5.2/manual.html#4
20130516 15:36:06< Crab_> Zazweda_: it describes the C api that lua has
20130516 15:36:14< Zazweda_> thanks
20130516 15:36:32< Crab_> and we're using it to make some lua tables which would contain lua functions which are actually implemented in C++
20130516 15:38:51< Zazweda_> ok
20130516 15:43:56-!- love1cat [~Adium@oatleym.uits.uconn.edu] has joined #wesnoth-dev
20130516 15:51:00-!- PL_kolek_ [~PL_kolek@dynamic-78-8-156-131.ssp.dialog.net.pl] has quit [Quit: Wychodzi]
20130516 15:51:20-!- mattsc [~mattsc@71-19-182-26.dedicated.allstream.net] has joined #wesnoth-dev
20130516 15:55:01-!- loonybot [~loonybot@ppp91-78-87-169.pppoe.mtu-net.ru] has joined #wesnoth-dev
20130516 15:55:01-!- loonybot [~loonybot@ppp91-78-87-169.pppoe.mtu-net.ru] has quit [Changing host]
20130516 15:55:01-!- loonybot [~loonybot@wesnoth/bot/loonybot] has joined #wesnoth-dev
20130516 15:56:52< Zazweda_> Crab_: So the LuaState is basically the stack ?
20130516 15:57:28< Crab_> Zazweda_: yes. part of it is 'active' stack, but there are also some special indexes
20130516 15:57:37< Zazweda_> ok
20130516 15:57:41< loonycyborg> My cpu fan sounds more akin to a jackhammer now :/
20130516 16:03:13< PL_kolek> Crab_, I'm going to start implementing some simple ideas as you suggested. Currently I'm playing with combat phase code. However, there is a call to get_attacks(), which returns a vector of possible attacks or something, but I can't figure out where and when it's filled. Do you understand what I'm asking about, or should I give more details?
20130516 16:05:04-!- dragos_ [~quassel@5-15-216-200.residential.rdsnet.ro] has joined #wesnoth-dev
20130516 16:05:23-!- dragos [~quassel@5-15-212-194.residential.rdsnet.ro] has quit [Ping timeout: 240 seconds]
20130516 16:07:25< Zazweda_> Crab_: and the get_side() function called on the luastate is one of this "special index" ?
20130516 16:08:44< loonycyborg> Ivanovic: 1.11.4 windows installer is uploaded
20130516 16:12:13-!- dragos [~quassel@5-15-217-9.residential.rdsnet.ro] has joined #wesnoth-dev
20130516 16:15:12-!- dragos_ [~quassel@5-15-216-200.residential.rdsnet.ro] has quit [Ping timeout: 252 seconds]
20130516 16:17:28-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has quit [Quit: bumbadadabum]
20130516 16:22:34-!- EdB [~edb@89-93-187-203.hfc.dyn.abo.bbox.fr] has joined #wesnoth-dev
20130516 16:30:04-!- timotei [~timotei@wesnoth/developer/timotei] has quit [Quit: Leaving]
20130516 16:33:39-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has joined #wesnoth-dev
20130516 16:35:16-!- flix [~flix@178.77.175.34] has joined #wesnoth-dev
20130516 16:41:07-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has quit [Quit: bumbadadabum]
20130516 16:50:35-!- jetrel2 [~jetrel2@64.208.23.76] has joined #wesnoth-dev
20130516 16:54:51-!- DCW [~Thunderbi@cpc1-finc14-2-0-cust12.4-2.cable.virginmedia.com] has joined #wesnoth-dev
20130516 16:56:36< irker832> wesnoth: mattsc wesnoth-old:master a1bc87 / data/campaigns/Under_the_Burning_Suns/ (ai/messenger-escort_engine.lua scenarios/08_Out_of_the_Frying_Pan.cfg): UtBS Out of the Frying Pan: Use Messenger Escort Micro AI for messenger side http://git.io/mge-uA
20130516 16:59:52-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has joined #wesnoth-dev
20130516 17:00:00-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has quit [Client Quit]
20130516 17:01:59-!- mattsc [~mattsc@71-19-182-26.dedicated.allstream.net] has quit [Quit: I'm asleep]
20130516 17:14:30-!- Blueblaze [~Blueblaze@adsl-99-158-45-47.dsl.hstntx.sbcglobal.net] has quit [Quit: Blueblaze]
20130516 17:23:57-!- dragos [~quassel@5-15-217-9.residential.rdsnet.ro] has quit [Read error: Connection reset by peer]
20130516 17:25:58-!- noy [~Noy@wesnoth/developer/noy] has quit [Ping timeout: 245 seconds]
20130516 17:37:09-!- LordBob_ [~chatzilla@2a01:e34:ee82:47e0:21e:c2ff:fe01:261f] has quit [Ping timeout: 245 seconds]
20130516 17:39:45-!- artisticdude [~artisticd@vsat-148-64-186-230.c005.g4.mrt.starband.net] has quit [Read error: Connection reset by peer]
20130516 17:42:36< Zazweda_> Crab_:  Is there right now a function which allows to create a unit from scratch ? (not recruit)
20130516 17:43:12-!- EdB [~edb@89-93-187-203.hfc.dyn.abo.bbox.fr] has quit [Quit: Konversation terminated!]
20130516 17:45:20-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has joined #wesnoth-dev
20130516 17:53:46-!- timotei [~timotei@79.119.100.244] has joined #wesnoth-dev
20130516 17:53:46-!- timotei [~timotei@79.119.100.244] has quit [Changing host]
20130516 17:53:46-!- timotei [~timotei@wesnoth/developer/timotei] has joined #wesnoth-dev
20130516 17:56:13< Zazweda_> Crab_: If I understand well, currently LUA calls the function (lets say ai_move), which will call the Actions execute_move_actions which will itself execute the action, right ?
20130516 17:58:50-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has quit [Quit: bumbadadabum]
20130516 17:59:49-!- DCW [~Thunderbi@cpc1-finc14-2-0-cust12.4-2.cable.virginmedia.com] has quit [Quit: DCW]
20130516 18:01:49-!- vultraz [~chatzilla@wesnoth/developer/vultraz] has quit [Ping timeout: 246 seconds]
20130516 18:03:07-!- flix [~flix@178.77.175.34] has quit [Quit: Leaving.]
20130516 18:04:54-!- LordBob_ [~chatzilla@2a01:e34:ee82:47e0:21e:c2ff:fe01:261f] has joined #wesnoth-dev
20130516 18:07:53-!- vultraz [~chatzilla@wesnoth/developer/vultraz] has joined #wesnoth-dev
20130516 18:16:49-!- Ivanovic [~ivanovic@x2f4faba.dyn.telefonica.de] has quit [Changing host]
20130516 18:16:49-!- Ivanovic [~ivanovic@wesnoth/developer/ivanovic] has joined #wesnoth-dev
20130516 18:17:51-!- Upthorn [~ogmar@108-85-91-228.lightspeed.frokca.sbcglobal.net] has quit [Ping timeout: 252 seconds]
20130516 18:20:37-!- boucman_work [~rosen@wesnoth/developer/boucman] has quit [Ping timeout: 256 seconds]
20130516 18:32:03< Crab_> Zazweda_: yes. AI can't do it, of course, but wesnoth lua code or wesnoth c++ code can.
20130516 18:32:49< Crab_> Zazweda_: for C++ example, see game_events.cpp , // If we should spawn a new unit on the map somewhere  WML_HANDLER_FUNCTION(unit, /*event_info*/, cfg)
20130516 18:33:07< Crab_> Zazweda_: or you can modify unit map directly, or see what unit_creator does (it does some extra things that you might not need)
20130516 18:35:08< Crab_> Zazweda_: lua ai calls a function like ai.move_full, which is a function from lua table which was put there by C++ code. so , C++ code for that function would be executed (for AI functions, it is in src/ai/lua/core.cpp , cfun_ai_execute_move_full , which will do (in ai_move C++ function) ai::actions::execute_move_action(side,exec,from,to,remove_movement, unreach_is_ok) which will check and execute the action.
20130516 18:35:17< Crab_> Zazweda_: so, yes, you're right.
20130516 18:35:41< Crab_> Zazweda_: see lua_ai_context::create(lua_State *L, char const *code, ai::engine_lua *engine) for the link between string names of lua AI functions and their C++ implementations
20130516 18:38:48< PL_kolek> Crab_, in case you read my question and want to answer, I figured it out (mostly, there is still some magic I don't wwant to dig into, but I know where wnd when). So no need to answer
20130516 18:39:14-!- perennate [~wizardus@MACGREGOR-THREE-TWENTY-EIGHT.MIT.EDU] has joined #wesnoth-dev
20130516 18:39:26-!- boucman [~rosen@wesnoth/developer/boucman] has joined #wesnoth-dev
20130516 18:39:31< Zazweda_> Crab_: Ok thanks !
20130516 18:39:33-!- Zazweda_ [c130f380@gateway/web/freenode/ip.193.48.243.128] has quit [Quit: Page closed]
20130516 18:41:00-!- Crab_ [Crab_@wesnoth/developer/crab] has left #wesnoth-dev []
20130516 18:52:03-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has joined #wesnoth-dev
20130516 19:01:37-!- jleldridge [~jleldridg@cpe-098-026-227-236.triad.res.rr.com] has joined #wesnoth-dev
20130516 19:02:14< bumbadadabum> wesbot: seen mattsc
20130516 19:02:15< wesbot> bumbadadabum: The person with the nick mattsc 2h ago was here and on the channel #wesnoth-umc-dev with the message: Quit: I'm asleep
20130516 19:03:23-!- artisticdude [~artisticd@vsat-148-64-186-230.c005.g4.mrt.starband.net] has joined #wesnoth-dev
20130516 19:03:24-!- artisticdude [~artisticd@vsat-148-64-186-230.c005.g4.mrt.starband.net] has quit [Client Quit]
20130516 19:05:29-!- love1cat [~Adium@oatleym.uits.uconn.edu] has quit [Quit: Leaving.]
20130516 19:05:58-!- PL_kolek [~PL_kolek@dynamic-78-8-156-131.ssp.dialog.net.pl] has quit [Quit: Wychodzi]
20130516 19:11:48-!- prkc [~negusnyul@BC069D1D.dsl.pool.telekom.hu] has joined #wesnoth-dev
20130516 19:16:55-!- LordBob_ [~chatzilla@2a01:e34:ee82:47e0:21e:c2ff:fe01:261f] has quit [Quit: ChatZilla 0.9.90 [Firefox 20.0/20130326150557]]
20130516 19:17:02-!- ancestral [~ancestral@65-128-236-198.mpls.qwest.net] has joined #wesnoth-dev
20130516 19:24:30-!- flix [~flix@178.77.175.34] has joined #wesnoth-dev
20130516 19:33:03< jleldridge> ls
20130516 19:33:06< jleldridge> mt
20130516 19:33:40-!- mattsc [~mattsc@fw.hia.nrc.ca] has joined #wesnoth-dev
20130516 19:36:18-!- ancestral [~ancestral@65-128-236-198.mpls.qwest.net] has quit [Quit: i go nstuf kthxbai]
20130516 19:43:02< jleldridge> ls
20130516 19:43:08< jleldridge> mt again >_>
20130516 19:47:13-!- prkc [~negusnyul@BC069D1D.dsl.pool.telekom.hu] has quit [Quit: Konversation terminated!]
20130516 19:52:54-!- trewe [~trewe@87-196-100-241.net.novis.pt] has joined #wesnoth-dev
20130516 19:53:25< bumbadadabum> mattsc: Your rushers recruitment doesn't work in my scenario
20130516 19:55:19< mattsc> bumbadadabum: in what way?
20130516 19:55:31< bumbadadabum> mattsc: The side doesn't recruit at all
20130516 19:56:31< mattsc> But no error messages at stdout or stderr?
20130516 19:56:34< bumbadadabum> no
20130516 19:57:06< mattsc> Could you paste the scenario code, so that I can give it a try?
20130516 19:57:08< bumbadadabum> Also, could I get the latest binary, if you don't mind
20130516 19:57:17< bumbadadabum> it's the same scenario as previously
20130516 19:57:30< bumbadadabum> I'm going to see if it also doesn't work in another scenario
20130516 19:57:35< mattsc> bumbadadabum: sure, let me compile
20130516 19:59:39< mattsc> I don't know (remember?) what you mean with "same scenario as before".
20130516 20:00:18< bumbadadabum> It's the same scenario as where I got my errors because of some stuff a few days ago
20130516 20:00:37< mattsc> Oh, right.  Sorry, too many things going on in parallel.  :)
20130516 20:01:04< mattsc> bumbadadabum: do you want me to do something right now or are you testing something else yourself first?
20130516 20:01:25< bumbadadabum> testing it in another scenario
20130516 20:01:41< jleldridge> mattsc, bumbadadabum: I just compiled a binary from the latest commits if you want mine to save some time.
20130516 20:01:55< bumbadadabum> jleldridge: A mac one?
20130516 20:02:00< jleldridge> nah, linux
20130516 20:02:22< jleldridge> Nevermind, forgot about the whole different platforms thing
20130516 20:03:14< bumbadadabum> mattsc: It does recruit
20130516 20:03:16< mattsc> jleldridge: thanks, it's really just clicking a few buttons for me (and waiting, more or less long, depending on how long it's been since my last compile)
20130516 20:03:20< bumbadadabum> but an error message appears every time
20130516 20:03:44< bumbadadabum> well
20130516 20:03:53< bumbadadabum> your "it took a really long time" one
20130516 20:04:27< mattsc> bumbadadabum: what?  That shouldn't happen if you just use the recruiter...
20130516 20:04:40< mattsc> That's something built into Fred, isn't it?
20130516 20:04:59< fabi> hello
20130516 20:05:10< bumbadadabum> I just did
20130516 20:05:10< bumbadadabum>             side=3
20130516 20:05:11< bumbadadabum>             ai_type=recruiting
20130516 20:05:11< bumbadadabum>             action=add
20130516 20:05:11< bumbadadabum>             recruiting_type=rushers
20130516 20:05:11< bumbadadabum>             randomness=1
20130516 20:05:38< mattsc> bumbadadabum: inside a [micro_ai] tag, right?
20130516 20:05:40< bumbadadabum> yes
20130516 20:05:59< mattsc> And you got the message that it took a long time, please report this?
20130516 20:06:07< bumbadadabum> the strange thing is, it didn't take take 90 seconds
20130516 20:06:29< bumbadadabum> in ca "recruit_rushers"
20130516 20:06:31< bumbadadabum> but it did recruit
20130516 20:06:53< mattsc> How long do you think it took per recruit?  Guesstimate?
20130516 20:06:57< bumbadadabum> 2 seconds
20130516 20:07:09< mattsc> Per recruit or for all of them together?
20130516 20:07:10< bumbadadabum> well
20130516 20:07:15< bumbadadabum> maybe even less
20130516 20:07:38< bumbadadabum> I think ~1 second from clicking one message to the other appearing
20130516 20:07:52< mattsc> bumbadadabum: let me look into that.  That message should not be in the MAIs at all.  Probably something left over from a testing phase.
20130516 20:08:23< mattsc> As for the recruiting itself: so it worked in one scenario, but not the other?
20130516 20:08:29< bumbadadabum> yes
20130516 20:08:42< bumbadadabum> so I'm seeing now if my other ai stuff is interfering
20130516 20:08:44< mattsc> Sounds again like there is something interfering in your scenario.
20130516 20:08:52< mattsc> exactly.
20130516 20:09:54< mattsc> bumbadadabum: once you figure it out, could you send me the interfering code?  I want to see if there is something I can do in the MAIs to avoid that from happening.
20130516 20:09:54< bumbadadabum> wow
20130516 20:10:06< bumbadadabum> "unknown unit: Aragwaith Swordsmaster"
20130516 20:10:09< bumbadadabum> uhh no
20130516 20:10:13< bumbadadabum> that's not unknown
20130516 20:10:17< fabi> The unit palette will work like: Left mouse button selects the "brush" unit, the one to be placed on the map by the unit tool.
20130516 20:11:08< mattsc> bumbadadabum: new binary: https://www.dropbox.com/s/bqhbxl9jeuktyeq/Wesnoth?v=1rc-c
20130516 20:11:14< fabi> The Right button will select a set of units that can be used to define recruit lists or recall filters or similar.
20130516 20:12:13< bumbadadabum> mattsc: Thanks!
20130516 20:15:35< mattsc> bumbadadabum: I see where those error messages come from.  They clearly shouldn't be in the recruiter MAI (and can't work the way they are set up).
20130516 20:16:50< bumbadadabum> fabi: I just opened the 1.10 editor again
20130516 20:16:59< bumbadadabum> and I thought
20130516 20:17:10-!- perennate [~wizardus@MACGREGOR-THREE-TWENTY-EIGHT.MIT.EDU] has left #wesnoth-dev []
20130516 20:17:30< bumbadadabum> "Wow they did this all in so little time?!"
20130516 20:17:48< fabi> bumbadadabum: Indeed, LordBob is very quick.
20130516 20:18:04< fabi> bumbadadabum: The code is maturing on my hard disc since a long time.
20130516 20:18:07< bumbadadabum> This and 1.9 are my favorite dev releases already
20130516 20:18:31< bumbadadabum> this all looks so different
20130516 20:19:44< fabi> bumbadadabum: You must now that I started the project back in 2009.
20130516 20:20:16< bumbadadabum> I know there were ideas long before 1.11.3/4
20130516 20:20:42< bumbadadabum> but they weren't in the game
20130516 20:21:12< bumbadadabum> mattsc: It gives the error twice now
20130516 20:21:15< bumbadadabum> and does nothing
20130516 20:21:21< bumbadadabum> I deactivated all other lua ai
20130516 20:22:10< bumbadadabum> now the leader moves
20130516 20:22:13< bumbadadabum> but still nothing
20130516 20:22:44< bumbadadabum> leader moves because I don't have passive_leader anymore
20130516 20:22:47< bumbadadabum> but still nothing
20130516 20:23:12< mattsc> bumbadadabum: just to confirm, with "error" you mean the message that it took too long, right?
20130516 20:23:20< bumbadadabum> yes
20130516 20:23:30< bumbadadabum> I'd call it an error message
20130516 20:23:48< irker832> wesnoth: mattsc wesnoth-old:master 04f799 / data/ai/lua/ (generic-recruit_engine.lua generic-rush_engine.lua): Micro and experimental AIs: turn off some debug output http://git.io/rTJpLQ
20130516 20:23:51< mattsc> bumbadadabum: that's fine, just making sure that we're talking about the same thing
20130516 20:24:03< mattsc> That commit should take care of that
20130516 20:24:45< bumbadadabum> ok lemme take a look
20130516 20:25:08< mattsc> bumbadadabum: but that won't fix your recruiting issue, it just won't show that (incorrect) message
20130516 20:25:41< mattsc> bumbadadabum: can you do a quick inspect:, just to make sure recruit= is set correctly for the side?
20130516 20:26:22< mattsc> Other than that, I think I'd have to test it myself with your slimmed-down scenario.  I'm out of ideas otherwise.
20130516 20:26:54< jleldridge> mattsc: Could I ask you a few general AI questions when you get a chance?
20130516 20:27:17< bumbadadabum> mattsc: You could checkout my campaign and try it
20130516 20:27:20< mattsc> jleldridge: right now is good, for another 20min or so
20130516 20:27:34< mattsc> bumbadadabum: remind me of the link
20130516 20:27:48< bumbadadabum> umcdev?
20130516 20:28:01< mattsc> to your campaign
20130516 20:28:33< jleldridge> mattsc: Ok. How do micro-ais interact with the "default" rca-ai loop? It seems to me that if a micro-ai is defined, the rca loop is completely ignored.
20130516 20:30:23< mattsc> jleldridge: no, not at all.
20130516 20:30:52< mattsc> jleldridge: for the most part (there are exceptions that are indicated on the wiki page), the RCA AI CAs are left untouched.
20130516 20:31:26< mattsc> It's just that the MAI CAs generally have a higher evaluation score for the units they control.
20130516 20:31:51< jleldridge> Ok that makes a lot of sense.
20130516 20:33:06< jleldridge> So adding a micro-ai to a side is just like adding a new CA that probably has higher priority than the other CAs?
20130516 20:33:58< mattsc> jleldridge: yes (and again, there are a couple exceptions to that)
20130516 20:36:33< mattsc> jleldridge: for example, check out the note here: http://wiki.wesnoth.org/Micro_AIs#Protect_Unit_specific_keys_for_the_.5Bmicro_ai.5D_tag:
20130516 20:37:03< jleldridge> mattsc: So is the LUA ai basically the same idea? It just adds another candidate action, defined in LUA?
20130516 20:37:49< mattsc> jleldridge: Umm, I think the answer to that is yes.  The MAIs are written using Lua AI code.
20130516 20:38:17< mattsc> Or in other words, Micro AIs consist of CAs written in Lua.
20130516 20:39:06< mattsc> And you can add CAs to the existing RCA CAs, or just have the Lua AI CAs, or a mix/subset of that.  Whatever works for your purpose.
20130516 20:40:08< jleldridge> mattsc: Is formula ai used at all anymore?
20130516 20:40:20< jleldridge> It looks like its being phased out
20130516 20:41:20< mattsc> It's still being used in some mainline campaigns and UMC.  I don't think it's going to be removed (according to Crab_), but it isn't maintained any more.
20130516 20:41:53< jleldridge> mattsc: Ok, that clears a lot up, thank you :)
20130516 20:42:18< mattsc> jleldridge: np
20130516 20:50:37-!- anonymissimus [~chatzilla@HSI-KBW-149-172-228-192.hsi13.kabel-badenwuerttemberg.de] has joined #wesnoth-dev
20130516 20:52:48< anonymissimus> jleldridge: looking at your pull request; instead of if a ~= nil then ... you can simply do if a then...
20130516 20:53:08< anonymissimus> to check whether a is not nil
20130516 20:53:54< anonymissimus> jleldridge: also, seems you could have merged the two if (next(value_table) ~= nil) then ...end blocks into one
20130516 20:54:47< jleldridge> I can change that if you'd like, I have a habit of using more explicit notation even if the code ends up being a little longer.
20130516 20:54:57< anonymissimus> mattsc: perhaps you should have left me a chance to say this before merging :P
20130516 20:55:44< mattsc> anonymissimus: oops ...
20130516 20:56:20< anonymissimus> jleldridge: shouldn't that be an additional operation then ?
20130516 20:56:30< anonymissimus> the ~= and the if
20130516 20:56:57< anonymissimus> I see it in the source as well, if bool_var == false and such, it's ugly
20130516 20:57:25< anonymissimus> takes longer to parse (for me)
20130516 20:57:35 * Soliton agrees.
20130516 20:58:15< jleldridge> with bool variables I agree that's ugly, I've used java most of the time so I'm used to checking for what I actually want when checking if a list is empty or something
20130516 20:58:44< jleldridge> though I know that nil == false and anything else is true and the notation can be shortened
20130516 20:59:04< jleldridge> But, if the convention in this code is to use the shorter notation I'll change it.
20130516 21:01:14< _8680_> jleldridge: Lua `nil` and `false` are both Booleanly false, yes, but they’re not equal.
20130516 21:03:51< jleldridge> That's what I meant, I know that they aren't exactly equal values.
20130516 21:04:10< anonymissimus> jleldridge: well, the convention, as far as any existing mainline lua is concerned (the one in data/lua) is the short form
20130516 21:04:51< anonymissimus> at least that's what anyone writing on it used
20130516 21:05:56< jleldridge> with associative tables, table.x is equivalent to table["x"] right?
20130516 21:06:47< _8680_> I believe `foo == nil` tests are used in some wml_actions handlers for tags with attributes that default to true.
20130516 21:06:51< _8680_> jleldridge: Yes.
20130516 21:07:43-!- trewe [~trewe@87-196-100-241.net.novis.pt] has quit [Ping timeout: 252 seconds]
20130516 21:07:48-!- timotei [~timotei@wesnoth/developer/timotei] has quit [Quit: SIGKILL]
20130516 21:08:57-!- Kexoth [~kex@89.205.70.185] has quit [Remote host closed the connection]
20130516 21:16:20< jleldridge> anonymissimus: Would this format be better? http://pastebin.com/Z3vWqqJa
20130516 21:19:05< anonymissimus> jleldridge: much better yes
20130516 21:19:26< anonymissimus> jleldridge: and leave away the () in if conditions
20130516 21:19:37< anonymissimus> this is not java or c++
20130516 21:20:29< anonymissimus> it's also not needed when calling a function but the wesnoth lua convention sort of is to use () there anyway...
20130516 21:21:35< anonymissimus> jleldridge: instead of the x_index variables you could use a bool var
20130516 21:22:10< anonymissimus> local use_xy = v.x and v.y
20130516 21:23:05< jleldridge> x_index and y_index are being used in the index() function to create an index for values[] as well though
20130516 21:23:18< anonymissimus> but you'd need to make an additional if later then
20130516 21:24:34-!- stikonas [~gentoo@128.232.240.234] has joined #wesnoth-dev
20130516 21:24:35-!- stikonas [~gentoo@128.232.240.234] has quit [Changing host]
20130516 21:24:35-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev
20130516 21:26:04< jleldridge> anonymissimus: btw, the code in that pastebin has a problem on line 23, I'm testing to make sure this syntax gives the same behavior and I fixed it.
20130516 21:27:13< anonymissimus> v.[x_index] ? yes, I just assume that's noticed and fixed
20130516 21:27:39< jleldridge> Ok
20130516 21:28:15< anonymissimus> next(table) is the normal way for checking whether a table is completely empty ? # gives 0 if there are only names indices
20130516 21:29:10< jleldridge> anonymissimus: I looked up what the standard way was to check if a table was empty and that was it
20130516 21:29:31< anonymissimus> okay, nice
20130516 21:33:31< jleldridge> Here's the version I want to submit: http://pastebin.com/5NqXzr5r
20130516 21:39:43< mattsc> anonymissimus: I'll stay out of this today and let you do it.  I have even less time today than yesterday and obviously I was rushing through it too quickly already yesterday ...  :)
20130516 21:40:11-!- LordBob_ [~chatzilla@2a01:e34:ee82:47e0:21e:c2ff:fe01:261f] has joined #wesnoth-dev
20130516 21:42:17< anonymissimus> jleldridge: okay, good
20130516 21:42:24< mattsc> jleldridge: sorry for that.  We even talked about some of that, but then didn't follow up on it.  ;P
20130516 21:42:48< anonymissimus> mattsc: that means that I finally can check out the pull request feature! :)
20130516 21:42:59< jleldridge> It was an easy enough fix, so np :)
20130516 21:43:35< mattsc> anonymissimus: on the github website, there's this really nice green button called "automatically merge this pull request" (or something like that)  :D
20130516 21:43:48< anonymissimus> I think push request would make more sense as a name though
20130516 21:44:25< mattsc> anonymissimus: that has confused me before as well ...
20130516 21:44:30< anonymissimus> after all, the request is that someone else pushes one's changes due to lack of commit rights
20130516 21:45:26< jleldridge> mattsc, anonymissimus: I essentially rewrote the history so that those formatting edits are merged into the original commit I made. IDK how this will effect the pull request so let me know if I need to change the commit history at all.
20130516 21:45:45< jleldridge> but the pull request has been submitted
20130516 21:46:21< anonymissimus> mattsc: AI0867 said there's also a description of how to do it locally, by checking out the pull request into a local branch
20130516 21:46:22< jleldridge> actually, hold on, it included extra commits with it
20130516 21:47:38< anonymissimus> jleldridge: well, I assume that since the first request was already merged, it's better to start a new one; at least we never "rewrite history" of a commit that was already made public
20130516 21:47:54< mattsc> anonymissimus: yes, if you click on the link "use the command line" left of the merge button, it gives you the commands you need to enter to do local tests first
20130516 21:48:12< anonymissimus> since that would break other dev's history; I don't know about the pull request aspect though
20130516 21:49:01< mattsc> jleldridge: I agree with anonymissimus.  Your new commit should read something like "fix things that mattsc was too stupid to notice"  :)
20130516 21:54:54< jleldridge> anonymissimus, mattsc: ok, the pull request has been fixed
20130516 21:55:11< jleldridge> on a side note, git is awesome
20130516 21:56:52< mattsc> jleldridge: I have to be off and will at the earliest be back in 5 or 6 hours.  But anonymissimus will do a much better job than I anyway.
20130516 21:57:24< jleldridge> mattsc: Ok, talk to you later!
20130516 21:57:56< mattsc> bye
20130516 21:58:00-!- mattsc [~mattsc@fw.hia.nrc.ca] has quit [Quit: bye]
20130516 22:11:33< jleldridge> anonymissimus: I'm going to be away for an hour or so. If I need to make any changes to my pull request let me know here and I'll read it in the logs.
20130516 22:33:55-!- anonymissimus [~chatzilla@HSI-KBW-149-172-228-192.hsi13.kabel-badenwuerttemberg.de] has quit [Ping timeout: 260 seconds]
20130516 22:34:19< LordBob_> fabi: there ?
20130516 22:46:25-!- EliDupree2 [~eli@dhip-029.rrw.residences.colby.edu] has quit [Ping timeout: 248 seconds]
20130516 22:47:35-!- bumbadadabum_ [~bumbadada@d155109.upc-d.chello.nl] has joined #wesnoth-dev
20130516 22:50:45-!- Netsplit *.net <-> *.split quits: Upth, Ivanovic, jetrel2, bumbadadabum
20130516 22:50:45-!- bumbadadabum_ is now known as bumbadadabum
20130516 22:52:56-!- Netsplit over, joins: jetrel2
20130516 22:57:47-!- Ivanovic [~ivanovic@x2f4faba.dyn.telefonica.de] has joined #wesnoth-dev
20130516 22:59:33-!- EliDupree2 [~eli@dhip-029.rrw.residences.colby.edu] has joined #wesnoth-dev
20130516 23:03:28-!- vultraz [~chatzilla@wesnoth/developer/vultraz] has quit [Read error: Connection reset by peer]
20130516 23:07:31-!- thunderstruck [~thunderst@cpc5-sgyl29-2-0-cust174.sgyl.cable.virginmedia.com] has quit [Quit: Leaving]
20130516 23:20:10-!- Gambit [~gambit@wesnoth/developer/grickit] has joined #wesnoth-dev
20130516 23:32:24-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has quit [Quit: bumbadadabum]
20130516 23:35:14< jleldridge> I'm back
20130516 23:36:31-!- shadowm_desktop [ignacio@wesnoth/developer/shadowmaster] has joined #wesnoth-dev
20130516 23:52:14-!- vultraz_laptop [~chatzilla@wesnoth/developer/vultraz] has quit [Ping timeout: 252 seconds]
20130516 23:52:37-!- zookeeper [~lmsnie@wesnoth/developer/zookeeper] has quit [Ping timeout: 248 seconds]
--- Log closed Fri May 17 00:00:32 2013