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

RLS: 0.23.1 #21312

Closed
TomAugspurger opened this issue Jun 4, 2018 · 19 comments
Closed

RLS: 0.23.1 #21312

TomAugspurger opened this issue Jun 4, 2018 · 19 comments
Labels
Milestone

Comments

@TomAugspurger
Copy link
Contributor

cc @pandas-dev/pandas-core

Targeting Friday June 8th.

@TomAugspurger TomAugspurger added this to the 0.23.1 milestone Jun 4, 2018
@jorisvandenbossche
Copy link
Member

@TomAugspurger Do you have time to do the backporting one of the coming days?

@TomAugspurger
Copy link
Contributor Author

Some, but not much... Any objections to enable the backport bot? https://github.com/integration/meeseeksdev/

@jreback
Copy link
Contributor

jreback commented Jun 7, 2018

looks ok to me
how do u use?

@TomAugspurger
Copy link
Contributor Author

TomAugspurger commented Jun 7, 2018 via email

@TomAugspurger
Copy link
Contributor Author

How are we looking for 0.23.1 sometime soon (today?)

@jreback
Copy link
Contributor

jreback commented Jun 11, 2018

#19029 needs some work and is not blocking here

@toobaz
Copy link
Member

toobaz commented Jun 11, 2018

Thoughts on #21313 (which now also fixes #21428 )?

@TomAugspurger
Copy link
Contributor Author

#21313 looks a bit "big", but I'm not familiar with the indexing code :)

@jorisvandenbossche OK with pushing #21423 / #19029 to 0.23.2? Or should it block?

#19535 might be nice to include, since we've had a couple people report issues trying to install the sdist. That'll fix it, but the smaller fix is to just fix this manifest.

@jorisvandenbossche
Copy link
Member

Yes, #21313 is for 0.24.0 I think.

OK with pushing #21423 / #19029 to 0.23.2? Or should it block?

I personally think it is a small and easy change, but not many people reacted the last days on the issue ..

Do you have something for #19976

I would maybe rather release just now, and do a relative quick next 0.23.2 release.

@TomAugspurger
Copy link
Contributor Author

Agreed with a short timeline for 0.23.2, maybe end of June.

@TomAugspurger
Copy link
Contributor Author

Going to do the rest of the backports for 0.23.1 now.

@TomAugspurger
Copy link
Contributor Author

TomAugspurger commented Jun 12, 2018

#21442 should have the remaining backports.

0.23.1 will then be frozen and I'll do the release.

@TomAugspurger
Copy link
Contributor Author

Backports done. Tagging.

@TomAugspurger
Copy link
Contributor Author

Tagged and updated docs / website.

Waiting on binary builds.

@cgohlke
Copy link
Contributor

cgohlke commented Jun 12, 2018

How do I test the installed pandas package? I get 100% test errors on Windows using python -m pytest --pyargs pandas:

============================= test session starts =============================
platform win32 -- Python 3.6.5, pytest-3.6.1, py-1.5.3, pluggy-0.6.0
rootdir: C:\Users\...\test, inifile:
plugins: xdist-1.22.2, pep8-1.0.6, forked-0.2, faulthandler-1.5.0, cov-2.5.1, palladium-1.1.0.1, hypothesis-3.57.0, celery-4.1.1
collected 26384 items / 2 skipped

pandas\tests\test_algos.py EEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEE [  0%]
EEEE

=================================== ERRORS ====================================
____________________ ERROR at setup of TestMatch.test_ints ____________________

self = <_pytest.config.Config object at 0x0000028177D2F4E0>, name = 'only_slow'
default = <NOTSET>, skip = False

    def getoption(self, name, default=notset, skip=False):
        """ return command line option value.

            :arg name: name of the option.  You may also specify
                the literal ``--OPT`` option instead of the "dest" option name.
            :arg default: default value if no option of that name exists.
            :arg skip: if True raise pytest.skip if option does not exists
                or has a None value.
            """
        name = self._opt2dest.get(name, name)
        try:
>           val = getattr(self.option, name)
E           AttributeError: 'Namespace' object has no attribute 'only_slow'

X:\Python36\lib\site-packages\_pytest\config.py:1254: AttributeError

During handling of the above exception, another exception occurred:

item = <Function 'test_ints'>

    def pytest_runtest_setup(item):
        if 'slow' in item.keywords and item.config.getoption("--skip-slow"):
            pytest.skip("skipping due to --skip-slow")

>       if 'slow' not in item.keywords and item.config.getoption("--only-slow"):
E       ValueError: no option named 'only_slow'

X:\Python36\lib\site-packages\pandas\conftest.py:24: ValueError
<snip>

@TomAugspurger
Copy link
Contributor Author

Does python -c "import pandas; pandas.test()" work?

@cgohlke
Copy link
Contributor

cgohlke commented Jun 12, 2018

Does python -c "import pandas; pandas.test()" work?

Yes, it does. Sorry, I get confused with the many ways to invoke tests...

@TomAugspurger
Copy link
Contributor Author

:) pandas is probably doing something wrong with our conftest here :)

@TomAugspurger
Copy link
Contributor Author

All done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants