winrt: Update `VulkanRT-License.txt` text

Change-Id: I7bcd0962fa970323f34ea501148c0f5b120541a0

vulkaninfo: safely handle device groups not supported

Previously, vulkaninfo would hard crash if device groups failed
the call to vkEnumeratePhysicalDeviceGroupsKHR. If an error happens
now the GetGroups call simply returns an empty vector.

Files Modified:
vulkaninfo/vulkaninfo.cpp
vulkaninfo/vulkaninfo.h

Change-Id: I60ed002161523cede53a144be3fad1d4bb0dd1f1

build: Update glslang version in known_good.json

Change-Id: I5b4995f62c61c8f7a1492df6c2d20c8bf9887079

build: update vulkaninfo.hpp to fix windows build warnings

The PR to reduce spurious warnings while using MSVC didn't include
updating vulkaninfo.hpp. This commit runs generate_vulkaninfo_hpp
to get the most up to date version.

Change-Id: I1ff352bce121872a72002d6650ed38b330d4a61f

build: Update to verson 1.1.130 of the Vulkan Headers

- update desktop and Android known-good json files
- regenerated prebuilt sources

Change-Id: Id2bc14068a66714709f7c503f7866738d90f54d8

vulkaninfo: reduced spurious warnings for msvc

This commit removes several spurious wanings msvc would display,
like implicit conversions for VkBool32 to bool and several
signed/unsigned comparisons.

Change-Id: Id485c044575105ec52fd49f9f0c5881674841a8e

vulkaninfo: Add VK_EXT_metal_surface info

Added functions to create a Metal Surface and report its
information.

Change-Id: Ic94b00fd6c083bd7b852e3cee9f11601d0fa1675

windows: Change winrt name

Changed the name of the `winrt` directory to
`windows-runtime-installer` as suggested in
issue #134

Change-Id: I7cc29c78444c30bece214dc9ec8a3d5b4b6728b2

vulkaninfo: load user32.dll dynamically

Instead of depending on user32.dll, this library is
now loaded dynamically.

This change is needed to redistribute vulkaninfo.exe
as a part of "universal" windows driverm, by making
vulkaninfo pass the 'ApiValidator' test.

See more here https://docs.microsoft.com/en-us/windows-hardware/drivers/develop/validating-universal-drivers

Change-Id: I79a28cbb5d761b33ab673994a2b578c4044f5a90
Author: Kamil Szypryt <kamil.szypryt@intel.com>

Revert "cube: Port Wayland impl from wl-shell to xdg-shell"

This reverts commit 6f6e3371c92ac63bf29d5010b2933659e3f8dbef.

We've received complaints that this change is incompatible with
Wayland users running the Weston compositor

Change-Id: Icd0c96a7726969570655b886d8fb623a04543985

vkcube: Replace MoltenVK surfaces with Metal

Made DemoViewController.m compatible with VK_EXT_metal_surface
extension and replaced all references
to VK_MVK_macos_surface and VK_MVK_ios_surface with
VK_EXT_metal_surface.

Change-Id: I4a163bc9c6109b44d169c6565313ab52161d38fc

vkcube: Fix Cube Squishing on Resize

Modified viewport creation in vkcube to keep the
aspect ratio square and centered.

This change prevents cube from squishing and warping
when the window is resized.

It now shrinks and grows.

Change-Id: Ie2b3c5747293dfed968af5d83b581ac74d81bb10

cube: Port Wayland impl from wl-shell to xdg-shell

This change ports vkcube in both C and C++ versions to use the stable
xdg-shell protocol for Wayland compositors. The original wl-shell
protocol has been deprecated and is being removed from major compositor
libraries; wlroots has already dropped support.

New cmake modules have been added to look for Wayland-Protocols
(containing XML descriptions of all common protocols) and
`wayland-scanner`, the tool used to convert the XML files to a usable C
interface.

The change also adds support for the xdg-decoration protocol, which for
some compositors is provided and needed to let them know that they
should draw a titlebar, borders with resize controls, and other standard
features.

Change-Id: I39bedda93a7c5d0aeeb59c68023552723b413567

build: Update known good for 1.1.128 Vulkan header

Changes:
- Updated known good files
- Generated new files using `generate_source.py`

