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

Assert Exception: _sell_asset->options.whitelist_markets.find #1018

Closed
litepresence opened this issue Jun 4, 2018 · 3 comments
Closed

Assert Exception: _sell_asset->options.whitelist_markets.find #1018

litepresence opened this issue Jun 4, 2018 · 3 comments
Assignees
Labels
1b User Story The User Story details a requirement. It may reference a parent Epic. It may reference child Task(s) 2c Ready for Development Status indicating the Requirements are sufficent to begin designing a solution 3c Enhancement Classification indicating a change to the functionality of the existing imlementation 6 API Impact flag identifying the application programing interface (API) 6 UX Impact flag identifying the User Interface (UX)

Comments

@litepresence
Copy link
Contributor

litepresence, [04.06.18 09:50]
can you help understand what this assertion means in human terms? occurring during pybitshares market buy/sell grapheneapi.exceptions.

RPCError: Assert Exception: _sell_asset->options.whitelist_markets.find(_receive_asset->id) != _sell_asset->options.whitelist_markets.end()

Stefan Schießl, [04.06.18 09:50]
The issuer of the asset has closed down trading for this market

It would be helpful if the exception arguments included the above statement

@ryanRfox ryanRfox added 1b User Story The User Story details a requirement. It may reference a parent Epic. It may reference child Task(s) 2b Gathering Requirements Status indicating currently refining User Stories and defining Requirements 3c Enhancement Classification indicating a change to the functionality of the existing imlementation 6 API Impact flag identifying the application programing interface (API) 6 UX Impact flag identifying the User Interface (UX) labels Jun 4, 2018
@litepresence
Copy link
Contributor Author

litepresence commented Jun 5, 2018

re: user story

I had botscript algo trading on the dex for several days, then suddenly it stopped working and I was getting this error repeatedly whenever algo was attempting buy/sell ops. I spent 2 days searching for a bug in my code (or pybitshares) that might have caused it, and the error argument given via the RPC call was not very discernible or helpful. Eventually, I found out that trading was disabled on all openledger; ie OPEN.* assets via telegram dev chat room. Hence the blue chip market OPEN.BTC/BTS was no longer whitelisted; randomly it didn't exist; it was a "black swan" event of sorts from my perspective, and there was indeed nothing wrong with my code; I just didn't realize that, in human speak:

The issuer of the asset has closed down trading for this market

@litepresence
Copy link
Contributor Author

related

Codaone/DEXBot#183

exceptions.RPCError: Assert Exception: is_authorized_asset( d, *_seller, *_receive_asset )

@pmconrad pmconrad added 2c Ready for Development Status indicating the Requirements are sufficent to begin designing a solution and removed 2b Gathering Requirements Status indicating currently refining User Stories and defining Requirements labels Nov 8, 2018
@jmjatlanta jmjatlanta self-assigned this Jan 24, 2019
@abitmore
Copy link
Member

Fixed by #1531.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1b User Story The User Story details a requirement. It may reference a parent Epic. It may reference child Task(s) 2c Ready for Development Status indicating the Requirements are sufficent to begin designing a solution 3c Enhancement Classification indicating a change to the functionality of the existing imlementation 6 API Impact flag identifying the application programing interface (API) 6 UX Impact flag identifying the User Interface (UX)
Projects
None yet
Development

No branches or pull requests

5 participants