Skip to content

Conversation

@hekaisheng
Copy link
Contributor

What do these changes do?

Check mt.random.rand and mt.random.randn parameters and raises ValueError if argument key not starts with '_' except chunk_size.

Related issue number

fixes #91

@hekaisheng hekaisheng added type: bug Something isn't working mod: tensor to be backported Indicate that the PR need to be backported to stable branch labels Jan 4, 2019
@hekaisheng hekaisheng added this to the v0.2.0a1 milestone Jan 4, 2019
@hekaisheng hekaisheng requested a review from qinxuye January 4, 2019 08:12
Copy link
Collaborator

@qinxuye qinxuye left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@qinxuye qinxuye left a comment

Choose a reason for hiding this comment

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

LGTM except the comments

hekaisheng added 2 commits January 4, 2019 17:28
Copy link
Collaborator

@qinxuye qinxuye left a comment

Choose a reason for hiding this comment

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

LGTM except the comments

Copy link
Collaborator

@qinxuye qinxuye left a comment

Choose a reason for hiding this comment

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

LGTM

@qinxuye qinxuye merged commit 30902ce into mars-project:master Jan 4, 2019
hekaisheng added a commit to hekaisheng/mars that referenced this pull request Jan 7, 2019
* fix rand&rand parameters * remove kw parameter in random functions * refine in random_integers (cherry picked from commit 30902ce)
hekaisheng added a commit to hekaisheng/mars that referenced this pull request Jan 7, 2019
* fix rand&rand parameters * remove kw parameter in random functions * refine in random_integers (cherry picked from commit 30902ce)
qinxuye pushed a commit that referenced this pull request Jan 7, 2019
* fix rand&rand parameters * remove kw parameter in random functions * refine in random_integers
@wjsi wjsi added backported already PR has been backported and removed to be backported Indicate that the PR need to be backported to stable branch labels Feb 28, 2019
@hekaisheng hekaisheng deleted the fix_rand_params branch July 16, 2019 11:18
fyrestone added a commit that referenced this pull request Mar 14, 2023
* Implements ColumnPruning optimization rule (#30) Co-authored-by: ChengjieLi <chengjieli23@outlook.com> Co-authored-by: aresnow <aresnow1@gmail.com> * Fix execution hang on optimization error (#33) * enh: Optimize groupby nunique implementation (#38) * enh: Support passing mars objects when call df.map_chunk (#41) * enh: Support `read_parquet` for GPU (#45) * enh: Merge small data on shuffle mapper side (#49) * Fix: `series.drop_duplicates()` failed (#53) * Fix: dataframe.isna on scaler args (#55) * enh: Support df or series type for apply function (#56) * enh: Support running TPC-H queries on GPU (#60) * Avoid creating too much thread pools (#62) * enh: Optimize DataFrame.isin (#67) * tst: Fix gpu CI (#42) * enh: Refactor `read_buffers` & `write_buffers` for GPU (#68) * Fix: ensure dataframe.all acts identical to pandas (#79) * fix: Fix wrong results of `DataFrame.replace` (#86) * enh: Add `skip_infer` paramter for all use defined functions (#76) * fix: Modify signature of `df.map_chunk` (#87) * BUG: DataFrame.agg with built-in functions (#91) Co-authored-by: UranusSeven <109661872+UranusSeven@users.noreply.github.com> * Fix dataframe.ewm (#97) * BUG: Fix dataframe.sample() (#99) * BUG: df.apply with list input returns wrong answer when axis=1 (#100) * BUG: Fix md.unique (#102) * Fix implement ordered in md.cut (#104) Co-authored-by: UranusSeven <109661872+UranusSeven@users.noreply.github.com> * Fix: map_chunk output type inference (#111) * Enh: support groupby plot (#113) * ENH: Clear default context after execution (#115) * BUG: fix supervisor start method (#116) * BUG: Fix `date_range` and pin sphinx<6.0.0 (#118) * BUG: handle missing merge key (#124) * BUG: Fix `read_csv` with specific names and header (#130) * BUG: needed cols being pruned (#134) * BUG: Fix `read_parquet` with latest pyarrow (#135) * BUG: Suppress `FutureWarning` (#108) * Fix merge * Add `_repr_mimebundle_` ... as `_repr_svg_` is deprecated in `Source` class of `graphviz`. --------- Co-authored-by: UranusSeven <109661872+UranusSeven@users.noreply.github.com> Co-authored-by: ChengjieLi <chengjieli23@outlook.com> Co-authored-by: aresnow <aresnow1@gmail.com> Co-authored-by: Chengjie Li <109656400+ChengjieLi28@users.noreply.github.com> Co-authored-by: aresnow1 <109642806+aresnow1@users.noreply.github.com> Co-authored-by: qianduoduo0904 <109654808+qianduoduo0904@users.noreply.github.com> Co-authored-by: 黄浩杰 <65081722+hoarjour@users.noreply.github.com> Co-authored-by: 刘宝 <po.lb@antgroup.com> Co-authored-by: Wenjun Si <swj0066@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backported already PR has been backported mod: tensor type: bug Something isn't working

3 participants