[ 
https://issues.apache.org/jira/browse/SPARK-16151?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Xiangrui Meng updated SPARK-16151:
----------------------------------
    Description: 
Currently, all generated param instances are final. There are several scenarios 
where we need them to be non-final:

1. We don't have a guideline about where we should document the param doc. Some 
were inherited from the generated param traits directly, while some were 
documented in the setters if we want to make changes. I think it is better to 
have all documented in the param instances, which appear at the top the 
generated API doc. However, this requires inherit the param instance.

{code}
/**
 * new doc
 */
val param: Param = super.param
{code}

We can use `@inherit_doc` if we just want to add a few words, e.g., the default 
value, to the inherited doc.

2. We might want to update the embedded doc in the param instance.

Opened this JIRA for discussion.

  was:
Currently, all generated param instances are final. There are several scenarios 
where we need them to be non-final:

1. We don't have a guideline about where we should document the param doc. Some 
were inherited from the generated param traits directly, while some were 
documented in the setters if we want to make changes. I think it is better to 
have all documented in the param instances, which appear at the top the 
generated API doc. However, this requires inherit the param instance.

{code}
/**
 * new doc
 */
val param: Param = super.param
{code}

We can use `@inherit_doc` if we just want to add a few words to the inherited 
doc.

2. We might want to update the embedded doc in the param instance.

Opened this JIRA for discussion.


> Make generated params non-final
> -------------------------------
>
>                 Key: SPARK-16151
>                 URL: https://issues.apache.org/jira/browse/SPARK-16151
>             Project: Spark
>          Issue Type: Improvement
>          Components: MLlib
>    Affects Versions: 2.0.0
>            Reporter: Xiangrui Meng
>
> Currently, all generated param instances are final. There are several 
> scenarios where we need them to be non-final:
> 1. We don't have a guideline about where we should document the param doc. 
> Some were inherited from the generated param traits directly, while some were 
> documented in the setters if we want to make changes. I think it is better to 
> have all documented in the param instances, which appear at the top the 
> generated API doc. However, this requires inherit the param instance.
> {code}
> /**
>  * new doc
>  */
> val param: Param = super.param
> {code}
> We can use `@inherit_doc` if we just want to add a few words, e.g., the 
> default value, to the inherited doc.
> 2. We might want to update the embedded doc in the param instance.
> Opened this JIRA for discussion.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org

Reply via email to