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

8.) delete old detect code #1598

Merged
merged 1 commit into from
Nov 12, 2019
Merged

Conversation

shanaxel42
Copy link
Collaborator

@shanaxel42 shanaxel42 commented Oct 1, 2019

This PR deletes all the old Detect code and references to it in the documentation. It should be merged after a version of starfish has been released with the deprecation warnings.

@codecov-io
Copy link

codecov-io commented Oct 1, 2019

Codecov Report

Merging #1598 into master will decrease coverage by 0.22%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1598      +/-   ##
==========================================
- Coverage   90.09%   89.87%   -0.23%     
==========================================
  Files         234      219      -15     
  Lines        8764     8117     -647     
==========================================
- Hits         7896     7295     -601     
+ Misses        868      822      -46
Impacted Files Coverage Δ
starfish/spots.py 100% <ø> (ø) ⬆️
starfish/core/pipeline/algorithmbase.py 100% <100%> (ø) ⬆️

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 6674d9b...8c0b6ba. Read the comment docs.

@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch 3 times, most recently from c5119e3 to f04f3a8 Compare October 7, 2019 16:33
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch from f04f3a8 to 2e83d51 Compare October 7, 2019 19:19
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch from 2e83d51 to 5c05508 Compare October 8, 2019 17:04
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch from 5c05508 to 31a6633 Compare October 8, 2019 17:31
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch 2 times, most recently from a1146ed to 4bab8a6 Compare October 8, 2019 21:34
@shanaxel42 shanaxel42 requested a review from ttung October 8, 2019 21:39
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch from 4bab8a6 to e256f23 Compare October 8, 2019 22:48
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch 6 times, most recently from cfc715f to 48fb35f Compare October 9, 2019 18:24
Copy link
Collaborator

@ttung ttung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Boy, I bet this was satisfying.

Please grep for the symbols you deleted to make sure the documentation is up to date before landing this.

@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch 3 times, most recently from 95da868 to 7e0f702 Compare October 9, 2019 22:03
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch from 7e0f702 to a67db56 Compare October 9, 2019 22:36
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch from a67db56 to a496f53 Compare October 9, 2019 23:20
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch 2 times, most recently from 954b4d0 to a75759b Compare October 10, 2019 15:15
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch from a75759b to 2422242 Compare October 10, 2019 18:02
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch from 2422242 to c2aa606 Compare October 10, 2019 19:42
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch 3 times, most recently from f3e7e24 to 8f72169 Compare October 15, 2019 16:42
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch 2 times, most recently from 1e62cbe to bf6dd05 Compare October 15, 2019 17:38
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch from bf6dd05 to 52e5912 Compare October 17, 2019 00:13
@shanaxel42 shanaxel42 changed the base branch from saxelrod-change-test-code to master October 17, 2019 17:24
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch from 52e5912 to 7e24890 Compare October 18, 2019 20:51
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch from 7e24890 to d0e97c4 Compare November 4, 2019 18:44
@shanaxel42 shanaxel42 force-pushed the saxelrod-delete-old-detect-code branch from d0e97c4 to 8c0b6ba Compare November 12, 2019 17:35
@shanaxel42 shanaxel42 merged commit 658ff1e into master Nov 12, 2019
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.

3 participants