microsoft / CNTK

Microsoft Cognitive Toolkit (CNTK), an open source deep-learning toolkit
https://docs.microsoft.com/cognitive-toolkit/
Other
17.53k stars 4.28k forks source link

CNTK failed when build with permissive- + MSVC on windows. #3172

Open QuellaZhang opened 6 years ago

QuellaZhang commented 6 years ago

CNTK failed with error C3861 and C2440 when build with permissive- on Windows, I use latest source on master branch. Could you please help take a look at this? Noted that this issue only found when compiles with unreleased vctoolset, that next release of MSVC will have this behavior.

You can repro this issue as the steps below:

  1. git clone https://github.com/Microsoft/cntk D:\CNTK\src
  2. open a clean x64 prompt and browse to D:\CNTK\src
  3. set CL=/permissive- /wd4643 /wd4244 /wd4326
  4. MSBuild /m /p:Configuration=Release_CpuOnly;Platform=x64 /p:WindowsTargetPlatformVersion= 10.0.16299.0 CNTK.sln /t:Rebuild

Error info: d:\cntk\src\source\common\include\numahelpers.h(47): error C3861: 'ppl_cores': identifier not found d:\cntk\src\source\common\include\numahelpers.h(64): error C3861: 'getcurrentnode': identifier not found d:\cntk\src\source\computationnetworklib\NonlinearityNodes.h(502): error C3861: 'GradientPtrRef': identifier not found d:\cntk\src\source\computationnetworklib\NonlinearityNodes.h(553): note: see reference to class template instantiation 'Microsoft::MSR::CNTK::TopKNode' being compiled ReshapingNodes.cpp(570): error C3861: 'ValuePtrRef': identifier not found ReshapingNodes.cpp(645): error C3861: 'ValuePtrRef': identifier not found TrainingNodes.cpp(180): error C3861: 'ValuePtrRef': identifier not found DSSMReader.cpp(203): error C2440: 'initializing': cannot convert from 'std::string' to 'Microsoft::MSR::CNTK::ConfigParameters' DSSMReader.cpp(203): note: No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called

thiagocrepaldi commented 6 years ago

Have you followed this ? https://docs.microsoft.com/en-us/cognitive-toolkit/Setup-CNTK-on-Windows

PhoebeHui commented 6 years ago

Any updates?

QuellaZhang commented 6 years ago

@thiagocrepaldi Yes, we built CNTK according to the page you mentioned. Could you help look at this? This issue still exists.