[jira] [Updated] (ARROW-13615) [R] Bindings for stringr::str_to_sentence
[ https://issues.apache.org/jira/browse/ARROW-13615?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Neal Richardson updated ARROW-13615: Fix Version/s: (was: 6.0.0) 7.0.0 > [R] Bindings for stringr::str_to_sentence > - > > Key: ARROW-13615 > URL: https://issues.apache.org/jira/browse/ARROW-13615 > Project: Apache Arrow > Issue Type: Improvement > Components: R >Reporter: Nicola Crane >Priority: Major > Labels: good-first-issue, kernel > Fix For: 7.0.0 > > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (ARROW-13615) [R] Bindings for stringr::str_to_sentence
[ https://issues.apache.org/jira/browse/ARROW-13615?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated ARROW-13615: --- Labels: good-first-issue kernel pull-request-available (was: good-first-issue kernel) > [R] Bindings for stringr::str_to_sentence > - > > Key: ARROW-13615 > URL: https://issues.apache.org/jira/browse/ARROW-13615 > Project: Apache Arrow > Issue Type: Improvement > Components: R >Reporter: Nicola Crane >Assignee: Dragoș Moldovan-Grünfeld >Priority: Major > Labels: good-first-issue, kernel, pull-request-available > Fix For: 7.0.0 > > Time Spent: 10m > Remaining Estimate: 0h > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (ARROW-13615) [R] Bindings for stringr::str_to_sentence
[
https://issues.apache.org/jira/browse/ARROW-13615?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Dragoș Moldovan-Grünfeld updated ARROW-13615:
-
Description:
There is more to this issue than meets the eye. The
{{stringr::str_to_sentence()}} does 2 things:
* capitalise the first word
* if there are multiple sentences provided as a single string, attempts to
find sentence breaks and capitalise the first word of each sentence.
The {{stringr}} implementation wraps {{stringi::str_trans_totitle()}}, which in
turns uses ICU’s BreakIterator to locate specific text boundaries. As a
consequence {{stringr::str_to_title()}} is not able to identify a full stop /
period (".") as a sentence end and does not capitalise words following it.
Thus, there is a discrepancy between behaviour of the {{utf8_capitalize}}
kernel (which capitalises the first word of a string without making any attempt
to break into sentences) and the behaviour of {{stringr::str_to_sentence()}}.
For more extensive discussions around the {{stringi / stringr}} implementation
see {{stringr}} issues [202|https://github.com/tidyverse/stringr/issues/202]
and [231|https://github.com/tidyverse/stringr/issues/231].
Due to the complexity of this issue and the relatively niche use cases, the
recommendation is to postpone implementation until someone requests it.
> [R] Bindings for stringr::str_to_sentence
> -
>
> Key: ARROW-13615
> URL: https://issues.apache.org/jira/browse/ARROW-13615
> Project: Apache Arrow
> Issue Type: Improvement
> Components: R
>Reporter: Nicola Crane
>Assignee: Dragoș Moldovan-Grünfeld
>Priority: Major
> Labels: good-first-issue, kernel, pull-request-available
> Fix For: 7.0.0
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> There is more to this issue than meets the eye. The
> {{stringr::str_to_sentence()}} does 2 things:
> * capitalise the first word
> * if there are multiple sentences provided as a single string, attempts to
> find sentence breaks and capitalise the first word of each sentence.
> The {{stringr}} implementation wraps {{stringi::str_trans_totitle()}}, which
> in turns uses ICU’s BreakIterator to locate specific text boundaries. As a
> consequence {{stringr::str_to_title()}} is not able to identify a full stop /
> period (".") as a sentence end and does not capitalise words following it.
> Thus, there is a discrepancy between behaviour of the {{utf8_capitalize}}
> kernel (which capitalises the first word of a string without making any
> attempt to break into sentences) and the behaviour of
> {{stringr::str_to_sentence()}}.
> For more extensive discussions around the {{stringi / stringr}}
> implementation see {{stringr}} issues
> [202|https://github.com/tidyverse/stringr/issues/202] and
> [231|https://github.com/tidyverse/stringr/issues/231].
> Due to the complexity of this issue and the relatively niche use cases, the
> recommendation is to postpone implementation until someone requests it.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
[jira] [Updated] (ARROW-13615) [R] Bindings for stringr::str_to_sentence
[
https://issues.apache.org/jira/browse/ARROW-13615?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Dragoș Moldovan-Grünfeld updated ARROW-13615:
-
Description:
There is more to this issue than meets the eye. The
{{stringr::str_to_sentence()}} does 2 things:
* capitalise the first word
* if there are multiple sentences provided as a single string, attempts to
find sentence breaks and capitalise the first word of each sentence.
The {{stringr}} implementation wraps {{stringi::str_trans_totitle()}}, which in
turns uses ICU’s BreakIterator to locate specific text boundaries. As a
consequence {{stringr::str_to_title()}} is not able to identify a full stop /
period (".") as a sentence end and does not capitalise words following it.
Thus, there is a discrepancy between behaviour of the {{utf8_capitalize}}
kernel (which capitalises the first word of a string without making any attempt
to break into sentences) and the behaviour of {{stringr::str_to_sentence()}}.
For more extensive discussions around the {{stringi / stringr}} implementation
see {{stringr}} issues [202|https://github.com/tidyverse/stringr/issues/202]
and [231|https://github.com/tidyverse/stringr/issues/231].
Due to the complexity of this issue and the relatively niche use cases, the
recommendation is to postpone implementation.
was:
There is more to this issue than meets the eye. The
{{stringr::str_to_sentence()}} does 2 things:
* capitalise the first word
* if there are multiple sentences provided as a single string, attempts to
find sentence breaks and capitalise the first word of each sentence.
The {{stringr}} implementation wraps {{stringi::str_trans_totitle()}}, which in
turns uses ICU’s BreakIterator to locate specific text boundaries. As a
consequence {{stringr::str_to_title()}} is not able to identify a full stop /
period (".") as a sentence end and does not capitalise words following it.
Thus, there is a discrepancy between behaviour of the {{utf8_capitalize}}
kernel (which capitalises the first word of a string without making any attempt
to break into sentences) and the behaviour of {{stringr::str_to_sentence()}}.
For more extensive discussions around the {{stringi / stringr}} implementation
see {{stringr}} issues [202|https://github.com/tidyverse/stringr/issues/202]
and [231|https://github.com/tidyverse/stringr/issues/231].
Due to the complexity of this issue and the relatively niche use cases, the
recommendation is to postpone implementation until someone requests it.
> [R] Bindings for stringr::str_to_sentence
> -
>
> Key: ARROW-13615
> URL: https://issues.apache.org/jira/browse/ARROW-13615
> Project: Apache Arrow
> Issue Type: Improvement
> Components: R
>Reporter: Nicola Crane
>Assignee: Dragoș Moldovan-Grünfeld
>Priority: Major
> Labels: good-first-issue, kernel, pull-request-available
> Fix For: 7.0.0
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> There is more to this issue than meets the eye. The
> {{stringr::str_to_sentence()}} does 2 things:
> * capitalise the first word
> * if there are multiple sentences provided as a single string, attempts to
> find sentence breaks and capitalise the first word of each sentence.
> The {{stringr}} implementation wraps {{stringi::str_trans_totitle()}}, which
> in turns uses ICU’s BreakIterator to locate specific text boundaries. As a
> consequence {{stringr::str_to_title()}} is not able to identify a full stop /
> period (".") as a sentence end and does not capitalise words following it.
> Thus, there is a discrepancy between behaviour of the {{utf8_capitalize}}
> kernel (which capitalises the first word of a string without making any
> attempt to break into sentences) and the behaviour of
> {{stringr::str_to_sentence()}}.
> For more extensive discussions around the {{stringi / stringr}}
> implementation see {{stringr}} issues
> [202|https://github.com/tidyverse/stringr/issues/202] and
> [231|https://github.com/tidyverse/stringr/issues/231].
> Due to the complexity of this issue and the relatively niche use cases, the
> recommendation is to postpone implementation.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
[jira] [Updated] (ARROW-13615) [R] Bindings for stringr::str_to_sentence
[
https://issues.apache.org/jira/browse/ARROW-13615?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Dragoș Moldovan-Grünfeld updated ARROW-13615:
-
Parent: ARROW-14865
Issue Type: Sub-task (was: Improvement)
> [R] Bindings for stringr::str_to_sentence
> -
>
> Key: ARROW-13615
> URL: https://issues.apache.org/jira/browse/ARROW-13615
> Project: Apache Arrow
> Issue Type: Sub-task
> Components: R
>Reporter: Nicola Crane
>Assignee: Dragoș Moldovan-Grünfeld
>Priority: Major
> Labels: good-first-issue, kernel, pull-request-available
> Fix For: 7.0.0
>
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> There is more to this issue than meets the eye. The
> {{stringr::str_to_sentence()}} does 2 things:
> * capitalise the first word
> * if there are multiple sentences provided as a single string, attempts to
> find sentence breaks and capitalise the first word of each sentence.
> The {{stringr}} implementation wraps {{stringi::str_trans_totitle()}}, which
> in turns uses ICU’s BreakIterator to locate specific text boundaries. As a
> consequence {{stringr::str_to_title()}} is not able to identify a full stop /
> period (".") as a sentence end and does not capitalise words following it.
> Thus, there is a discrepancy between behaviour of the {{utf8_capitalize}}
> kernel (which capitalises the first word of a string without making any
> attempt to break into sentences) and the behaviour of
> {{stringr::str_to_sentence()}}.
> For more extensive discussions around the {{stringi / stringr}}
> implementation see {{stringr}} issues
> [202|https://github.com/tidyverse/stringr/issues/202] and
> [231|https://github.com/tidyverse/stringr/issues/231].
> Due to the complexity of this issue and the relatively niche use cases, the
> recommendation is to postpone implementation.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
[jira] [Updated] (ARROW-13615) [R] Bindings for stringr::str_to_sentence
[ https://issues.apache.org/jira/browse/ARROW-13615?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Neal Richardson updated ARROW-13615: Labels: good-first-issue kernel (was: ) > [R] Bindings for stringr::str_to_sentence > - > > Key: ARROW-13615 > URL: https://issues.apache.org/jira/browse/ARROW-13615 > Project: Apache Arrow > Issue Type: Improvement > Components: R >Reporter: Nic Crane >Priority: Major > Labels: good-first-issue, kernel > Fix For: 6.0.0 > > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (ARROW-13615) [R] Bindings for stringr::str_to_sentence
[ https://issues.apache.org/jira/browse/ARROW-13615?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Neal Richardson updated ARROW-13615: Fix Version/s: 6.0.0 > [R] Bindings for stringr::str_to_sentence > - > > Key: ARROW-13615 > URL: https://issues.apache.org/jira/browse/ARROW-13615 > Project: Apache Arrow > Issue Type: Improvement > Components: R >Reporter: Nic Crane >Priority: Major > Fix For: 6.0.0 > > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (ARROW-13615) [R] Bindings for stringr::str_to_sentence
[ https://issues.apache.org/jira/browse/ARROW-13615?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alessandro Molina updated ARROW-13615: -- Fix Version/s: (was: 6.0.0) 7.0.0 > [R] Bindings for stringr::str_to_sentence > - > > Key: ARROW-13615 > URL: https://issues.apache.org/jira/browse/ARROW-13615 > Project: Apache Arrow > Issue Type: Improvement > Components: R >Reporter: Nicola Crane >Priority: Major > Labels: good-first-issue, kernel > Fix For: 7.0.0 > > -- This message was sent by Atlassian Jira (v8.3.4#803005)
