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
[MRG] Add warning when setting element values with unknown keyword #1199
Conversation
Hello @scaramallion! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2020-10-01 21:51:54 UTC |
Codecov Report
@@ Coverage Diff @@
## master #1199 +/- ##
==========================================
+ Coverage 94.05% 94.07% +0.01%
==========================================
Files 54 54
Lines 7979 7987 +8
==========================================
+ Hits 7505 7514 +9
+ Misses 474 473 -1
Continue to review full report at Codecov.
|
pydicom/config.py
Outdated
@@ -292,6 +292,27 @@ def needs_to_convert_to_RGB(ds): | |||
values within the dataset when applying corrections. | |||
""" | |||
|
|||
WARN_INVALID_KEYWORD = True |
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.
Maybe WARN_ON_INVALID_KEYWORD
?
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.
Done
OK, changes done |
I think everything has been addressed. Merging... |
@SimonBiggs, we are having problems on push to master... lot of stuff blowing up for pymedphys - I had a quick look but nothing obvious, figured you could solve it much quicker... |
Yup, I think I know the issue. Will send a PR. Thanks Darcy. |
Describe the changes
Closes #1014
Tasks