core icon indicating copy to clipboard operation
core copied to clipboard

Coinbase integration reports incorrect volume for SHIBA and DOGE by a factor of 10

Open perstromqvist opened this issue 1 year ago • 11 comments

The problem

2000 DOGE on Coinbase is being reported as 20000 DOGE in home assistant. Same for SHIB where 20,000,000 is being reported as 200 million.

What version of Home Assistant Core has the issue?

Core-2024.12.3

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Coinbase

Link to integration documentation on our website

No response

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

perstromqvist avatar Dec 15 '24 12:12 perstromqvist

Hey there @tombrien, mind taking a look at this issue as it has been labeled with an integration (coinbase) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of coinbase can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign coinbase Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


coinbase documentation coinbase source (message by IssueLinks)

home-assistant[bot] avatar Dec 15 '24 19:12 home-assistant[bot]

Please can you download the diagnostics for the Coinbase serivce and attach them here.

TomBrien avatar Dec 16 '24 16:12 TomBrien

Please attach via the github website and not as an email reply

TomBrien avatar Dec 16 '24 17:12 TomBrien

As seen in the diagnostics the integration won't read SOL at all.

perstromqvist avatar Dec 17 '24 09:12 perstromqvist

So at the moment I'm unable to recreate this. I got about 20,000 SHIB and see the same in Home Assistant as on the Coinbase website. Has it ever provided the correct reading?

Also, looking at the diagnostics I can't see a reason SOL wouldn't be working as the data is being collected from the API.

TomBrien avatar Dec 18 '24 21:12 TomBrien

I did a fresh install on an HA Green with the integration. In short, it has never shown the right amount for SHIB, DOGE, nor the value of SOL.

I'll just make a helper then to recalculate the numbers.

Den ons 18 dec. 2024 kl 22:57 skrev Tom Brien @.***>:

So at the moment I'm unable to recreate this. I got about 20,000 SHIB and see the same in Home Assistant as on the Coinbase website. Has it ever provided the correct reading?

Also, looking at the diagnostics I can't see a reason SOL wouldn't be working as the data is being collected from the API.

— Reply to this email directly, view it on GitHub https://github.com/home-assistant/core/issues/133268#issuecomment-2552339968, or unsubscribe https://github.com/notifications/unsubscribe-auth/A5PIBKR4P4GIG26DHU4HTBD2GHVURAVCNFSM6AAAAABTUPKC3WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJSGMZTSOJWHA . You are receiving this because you authored the thread.Message ID: @.***>

perstromqvist avatar Dec 20 '24 09:12 perstromqvist

Is a sensor being created for SOL? The Coinbase API is unreliable and inconsistent at times. I've had a good look at raw response from Coinbase and there doesn't seem to be a scaling factor ever being provided on any wallets.

TomBrien avatar Dec 20 '24 09:12 TomBrien

Yes, a sensor is created. It reads 0,00 SOL//Per+46 70 555 70 7720 dec. 2024 kl. 10:49 skrev Tom Brien @.***>: Is a sensor being created for SOL? The Coinbase API is unreliable and inconsistent at times. I've had a good look at raw response from Coinbase and there doesn't seem to be a scaling factor ever being provided on any wallets.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

perstromqvist avatar Dec 20 '24 13:12 perstromqvist

I'm interested if this is now resolved. I still can't reproduce it but there's a change in 2025.2.2 which addresses a different issue but I think may help here

TomBrien avatar Feb 11 '25 08:02 TomBrien

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.