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

[Bug]: duplicate_entries_found triggered on setID #223

Open
zargot opened this issue Nov 4, 2024 · 8 comments · May be fixed by #231
Open

[Bug]: duplicate_entries_found triggered on setID #223

zargot opened this issue Nov 4, 2024 · 8 comments · May be fixed by #231
Assignees
@zargot
Labels
bug Something isn't working
Milestone
v0.6.0

Comments

@zargot
Copy link
Collaborator

zargot commented Nov 4, 2024 •
edited
Loading

Bug Description

unclear if this is an error in the ODM definition, or if the code is supposed to handle this primary-key exception, or if this is user error.

error msg: sets(setID, 4,5): [duplicate_entries_found]

image

The operating system on which the bug occured

linux

Python Version

3.8
@zargot zargot added the bug Something isn't working label Nov 4, 2024
@zargot zargot assigned yulric and zargot and unassigned yulric and zargot Nov 4, 2024
@zargot
Copy link
Collaborator Author

zargot commented Nov 5, 2024

@yulric, do you know who's at fault here?

@yulric
Copy link
Collaborator

yulric commented Nov 5, 2024

Was this error generated when creating new schema files? Or when validating a data file?

@zargot
Copy link
Collaborator Author

zargot commented Nov 5, 2024

Was this error generated when creating new schema files? Or when validating a data file?

all the bug reports I made recently are just from validating the delatolla dataset.

@zargot
Copy link
Collaborator Author

zargot commented Nov 5, 2024

oh, it's not supposed to validate the parts and sets tables, is it?

@yulric
Copy link
Collaborator

yulric commented Nov 6, 2024

Nope

@zargot
Copy link
Collaborator Author

zargot commented Nov 6, 2024

right, then I will automatically exclude those.

@yulric
Copy link
Collaborator

yulric commented Nov 6, 2024

Why was the tool validating the dictionary? Were they in the data you were testing?

@zargot
Copy link
Collaborator Author

zargot commented Nov 6, 2024

Why was the tool validating the dictionary? Were they in the data you were testing?

yeah, the delatolla excel file including the dictionary, apparently.

@zargot zargot added this to the v0.6.0 milestone Nov 11, 2024
@zargot zargot linked a pull request Nov 11, 2024 that will close this issue
exclude dictionary tables from validation #231
Open
@zargot zargot linked a pull request Nov 11, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zargot zargot

Labels
bug Something isn't working
Projects
None yet
Milestone
v0.6.0
Development

Successfully merging a pull request may close this issue.

exclude dictionary tables from validation Big-Life-Lab/PHES-ODM-Validation
2 participants
@zargot @yulric