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.

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

v.random: do not transfer cat from area if attributes are coppied too #2537

Merged
merged 2 commits into from
Sep 25, 2022

Conversation

marisn
Copy link
Contributor

@marisn marisn commented Aug 26, 2022

Current code creates two cat entries for each point on the same level -
one from area point is in and one matching new entry in the database.
As entries in db have new cat values, old cat value points to a random
or non-existing entry in the db thus causing a mess.
Fixes #2475.

Current code creates two cat entries for each point on the same level -
one from area point is in and one matching new entry in the database.
As entries in db have new cat values, old cat value points to a random
or non-existing entry in the db thus causing a mess.
Fixes OSGeo#2475.
@marisn marisn requested a review from metzm August 26, 2022 15:06
@marisn marisn added the vector Related to vector data processing label Aug 26, 2022
@marisn marisn added this to the 8.2.1 milestone Aug 26, 2022

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@wenzeslaus
Copy link
Member

I resolved the conflict in the GitHub web interface by using the new line.

@marisn marisn merged commit 82fb7b8 into OSGeo:main Sep 25, 2022
neteler pushed a commit that referenced this pull request Oct 17, 2022
…#2537)

Current code creates two cat entries for each point on the same level -
one from area point is in and one matching new entry in the database.
As entries in db have new cat values, old cat value points to a random
or non-existing entry in the db thus causing a mess.
Fixes #2475.
ninsbl pushed a commit to ninsbl/grass that referenced this pull request Oct 26, 2022
…OSGeo#2537)

Current code creates two cat entries for each point on the same level -
one from area point is in and one matching new entry in the database.
As entries in db have new cat values, old cat value points to a random
or non-existing entry in the db thus causing a mess.
Fixes OSGeo#2475.
ninsbl pushed a commit to ninsbl/grass that referenced this pull request Feb 17, 2023
…OSGeo#2537)

Current code creates two cat entries for each point on the same level -
one from area point is in and one matching new entry in the database.
As entries in db have new cat values, old cat value points to a random
or non-existing entry in the db thus causing a mess.
Fixes OSGeo#2475.
@marisn marisn deleted the v_random_fix_cats branch October 22, 2023 10:01
neteler pushed a commit to nilason/grass that referenced this pull request Nov 7, 2023
…OSGeo#2537)

Current code creates two cat entries for each point on the same level -
one from area point is in and one matching new entry in the database.
As entries in db have new cat values, old cat value points to a random
or non-existing entry in the db thus causing a mess.
Fixes OSGeo#2475.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vector Related to vector data processing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] v.to.rast use=attr does not convert all points
3 participants