v.net: Add JSON output support by tangelll · Pull Request #7088 · OSGeo/grass

tangelll · 2026-02-14T20:05:07Z

This PR adds JSON output support to the v.net module using the gjson/parson library.

Description

This PR introduces JSON output support for the v.net module, specifically for the operation=nodes task. This enhancement allows users to programmatically capture network statistics (such as new node counts) for use in automated workflows and testing pipelines.

Changes

Core Logic: Integrated G_JSON support in main.c to handle structured output.
CLI Arguments: Added a new format parameter to choose between plain (default) and json.
Documentation: Updated v.net.md with usage examples and guidelines for JSON output.
Testing: Added an automated Python test case in testsuite/test_v_net.py to ensure output consistency.

How to Test

To verify the implementation, you can use the North Carolina sample dataset.

Manual Verification:

# Copy sample data g.copy vector=streets_wake,my_streets # Run the module with JSON format for nreport testsuite % v.net input=my_streets_ready operation=nreport nlayer=2 format=json #Run the module with JSON format for report v.net input=streets output=test_vnet operation=report format=json

Expected JSON Output :

[
{
"line_cat": 49599,
"start_node_cat": -1,
"end_node_cat": -1
},
...
]

Automated Test Suite: Run the newly added test case:

python3 vector/v.net/testsuite/test_v_net.py

Copying features...
100%
Building topology for vector map test_vnet@PERMANENT...
Registering primitives...
40000
Copying attributes...
Building topology for vector map test_vnet@PERMANENT...
Registering primitives...
50000
v.net complete. 323 lines (network arcs) written to output.

.
Copying features...
100%
Building topology for vector map test_vnet@PERMANENT...
Registering primitives...
40000
Copying attributes...
Building topology for vector map test_vnet@PERMANENT...
Registering primitives...
50000
v.net complete. 156 lines (network arcs) written to output.

.
Copying attributes...
Building topology for vector map test_vnet@PERMANENT...
Registering primitives...
90000
v.net complete. 41813 new points (nodes) written to output.

.v.net complete.
.v.net complete.
.OK

vector/v.net/testsuite/test_v_net.py

petrasovaa

Please use pre-commit.

vector/v.net/testsuite/test_v_net.py

vector/v.net/args.c

vector/v.net/report.c

vector/v.net/v.net.md

vector/v.net/testsuite/test_v_net.py

tangelll · 2026-02-19T20:15:27Z

I added a check for G_OPT_F_FORMAT in args.c because it was causing a segmentation fault and UI warnings on my environment when the description was missing.

petrasovaa · 2026-02-20T03:23:10Z

vector/v.net/args.c

+ opt->format->key = "format";
+ if (!opt->format->description)
+ opt->format->description = _("Output format");


There should not be any need to add these.

Suggested change

opt->format->key = "format";

if (!opt->format->description)

opt->format->description = _("Output format");

"After removing those parts and running make, the build gets stuck at the UI description generation step with the following warnings: 'Missing option key' and 'Description for option <?> missing'.

I tested it and it works fine for me. Look at other usages of G_OPT_F_FORMAT.

Maybe you'd need to clean before make, like make clean, make libsclean, or make distclean. With the last one, you'd need to run configure again

This still needs to be addressed.

vector/v.net/report.c

.pre-commit-config.yaml

vector/v.net/Makefile

tangelll · 2026-02-24T21:22:36Z

Hi @petrasovaa , just wanted to check if you had a chance to look at this PR when you have a moment. Thanks!

vector/v.net/report.c

vector/v.net/Makefile

petrasovaa · 2026-02-25T22:12:19Z

Was it closed by mistake?

echoix · 2026-02-26T00:24:13Z

Reviewing is hard when force merged are made after there was a first review. Not only we Connor just read where we left off, but comments posted are on commits (and lines) that don't exist anymore, and are now kind of "detached" from the PR changed files view (and commits view)

echoix · 2026-02-26T00:26:26Z

The commit 72dd6b2 seems affected by a bad rebase, its tricky when we force-push.

tangelll · 2026-02-26T02:04:26Z

The commit 72dd6b2 seems affected by a bad rebase, its tricky when we force-push.

Hi!, yes you are right I opened a clean new branch and I will pull request it from there today.. thank you!.

echoix · 2026-02-26T03:14:14Z

We didn't review more since, you might as well fix up the same branch, so we still have the context of what we already discussed (instead of having it lost in a new PR)

tangelll · 2026-02-28T15:28:11Z

@echoix I've verified the changes locally with super-linter. Could you please approve the workflows to run the CI tests?

vector/v.net/testsuite/test_v_net.py

echoix · 2026-02-28T16:39:42Z

vector/v.net/testsuite/test_v_net.py