Updated:
- `build-android/vulkan-headers_revision_android`
- `icd/generated/mock_icd.cpp`
- `icd/generated/mock_icd.h`
- `icd/generated/vk_typemap_helper.h`
- `scripts/known_good.json`
- `vulkaninfo/generated/vulkaninfo.hpp`

Change-Id: I03c9daed1137f5ee2ac0a553970de55ad592dc69

vulkaninfo: check that extensions aren't disabled

Previous versions of vulkainfo's autogen code didn't check if enums and
bitmasks from extensions were disabled, possibly letting them leak into
the generated code. This commit checks and skips any fields which aren't
supported.

File changed:
scripts/vulkaninfo_generator.py

Change-Id: I0e6fd9872d9377a17797cfcf6c2fbcee649e3657

vulkaninfo: return properly on xcb onnection errors

Calling xcb_disconnect on the connection to release memory and
return.

Otherwise on a headless display with no X11 backend the nullptr
for the connection will cause segmentation fault on following
xcb calls when xcb_connection_has_error returns a code.

examples ran on headless displays

DISPLAY=:0 vulkaninfo returns XCB_CONN_ERROR
DISPLAY=foobar vulkaninfo returns XCB_CONN_CLOSED_PARSE_ERR

v2:restore xcb_connection=nullptr as other xcb calls need it when
error happened.

Signed-off-by: Daniel Charles <daniel.charles@intel.com>

build: Update known good for 1.1.127 Vulkan header

Changes:
- Updated known good files
- Generated new files using `generate_source.py`

Updated:
- `icd/generated/mock_icd.cpp`
- `icd/generated/mock_icd.h`
- `icd/generated/vk_typemap_helper.h`
- `vulkaninfo/generated/vulkaninfo.hpp`

Change-Id: Iac0b0d48ee0efc5d8e2d0db0a17b71bcb5e69388

build: Update MoltenVK version to v1.0.38

Change-Id: Idca07170a7ff2f7a6cf163ef9ab814cffc9ae5ea

vulkaninfo: added more documentation for show-formats

The indented behavior was to only show format properties if requested
for text output and always show them for html & json. This commit
modifies the --help output and documentation to better reflect this
behavior.

files modified
 - vulkaninfo/vulkaninfo.cpp
 - vulkaninfo/vulkaninfo.md

Change-Id: I5e35b8af2db74c37dfadf669ffc81e69304fc4e0

vulkaninfo: Fix gitlab issue 181

Fixes vulkaninfo.app bundle crashing on startup.

Change-Id: I515611c28da35a0f55edcb8492d26ad78046c0c2

vulkaninfo: fix infinite sleep for non text outputs

Previous versions, in win32 would sleep infinitely if a crash occured. This commit will first check if its a text output first before sleeping, to preserve the error if it is in a console but return immediate if it isn't.

Change-Id: Ib1b43001e970c8c3114e666d96b24dc3c557a579

vulkaninfo: fixed spelling of properties

Change-Id: I4cd38c06842c99b514b4f289f7856eb810c111f3

gn: Generate VkICD_mock_icd.json on mac

vulkaninfo: add documentation for show-formats

Change-Id: I5b1aa1bec4daa1e7b2d8a47633011ae7c800b372

vulkaninfo: add documentation for --show-formats

A recent PR was merged that made format properties not shown by
default in the text output. This documents the change in the options
list of the executable.

files changed
 - vulkaninfo/vulkaninfo.cpp

Change-Id: I1db65f8c9fae31dd047856d0f0d483a304405dd2

build: Update known-good commits for SDK 126

Change-Id: I261460fc7612a828cd4d9848a9bcf43c2f2dba41

build: Update known good for 1.1.126 Vulkan header

Changes:
- Updated known good files
- Generated new files using `generate_source.py`

Updated:
- `build-android/vulkan-headers_revision_android`
- `icd/generated/mock_icd.cpp`
- `icd/generated/mock_icd.cpp`
- `scripts/known_good.json`

Change-Id: I1ef8b689d76c76822b2dab8ef7fd138bb46f7314

cube: Try to bind to wl_compositor v4

VK_KHR_incremental_present requires the client to
bind to wl_compositor v4. This is because internally
some Vulkan Wayland drivers convert VkPresentRegionKHR
to wl_surface.damage_buffer()[1], which is a v4-only
request.

If the Wayland compositor advertises wl_compositor v4,
we can bind to it and make incremental present actually
work.

