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

Modify black scholes example to use internal RNG from library instead of std::rand #418

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

roshanr95
Copy link
Contributor

This adresses #276 . Also check if the values are correct... (esp. the put price)
On my setup,

$ ./example/black_scholes
device: GeForce GT 650M
option 0 call price: 15.189
option 0 put price: 0

UPDATE: Travis also reports the same value, so I guess the problem of different values on different platforms is solved. I have no clue about correctness though...

@kylelutz
Copy link
Collaborator

The black-scholes code should still be correct, but different input values will give different output values. Having 0 exactly for the put price is a little weird, I'd try changing the RNG seed value to something else and then use those results.

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.

2 participants