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.

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

Not working with python2 (Was: MACOS Sierra) #3

Open
ygageot opened this issue Mar 12, 2017 · 9 comments
Open

Not working with python2 (Was: MACOS Sierra) #3

ygageot opened this issue Mar 12, 2017 · 9 comments

Comments

@ygageot
Copy link

ygageot commented Mar 12, 2017

I just installed the software on MACOS Sierra version 10.12.3.
I had a first test with ./fdxread dumps/gnd10-and-displays.dump

I have a lot of messages like WARNING:root:Ignoring exception: missing tailer.

I tried also the testall program and it is the same.
A CR/LF trouble ?

Thanks for your help

@ygageot
Copy link
Author

ygageot commented Mar 12, 2017

For your information: I just install the version on my RPi2 (Raspbian 7 wheezy) and I have the same warning.

@lkarsten
Copy link
Owner

Hi.

Nice to hear from you. Where did you install this version from? If it was pypitest, that shouldn't be used at the moment.

I reran using the version in git now (0ab2027) and that works as expected for me:

./fdxread --seek 3000 dumps/onsdagsregatta-2016-08-17.dump | head
$FVMWV,216.78,R,1.94,K,A*2E
$SDDBT,,f,5.21,m,,F*10
$SDVHW,0.0,T,0.0,M,0.00,N,0.0,K*72
$FVMWV,213.50,R,0.62,K,A*29
$ZZXDR,P,101.42000,B,Barometer*21
$ZZXDR,C,21.11,C,TempDir*10
$SDDBT,,f,5.21,m,,F*10
$SDVHW,0.0,T,0.0,M,0.00,N,0.0,K*72
$FVMWV,213.50,R,0.62,K,A*29
$SDDBT,,f,5.21,m,,F*10

The gnd10-and-displays.dump file isn't the best file to test with, recommend you use onsdagsregatta-2016-08-17.dump instead.

@ygageot
Copy link
Author

ygageot commented Mar 12, 2017

I made : git clone https://github.com/lkarsten/GND10read.git to obtain the version

If i do : ./fdxread --seek 3000 dumps/onsdagsregatta-2016-08-17.dump | head like you

I have only manies : WARNING:root:Ignoring exception: missing tailer

@lkarsten
Copy link
Owner

Ah, sorry! Now I understand - it is that you're using Python 2.

Travis is detecting the same on https://travis-ci.org/lkarsten/fdxread , but I haven't gotten around to look at it yet. Something about matching inside a bytes() being different.

Can you install python 3 and try again? I'm using 3.6, but 3.5 should work fine as well.

@ygageot
Copy link
Author

ygageot commented Mar 12, 2017

Sorry, i am not familiar with python;

On MACOS I

  • installed python 3.6.0
  • reinstalled fdxread with virtualenv --python=python3.6 --system-site-packages venv

and it works

Thanks

NB : I have to do the same on my RPi ...

@ygageot
Copy link
Author

ygageot commented Mar 12, 2017

It is done on my RPi and it works. Thanks.

NB:
I plan these week to connect my Mac and/or my RPI to Nexus WSI box with an USB/RS485 cable.
I hope to obtain a connection and log the traffic.
I will try your software with serial entry.
And I will try also a small nodejs program I wrote to dump the traffic with your dump format
I will give you news !!!

@lkarsten
Copy link
Owner

Hi.

Good to hear you got it working!

I'll make an effort to clean up the problems you ran into in the coming days.

I'd love some data dumps from your WSI box. The libfdx/dumpserial.py program can be used for that. Please, if possible, also record (write down, or take a picture of) what your displays/instruments are showing when you dump it, otherwise we just have data and not information. :)

@lkarsten lkarsten changed the title MACOS Sierra Not working with python2 (Was: MACOS Sierra) Mar 12, 2017
@ygageot
Copy link
Author

ygageot commented Mar 12, 2017

I will move tomorrow and I will have access to my static boat.
I plan to take pictures of wind+depth+watertemp+battery+pression+rsi
the boat speed/VMG will be zero !
A first step !!

@lkarsten
Copy link
Owner

Re installation issue: Running on Python 2 should work fine again now with 3618b17 (incl. dumpreader.py refactor in 7cc518f) commited and pushed today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants