Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wxGUI: Fixed bare exceptions in mapdisp/ #4979

Closed
wants to merge 5 commits into from
Closed

wxGUI: Fixed bare exceptions in mapdisp/ #4979

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
78420ef
update
Jan 22, 2025
bf8091f
update
Jan 23, 2025
40682bf
flake update
Jan 23, 2025
e8889fc
Merge branch 'main' into 722-mapdisp
arohanajit Jan 28, 2025
6cf2808
Update .flake8
arohanajit Jan 28, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 0 additions & 4 deletions .flake8
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ per-file-ignores =
gui/wxpython/animation/g.gui.animation.py: E501
gui/wxpython/tplot/g.gui.tplot.py: E501
gui/wxpython/iclass/g.gui.iclass.py: E501
gui/wxpython/mapdisp/main.py: E722
gui/wxpython/mapdisp/test_mapdisp.py: E501
gui/wxpython/mapswipe/g.gui.mapswipe.py: E501
gui/wxpython/mapwin/base.py: E722
Expand All @@ -40,9 +39,6 @@ per-file-ignores =
# TODO: Is this really needed?
python/grass/jupyter/__init__.py: E501
python/grass/pygrass/vector/__init__.py: E402
# Current benchmarks/tests are changing sys.path before import.
# Possibly, a different approach should be taken there anyway.
python/grass/pygrass/tests/benchmark.py: F821
# Configuration file for Sphinx:
# Ignoring import/code mix and line length.
# Files not managed by Black
Expand Down
2 changes: 1 addition & 1 deletion gui/wxpython/mapdisp/main.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ def GetLayersFromCmdFile(self):
mapFile = line.split("=", 1)[1].strip()
try:
k, v = line[2:].strip().split("=", 1)
except:
except (ValueError, IndexError):
pass
render_env[k] = v
continue
Expand Down
2 changes: 1 addition & 1 deletion gui/wxpython/vdigit/wxdisplay.py
View file
Open in desktop
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unrelated changes

Original file line number Diff line number Diff line change
Expand Up @@ -20,14 +20,14 @@
import locale
import os
import sys

from ctypes import CFUNCTYPE, byref, c_double, c_int, pointer
from grass.lib.ctypes_preamble import UNCHECKED, String

import wx
from core.debug import Debug
from core.gcmd import DecodeString
from core.settings import UserSettings

try:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[black] reported by reviewdog 🐶

Suggested change
try:
try:

from grass.lib.gis import (
G_gisinit,
Expand Down
Loading