-
Notifications
You must be signed in to change notification settings - Fork 244
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
[release] updates changelog for 0.2.118 #13127
Conversation
0258ba2
to
8d8bf87
Compare
@@ -33,6 +33,18 @@ Please note that **forward compatibility should not be expected, especially | |||
relating to file formats**: this means that it may not be possible to use | |||
an earlier version of Hail to read files written in a later version. | |||
|
|||
## Version 0.2.118 | |||
|
|||
Released 2023-05-30 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you change the date?
hail/python/hail/docs/change_log.md
Outdated
|
||
- (hail#13113) `hail.ggplot2.geom_point` now displays a legend group for a column even when it has only one value in it. | ||
- (hail#13089) Requester pays buckets now work in `hailtop.fs` and `hl.hadoop_*`. This has been broken since at least 0.2.115. | ||
- (hail#12995) Fixed bug causing poor performance and memory leaks for `Matrix.annotate_rows` aggregations. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about #12569 as new features?
#13081 as a new feature?
I don't think #13126 is a bug fix, but @chrisvittal can chime in?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#13113 was the reason I wanted to cut a release in the first place, so I'd like to wait for it to go in first.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about #12569 as new features? #13081 as a new feature?
I don't think #13126 is a bug fix, but @chrisvittal can chime in?
That is a bugfix. One partition pipelines that we execute on the driver that then failed because of transient errors would not retry.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added changelog messages for #12569 and #13126. I'll change the date as well once this is actually ready to go in. This PR is WIP pending the merge of at least #13113, #13147, and any other PRs that are necessary to make CI work again. We could also wait for #13089 and #12995, but I don't have strong opinions either way about that. @danking, thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@iris-garden your PR merged, let's fix up the change log and get this release out!
@iris-garden bump! |
No description provided.