Re: [Flightgear-devel] Multiplayer chat patch
On Saturday 30 September 2006 16:57, Buchanan, Stuart wrote: Any chance this could be committed sometime, or objections raised? Yes, I will do so this weekend. I would like to double check that it does not break the current protocol ... Greetings Mathias - Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.phpp=sourceforgeCID=DEVDEV ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] Multiplayer chat patch
--- Mathias Fröhlich wrote: ember 2006 16:57, Buchanan, Stuart wrote: Any chance this could be committed sometime, or objections raised? Yes, I will do so this weekend. I would like to double check that it does not break the current protocol ... Thanks. I've only been able to test the recent minor changes on a single computer, so a bit or paranoia would be good. If you need another person to help, let me know. I'll be on IRC. -Stuart ___ All new Yahoo! Mail The new Interface is stunning in its simplicity and ease of use. - PC Magazine http://uk.docs.yahoo.com/nowyoucan.html - Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.phpp=sourceforgeCID=DEVDEV ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] Multiplayer chat patch
Stuart, I have problems applying that patches. I am not sure why but patch claims: patch: Only garbage was found in the patch input. Could you rediff that only with cvs diff -u instead of cvs diff -u -p -8 -w I would expect that this helps Thanks! On Sunday 01 October 2006 11:26, Buchanan, Stuart wrote: Thanks. I've only been able to test the recent minor changes on a single computer, so a bit or paranoia would be good. If you need another person to help, let me know. I'll be on IRC. Thanks! Greetings Mathias - Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.phpp=sourceforgeCID=DEVDEV ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] Multiplayer chat patch
--- Mathias Fröhlich wrote: Stuart, I have problems applying that patches. I am not sure why but patch claims: patch: Only garbage was found in the patch input. Could you rediff that only with cvs diff -u instead of cvs diff -u -p -8 -w Done. I've also run dos2unix on it. Available from http://www.nanjika.co.uk/flightgear/chat.tar.gz -Stuart ___ All new Yahoo! Mail The new Interface is stunning in its simplicity and ease of use. - PC Magazine http://uk.docs.yahoo.com/nowyoucan.html - Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.phpp=sourceforgeCID=DEVDEV ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] Multiplayer chat patch
--- Buchanan, Stuart wrote: --- Buchanan, Stuart wrote: --- Mathias Fröhlich wrote: snip Can you update your patch to avoid transmitting the type? Yes, but I can't provide any guarantee as to when I'll get it done. Well, whatcha know - it was less work than expected. New version available from http://www.nanjika.co.uk/flightgear/chat.tar.gz -Stuart Any chance this could be committed sometime, or objections raised? -Stuart ___ All New Yahoo! Mail Tired of [EMAIL PROTECTED]@! come-ons? Let our SpamGuard protect you. http://uk.docs.yahoo.com/nowyoucan.html - Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.phpp=sourceforgeCID=DEVDEV ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] Multiplayer chat patch
On Saturday 23 September 2006 18:47, Buchanan, Stuart wrote: Hi All, I've written a patch to the MP code that allows text-based chat within the simulator. It provides the following: 1) Improved MP property handling. The MP code no-longer assumes all properties are floats. Props now turn at the correct RPM, and the MP message format now supports strings and integers. 2) MP chat. Users can type messages that are displayed to all users within MP range. The messages are displayed within a dialog box, and optionally to the screen using the current Nasal message system. This allows the messages to be TTS'd by festival. Currently the patch is intentionally simple. There are a couple of obvious enhancements that could be made once this is committed: A) Frequency specific transmissions. Currently everyone in the MP range hears all transmissions. It would be better to restrict users to transmitting/receiving on their current COM1 frequency. The source code requirements for this are already present in this patch (the /sim/multiplay/transmission-freq-hz property is exported), but the Nasal code is not. B) Quick-phrases. Typing while flying is not particularly efficient. A tree of phrases with click-to-say navigation would be easier to use. The patch is available from http://www.nanjika.co.uk/flightgear/chat.tar.gz We may want to increment the MP protocol version for this, but I'll leave that to other's discretion. As always, comments are welcome. Nice patch! But anyway: You will actually double the amount of memory that is transmitted for one property. While it is good to be able to send different typed properties, I think we can, with the current scheme, store the properties type in the static property configuration where we also store the id to property path mapping. This way we can avoid sending additional data on each property which must be known on both sides anyway. In the longer term, I would like to have a dynamic property path/property type negotiation that is done at the start of a connection. That way we will build up that currently static 'id to path' map at connection start and send only the property values in the fast path. Can you update your patch to avoid transmitting the type? Greetings Mathias - Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.phpp=sourceforgeCID=DEVDEV ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] Multiplayer chat patch
--- Mathias Fröhlich wrote: snip But anyway: You will actually double the amount of memory that is transmitted for one property. That is true. However, transmitting the chat strings is going to represent a much bigger increase in message size, though I've restricted the chat size to 128 characters per message. I should have highlighted this in my original patch just in case we're worried about bandwidth. While it is good to be able to send different typed properties, I think we can, with the current scheme, store the properties type in the static property configuration where we also store the id to property path mapping. This way we can avoid sending additional data on each property which must be known on both sides anyway. Good point. In the longer term, I would like to have a dynamic property path/property type negotiation that is done at the start of a connection. Me too, but re-writing the message format seemed quite sufficient for the moment :) Can you update your patch to avoid transmitting the type? Yes, but I can't provide any guarantee as to when I'll get it done. -Stuart ___ All New Yahoo! Mail Tired of [EMAIL PROTECTED]@! come-ons? Let our SpamGuard protect you. http://uk.docs.yahoo.com/nowyoucan.html - Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.phpp=sourceforgeCID=DEVDEV ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] Multiplayer chat patch
--- Buchanan, Stuart wrote: --- Mathias Fröhlich wrote: snip Can you update your patch to avoid transmitting the type? Yes, but I can't provide any guarantee as to when I'll get it done. Well, whatcha know - it was less work than expected. New version available from http://www.nanjika.co.uk/flightgear/chat.tar.gz -Stuart ___ All new Yahoo! Mail The new Interface is stunning in its simplicity and ease of use. - PC Magazine http://uk.docs.yahoo.com/nowyoucan.html - Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.phpp=sourceforgeCID=DEVDEV ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] Multiplayer chat patch
Hi, On Sunday 24 September 2006 18:19, Buchanan, Stuart wrote: You will actually double the amount of memory that is transmitted for one property. That is true. However, transmitting the chat strings is going to represent a much bigger increase in message size, though I've restricted the chat size to 128 characters per message. I should have highlighted this in my original patch just in case we're worried about bandwidth. Yep, but the chat messages will happen every now and than. The properties will be sent at each step. So this will hurt much more than the chat messages. In the longer term, I would like to have a dynamic property path/property type negotiation that is done at the start of a connection. Me too, but re-writing the message format seemed quite sufficient for the moment :) So, if we just store the type with that map, we do not even need to change the message format. - All old properties are floats. - The old code ignores id's that it does not know about. - The updated code will interpret old style floats correctly. So, in this case there is no need to increase the protocol version. Did I miss something? Can you update your patch to avoid transmitting the type? Yes, but I can't provide any guarantee as to when I'll get it done. Not a problem. Greetings Mathias - Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.phpp=sourceforgeCID=DEVDEV ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
[Flightgear-devel] Multiplayer chat patch
Hi All, I've written a patch to the MP code that allows text-based chat within the simulator. It provides the following: 1) Improved MP property handling. The MP code no-longer assumes all properties are floats. Props now turn at the correct RPM, and the MP message format now supports strings and integers. 2) MP chat. Users can type messages that are displayed to all users within MP range. The messages are displayed within a dialog box, and optionally to the screen using the current Nasal message system. This allows the messages to be TTS'd by festival. Currently the patch is intentionally simple. There are a couple of obvious enhancements that could be made once this is committed: A) Frequency specific transmissions. Currently everyone in the MP range hears all transmissions. It would be better to restrict users to transmitting/receiving on their current COM1 frequency. The source code requirements for this are already present in this patch (the /sim/multiplay/transmission-freq-hz property is exported), but the Nasal code is not. B) Quick-phrases. Typing while flying is not particularly efficient. A tree of phrases with click-to-say navigation would be easier to use. The patch is available from http://www.nanjika.co.uk/flightgear/chat.tar.gz We may want to increment the MP protocol version for this, but I'll leave that to other's discretion. As always, comments are welcome. -Stuart ___ All new Yahoo! Mail The new Interface is stunning in its simplicity and ease of use. - PC Magazine http://uk.docs.yahoo.com/nowyoucan.html - Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.phpp=sourceforgeCID=DEVDEV ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
