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

[OpTestPy] add greater grid_sampler 2 op all utest #7994

Merged

Conversation

daming5432
Copy link
Collaborator

No description provided.

@paddle-bot-old
Copy link

Thanks for your contribution!

zhenlin-work
zhenlin-work previously approved these changes Dec 23, 2021
Copy link
Collaborator

@zhenlin-work zhenlin-work left a comment

Choose a reason for hiding this comment

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

LGTM

outputs={"Out": ["cast_data_out"], },
attrs={"in_dtype": int(0),
"out_dtype": int(2)})
cast_out.outputs_dtype = {"cast_data_out": np.int32}
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个outputs_dtype为啥不写到opconfig里面呢

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

也行,改了

st.sampled_from(["type_int64", "type_float", "type_int32"]))

############### ToDo ####################
assume(process_type != "type_int32")
Copy link
Collaborator

Choose a reason for hiding this comment

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

将类型约束放在is_program_valid函数中

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

已改

return True

def sample_program_configs(self, draw):
in_shape = draw(
Copy link
Collaborator

Choose a reason for hiding this comment

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

in_shape -> in_shape_x

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

已改

return self.get_predictor_configs(), ["grid_sampler"], (1e-5, 1e-5)

def add_ignore_pass_case(self):
def teller1(program_config, predictor_config):
Copy link
Collaborator

Choose a reason for hiding this comment

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

grid_sampler 输出有diff是吗?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

嗯,cpu和gpu都有diff

zhaoyang-star
zhaoyang-star previously approved these changes Dec 24, 2021
Copy link
Collaborator

@zhaoyang-star zhaoyang-star 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

@zhaoyang-star zhaoyang-star left a comment

Choose a reason for hiding this comment

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

LGTM

@zhaoyang-star zhaoyang-star merged commit 50680c9 into PaddlePaddle:develop Dec 30, 2021
WeiLi233 pushed a commit to WeiLi233/Paddle-Lite that referenced this pull request Mar 29, 2022
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