Re: Contributions - Image package function

[Carnë Draug]

On 4 May 2012 01:20, Carnë Draug <address@hidden> wrote:
>> On 2 May 2012 18:57, Pablo Rossi <address@hidden> wrote:
>>
>> Dear,
>>
>> The images’ package has not   the "imchop" from Matlab to crop an
>> image. Instead, I have developed a simple routine called "imgchop" trying
>> to fulfill this role.
>>
>> I send the file attachment.
>>
>> This package is published on the Chrishtin package to TLC.
>>
>> http://www.ing.unrc.edu.ar/grupos/simap/developments/index.html
>>
>> I hope my contribution will serve you
>>
>> Sincerely,
>>
>> Ing. Pablo C. Rossi
>>
> Hi Pablo
>
> we don't have an implementation of matlab's imcrop and your imgchop is
> already an approximation so I'd call it imcrop and add it to the image
> package with that name. The API you use seems to be the same albeit
> without all the options, it's just the GUI that it's just not so
> pretty (but I think that's all we can do with the current octave
> version). People will work on top of it. What do you think?
>
> One thing that should easily be implemented is to check nargin and if
> 0, to use the current figure. Another is if 2, and second argument is
> a colormap,display the image with it. If fisrt argument ishghandle,
> then use that figure. If second argument is a 4 elementss vector, use
> those coordinates rather than asking the user to draw a box. These are
> just some ideas that shouldn't take more than 30min to implement if
> you're interested on giving it some more time.
>
> Plus, do you have more functions on your package that would also cover
> some of the missing functions (or maybe something nice, of general
> use, that neither octave or matlab already has)?
>
> Finnaly, if you're interested, you could move your package into
> octave-forge. This would make it easier for people to see it when
> looking at octave (rather than chromatography) and it would allow
> users to easily install it with the -forge option.
>
> Carnë

Hi Pablo,

only now did I notice that you had originally mailed the octave
maintainers and help mailing list. Since this pertains to the image
package, it should have been mailed to the octave forge mailing list.
I'm sending it there now.

Anyway, I just added your function to the image package under the name
of imcrop. I had to replace errordlg by display since octave does not
have it yet.

Thank you very much for the code, I was planning on implementing it
for ages now. Hopefully someone will find time to further improve it
Carnë