bugGNU Octave - Bugs: bug #62407, datenum() handles nD array inputs...

 
 

bug #62407: datenum() handles nD array inputs incorrectly

Submitter:  Nir Krakauer <nir_krakauer>
Submitted:  Tue 03 May 2022 02:57:46 PM UTC
 
Category:  Octave Function Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * 7.1.0 Operating System:  * Any
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       

 

Sun 29 May 2022 04:46:49 AM UTC, comment #2: 

It's been 3 weeks without comment.  Assuming fix works.  Closing report.

Rik <rik5>
Project Administrator
Thu 05 May 2022 06:51:55 PM UTC, comment #1: 

Thank you for the report and the proposed solution.

I pushed a slightly different fix to the stable branch here:
https://hg.savannah.gnu.org/hgweb/octave/rev/1ce1c4552d5b

Marking as ready for test.

Markus M├╝tzel <mmuetzel>
Project Member
Tue 03 May 2022 02:57:46 PM UTC, original submission:  

If one of the first 3 inputs to datenum is a 3+ dimensional array with 1 column, the output has the wrong size. For example,

size (datenum (2000, 1, ones(1, 1, 3)))

is [3 1 3].

The attached patch fixes this.

Nir Krakauer <nir_krakauer>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #53171:  datenum.patch added by nir_krakauer (1KiB - text/x-patch)
file #53172:  datenum_fixed.m added by nir_krakauer (10KiB - text/x-objcsrc)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by mmuetzel (Posted a comment)
  • -email is unavailable- added by nir_krakauer (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 project members can vote.

     

     

     

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-05-29 rik5 StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2022-05-05 mmuetzel StatusNone Ready For Test
        Releasedev 7.1.0
    2022-05-03 nir_krakauer Attached File- Added datenum.patch, #53171
        Attached File- Added datenum_fixed.m, #53172

    Back to the top


    Powered by Savane 3.9