Thanks to visit codestin.com
Credit goes to github.com

Skip to content

En 1570 no cython event handlers #16

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
Merged
merged 72 commits into from
Jun 19, 2020
Merged

En 1570 no cython event handlers #16

merged 72 commits into from
Jun 19, 2020

Conversation

@iamishalkin
Copy link
Member

@iamishalkin iamishalkin commented Jun 11, 2020

Technical part for review. Documentation in progress

iamishalkin added 30 commits May 29, 2020 14:38
@iamishalkin
[EN-1570] initial working realisation
95116ad
@iamishalkin
[EN-1570] manual py func added
99fd255
@iamishalkin
[EN-1570] No crash implementation
24c24a2
@iamishalkin
[EN-1570] DXFeedPy to new logic
bd93a69
@iamishalkin
[EN-1570] dq deleted
ecdf180
@iamishalkin
[EN-1570] fixed u_data pointer
d6c6f42
@iamishalkin
[EN-1570] event handler in separate class
8256fed
@iamishalkin
[EN-1570] warn message added
23e8a50
@iamishalkin
[EN-1570] Merge branch 'master' of github.com:dxFeed/dxfeed-python-ap…
a09ff07 
…i into EN-1570-no-cython
@iamishalkin
[EN-1570] pyproject quotes refactor + build_inplace task added
286c5f8
@iamishalkin
[EN-1570] redundant logic removed from process last error
4cf8f9a
@iamishalkin
[EN-1570] common method to close con users added
4583389
@iamishalkin
[EN-1570] write data columns to event handler instead of subs class
ac20760
@iamishalkin
[EN-1570] default handler added
2ad73be
@iamishalkin
[EN-1570] Endpoint class corrected
93d1a99
@iamishalkin
[EN-1570] Subscription class changes added
675f1dc
@iamishalkin
[EN-1570] all listeners support event handlers
6d5cc86
@iamishalkin
[EN-1570] default event handler test added
a75b610
@iamishalkin
[EN-1570] refactoring of Subscription class
b97ebdd
@iamishalkin
[EN-1570] subscription example added
1dbd654
@iamishalkin
[EN-1570] subscription example edited
a1f322a
@iamishalkin
[EN-1570] handler docstrings
a5bd029
@iamishalkin
[EN-1570] Docstrings to dxfeedpy added
dcfa686
@iamishalkin
[EN-1570] Docstrings to Subscription added
522771b
@iamishalkin
[EN-1570] data_len parameter removed from Subscription
59ec5b2
@iamishalkin
[EN-1570] weakref tests added
522a7bf
@iamishalkin
[EN-1570] event handle tests added
05fa9cb
@iamishalkin
[EN-1570] hot event handle reattaching
070448c
@iamishalkin
[EN-1570] custom handler draft example
aa10ecc
@iamishalkin
[EN-1570] custom handler example description added
7b6744f
@iamishalkin iamishalkin requested a review from asalynskii June 19, 2020 13:08
asalynskii
asalynskii requested changes Jun 19, 2020
View reviewed changes
@iamishalkin iamishalkin requested a review from asalynskii June 19, 2020 15:57
asalynskii
asalynskii requested changes Jun 19, 2020
View reviewed changes
@iamishalkin iamishalkin requested a review from asalynskii June 19, 2020 17:05
@asalynskii asalynskii merged commit 7ebe79d into master Jun 19, 2020
@asalynskii asalynskii deleted the EN-1570-no-cython branch July 30, 2020 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@asalynskii asalynskii asalynskii approved these changes

+1 more reviewer

@10010111 10010111 10010111 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@iamishalkin @asalynskii @10010111