Skip to content

wxGUI/gmodeler: Correct variable replacement in model_convert.py #7241

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

Open
gureckyfrantisek wants to merge 10 commits into
base: main
Choose a base branch
from
Open

wxGUI/gmodeler: Correct variable replacement in model_convert.py #7241

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
ae826ab
fix: Correct variable replacement
gureckyfrantisek Mar 30, 2026
8f67aa1
Merge branch 'main' into issue-7212
echoix Apr 15, 2026
4c70abc
wxGUI/gmodeler: backward compatibility for curly braces
gureckyfrantisek Apr 15, 2026
5a1f52d
wxGUI/gmodeler: parsing without curly braces fix
gureckyfrantisek Apr 15, 2026
5b128c6
Merge branch 'main' into issue-7212
gureckyfrantisek Apr 29, 2026
20ccd1c
regex: Allow only both or none curly braces
gureckyfrantisek Apr 29, 2026
bb54a2a
Merge branch 'main' into issue-7212
gureckyfrantisek May 7, 2026
952c3dd
fix: Proper map name from model
gureckyfrantisek May 15, 2026
d7eb782
fix: Map variable replacement after Python
gureckyfrantisek May 15, 2026
d9c9070
fix: Map variable replacement on Display change
gureckyfrantisek May 15, 2026
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
7 changes: 4 additions & 3 deletions gui/wxpython/gmodeler/model_convert.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -969,14 +969,15 @@ def _substitutePythonParamValue(
# check for variables
formattedVar = False
for var in variables["vars"]:
pattern = re.compile("%{" + var + "}")
found = pattern.search(value)
# curly braces are optional
Copy link
Copy Markdown
Member

@landam landam Apr 15, 2026

Choose a reason for hiding this comment

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

Curly braces enclosing variables were introduced in version 8.5. I assumed this change was meant to ensure backward compatibility, which would be nice. I tested to load model without braces (attached in the issue), the conversion to Python works. But attempt to run the model (directly from the modeler, not converted Python code) fails - no variables are substituted.

p.in.pdal -e -o --overwrite input=%path/dmp1g/%tile.laz output=%tile_dmp method=mean type=FCELL zscale=1.0 iscale=1.0 resolution=%resolution dimension=z
...

Backward compatibility wasn’t explicitly mentioned in the issue. I think it’s fine to keep this change in the PR, but in that case the model execution should be fixed as well.

Copy link
Copy Markdown
Author

@gureckyfrantisek gureckyfrantisek Apr 15, 2026

Choose a reason for hiding this comment

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

With the latest commits I changed every re.compile in the gmodeler folder have optional curly braces.

I managed to both run the model directly and have the values parametrized in the generated python code, but I had a leftover empty raster in my map at the end of the process:

Failed to run command 'd.rast map=%tile_chm'. Details:

GRASS_INFO_ERROR(70582,1): Raster map <%tile_chm> not found
GRASS_INFO_END(70582,1)
Failed to run command 'd.rast map=%tile_chm'. Details:

GRASS_INFO_ERROR(70665,1): Raster map <%tile_chm> not found
GRASS_INFO_END(70665,1)

I couldn't see, where the d.rast function is called, it's not in the model or the python code.

It doesn't break it, so I didn't look for a solution yet.

Copy link
Copy Markdown
Member

@landam landam May 6, 2026

Choose a reason for hiding this comment

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

@gureckyfrantisek It should be relatively easy to fix see https://github.com/OSGeo/grass/blob/main/gui/wxpython/gmodeler/model_items.py#L692

Copy link
Copy Markdown
Member

@landam landam May 6, 2026

Choose a reason for hiding this comment

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

It would be nice to fix in this PR.

pattern = re.compile("%{?" + var + "}?")
found = pattern.search(parameterizedValue)
if found:
foundVar = True
if found.end() != len(value):
formattedVar = True
parameterizedValue = pattern.sub(
"{options['" + var + "']}", value
"{options['" + var + "']}", parameterizedValue
)
else:
parameterizedValue = f'options["{var}"]'
Expand Down
Loading