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

 
 

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

Submitter:  Ricardo Fantin da Costa <ricardofantin>
Submitted:  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
* Mandatory Fields

Add a New Comment Rich Markup
   

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>
Group 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>
Group 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)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    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.13-cf05.
    Corresponding source code