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

Update to python-0.9.0 - only unix - with ARM support #10

Closed
wants to merge 24 commits into from

Conversation

jan-janssen
Copy link
Member

@jan-janssen jan-janssen commented Jul 7, 2024

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

@conda-forge-webservices
Copy link
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@jan-janssen
Copy link
Member Author

@conda-forge-admin, please rerender

@jan-janssen
Copy link
Member Author

/Users/runner/miniforge3/conda-bld/lancedb_1720331978333/test_tmp/conda_test_runner.sh: line 3: /Users/runner/miniforge3/conda-bld/lancedb_1720331978333/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/bin/python: Bad CPU type in executable

@jan-janssen jan-janssen marked this pull request as draft July 7, 2024 06:48
@jan-janssen
Copy link
Member Author

@conda-forge-admin, please rerender

@jan-janssen
Copy link
Member Author

We stopped hearing from agent Azure Pipelines 133. Verify the agent machine is running and has a healthy network connection. Anything that terminates an agent process, starves it for CPU, or blocks its network access can cause this error. For more information, see: https://go.microsoft.com/fwlink/?linkid=846610

@moritzwilksch moritzwilksch mentioned this pull request Jul 8, 2024
5 tasks
@jan-janssen
Copy link
Member Author

@conda-forge/core The Linux ARM tests timeout - most likely because they require too much memory. I previously had a similar issue on Windows #11 . Is there any way to get access to native ARM builds?

@h-vetinari
Copy link
Member

Before we think about native arm runners, can you try adding

azure:
  free_disk_space: true
  settings_linux:
    swapfile_size: 15GiB

to conda-forge.yml and rerender?

@jan-janssen
Copy link
Member Author

@conda-forge-admin, please rerender

@jan-janssen
Copy link
Member Author

@h-vetinari Is there a similar trick for windows to fix the out of memory issue in #11 ?

@h-vetinari
Copy link
Member

@h-vetinari Is there a similar trick for windows to fix the out of memory issue in #11 ?

We don't have a common interface unfortunately, and windows works... less reliably... but in principle you can do:

azure:
  settings_win:
    variables:
      CONDA_BLD_PATH: C:\\bld\\  # <- potentially not necessary, but C: tends to have more space (~70 GB) available
      SET_PAGEFILE: "True"

This will set a swapfile somewhere between 8GB & 16GB, depending what the system returns.

Here's some other related issues: conda-forge/conda-smithy#1949 & conda-forge/conda-smithy#1971

@jan-janssen jan-janssen marked this pull request as ready for review July 9, 2024 06:34
recipe/meta.yaml Outdated Show resolved Hide resolved
@jan-janssen
Copy link
Member Author

@h-vetinari Thanks a lot, based on your suggestion, I was able to successfully build the windows version in #11 and also the ARM version works fine now.

@jan-janssen jan-janssen closed this Jul 9, 2024
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