-
Notifications
You must be signed in to change notification settings - Fork 3
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
Enable basket export for kek #183
Merged
Merged
Changes from 5 commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
bc7c06b
Enable basket export for KEK
domi4484 53b36e5
Fix metaattribute
domi4484 ea1a26f
Org with or whitout basket
domi4484 c459616
Fix topic name
domi4484 ab7af95
Print current basket id
domi4484 7a2be0b
Enable basket for kek import
domi4484 0fccdd9
more debug info
domi4484 1813350
Try disable selection test
domi4484 0144e74
debug
domi4484 d6df629
debug
domi4484 4326a98
reset handling of org basket in base_common
domi4484 1048e96
Skip another selection test for now
domi4484 5e326a1
Apply suggestions from code review
domi4484 0bebb06
fix skip syntax
domi4484 a2307ce
sia405 selection export should wirk because baskets are not used
domi4484 1e4d34e
Rename export xtf according to exported model
domi4484 900333f
raise exception for debug
domi4484 6277fde
here its still a dict
domi4484 5561f96
Change back
domi4484 3fe7d37
Basket should not be enabled for dss import
domi4484 9fa9ca5
comment debug exception
domi4484 fb92bf2
More debug infos
domi4484 cfef36e
warning
domi4484 dfa2f30
debug
domi4484 4e0d3f9
Explicitly setup DB for test case f
domi4484 ce5a116
Re-enable case-e
domi4484 68924da
tmate
domi4484 4bbb46b
on failure
domi4484 a160f19
set --exportModels for KEK
domi4484 b6d2a5f
Cleanup
domi4484 b9e6f39
Missing NAME
domi4484 84a3fc0
Output exported file content
domi4484 867e566
Change current basket
domi4484 f633bd0
Change basket back for labels
domi4484 190496f
Merge branch 'master' into enableBasktetExportForKEK
ponceta 535e2ec
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
@domi4484 Here you are not writing the current_basket if you have class organisation - is this the problem? The tests fail on organisation 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.
Because it was giving an error. But it is set later here https://github.com/QGEP/qgepqwat2ili/pull/183/files#diff-eafeae0f9c5218fb04f5c903a4a2f22919fb6233a3cbb0e0746f611ad6906afcR287