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

ATV modulator no longer available in Windows build #2248

Closed
pa3byv opened this issue Sep 8, 2024 · 8 comments
Closed

ATV modulator no longer available in Windows build #2248

pa3byv opened this issue Sep 8, 2024 · 8 comments
Assignees

Comments

@pa3byv
Copy link

pa3byv commented Sep 8, 2024

Hi, we are still using v6.16 in Windows 10 because it has an ATV modulator. It seems to have disappeared from Windows builds in V7, probably from V6.20 onwards. But I see it is still maintained and available in the Linux builds. Is there a special reason for this and if not could you please include it in the V7 Windows build again?

Many thanks,
Poll

@srcejon
Copy link
Collaborator

srcejon commented Sep 9, 2024

From the github action log, I see:

Not building modatv (ENABLE_CHANNELTX_MODATV=ON OpenCV_FOUND=0)

I think this is probably due to the version of OpenCV in sdrangelwindowslibraries not working with the latest version of Visual C. Updating to 4.10 seems to work.

@pa3byv
Copy link
Author

pa3byv commented Sep 9, 2024

Thanks for looking into it. Hopefully the ATV module will reappear in the next build then.

@srcejon
Copy link
Collaborator

srcejon commented Sep 10, 2024

@f4exb have submitted a PR to sdrangel-windows-library

We'll also need a small update in CMakeLists.txt

if(VS2019)
    set(OpenCV_DIR "${EXTERNAL_LIBRARY_FOLDER}/opencv4" CACHE INTERNAL "")

Should be VS2019 OR VS2022

@0x0V3RF10W
Copy link

Hello! Any plans to return modatv to windows builds?

@srcejon
Copy link
Collaborator

srcejon commented Sep 15, 2024

@f4exb - unfortunately need a small patch to the opencv cmakefiles: f4exb/sdrangel-windows-libraries#26

@srcejon
Copy link
Collaborator

srcejon commented Sep 15, 2024

Hello! Any plans to return modatv to windows builds?

Hopefully should be in the next release.

@srcejon
Copy link
Collaborator

srcejon commented Oct 11, 2024

Back in 7.22.1

@srcejon srcejon closed this as completed Oct 11, 2024
@pa3byv
Copy link
Author

pa3byv commented Oct 11, 2024 via email

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

3 participants