Also tolerate IndexError while trying to delete a field in mw#634
Also tolerate IndexError while trying to delete a field in mw#634yarikoptic wants to merge 1 commit intonipy:masterfrom
Conversation
Closes nipy#633 Somewhat of a blind fix without looking to deep why we get IndexError instead of KeyError
|
Yes will do it now |
Codecov ReportBase: 81.34% // Head: 81.29% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #634 +/- ##
==========================================
- Coverage 81.34% 81.29% -0.06%
==========================================
Files 42 42
Lines 3828 3828
==========================================
- Hits 3114 3112 -2
- Misses 714 716 +2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
|
so, do you think this fix is worth it or not really (given that alone doesn't provide remedy) @neurorepro ? |
|
@yarikoptic this fix can be useful if the error raised from |
Closes #633
Somewhat of a blind fix without looking to deep why we get IndexError instead of KeyError
@neurorepro -- can you check if that would be sufficient?