-
Notifications
You must be signed in to change notification settings - Fork 8
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
various package status improvement #8
Conversation
Thank you for all the work you have done. Getting the dependencies down is a good thing. However, it looks like AbstractFFTs needs to be reinstated (or it is being unnecessarily used somewhere) You are most welcome to make improvements to this package as you wish. I am delighted that you are interested in the package. On my own all I would be doing is trying to keep compat entries maintained and otherwise doing the bare minimum to keep the package useable. |
Images is a big dependency that most package developers should not use directly. Instead they're encouraged to use exactly the ones they need. There are some test and documentation related dependencies. This commit removes them.
Right, I missed committing one change. It seems that this repo has messed up with whitespace issues. VSCode makes a bunch of auto-fixes and it makes tracking changes quite hard 😂 Can you run some whitespace fixes on the git history? For instance, the DemoCards is used to support all these kinds of examples and tutorials. It follows a mixed syntax of Documenter and Literate.jl. Currently, we need to wait for Images v0.25 out first. But if you're interested to check it locally, you can try to add JuliaImages/Images.jl#971 branch. I'm definitely interested in improving the status of this package; you're really an expert on this and I think people should be grateful to use them. I'm also considering making this package an external extension to ImageFiltering, or forking the functionality into ImageFiltering. I'm not sure yet which is better, but I do want to advertise your packages in JuliaImages. Or if you want to make them exposed more, you're welcome to transfer your packages to JuliaImages. |
Codecov Report
@@ Coverage Diff @@
## master #8 +/- ##
=========================================
Coverage ? 87.45%
=========================================
Files ? 5
Lines ? 765
Branches ? 0
=========================================
Hits ? 669
Misses ? 96
Partials ? 0 Continue to review full report at Codecov.
|
Thanks for the invitation! |
I am not sure about the whitespace fixing. All I know is the basics of git - just enough to get me into trouble! |
Enjoy the trip and your retired life 😆 |
various package status improvement
Done. You'll have to do a force-reset on your local repo to get the updated history: git fetch --all
git checkout master
git reset --hard origin/master |
Instead of doing JuliaImages/ImageFiltering.jl#235 I think it might be worth improving this package first 😆
This PR doesn't touch the core functionality with one exception
rayleighmode
reworked using ImageContrastAdjustment.Depending on how you want this package to work closely with the JuliaImages ecosystem, I can contribute more to e.g., improve test codes, docstrings, and potentially improve the core implementation when I'm confident.