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

Turn on tests when building through conda-build #493

Closed
wants to merge 18 commits into from

Conversation

peterjc123
Copy link
Contributor

@peterjc123 peterjc123 commented Apr 2, 2020

Relates to #425

@peterjc123
Copy link
Contributor Author

@vincentqb The Windows tests are enabled here. You may use this PR for testing purposes.

@vincentqb
Copy link
Contributor

Rebased.

@vincentqb
Copy link
Contributor

Is it intentional that config.yml and config.yml.in are not in sync, see here?

@peterjc123
Copy link
Contributor Author

@vincentqb Nope, I used python regenerate.py but it doesn't work.

@peterjc123 peterjc123 force-pushed the conda_test branch 3 times, most recently from 44900ad to c9cd1fb Compare May 1, 2020 03:45
@vincentqb
Copy link
Contributor

Rebased and regenerated yaml.

@vincentqb
Copy link
Contributor

The windows build appear to get stuck at the same place:

Packaging torchaudio
INFO:conda_build.build:Packaging torchaudio
INFO conda_build.build:build(1571): Packaging torchaudio
Packaging torchaudio-0.6.0.dev20200501-py36
INFO:conda_build.build:Packaging torchaudio-0.6.0.dev20200501-py36
INFO conda_build.build:bundle_conda(891): Packaging torchaudio-0.6.0.dev20200501-py36
Importing conda-verify failed.  Please be sure to test your packages.  conda install conda-verify to make this message go away.
WARNING:conda_build.build:Importing conda-verify failed.  Please be sure to test your packages.  conda install conda-verify to make this message go away.
WARNING conda_build.build:bundle_conda(1042): Importing conda-verify failed.  Please be sure to test your packages.  conda install conda-verify to make this message go away.
compiling .pyc files...
number of files: 49
Fixing permissions
Packaged license file/s.
WARNING: Detecting which files contain PREFIX is slow, installing ripgrep makes it faster. 'conda install ripgrep'
TEST START: C:\tools\miniconda3\conda-bld\win-64\torchaudio-0.6.0.dev20200501-py36.tar.bz2
Renaming host prefix directory,  C:\tools\miniconda3\conda-bld\torchaudio_1588376129751\_h_env  to  C:\tools\miniconda3\conda-bld\torchaudio_1588376129751\host_prefix_moved_torchaudio-0.6.0.dev20200501-py36_win-64
Renaming work directory,  C:\tools\miniconda3\conda-bld\torchaudio_1588376129751\work  to  C:\tools\miniconda3\conda-bld\torchaudio_1588376129751\work_moved_torchaudio-0.6.0.dev20200501-py36_win-64
Collecting package metadata (repodata.json): ...working... done
Solving environment: ...working... done

## Package Plan ##

  environment location: C:\tools\miniconda3\conda-bld\torchaudio_1588376129751\_test_env


The following NEW packages will be INSTALLED:

    atomicwrites:       1.3.0-py36_1                                 
    attrs:              19.3.0-py_0                                  
    blas:               1.0-mkl                                      
    certifi:            2020.4.5.1-py36_0                            
    colorama:           0.4.3-py_0                                   
    cpuonly:            1.0-0                         pytorch-nightly
    icc_rt:             2019.0.0-h0cc432a_1                          
    importlib_metadata: 1.5.0-py36_0                                 
    intel-openmp:       2020.0-166                                   
    mkl:                2020.0-166                                   
    mkl-service:        2.3.0-py36hb782905_0                         
    mkl_fft:            1.0.15-py36h14836fe_0                        
    mkl_random:         1.1.0-py36h675688f_0                         
    more-itertools:     8.2.0-py_0                                   
    ninja:              1.9.0-py36h74a9793_0                         
    numpy:              1.18.1-py36h93ca92e_0                        
    numpy-base:         1.18.1-py36hc3f5095_1                        
    packaging:          20.3-py_0                                    
    pip:                20.0.2-py36_1                                
    pluggy:             0.13.1-py36_0                                
    py:                 1.8.1-py_0                                   
    pyparsing:          2.4.7-py_0                                   
    pytest:             5.4.1-py36_0                                 
    python:             3.6.10-h9f7ef89_1                            
    pytorch:            1.6.0.dev20200501-py3.6_cpu_0 pytorch-nightly [cpuonly]
    scipy:              1.4.1-py36h9439919_0                         
    setuptools:         46.1.3-py36_0                                
    six:                1.14.0-py36_0                                
    sqlite:             3.31.1-h2a8f88b_1                            
    torchaudio:         0.6.0.dev20200501-py36        local          
    vc:                 14.1-h0510ff6_4                              
    vs2015_runtime:     14.16.27012-hf0eaf9b_1                       
    wcwidth:            0.1.9-py_0                                   
    wheel:              0.34.2-py36_0                                
    wincertstore:       0.2-py36h7fe50ca_0                           
    zipp:               3.1.0-py_0                                   
    zlib:               1.2.11-h62dcd97_4                            

Preparing transaction: ...working... done
Verifying transaction: ...working... done
Executing transaction: ...working... done

@peterjc123
Copy link
Contributor Author

Yes, I'm looking into it. Looks like a bug in conda-build that blocks installing wheel packages and using pytest.

@peterjc123
Copy link
Contributor Author

Well, it's weird because I cannot reproduce it on my machine.

@peterjc123
Copy link
Contributor Author

peterjc123 commented May 2, 2020

@vincentqb Could you please give me some write permission to this repo so that I can debug using this tutorial? Currently, I cannot rerun a CircleCI job with SSH.

@peterjc123
Copy link
Contributor Author

@vincentqb It seems that the tests in test_datasets.py leads to the hang.

@peterjc123
Copy link
Contributor Author

@vincentqb There's only one test failing now: test_batch_mfcc. Could you please look into it?

@vincentqb
Copy link
Contributor

@vincentqb There's only one test failing now: test_batch_mfcc. Could you please look into it?

I'm ok skipping this test on windows for now.

E       AssertionError: Not within tolerance rtol=1e-05 atol=1e-05 at input[1, 0, 2, 1384] (0.5488020181655884 vs. 0.5488258004188538) and 1 other locations (0.00%)

@peterjc123
Copy link
Contributor Author

@vincentqb Test failures are unrelated. Do you want to get this into master?

@vincentqb
Copy link
Contributor

Yes, I would like to merge this PR in.

  • nit: should we have the windows test under ci/circleci: unittest_windows_py3.6? there is no equivalent ci/circleci: binary_windows_*_py3.6 that would make sense, right?
  • once this PR as landed, we should update the upload the conda/wheel nightlies too :)

@peterjc123
Copy link
Contributor Author

peterjc123 commented May 3, 2020

[ ] nit: should we have the windows test under ci/circleci: unittest_windows_py3.6? there is no equivalent ci/circleci: binary_windows_*_py3.6 that would make sense, right?

Yes, we should simply rename the prefix of the current working job into unittest.

  • once this PR as landed, we should update the upload the conda/wheel nightlies too :)

Yes, and that will fairly easy since torchaudio on Windows will be a Python-only package.

@peterjc123
Copy link
Contributor Author

@vincentqb The unittest jobs are enabled in #604.

@vincentqb
Copy link
Contributor

Closed by #604

@vincentqb vincentqb closed this May 4, 2020
mthrok pushed a commit to mthrok/audio that referenced this pull request Dec 13, 2022
Co-authored-by: holly1238 <77758406+holly1238@users.noreply.github.com>
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