Change request #1350
Separate parameter reading from file loading
| Status: | Rejected | Start date: | 10/30/2014 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assigned To: | - | % Done: | 0% | |
| Category: | - | |||
| Target version: | 1.0.0 | |||
| Duration: |
Description
All ctools should only read task parameters in get_parameters(), and loading of files should only be done later in the run() method. Most of the tools do respect this rule, but not all of them (see #564). We should make sure that all ctools follow that rule.
Recurrence
No recurrence.
Related issues
History
#1
Updated by Knödlseder Jürgen over 9 years ago
I’m not fully sure that this always can be accommodated as sometimes the request of parameters depend on information found (or not found) in a file. The actual logic with the ctool base class even prevents this. I think that the actual response times of the ctools is reasonably fast, hence I propose to reject that issue.
#2
Updated by Knödlseder Jürgen over 9 years ago
- Status changed from New to Rejected
Decided to not do anything about that.