Skip to content
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

REL: Release version 0.7.1 #136

Merged
merged 25 commits into from May 22, 2020
Merged

REL: Release version 0.7.1 #136

merged 25 commits into from May 22, 2020

Conversation

scottclowe
Copy link
Member

Ships PRs #130, #132 and #135.

Fixed

  • Loading oval, ellipse, brush/freehand, freeline, and polyline ImageJ ROIs on Python 3. (#135)

Added

  • Support for rotated rectangle and multipoint ROIs on Python 3. (#135)

and not to just test one iteration. The results of the previous
test were too stringent and changed when scikit-learn moved from
version 0.22.2.post1 to 0.23.0.
All ROI types which possess x, y and n attributes are handled in
the same way as a polygon ROI, constructing a polygon using the
points supplied.
ROI types "freehand", "freeline", "multipoint", "point", "polygon",
"polyline", and "trace" are now handled correctly.
Co-ordinates in ImageJ have top at 0, so it should be
bottom = top + height. Previously it was as if bottom was at 0,
and the result was incorrect polygon y co-ordinates.
When we use `np.assert_all_close` using `self.assertEqual` as
declared with `addTypeEqualityFunc`, it is passed a `msg` argument
which `np.assert_all_close` doesn't accept. So we need to
ferry this argument away.
For checking two dictionaries of numpy.ndarrays are the same.
This is when the rotation argument was added to
skimage.draw.ellipse.
We don't deal with the potential superresolution in the
pixel locations, and so the output is incorrect.
Their release is not compatible with the latest skimage==0.17.2.
Having problems pushing with the python script (installed via pip)
and it seems like the big name python packages (numpy, pytest)
tend to use the bash script instead, which is working for them.
@codecov
Copy link

codecov bot commented May 22, 2020

Codecov Report

Merging #136 into v0.7.x will increase coverage by 11.17%.
The diff coverage is 98.11%.

Impacted file tree graph

@@             Coverage Diff             @@
##           v0.7.x     #136       +/-   ##
===========================================
+ Coverage   80.40%   91.57%   +11.17%     
===========================================
  Files           9        9               
  Lines         740      760       +20     
  Branches      139      142        +3     
===========================================
+ Hits          595      696      +101     
+ Misses        109       37       -72     
+ Partials       36       27        -9     
Impacted Files Coverage Δ
fissa/readimagejrois.py 89.69% <98.07%> (+38.54%) ⬆️
fissa/__meta__.py 100.00% <100.00%> (ø)
fissa/neuropil.py 89.18% <0.00%> (+21.62%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 37f6133...073fa5a. Read the comment docs.

@scottclowe scottclowe merged commit 6efdd57 into v0.7.x May 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant