devcartel / pyrfa

Open sourced Python API for Refinitiv (Thomson Reuters) Enterprise Platform.
http://devcartel.com/pyrfa
MIT License
50 stars 15 forks source link

Pause/Resume with Views #32

Closed peervm closed 6 years ago

peervm commented 6 years ago

Hi,

Does the pause/resume feature work with views?

I used the examples/view.py code and added pause/resume. Unfortunately after resume was called then I started to receive all updates and not just my filtered view.

Thanks, Peer

wiwat-tharateeraparb commented 6 years ago

I think it’s a bug. When resuming, it doesn’t pass the set views to the server. I will fix this in the next release. ETA is about 15 days.

peervm commented 6 years ago

Awesome, thanks for the prompt reply and future fix.

Thanks, Peer

On Tue, Sep 5, 2017 at 16:27 Wiwat Tharateeraparb notifications@github.com wrote:

I think it’s a bug. When resuming, it doesn’t pass the set views to the server. I will fix this in the next release. ETA is about 15 days.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/devcartel/pyrfa/issues/32#issuecomment-327094142, or mute the thread https://github.com/notifications/unsubscribe-auth/ADBeQwwmvSccU7n1pWdT_ype-Z-yd62Eks5sfPfFgaJpZM4PMSd0 .

wiwat-tharateeraparb commented 6 years ago

Seems that pauseAll() and resumeAll() work with views but not marketPriceResume(ric)

peervm commented 6 years ago

I will try using pauseAll() and resumeAll() instead, but would prefer to do it on a ric basis if possible.

Do you know if this is a limitation on the Reuters RFA library or an issue with PyRFA? Thanks.

peervm commented 6 years ago

Confirmed that views combined with pauseAll/resumeAl behaves as expected. Thanks.

wiwat-tharateeraparb commented 6 years ago

Please try PyRFA 8.2.1 released today for which addresses this issue.

peervm commented 6 years ago

Great, thanks for the quick fix. I`ll try it out this friday. Thanks.

peervm commented 6 years ago

Hi, I tried PyRFA 8.2.1 using marketPricePause(ric) and marketPriceResume(ric) but it still loses the view settings after the resume. Other than checking the init.py version number how can I programmatically check the pyrfa.pyd version? I want to ensure I am running the correct version.

Also, I am using Python 3.6 win64. Thanks.

wiwat-tharateeraparb commented 6 years ago

You can run below to check the version:

>>> import pyrfa
>>> pyrfa.__version__

I will double check it

wiwat-tharateeraparb commented 6 years ago

I have checked again with PyRFA 8.2.1 Win64 and can confirm that view works with marketPricePause and marketPriceResume. Maybe you can try to update PyRFA with:

Python3.6.exe -m pip install -U pyrfa
wiwat-tharateeraparb commented 6 years ago

Fixed in 8.2.1