bugpyFormex - Bugs: bug #43005, Why out and res parameters are...

 
 

bug #43005: Why out and res parameters are deprecated in ABQData?

Submitter:  francesco <francio>
Submitted:  Sun 17 Aug 2014 01:02:32 PM UTC
   
 
Category:  Plugins Severity:  3 - Normal
Item Group:  None Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 21 Aug 2014 10:29:20 AM UTC, comment #2: 

The deprecation has been removed.

francesco <francio>
Group Member
Wed 20 Aug 2014 03:56:39 PM UTC, comment #1: 

Since out,res were made part of the Step class, I think the  general out,res are not needed anymore. They are written out on every step, even when they are the same for all steps.

If you have fixed out,res for all steps, it is just as easy to define your steps with these fixed refs as parameters, as to put them as arguments to the AbqData.

However, keeping the global out,res is useful when you want to ADD some global output requests to specific per step requests. That is why the feature was left in. So we should probably remove the deprecation.

Note however that the new OutputDB class does not allow this easy addition of global and per step requests (as in out+self.out).
In order to do that, the OutputDB class should probably be subclassed from a list. I do not see any particular reason for that class to be a Dict anyway.

Benedict Verhegghe <bverheg>
Group administrator
Sun 17 Aug 2014 01:02:32 PM UTC, original submission:  

I dont get this deprecation. Why cannot be passed in ABQDAta and only in Step?

Also now with the new OutputDB class can be convenent to keep it.
Can we remove this deprecation or can explain why is this done?

francesco <francio>
Group Member

 

(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 (Posted a comment)
  • -email is unavailable- added by francio (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.

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-08-21 francio Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code