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

Dev type as module #5349

Merged
merged 43 commits into from
Jun 30, 2021
Merged

Dev type as module #5349

merged 43 commits into from
Jun 30, 2021

Conversation

Flowingsun007
Copy link
Contributor

@Flowingsun007 Flowingsun007 commented Jun 29, 2021

新增:type_as、long module及test case

  • api docs:
    Snip20210629_3
    Snip20210629_5

  • doc test:
    Snip20210629_1

Flowingsun007 and others added 30 commits June 10, 2021 11:15
not flow.unittest.env.eager_execution_enabled(),
".numpy() doesn't work in lazy mode",
)
class TestPow(flow.unittest.TestCase):
Copy link
Contributor

Choose a reason for hiding this comment

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

名字改一下

".numpy() doesn't work in lazy mode",
)
class TestPow(flow.unittest.TestCase):
def test_pow_forward(test_case):
Copy link
Contributor

Choose a reason for hiding this comment

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

这里也是


def _test_type_as(test_case, shape, device):
np_input = np.random.rand(*shape)
input = flow.tensor(np_input, dtype=flow.float, device=device)
Copy link
Contributor

Choose a reason for hiding this comment

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

这里感觉测试不全面,我理解可以写两个dtype的数组来遍历着测试

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好的

@oneflow-ci-bot oneflow-ci-bot requested review from oneflow-ci-bot and removed request for oneflow-ci-bot June 29, 2021 15:54
@oneflow-ci-bot oneflow-ci-bot requested review from oneflow-ci-bot and removed request for oneflow-ci-bot June 29, 2021 17:20
@oneflow-ci-bot oneflow-ci-bot requested review from oneflow-ci-bot and removed request for oneflow-ci-bot June 29, 2021 18:22
@oneflow-ci-bot oneflow-ci-bot requested review from oneflow-ci-bot and removed request for oneflow-ci-bot June 29, 2021 20:53
@oneflow-ci-bot oneflow-ci-bot merged commit ec5dc37 into master Jun 30, 2021
@oneflow-ci-bot oneflow-ci-bot deleted the dev_type_as_module branch June 30, 2021 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants