Skip to content
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: VpinEstimation v0.1.0 #117292

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JuliaRegistrator
Copy link
Contributor

@JuliaRegistrator JuliaRegistrator commented Oct 15, 2024

  • Registering package: VpinEstimation
  • Repository: https://github.com/jinyunfan/VpinEstimation.jl
  • Created by: @jinyunfan
  • Version: v0.1.0
  • Commit: 95ea205b4e4aee760ee35d4181f6f7e9f94a8f9b
  • Git reference: main
  • Description: With reference to Easley et al(2014), the volume synchronized probability of informed trading is estimated under high-frequency environment, and the index VPIN(volume synchronized probability of informed trading) is calculated.

Copy link
Contributor

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 registration

Please 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 registration

If 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 [noblock] in your comment.

Tip: You can edit blocking comments to add [noblock] in order to unblock auto-merging.

@goerz
Copy link
Member

goerz commented Oct 15, 2024

Thank you for resubmitting. This still needs a little bit more documentation in order to be registered. Packages in General are intended to be useful to a wider audience, which requires enough documentation to be able to figure out how to use the package. As a minimum, an initial registration might provide a usage example (with some explanations) in the README. Of course, a more complete Documenter-based documentation with tutorials etc. is always a good idea, and usually a requirement in the long term (for a v1.0 release) at least for more complex packages.

UUID: 98b75bef-cec7-42fd-9fd8-1608d125fef6
Repo: https://github.com/jinyunfan/VpinEstimation.jl.git
Tree: 48779900ce7da6676ee03cd1ba6716b2018d3189

Registrator tree SHA: 191228b6dd8b9d0e2965ae3e705fe54c51dcfee8
@JuliaRegistrator JuliaRegistrator force-pushed the registrator-vpinestimation-98b75bef-v0.1.0-4d7a742647 branch from f0e6c96 to 9b616be Compare October 24, 2024 09:59
@jinyunfan
Copy link

Ok, I changed the version number to v0.1.0. Is it OK now? @goerz

@goerz
Copy link
Member

goerz commented Oct 24, 2024

No… Didn’t you see my comment?

@jinyunfan
Copy link

Sorry,I misunderstood. So I will add a usage example and function descriptions before submitting.

@JuliaTagBot JuliaTagBot added the AutoMerge: last run blocked by comment PR blocked by one or more comments lacking the string [noblock]. label Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoMerge: last run blocked by comment PR blocked by one or more comments lacking the string [noblock]. new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants