Skip to content
This repository was archived by the owner on Jan 18, 2023. It is now read-only.

Conversation

@bweick
Copy link
Contributor

@bweick bweick commented Jan 29, 2019

No description provided.

Copy link
Contributor

@felix2feng felix2feng left a comment

Choose a reason for hiding this comment

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

Looks good to me

* @param _setTokenFactory The address of the SetTokenFactory
* @param _auctionLibrary The address of auction price curve to use in rebalance
* @param _auctionTimeToPivot The amount of time until pivot reached in rebalance
* @param _btcMultiplier With _ethMultiplier, the ratio amount of wbtc to include
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we can add a comment above about why / how this multiplier can be used.

btcMultiplier,
ethMultiplier
);
console.log(expectedNextSetParams['units']);
Copy link
Contributor

Choose a reason for hiding this comment

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

remove me

ethMultiplier
);
console.log(expectedNextSetParams['units']);
expect(JSON.stringify(nextSetUnits)).to.be.eql(JSON.stringify(expectedNextSetParams['units']));
Copy link
Contributor

Choose a reason for hiding this comment

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

Whats the [units] for?

@coveralls
Copy link

coveralls commented Jan 29, 2019

Pull Request Test Coverage Report for Build 3939

  • 9 of 9 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 3926: 0.0%
Covered Lines: 865
Relevant Lines: 865

💛 - Coveralls

@bweick bweick force-pushed the brian/rebalance_manager_update branch from 97f1a0a to 969388f Compare January 29, 2019 15:56
@bweick bweick merged commit d1fdaf2 into master Jan 29, 2019
@bweick bweick deleted the brian/rebalance_manager_update branch January 29, 2019 18:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants