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

add Windows compatibility #4

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

add Windows compatibility #4

wants to merge 2 commits into from

Conversation

LeoDJ
Copy link

@LeoDJ LeoDJ commented Jul 21, 2023

Hopefully without breaking Linux support ;)

Should fix #2 too, without breaking whichever system required False instead of 0.

If you installed the Topdon Windows software already, you need to switch the driver as explained here:
LeoDJ/P2Pro-Viewer#9 (comment)

LeoDJ added 2 commits July 21, 2023 19:37
hopefully without breaking Linux support
@nemi1
Copy link

nemi1 commented Jul 21, 2023

It works! Thank you both so much for your work on this.

I just tested on a TC001 on Windows 11 with USB driver as "USB composite device".

@nemi1
Copy link

nemi1 commented Jul 21, 2023

Hi I am new to Python and learning fast to add features today to this project, also very new to GitHub.

I've modified this open contribution further to add working support to save CSV temperature data files. How may I contribute it here?

@leswright1977
Copy link
Owner

leswright1977 commented Jul 22, 2023 via email

@ikin5050
Copy link

ikin5050 commented Nov 2, 2023

Uninstalling the topdon driver so that your camera gets recognized as a USB device enables cv2 and VLC to recognize the camera, then its just a case of getting rid of the r-pi specific code at top

@villar10
Copy link

Hi I am new to Python and learning fast to add features today to this project, also very new to GitHub.

I've modified this open contribution further to add working support to save CSV temperature data files. How may I contribute it here?

Hey nemi1, I would be very interested in taking a look at your changes! Can you submit a pull request?

@mikelmujika11
Copy link

Hi,

I am trying to connect to the cam using this script but it gices me an error. lo = lo * 256 -> Python integer 256 out of bounds for uint8. How can I fix it?

@mikelmujika11
Copy link

Hi,

I am trying to connect to the cam using this script but it gices me an error. lo = lo * 256 -> Python integer 256 out of bounds for uint8. How can I fix it?

I just find the solution by adding the following line:
thdata = thdata.astype(np.int16)

However, for some reason the image appears rotated, why?

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

Successfully merging this pull request may close these issues.

6 participants