-
Notifications
You must be signed in to change notification settings - Fork 465
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
New package: WykoOPDReader v0.1.2 #118120
New package: WykoOPDReader v0.1.2 #118120
Conversation
JuliaRegistrator
commented
Oct 26, 2024
- Registering package: WykoOPDReader
- Repository: https://github.com/jaakkor2/WykoOPDReader.jl
- Created by: @jaakkor2
- Version: v0.1.2
- Commit: a55c4c1d3c6400b43c9cdbf01b077e77279d6647
- Reviewed by: @jaakkor2
- Reference: Register jaakkor2/SurfaceProfileReaders.jl#1 (comment)
- Description: Reader for Wyko datasets (.opd)
UUID: 6da23c8c-444c-4310-aaa6-9150b66cb193 Repo: https://github.com/jaakkor2/WykoOPDReader.jl.git Tree: 293dcdf1811b3cbc1210dbafac04712878108375 Registrator tree SHA: 17aec322677d9b81cdd6b9b9236b09a3f1374c6a
Hello, I am an automated registration bot. I help manage the registration process by checking your registration against a set of AutoMerge guidelines. If all these guidelines are met, this pull request will be merged automatically, completing your registration. It is strongly recommended to follow the guidelines, since otherwise the pull request needs to be manually reviewed and merged by a human. 1. New package registrationPlease make sure that you have read the package naming guidelines. 2. AutoMerge Guidelines are all met! ✅Your new package registration met all of the guidelines for auto-merging and is scheduled to be merged when the mandatory waiting period (3 days) has elapsed. 3. To pause or stop registrationIf you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text Tip: You can edit blocking comments to add |
Thank you for submitting your package! However, could you extend the documentation in the README a little bit more? It would probably be good for "Wyko OPD profilometer datasets" to be a link to some page that explains the format and gives some context. Please also explain what the type of |
[noblock] Thank you @goerz for taking the time to review new package entries. Now I found Matlab script many by Veeco https://mtl.mit.edu/pipermail/labnetwork/attachments/20110711/1f04a0e6/attachment.ksh that seems be the best available description of the OPD format. I added a link to that to README.md. Also, docstring for |
Closing in favor of #118158 In the future, please do not change the version number when addressing issues with the registration.
My question was about the type of |