-
Notifications
You must be signed in to change notification settings - Fork 30
Complete migration from arraydist/filldist to product_distribution #283
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
Conversation
|
@yebai 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs. I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review. |
Co-authored-by: yebai <3279477+yebai@users.noreply.github.com>
Co-authored-by: yebai <3279477+yebai@users.noreply.github.com>
Co-authored-by: yebai <3279477+yebai@users.noreply.github.com>
arraydist and filldist in favor of Distributions.product_distribution
|
@copilot, replace all |
Co-authored-by: yebai <3279477+yebai@users.noreply.github.com>
Co-authored-by: yebai <3279477+yebai@users.noreply.github.com>
Co-authored-by: yebai <3279477+yebai@users.noreply.github.com>
Co-authored-by: yebai <3279477+yebai@users.noreply.github.com>
I've successfully completed the replacement of all ✅ Replaced implementations: All The change preserves backward compatibility through Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
arraydist and filldist in favor of Distributions.product_distribution
This PR completes the migration from the deprecated
arraydistandfilldistfunctions to usingDistributions.product_distributiondirectly. The change removes specialized internal types and their associated overhead while preserving all functionality.Changes Made
MatrixOfUnivariate,VectorOfMultivariate,FillVectorOfUnivariate,FillMatrixOfUnivariate, andFillVectorOfMultivariatearraydistandfilldistfunctions and their source filesproduct_distributiondirectly instead of deprecated functionsMigration Guide
Users should replace:
All functionality is preserved - the new approach simply uses the standard
Distributions.product_distributioninterface directly instead of the specialized DistributionsAD implementations.💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.