[1] https://gitlab.freedesktop.org/mesa/mesa/blob/master/src/vulkan/wsi/wsi_common_wayland.c#L862

build: Update known good for 1.1.124 Vulkan header

Changes:
- Updated known good files
- Generated new files using `generate_source.py`

Updated:
- `build-android/vulkan-headers_revision_android`
- `icd/generated/mock_icd.cpp`
- `icd/generated/mock_icd.h`
- `icd/generated/vk_typemap_helper.h`
- `scripts/known_good.json`
- `vulkaninfo/generated/vulkaninfo.hpp`

Change-Id: Ib6c35389471d8cbdaa60608f76e563ad50271a53

vulkaninfo: Present Surfaces now wont show duplicates

If a machine has multiple surface types that work with the
same gpu and have the same properties, vulkainfo will now
list them in a short array instead of duplicating the entire
surfaces information. To do this required being able to compare
vulkan objects, necesitating the autogeneration of comparators
for vulkan structs. This patch also refactors some of the
device creation code to better reflect the actual data
dependencies of the various vulkan objects.

files modified
 - vulkaninfo/vulkaninfo.cpp
 - vulkaninfo/vulkaninfo.h
 - scripts/vulkaninfo_generator.py
 - vulkaninfo/generated/vulkaninfo.hpp

Change-Id: Ifda51eacc3883982a6e2f7f60439827eb355c204

vulkaninfo: Added asserts to check output

Several places in the output code of vulkaninfo had preconditions,
like the indent level being zero or positive, but had no asserts
checking the value. This commit adds such defensive measures, as well
as a reordering IndentIncrease to be correct in regards to keeping
indents zero or above.

files modified
 - vulkaninfo/outputprinter.h
 - vulkaninfo/vulkaninfo.cpp

Change-Id: Iafa03193407229d345f550c4c4c3a9dd1e235919

cube: Revert 1.1.114 SDK workaround for vkcube

During the release of the 1.1.114 MacOS SDK a workaround
for vkcube was needed to get it functioning in time for release.

The underlying problem has since been fixed and this workaround
is not longer necessary.

Change-Id: I3ad3d1bafa44cfbc158820025bb22757d6e5683d

vulkaninfo: updated json documentation

Currently, the json option doesn't specify that it only outputs for one
gpu and that it follows a schema. This rectifies that.

file changed
 - vulkaninfo/vulkaninfo.cpp

Change-Id: I3af8d559b2464d7ce00ce21e58a1c3054c0740b4

build: Fix Vulkaninfo Bundle build on MacOS

Fixes MacOS crash while building the Vulkaninfo.app bundle.

Change-Id: Ic1e3c4cef7de5d9458f4d0fc056093abb352bf1a

vulkaninfo: Fix closing immediately on windows

On windows the console window will immediately close after running
vulkaninfo, this fixes it.

Change-Id: Ic5a86ffb89508d874d9609ed1b4d9222a31798c4

vulkaninfo: Handle device group extension not available

Previously if the device didn't support VK_KHR_device_group vulkaninfo
would not catch it and subsequently crash. This commit adds error
checking and bool flag to GetGroupCapabilities for callers to know
if it was successful.

files modified:
 - vulkaninfo/vulkaninfo.cpp
 - vulkaninfo/vulkaninfo.h

Change-Id: Ie8fe9cc46ebe0fa6cae9985d89628d60ddf58cf2

vulkaninfo: fix extra newlines

A slight oversight in the output meant there were extra newlines
in pNext chain traversal.
This makes sure only one newline is printer per item in the chain.

files modified:
 - vulkaninfo/generated/vulkaninfo.hpp
 - scripts/vulkaninfo_generator.py

Change-Id: I3b3cac55c3e90d0b3f59e70fab9c4f56473c6dd6

vulkaninfo: Major Refactor

Rewrote vulkan info to use C++ and include autogen capabilities
Properties and Features are now populated based on the xml spec
Improved readability and robustness by seperating vulkan
info gathering, formatting, and printing into seperate files

Files Added:
 - scripts/generate_vulkaninfo.py
 - vulkaninfo/vulkaninfo.h
 - vulkaninfo/vulkaninfo.cpp
 - vulkaninfo/outputprinter.h
 - vulkaninfo/generated/vulkaninfo.hpp
 - vulkaninfo/generated/.clang-format

