Please note that as of October 24, 2014, the Nokia Developer Wiki will no longer be accepting user contributions, including new entries, edits and comments, as we begin transitioning to our new home, in the Windows Phone Development Wiki. We plan to move over the majority of the existing entries. Thanks for all your past and future contributions.

Revision as of 04:01, 19 December 2013 by somnathbanik (Talk | contribs)

Talk:Edge detection filter in Windows Phone

From Wiki
Jump to: navigation, search


Hamishwillee - Very first thought

Hi Somnath

Thanks for the article.

My first thought is that this filter would be much better if created as a custom filter in the Nokia Imaging SDK. That way others could use it in the pipeline and with other filters. Can you modify to support that?

This is a lot like "sketch" filter h- see!index.html

My only other thought is that the UI itself is not as interesting as the filter, so perhaps reorder and discuss the UI only very briefly at end.

I hope to review more thoroughly tomorrow



hamishwillee (talk) 08:09, 12 December 2013 (EET)

Hamishwillee - Still the same feedback

Hi Somnath

I fixed the categories and gave the introduction a subedit to improve the English.

Basically though my feedback is the same. This is an interesting filter, but it's value would be much higher if it was created for use with the Imaging SDK. I'm also more interested in the algorithm and integrating with the SDK than with the specific UI used.

So "high potential", but not yet realised :-)



hamishwillee (talk) 02:57, 13 December 2013 (EET)

Yan -

WriteableBitmap properties are slow. You should not use them in your loop. You can do something like

var with = bmp.PixelWidth;
var height = bmp.PixelHeight;
var bufferIn = bmp.Pixels;
var bufferOut = bitmap.Pixels;
for(int i= 0;i<height;++i)
for(int j = 0; j < width; ++j)
int index = i*with+j;
bufferOut [index] = bufferIn [index];
return bufferOut

yan_ (talk) 22:23, 14 December 2013 (EET)

Somnathbanik - Thanks

Hi Guys,

Thanks for your feedback, I will update the code soon.



somnathbanik (talk) 13:27, 15 December 2013 (EET)

Somnathbanik - The article is been updated

Hi Friends,

This article is been updated using Nokia Imaging SDK.



somnathbanik (talk) 12:45, 17 December 2013 (EET)

Hamishwillee - Thank you very much

Hi Somnath

Much more useful now - thanks. Did you update the code example? I suggest you look at Yan's comment - he is pretty much the community expert now days. Also your code doesn't handle exceptions - copy the ND example to do so.

It is up to you, but I'd suggest that you integrate fully and dump your old approach - make the update section the article and move to the top. The UI section isn't all that interesting and can move to the bottom (though keep the images in the introduction.

Thanks again



hamishwillee (talk) 01:34, 19 December 2013 (EET)

Somnathbanik - Feedback has been taken care of

Hi Hamish,

In this article there are two source code. And the new source code with Nokia Imaging SDK takes care of Yan's comment as well. Yes the article has both the old and the new code, but there are two separate projects one of which ( has followed Yan's technique.

I will check and will try to come up with more beautiful UI.



somnathbanik (talk) 06:01, 19 December 2013 (EET)