-
Notifications
You must be signed in to change notification settings - Fork 0
Fix/235 update test data #269
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
Conversation
…e into 235-update-test-data
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Besides my comments inline, it seems to me that the test data still contains lots of unused files. Is that so?
Other thing I noticed is that make test
+ make test_integration
succeeds, but make test_all
fails at the test_party_walls
test with an error that the partition 0/0/0
does not exist. I don't know why is this happening, but it suggests that there is still some mismatch between the test configuration and the test files.
if env == "test": | ||
root_dir = Path(os.getenv("BAG3D_FILESTORE")) / "reconstruction_input" | ||
root_dir = ( | ||
Path(os.getenv("BAG3D_FILESTORE")) / "integration_party_walls/file_store" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did you change this? Why set to "integration_party_walls/file_store"
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is not nested, for each integration test I have a filestore and a filestore_fastssd, so that it is clear (like we havei n production)
…peline into fix/235--update-test-data
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much much cleaner now, thanks for this.
No description provided.