1487
91%

Ran 28 Dec 2019 03:02AM UTC

Jobs 6

Files 210

Run time 8min

Badge

Embed ▾

pending completion

Build # 1487

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Merge #842

842: Missing conversion definition when mutating context transformations. r=hgrecco a=dcupif

Thanks for the library, we use it at our company and it's proven
really helpful so far. We are currently moving to Python 3.7 which
made me spot a potential bug in the `BaseRegistry` class (some
of our tests failed when upgrading).

Everything runs fine in Python 3.5, but in 3.7, we are getting a
`DimensionalityError` resulting from the ill-definition of a conversion.
Root cause is that we are mutating context transformations while
iterating over it.

This PR aims to avoid this potential problem. Since we modify the
transformations dictionary structure while iterating over it, we may
fail to iterate over all entries.

Explanation: "Iterating views while adding or deleting entries in the
dictionary may raise a `RuntimeError` or fail to iterate over all
entries"

https://docs.python.org/3/library/stdtypes.html#dictionary-view-objects

This is my first time contributing to Pint, so please let me know if
something needs to be improved / changed.

Co-authored-by: Damien Cupif <dcupif@kpler.com>

Run Details

38249 of 42915 relevant lines covered (89.13%)

5.22 hits per line

Jobs

ID	Job ID	Ran	Coverage
4	1487.4 (PKGS="python=3.8")	28 Dec 2019 03:02AM UTC	74.75	Travis Job 1487.4
5	1487.5 (PKGS="python=3.6 uncertainties=3.0")	28 Dec 2019 03:02AM UTC	91.99	Travis Job 1487.5
6	1487.6 (PKGS="python=3.7 uncertainties=3.0")	28 Dec 2019 03:10AM UTC	91.99	Travis Job 1487.6
9	1487.9 (PKGS="python=3.8 numpy=1.17 matplotlib")	28 Dec 2019 03:05AM UTC	90.25	Travis Job 1487.9
10	1487.10 (PKGS="python=3.6 numpy=1.14 uncertainties=3.0")	28 Dec 2019 03:04AM UTC	88.64	Travis Job 1487.10
11	1487.11 (PKGS="python=3.7 numpy=1.14 uncertainties=3.0")	28 Dec 2019 03:06AM UTC	88.6	Travis Job 1487.11

Source Files on build 1487

Detailed source file information is not available for this build.

hgrecco / pint / 1487
91%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1487

hgrecco / pint / 1487 91%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1487

hgrecco / pint / 1487
91%

README BADGES
x