You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "paleolimbot (via GitHub)" <gi...@apache.org> on 2024/05/09 20:28:17 UTC
[PR] feat(r): Add experimental `nanoarrow_vctr` to wrap a list of arrays [arrow-nanoarrow]
paleolimbot opened a new pull request, #461:
URL: https://github.com/apache/arrow-nanoarrow/pull/461
Work in progress!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat(r): Add experimental `nanoarrow_vctr` to wrap a list of arrays [arrow-nanoarrow]
Posted by "Robinlovelace (via GitHub)" <gi...@apache.org>.
Robinlovelace commented on PR #461:
URL: https://github.com/apache/arrow-nanoarrow/pull/461#issuecomment-2112053694
Amazing work, great to see these changes rolling in in real time!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat(r): Add experimental `nanoarrow_vctr` to wrap a list of arrays [arrow-nanoarrow]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #461:
URL: https://github.com/apache/arrow-nanoarrow/pull/461#issuecomment-2104900503
## [Codecov](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/461?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
Attention: Patch coverage is `97.20280%` with `8 lines` in your changes are missing coverage. Please review.
> Project coverage is 89.09%. Comparing base [(`9935713`)](https://app.codecov.io/gh/apache/arrow-nanoarrow/commit/9935713a4a853e8cdcd32ec42b983df04093ae23?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) to head [(`6aefdd2`)](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/461?dropdown=coverage&src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
> Report is 39 commits behind head on main.
| [Files](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/461?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [r/R/vctr.R](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/461?src=pr&el=tree&filepath=r%2FR%2Fvctr.R&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ci9SL3ZjdHIuUg==) | 96.29% | [5 Missing :warning: ](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/461?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [r/src/convert\_array\_stream.c](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/461?src=pr&el=tree&filepath=r%2Fsrc%2Fconvert_array_stream.c&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ci9zcmMvY29udmVydF9hcnJheV9zdHJlYW0uYw==) | 91.30% | [2 Missing :warning: ](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/461?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [r/src/materialize.c](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/461?src=pr&el=tree&filepath=r%2Fsrc%2Fmaterialize.c&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ci9zcmMvbWF0ZXJpYWxpemUuYw==) | 98.11% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/461?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## main #461 +/- ##
==========================================
+ Coverage 88.76% 89.09% +0.32%
==========================================
Files 82 89 +7
Lines 14633 15958 +1325
==========================================
+ Hits 12989 14217 +1228
- Misses 1644 1741 +97
```
</details>
[:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/461?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
:loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat(r): Add experimental `nanoarrow_vctr` to wrap a list of arrays [arrow-nanoarrow]
Posted by "paleolimbot (via GitHub)" <gi...@apache.org>.
paleolimbot merged PR #461:
URL: https://github.com/apache/arrow-nanoarrow/pull/461
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org