Skip to content
This repository has been archived by the owner on Feb 1, 2024. It is now read-only.

Fix divide by zero error #22

Merged
merged 6 commits into from
Oct 11, 2018
Merged

Fix divide by zero error #22

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
5e86b11
Print strat (#1)
Reidmcc Oct 2, 2018
4506fb2
doc clarify need for full paths for config files (#2)
Reidmcc Oct 2, 2018
d1fb1a2
Catch up to root master (#3)
Reidmcc Oct 2, 2018
0bcf4ef
Merge pull request #4 from lightyeario/master
Reidmcc Oct 2, 2018
a48b282
Update sellSideStrategy.go
Reidmcc Oct 4, 2018
d87e7da
Update sellSideStrategy.go
Reidmcc Oct 11, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions plugins/sellSideStrategy.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,13 @@ func (s *sellSideStrategy) UpdateWithOps(offers []horizon.Offer) (ops []build.Tr
// hitCapacityLimit can be updated below
targetPrice := s.currentLevels[i].Price
targetAmount := s.currentLevels[i].Amount
if targetPrice.AsFloat() == 0 {
return nil, nil, fmt.Errorf("targetPrice is 0")
}
if targetAmount.AsFloat() == 0 {
return nil, nil, fmt.Errorf("targetAmount is 0")
}

if s.divideAmountByPrice {
targetAmount = *model.NumberFromFloat(targetAmount.AsFloat()/targetPrice.AsFloat(), targetAmount.Precision())
}
Expand Down