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

Jump to bottom

collect_attr_by_cond does not generate subitems in this case #782

Closed
jipclaassens opened this issue Aug 8, 2024 · 5 comments
Closed

collect_attr_by_cond does not generate subitems in this case #782

jipclaassens opened this issue Aug 8, 2024 · 5 comments
Assignees
@MaartenHilferink @jipclaassens
Labels
OVSRV09 testcase available on ovsrv09 tiny issue Work-load: probably tiny

Comments

@jipclaassens
Copy link
Member

jipclaassens commented Aug 8, 2024 •
edited
Loading

OVSRV09:
C:\ProjDir_Jip\_Issues\RSopen_issue782\cfg\main.dms

/SourceData/Vastgoed/BAG/PerJaar/Y2022/pand/met_vbo/Counts/vbos_inPand_per_vbo_functie

image
@jipclaassens jipclaassens changed the title collect_attr_by_cond does not generate subitems collect_attr_by_cond does not generate subitems in this case Aug 8, 2024
@jipclaassens
Copy link
Member Author

The selection_condition specified is the one from the select_with_att and thus wrong.

I've added this scenario to the operator test

/Relational/UnTiled2UnTiled/results/Tests

@jipclaassens jipclaassens added OVSRV09 testcase available on ovsrv09 tiny issue Work-load: probably tiny labels Sep 4, 2024
@MaartenHilferink
Copy link
Collaborator

MaartenHilferink commented Sep 12, 2024 •
edited
Loading

unable to reproduce

image

@jipclaassens
Copy link
Member Author

This test commit was probably undone in some merge. I've committed it to the GeoDMS-test_Jip branch:

/Relational/UnTiled2UnTiled/collect_attr_by_cond_Jip/test_wrongCondition

@jipclaassens
Copy link
Member Author

De sub attributen hebben DomainUnit = "/SourceData/Vastgoed/BAG/PerJaar/Y2022/pand"

sourceDomain = 0x0000027d4c5d2a60 "/SourceData/Vastgoed/BAG/PerJaar/Y2022/pand/met_vbo"

jipclaassens added a commit that referenced this issue Oct 1, 2024
@jipclaassens
Fixed #782
cc5774b
@jipclaassens
Copy link
Member Author

image

jipclaassens added a commit that referenced this issue Oct 2, 2024
@jipclaassens
Fixed #782, supplemental
b79a5d5
MaartenHilferink pushed a commit that referenced this issue Oct 2, 2024
@jipclaassens @MaartenHilferink
Fixed #782, supplemental
ea9caeb
MaartenHilferink added a commit that referenced this issue Oct 2, 2024
@MaartenHilferink
#782, supplemental
83b985c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MaartenHilferink MaartenHilferink

@jipclaassens jipclaassens

Labels
OVSRV09 testcase available on ovsrv09 tiny issue Work-load: probably tiny
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@MaartenHilferink @jipclaassens