bugpyFormex - Bugs: bug #49273, PGF file readback fails if an...

 
 

bug #49273: PGF file readback fails if an attribute is a multidimensional array.

Submitter:  Benedict Verhegghe <bverheg>
Submitted:  Wed 05 Oct 2016 05:25:06 PM UTC
   
 
Category:  Core Severity:  4 - Important
Item Group:  Functionality error Status:  Fixed
Privacy:  Public Assigned to:  bverheg
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 15 Mar 2017 08:40:37 PM UTC, comment #4: 

In 665a6cfe the output of attributes has been restricted to simple types, and the 'color' attribute is skipped because it is already handled separately by the objtype header.

Benedict Verhegghe <bverheg>
Group administrator
Tue 14 Mar 2017 05:25:54 PM UTC, comment #3: 

In 5fb1bed4, we added a numpy print_option to make sure the full array is printed out instead of the truncated form.

Please test and report.

Benedict Verhegghe <bverheg>
Group administrator
Tue 14 Mar 2017 03:46:44 PM UTC, comment #2: 

Tim (tdezutter) has correctly reported that the solution from  cd694a331 is invalid in case of large arrays, because numpy writes an abbreviated form by default.

Therefore this bug has been reopened.

Benedict Verhegghe <bverheg>
Group administrator
Sat 17 Dec 2016 02:48:24 PM UTC, comment #1: 

Fixed in cd694a331

Array type values for Attributes can now properly be stored and read back in PGF format. THis fixes the problem with the color attribute.

Note that we do not intend to store any other classes but a generic array() in the PGF format. PGF should be application agnostic and thus not depend on the complex class hierarchies of pyFormex.

numpy's array interface however is considered generic enough.

Benedict Verhegghe <bverheg>
Group administrator
Wed 05 Oct 2016 05:25:06 PM UTC, original submission:  

New 2.0 format PGF files store color attributes of the geometry.
If the color is a 2-dim array ('vertex' color), readback fails.

This can e.g. be seen by running the GeomFile example.

Note:
It has been proposed to store 'vertex' color (and maybe also 'elem' color) in a Field, but that has not been implemented yet.

Benedict Verhegghe <bverheg>
Group administrator

 

(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 bverheg (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
    2017-03-22 bverheg StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2017-03-14 bverheg StatusIn Progress Ready For Test
    2017-03-14 bverheg StatusFixed In Progress
        Open/ClosedClosed Open
    2016-12-17 bverheg StatusNone Fixed
        Assigned toNone bverheg
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code