Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Brightness table special effects try #517

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

xiaodi86
Copy link

大师兄好,我在这个比较好理解的波形图版本上按照你提供的思路(使用imageAtomicAdd实现)进行了修改,尝试对亮度直方图实现,发现现在的version 310,version 320环境都不支持imageAtomicAdd,希望大师兄抽空指导一下。

xiaodi86 and others added 4 commits February 18, 2023 14:17
修正获取纹理像素,直方图数据传入图像数据方式
在FrameBufferWithTexture类中如何定义mapBuffer()和unmapBuffer()
在FrameBufferWithTexture类中如何定义mapBuffer()和unmapBuffer()
@xiaodi86
Copy link
Author

xiaodi86 commented Mar 1, 2023

修正了亮度表像素数据获取与写入,添加FrameBufferWithTexture类mapBuffer(),unmapBuffer()方法。未修复glGetTexImage支持和TextureDrawer类

xiaodi86 added 3 commits March 1, 2023 09:48
delete glGetTexImage
删除背景设设置setColor
修改Histogram 参数
@wysaid
Copy link
Owner

wysaid commented Mar 1, 2023

最近有点忙哈... 我看你自己也修复了一些东西, 如果有啥不好解决的, 可以再补充一些信息

xiaodi86 added 2 commits March 2, 2023 17:29
Fix error: use of undeclared identifier 'glGetTexImage'
try fix No matching function for call to 'imageAtomic'
@xiaodi86
Copy link
Author

xiaodi86 commented Mar 2, 2023

大师兄好,可以运行起来了不报错了,但点击直方图特效时报错No matching function for call to 'imageAtomic',尝试了几种办法还是无法解决,是不是要把环境支持升级到420 es才能解决这个问题啊?大师兄有空时帮忙指导一下。感谢!

@wysaid
Copy link
Owner

wysaid commented Apr 6, 2023

大师兄好,可以运行起来了不报错了,但点击直方图特效时报错No matching function for call to 'imageAtomic',尝试了几种办法还是无法解决,是不是要把环境支持升级到420 es才能解决这个问题啊?大师兄有空时帮忙指导一下。感谢!

最近时间比较紧, 才看到你这个。 你这边问题解决了吗, 还有问题的话可以帮你看看, 你可以描述一下最新的进展

@wysaid
Copy link
Owner

wysaid commented Aug 23, 2023

Hello, 这个 PR 还有补充吗, 如果补充好了的话可以合入进去。

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