I'm glad we could find this bug. Rolling back to the older version of gegl-operation-point-filter.c and adding support for enums in gegl-operation.c allows my opencl kernel to run (among other changes). I will rebase my repo on top of master once it's updated. The last issue that I'm having is that I get no entry for gegl:video-degradation when I have instrumentation enabled (GEGL_DEBUG_TIME=1). I've been parsing the output to determine the speed of other opencl implementations. Any suggestions?
Thanks, Nanley On Wed, Nov 19, 2014 at 2:26 PM, Nanley Chery <nanleych...@gmail.com> wrote: > It seems like the code to initialize and run the opencl kernel was lost in > this commit: > > > https://git.gnome.org/browse/gegl/commit/gegl?id=a206f032f77064cf9bff8590ac83ca5b086b53fd > > I'm not familiar enough with the codebase to understand the commit > message. Why was this functionality removed? > Should I add the deleted code into video degradation's process function? > > Thanks, > Nanley > > On Wed, Nov 19, 2014 at 12:57 AM, Nanley Chery <nanleych...@gmail.com> > wrote: > >> I noticed there was more to the brightness-contrast example. I made the >> adjustments concerning the kernel name and parameter values. >> The code compiles now. The current problem that I'm experiencing is that >> the run-composition.py test for video-degradation passes with an empty >> kernel. >> I'm not sure which code paths are executing to make this work. Any >> pointers? I'll do some grepping of the source tree in the meantime. >> >> Thanks, >> Nanley >> >> On Tue, Nov 18, 2014 at 8:22 PM, Nanley Chery <nanleych...@gmail.com> >> wrote: >> >>> Wow. Thank you for the tip, CL_CHECK is now giving me an output. >>> >>> This is the error message: >>> (lt-gegl:10486): GEGL-video-degradation.c-WARNING **: Error in >>> video-degradation.c:236@cl_process - invalid kernel >>> >>> I thought that I had followed the kernel compilation process correctly. >>> Do you notice any mistake? I have pushed my latest change to the branch. >>> >>> Nanley >>> >>> >>> >>> On Tue, Nov 18, 2014 at 8:06 PM, Victor Oliveira < >>> victormath...@gmail.com> wrote: >>> >>>> Hi Nanley, >>>> >>>> I'd recommend you follow operations/common/brightness-contrast.c file >>>> for a point-filter operation (i.e. a pixel-wise filter) instead of >>>> doing what you did. >>>> >>>> Notice that in operations/common/brightness-contrast.c#n153 there's a >>>> string brightness_contrast_cl_source which is a string in >>>> opencl/brightness-contrast.cl.h, these are auto-generated files from >>>> the kernels in the opencl folder. >>>> >>>> Let me know what happens from that. >>>> >>>> Victor >>>> >>>> On Tue, Nov 18, 2014 at 4:45 PM, Nanley Chery <nanleych...@gmail.com> >>>> wrote: >>>> > Hi Victor, >>>> > >>>> > Thank you very much for taking a look. I understand about the time. >>>> > >>>> > Here's the link to my bitbucket branch: >>>> > https://bitbucket.org/nanoman281/gegl-cse6230/branch/vid_upstrm >>>> > >>>> > The latest commit is what's causing the video-degradation.xml test to >>>> fail >>>> > (I'm testing using run-compositions.py). >>>> > >>>> > Nanley >>>> > >>>> > On Tue, Nov 18, 2014 at 5:11 PM, Victor Oliveira < >>>> victormath...@gmail.com> >>>> > wrote: >>>> >> >>>> >> Hi Nanley, >>>> >> >>>> >> Just to let you know, I'll need some time to answer that because I'll >>>> >> need to build GIMP on my new laptop. >>>> >> >>>> >> Can you share your code so I can give a look? >>>> >> >>>> >> Victor >>>> >> >>>> >> On Tue, Nov 18, 2014 at 12:49 PM, Nanley Chery < >>>> nanleych...@gmail.com> >>>> >> wrote: >>>> >> > Hi Victor, >>>> >> > >>>> >> > I'm a student working on OpenCL porting work for my High >>>> Performance >>>> >> > Computing class. I'm trying to implement an OpenCL port for the >>>> >> > newly-committed video-degradation operation. Are you willing to >>>> provide >>>> >> > guidance on the following roadblock? >>>> >> > >>>> >> > >>>> >> > The issue that I'm finding is that creating a cl_process method and >>>> >> > setting >>>> >> > the following variables in gegl_op_class_init is not enough to get >>>> the >>>> >> > cl_process method called: >>>> >> > >>>> >> > operation_class->opencl_support = TRUE; >>>> >> > point_filter_class->cl_process = cl_process; >>>> >> > >>>> >> > If I manually try to call the cl_process function in the process >>>> method >>>> >> > (like in edge-laplace.c), the program terminates in the >>>> >> > gegl_cl_set_kernel_args method without an error from CL_CHECK; >>>> >> > >>>> >> > Is there something I'm missing? I apologize for mailing you >>>> directly >>>> >> > instead >>>> >> > of writing to the mailing list. I'm a little pressed for time, so I >>>> >> > opted >>>> >> > for this option. >>>> >> > >>>> >> > Regards, >>>> >> > Nanley >>>> > >>>> > >>>> >>> >>> >> >
_______________________________________________ gegl-developer-list mailing list List address: gegl-developer-list@gnome.org List membership: https://mail.gnome.org/mailman/listinfo/gegl-developer-list