PowerModelsDistribution.jl
PowerModelsDistribution.jl copied to clipboard
FIX: `parse_file` in `.dss` files with character UTF-8 0x09 (Tabulation)
This PR Closes #394. and it is not a breaking change.
Changes
- Adding a new test case that represents the #394 bug.
- Adding a new test for the new test case.
- Changing space validations as discussed in https://github.com/lanl-ansi/PowerModelsDistribution.jl/issues/394#issuecomment-1216697932.
- Update CHANGELOG.md.
Codecov Report
Merging #395 (b6ca4f7) into main (28995a1) will increase coverage by
0.01%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #395 +/- ##
==========================================
+ Coverage 69.58% 69.60% +0.01%
==========================================
Files 59 59
Lines 14607 14607
==========================================
+ Hits 10165 10167 +2
+ Misses 4442 4440 -2
Impacted Files | Coverage Δ | |
---|---|---|
src/io/dss/dss_parse.jl | 93.10% <100.00%> (ø) |
|
src/io/utils.jl | 82.50% <0.00%> (+0.38%) |
:arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
I don't have a MacOs to test why the tests didn't pass :/
The problem with the unit test should be fixed, could you merge in the latest main branch to trigger the tests to run again?
Thanks for the contribution!