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.

Sign up for GitHub

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

Jump to bottom

Add interruption ability in case rate limit #379

Closed
wants to merge 111 commits into from
Closed

Add interruption ability in case rate limit #379

wants to merge 111 commits into from

Conversation

ZIMkaRU
Copy link
Member

@ZIMkaRU ZIMkaRU commented May 17, 2024

This PR adds interruption ability in case rate limit. When getting Rate Limit or cool down due to Rate Limit for 1min occurs, it needs to provide a feature to interrupt the tax report and sync being processed

Depends on this PR:

ZIMkaRU added 30 commits April 26, 2024 09:10
@ZIMkaRU
Add TransactionTaxReport service type
fda5cec
@ZIMkaRU
Add TransactionTaxReport service
a262335
@ZIMkaRU
Add TransactionTaxReport service into di
013a0c3
@ZIMkaRU
Add data consistency checker for TransactionTaxReport
65d331d
@ZIMkaRU
Add params schema for TransactionTaxReport
415e184
@ZIMkaRU
Add getTransactionTaxReport entrypoint to main service
084a43b
@ZIMkaRU
Add report file job data for TransactionTaxReport
b99c94e
@ZIMkaRU
Add getTransactionTaxReportFile entrypoint to main service
cf609b0
@ZIMkaRU
Add ability to get trades for trx tax report
84b3f84
@ZIMkaRU
Implement transaction tax report calc
095cf3c
@ZIMkaRU
Consider trades in tax reports that were not closed in prev period
219a03e
@ZIMkaRU
Add CurrencyConversionError class
86dee5f
@ZIMkaRU
Use CurrencyConversionError class
fd317bf
@ZIMkaRU
Add CurrencyPairSeparationError class
b1205ef
@ZIMkaRU
Use CurrencyPairSeparationError class
4e84ad1
@ZIMkaRU
Speed up back iteration
a5d2408
@ZIMkaRU
Optimize trx tax report calc performance
e11ab8b
@ZIMkaRU
Increase request timeouts for dev mode
1bb8ec9
@ZIMkaRU
Prevent event loop locking for trx tax report
7ad37ec
@ZIMkaRU
Use sale trx to declare into tax report
921c0bf
@ZIMkaRU
Improve lookup buy trx with unrealized profit
aaa1bce
@ZIMkaRU
Rework trx tax report to consider deposit-withdrawal
3e91d51
@ZIMkaRU
Implement currency conversion for trx tax report
3a41322
@ZIMkaRU
Speed up currency conversion for trx tax report
f20ba5f
@ZIMkaRU
Add TrxTaxReportGenerationTimeoutError error class
ec92940
@ZIMkaRU
Use TrxTaxReportGenerationTimeoutError error class
40c6e8b
@ZIMkaRU
Add ability to send ws event when trx tax report generated
13c8eba
@ZIMkaRU
Add ability to make trx tax report in background
7ed653c
@ZIMkaRU
Add endpoint to make trx tax report in background
2d03cbb
@ZIMkaRU
Improve data consistency checker for trx tax report
912d8e1
ZIMkaRU added 19 commits May 14, 2024 09:24
@ZIMkaRU
Add interrupter into lookUpTrades helper
186c8d0
@ZIMkaRU
Add ability to interrupt trx tax report
8f41e02
@ZIMkaRU
Simplify orig session obj ref return
9e1efed
@ZIMkaRU
Fix pub trades args for trx tax report
a63e601
@ZIMkaRU
Set name into sync interrupter
d5ba950
@ZIMkaRU
Add ability to set name into interrupter using factory
7a94571
@ZIMkaRU
Set name into trx tax report interrupter
521138a
@ZIMkaRU
Add ability to interrupt user operation
28e742d
@ZIMkaRU
Add interruptOperations endpoint
2524513
@ZIMkaRU
Add params schema for interruptOperations endpoint
a3e88d1
@ZIMkaRU
Add TransactionTaxReport service into di singleton scope
43fc721
@ZIMkaRU
Add test coverage for transaction tax report interruption
dd66db3
@ZIMkaRU
Add test case with wrong param for interruptOperations endpoint
56242a6
@ZIMkaRU
Fix interrupter name for trx tax report
cfb8636
@ZIMkaRU
Implement interruption ability in BfxApiRouter service
5690453
@ZIMkaRU
Turn off interrupter for full snapshot report csv writer
850abda
@ZIMkaRU
Turn off interrupter for full tax report csv writer
b2392c7
@ZIMkaRU
Turn off interrupter for sub.account.api.data service
b0b69d3
@ZIMkaRU
Improve interrupt handling in main service
c266e2f
This was referenced May 21, 2024
Merged
Merged
Merged
@ZIMkaRU ZIMkaRU mentioned this pull request May 30, 2024
Merged
This was referenced Jun 20, 2024
Merged
Merged
@ZIMkaRU ZIMkaRU changed the title [WAITING] Add interruption ability in case rate limit Add interruption ability in case rate limit Jun 25, 2024
@ZIMkaRU
Copy link
Member Author

ZIMkaRU commented Jun 25, 2024

done in small separated PRs

@ZIMkaRU ZIMkaRU closed this Jun 25, 2024
This was referenced Jul 31, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ZIMkaRU