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

Fix/weight factor #15

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix/weight factor #15

wants to merge 2 commits into from

Conversation

pranaybaldev
Copy link
Contributor

@pranaybaldev pranaybaldev commented Apr 26, 2022

fixes: #14

@eitjuh
Copy link

eitjuh commented May 3, 2022

@pranaybaldev As mentioned: Not sure how to review this, please instruct us how to continue. (I think you mentioned you want to have a good understanding about the formula that we are supposed to use here? Please document it - or whomever can give us that answer)

@sschiessl-bcp
Copy link
Contributor

@sschiessl-bcp
Copy link
Contributor

Ping

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.

weightingFactor missing in calculation if isReverse=true
3 participants