patchGNU Octave - Patches: patch #9608, [octave forge] (image) new...

 
 

patch #9608: [octave forge] (image) new function colorangle

Submitted by:  Ricardo Fantin da Costa <ricardofantin>
Submitted on:  Mon 26 Mar 2018 11:57:59 PM UTC  
 
Category:  Forge : new function Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  None Open/Closed:  Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 23 Nov 2018 02:10:35 PM UTC, comment #4:

Thank you for the contributions. I have pushed this with a few more changes, mainly to support cases where there's multiple colours in any of the input.

Carnë Draug <carandraug>
Project Member
Thu 22 Nov 2018 09:05:50 PM UTC, comment #3:

(adding Avinoam to cc)

Hartmut <hardy>
Thu 15 Nov 2018 09:18:57 PM UTC, comment #2:

Hi, I modified Ricardo's code to use rad2deg, added some BIST tests (numbers taken from MATLAB docs), argument validation, a little formatting and changed the output to 0 instead of a NaN if given two zero vectors to be MATLAB compatible. I'm attaching a changeset with the changes.

(file #45437)

Martin Janda <tojtoj>
Thu 05 Apr 2018 05:59:59 PM UTC, comment #1:

rad2deg (and deg2rad) is in core octave, surely in the coming 4.4.0, and also in several octave-forge packages (mapping, geometry)

Philip Nienhuis <philipnienhuis>
Project Member
Mon 26 Mar 2018 11:57:59 PM UTC, original submission:

This function include the tests cases and a formula in the description.

The function is very simple.

Any feedback is welcome.

The function rad2deg is not implemented yet, so I use a multiplication by (pi/180) to convert. To make the function more readable when this function is available in octave a small change should be done. I left a comment in the file.

Ricardo Fantin da Costa <ricardofantin>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #45437:  colorangle.patch added by tojtoj (4KiB - application/octet-stream)
file #43692:  colorangle.m added by ricardofantin (2KiB - text/x-objcsrc - colorangle function implementation)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by hardy (Posted a comment)
  • -email is unavailable- added by hardy
  • -email is unavailable- added by tojtoj
  • -email is unavailable- added by tojtoj (Updated the item)
  • -email is unavailable- added by ricardofantin (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

     

     

    Follow 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-03-07 mtmiller Carbon-CopyRemoved 80942 => -
    2018-11-23 carandraug StatusNone => Done
        Open/ClosedOpen => Closed
    2018-11-22 hardy Carbon-Copy- => Added avinoam
    2018-11-16 tojtoj Carbon-Copy- => Added -email is unavailable-
    2018-11-15 tojtoj Attached File- => Added colorangle.patch, #45437
    2018-03-31 mtmiller Summaryimage package: new function colorangle => [octave forge] (image) new function colorangle
    2018-03-26 ricardofantin Attached File- => Added colorangle.m, #43692

    Back to the top


    Powered by Savane 3.4