Skip to content

Tags: magnopus-opensource/connected-spaces-platform

Tags

v5.9.1+572

Toggle v5.9.1+572's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
[OF-1415] feat: change AssetPlatform to DeviceID for SignalR (#615)

* [OF-1415] feat: change AssetPlatform to DeviceID for SignalR

This change affects:
- the HTTP header in SignalRConnection
- the initial URL in EmscriptenSignalRClient

This change does not affect HttpPayload, which still uses AssetPlatform.

v5.9.0+570

Toggle v5.9.0+570's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
CSP 5.9.0 (#611)

* Terraform to deploy CSP docs site

* Upload files to S3

* Use appropriate cert

* Add missing index

* terraform fmt

* Remove unnecessary variables

* Update build dir

* Try this build dir

* Make build_dir a variable

* Use aws_cloudfront_origin_access_control instead of aws_cloudfront_origin_access_identity

* Redirect to /index.html

* redirect-to-https

* default_root_object

* mime_types

* Add CSS MIME type

* response_code = 200

* Invalidate CloudFront cache when files change

* [NT-0] doc: Multiplayer architecture and events (#607)

This change introduces two new sections to the Spaces documentation module:

* Multiplayer Architecture
* Events

Note:

The change also introduces/demonstrates the pattern to be followed should it be helpful to provide the reader with a link to API documentation inline. For examples, please see references to eval_rst in Library\docs\source\learn\spaces\events.md.

* [NT-0] feat!: Add metadata and tag parameters to material creation (#608)

- Add parameters for metadata and tags to the create material method to support being able to filter them out from other types of assets
- Update the relevant tests

---------

Co-authored-by: Ray Saltrelli <ray.saltrelli@magnopus.com>
Co-authored-by: Ray Saltrelli <146846018+raynopus@users.noreply.github.com>
Co-authored-by: Sam Birley <99482500+MAG-SamBirley@users.noreply.github.com>
Co-authored-by: Daniel <100562726+MAG-DanielMurdolo@users.noreply.github.com>

v5.8.1+569

Toggle v5.8.1+569's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Merge pull request #606 from magnopus-opensource/develop

CSP 5.9.0

The new changes are:
- Enable warnings as errors
- Remove unused serialization vector methods
- Only log to console if there is no log callback
- Fix SpaceTransform multiplication
- Emscripten release mode build warning
- Disable error warnings on Android builds
- Add Connected callback
- Let systems register callbacks before multiplayer connection is established
- Updated links to the docs to direct users towards the new URL
- Remove model component tests

v5.8.0+566

Toggle v5.8.0+566's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
CSP 5.8.0 (#590)

* [NT-0] doc: Assets (#576)

This change introduces a new module to the CSP documentation site: 'Assets'.

The module covers the following topics:

Understanding Asset Collections
Asset CRUD Operations
Relationship with Entity Components
Querying for Assets
Automated Model Decimation

* [OF-1437] refac: Introduce a new Event Bus for SignalR Multiplayer events (#533)

- create EventBus class and move event handling to it
- separate NetworkEventsMap into two, one for Systems
  and one for Callbacks
- update the SystemBase base class to handle default
  callbacks and serialization
- make default SystemBase constructor private as it only
  exists to appease the wrapper generator
- move System callbacks and deserializers (OnEvent) to their
  respective System: Systems can register to events by
  passing their instance; clients can register with a
  parameterised callback as before
- update tests to work with the new EventBus
- move event tests from MultiPlayerTests.cpp
  to new EventBusTests.cpp and add test case for the new 
 Callback vs Systems functionality
- add checks for expected log messages to new test
- fix wrapper generator enum parsing

* [OF-1471] test: Integrate multiplayer test runner with Tests project (#575)

* [OF-1471] test: Add tiny-process-lib

Adds tiny-process-lib, for the purposes
of the multiplayer test runner, and
invoking it within the tests.

I laboured over making this a submodule vs
having it directly in thirdparty, and eventually
came to the conclusion that direct thirdparty
is more standard to the codebase.

* [OF-1471] test: Build & Install TPL

Build TPL using cmake during the solution creation.
Copy it to the directory of the tests so its easily
invokable.

We go to the effort to install tiny-process-lib to
an install directory here, which isn't something we
do for other cmake libs, but really we should.

* [OF-1471] test: Dont test MTR by default

During dev it seemed cute to have debug builds
run the tests by default, but it immediately became
frustrating during integration.

* [OF-1471] test: Test Runner Shakedowns

A couple of changes to the test runner that emerged
during integration.

1. I hit the classic problem of calling a private header
in a public one.

2. The process descriptors were too generic,
and I actually accidentlyoutput the same text during
debugging, which caused pain due to
how we monitor stdout to flag futures.

* [OF-1471] test: MTR process manager

Add the process controller object for use in our tests.

Allows setting command line arguments and invoking
the multiplayer test runner, and managing termination
via RAII.

The manager works via listening for set strings on
stdout, and then setting promises to notify the main
test of when events have occured on each process.

* [OF-1471] test: MTR integration test

Add an integration test that launches the
multiplayer test runner and checks that
its operations are performed.

* [OF-1471] test: Unit tests for runner process

Unit tests for the runner process object itself.
Not tests for the runner itself, which has its own tests,
(although they're not run automatically yet / ever)

* [NT-0] doc: updated stack graphic (#581)

Updating the apps/CSP/services stack graphic with a new visual treatment.

* [NT-0] doc: visual treatment for diagrams (#586)

Updated various diagrams throughout the documentation with a new visual treatment.

* [NT-0] style: In hooks, Remove line limitations from both git commit title and git commit body

I am intentionally writing an overly long subject line and overly long message to test that this works.

* [OF-1527] Allow test endpoint to be configurable (#585)

Exposed the services endpoint to be configurable or default to live services if not defined.

* [OF-1495] style: update clang-format (#587)

- Update clang-format to use the WebKit preset style
- Set the number of columns to 150 (as it was before)
- Use the Allman brace breaking style (as it was before)
- Run new clang-format on all *.cpp/*.hpp/*.c/*.h files,
  excluding modules and third parties

* [OF-1492] test: Review all tests and either put them on the CI or delete them (#583)

- Re-enable commented out/disabled C++ tests that pass, and use
  DISABLE_CSPEngine for the tests that fail/that take 40 minutes
- Delete C++ tests that are no longer needed
- Update C++ tests that no longer build
- Add link to story for every C++ test that has one
- Remove all CSharp tests, as these are no longer run nor 
  maintained, except the Wrapper Generator UnitTesting ones, which
  we run on the CI
- Remove all JavaScript tests, as these are no longer run nor
  maintained, except the Wrapper Generator UnitTesting ones, which
  we run as a GitHub Action

* [OF-1441] feat: Video Material Classes (#530)

* [OF-1441] feat: added material classes

* [OF-1441] feat: updated defult values

* [OF-1441] fix: fixed enums for wrapper gen

* [OF-1441] feat: added IsSet for textures

* [OF-1441] style: updated comments

* [OF-1441] feat: adding material base class

* [OF-1441] fix: Set textures when deserializing

* [OF-1440] feat: Material AssetSystem Interface (#547)

* [OF-1441] fix: Set textures when deserializing

* [OF-1440] feat: add asset system interface

* [OF-1440] feat: Add Asset interface for materials

* [OF-1440] test: Add Material tests

* [OF-1440] refac: inline docs + interface update

* [OF-1440] fix: make assetcollectionid unique

* [OF-1440] feat: creating unique asset names

* [OF-1440] fix: Fix GetMaterials + style

* [OF-1440] refac: pr reviews

* [OF-1440] refac: remove const cast

* [OF-1442] feat: Material Overrides (#540)

* [OF-1441] fix: Set textures when deserializing

* [OF-1442] feat: add material overrides

* Update Library/include/CSP/Multiplayer/Components/StaticModelSpaceComponent.h

Co-authored-by: Elliot Morris <Elliot.Morris@magnopus.com>

* [OF-1442] refac: pr reviews

---------

Co-authored-by: Elliot Morris <Elliot.Morris@magnopus.com>

* [OF-1441] fix: name changes

* [OF-1441] refac: update to new csp api

* [OF-1408] fix: update AvatarInfoResult type (#566)

* [NT-0] doc: Physical and Digital Realities (#565)

This change introduces a new module to the CSP documentation site: 'Physical and Digital Realities'.

The module covers the following topics:

Cross-Reality Applications (and digital twins)
Anchoring (with a focus on how to use CSP and Google Cloud Anchors)
Fiducial Markers

* [OB-3643] fix: update callbacks

* Ensured callbacks are now called on main thread

* Revert "[OB-3643] fix: update callbacks"

* [OF-1519] fix: replicated map (#573)

* [OF-1519] fix: update to string map

* [OF-1519] fix: update map serialization

* Map serialization has been changed to use a string map

* [OF-1519] test: add tests

* [OF-1519] refac: remove test vars

* [OF-1519] refac: fix typo

* [OF-1519] test: update tests

* [OF-1441] fix: main thread crash

* [OF-1441] fix: remove pthread proxy

* [OF-1441] fix: wasm crash (#578)

* [OF-1441] fix: wasm crash

* [OF-1441] refac: simplify material downloading

* [OF-1518] feat: add material change event (#580)

* [OF-1441] doc: update comment

* [OF-1441] fix: elegantly fail when error (#582)

* [OF-1441] fix: elegantly fail when error

* [OF-1441] doc: Add comment

* [OF-1441] fix: update to new login method

* [OF-1441] test: update to new login

---------

Co-authored-by: Elliot Morris <Elliot.Morris@magnopus.com>
Co-authored-by: Sam Birley <99482500+MAG-SamBirley@users.noreply.github.com>

---------

Co-authored-by: Sam Birley <99482500+MAG-SamBirley@users.noreply.github.com>
Co-authored-by: MAG-OR <185054971+MAG-OR@users.noreply.github.com>
Co-authored-by: Elliot Morris <Elliot.Morris@magnopus.com>
Co-authored-by: MAG-ChristopherAtkinson <christopher.atkinson@magnopus.com>

v5.7.0+565

Toggle v5.7.0+565's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Merge pull request #579 from magnopus-opensource/develop

CSP 5.7.0

v5.6.0+564

Toggle v5.6.0+564's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
CSP 5.6.0 (#567)

* [OF-1468] test: MultiplayerTestRunner-Add Project

Add the premake project for the multiplayer test runner.
Link googletest and CSP, linking CSP in place, expecting it
to be where the solution puts it.

* [OF-1468] test: MultiplayerTestRunner-Main

Add main file.
Takes CLI input, logs in, creates and joins a space,
then runs the selected test.

* [OF-1468] test: Add example test

Add the example avatar test.
This test isn't designed to actually do anything useful,
just to define the structure of how to add your own
tests later on down the line.

* [OF-1468] test:  Errorcodes & Process Descriptors

Add the public headers concerned with handling result communication.
Calling processes may use these headers to see what the errors coming
out of the code mean, and can compare what's coming out of stdout
to the process descriptors.

* [OF-1468] test: Add CLI11 dependency

Adds thirdparty headeronly dependency on CLI11

* [OF-1468] test: Add uuid_v4 dependency

Add header only thirdparty dependency to uuid_v4.
Also requires endianness file, from the same repo.

* [OF-1468] test: Utils

Add general utilities file.
If the project was bigger, these could have justified their own
categories, but I think this if fine for this scale.

* [OF-1468] test: CLI handling

Create the CLI and test it
Arguments taken from the CLI are validated, and converted to
RunnerSettings, which is the main configuration object used
about the program.

* [OF-1468] test: Login behaviour

Add Login RAII behaviour and tests.

* [OF-1468] test: Space behaviour

Add RAII type to manage space creation and joining.

* [OF-1468] test: Add copyright blocks

Add copyright blocks to every file, since I forgot.

* [NT-0] doc: removing references to the GH wiki from the readme (#562)

This change replaces references to GitHub wiki pages with CSP API docs links. 

It also restructures the readme to flow better from a first-time reader's point of view, and now includes the CSP logo instead of the emoji-based header.

* [NT-0] fix: Workaround CI std::pair comparisons

Our teamcity runners seem to be rejecting the std::pair comparison,
even though they work fine locally.
Just workaround it to unblock things.

* Asset: make ThirdParty properties public, remove methods (#561)

* Asset: make ThirdParty properties public, remove methods

* Update tests to remove method for Asset ThirdParty*

* Update tests to remove method for Asset ThirdParty*

* Fix misspelling in test replaced code

* Use variables to store identifier string in tests

* [OF-1408] fix: update AvatarInfoResult type (#566)

---------

Co-authored-by: Elliot Morris <Elliot.Morris@magnopus.com>
Co-authored-by: Sam Birley <99482500+MAG-SamBirley@users.noreply.github.com>
Co-authored-by: mag-richardzirbes <100873114+mag-richardzirbes@users.noreply.github.com>

v5.5.2+562

Toggle v5.5.2+562's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Merge pull request #560 from magnopus-opensource/develop

CSP 5.5.2

v5.5.1+561

Toggle v5.5.1+561's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
CSP 5.5.1 (#546)

* [OF-1477] test: Disable CopyAssetCollectionTest

This test is broken due to a presumed change in CHS.
We are disabling it to unblock other PRs.
Ticket linked in the title is the ticket to re-enable.

* [OB-3906] fix: incorrect reading of components (#544)

v5.5.0+553

Toggle v5.5.0+553's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
CSP 5.5.0 (#539)

* [OF-1432] refac: remove sorted hierarchy feature (#526)

* Removed the use of Sequence Hierarchy
 * Removed the use of Sequence Hierarchy Tests
 * Removed event parameter and deserialization

Co-authored-by: MAG-ChristopherAtkinson <Christopher.Atkinson@magnopus.global>
Co-authored-by: MAG-mv <109593178+MAG-mv@users.noreply.github.com>

* [OF-1455] feat: Set tags on space metadata (#529)

- Adding support for setting and updating tags on the space metadata prototype
- This is so we can use it to identify types of spaces using the first class property from the cloud services endpoint

* [OF-1459] fix: update cinematic camera GetFov() to match aspect ratio (#537)

* [OF-1456] feat: Children now reparent after parent deletion (#538)

* [OF-1456] feat: children now reparent correctly

* Update Tests/src/PublicAPITests/MultiPlayerTests.cpp

Co-authored-by: Elliot Morris <Elliot.Morris@magnopus.com>

* Update Tests/src/PublicAPITests/MultiPlayerTests.cpp

Co-authored-by: Elliot Morris <Elliot.Morris@magnopus.com>

* [OF-1456] fix: updated CreateSpace call

* [OF-1456] fix: remove define

---------

Co-authored-by: Elliot Morris <Elliot.Morris@magnopus.com>

---------

Co-authored-by: MAG-ChristopherAtkinson <christopher.atkinson@magnopus.com>
Co-authored-by: MAG-ChristopherAtkinson <Christopher.Atkinson@magnopus.global>
Co-authored-by: Daniel <100562726+MAG-DanielMurdolo@users.noreply.github.com>
Co-authored-by: MAG-AdrianMeredith <100127317+MAG-AdrianMeredith@users.noreply.github.com>
Co-authored-by: Elliot Morris <Elliot.Morris@magnopus.com>

v5.4.0+552

Toggle v5.4.0+552's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Merge pull request #536 from magnopus-opensource/develop

CSP 5.4.0 - fix to build script required.