- 
                Notifications
    
You must be signed in to change notification settings  - Fork 37
 
Fix ignored docstring for Model{Missing} constructor #219
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
Conversation
| 
           Oh, nice catch, I didn't know that it is problematic to add docstrings to inner constructors. Maybe we could clean up and merge the three different docstrings for   | 
    
| 
           Sounds good, I'll try to come up with a proposal (though I'm quite a beginner with Turing 😊)  | 
    
| 
           bors r+  | 
    
| 
           👎 Rejected by too few approved reviews  | 
    
| 
           bors r+  | 
    
Currently the help for `Model` doesn't show the documentation for the inner constructor `Model{Missing}`, due to JuliaLang/julia#16730 .
This PR fixes it using the `@doc` workaround mentioned in that issue.
With this PR, the `Model{Missing}` doc shows up at the top, before the `struct Model` doc. I hope it's not a problem.
Co-authored-by: Hong Ge <hg344@cam.ac.uk>
    | 
           This PR was included in a batch that timed out, it will be automatically retried  | 
    
Currently the help for `Model` doesn't show the documentation for the inner constructor `Model{Missing}`, due to JuliaLang/julia#16730 .
This PR fixes it using the `@doc` workaround mentioned in that issue.
With this PR, the `Model{Missing}` doc shows up at the top, before the `struct Model` doc. I hope it's not a problem.
Co-authored-by: Hong Ge <hg344@cam.ac.uk>
    | 
           Timed out.  | 
    
Currently the help for
Modeldoesn't show the documentation for the inner constructorModel{Missing}, due to JuliaLang/julia#16730 .This PR fixes it using the
@docworkaround mentioned in that issue.With this PR, the
Model{Missing}doc shows up at the top, before thestruct Modeldoc. I hope it's not a problem.