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

Automation of the detection system for vcredist. #90

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

Automation of the detection system for vcredist. #90

wants to merge 2 commits into from

Conversation

Mrproex
Copy link
Contributor

@Mrproex Mrproex commented Dec 23, 2019

The solution has been tested on my own project.
The execution time of the function varies between 0 and 1ms.

The solution has been tested on my own project.
The execution time of the function varies between 0 and 1ms.
@tinohager
Copy link
Contributor

Sorry but I don't see that as an alternative this is the same logic as using the registry I search for a completely different approach.

Improved detection with a view to making it more automatic
@Mrproex
Copy link
Contributor Author

Mrproex commented Dec 23, 2019

I understand, I modified the function again to match the idea of automatic detection.
I think it's more or less the best alternative to the existing code, at least until something really different comes along.
There will be no more need to add new versions.

@tinohager
Copy link
Contributor

The check on the WOW6432Node i a bad idea. The WOW6432Node is used for 32bit software. Yolo require the 64bit version of C++. https://en.wikipedia.org/wiki/WoW64

@Mrproex
Copy link
Contributor Author

Mrproex commented Dec 23, 2019

The 32b and 64b version information is contained in WOW6432Node so I think any computer in 64b has this part of the registry with the version information we're looking for.
I might be wrong but i believe this specific part of the registry "WOW6432Node" is used by 64b and 32b softwares.

@ZPiDER666
Copy link
Contributor

@Mrproex , if can you resolve the conflicts i will accept the PR

@Mrproex
Copy link
Contributor Author

Mrproex commented Oct 21, 2020

Hi, so i wasn't able to edit the previous pull request, here is the new file with my detection system in it !

DefaultYoloSystemValidator.zip

@zgabi
Copy link

zgabi commented Jan 17, 2021

@SpiRaiL The previous maintainer of this package left the company, see #143.
I've created a copy from this repository and added similar functianlity to it.
https://github.com/zgabi/Yolo.Net

Also created a nuget package: https://www.nuget.org/packages/Yolo.Net/3.0.6-alpha.1

Feel free to create PRs in my repository.

@ZPiDER666 : It seems that this repository is not maintained anymore by Alturos, this is why I created a new repository. Of course I'll try to make everything acccording the MIT license, "Alturos Destinations" is written on the LICENSE.md file. If you find any legal problem, please let me know, and I'll try to solve it. But I still hope that one time you will merge the 2 repositories. You can freely merge it any time without my approval.

@SandipBR
Copy link

Hi
Thanks a lot for all the effort

I am trying to use the 3.0.6 alpha 2 version with cuda 10.2 and cudnn 7.6.5 in vs 2019 and getting the following error.

System.BadImageFormatException
HResult=0x8007000B
Message=An attempt was made to load a program with an incorrect format. (Exception from HRESULT: 0x8007000B)
Source=Alturos.Yolo
StackTrace:
at Alturos.Yolo.YoloWrapper.InitializeYoloCpu(String configurationFilename, String weightsFilename, Int32 gpuIndex)
at Alturos.Yolo.YoloWrapper.Initialize(String configurationFilename, String weightsFilename, String namesFilename, GpuConfig gpuConfig)
at Alturos.Yolo.YoloWrapper..ctor(String configurationFilename, String weightsFilename, String namesFilename, GpuConfig gpuConfig, IYoloSystemValidator yoloSystemValidator)
at TrackNow_YOLO.Form1.detectRTSP() in C:\Users\sandi\Desktop\TrackNow_YOLO - Copy\TrackNow_YOLO\Form1.cs:line 168
at TrackNow_YOLO.Form1.Form1_Load(Object sender, EventArgs e) in C:\Users\sandi\Desktop\TrackNow_YOLO - Copy\TrackNow_YOLO\Form1.cs:line 96
at System.Windows.Forms.Form.OnLoad(EventArgs e)
at System.Windows.Forms.Form.OnCreateControl()
at System.Windows.Forms.Control.CreateControl(Boolean fIgnoreVisible)
at System.Windows.Forms.Control.CreateControl()
at System.Windows.Forms.Control.WmShowWindow(Message& m)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.Form.WmShowWindow(Message& m)
at System.Windows.Forms.NativeWindow.DebuggableCallback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
at System.Windows.Forms.UnsafeNativeMethods.SendMessage(HandleRef hWnd, Int32 msg, Int32 wParam, Int32 lParam)
at System.Windows.Forms.Form.SetVisibleCore(Boolean value)
at System.Windows.Forms.Application.ThreadContext.RunMessageLoopInner(Int32 reason, ApplicationContext context)
at System.Windows.Forms.Application.ThreadContext.RunMessageLoop(Int32 reason, ApplicationContext context)
at TrackNow_YOLO.Program.Main() in C:\Users\sandi\Desktop\TrackNow_YOLO - Copy\TrackNow_YOLO\Program.cs:line 19

I am using x64 as target platform. What can be the issue ?
I am in urgent need... please help

@zgabi
Copy link

zgabi commented Jan 13, 2022

Are you sure that all of the native dlls you use are 64bit? Badimageformat exception usually means that you are loading 32 bit dll in 64 bit appication or otherwise.

@SandipBR
Copy link

Are you sure that all of the native dlls you use are 64bit? Badimageformat exception usually means that you are loading 32 bit dll in 64 bit appication or otherwise.

Thanks a lot for the indication. I could resolve the problem. Everything works fine.
You are a life saver

🙏

@zgabi
Copy link

zgabi commented Jan 13, 2022

What did you do?

@SandipBR
Copy link

Nothing much actually... just started a new project and installed zgabi/Yolo.Net and all other necessary packages. I was having problem in using GPU but in the new project this got resolved too. Even in muti-threading it works fine. I am really thankful.

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.

None yet

5 participants