search

avaframe / AvaFrame

https://avaframe.org
European Union Public License 1.2
16 stars 2 forks source link

Handle bug about simName / scenario mismatch in report (Windows QGis GUI); issue #872 [com1] #1024

Closed fso42 closed 1 month ago

fso42 commented 1 month ago

Fixes the problem with report sinNames on Windows (#872)

This is a dirty fix, because it does not address the root cause, simply because I cannot find it. It only treats the effect...

pep8speaks commented 1 month ago

Hello @fso42! Thanks for updating this PR.

Line 2830:121: E501 line too long (132 > 120 characters) Line 2102:69: E127 continuation line over-indented for visual indent Line 1051:121: E501 line too long (124 > 120 characters)

Comment last updated at 2024-09-16 20:26:27 UTC
fso42 commented 1 month ago

Standardtests ok, apart from know Kot and PFA resistance. Pfa needs resistance parameter adjustments for next release

codeclimate[bot] commented 1 month ago

Code Climate has analyzed commit 6282d986 and detected 3 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1
Duplication 2

The test coverage on the diff in this pull request is 66.6% (50% is the threshold).

This pull request will bring the total coverage in the repository to 69.6% (0.0% change).

View more on Code Climate.