Reorganize source tree

[e117649]

Closes #335 and closes #212

[gxuu]

Why are we no longer using scaler/ as our include directory? For Python examples we had the same thing:

# examples/simple_client.py
from scaler import Client
from scaler.cluster.combo import SchedulerClusterCombo

I think we should keep them consistent. @sharpener6 @rafa-be .

[sharpener6]

Why are we no longer using scaler/ as our include directory? For Python examples we had the same thing:

# examples/simple_client.py
from scaler import Client
from scaler.cluster.combo import SchedulerClusterCombo

I think we should keep them consistent. @sharpener6 @rafa-be .

User should pip install opengris-scaler and run it, not just git clone and run those examples, eventually for all the CI unit tests, should build wheel, and install it then run the tests, not run the tests before making wheels

[rafa-be]

All the Python changes are fine to me.

However, looking at the #include, I'm wondering if we shouldn't add a top scaler/ directory to all C++ files. That is:

src/
| - cpp/
    | - scaler/
        | - ymq/
            | - ...
        | - object_storage/
            | - ...
        | - CMakeLists.txt
| - scaler/
    | - __init__.py
    | - ...

Thus instead of having includes like this:

#include "ymq/logging.h"

We would have something like this:

#include "scaler/ymq/logging.h"

Which matches Python's module structure.

[gxuu]

I get it

[sharpener6]

All the Python changes are fine to me.

However, looking at the #include, I'm wondering if we shouldn't add a top scaler/ directory to all C++ files. That is:

src/
| - cpp/
    | - scaler/
        | - ymq/
            | - ...
        | - object_storage/
            | - ...
        | - CMakeLists.txt
| - scaler/
    | - __init__.py
    | - ...

Thus instead of having includes like this:

#include "ymq/logging.h"

We would have something like this:

#include "scaler/ymq/logging.h"

Which matches Python's module structure.

Agree

[sharpener6]

@e117649 Did you tried to build the wheel and install the wheel and import scaler working?

[e117649]

@e117649 Did you tried to build the wheel and install the wheel and import scaler working?

it is not working, I'll have it fixed in 10 minutes

[e117649]

@e117649 Did you tried to build the wheel and install the wheel and import scaler working?

@sharpener6 yes, importing and using scaler from a built and installed wheel now works, and all the tests running against the source (not built wheel) still work

I also moved src/cpp/scaler/ymq (previously scaler/io/ymq) to src/cpp/scaler/io/ymq to keep in line with matching the python module structure