Could you double check which of the tests take long? In the error logs, these 6 tests in the file took 69 seconds to run on Ubuntu, which is quite long in my opinion. Maybe they were already long, but take a look to see if it is the newly added tests.

About 38.4sec on macOS (the Apple silicon chips are really fast, so it's normal, about 2x as fast as the Linux runners usually).

66 sec for the Linux CMake

Windows didn't finish run it yet.

I've optimized the tests to address the execution time concerns. The v.net network preparation steps are now moved to setUpClass, so they only run once for the entire test suite. My local tests show a significant reduction in total runtime. I've also ensured that existing tests and the tearDown logic remain intact.

What was the before and after (approx), and what was the other tests to compare to?

I realized I was preparing the heavy streets dataset twice (once for each new test). I have now moved the network preparation to setUpClass so it only runs once. This fixed the slowdown. Before it was 38 sec. in my local mac now it is 28 sec. in my local mac.

This still takes too long (it takes 40 s on my laptop). It should be done in less than 5 s. Try using a different vector, e.g. roadsmajor.

petrasovaa · 2026-03-17T03:08:52Z

vector/v.net/testsuite/test_v_net.py

This still takes too long (it takes 40 s on my laptop). It should be done in less than 5 s. Try using a different vector, e.g. roadsmajor.

petrasovaa · 2026-03-17T03:09:11Z

vector/v.net/args.c

+ opt->format->key = "format";
+ if (!opt->format->description)
+ opt->format->description = _("Output format");


This still needs to be addressed.

tangelll mentioned this pull request Feb 14, 2026

v.net: Add JSON output support for nodes operation #6999

Closed

github-actions bot added vector Related to vector data processing Python Related code is in Python C Related code is in C module docs markdown Related to markdown, markdown files tests Related to Test Suite labels Feb 14, 2026

github-actions bot reviewed Feb 15, 2026

View reviewed changes

github-actions bot reviewed Feb 17, 2026

View reviewed changes

vector/v.net/testsuite/test_v_net.py Outdated Show resolved Hide resolved

petrasovaa requested changes Feb 17, 2026

View reviewed changes

tangelll force-pushed the vnet-json branch from 474c4a3 to 3728e05 Compare February 19, 2026 20:13

github-actions bot added the CI Continuous integration label Feb 19, 2026

petrasovaa requested changes Feb 20, 2026

View reviewed changes

tangelll force-pushed the vnet-json branch from 550430c to 0ba9283 Compare February 21, 2026 15:40

github-actions bot added the CMake label Feb 21, 2026

petrasovaa reviewed Feb 25, 2026

View reviewed changes

vector/v.net/report.c Outdated Show resolved Hide resolved

petrasovaa reviewed Feb 25, 2026

View reviewed changes

vector/v.net/Makefile Outdated Show resolved Hide resolved

tangelll force-pushed the vnet-json branch from fccb894 to 72dd6b2 Compare February 25, 2026 15:10

github-actions bot added translation Message translation related libraries general labels Feb 25, 2026

tangelll closed this Feb 25, 2026

tangelll and others added 2 commits February 27, 2026 02:24

Tools is used for test , pre-commit fix is made and v.net.md is fixed.

6a837f6

fixing the pull problem, i used rebase instead and back to my works.

4b3e8dc

tangelll force-pushed the vnet-json branch from 5ad00b5 to 4b3e8dc Compare February 26, 2026 23:34

github-actions bot added vector Related to vector data processing Python Related code is in Python C Related code is in C module docs markdown Related to markdown, markdown files tests Related to Test Suite labels Feb 27, 2026

CMakeList grass parson is added

d117aca

github-actions bot added the CMake label Feb 27, 2026

echoix reviewed Feb 28, 2026

View reviewed changes

tangelll added 3 commits March 1, 2026 00:24

remove the te street_net and optimize the test time.

f490ab0

bring test_nodes back

107dda4

bring back the commad line

35fbcf6

petrasovaa requested changes Mar 17, 2026

View reviewed changes

	opt->format->key = "format";
	if (!opt->format->description)
	opt->format->description = _("Output format");

Uh oh!

Conversation

tangelll commented Feb 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Changes

How to Test

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

petrasovaa left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

tangelll commented Feb 19, 2026

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

tangelll commented Feb 24, 2026

Uh oh!

Uh oh!

petrasovaa commented Feb 25, 2026

echoix commented Feb 26, 2026

echoix commented Feb 26, 2026

tangelll commented Feb 26, 2026

echoix commented Feb 26, 2026

tangelll commented Feb 28, 2026

Uh oh!

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tangelll Mar 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Labels

3 participants

tangelll commented Feb 14, 2026 •

edited

Loading

tangelll Mar 1, 2026 •

edited

Loading