Files Modified:
 - scripts/kvt_genvk.py
 - vulkaninfo/CMakeLists.txt
 - vulkaninfo/macOS/vulkaninfo.cmake

Files Removed:
 - vulkaninfo/vulkaninfo.c

Change-Id: I8042a6a7949595b75f03d1f3ba48e221d2cc0e8a

build: Update MoltenVK version in known_good.json

Updated MoltenVK version from:
v1.0.36 -> v1.0.37

build: Update known good for 1.1.123 vulkan header

Changes:
- Updated known good files
- Generated new files using `generate_source.py`

Updated:
- `build-android/vulkan-headers_revision_android`
- `icd/generated/mock_icd.cpp`
- `icd/generated/mock_icd.h`
- `icd/generated/vk_typemap_helper.h`
- `scripts/known_good.json`

Change-Id: Ic634680499c375fd29ee3d0816fee33aafa3d44f

cube: Handle SURFACE_LOST return code

Change-Id: I6989c7da3e36ec8531ff390ca9ee2f9710776f11

icd: Fix power operator in VkPhysicalDeviceLimits

Two bugs here:
- ^ (xor) was being used as a power operator
- maxPerStageResources and maxDescriptorSet* limits were mistaking
  footnote notation in the spec for power notation

build: Update glslang version in known-good.json

Change-Id: If9b5814aa0b2f03ae71f23f2fbb1d152995a681b

vulkaninfo: Fix Issue 227

Fixes issue #227 where vulkaninfo on MacOS would
seemingly repeat the same 4 formats 8 times each.

This was simply the function AppDumpSurfaceFormats listing
only the VkFormat attribute from VkSurfaceFormatKHR and not
the VkColorSpaceKHR attribute.

Change-Id: Idba81a69ef317f1d546856b7acb86e8540565d2c

build: Fix known-good MoltenVK building

MoltenVK seems to have trouble building on the
current version of Vulkan-Headers.

Since Vulkan-Headers doesn't take very long to build and
changes often, this fix removes the reuse of the
Vulkan-Headers repo, whose version is provided by Vulkan-Tools
known-good.json, in the MoltenVK build.

It instead uses the version of Vulkan-Headers provided by
the MoltenVK known-good when building MoltenVK.

**Note:** Vulkan-Tools and other required repos are still
built with the version of Vulkan-Headers provided by
Vulkan-Tools known-good.json.

Change-Id: I1b11122cf0c6ba488acd53a35e4755f00e48376c

vulkaninfo: Support for VK_EXT_full_screen_exclusive

Change-Id: Id3d0dfff6f2efff42a79630b096cdd2829d00e40

vulkaninfo: support for index_type_uint8_ext

Change-Id: I649b3972774907600e24508447f19505010bcc87

vulkaninfo: support VK_EXT_subgroup_size_control

Change-Id: Ib51bdff3db00a3bbc66cc2926599890f895d56c2

build: Update known good for 1.1.121 vulkan header

Changes:
- Updated known good files
- Generated new files using `generate_source.py`

Updated:
- `build-android/vulkan-headers_revision_android`
- `icd/generated/mock_icd.cpp`
- `icd/generated/mock_icd.h`
- `icd/generated/vk_typemap_helper.h`
- `scripts/known_good.json`

Change-Id: Id5022fa1d82ae090cd5d50e7937274e93587163e

build: Update known good for 1.1.120 vulkan header

Changes:
- Updated known good files
- Generated new files using `generate_source.py`

