CERTI - Tasks: task #10290, Implement [part of] IEEE-1516 API
You are not allowed to post comments on this tracker with your current authentication level.
task #10290: Implement [part of] IEEE-1516 API
Submitter: | Eric NOULARD <erk> | ||
Submitted: | Mon 22 Mar 2010 08:13:42 AM UTC | ||
Should Start On: | Sun 21 Mar 2010 11:00:00 PM UTC | Should be Finished on: | Sun 21 Mar 2010 11:00:00 PM UTC |
Category: | IEEE-1516 compliance | Priority: | 5 - Normal |
Status: | Done | Privacy: | Public |
Percent Complete: | 100% | Assigned to: | vyperx |
Open/Closed: | Closed | Planned Release: | CERTI 3.4.0 |
RTI Service Groups: | None |
( Jump to the original submission )
Thu 11 Nov 2010 11:17:24 AM UTC, comment #22: |
Eric NOULARD <erk>![]() |
Mon 31 May 2010 09:38:14 AM UTC, comment #21: I've just checked-in the latest attached patch.
Some remarks concerning modifications I did include myself before check-in:
- try to avoid mixin C style include and [new] C++ style ones.
- size_t is usually unsigned so beware of
- you may use STL algorithm std::for_each
Declare:
template <class T>
then use:
|
Eric NOULARD <erk>![]() |
Sat 29 May 2010 09:27:30 PM UTC, comment #20: Update Jeroen Modif as a patch (with modifications)
|
Eric NOULARD <erk>![]() |
Thu 27 May 2010 12:23:36 PM UTC, comment #19: Hi Jeroen,
Thank you for all the updates.
Concerning the test modules,
I'll have a tight look at the static lib solution.
Alternate logical time may be useful, we'll see
|
Eric NOULARD <erk>![]() |
Thu 27 May 2010 11:54:42 AM UTC, comment #18: Hi Eric,
I've added a new version of our CERTI-branch... I'm afraid we haven't merged any changes you may have made in the mean time, but I hope the overlap will not be that large. The test modules are also still under the CERTI project, so you may want to keep those out when merging.
Additionally, I have attached the overview of the implemented services. Please note that some services are not tested, although this is not indicated in the overview yet.
Also, I have added another LogicalTime implementation to the FedTime1516 library. This is the one that is used in our project (BITS), and was difficult to include for us otherwise. Things like that I think we should better keep as a user-patch to the main CERTI stuff, but it's there now.
Finally, I think I have solved the circular dependency problem by introducing a new static libary (ExportRTI1516), which contains the classes (Exceptions, etc) needed by the FedTime1516 dynamic library.
Kind regards,
Jeroen |
Jeroen van IJperen <vyperx>![]() ![]() |
Tue 25 May 2010 01:48:15 PM UTC, comment #17: Hello Eric,
Things are going quite well on this side. We've completed the RTI1516 services we require and are now moving our original project to make use of CERTI.
We've partially tested the services we've ported, but we won't be testing all of them, because of budget constraints.
I'll fill in the spreadsheet tomorrow, along with a new version of the RTI1516 modules.
During the port of our project, I'm sure we will find some additional stuff in Certi that needs porting/fixing, so we're not completely done yet. :)
After this I'm not sure yet how things will go for me personally (with regard to new projects), but if I have the time, I'll put in some work to 'round off' the work we did for CERTI. (I.e. better responses for the test cases, using correct .cc/.hh files, etc.) |
Jeroen van IJperen <vyperx>![]() ![]() |
Tue 25 May 2010 01:38:18 PM UTC, comment #16: Hi Jeroen,
Maxime is currently working on test case based on
The idea is to use the interactive federate which is kind
I'll ask him to give all of us an
Last message from Maxime (with the spreadsheet attached)
If you tell us where you are in the IEE-1516 service implementation (done or on-going) we can jump to implement
Do you have any updated patch?
|
Eric NOULARD <erk>![]() |
Tue 04 May 2010 12:22:29 PM UTC, comment #15: I have attached table with all IEEE 1516 services.
|
BAUDRAIS <mbaudrai> |
Thu 29 Apr 2010 09:20:20 PM UTC, comment #14: 2) Compiling CERTI with cygwin hasn't been done since a while
see
4) OK. And yes good idea to put a separate zip containing
6) Fine thank you, we'll try to handle/merge those bug fixes
|
Eric NOULARD <erk>![]() |
Thu 29 Apr 2010 08:19:59 AM UTC, comment #13: Hello Eric,
Thanks for the feedback.
2) throw declarations should match - Good call on that. Unfortunatly VS doesn't warn us about that. I'll see if I can easily get GCC to compile it under Cygwin. At least for the warnings...
3) Add messagetypes/enums at end - I can see how that's required for backwards compatibility. I'll do that from now on.
4) Move tests to HLA_TestSuite - For now I think I'll keep them in the 'certi' part. If I'd move them to the 'applications/HLA_TestSuite' they would end up in a different VS solution, meaning I would have to open two VS instances... Near the end of development I will definetly move them there.
5) Commit HLA1516 to CVS - Thanks! I hope people have a chance to experiment a bit with it... Perhaps I can add the RTI1516Test as a seperate Zip here so people can take a look at the tests.
6) fixed bugs Interaction publish/subscribe - Yeah, it would have been better to report that as a bug, with a patch for a fix. I have just uploaded a patch for another bug I found... (waitForMessage deadlock on exception). I'll keep doing that for any bugs I may find.
7) Warnings on handles - It seems the derived handles override the base 'operator=' operator. I actually think the base does not need the 'operator='... I'll take a look into it.
8) Double macro definition in RTI1516fedtime.h - I added that part because the VS 'IntelliSense' kept insisting it was defined as import, while it should export. Removing the macro still allowes it to compile, so I'll remove it.
The merge went fine, so we'll continue development today. :) |
Jeroen van IJperen <vyperx>![]() ![]() |
Wed 28 Apr 2010 06:56:10 PM UTC, comment #12: Hi Jeroen,
That's good work, I did partially checkin
Here comes my more detailed remarks.
1) Beware of the fact that
win32 specific part of code should be surrounded
2) Beware of the fact that header (.h/.hh)
3) When adding and/or modifying message classes
CERTI_Message.msg for M_Classes
try to add your messages at the end of the file
In the same way, please add the concerned
This way you'll avoid to make old enum values move
In fact this is not currently "mandatory" but it will
So please put new messages at the end unless you have to
4) Concerning the RTI1516tests your initiative is very good
However I'd rather put those tests inside HLA_TestSuites
Moreover there is inside HLA_TestSuites some python helper
5) After some Linux compile fix I did run usual and current CERTI HLA 1.3
However I did not checked-in the RTI1516tests.
6) I've seen you have fixed bugs concerning
7) There remains weird compile warnings like:
/home/eric/workspace/certi-integration/libRTI/RTI1516HandleImplementation.h:41: warning: ‘virtual rti1516::HandleImplementation& rti1516::HandleImplementation::operator=(const rti1516::HandleImplementation&)’ was hidden
and I did add a // FIXME Eric-->Jeroen in RTI1516fedtime.h
but we will fix those gradually. I'd rather check-in gradually
I invite you to update (carefully as usual in order not to loose
Again that's good work. |
Eric NOULARD <erk>![]() |
Wed 28 Apr 2010 05:09:55 PM UTC, comment #11: Ok thanks,
Currently testing on Linux. |
Eric NOULARD <erk>![]() |
Wed 28 Apr 2010 12:42:27 PM UTC, comment #10: I've attached a new version of the HLA1516 development... Actually, some test are now broken, but I guess that shows that we're still working on it. :)
We've only done compilation under Visual Studio (2005 & 2008), so please let us know if things aren't working under Linux (and how we can fix them :) ) |
Jeroen van IJperen <vyperx>![]() ![]() |
Tue 27 Apr 2010 07:35:23 AM UTC, comment #9: Hello Eric,
Glad to see you're back. :)
So far, things are progressing quite well, with a lot of the services ported directly to HLA1516...
I've also been working on a unit test system, which already identified some issues in the RTIA/RTIG code... So far, the unit tests still require manual handling, but I'm certain it could be adapted to be automatic, making it more useful in general. |
Jeroen van IJperen <vyperx>![]() ![]() |
Tue 27 Apr 2010 07:28:38 AM UTC, comment #8: Hi Jeroen,
I'm back now :-)
I've fast-tried HLA1516-2010-04-13.zip on Linux and got some
|
Eric NOULARD <erk>![]() |
Thu 15 Apr 2010 01:19:23 PM UTC, comment #7: Thanks for keeping us informed, Eric. :)
Things are going quite well here. Services are being implemented, and I have started working on some unit tests. So far only found bugs in the rtia/rtig, so I guess that's good. :)
Enjoy your trip, and we'll see where we stand in a week. |
Jeroen van IJperen <vyperx>![]() ![]() |
Thu 15 Apr 2010 07:12:19 AM UTC, comment #6: Hi Jeroen,
I won't be very responsive for 7/8 days because I'm travelling
|
Eric NOULARD <erk>![]() |
Mon 12 Apr 2010 11:17:43 AM UTC, comment #5: Hi Eric,
I'd be happy to have some (more) help. :) Let me know if/when he needs any info.
I've uploaded a new patch of my work so far. Unfortunatly I have had to make a 'branch' of some of the HLA1516 header files, in order to get the project compiling under Visual Studio. I've renamed them with a 'certi' prefix, and added an explanation in the file. Still trying to figure out a better way to solve this though.
Right now I'll start working on CERTI being able to parse HLA1516 FOM files. My collegue in the mean time will work on implementing the rti1516::handles and porting the various services. |
Jeroen van IJperen <vyperx>![]() ![]() |
Wed 07 Apr 2010 12:53:03 PM UTC, comment #4: Hi Jeroen,
We (Pierre and myself)
He is currently beginning to learn HLA with CERTI
|
Eric NOULARD <erk>![]() |
Mon 22 Mar 2010 04:57:00 PM UTC, comment #3: Hi Jeroen,
Here is a first feedback on your work.
1) I think you don't have to modify
2) I don't think we need separate RTIA for 1.3 and 1516
There may be 1516/1.3 specific messages but I think
3) libRTI1516
In fact you can modify the existing libRTI/CMakeLists.txt
and then
ADD_LIBRARY(RTI1516 ${RTI_LIB_SRCS} ${RTI_LIB_INCLUDE} ${RTI1516_LIB_SRCS} ${RTI1516_LIB_INCLUDE})
I think you get the idea.
4) Please keep the same naming convention for file extension
C++ header = .hh
5) I did merge-in your modifications of certi.hh
|
Eric NOULARD <erk>![]() |
Mon 22 Mar 2010 09:26:58 AM UTC, comment #2: I needed your Savannah account :-)
As said on the ML, please attach some patches here
Thank you for contributing to CERTI
|
Eric NOULARD <erk>![]() |
Mon 22 Mar 2010 09:09:24 AM UTC, comment #1: Hi Eric,
As indicated on the mailing list, I have taken up this task. Can you assign it to me here? Thanks.
Kind regards,
Jeroen v. IJperen (vyperx)
|
Jeroen van IJperen <vyperx>![]() ![]() |
Mon 22 Mar 2010 08:13:42 AM UTC, original submission:
Many people want to use 1516 API instead of 1.3.
May be more than taht afterwards... |
Eric NOULARD <erk>![]() |
Depends on the following items: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 20 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2010-11-11 | erk | Status | In Progress | ![]() |
Done |
Percent Complete | 70% | ![]() |
100% | ||
Open/Closed | Open | ![]() |
Closed | ||
Planned Release | UnplannedRelease | ![]() |
CERTI 3.4.0 | ||
2010-05-31 | erk | Dependencies | - | ![]() |
bugs #30003 is dependent |
2010-05-31 | erk | Attached File | - | ![]() |
Added task10290-erk-jeroen4-1516only.patch, #20661 |
Percent Complete | 60% | ![]() |
70% | ||
2010-05-30 | erk | Attached File | - | ![]() |
Added task10290-erk-jeroen4.patch, #20654 |
2010-05-29 | erk | Attached File | - | ![]() |
Added task10290-erk-jeroen3.patch, #20652 |
2010-05-27 | erk | Percent Complete | 0% | ![]() |
60% |
2010-05-27 | vyperx | Attached File | - | ![]() |
Added tab2.ods, #20635 |
Attached File | - | ![]() |
Added HLA1516-2010-05-27.zip, #20636 | ||
2010-05-04 | mbaudrai | Attached File | - | ![]() |
Added tab implementation.ods, #20423 |
2010-04-28 | vyperx | Attached File | - | ![]() |
Added HLA1516-2010-04-28.zip, #20358 |
2010-04-12 | vyperx | Attached File | - | ![]() |
Added HLA1516-2010-04-13.zip, #20203 |
2010-04-07 | erk | Carbon-Copy | - | ![]() |
Added mbaudrai |
2010-03-22 | vyperx | Attached File | - | ![]() |
Added HLA1516-2010-03-22.zip, #20003 |
2010-03-22 | erk | Status | None | ![]() |
In Progress |
Assigned to | None | ![]() |
vyperx | ||
Carbon-Copy | - | ![]() |
Added siron |
This work has been integrated into CERTI 3.4.0.