Skip to content

[r] Release 0.1.19#621

Merged
aaronwolen merged 6 commits intomain-oldfrom
aaronwolen/release-0-1-19
Dec 19, 2022
Merged

[r] Release 0.1.19#621
aaronwolen merged 6 commits intomain-oldfrom
aaronwolen/release-0-1-19

Conversation

@aaronwolen
Copy link
Copy Markdown
Member

@aaronwolen aaronwolen commented Dec 19, 2022

Updates package version number to 0.1.19 in preparation for release.

Other changes:

  • Updated setup-r GitHub Action to v2

Note: CI test failures are due to a core issue that will be fixed in 2.13.1.

@aaronwolen aaronwolen changed the base branch from main to main-old December 19, 2022 20:20
@aaronwolen aaronwolen requested a review from johnkerl December 19, 2022 20:22
@aaronwolen aaronwolen marked this pull request as ready for review December 19, 2022 20:26
@eddelbuettel
Copy link
Copy Markdown
Contributor

eddelbuettel commented Dec 19, 2022

Please change

- uses: actions/checkout@v2
as well as
- uses: actions/checkout@v2

from v2 to v3

Copy link
Copy Markdown
Contributor

@johnkerl johnkerl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With one change requested 🚢

apis/r/NEWS.md Outdated

## Changes

- Increased capacity of `AssayMatrix` arrays from 1,000 to 100,000 to improve remote-read performance (#543)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was a move from 100,000 down to 1,000 in order to increase read-locality for point queries

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

@aaronwolen aaronwolen merged commit 85d7e6e into main-old Dec 19, 2022
@aaronwolen aaronwolen deleted the aaronwolen/release-0-1-19 branch December 19, 2022 20:40
Copy link
Copy Markdown
Contributor

@eddelbuettel eddelbuettel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for updating the checkout actions too

@aaronwolen
Copy link
Copy Markdown
Member Author

Thanks for spotting that @eddelbuettel!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants