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 almost 10 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.