Skip to content

Update preset json format + Add device info - #9465

Merged
maloel merged 19 commits into
realsenseai:developmentfrom
Nir-Az:add_camera_source_to_json
Jul 22, 2021
Merged

Update preset json format + Add device info#9465
maloel merged 19 commits into
realsenseai:developmentfrom
Nir-Az:add_camera_source_to_json

Conversation

@Nir-Az

@Nir-Az Nir-Az commented Jul 22, 2021

Copy link
Copy Markdown
Collaborator
  • Change preset json format to SCHEMA_VERSION = 1
  • Reading backward compatible - old style (flat) preset json files is supported

Tracked on [LRS-210]

@Nir-Az
Nir-Az requested review from ev-mp and maloel July 22, 2021 07:19
@Nir-Az
Nir-Az force-pushed the add_camera_source_to_json branch from d1b63d6 to bf3eb5f Compare July 22, 2021 07:36
Comment thread src/ds5/advanced_mode/json_loader.hpp
Comment thread src/ds5/advanced_mode/json_loader.hpp Outdated
}

auto str = j.dump(4);
auto str = preset_writer.get_root().dump(4);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't we have something like a to_string() call for the writer?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good idea 👍

Comment thread src/serialized-utilities.h
Comment thread src/serialized-utilities.h Outdated
Comment thread src/serialized-utilities.h Outdated
Comment thread src/serialized-utilities.h Outdated
Comment thread src/serialized-utilities.h Outdated
Comment thread src/serialized-utilities.h Outdated
Comment thread src/serialized-utilities.cpp Outdated
Comment thread src/serialized-utilities.cpp Outdated
Comment thread src/serialized-utilities.cpp Outdated
Comment thread src/serialized-utilities.cpp
Comment thread src/serialized-utilities.cpp Outdated
Comment thread src/serialized-utilities.cpp Outdated
Comment thread src/ds5/advanced_mode/json_loader.hpp Outdated
Comment thread src/ds5/advanced_mode/json_loader.hpp Outdated
Comment thread src/l500/l500-serializable.cpp Outdated
Comment thread src/l500/l500-serializable.cpp Outdated
Comment thread src/serialized-utilities.cpp Outdated
Comment thread src/serialized-utilities.h Outdated
Comment thread src/serialized-utilities.cpp Outdated
Comment thread src/serialized-utilities.h Outdated
Comment thread src/serialized-utilities.cpp
Comment thread src/serialized-utilities.cpp
@maloel
maloel merged commit e69cd66 into realsenseai:development Jul 22, 2021
@Nir-Az
Nir-Az deleted the add_camera_source_to_json branch July 27, 2021 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants