Skip to content

Add image dump in PNG#301

Closed
hevrard wants to merge 3 commits intogoogle:masterfrom
hevrard:export-png
Closed

Add image dump in PNG#301
hevrard wants to merge 3 commits intogoogle:masterfrom
hevrard:export-png

Conversation

@hevrard
Copy link
Collaborator

@hevrard hevrard commented Feb 19, 2019

This change adds support to dump image to PNG format, it relies on the third party 'lodepng' library (zlib license).

Maybe PPM support can be dropped subsequently, but it would break backward compatibility with vkrunner.

Note that in the cmake we add third_party/ as an include directory in order to please the linter about the import line of lodepng, which the linter insists to have the directory in it.

DEPS Outdated
vars['glslang_revision'],

'third_party/lodepng': vars['lvandeve_git'] + '/lodepng.git@' +
vars['glslang_revision'],
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should be lodepng_revision not glslang_revision

samples/png.cc Outdated

#include "amber/result.h"
#include "amber/value.h"
#include "lodepng/lodepng.h"
Copy link
Collaborator

Choose a reason for hiding this comment

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

#include "third_party/lodepng/lodepng.h" ? If you do that, can you drop the adding of third_party/ to the include path?

@dj2
Copy link
Collaborator

dj2 commented Feb 19, 2019

You're probably going to have to move the building of lodepng into the third_party/CmakeLists.txt file, have it build a library and then you can disable all of the warnings it triggers similar to how we do other third_party dependencies.

@dj2 dj2 mentioned this pull request Feb 20, 2019
dj2 added a commit that referenced this pull request Feb 20, 2019
This change adds support to dump image to PNG format, it relies on the
third party 'lodepng' library (zlib license).

This CL is based on Pull #301 by @hevrard.
@dneto0
Copy link
Collaborator

dneto0 commented Feb 20, 2019

This is superceded by #302 which has been merged.

@dneto0 dneto0 closed this Feb 20, 2019
@hevrard hevrard deleted the export-png branch March 15, 2019 17:12
dneto0 added a commit to dneto0/amber that referenced this pull request Dec 8, 2022
KhronosGroup/SPIRV-Headers@0bcc624...1d31a10

