Skip to content

Conversation

@AgainstEntropy
Copy link

No description provided.

…_pos

* Removed the overridden max_nmse_err method in the test_get_rel_pos struct as it was deemed unnecessary.
* Added multiple new test cases for various attention configurations, enhancing coverage for the get_rel_pos functionality.
@AgainstEntropy
Copy link
Author

Test results of ./build-cuda/bin/test-backend-ops -o GET_REL_POS:

image

@bluebread
Copy link
Owner

@AgainstEntropy Please take a look at the error message for the failures and feel free to tell me what I can help you.

@AgainstEntropy
Copy link
Author

@AgainstEntropy Please take a look at the error message for the failures and feel free to tell me what I can help you.

Seems the error was raised from this line:

*((T *) dp) = 0;

I think we should be able to fix it by replacing the current *((T *) dp) = 0; with *((T *) dp) = 0.0;

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.

2 participants