Skip to content
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

Add continuous uniform distribution #15813

Closed
wants to merge 5 commits into from

Conversation

dowi
Copy link

@dowi dowi commented Mar 11, 2021

Test plan - (Please fill in how you tested your changes)

Please make sure your submission complies with our Development, Formatting, and Commit Message guidelines. Don't forget to follow our attribution guidelines for any code copied from other projects.

Fill in the release notes towards the bottom of the PR description.
See Release Notes Guidelines for details.

== RELEASE NOTES ==

General Changes
added continuous uniform distribution
* ...

Hive Changes
* ...
* ...

If release note is NOT required, use:

== NO RELEASE NOTE ==

@leepface
Copy link
Contributor

Code review:

  • squash commits together into one commit

@dowi dowi force-pushed the continuous_uniform_distribution branch from d76cf83 to 217cf54 Compare March 16, 2021 07:26
@dowi
Copy link
Author

dowi commented Mar 16, 2021

Commits squashed

@dowi dowi force-pushed the continuous_uniform_distribution branch from 217cf54 to ab00bb5 Compare March 16, 2021 12:59
@dowi dowi force-pushed the continuous_uniform_distribution branch from ab00bb5 to 07cc088 Compare March 17, 2021 08:34
@talgalili
Copy link
Contributor

talgalili commented May 10, 2021

Hey @dowi - I see that this diff has merge conflicts, could you please resolve them?

@ajaygeorge
Copy link
Contributor

@dowi Can you please take a look at resolving the merge conflicts.
cc @jbapple

@rongrong
Copy link
Contributor

For commit message, please use Add instead of added. Add continuous uniform distribution functions

@dowi dowi changed the title added continuous uniform distribution Add continuous uniform distribution Oct 28, 2021
@stale
Copy link

stale bot commented Apr 28, 2022

This pull request has been automatically marked as stale because it has not had recent activity. If you'd still like this PR merged, please comment on the task, make sure you've addressed reviewer comments, and rebase on the latest master. Thank you for your contributions!

@stale stale bot added the stale label Apr 28, 2022
@wanglinsong wanglinsong requested review from steveburnett and a team as code owners July 6, 2024 04:32
@wanglinsong wanglinsong requested a review from presto-oss July 6, 2024 04:32
@steveburnett
Copy link
Contributor

Closing due to lack of activity and age. If this is still relevant, please re-open this PR, rebase, and resolve any file conflicts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants