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

[tests] Make repeat.py compatible with python 3 #1747

Merged
merged 1 commit into from
Feb 6, 2019

Conversation

kumpera
Copy link
Collaborator

@kumpera kumpera commented Feb 6, 2019

Fixes test_race_condition on OSX High Sierra that comes with python 3 by default.

@kumpera kumpera mentioned this pull request Feb 6, 2019
@JohnLangford JohnLangford merged commit c911042 into VowpalWabbit:master Feb 6, 2019
@JohnLangford
Copy link
Member

Merged, thanks :-)

rajan-chari pushed a commit to VowpalWabbit/reinforcement_learning that referenced this pull request Feb 22, 2019
RunTests: use test label number instead of counter (VowpalWabbit/vowpal_wabbit#1753)
Small Json parser cleanup (VowpalWabbit/vowpal_wabbit#1759)
Type erase json parser context for easier deletion ((VowpalWabbit/vowpal_wabbit#1760)
Fix static linking (VowpalWabbit/vowpal_wabbit#1758)
Fix build scripts forcing Debug builds. Add LTO mode and fix VW defau…(VowpalWabbit/vowpal_wabbit#1735)
Do not define BOOST_TEST_DYN_LINK when statically linking (VowpalWabbit/vowpal_wabbit#1750)
Convert TopK reduction to be multiline example based (VowpalWabbit/vowpal_wabbit#1752)
vw java 11 compatibility (VowpalWabbit/vowpal_wabbit#1700)
cbify: --cbify_ldf for multiline (csoaa_ldf) input datasets (VowpalWabbit/vowpal_wabbit#1681)
Merge pull request #1751 from yannstad/fix-tests (VowpalWabbit/vowpal_wabbit#1751)
[tests] Make repeat.py compatible with python 3 (VowpalWabbit/vowpal_wabbit#1747)
jackdoe pushed a commit to lucjb/vowpal_wabbit that referenced this pull request Mar 7, 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.

2 participants