-
Notifications
You must be signed in to change notification settings - Fork 15
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
adding build using binary downloads #8
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some comments but otherwise happy for it to go in. Thanks for resolving this quickly!
modules/utils/build.rs
Outdated
|
||
let (prebuilt_url, prebuilt_hash) = match target.as_str() { | ||
"aarch64-apple-darwin" => ( | ||
"https://parcel.pyke.io/v2/delivery/ortrs/packages/msort-binary/1.16.3/ortrs-msort_static-v1.16.3-aarch64-apple-darwin.tgz", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The versions and shared prefixes can be extracted. This guarantees that we don't have version typos then.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've now moved over to just using the download binaries feature in the ort
for the dev-dependencies
and then using the binary for the include bytes, all builds are now passing
output/ | ||
downloaded_onnx_package/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tbh we could probably move the package to output/downloaded_onnx
instead. Because it is a byproduct of building, even if its required for the build.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've now removed the reliance on downloads and it's all now in thetarget
directory
Due to the new release of rust, how some of the files were stored in different directories which broke the embedding of the ONNX. This pull request fixes this by getting `ort` to download and compile ONNX for the specific platform in `dev-dependencies` and this compiled ONNX library is then embedded.
* adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * Kings college london integration (#23) * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London
* merging bioengineering code for ML pipelines (#24) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * Kings college london integration (#23) * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * adding python bindings to data access for ML training
* adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * Kings college london integration (#23) * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * adding python bindings to data access for ML training
* adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com>
* adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com>
* adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version * updating the README and module --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com>
* adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version * updating the README and module * updating the surrealml-core deployment workflow * updating the surrealml-core deployment workflow --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com>
* adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version * updating the README and module * updating the surrealml-core deployment workflow * updating the surrealml-core deployment workflow * updating cargo --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com>
* Develop (#35) * adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version * updating the README and module * updating the surrealml-core deployment workflow * updating the surrealml-core deployment workflow * updating cargo --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com> * merging error modules into the core * merging error modules into the core * merging error modules into the core --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com>
* adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version * updating the README and module * updating the surrealml-core deployment workflow * updating the surrealml-core deployment workflow * updating cargo * Error modules (#36) * Develop (#35) * adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version * updating the README and module * updating the surrealml-core deployment workflow * updating the surrealml-core deployment workflow * updating cargo --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com> * merging error modules into the core * merging error modules into the core * merging error modules into the core --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com> --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com>
* adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version * updating the README and module * updating the surrealml-core deployment workflow * updating the surrealml-core deployment workflow * updating cargo * Error modules (#36) * Develop (#35) * adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version * updating the README and module * updating the surrealml-core deployment workflow * updating the surrealml-core deployment workflow * updating cargo --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com> * merging error modules into the core * merging error modules into the core * merging error modules into the core --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com> * Index overflow (#40) * adding buffer out of index check * adding buffer out of index check --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com>
* adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version * updating the README and module * updating the surrealml-core deployment workflow * updating the surrealml-core deployment workflow * updating cargo * Error modules (#36) * Develop (#35) * adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version * updating the README and module * updating the surrealml-core deployment workflow * updating the surrealml-core deployment workflow * updating cargo --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com> * merging error modules into the core * merging error modules into the core * merging error modules into the core --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com> * Index overflow (#40) * adding buffer out of index check * adding buffer out of index check * updating testing around meta data (#42) * updating the naming and increasing tests around the meta data of the stored ML models * updating the naming and increasing tests around the meta data of the stored ML models * updating the naming and increasing tests around the meta data of the stored ML models --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com>
* adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version * updating the README and module * updating the surrealml-core deployment workflow * updating the surrealml-core deployment workflow * updating cargo * Error modules (#36) * Develop (#35) * adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version * updating the README and module * updating the surrealml-core deployment workflow * updating the surrealml-core deployment workflow * updating cargo --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com> * merging error modules into the core * merging error modules into the core * merging error modules into the core --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com> * Index overflow (#40) * adding buffer out of index check * adding buffer out of index check * updating testing around meta data (#42) * updating the naming and increasing tests around the meta data of the stored ML models * updating the naming and increasing tests around the meta data of the stored ML models * updating the naming and increasing tests around the meta data of the stored ML models * Tensorflow support (#44) * stashing for branch switch * adding tests for tensorflow * adding tests for tensorflow * fixing requirement conflicts * fixing requirement conflicts * fixing requirement conflicts * fixing requirement conflicts * fixing requirement conflicts --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com>
* adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version * updating the README and module * updating the surrealml-core deployment workflow * updating the surrealml-core deployment workflow * updating cargo * Error modules (#36) * Develop (#35) * adding glue module to enable error handling (#27) * added batching runner (#28) * Kings college london integration (#30) * adding build using binary downloads (#8) * adding build using binary downloads * sorting out the build.rs * updating build.rs for surrealml package * prepping version for release * now has target tracking (#10) * adding check in build.rs for docs.rs * removing build.rs for main surrealml to ensure that libraries using the core do not need to do anything in their build.rs * adding machine learning pipelines for bioengineering projects at Kings College London * Remove integrated_training_runner/run_env/ from tracking * adding machine learning pipelines for bioengineering projects at Kings College London * Update FFmpeg data access module and README (#29) * adding run_env to the gitignore --------- Co-authored-by: Yang Li <oliverlee2018@163.com> * bumping the version * updating the README and module * updating the surrealml-core deployment workflow * updating the surrealml-core deployment workflow * updating cargo --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com> * merging error modules into the core * merging error modules into the core * merging error modules into the core --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com> * Index overflow (#40) * adding buffer out of index check * adding buffer out of index check * updating testing around meta data (#42) * updating the naming and increasing tests around the meta data of the stored ML models * updating the naming and increasing tests around the meta data of the stored ML models * updating the naming and increasing tests around the meta data of the stored ML models * Tensorflow support (#44) * stashing for branch switch * adding tests for tensorflow * adding tests for tensorflow * fixing requirement conflicts * fixing requirement conflicts * fixing requirement conflicts * fixing requirement conflicts * fixing requirement conflicts * adding support for raw ONNX models * adding support for raw ONNX models (#46) adding support for raw ONNX models * updating * fixing conflicts --------- Co-authored-by: Sam Hillman <116303632+SHillman836@users.noreply.github.com> Co-authored-by: Yang Li <oliverlee2018@163.com>
Instead of doing a
cargo build
on theort
library, thebuild.rs
file now directly downloads the onnx runtime and links it. The new update to rust automatically linked workspaces breaking the previous approach