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

Build conda package and upload to anaconda #515

Merged
merged 80 commits into from
May 15, 2024
Merged

Conversation

Aristoeu
Copy link
Collaborator

Added a workflow to build initiate conda environment, build conda package, and upload conda package to anaconda.

meta.yaml and build.sh are configuration files to build conda package. MANIFEST.in is a necessary file to build python package.

For now the conda package is uploaded to my account, and the token is written explicitly, it should be a secret defined in github repo.

I directly build the conda package from source code, rathen than building the wheel and uploading the wheel to pypi. I tried to build wheels for manylinux, but there are still problems:

  • cannot find -lpython3.8. It seems manylinux docker containers don't have the lpython, and no libpython3.8.so or libpython3.8.a. I didn't find a good way to install them when building the wheel
  • openblas error. It still throws many errors after I specified the lowest target version

Copy link

codecov bot commented Feb 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (master@cdf07be). Click here to learn what that means.

❗ Current head 1c226e4 differs from pull request most recent head 84e2314. Consider uploading reports for the commit 84e2314 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #515   +/-   ##
=========================================
  Coverage          ?   54.56%           
=========================================
  Files             ?      146           
  Lines             ?    22357           
  Branches          ?     1336           
=========================================
  Hits              ?    12200           
  Misses            ?     9817           
  Partials          ?      340           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Aristoeu Aristoeu marked this pull request as ready for review February 23, 2024 07:29
@Aristoeu
Copy link
Collaborator Author

Aristoeu commented Feb 23, 2024

Different python versions, test using docker, then arm
test in colied locally or in container

@Aristoeu Aristoeu marked this pull request as draft February 29, 2024 23:44
@Aristoeu
Copy link
Collaborator Author

Aristoeu commented May 11, 2024

Tried to change an older cmake version to debug the slash missing bug for arm64, but the cmake version is always set to 3.18.4-2+deb11u1

slash missing bug for arm64: https://github.com/mspass-team/mspass/actions/runs/9033197024/job/24822993108
line 2244
#11 2988.6 OSError: Failed to copy /mspass/cxx/includealgorithms/deconvolution/MTPowerSpectrumEngine.h to /opt/conda/conda-bld/mspasspy_1715349086006/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/info/recipe/cxx/includealgorithms/deconvolution/MTPowerSpectrumEngine.h. Error was: Command 'cp -a /mspass/cxx/includealgorithms/deconvolution/MTPowerSpectrumEngine.h /opt/conda/conda-bld/mspasspy_1715349086006/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/info/recipe/cxx/includealgorithms/deconvolution/MTPowerSpectrumEngine.h' returned non-zero exit status 1.

@wangyinz wangyinz merged commit 57afad5 into master May 15, 2024
6 of 10 checks passed
@wangyinz wangyinz deleted the build-to-wheel-for-conda branch May 15, 2024 05:22
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.

3 participants