bugGNU Octave - Bugs: bug #64595, [Octave forge] (netcdf) Make...

 
 

bug #64595: [Octave forge] (netcdf) Make Matlab compatible +package version of netcdf functions for compatibility

Submitter:  Richard <crobar>
Submitted:  Fri 25 Aug 2023 11:44:02 AM UTC
   
 
Category:  Octave Forge Package Severity:  1 - Wish
Priority:  3 - Low Item Group:  Feature Request
Status:  Fixed Assigned to:  lostbard
Originator Name:  Richard Crozier Open/Closed:  * Closed
Release:  * dev Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Thu 21 Sep 2023 07:39:43 PM UTC, comment #10: 

No further problems reported. Closing as fixed.

Arun Giridhar <arungiridhar>
Group Member
Tue 05 Sep 2023 09:16:55 PM UTC, comment #9: 

Great, thanks for responding!

Richard <crobar>
Tue 05 Sep 2023 04:05:46 PM UTC, comment #8: 

I got around to looking at it and pushed a change to add the netcdf.xxxxxx files to the repo

John Donoghue <lostbard>
Group Member
Tue 29 Aug 2023 12:42:03 AM UTC, comment #7: 

I'll look at a patch if submitted - please provide as a git patch, rather than as a tarball of changes files.
 
I would imagine that as part of the generation, it will need to provide some kind of documentation so users can get information of function usage rather than just a see netcf_XXXXX

John Donoghue <lostbard>
Group Member
Mon 28 Aug 2023 09:54:21 PM UTC, comment #6: 

Purely compatibility with existing code written for Matlab, so it can be run without extensive modification. I'm looking at possibly supplying a patch to generate the +package directory on installation. Would such a patch be accepted?

Richard <crobar>
Mon 28 Aug 2023 09:53:42 PM UTC, comment #5: 

I marked it as a wish feature request and lowered priority

John Donoghue <lostbard>
Group Member
Mon 28 Aug 2023 09:51:11 PM UTC, comment #4: 

Yes thats what it does - and I agree that its not quite the same.

The struct file is auto generated from the other functions.

Is there added benefit in adding extra files to support it? It runs faster ? Apart from the import statement, matlab code wont run?

John Donoghue <lostbard>
Group Member
Mon 28 Aug 2023 09:29:33 PM UTC, comment #3: 

I feel it would be relatively easy to generate the +package by just creating files in a +netcdf directory like the following in a similar way


function getConstant(varargin)

    netcdf_getConstant(varargin{:});

end


Richard <crobar>
Mon 28 Aug 2023 09:23:46 PM UTC, comment #2: 

Thanks for the tip, but at first glance, I understand that what this does is create a structure in the local workspace with a load of function handles that match the netcdf function names.

This really doesn't provide compatibility as all existing matlab code using the netcdf package would have to be modified to add this import_netcdf statement in every function which uses the netcdf package.

Richard <crobar>
Mon 28 Aug 2023 07:59:37 PM UTC, comment #1: 

That functionality is already provided.

run

import_netcdf

after loading the netcdf package and you can use netcdf.XXXXX functions.

John Donoghue <lostbard>
Group Member
Fri 25 Aug 2023 11:44:02 AM UTC, original submission:  

The netcdf packages are all of the naming format "netcdf_*"

The matlab version of most of the same functions are instead in a +package directory "+netcdf" and called like netcdf.abort etc.

It would be very helpful if it were possible to provide the same interface in the Octave netcdf package. This could be generated automatically, and could coexist with the existing functions, so no need to break existing user code.

Richard <crobar>

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by arungiridhar (Posted a comment)
  • -email is unavailable- added by lostbard (Posted a comment)
  • -email is unavailable- added by nrjank (Updated the item)
  • -email is unavailable- added by crobar (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 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2023-09-21 arungiridhar StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2023-09-05 lostbard StatusNone Ready For Test
        Assigned toNone lostbard
    2023-08-28 lostbard Severity3 - Normal 1 - Wish
        Priority5 - Normal 3 - Low
    2023-08-25 nrjank Summary[netcdf] Make Matlab compatible +package version of netcdf functions for compatibility [Octave forge] (netcdf) Make Matlab compatible +package version of netcdf functions for compatibility

    Back to the top

    Powered by Savane 3.12