Skip to content

Conversation

@whitesockcat
Copy link

No description provided.

a = [0.0 , 0.0]
i = pos
nu = 0.0
while i < n and nu < 1e-5:
Copy link
Owner

Choose a reason for hiding this comment

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

Oh, the main reason for implement compute_weight in C++ is that python's loops are too slow and this loop will be a too slow for large batch size. Can you provide performance report which show that python code isn't slower than c++ implementation?

Copy link
Author

Choose a reason for hiding this comment

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

Good point. However i haven't compared yet. Thanks for your patience again.

Copy link

Choose a reason for hiding this comment

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

I cann't run build.py. So i try the python version , but it doesn't work. Have you succeed?

Copy link
Owner

Choose a reason for hiding this comment

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

@01lin currently this implementation is not compatible with pytorch 0.5+

Copy link

Choose a reason for hiding this comment

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

@belbes With pytorch 0.4.0 and python3, I also cann't run build.py. And the error is " cffi.error.VerificationError: CompileError: command 'gcc' failed with exit status 1". Can you help me?

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