Bug report #21637

r.sun cannot be executed because it calls on two incompatible options

Added by vince - about 5 years ago. Updated almost 5 years ago.

Status:Closed
Priority:Normal
Assignee:Alexander Bruy
Category:Processing/GRASS
Affected QGIS version:3.6.0 Regression?:No
Operating System: Easy fix?:Yes
Pull Request or Patch supplied:Yes Resolution:
Crashes QGIS or corrupts data:No Copied to github as #:29453

Description

Trying to run the r.sun grass7 function from the Processing toolbox gives the following output:

"ERROR: insol_time and incidout are incompatible options"

And the process exits without producing any output. As was the case with r.sunmask, would it be possible to split that function into two mutually exclusive sub-functions?

Associated revisions

Revision ab646c4d
Added by Alexander Bruy almost 5 years ago

[processing] split r.sun into two algorithms as it has mutually
exclusive options (fix #21637)

Revision 121a92df
Added by Alexander Bruy almost 5 years ago

Merge pull request #10074 from alexbruy/grass-r-sun

[processing] split GRASS r.sun algorithm into two (fix #21637)

Revision f17accf1
Added by Alexander Bruy almost 5 years ago

[processing] split r.sun into two algorithms as it has mutually
exclusive options (fix #21637)

(cherry picked from commit ab646c4d75c6eac3c37cfa6a284191040048ffe8)

Revision de0544c1
Added by Alexander Bruy almost 5 years ago

Merge pull request #25793 from alexbruy/backport-r-sun

[processing] split GRASS r.sun algorithm into two (fix #21637)

History

#1 Updated by Giovanni Manghi about 5 years ago

  • Operating System deleted (MacOS X)
  • Assignee set to Giovanni Manghi

#2 Updated by Alexander Bruy almost 5 years ago

  • Assignee changed from Giovanni Manghi to Alexander Bruy
  • Status changed from Open to In Progress

#3 Updated by Alexander Bruy almost 5 years ago

  • Pull Request or Patch supplied changed from No to Yes

#4 Updated by Alexander Bruy almost 5 years ago

  • Status changed from In Progress to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF