Add proba.MultivariateGaussian #1297
Replies: 1 comment 2 replies
-
Hey hey 👋
I like the simplicity of your method! I was expecting to see a citation of the River paper though 🙈
Kind words.
I think it's something valuable to have in River. It would be the first multivariate (continuous) distribution, so I would be inclined to introduce a new base class. There are also a couple of details which could be improved in terms of design. In any case, I encourage you to open a pull request. I can provide some comments. |
Beta Was this translation helpful? Give feedback.
-
Hello everyone! 👋
I'm currently working on an exciting project called Unsupervised Online Outlier Detection Service with Dynamic Process Limits. During my development journey, I came across the fantastic rivers beautiful API, which has been instrumental in my progress.
As the project evolved, I recognized the need to introduce support for multivariate Gaussian distribution. After exploring rivers' API, I found it to be straightforward and aligned with my requirements. To address this, I implemented proba.MultivariateGaussian function.
Now, I believe this addition could be beneficial to the broader community, and I'd like to discuss whether it aligns with the library's broader goals and objectives.
You can find my fork of river with the new feature here. I have taken care to ensure it passes the pre-commit tests and is thoroughly documented.
I welcome your valuable input and feedback on this implementation. If it proves to be valuable, I'd be thrilled to make it a part of the official library to assist others in their projects.
Thank you all for your time and consideration. Looking forward to our discussion!
Beta Was this translation helpful? Give feedback.
All reactions