$ git log 0bcc62492..1d31a1004 --date=short --no-merges --format='%ad %ae %s'
2022-12-01 admin Update spir-v.xml
2022-12-01 admin Update spir-v.xml
2022-11-25 stephen.clarke Update spir-v.xml
2022-11-17 admin Register Taichi as SPIR-V generator
2022-11-16 dmitry.sidorov Remove unnecessary extensions addition
2022-11-14 alele Remove extension
2022-11-07 alele Fix typo.
2022-11-07 alele Review feedback 1.
2022-11-04 dneto Revert "Merge pull request google#301 from alan-baker/hpp1" (google#302)
2022-11-03 alanbaker Add spirv.hpp11 to BUILD.gn
2022-10-05 alele Add headers for SPV_NV_shader_invocation_reorder.
2022-10-17 dmitry.sidorov Add SPV_INTEL_runtime_aligned
2022-10-17 dmitry.sidorov Add SPV_INTEL_fpga_dsp_control
2022-10-17 dmitry.sidorov Add SPV_INTEL_fpga_invocation_pipelining_attributes
2022-10-17 dmitry.sidorov Update SPV_INTEL_fpga_loop_controls to rev I
2022-10-05 dkoch Fix extension for RayTracingOpacityMicromapEXT
2022-10-04 kpet Add support for texel buffers to clspv non-semantic instruction set
2022-10-04 hugo registered ID for Shady
2022-08-23 ewerness VK_EXT_opacity_micromap
2021-11-29 kevin.petit Add definitions for SPV_ARM_core_builtins
2021-08-23 pmistry Implement headers definitions for SPV_EXT_mesh_shader.
2022-07-27 jeremy Remove custom target if skipping install

Created with:
  roll-dep third_party/spirv-headers
dneto0 added a commit to dneto0/amber that referenced this pull request Dec 9, 2022
KhronosGroup/SPIRV-Headers@0bcc624...1d31a10

$ git log 0bcc62492..1d31a1004 --date=short --no-merges --format='%ad %ae %s'
2022-12-01 admin Update spir-v.xml
2022-12-01 admin Update spir-v.xml
2022-11-25 stephen.clarke Update spir-v.xml
2022-11-17 admin Register Taichi as SPIR-V generator
2022-11-16 dmitry.sidorov Remove unnecessary extensions addition
2022-11-14 alele Remove extension
2022-11-07 alele Fix typo.
2022-11-07 alele Review feedback 1.
2022-11-04 dneto Revert "Merge pull request google#301 from alan-baker/hpp1" (google#302)
2022-11-03 alanbaker Add spirv.hpp11 to BUILD.gn
2022-10-05 alele Add headers for SPV_NV_shader_invocation_reorder.
2022-10-17 dmitry.sidorov Add SPV_INTEL_runtime_aligned
2022-10-17 dmitry.sidorov Add SPV_INTEL_fpga_dsp_control
2022-10-17 dmitry.sidorov Add SPV_INTEL_fpga_invocation_pipelining_attributes
2022-10-17 dmitry.sidorov Update SPV_INTEL_fpga_loop_controls to rev I
2022-10-05 dkoch Fix extension for RayTracingOpacityMicromapEXT
2022-10-04 kpet Add support for texel buffers to clspv non-semantic instruction set
2022-10-04 hugo registered ID for Shady
2022-08-23 ewerness VK_EXT_opacity_micromap
2021-11-29 kevin.petit Add definitions for SPV_ARM_core_builtins
2021-08-23 pmistry Implement headers definitions for SPV_EXT_mesh_shader.
2022-07-27 jeremy Remove custom target if skipping install

Created with:
  roll-dep third_party/spirv-headers
dneto0 added a commit to dneto0/amber that referenced this pull request Dec 10, 2022
KhronosGroup/SPIRV-Headers@0bcc624...1d31a10

$ git log 0bcc62492..1d31a1004 --date=short --no-merges --format='%ad %ae %s'
2022-12-01 admin Update spir-v.xml
2022-12-01 admin Update spir-v.xml
2022-11-25 stephen.clarke Update spir-v.xml
2022-11-17 admin Register Taichi as SPIR-V generator
2022-11-16 dmitry.sidorov Remove unnecessary extensions addition
2022-11-14 alele Remove extension
2022-11-07 alele Fix typo.
2022-11-07 alele Review feedback 1.
2022-11-04 dneto Revert "Merge pull request google#301 from alan-baker/hpp1" (google#302)
2022-11-03 alanbaker Add spirv.hpp11 to BUILD.gn
2022-10-05 alele Add headers for SPV_NV_shader_invocation_reorder.
2022-10-17 dmitry.sidorov Add SPV_INTEL_runtime_aligned
2022-10-17 dmitry.sidorov Add SPV_INTEL_fpga_dsp_control
2022-10-17 dmitry.sidorov Add SPV_INTEL_fpga_invocation_pipelining_attributes
2022-10-17 dmitry.sidorov Update SPV_INTEL_fpga_loop_controls to rev I
2022-10-05 dkoch Fix extension for RayTracingOpacityMicromapEXT
2022-10-04 kpet Add support for texel buffers to clspv non-semantic instruction set
2022-10-04 hugo registered ID for Shady
2022-08-23 ewerness VK_EXT_opacity_micromap
2021-11-29 kevin.petit Add definitions for SPV_ARM_core_builtins
2021-08-23 pmistry Implement headers definitions for SPV_EXT_mesh_shader.
2022-07-27 jeremy Remove custom target if skipping install

Created with:
  roll-dep third_party/spirv-headers
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.

3 participants