Some tests show a loading time decrease by a factor 3 to 4.

We need to access a private data to optimize geojson loading, hence
that fork containing a very small patch (a 'private' data demoted
to 'protected').

This will ease the change from jsnocpp to tao/cpp

This avoids the build of other-src/ to use header files from src/

Each Makefile.am now has to set its 'local' inclusion directive.
For instance, in src/Makefile.am:

AM_CXXFLAGS = @MY_CXXFLAGS@ -iquote ${top_srcdir}/src

Dauster Guillaume authored 3 years ago and sdegrande committed 2 years ago

Some Feature's properties are handled, such as the type of the
element, its height...

A new MultiMeshes 3d object is added, to 'pack' several meshes into
one single OpenGL buffer. It is needed to fast-render the huge numbers
of buildings that can be generated from a GeoJson file...

A MeshContainer is also added. It internally handles a vector of
MultiMeshes, automatically allocating a new MultiMeshes if needed when
a new mesh is added to the container.

Note: this commit is the result of a work from Paul-Elian Tabarant
and Guillaume Dauster. The individual commits comes from two separate
git repos, hence this 'merged' single commit.

The AssetManager's ctor is called during the global initialization,
due to uses of AssetProviderImpl() in order to register some
additional asset dirs.

In that ctor, logstreams is used to output some infos.

The default initialization of some logstreams's members seems to be
different with clang64, and so the use of logstreams crashes.

This patch transfers all the code from the ctor into a new
AssetManager::init() function, called by main().

In the same way, the actual registration of the asset dirs is defered
into AssetManager::init().

Since C++17, std::iterator is deprecated.
We now have to define 5 type aliases used by iterators.

See https://www.fluentcpp.com/2018/05/08/std-iterator-deprecated/, for
instance.

A new cmdline parameter was added to VAirDraw in a previous commit
(7fa36921). This is currently the only way to know which asset dirs
are known by VAirDraw, in order to fill the launcher panels with
the right palettes, sceneries, configs...

This patch uses that VAirDraw command line parameter to set the @ALL
setup data, instead of "hardcoding" it.

It currently displays the VAirDraw version, the list of the registered
assets dirs, and terminates the execution.

Use std::find_if, instead of a loop.
Add 'const' to some parameters.

A parallel execution of std::reduce() was unintentionally left in the
code.

Enhance readability.

Use it to register VAirDraw's Application and the GeoJson loader
as asset providers.

Implement a 'comutative and associative' accum functor, to reuse
std:reduce() instead of std::accumulate().

TODO: detect if LLVM libc++ or GNU libstd++ is used, so that we can
use std::reduce() with a parallel execution policy.

The binaryop of std::reduce, without the parallel execution policy,
must be able to be commutative, and so accept parameters in any order.

Our accum binaryop is not commutative, so we get back to using
std::accumulate.

The parallelism specification is not yet implemented in LLVM libc++,
and gcc on mingw64 complains about a deprecation.

Generation of vairdraw.pdb must also be commented out in
src/Makefile.am

We intend to remove it, and use clang-ld to generate the .pdb file
Meanwhile, we comment out the use of cv2pdb in configure.ac

Deep change in AssetManager on the way to define the directories to
look for an asset.

Let's call 'topDir' the directory containing the VAirDraw's assets/
subdir.

3 assets dirs are hardcoded (just as before this commit):
topDir, topDir/assets and topDir/extra-assets.

Applications and Modules can now also register their own assets dir,
using AssetManager::regiterAssetDir(path, priority).

Six levels of look-for priority are defined (1 being the highest
priority):
 1 : for Apps assets
 2 : for Modules assets
 3 : for local assets (the new local asset dir)
 4 : for distrib assets (VAirDraw's assets/)
 5 : for extra-assets
 6 : reserved for hard-coded dirs

Add a GUI to set a local asset dir in the VAirDraw launcher panel.
The content of the panels is now extracted from the following
directories (see the '@ALL' option in Setup.cpp):
  $TOP/src/$APP_DIR/assets,
  $LOCAL_ASSET,
  $TOP/assets,
  $TOP/extra-assets

(Modules assets are still to be added).

A check box is also added, so that only the local asset dir is
used to fill the content of the panels.

The local-asset-dir is written into the config_local.json file before
to launch VAirDraw.

NOTE : the launcher-app.ini MUST BE REMOVED, to be regenerated, due to
changes in the Setup's options.

This is in preparation to the creation of an Installer.
That installer would potentially put the launcher's assets in the
top install dir.
And so we need to differentiate them with the VAirDraw's assets.

For the CoFadeVR apps, externals/libhttpserver is build, and
during its build rule, autoconf is called.

On the mingw64 VM, the installed autoconf is newer than on Fedora,
and it generates warnings.

Those warnings are captured by the 'check warning' job, which
consequently fails.

TODO: we should find a way to call the libhttpserver's configure from
the whole project's configure, and not from a make rule...

When running the build with "make -j N", a warning is generated:
‘warning: jobserver unavailable: using -j1. Add `+' to parent make rule.’

This fix consists in replacing a call to "make" in a Makefile by
a call to "$(MAKE)", so that is '-j' parameter is correctly passed to
sub-makes.

See https://www.gnu.org/software/make/manual/html_node/Error-Messages.html
for an explanation.

If docpad is not found, a warning is generated, caught by the CI
pipeline (we do not make the doc on the current pipeline).

We usually avoid parallel build on CI, but the Windows VM is much
too slow !

It is only built if that module is requested by an App in its
configure_app.m4, with :

AS_VAR_SET([WITH_EXTERNAL_LIBHTTPSERVER], [true])

CXXFLAGS and LIBS are 'automatically' set accordingly.

This type of controller used with the HP Reverb G1 is now supported in VR, with the proper bindings.

The supposedly unused 'needExpansion' structure member is actually
used in Setup::ReadSting()...

'Polygon' is a function defined in wingdi.h, colliding our
SceneGraph::Polygon.

We have to explicitly use the 'SceneGraph::' namespace to help the
compiler knows which 'Polygon' to use.

If a scene can not be loaded, a nullptr is returned and its insertion
into the mainScene crashes VAirDraw.

This patch prevents to add a nullptr node, and notifies about a
potential parsing error.

Apply a factor 10 to each coordinate, to get some "more plausible"
values.