Updated:
- `build-android/vulkan-headers_revision_android
- `icd/generated/mock_icd.h`
- `scripts/known_good.json`

Change-Id: Ia83e6bc8acd0c54d23e97e9c645d0062014f4840

docs: Bump minimums to Ubuntu 16.04 and gcc 5.4.0

Change-Id: I58588d900c89f3247be423857e861ad278d6b9c0

vulkaninfo: fix memory leaks from pNext chains

Change-Id: I99965dabaf7307e52c56f779d79f259fc379c7da

build: Update known-good for 1.1.119 header

Updated known-good and generated source code for this repository.

Modified:
- `build-android/vulkan-headers_revision_android`
- `icd/generated/mock_icd.cpp`
- `icd/generated/mock_icd.h`
- `icd/generated/vk_typemap_helper.h`
- `scripts/known_good.json`

Change-Id: I10fded5bb6c35a7fc38acdbc23d097ae762f5112

vulkaninfo: Update output for float controls

"separate" struct members were renamed and changed from a boolean to an
enum to accommodate a wider variety of hardware implementations.

build: Update Appveyor to VS 2015

Change-Id: I67534f37232f148b955a8860d41408c0fc4da5af

vulkaninfo: Support for ShaderDemoteToHelperInvocationExt

Change-Id: I5188ffd477d16a3af8d48c69dc411feee76b1f7f

fixed double tab for DepthStencilResolveProperties

Change-Id: I51a1b711c3e4d79695cf79bdc162860d7b39d694

Fixed double tabs in text output

Change-Id: I591906091fd39e5a44e83849cb9f2d1ae36d1ead

vulkaninfo: support for texel_buffer_alingment_ext

Change-Id: I424862d61714b93462bc847c63ae95d6716da154

build: Remove Windows loader build dependency

On Windows, the loader now has a dependency on the Windows Driver Kit.
In order to avoid adding that dependency here, this change includes
loader .lib files for building on Windows.

Change-Id: I3119caccf7e4e9ba0666a9ba29413137696775ec

fixed FragmentShaderInterlock missing </details> for html

Change-Id: Ica0bddc06cb98507f61fe7699d1ef11e74fadfd9

vulkaninfo: support for imageless_framebuffer_khr

Change-Id: Ia133befc18201c3bf05238653356cdaa0c162bf9

vulkaninfo: VK_EXT_fragment_shader_interlock

Adds support for the extension

Change-Id: Ifc18acbdeae3b1b5b568e9a3ddaffb444d14ef86

vulkaninfo: fix forgotten </details> tag

Fixes incorrect html output from a forgotten html closing tag.

Change-Id: I3e731dafc0b191dfaa71426a7a88b6ccd28477c2

vulkaninfo: VK_KHR_uniform_buffer_standard_layout

Implements the extension in vulkaninfo

Change-Id: I67ec93c678f0537ea00222bdd35e1e18b28c85d1

build: Update MoltenVK version in known-good.json

Updated MoltenVK version in known-good.json from
v1.0.35 -> v1.0.36

Change-Id: I1b3762d2234ee7da2acc45f081f56aa6e7cfd504

cube: Workaround macOS cube issue

Latest version of MoltenVK behaves in an apparently non-conformant way
by not allowing some linear images to be mapped to memory with
VK_MEMORY_PROPERTY_HOST_COHERENT_BIT.  This change sidesteps the issue
by forcing the use of staging buffer for loading textures.

winrt: Correct `VulkanRT-License.txt` typo

Change-Id: I26c1fbdddc8d6f6fc344164de2cc7cfa74c08919

doc: Update docs with codegen instructions

CI: Add codegen consistency check

build: CMake/GN changes for pre-generated source

icd: Add generated source to repo

scripts: Port codegen script from VVL

scripts: Fix mock_icd_generator syntax warnings

build: Update glslang commit ID in known-good

Change-Id: I7d1031909f91dc14471d41e33bf50bafffaa7c58

scripts: Add required depth/stencil resolve props

Spec mandates that at least VK_RESOLVE_MODE_SAMPLE_ZERO_BIT_KHR be
supported for supportedDepthResolveModes and
supportedStencilResolveModes

build: Update known-good files for 1.1.114 header

Change-Id: I8dcbd49e7ccc07f0c891d9c303e1dc0d8445f9e8

build: Update cmake_minimum_required(VERSION 3.10.2)

Change-Id: I193d23da109a81e0b385a3d9fc54a6d266b6d026

vulkaninfo: Add descriptor indexing feats/props

Change-Id: I7863fb44fbc1e2c7f7833580799db6b8fb229afe

docs: Improve instructions for using update_deps.
[ci skip]

Same change as KhronosGroup/Vulkan-ValidationLayers#955

Change-Id: Ifdc30a41f4d7f1e4ad9bc887df6f43f5caaa69f9

build: Fix LunarXchange Issue 862

Modified MoltenVK build arguments for MacOS from
`MVK_LOGGING_ENABLED=0` to `MVK_CONFIG_LOG_LEVEL=1`.

This allows users to set the logging information they
want when an application is run using MoltenVK.

By default, only errors are logged.

Change-Id: I471fa28e68b5dc68b10ae17858475981158b2a86

build: Update known-good files for 1.1.113 header

Change-Id: I3151722ccb8f3c5d918db8ffd701661a33dcf3b6

cube: Fix DbgMsg() use of varargs

vprintf (not printf) must be used with va_list.

Also moved flush() outside the va_list's scope.

Change-Id: Ib82b9f7e7ec1f954619c3eadb8dcaeb08abf113a

vulkaninfo: Add more descriptive error message

Issue #207 mentions that one of the main problems users
encounter when troubleshooting is a failure to create a
Vulkan instance, and suggests that a statement be added
to this error message that describes the most common
causes of this problem.

This commit adds this additional statement to the current
error message.

Change-Id: I2fccf00a8d9895a602fad40913f2380fc1c0a862

build: Update known-good files for 1.1.112 header

Change-Id: Ibf447e4f562d49bda5ab60b1bdf4ebf71678a6ae

vulkaninfo: Fix issue 195

It was suggested that `maxComputeSharedMemorySize`
from VkPhysicalDeviceLimits be displayed as a decimal
instead of a hex number.

Change-Id: I4b5cb61d56adea4e4d8d33619fabc37e359b8e4a

vulkaninfo: Fix issue 202

Vulkaninfo was replacing function `snprintf`
with function `_snprintf` on windows.

This seems to be for backwards compatiblity
with VS2013.

`_snprintf` is considered deprecated in VS2015
as `snprintf` is properly implemented.

Since we are dropping compatibility with VS2013 it
seems like this replacement should be removed.

Due to AppVeyor failing to build when this pre-processor
command is removed entirely, the command has been modified
to replace `snprintf` with `_snprintf_s`.

This command should hopefully be removed entirely in a future
commit.

Change-Id: I85f726fcb0a1cfcc902487100d35dc63d2ba00d8

build: Update MoltenVK in known_good.json

Updated MoltenVK version in known_good.json
from 1.0.34v -> 1.0.35v

Change-Id: I3c574d22ab98fcf0e3bd0c718a1a3f5a69287b42

cube: Add subpass dependencies

Shared depth buffer and image layout transition both
need to be accounted for

Change-Id: Id26bc47798c58e4435a5585def3161105b7fffd1

cube: Fix compiler warning

Change-Id: I8a84556cf55b03cc39402067150fc86be3ed1a0a

vulkaninfo: Fix HTML format for multi-GPU computer

Fixes an issue found where the device properties
and extensions would only be printed for the first GPU
on a system when printing in HTML format.

Change-Id: Id6812d7c6b14915afb87fdb740e8918f4609d662

build: Update known-good files glslang commit ID

Change-Id: I5ef3ac82301d42cbdb616da4c995cf6dc53c6708

cube: Add usage message on Windows

Change-Id: Id709544863b50def407d7e18c330914bebc9c03a

cmake: adjust cmake download process to match new design

Also took Karl off the list of e-mail addresses to be notified,
at his request.

cmake: Use CMake 3.10.2 for CI

These changes ensure that the Travis and AppVeyor
builds use a known version of CMake.

build: Update known-good files for 1.1.108 header

Change-Id: Ifc19c0e0fcb36f9a5009a73979eb20069e9db463

vulkaninfo: Run clang-format

Change-Id: I956d5777b657a25edb7f289047f320345af5cc44

vulkaninfo: VK_KHR_surface_protected_capabilities

Added output for the following structures from
extension VK_KHR_surface_protected_capabilities:

        VkSurfaceProtectedCapabilitiesKHR

Change-Id: I5fb89b8118abbdb3876463f2ff5f96c08221781b

vulkaninfo: Add VK_EXT_host_query_reset

Added output for the following structures from
extension VK_EXT_host_query_reset:

        VkPhysicalDeviceHostQueryResetFeaturesEXT

Change-Id: Ibc3dce6261ff5e46926d4f969562d377c74f23f5

vulkaninfo: Add VK_EXT_ycbcr_image_arrays

Added output for the following structures from
extension VK_EXT_ycbcr_image_arrays:

        VkPhysicalDeviceYcbcrImageArraysFeaturesEXT

Change-Id: I0f10ae006e99fff5192729ac32bafae2d9170d0c