Skip to content

CLN: remove unused PeriodEngine methods #33796

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 2 commits into from
Apr 25, 2020
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
35 changes: 1 addition & 34 deletions pandas/_libs/index.pyx
Original file line number Diff line number Diff line change
Expand Up @@ -21,14 +21,13 @@ cnp.import_array()

cimport pandas._libs.util as util

from pandas._libs.tslibs import Period
from pandas._libs.tslibs import Period, Timedelta
from pandas._libs.tslibs.nattype cimport c_NaT as NaT
from pandas._libs.tslibs.c_timestamp cimport _Timestamp

from pandas._libs.hashtable cimport HashTable

from pandas._libs import algos, hashtable as _hash
from pandas._libs.tslibs import Timedelta, period as periodlib
from pandas._libs.missing import checknull


Expand Down Expand Up @@ -501,38 +500,6 @@ cdef class PeriodEngine(Int64Engine):
cdef _call_monotonic(self, values):
return algos.is_monotonic(values, timelike=True)

def get_indexer(self, values):
cdef:
ndarray[int64_t, ndim=1] ordinals

super(PeriodEngine, self)._ensure_mapping_populated()

freq = super(PeriodEngine, self).vgetter().freq
ordinals = periodlib.extract_ordinals(values, freq)

return self.mapping.lookup(ordinals)

def get_pad_indexer(self, other: np.ndarray, limit=None) -> np.ndarray:
freq = super(PeriodEngine, self).vgetter().freq
ordinal = periodlib.extract_ordinals(other, freq)

return algos.pad(self._get_index_values(),
np.asarray(ordinal), limit=limit)

def get_backfill_indexer(self, other: np.ndarray, limit=None) -> np.ndarray:
freq = super(PeriodEngine, self).vgetter().freq
ordinal = periodlib.extract_ordinals(other, freq)

return algos.backfill(self._get_index_values(),
np.asarray(ordinal), limit=limit)

def get_indexer_non_unique(self, targets):
freq = super(PeriodEngine, self).vgetter().freq
ordinal = periodlib.extract_ordinals(targets, freq)
ordinal_array = np.asarray(ordinal)

return super(PeriodEngine, self).get_indexer_non_unique(ordinal_array)


cdef class BaseMultiIndexCodesEngine:
"""
Expand Down
7 changes: 0 additions & 7 deletions pandas/core/indexes/period.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
from datetime import datetime, timedelta
from typing import Any
import weakref

import numpy as np

Expand Down Expand Up @@ -322,12 +321,6 @@ def _formatter_func(self):
# ------------------------------------------------------------------------
# Indexing

@cache_readonly
def _engine(self):
# To avoid a reference cycle, pass a weakref of self._values to _engine_type.
period = weakref.ref(self._values)
return self._engine_type(period, len(self))

@doc(Index.__contains__)
def __contains__(self, key: Any) -> bool:
if isinstance(key, Period):
Expand Down