From: Alberto V. <va...@di...> - 2008-10-17 08:36:54
|
Hi Brian, thank you for the work. I have checked the robot_params.h and robot_params.cpp. In the .h lacks the bumper_pose_t struct that is required by p2os.cc (version 2.0.5) The parser does not create this struct. We have added to the .h and it compiled correctly. I have checked the patrobol-sh parmas in the files you have sent. I have dettected one mistake. -The fied "char* LaserIgnore", is generated by the parser as "int LaserIgnore". Because in the .p you have: LaserIgnore 73 74 75 -73 -74 -75 Ther parser takes the first value (73) and sets it as a int. If you change in the .cc from int to char* it compiles. But there is a mistake in the data. It shoud not be "73", but "73 74 75 -73 -74 -75". This can be easily solved by changing the .p line to LaserIgnore "73 74 75 -73 -74 -75" --Alberto. El jue, 16-10-2008 a las 09:28 -0700, Brian Gerkey escribió: > On Oct 15, 2008, at 9:14 AM, Alberto Valero wrote: > > > These are the params of all robots from ARIA. We got the robot 6 > > months > > ago. > > > > Please, if you update p2os let us know how to download it. > > Ok, I updated the parser to handle the new files. There were two > issues: > > - There's a new string-type parameter, Map, that wasn't there > before. The parser is pretty brittle, and needs to have hard-coded > the names of all string-type parameters. > > - The .p file can now have "Section" lines that weren't there before. > The parser will ignore them. This shouldn't be a problem, unless the > same parameter name can be reused in different sections. > > The new .cc and .h files are attached. Drop them in player/server/ > drivers/mixed/p2os and 'make; make install'. > > I don't have any Pioneer hardware around these days, so I'd appreciate > if folks would try these new files and report back on whether they > work. Once we have a consensus that the new files work, I'll check > them in. > > brian. > > > > > > El mié, 15-10-2008 a las 08:38 -0700, Brian Gerkey escribió: > >> On Oct 15, 2008, at 6:37 AM, Alberto Valero wrote: > >> > >>> We have just configured player to work with a PatrolBot, > >> > >>> > >>> Connected to UPMDISAM_5020, a Pioneer patrolbot-sh > >>> P2OS: Warning: couldn't find parameters for this robot; using > >>> defaults > >>> > >>> Question: Where can I write the parameters of my robot. > >> > >> The parameters are not settable at runtime. They're hardcoded into > >> the p2os driver (not the best solution, but it's worked well enough > >> so > >> far). The problem here is that the p2os driver hasn't been updated > >> in > >> a while, and so doesn't have parameters for a robot with class/ > >> subclass == Pioneer/patrolbot-sh. > >> > >> There's a parser in player/server/codetools/saphconv that reads in > >> Saphira .p files (MobileRobots sends out robot parameters in these > >> files) and emits a pair of source files: robot_params.cc and > >> robot_params.h. These two files get compiled into the p2os driver. > >> > >> If someone can send the latest .p files for MobileRobots' robots (all > >> of them, not just the PatrolBot), I'll generate updated sources for > >> the p2os driver. Or you can run the parser yourself (look at the > >> README in the parser directory) and send the new sources and I'll > >> check them in. > >> > >>> -The laser readings are inversed in comparison to the laser readings > >>> got > >>> from Player/Stage with a P2AT, which creates confunssion with mi > >>> robot > >>> program. > >> > >> Add this line to the sicklms200 driver block in your .cfg file: > >> invert 1 > >> > >> brian. > >> > >> ------------------------------------------------------------------------- > >> This SF.Net email is sponsored by the Moblin Your Move Developer's > >> challenge > >> Build the coolest Linux based applications with Moblin SDK & win > >> great prizes > >> Grand prize is a trip for two to an Open Source event anywhere in > >> the world > >> http://moblin-contest.org/redirect.php?banner_id=100&url=/ > >> _______________________________________________ > >> Playerstage-users mailing list > >> Pla...@li... > >> https://lists.sourceforge.net/lists/listinfo/playerstage-users > > < > > params > > .tar > > .gz > > > > > ------------------------------------------------------------------------- > > This SF.Net email is sponsored by the Moblin Your Move Developer's > > challenge > > Build the coolest Linux based applications with Moblin SDK & win > > great prizes > > Grand prize is a trip for two to an Open Source event anywhere in > > the world > > http://moblin-contest.org/redirect.php?banner_id=100&url=/_______________________________________________ > > Playerstage-users mailing list > > Pla...@li... > > https://lists.sourceforge.net/lists/listinfo/playerstage-users > > ------------------------------------------------------------------------- > This SF.Net email is sponsored by the Moblin Your Move Developer's challenge > Build the coolest Linux based applications with Moblin SDK & win great prizes > Grand prize is a trip for two to an Open Source event anywhere in the world > http://moblin-contest.org/redirect.php?banner_id=100&url=/ > _______________________________________________ Playerstage-users mailing list Pla...@li... https://lists.sourceforge.net/lists/listinfo/playerstage-users |