Skip to content

[BLD] enable cython coverage, use cythonize #21879

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

Merged
merged 12 commits into from
Jul 20, 2018
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .coveragerc
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
[run]
branch = False
omit = */tests/*
plugins = Cython.Coverage

[report]
# Regexes for lines to exclude from consideration
Expand All @@ -22,6 +23,7 @@ exclude_lines =
if __name__ == .__main__.:

ignore_errors = False
show_missing = True

[html]
directory = coverage_html_report
4 changes: 2 additions & 2 deletions pandas/_libs/tslibs/period.pyx
Original file line number Diff line number Diff line change
Expand Up @@ -1655,8 +1655,8 @@ cdef class _Period(object):
return value

def __setstate__(self, state):
self.freq=state[1]
self.ordinal=state[2]
self.freq = state[1]
self.ordinal = state[2]

def __reduce__(self):
object_state = None, self.freq, self.ordinal
Expand Down
6 changes: 3 additions & 3 deletions pandas/_libs/src/util.pxd → pandas/_libs/tslibs/util.pxd
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
from numpy cimport ndarray, NPY_C_CONTIGUOUS, NPY_F_CONTIGUOUS
from numpy cimport ndarray
cimport numpy as cnp
cnp.import_array()

Expand Down Expand Up @@ -64,7 +64,7 @@ cdef inline bint is_datetime64_object(object obj) nogil:

# --------------------------------------------------------------------

cdef extern from "numpy_helper.h":
cdef extern from "../src/numpy_helper.h":
void set_array_not_contiguous(ndarray ao)

int assign_value_1d(ndarray, Py_ssize_t, object) except -1
Expand All @@ -87,7 +87,7 @@ ctypedef fused numeric:
cnp.float32_t
cnp.float64_t

cdef extern from "headers/stdint.h":
cdef extern from "../src/headers/stdint.h":
enum: UINT8_MAX
enum: UINT16_MAX
enum: UINT32_MAX
Expand Down
1 change: 1 addition & 0 deletions pandas/_libs/util.pxd
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
from tslibs.util cimport *
2 changes: 1 addition & 1 deletion pandas/_libs/window.pyx
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ from numpy cimport ndarray, double_t, int64_t, float64_t
cnp.import_array()


cdef extern from "../src/headers/cmath" namespace "std":
cdef extern from "src/headers/cmath" namespace "std":
int signbit(double) nogil
double sqrt(double x) nogil

Expand Down
Loading