-
-
Notifications
You must be signed in to change notification settings - Fork 522
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 (Token detection): Do after decoding ends #8340
Conversation
self._post_process(refresh_balances=refresh_balances) | ||
maybe_detect_new_tokens(self.database) | ||
return events |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this doesn't make any external request right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no it only process the data in the db
a2d41f8
to
a0275b5
Compare
rotkehlchen/tasks/assets.py
Outdated
event.location_label not in database.get_single_blockchain_addresses( | ||
cursor=cursor, | ||
blockchain=chain, | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mmm this is one query per event... can't you just do
{address_tuple[0] for address_tuple in cursor.execute(
'SELECT UNIQUE account FROM blockchain_accounts;',
)}
at the start of the function?
Updated aave test due to
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## bugfixes #8340 /- ##
=========================================
Coverage 53.80% 53.81%
=========================================
Files 1707 1708 1
Lines 168704 168773 69
Branches 13956 13969 13
=========================================
Hits 90774 90824 50
- Misses 75505 75528 23
Partials 2425 2421 -4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: OjusWiZard <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
rotki/test-caching/tree/fix/decoder was successfully merged |
Checklist