-
Notifications
You must be signed in to change notification settings - Fork 37
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 package.d doc for median filter example #468
Conversation
Why not use the median function from mir.math.stat?
http://mir-algorithm.libmir.org/mir_math_stat.html#.median
If I recall correctly, that text was from before mir.math.stat existed.
…On Fri, Mar 29, 2024 at 4:18 AM aferust ***@***.***> wrote:
I only changed the code comments to update the median filter example for docs. did not touch anything else.
________________________________
You can view, comment on, or merge this pull request online at:
#468
Commit Summary
7c90d9a Update package.d doc for median filter example
File Changes (1 file)
M source/mir/ndslice/package.d (32)
Patch Links:
https://github.com/libmir/mir-algorithm/pull/468.patch
https://github.com/libmir/mir-algorithm/pull/468.diff
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>
|
I believe the docs there are for explaining the usage of the ndslice not to show how to perform a median filter. |
Thank you for the update! Unfortunately, my Circle CI account is under sanctions, and it doesn't update online docs anymore. |
sorry to hear that. It is always weird that politics affects our lives in this way. There must be some third-party solutions for it. |
Sure, it just needs some time. Mir's solution uses Circle CI + Docker + AWS. Circle CI is OK, but I don't have time to work on Mir anymore. It requires creating a new Circle CI account, setting up AWS keys in GitHub, and digging into the AWS setup, which I completely forgot. I will accept changes if you or other Mir members want to transfer documentation generation to another AWS bucket and website. |
@9il I haven't had time to work on mir.stat lately, but when I was submitting PRs last year I didn't have any issues updating the documentation. My understanding was that the documentation for mir.algorithm wasn't updating for other reasons than what you highlight. The issue may be that you're not able to see the results of the checks [1]. The checks have been failing for a while. Right now, this is what I see:
|
Wow, thank you for the great news! Then it is just really very outdated:
It's still a lot of work, but at least twice less than I thought. |
I only changed the code comments to update the median filter example for docs. did not touch anything else.