Bug 517172 - Implement a generic architecture initialization interface
Summary: Implement a generic architecture initialization interface
Status: REOPENED
Alias: None
Product: 4DIAC
Classification: IoT
Component: FORTE (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows NT
: P3 normal
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-05-24 05:57 EDT by Jose Maria Jesus Cabral Lassalle CLA
Modified: 2018-08-24 03:18 EDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jose Maria Jesus Cabral Lassalle CLA 2017-05-24 05:57:06 EDT
Having a generic architecture interface, will make easier to implement initialization and de-initialization of features specifc of each platform
Comment 1 Eclipse Genie CLA 2017-05-24 06:03:35 EDT
New Gerrit change created: https://git.eclipse.org/r/97837
Comment 3 Jose Maria Jesus Cabral Lassalle CLA 2018-08-23 09:47:29 EDT
This is still open. the idea is to have the same interface for all architectures. What has been done in rcX was a test to see if it fits. 

forte_instance.cpp/h files are the proposed interface, and forte_architecture.cpp is the file that has to be implemented for each arch. A unique main file only could then use these files. 

I thought that modules could be also defined in CMake and then generate somehow the initialization, since many of them must be initialized first before using it. A special case arose with ROS in posix, where the module needs a parameter.
Comment 4 Martin Melik-Merkumians CLA 2018-08-23 15:47:50 EDT
So as the last try was done in 2017 according to the bugs gerrit comments, and your comments is this even feasible? Or to ask it in another way, is the effort worth the benefit?
Comment 5 Jose Maria Jesus Cabral Lassalle CLA 2018-08-24 02:54:30 EDT
I like the benefit and I think the effort is not too much, at least for most of the archs, since no new code is necessary, mostly reorganizing. The only thing missing is to define a proper interface in forte_instance. Right now, depending on the architecture, starting forte is different for each one, and when a new platform is developed, this part of the code must be copied. Having the same interface will help specially when forte is used as library. 

The modules idea is another thing, and maybe that could go in another bug, if necessary.
Comment 6 Martin Melik-Merkumians CLA 2018-08-24 02:59:46 EDT
Yes, lets move the modules into a new bug, so that we get manageable chunks. So what's then the proposed timeline for this bug?
Comment 7 Jose Maria Jesus Cabral Lassalle CLA 2018-08-24 03:18:11 EDT
My idea was to have it for 1.10 release. I'd say at least discuss and define a proper interface (if possible) before the release, and if time and human-resource are avaiable, implement it before it.