move3d

up: forget to put watchers and assignee

A few comments about the commit: Changing the facts architecture

• enums shall be put into namespace or class.
• logger names may be shortened (planners.Facts.FactTypeTranslation => planners.facts.translation)
• VirtualFact::Compute() => compute() (lower case) (idem for VirtualProperty and all reimplemented)
• document contents needed for json files for each type (maybe in both code and README* file in data)
• dirname FactsAndProperties => Facts ?
• remove FactManager::getFactTypeFromObject
• FactManager::getClassFromFactType => factFactory ?
• FactTypeTranslation: do you know what you are doing (singleton implementation with reference) (anyway, need to change stuff IMHO)
• comments to remove
• retrieveAndSet*Parameter => fetchParameters

I'm not done yet for this commit, as for the pullrequest.

Compile, no segfault and work with SST

+1

Checked with François-Jean
+1

The comments for this and #784 are taken into account in the following repos:

/home/magharbi/pull-request/dev/planners
branch: pullReqMam
/home/magharbi/pull-request/dev/studio
branch: pullReqMam

WARN: the history changed, go to actual master (1933ae10ffcea8374aea2f4adf1e7b294adf1721 - DB move files of EntityTypes into data) to pull

refactor: FactManager: computefactFor => computeFactFor
squash 2 last commit together (just before pulling)

the commit "[GTP] bugfix: hatpconnection" introduce 2 cout<<, which should be removed/replaced by M3DLOG messages.

Otherwise seems ok.