-
Notifications
You must be signed in to change notification settings - Fork 168
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
Automate Wheel Builds for multi-operating systems, including c++ keras-retinanet. #60
Comments
Its really looking like travis build pushes to pypi are only linux and mac. Reading more about azure services https://medium.com/raa-labs/creating-a-pipeline-in-azure-devops-to-build-and-publish-python-packages-artifacts-ea2f99410e6c |
wrong issue there Henry?
…On Wed, Jan 8, 2020 at 11:40 AM henry senyondo ***@***.***> wrote:
- Add Fia - tree - plot - cond join
- Includes filtering duplicate row values from required columns
- Includes adding tests for expected query statement.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#60?email_source=notifications&email_token=AAJHBLCGX4M236LI6DJMP7DQ4YT4RA5CNFSM4KC7IRG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEINXLYY#issuecomment-572224995>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJHBLG55TTWV6E45LFWSGDQ4YT4RANCNFSM4KC7IRGQ>
.
--
Ben Weinstein, Ph.D.
Postdoctoral Fellow
University of Florida
http://benweinstein.weebly.com/
|
my update on pypi builds trigged on master build on azure pipeline
|
complete. https://pypi.org/project/deepforest/#files uploaded! need to check if the pipeline gets the source distribution as well. Needs to be checked on azure instances tomorrow. If so, can close. |
Checked on azure machine for windows and locally on osx, looks great. see #56 |
Goal: An automated tool that builds both DeepForest and keras-retinanet into a single pypi wheel for each operating system. Separate builds for osx, linux and windows.
Current Status: OSX wheel is built on travis for DeepForest. User needs to pip install keras-retinanet from github and have valid c++ compiler on path for the Cython code. This makes it hard for windows users (
#56).
Resources being investigated:
Azure build pipelines: https://iscinumpy.gitlab.io/post/azure-devops-python-wheels/
we have Microsoft $, so cost is not an issue here.
https://github.com/joerick/cibuildwheel . Not immediately obvious if c++ are built across operating systems. Not encouraged by this thread here: https://travis-ci.community/t/python-support-on-windows/241
Additional thoughts: The keras-retinanet fork and the DeepForest repo are currently separate. I do not see pulling from keras-retinanet upstream. So I am open to the idea of copying the source and having both packages in the same repo. This is more of an organizational change, it doesn't help much with the above challenges (but would address #41).
@ethanwhite @henrykironde
The text was updated successfully, but these errors were encountered: