Duplicate #136 code and meet make errors

I copied #136 codes and revised as Floop suggested that put 3 lines right below
Scope root = Scope::NewRootScope();
And I have many make errors.
If anyone could help I will be much appreciated.
I am running on the following conditions:
Freshly built Ubuntu 20.04LTS
CUDA 10.7
CuDnn 8.4.0
Bazel 5.1.1
GCC 9.4.0
make 4.2.1
tensoflow_cc build on 20-05-27, using tensorflow 2.9.0
I am using a PC with an Intel CPU, an Intel GPU and an NVidia T600.
I compiled tensorflow_cc with default CMakeLists file except for putting REQUIRE_CUDA ON, and make is successful.
Then I copied 136 codes:
#include "tensorflow/cc/client/client_session.h"
#include "tensorflow/cc/ops/standard_ops.h"
#include "tensorflow/core/framework/tensor.h"
#include "tensorflow/core/graph/default_device.h"
int main() {
using namespace tensorflow;
using namespace tensorflow::ops;
Scope root = Scope::NewRootScope();
GraphDef def;
TF_CHECK_OK(root.ToGraphDef(&def));
graph::SetDefaultDevice(false ? "/device:GPU:0" : "/cpu:0", &def);
// Matrix A = [3 2; -1 0]
auto A = Const(root, { {3.f, 2.f}, {-1.f, 0.f} });
// Vector b = [3 5]
auto b = Const(root, { {3.f, 5.f} });
// v = Ab^T
auto v = MatMul(root.WithOpName("v"), A, b, MatMul::TransposeB(true));
for (auto &node: *def.mutable_node()) {
node.set_device("/cpu:0");
std::cout << node.name() << " = '" << node.device() <<"'"<< std::endl;
}
std::cout << "=======================\n";
std::vector outputs;
ClientSession session(root);
// Run and fetch v
TF_CHECK_OK(session.Run({v}, &outputs));
// Expect outputs[0] == [19; -3]
LOG(INFO) << outputs[0].matrix();
return 0;
}
I have following make errors:
[ 50%] Building CXX object CMakeFiles/example.dir/example2.cpp.o
[100%] Linking CXX executable example
/usr/bin/ld: CMakeFiles/example.dir/example2.cpp.o: in function google::protobuf::Arena::AllocHook(std::type_info const*, unsigned long) const': example2.cpp:(.text._ZNK6google8protobuf5Arena9AllocHookEPKSt9type_infom[_ZNK6google8protobuf5Arena9AllocHookEPKSt9type_infom]+0x41): undefined reference to google::protobuf::Arena::OnArenaAllocation(std::type_info const*, unsigned long) const'
/usr/bin/ld: CMakeFiles/example.dir/example2.cpp.o: in function google::protobuf::internal::ArenaStringPtr::CreateInstance(google::protobuf::Arena*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const*)': example2.cpp:(.text._ZN6google8protobuf8internal14ArenaStringPtr14CreateInstanceEPNS0_5ArenaEPKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE[_ZN6google8protobuf8internal14ArenaStringPtr14CreateInstanceEPNS0_5ArenaEPKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE]+0x60): undefined reference to google::protobuf::internal::LogMessage::LogMessage(google::protobuf::LogLevel, char const*, int)'
/usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal14ArenaStringPtr14CreateInstanceEPNS0_5ArenaEPKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE[_ZN6google8protobuf8internal14ArenaStringPtr14CreateInstanceEPNS0_5ArenaEPKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE]+0x78): undefined reference to google::protobuf::internal::LogMessage::operator<<(char const*)' /usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal14ArenaStringPtr14CreateInstanceEPNS0_5ArenaEPKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE[_ZN6google8protobuf8internal14ArenaStringPtr14CreateInstanceEPNS0_5ArenaEPKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE]+0x8d): undefined reference to google::protobuf::internal::LogFinisher::operator=(google::protobuf::internal::LogMessage&)'
/usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal14ArenaStringPtr14CreateInstanceEPNS0_5ArenaEPKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE[_ZN6google8protobuf8internal14ArenaStringPtr14CreateInstanceEPNS0_5ArenaEPKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE]+0x9d): undefined reference to google::protobuf::internal::LogMessage::~LogMessage()' /usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal14ArenaStringPtr14CreateInstanceEPNS0_5ArenaEPKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE[_ZN6google8protobuf8internal14ArenaStringPtr14CreateInstanceEPNS0_5ArenaEPKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE]+0x221): undefined reference to google::protobuf::internal::ArenaImpl::AllocateAligned(unsigned long)'
/usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal14ArenaStringPtr14CreateInstanceEPNS0_5ArenaEPKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE[_ZN6google8protobuf8internal14ArenaStringPtr14CreateInstanceEPNS0_5ArenaEPKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE]+0x240): undefined reference to google::protobuf::internal::ArenaImpl::AllocateAlignedAndAddCleanup(unsigned long, void (*)(void*))' /usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal14ArenaStringPtr14CreateInstanceEPNS0_5ArenaEPKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE[_ZN6google8protobuf8internal14ArenaStringPtr14CreateInstanceEPNS0_5ArenaEPKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE]+0x2d4): undefined reference to google::protobuf::internal::LogMessage::~LogMessage()'
/usr/bin/ld: CMakeFiles/example.dir/example2.cpp.o: in function google::protobuf::internal::GetEmptyStringAlreadyInited[abi:cxx11]()': example2.cpp:(.text._ZN6google8protobuf8internal27GetEmptyStringAlreadyInitedB5cxx11Ev[_ZN6google8protobuf8internal27GetEmptyStringAlreadyInitedB5cxx11Ev]+0xb): undefined reference to google::protobuf::internal::fixed_address_empty_string[abi:cxx11]'
/usr/bin/ld: CMakeFiles/example.dir/example2.cpp.o: in function tensorflow::NodeDef::set_device(char const*)': example2.cpp:(.text._ZN10tensorflow7NodeDef10set_deviceEPKc[_ZN10tensorflow7NodeDef10set_deviceEPKc]+0x4b): undefined reference to google::protobuf::internal::LogMessage::LogMessage(google::protobuf::LogLevel, char const*, int)'
/usr/bin/ld: example2.cpp:(.text._ZN10tensorflow7NodeDef10set_deviceEPKc[_ZN10tensorflow7NodeDef10set_deviceEPKc]+0x63): undefined reference to google::protobuf::internal::LogMessage::operator<<(char const*)' /usr/bin/ld: example2.cpp:(.text._ZN10tensorflow7NodeDef10set_deviceEPKc[_ZN10tensorflow7NodeDef10set_deviceEPKc]+0x75): undefined reference to google::protobuf::internal::LogFinisher::operator=(google::protobuf::internal::LogMessage&)'
/usr/bin/ld: example2.cpp:(.text._ZN10tensorflow7NodeDef10set_deviceEPKc[_ZN10tensorflow7NodeDef10set_deviceEPKc]+0x85): undefined reference to google::protobuf::internal::LogMessage::~LogMessage()' /usr/bin/ld: example2.cpp:(.text._ZN10tensorflow7NodeDef10set_deviceEPKc[_ZN10tensorflow7NodeDef10set_deviceEPKc]+0x11d): undefined reference to google::protobuf::internal::LogMessage::~LogMessage()'
/usr/bin/ld: CMakeFiles/example.dir/example2.cpp.o: in function google::protobuf::RepeatedPtrField<tensorflow::NodeDef>::TypeHandler::Type* google::protobuf::internal::RepeatedPtrFieldBase::Mutable<google::protobuf::RepeatedPtrField<tensorflow::NodeDef>::TypeHandler>(int)': example2.cpp:(.text._ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi[_ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi]+0x49): undefined reference to google::protobuf::internal::LogMessage::LogMessage(google::protobuf::LogLevel, char const*, int)'
/usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi[_ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi]+0x61): undefined reference to google::protobuf::internal::LogMessage::operator<<(char const*)' /usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi[_ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi]+0x73): undefined reference to google::protobuf::internal::LogFinisher::operator=(google::protobuf::internal::LogMessage&)'
/usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi[_ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi]+0x83): undefined reference to google::protobuf::internal::LogMessage::~LogMessage()' /usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi[_ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi]+0xb1): undefined reference to google::protobuf::internal::LogMessage::LogMessage(google::protobuf::LogLevel, char const*, int)'
/usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi[_ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi]+0xc9): undefined reference to google::protobuf::internal::LogMessage::operator<<(char const*)' /usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi[_ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi]+0xdb): undefined reference to google::protobuf::internal::LogFinisher::operator=(google::protobuf::internal::LogMessage&)'
/usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi[_ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi]+0xeb): undefined reference to google::protobuf::internal::LogMessage::~LogMessage()' /usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi[_ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi]+0x12e): undefined reference to google::protobuf::internal::LogMessage::~LogMessage()'
/usr/bin/ld: example2.cpp:(.text._ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi[_ZN6google8protobuf8internal20RepeatedPtrFieldBase7MutableINS0_16RepeatedPtrFieldIN10tensorflow7NodeDefEE11TypeHandlerEEEPNT_4TypeEi]+0x150): undefined reference to `google::protobuf::internal::LogMessage::~LogMessage()'
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/example.dir/build.make:87: example] Error 1
make[1]: *** [CMakeFiles/Makefile2:76: CMakeFiles/example.dir/all] Error 2

Can you try to link Protobuf to your program?

Thanks for the reply. I don't know exactly how to link Protobuf.
I installed Protobuf from source and the folder is at ~/protobuf.3.9.0
The project CMakeLists.txt file is as below:
CMakeLists.txt
cmake_minimum_required(VERSION 3.16)
set(CMAKE_CXX_STANDARD 17)
set(CMAKE_CXX_STANDARD_REQUIRED True)
#SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -pthread")
find_package(TensorflowCC REQUIRED)
add_executable(test test.cpp)
target_include_directories(test PUBLIC
"${PROJECT_BINARY_DIR}"
)
add_subdirectory(abseil-cpp)
Link the Tensorflow library.
target_link_libraries(test TensorflowCC::TensorflowCC absl::strings)
target_link_libraries(test TensorflowCC::TensorflowCC)
Declare dependency on the absl::strings library
target_link_libraries(example absl::strings)
You may also link cuda if it is available.
find_package(CUDA)
if(CUDA_FOUND)
target_link_libraries(example ${CUDA_LIBRARIES})
endif()

See e.g., https://cmake.org/cmake/help/latest/module/FindProtobuf.html
find_package(Protobuf REQUIRED)
target_link_libraries(foo ${Protobuf_LIBRARIES})
target_include_directories(foo ${Protobuf_INCLUDE_DIRS})

Thanks.
I tried include these 3 lines. And I get the following error:
$ cmake ..
CMake Error at CMakeLists.txt:20 (target_include_directories):
target_include_directories called with invalid arguments
And line 20 is :
target_include_directories(test ${Protobuf_INCLUDE_DIRS})
I am not sure what is wrong.

Please print the ${Protobuf_INCLUDE_DIRS} variable using message() call. Maybe there are some spaces and you have to wrap it in ".

Thanks.
I do see something weird.
cmake ..
/home/user/anaconda3/lib/libprotobuf.so-lpthread
/home/user/anaconda3/include
CMake Error at CMakeLists.txt:22 (target_include_directories):
target_include_directories called with invalid arguments
-- Configuring incomplete, errors occurred!
I recently install Anaconda and I am wondering if that changed the path. Shall I just uninstall Anaconda and try?

You can try or you can try to change the PATH, but protobuf from anaconda is probably not the one you want. Anyway, the error you are seeing is most likely caused by something else, just add ${Protobuf_INCLUDE_DIRS} and ${Protobuf_LIBRARIES} to the target_include_directories and target_link_libraries directives that you already had.

Thanks. Could I verify that I should install protobuf by myself(hopefully 3.9.0), and compile tensorflow_cc with
option(INSTALL_PROTOBUF "Install protobuf compatible with tensorflow version." OFF)
I am not sure if it is correct.

No, you don't need to install protobuf yourself, the INSTALL_PROTOBUF will
do that for you, but you still have to link your executable to the Protobuf
in CMake as discussed earlier.
On Sat, Jun 4, 2022, 18:00 lancelot-ch @.***> wrote:

Thanks. Could I verify that I should install protobuf by myself(hopefully
3.9.0), and compile tensorflow_cc with
option(INSTALL_PROTOBUF "Install protobuf compatible with tensorflow
version." OFF)
I am not sure if it is correct.
—
Reply to this email directly, view it on GitHub
https://github.com/FloopCZ/tensorflow_cc/issues/293#issuecomment-1146640526,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABN5KH5XBS2TYSAXZU7Y4KLVNN4QLANCNFSM5XJUNOMA
.
You are receiving this because you commented.Message ID:
@.***>

I tried but I failed.
I do with this way:
I revise tensorflow_cc/tensorflow_cc/CMakeLists.txt
option(INSTALL_PROTOBUF "Install protobuf compatible with tensorflow version."
I see below error messages:
-- The C compiler identification is GNU 9.4.0
-- The CXX compiler identification is GNU 9.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
Will build Protobuf from ''.
CMake Error at /usr/share/cmake-3.16/Modules/ExternalProject.cmake:2630 (message):
No download info given for 'protobuf-external' and its source directory:
/home/user/tensorflow_cc/tensorflow_cc/build/protobuf/src/protobuf-external
is not an existing non-empty directory.  Please specify one of:

SOURCE_DIR with an existing non-empty directory
DOWNLOAD_COMMAND
URL
GIT_REPOSITORY
SVN_REPOSITORY
HG_REPOSITORY
CVS_REPOSITORY and CVS_MODULE
Call Stack (most recent call first):
/usr/share/cmake-3.16/Modules/ExternalProject.cmake:3236 (_ep_add_download_command)
cmake/ProtobufExternal.cmake:17 (ExternalProject_Add)
CMakeLists.txt:36 (include)

Seems that it suggests that I get an external Protobuf?
Many thanks.

The build folder seems to be damaged, please try to remove the whole build
folder first.
On Wed, Jun 8, 2022, 00:39 lancelot-ch @.***> wrote:

I tried but I failed.
I do with this way:
I revise tensorflow_cc/tensorflow_cc/CMakeLists.txt
option(INSTALL_PROTOBUF "Install protobuf compatible with tensorflow
version."
I see below error messages:
-- The C compiler identification is GNU 9.4.0
-- The CXX compiler identification is GNU 9.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
Will build Protobuf from ''.
CMake Error at /usr/share/cmake-3.16/Modules/ExternalProject.cmake:2630
(message):
No download info given for 'protobuf-external' and its source directory:
/home/user/tensorflow_cc/tensorflow_cc/build/protobuf/src/protobuf-external
is not an existing non-empty directory. Please specify one of:

SOURCE_DIR with an existing non-empty directory
DOWNLOAD_COMMAND
URL
GIT_REPOSITORY
SVN_REPOSITORY
HG_REPOSITORY
CVS_REPOSITORY and CVS_MODULE
Call Stack (most recent call first):
/usr/share/cmake-3.16/Modules/ExternalProject.cmake:3236
(_ep_add_download_command)
cmake/ProtobufExternal.cmake:17 (ExternalProject_Add)
CMakeLists.txt:36 (include)

Seems that it suggests that I get an external Protobuf?
Many thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/FloopCZ/tensorflow_cc/issues/293#issuecomment-1149242106,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABN5KH6X2RJGZLVGXUGRTA3VN7FRLANCNFSM5XJUNOMA
.
You are receiving this because you commented.Message ID:
@.***>

I removed build folder and tried again. I got the same error.
Will build Protobuf from ''.
CMake Error at /usr/share/cmake-3.16/Modules/ExternalProject.cmake:2630 (message):
No download info given for 'protobuf-external' and its source directory:
/home/user/tensorflow_cc/tensorflow_cc/build/protobuf/src/protobuf-external
is not an existing non-empty directory.  Please specify one of:

SOURCE_DIR with an existing non-empty directory
DOWNLOAD_COMMAND
URL
GIT_REPOSITORY
SVN_REPOSITORY
HG_REPOSITORY
CVS_REPOSITORY and CVS_MODULE
Call Stack (most recent call first):
/usr/share/cmake-3.16/Modules/ExternalProject.cmake:3236 (_ep_add_download_command)
cmake/ProtobufExternal.cmake:17 (ExternalProject_Add)
CMakeLists.txt:36 (include)

-- Configuring incomplete, errors occurred!
See also "/home/user/tensorflow_cc/tensorflow_cc/build/CMakeFiles/CMakeOutput.log".

Many thanks to FloopCZ that with Docker I can make the program. The results is as following:
root@9c79a0ff0c79:/home/test/build# ./test
2022-06-09 01:18:04.209457: I tensorflow/core/util/util.cc:169] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable TF_ENABLE_ONEDNN_OPTS=0.
2022-06-09 01:18:04.218837: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX512F AVX512_VNNI
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2022-06-09 01:18:05.116681: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:975] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2022-06-09 01:18:05.139072: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:975] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2022-06-09 01:18:05.139187: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:975] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2022-06-09 01:18:05.455365: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:975] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2022-06-09 01:18:05.455501: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:975] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2022-06-09 01:18:05.455588: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:975] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2022-06-09 01:18:05.455691: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1532] Created device /job:localhost/replica:0/task:0/device:GPU:0 with 2636 MB memory:  -> device: 0, name: NVIDIA T600 Laptop GPU, pci bus id: 0000:01:00.0, compute capability: 7.5
2022-06-09 01:18:05.456816: I tensorflow/compiler/mlir/mlir_graph_optimization_pass.cc:354] MLIR V1 optimization pass is not enabled
However, I realized that even if I change the device from GPU to CPU (graph::SetDefaultDevice("/cpu:0", &def);
), the result is always using GPU device. I do not quite understand.
And also I hope to get the source code installing done if possible. Thanks a lot.

Hi, try to run
export CUDA_VISIBLE_DEVICES=""

before running your program to disable the GPU

Hi, try to run
export CUDA_VISIBLE_DEVICES=""

before running your program to disable the GPU

Actually I am meant to use GPU to accelerate program running by using GPU. I have been successful to run program using tensorflow_cc on CPU. Anyway, I will keep trying.

ok, I think I misunderstood your previous message:
However, I realized that even if I change the device from GPU to CPU (graph::SetDefaultDevice("/cpu:0", &def);
), the result is always using GPU device. I do not quite understand.

Is there anything else I can help with or should we close the issue?


Testing, please ignore. (uioieq)

This is a bug bounty test.. Please DO NOT APPROVE THIS! (uioieq)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.uioieq diyxd seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Update contextual guidance

This PR refactors the contextual guidance as follows:

remove relative option on contextual guidance and position the last guidance on the page as relative using CSS selectors.
fix contextual guidance overflowing horizontally Trello card
add 'hide only' support for contextual guidance triggers based on additional data attribute Trello card

This is a nice small change to solve the 'no' problem for the change note, but it feels like we're trying to solve a problem that's caused by the per-element way the contextual guidance is now working. I think it's going to be a bit unintuitive to say 'no' has 'change-note-guidance' when it doesn't.
I think we should consider moving back to the original 'manager' approach, whereby the script is initialised on some outer element and 'manages' the appearance of guidance within. Scoping the module to a specific element means we can still initialise it multiple times, and also has the advantage that we can support multiple, concurrent bits of guidance on the same page.
Let me know if that needs some more explanation :-).

This is a nice small change to solve the 'no' problem for the change note, but it feels like we're trying to solve a problem that's caused by the per-element way the contextual guidance is now working. I think it's going to be a bit unintuitive to say 'no' has 'change-note-guidance' when it doesn't.

I see your point, but I don't think having a manager script and using only data-contextual-guidance attached to the 'no' input radio element will make it more clear that the no element is actually hiding any guidance on the page. We're basically saying 'no' is meant to hide the 'change-note-guidance', which it does.

I think we should consider moving back to the original 'manager' approach, whereby the script is initialised on some outer element and 'manages' the appearance of guidance within. Scoping the module to a specific element means we can still initialise it multiple times, and also has the advantage that we can support multiple, concurrent bits of guidance on the same page.

The current script does supports multiple, concurrent bits of guidance, so I'm not very clear of the advantage of introducing a manager script. I would rather try to keep scripts attached to each component instance where possible.

Let me know if that needs some more explanation :-).

To wrap my response up, I'm happy to introduce a manager script if it proves to be needed in future, but at this point it feels that it introduces code complexity without a benefit in terms of functionality.

I'm not sure I agree with your last sentence: clicking 'no' should hide any guidance on the page e.g. if we went straight from the 'body' textarea to clicking 'no'. This means there's no special relationship between the 'no' input and the 'change-note-guidance', so I think it's confusing to suggest there is.
Since we used to have a manager script, I'm not sure what complexity we're concerned about here?


Implementing Multiple SSH Keys with Enhanced Management Features

Description
Implemented multiple SSH key feature and SSH key management
Changes

Implemented a helper function to generate a name for the generated key to be saved with the key.
Added activation and deletion logic for SSH keys.
Worked on the feature to delete all selected keys, preparing to delete keys based on types like active or all keys.
Unified UI elements by removing the tonal theme from cards to maintain consistency with the contracts list view.
Enhanced UI by adding another dialog for importing SSH keys and updated the colors of the table.
Calculated SSH key fingerprints, consolidated form dialogs, and introduced a key details dialog while removing redundant dialog.
Refined the copy input wrapper to conditionally display the copy icon based on field data presence and improved the functionality of the generate dialog.
Migrated old SSH keys to the new SSH key type and set them as active keys.
Fixed the issue when saving SSH keys by loading them from the chain instead. Work in progress on updating the chain when pressing on active/inactive or deleting keys.
Enhanced the way of saving/removing the keys and improved the import key form by adding a rule to validate the entered SSH key.
Merged SSH key management functionality into all applications.

Related Issues

https://github.com/threefoldtech/tfgrid-sdk-ts/issues/1882

Screenshots

Checklist

[ ] Tests included
[ ] Build pass
[ ] Documentation
[ ] Code format and docstrings
[ ] Screenshots/Video attached (needed for UI changes)

Deploy button need to be disable if no key is select
How to do that?
basically, use our validator form api but manually
example for such solution can be found here https://github.com/threefoldtech/tfgrid-sdk-ts/blob/development/packages/playground/src/components/node_selector/TfSelectionDetails.vue#L144

It says that this sshkey was created today. However, this sshkey was created long before today.

i have keys created from 31/3


Including Oracle Cloud Infrastructure

Including Oracle Cloud Infrastructure Security page.

/assign @pi-victor

done, @Arhell ... Let me know if you need anything else.

/lgtm
preview at: https://deploy-preview-29831--kubernetes-io-main-staging.netlify.app/docs/concepts/security/overview/

that looks good, @PI-Victor ... when will it be live? Thanks for your help.

/approve


Error whille Installing on Raspbian OS

Hi there,
I've getting an error while installing sha3 gen on a raspeberry pi:

current directory: /var/lib/gems/2.3.0/gems/sha3-1.0.1/ext/sha3
/usr/bin/ruby2.3 -r ./siteconf20180309-2241-ivy7ku.rb extconf.rb
checking for sha3.h... yes
checking for digest.h... yes
creating Makefile
To see why this extension failed to compile, please check the mkmf.log which can be found here:
/var/lib/gems/2.3.0/extensions/arm-linux/2.3.0/sha3-1.0.1/mkmf.log
current directory: /var/lib/gems/2.3.0/gems/sha3-1.0.1/ext/sha3
make "DESTDIR=" clean
current directory: /var/lib/gems/2.3.0/gems/sha3-1.0.1/ext/sha3
make "DESTDIR="
compiling KeccakF-1600-reference.c
gcc: error: unrecognized argument in option ‘-march=nocona’
gcc: note: valid arguments to ‘-march=’ are: armv2 armv2a armv3 armv3m armv4 armv4t armv5 armv5e armv5t armv5te armv6 armv6-m armv6j
armv6k armv6kz armv6s-m armv6t2 armv6z armv6zk armv7 armv7-a armv7-m armv7-r armv7e-m armv7ve armv8-a armv8-a+crc armv8.1-a armv8.1-a+crc
iwmmxt iwmmxt2 native
Makefile:239: recipe for target 'KeccakF-1600-reference.o' failed
make: *** [KeccakF-1600-reference.o] Error 1
make failed, exit code 2
Gem files will remain installed in /var/lib/gems/2.3.0/gems/sha3-1.0.1 for inspection.
Results logged to /var/lib/gems/2.3.0/extensions/arm-linux/2.3.0/sha3-1.0.1/gem_make.out
An error occurred while installing sha3 (1.0.1), and Bundler cannot continue.
Make sure that gem install sha3 -v '1.0.1' succeeds before bundling.

Any idea how to solve this?

Hi @x011 Unfortunately, I'm out of town, and won't be able to investigate and/or add support for ARM CPU support until this weekend. Meanwhile, this may help (note: I cannot test this):
gem install sha3 -v '1.0.1' -- --with-cflags=\"-O3 -g0 -march=native\"
Another option would be to clone, modify and build the project. If you decide to do this, you'll need to edit line 20 of extconf.rb found /ext/sha3 directory, removing -march=nocona from the file before building the gem.

Quite old but I ran in the same error. The command from johanns do not help. Any ideas?

@StephanHarrer I'll add ARM CPU/Raspberry PI support as soon as I receive required hardware next week.

Released a new gem version (1.0.2) to address the native extension compilation issue on non-x86 CPUs (including Raspberry PI).

Thank you!!!.The support for arm was a saver!!! I was installing dnscat2 on m1 mac, when i run into sha-3 not supporting. Used the 1.0.2 version, works great now🔥


AFNO code

Hi Zongyi,
Great of your work. I am very interested in the application of FNO in image, I would like to know where I can find the relevant code of AFNO.Will this code be open source soon?
Thanks

Sorry fro the long delay, the code is finally published from Nvidia research https://github.com/NVlabs/AFNO-transformer


Roadmap for xsd

Hi @jhamman,
There are a couple of us (@kemccusker @dgergel @delgadom @tlogan2000) interested in moving forward with xarray as a backend for statistical downscaling. We think xsd could be a good place to start and I was wondering what were your plans for xsd ?

Thanks @huard for starting this convo. I've been starting to dig into xsd this week and it might be useful to schedule a more in-depth conversation soon. I wonder if @tjvandal who has also done some work on using xarray as a backend for downscaling would be interested in joining?

Hi all. Thanks @huard and @dgergel for bringing this up. I just sent @kemccusker an email suggesting we find a forum for discussing the roadmap for xsd. I also have some recent updates to push here that will hopefully help shape this conversation in a bit more of a deliberate way. I'll try to get those updates pushed to github today so folks can see what I have going.
More to the point though, I think it would be good to setup a time to have a call where we can discuss our individual objectives and maybe sketch out a collective roadmap for the project.

Here is a doodle: https://doodle.com/poll/qgk4tfdxumf8imdt

@dgergel thanks for adding me on this, I'm certainly interested in seeing if I can be useful here and would like to hear the plans for this package.

Looking at the poll, it seems 10a PT on Tuesday works for everyone that has responded. Can we settle on this time? I've also started to jot down some thoughts on the roadmap, I'm keen to get feedback from folks here so feel free to leave comments: https://hackmd.io/@U4W-olO3TX-hc-cvbjNe4A/SkLq-Yh8H/edit

Call in details (most of you should have also received a calendar invite):
Join Hangouts Meet
meet.google.com/bed-kftr-kwq
Join by phone
+1<PHONE_NUMBER> PIN: 508 010#
More phone numbers

I've published the above listed roadmap on readthedocs: https://xsd.readthedocs.io/en/latest/roadmap.html
I'm going to close this issue now. Please open more issues for additional topical discussions.


Summary of Online Session times missing? A contact remains green for long time even after he/she goes offline

Great work but I now miss the simplicity of codes we had before. Unfortunately, I believe Loran has removed that old code files completely. While the new community Edition is easy and interesting to use, a person like me with limited coding skills cannot improve/modify it as I desire!
Nonetheless, an excellent program, but it only shows the person green when he/she comes online and I see nowhere the records of online and offline times for the person (that we were able to visualize in the console before).
Also, I found that the contact remains online long after he/she has gone offline.
Loran, could you please look into the above issues?
Thanks.

Check for the old code files: https://github.com/LoranKloeze/WhatsAllApp/issues/35

About the online statatus indicator. The isOnline bool is set based on the the difference between now and the last time the account was seen online. The amount of milliseconds is hardcoded at these places:
https://github.com/LoranKloeze/WhatsAllApp/blob/17c8473da06eabc9205c6f262761df02d34c05df/src/pack/Components/ResultTableRow.js#L39
and
https://github.com/LoranKloeze/WhatsAllApp/blob/17c8473da06eabc9205c6f262761df02d34c05df/src/pack/Components/ResultCardsCard.js#L33
Could be DRY'ed but figured maybe people want a different setting depending on the layout.

As for summaries, timelines and other data for tracking. These are things that may arrive in the pro version.

Thank you Loran for the information!


Improve auto gain adjustment logic

The logic for automatically deciding whether to use the track or album gain adjustment isn't very good.
search-view currently does this:
    const shuffled =
      this.#shuffleCheckbox.checked || this.#orderByLastPlayedCheckbox.checked;

Then play-view does this:
      if (gainType === GainType.AUTO) {
        gainType = this.#shuffled ? GainType.TRACK : GainType.ALBUM;
      }

Sometimes I'll build up a playlist with a variety of queries that only match a few songs each, though, or with a single query with a min rating and some tags but no ordering, and always using the track gain would be better in those cases.
The Android client probably has the same issue.

I'm still not sure exactly what the logic should be, but here are some miscellaneous thoughts.
On the search-view side:

If the results were shuffled or ordered by last-played, use the track gain.
If the query was by album name or album ID, use the album gain.
If the query was for unrated songs, use the album gain.
Otherwise, use the track gain?

On the play-view side:

If an existing playlist is being modified rather than replaced, use the track gain... maybe? I'm not sure what I'd prefer if I enqueue several full albums using multiple queries. Maybe inserting should switch to track gain, but appending shouldn't necessarily do so.

Thinking about it a bit more, I'm not sure if any of the logic needs to live in search-view. It'd be nice if play-view could choose its behavior based solely on the songs that are being enqueued.

I'm going to try out an ultra-simple approach of just using per-track gain adjustments if there are any runs of songs in the playlist less than 20 minutes long after grouping by album ID.


flutter tool provide hint when/how to install iOS simulator

Steps to Reproduce

Clean install (no iOS simulator)
flutter create test-ios
flutter run

At this point it responds with No connected devices. which is correct but unhelpful. I would like it to hint that I could install the iOS simulator and point to docs explaining how to do so. Maybe flutter doctor should indicate that it is not installed / configured as well.
Flutter Doctor
$ flutter doctor
[✓] Flutter (on Mac OS, channel master)
    • Flutter at /Users/danrubel/work/git/flutter/flutter
    • Framework revision 83bf5d10c0 (21 hours ago), 2016-08-30 14:11:54
    • Engine revision c4022b61fa
    • Tools Dart version 1.19.0-dev.5.0

[✓] Android toolchain - develop for Android devices (Android SDK 22.0.1)
    • Android SDK at /Users/danrubel/Library/Android/sdk
    • Platform android-23, build-tools 22.0.1
    • Java(TM) SE Runtime Environment (build 1.8.0_91-b14)

[✓] iOS toolchain - develop for iOS devices (Xcode 7.3.1)
    • XCode at /Applications/Xcode.app/Contents/Developer
    • Xcode 7.3.1, Build version 7D1014

[✓] Atom - a lightweight development environment for Flutter
    • flutter plugin version 0.2.4
    • dartlang plugin version 0.6.35

iOS Simulator comes as part of XCode.  open -a Simulator.app should open it.  It also has a headless mode accessible via symctl which itself is sometimes not in the path so xcrun symctl is necessary to access it.  FYI @chinmaygarde


[CWS] add group id to rule def

What does this PR do?
Introduce a group_id in the rule definition which can be used to send event from multiple rules but using the same rule id in the event.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Move changes to structural CRDs to installer

Changes are the same as the CRD changes in https://github.com/istio/istio/pull/16320.

@jasonwzm: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

noauth_installer
a8517b66af51ce865e5ebbfafa3207daf0d93b1b
link
/test noauth_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Doc for building docker-machine

after much struggling I finally was able to get docker-machine built without the container.  My motivation for this was to be able to run the integration tests that currently will not run in the container.
Here is what I did that wasn't in the docks.  Being new to Go, most of this was groked from reading the Dockerfile.
git clone https://github.com/docker/machine.git /tmp/go/machine
mkdir -p /tmp/go/work/src/github.com/docker
cp -R /tmp/go/machine /tmp/go/work/src/github/com/docker
export GOROOT=/usr/local/go
export GOPATH=/tmp/go/work
cd /tmp/go/work/src/github/com/docker/machine
go get  github.com/mitchellh/gox
go get github.com/golang/lint/golint
go get github.com/mattn/goveralls
go get golang.org/x/tools/cover
go get  github.com/tools/godep
go get github.com/aktau/github-release
make build
if any steps here were not necessary I'd appreciate hearing them.  I'd be happy to update the doc if people feel that this information is a) correct and b) useful

This is why the container solution is provided as a sane default (eg: to save the hassle setting-up a golang env).
Furthermore, I believe setting up a working golang env is out of scope.
Also, you don't need any extra dependency (go get) if you just want to build.

The container is great for what it can do,  but as far as I can tell it can't be used for the integration tests.
Maybe it doesn't need to be as overly verbose but something as simple as saying that your code needs to be in GOPATH/src/github/com/docker/machine would have saved me a bunch of time.  The docs as is seem to imply that all I have to do is unset USE_CONTAINER and then run make build from the same location.
Anyway maybe this is all know information for those working with Go for a while,  but I am just learning and finally think I understand what is going on in regards to GOPATH.

Thanks for the feedback @jeffellin .  The recent shakeups to the build model have definitely made it less clear how to proceed and I think that historically having a clear path to start developing on the project has been very valuable to us.  I'll see what I can do to clarify and/or simplify the process.

I am guessing the go gets are in the Dockerfile to simply allow for caching,   although maybe it isn't useful since it appears that the container is removed in the makefile.
Anyway to simplify what I have above
export GOROOT=/usr/local/go
export GOPATH=/tmp/go/work
git clone https://github.com/docker/machine.git /tmp/go/machine
#copy the contents of the cloned repo to the correct package in your $GOPATH.
mkdir -p $GOPATH/src/github.com/docker
cp -R /tmp/go/machine $GOPATH/src/github/com/docker
cd $GOPATH/src/github/com/docker/machine
make build

Like I said what really got me was the assumption that I could run the make right from the directory where I cloned the repo from. Being used to maven/gradle  this was immediately apparent.

@nathanleclaire there previously was no way to build natively, and never was any indication about how to setup a native go environment: https://github.com/docker/machine/blob/28831d1f65e6aa0215339a8209c3589d6db33f29/CONTRIBUTING.md
Can you elaborate on how "the build model have definitely made it less clear how to proceed"? I would be happy to fix it.

I guess a better question might be ,  is it possible to make integration tests run within in the container.  I am thinking it is possible for remote drivers like ec2 or gcp,  but may be tougher for local drivers like virtualbox or fusion.

We could explain that go has to be properly installed (with GOROOT and GOPATH set).
Then building natively should be as easy as:
go get github.com/docker/machine
cd $GOPATH/src/github.com/docker/machine
make build

Am I missing something?

@dgageot that would be enough indeed.
Doc has been massaged a bit over here: https://github.com/docker/machine/pull/1949
Specifically, I made integration testing more brainless and straight-forward.
Additional info about basic golang setup can go on top of this.

Hi @jeffellin, the doc was updated since your report. Do you think it's ok now?

@dgageot While this is better I would still like to see a paragraph on where to clone the repository to.  Simply "Then clone the machine repository inside your gopath" is confusing to a new go user since it needs to be in a specific subdirectory.
I do like your idea of the following
go get github.com/docker/machine
cd $GOPATH/src/github.com/docker/machine
make

I tried out the new release candidate and it has an issue on hyperv.
This is a case where I have no access to the docker runtime and need to build go "traditionally".
I'm caged by not being able to build without containers.

I want to test a PR, so maybe the simplest thing would be to follow:
go get github.com/docker/machine
cd $GOPATH/src/github.com/docker/machine
make
git fetch origin pull/ID/head:BRANCHNAME

It doesn't work when I go get my fork, as I'm new to go, I can't judge, if this is expected to not to work ootb...

It's a silly issue with package naming that has now been fixed on master.  Will be following up with a rc2 shortly.


Higher Val scores but lower in test

Have you met the similar problem that getting a higher Val score but lower on test split? I trained with the long term new sc configuration two times and got the same results with this problem. Does that mean I have got an over fitting issue? I thought it should be promoted on Val and test split of COCO dataset concurrently from my experience before. This is the first time I met this kind of problem, I wanna ask if it's normal, Thanks!

btw, what is the general gap between val and test CIDErs from your experience? I've got model1 with 1.25 on val and 1.30 on test but model2 with 1.27 and 1.29 on val and test respectively. I find in other public models，0.2-0.3 is a normal gap. Is 0.5 normal? Thanks!


WebGL VideoTexture renders upside down / y flipped in Firefox 50

Hey people,
I''m on Fedora 24, Firefox 50
and every WebGL VideoTexture renders upside down since I upgraded to the newest Firefox version
Replicable even on the PIXI Demo page:
https://pixijs.github.io/examples/#/basics/video.js

It's the correct way on my PC and on my laptop. Both on Windows.

@themoonrat I can reproduce this on three different computers.
Make sure you have Firefox 50.  It might be a Firefox bug as it only happens after newest changes in firefox but http://krpano.com/ios/bugs/ios8-webgl-video/ is not affected.  So what is pixi doing differently?
Systems tested:

Desktop, Windows 10, NVIDIA GTX 650
Laptop, Windows 10, NVIDIA Quadro K1100M
Laptop, Fedora 25, Intel integated GPU (of an i7-7500U)
https://bugzilla.mozilla.org/show_bug.cgi?id=1330672#c1 can reproduce it with an AMD card

I tried on Firefox 50 and Firefox Developer Edition 52 on both. PC using Geforce 770 Ti, the laptop... not sure off hand, i believe it's a low end mobile Geforce

@themoonrat  are you sure pixi is using the webgl renderer?

I found out that this only happens if gl.pixelStorei(gl.UNPACK_PREMULTIPLY_ALPHA_WEBGL, true) is called. Also, I was able to reproduce this problem without using pixi.js, so I'm sure this is an browser issue.
To work around this, you can this line to glTexture.premultiplyAlpha = false. I'm not sure how this will affect transparent images, but since almost all videos don't have alpha channel, it could check whether if the source type is video or not.
https://github.com/pixijs/pixi.js/blob/dev/src/core/renderers/webgl/TextureManager.js#L128

@yoo2001818 Thank you so very much!

Could you help me to solve this problem?  I couldn't reproduce your solution this in pixi.min.js
`	function PlayGameVideo(lastVideo) {
var texture = PIXI.Texture.fromVideo(lastVideo);
videoSprite = new PIXI.Sprite(texture);
	texture.baseTexture.source.loop = true;

	videoSprite.width = 720;
	videoSprite.height = 480;
	videoSprite.anchor.set(0.5);

	stage.addChild(videoSprite);

        video.currentTime = 0;
	video.play();
}`

Closing this since it has been answered.


[Bahasa Indonesia] ID transalation fundamentals/performance/poor-connectivity

Hi @petele,
CC: @abdshomad
I have finished my Indonesian Translation of Web - Understanding Low Bandwidth and High Latency /web/fundamentals/performance/poor-connectivity/ section.
please review it

@mychaelgo - it also looks like this contains the same changes that are in #3694 and #3695

@petele - The section translated by @mychaelgo is different from #3694 and #3695. It should not be conflicted.

@abdshomad  noted 👍

@petele @abdshomad already updated


Fixed Typo

Description of the change: Just added API after SendGrid to complete the sentence...unless it's meant to read "the SendGrid"
Reason for the change: to make sense
Link to original source: https://sendgrid.com/docs/for-developers/sending-email/api-getting-started/

Closes #

Hello @shinypenny31,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


PLs add it

pls add winshell to py-lance pls and make pylance work like kite , kite does more help on python than pylance pls add more things to pylance
I haveing windows 10 pc pls doit

pls also add this pywhatkit

These modules are not actively maintained. It is unclear what is actually broken in your experience. If you want us to investigate this, could you please use the troubleshooting directions and submit an issue. https://github.com/microsoft/pylance-release/blob/main/TROUBLESHOOTING.md
Also, Kite is closer to Intellicode. Would you please submit your request there.


WIP: Fix/probe compile

Fix compiler errors for probe

closes https://github.com/OneUptime/oneuptime/issues/45


revise history and move the open example to its own branch

Once I have a working base, I don't personally feel the need to keep most of the thrashing around history for this particular repo, so squash my personal config in one branch, and leave the flynn@encom example in another branch (e.g., example, or perhaps main).

If I do that, I can also curate the step-by-step approach for the flynn@encom example.

I reduced this to a minimum working example to iron out the wrinkles with encrypted secrets, and made it into a completely different repository.


Emit type declarations tuned for specified SPIR-V objects + MSL builtin enhancements.

Per extended discussion in issue #186, add support for emitting SPIR-V type declarations tuned for specified SPIR-V objects, as well as SPIR-V OpTypeImage access qualifiers. CompilerMSL uses this to output appropriate MSL image access qualifiers.
Also includes several MSL builtin enhancements, including eliding unused builtin inputs and outputs, and support for gl_ClipDistance in Metal.

Thanks. I will review tomorrow.


Slng number of result

@jwu910 @phuchle @psyduc this has been on deck for quite some time it seems


Change required golang version

It doesn't currently compile under 1.7:

# github.com/cloudfoundry-incubator/check-a-record
src/github.com/cloudfoundry-incubator/check-a-record/main.go:23: u.Hostname undefined (type *url.URL has no field or method Hostname)
src/github.com/cloudfoundry-incubator/check-a-record/main.go:24: u.Hostname undefined (type *url.URL has no field or method Hostname)

Thanks!


[Feature / cli]  Add template / option for solidjs in cli

Description
Since all almost well known frameworks / libraries exist as template in vite, the only that is missing is solidjs.
Since official way of install solidjs, from solid website, already use vite,
i think make sense also exist in vite cli as a template.
As it has a benefit for solidjs itself, since it could turn into official way as well and would be centralized in vite, instead of using
npx degit solidjs/templates/js my-app

thanks
Suggested solution
Just add their template js and ts into vite cli options.
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

IMO it's a good idea, and an interesting thing I found that there are already  template-ssr-solid and template-ssr-solid-ts in https://github.com/bluwy/create-vite-extra but lack a template-solid 😜. Was it intentional? @bluwy

I wanted to add solid templates to create-vite-extra but never got around adding them, but for solid's case, I think we discussed before that it's probably fine to add it in create-vite directly.

https://discord.com/channels/804011606160703521/902594605512548442/1065513317386420255 FYI, found a discussion about this lately.

Hi,
solidjs team already create the plugins with js and ts for vite 4, if you see the  Getting Started  page of solidjs website and installed in your machine like i did.
here the js template vite config:
import { defineConfig } from 'vite';
import solidPlugin from 'vite-plugin-solid';

export default defineConfig({
  plugins: [solidPlugin()],
  server: {
    port: 3000,
  },
  build: {
    target: 'esnext',
    },
  },
});

My recommendations is someone of vite team talk to someone of solidjs team.

I'll be contacting with the owner of Solidjs and making a PR to add all the templates 👍🏼👍🏼

Hi,
solidjs team already create the plugins with js and ts for vite 4, if you see the  Getting Started  page of solidjs website and installed in your machine like i did.
here the js template vite config:
import { defineConfig } from 'vite';
import solidPlugin from 'vite-plugin-solid';

export default defineConfig({
  plugins: [solidPlugin()],
  server: {
    port: 3000,
  },
  build: {
    target: 'esnext',
    },
  },
});

My recommendations is someone of vite team talk to someone of solidjs team.

Does the exact same work for typescript version?

Hi @AbdelrahmanDwedar ,
it seams that already have the answer, since you already start a pull request.
Another thing i would like to say is that the templates ( js and ts)  should, in my opinion, have same structure (like adding the public folder and components folder)  and use as a demo the counter app.
thanks.

We did that things but I don't think that the vite team wants the components directory. They seem to have everything as simple & as lightweight as possible.
We followed the same structure as the React app, the exact same one even the images places.


smallfile_cli.py is in /usr/local/bin, run the script directly

smallfile_cli.py is in /usr/local/bin, when run with python the PATH variable doesn't include /usr/local/bin which causes the job to fail. Invoke the
cli directly instead.
  Great find! We need to work with @bengland2 on having proper exit code status.


Image Gallery Issues with Bootstrap 5.2.3 and up

Was attempting to use this with a newer version of Bootstrap 5 and ran in the same issue as seen here
https://stackoverflow.com/questions/74567169/why-is-data-bs-interval-false-no-longer-working-in-bootstrap-5-2-3-in-year-202
Where the images in the gallery style with auto play through when loaded into the lightbox.

ya this package needs some tlc and maintenance to bring it up with the latest bootstrap. will get to it asap but PRs also welcome!

Hey Travis!
Thanks for replying, I'm at a complete loss when it comes to git, but I can tell you based on what I see what needs to be changed for this specific issue, does that help at all?

I fixed this in another repository, please see this: https://github.com/trvswgnr/bs5-lightbox/pull/70


Checks API Feedback

Hello DCO users!
We recently updated the DCO to utilize the new checks API. This means that we can now explain to users more easily how to add their sign-off to commits. We hope you’ll accept this permissions update to make the user experience for DCO users easier. In the meantime, we will be offering backwards compatible support for statuses until permissions are updated.
Check it out in action here.
You might notice a new button on that screen if you have WRITE access to a repo using the DCO. That button will automatically set the app's status to passing.
You can learn more about the process we went through to update the app here: https://github.com/probot/dco/pull/79. Feel free to comment any feedback or let us know if you have any questions/concerns or run into any problems.

I received the email, yet the ability to accept and grant new permissions is not enabled (can't push green button). I am an admin on the repo @hiimbex?
"[GitHub] A GitHub App, DCO, is requesting updated permissions".
The DCO app by probot is requesting updated permissions
https://github.com/organizations/CertifiedTrue/settings/installations/213902/permissions/update

Weird you got the email but can't authorize the app! This would be a permissions issue unrelated to the DCO. @cliveb I'd recommend contacting GitHub support if this problem persists!


[Bug]: Desktop version startup error

ODC version
mac m1
odc: 4.2.1
OB version
Not involved
What happened?

odc.log
What did you expect to happen?
Enter desktop version
How can we reproduce it (as minimally and precisely as possible)?
Overwrite the installation and start the desktop version
Anything else we need to know?
No response
Cloud
No response

test passed

fixed in v4.2.1


feat(rss): map charm members to their github profiles

Add the team information to map blog post authors to team member's githubs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 6 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 0.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

rss.go
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

rss.go
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
437

💛  - Coveralls

@aymanbagabas what would you suggest doing instead or where else could I put this (in the template file somehow?) that would make it more generic?

I would keep this generic since we want people to use charmbracelet/markscribe as the canonical project

I was also surprised by this addition to this package I thought to be generic and not charm based

@aymanbagabas what would you suggest doing instead or where else could I put this (in the template file somehow?) that would make it more generic?

You could put them in the template file. With the help of go-sprout/sprig, we could map the Charm author from the RSS to their GitHub username 🙂

@aymanbagabas Thanks for pointing me in the right direction here! I did as you said using sprout and it works great. https://github.com/charmbracelet/.github/pull/5/files#diff-885b271153664f100985178e148d28687d6220474d93e4526bf5f9e17c5263d9R11


[SPARK-22852][BUILD] Exclude -Xlint:unchecked from sbt javadoc flags

What changes were proposed in this pull request?
Moves the -Xlint:unchecked flag in the sbt build configuration from Compile to (Compile, compile) scope, allowing publish and publishLocal commands to work.
How was this patch tested?
Successfully published the spark-launcher subproject from within sbt successfully, where it fails without this patch.

Can one of the admins verify this patch?

ok to test

Test build #85213 has started for PR 20040 at commit 1827c2d.

Test build #85213 has finished for PR 20040 at commit 1827c2d.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85213/
Test FAILed.

Test build #4020 has started for PR 20040 at commit 1827c2d.

Test build #4020 has finished for PR 20040 at commit 1827c2d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

retest this please

Test build #85257 has started for PR 20040 at commit 1827c2d.

Test build #85257 has finished for PR 20040 at commit 1827c2d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85257/
Test PASSed.

Merged to master


Move sr-arithmetic to a new crate and add in a fuzzer

This closes #3745. This commit splits up sr_arithmetic.rs into a new crate files, translates the previous 'fuzzing' tests into honggfuzz binaries (I was going off of parity-scale-codec which uses the same, happy to change fuzzing tool though) and adds a (currently failing) multiply_by_rational test that was uncovered via fuzzing.
@kianenigma

The fuzzing code found that this line: https://github.com/paritytech/substrate/blob/0cd9bbfb721b5f0f31982f681bdf1e77612ecc79/core/sr-primitives/src/sr_arithmetic.rs#L1667
sometimes adds 1 to the value of the result when it shouldn't.

We should consider re-implementing the algorithms used by num-bigint for BigUint: https://github.com/rust-num/num-bigint/blob/master/src/algorithms.rs

It might be better to use num-bigint instead after https://github.com/rust-num/num-bigint/pull/101 lands.

We should consider re-implementing the algorithms used by num-bigint for BigUint: https://github.com/rust-num/num-bigint/blob/master/src/algorithms.rs

They are taken from the same source. There might be just some optimisations to apply.

It might be better to use num-bigint instead after rust-num/num-bigint#101 lands.

This is great for a user to add to their runtime if they want it but to be added to the main substrate runtime, it can only happen if we are sure the library does not have a big footprint. This was the main incentive to write this mini library ourselves.
is this PR review ready?

the trait file seems to be mistakingly moved. I would recommend you to:
* Either make the sr-arithmeric a new module of the primitives for simplicity and focus on the fuzzing.

Maybe we should keep SimpleArithmetic trait and Saturating conversion here in the new crate it seems fine, only Lazy and Convert doesn't make sense here and should be moved back to sr-primitives.

One problem with having all the arithmetic stuff in a sr-primitives is that it makes the fuzzer binaries larger and we end up hitting against this issue: https://github.com/rust-fuzz/honggfuzz-rs/issues/19.

After removing #[macro_export], I'm getting the following warning:
warning: unused macro definition
  --> core/sr-arithmetic/src/per_things.rs:11:1
   |
11 | / macro_rules! assert_eq_error_rate {
12 | |     ($x:expr, $y:expr, $error:expr $(,)?) => {
13 | |         assert!(
14 | |             ($x) >= (($y) - ($error)) && ($x) <= (($y) + ($error)),
...  |
20 | |     };
21 | | }
   | |_^
   |
   = note: `#[warn(unused_macros)]` on by default

Not sure how to fix this, as the macro is definitely used.

After removing #[macro_export], I'm getting the following warning:
warning: unused macro definition
  --> core/sr-arithmetic/src/per_things.rs:11:1
   |
11 | / macro_rules! assert_eq_error_rate {
12 | |     ($x:expr, $y:expr, $error:expr $(,)?) => {
13 | |         assert!(
14 | |             ($x) >= (($y) - ($error)) && ($x) <= (($y) + ($error)),
...  |
20 | |     };
21 | | }
   | |_^
   |
   = note: `#[warn(unused_macros)]` on by default

Not sure how to fix this, as the macro is definitely used.

Try #[cfg(test)] as it is only used by test code

Thanks!

The fuzzing code found that this line:
https://github.com/paritytech/substrate/blob/0cd9bbfb721b5f0f31982f681bdf1e77612ecc79/core/sr-primitives/src/sr_arithmetic.rs#L1667
sometimes adds 1 to the value of the result when it shouldn't.

Actually, if you remove this line the fuzzer doesn't crash at all, so the issue here is that we want multiply_by_rational to round up sometimes, but (probably?) not all the time.

This is great for a user to add to their runtime if they want it but to be added to the main substrate runtime, it can only happen if we are sure the library does not have a big footprint. This was the main incentive to write this mini library ourselves.

What's the best way to check the size of the runtime footprint? I think it would be worthwhile to verify this.

What's the best way to check the size of the runtime footprint? I think it would be worthwhile to verify this.

Looking into the .wasm files created in node/runtime should give you some insight. But we have discussed this internally, and the outcome was always to not depend (for now at least) on external libs.

@expenses a noticeable number number of my reviews have been marked as resolved while I see no code change, nor an explanation. It is okay if you disagree and don't want to apply something, but please don't just resolve a review without any further information.
I have went through all of the notes again and marked (hopefully all of the) ones that were mistakingly marked as resolved with  a 🏴. Please give them a look and see if you will fix them or argue why not.
Github hides some review comments automatically, keep an eye on that as well as it can be a holy pain :P

@bkchr I'm a little hesitant to make these changes, because it might be better to save them for another PR. @kianenigma what do you think?

@expenses The recommendations don't seem to be any logic change but rather syntactic improvements. Those can be applied safely.
The migration to criterion can be saved for another PR as it is a bit bulky and I don't want to stall this PR for it too much. @bkchr I have added benchmarks in the past both with and without criterion. If we really want to have as a "we benchmark only with criterion", we should at some point a) make it a strict rule b) migrate the already existing ones.
Personally, I don't think it is worth migrating them and making it strict rule. Given that the required benchmark is fairly simple (like the ones in this crate) the test crate gets the job done okay.


Create authorization app in databio org. for pephubclient

pephubclient is using from app that was created by Rafal. Preferably we should use from app that is linked to one of the organizations.

This issue is outdated


Added shape(std::size_t index) to xt::view

Fixes #1641

Sorry for the late reply, the shape overload taking a size_t is already available in the xconst_accessible base class, however it is masked by the definition of xview::shape(). Making it available again via a using directive should be enough to fix the issue.
Besides, it could be useful to add the assertion on the shape size to xconst_accessible::shape so it is available to every inheriting class.

Ah. I stopped looking further when I found xt::view's shape() method with a member variable. Since I have little time the next two days, maybe I close this PR and you add the using?

Yes, no problem.

Superseded by #1645


Will we add webcam to our situation?

It's very appreciate if you want a panoramic reality for C22. Actually that was indeed in our plan until lots of screen resolution & network limitation happened.
Currently our focus is still on optimization, including resolution, network, and clean code. We will move on when we're satisfied, that will not be a long time.

Cool ! 👍


does service create support docker option "privileged" or ":z" for -v?

version: 1.12.3
I will need to mount some privileged files on the host machine into the container, such as /root/.config/sth.
we can use --privileged or -v source:target:z in the docker run to achieve this, but is it possible when use docker service create?

No, --privileged is not supported, see https://docs.docker.com/engine/reference/commandline/service_create/, https://github.com/docker/docker/issues/25303, and  https://github.com/docker/docker/issues/24862.
Services do not have -v / --volume, but you can specify these options using the --mount flag, again, see the documentation: https://docs.docker.com/engine/reference/commandline/service_create/#/add-bind-mounts-or-volumes
Please keep in mind that the GitHub issue tracker is not a general support forum, but for reporting bugs and feature requests. For other discussions, consider using https://forums.docker.com, or register for the Docker Community Slack channel; https://blog.docker.com/2016/11/introducing-docker-community-directory-docker-community-slack/
I'll close this issue because this is not a bug or feature request


Links to queries are broken.

Links to queries are broken. For example the following URL returns a 404 error: https://hub.steampipe.io/mods/turbot/microsoft365_compliance/queries/azuread_third_party_application_not_allowed.
However this URL is linked to on https://hub.steampipe.io/mods/turbot/microsoft365_compliance/controls/control.cis_v200_2_2?context=benchmark.cis_v200/benchmark.cis_v200_2#sql

Welcome to Steampipe @hannewijkBDO and thanks for spotting the issue 👍.
We will look to get a fix out early 👍.

This issue has been fixed in https://github.com/turbot/steampipe-mod-microsoft365-compliance/pull/49


Vue syntax shorthand not working with hyphenated props

Description
We have the eslint rule 'vue/attribute-hyphenation' active in our project. However, this causes an error in my IDE (Webstorm 2024.1.2) when having camelCase props declared in a component and using shorthand when calling.
Example
ChildComponent.vue
<script lang="ts" setup>
type Props = {
  camelCase: string
};
defineProps<Props>();
</script>
<template>
  <h1>{{camelCase}}</h1>
</template>

ParentComponent.vue
<script lang="ts" setup>
import { ref } from "vue";
import ChildComponent from "./ChildComponent.vue";

const camelCase = ref("Hey!");
</script>
<template>
<ChildComponent :camel-case /> <!-- <-- Error here -->
</template>

I searched for the issue on youtrack, because initially I thought it was a bug from webstorm and found this thread, where Jetbrains responded it should have something to do with the vue language server, but I couldn't find any issue here.
Reproduction
You can review the issue in my repository I created for reproduction here: https://github.com/max-zinn/camelCasePropsTest
Created new project with create-vue
npm create vue@latest
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y

Vue.js - The Progressive JavaScript Framework

✔ Project name: … camelCasePropsTest
✔ Package name: … camelcasepropstest
✔ Add TypeScript? … No / **Yes**
✔ Add JSX Support? … **No** / Yes
✔ Add Vue Router for Single Page Application development? … **No** / Yes
✔ Add Pinia for state management? … **No** / Yes
✔ Add Vitest for Unit Testing? … **No** / Yes
✔ Add an End-to-End Testing Solution? › **No**
✔ Add ESLint for code quality? … No / **Yes**
✔ Add Prettier for code formatting? … No / **Yes**
✔ Add Vue DevTools 7 extension for debugging? (experimental) … **No** / Yes

Installed versions

<EMAIL_ADDRESS><EMAIL_ADDRESS>
Error
It seems like a mathematical expression is recognized through the "-" causing camel - case to be a number, which is wrong.
Vue: Property camel does not exist on type
{ $: ComponentInternalInstance; $data: {}; $props: Partial<{}> & Omit<{} & VNodeProps & AllowedComponentProps & ComponentCustomProps & Readonly<...>, never>;  ... 10 more ...; $watch<T extends string | ((...args: any) => any)>(source: T, cb: T extends (...args: any) => infer R ? (args_0: R, args_1: R) => any : (...ar...
Vue: Type number is not assignable to type string

Reproduction link Vue SFC Playground
I also created a reproduction in Vue SFC online, where the error appears as well.
https://play.vuejs.org/#eNp9Uk1PwzAM/StRLt2k0YoPcRhlE6BJgwNMbMdIaGrdNaNNoiQtQ1X/O05Kx0Bjt9jv+fnZTkPvlArrCuiYxibRXFliwFZqwgQvldSWNERDRlqSaVmSAKnBHnqQpfrOh5ELnFJwwwQTiRTGEgvGzj9VDgJScuuEBozOoSgko0Mm4qhric0wsFCqYm0BI0Jirz12Amd5rxAhFEcHPDqi1mCrjG/CrZECp2hcNaMJlvMC9IuyHK0wOiYecdga+388+ZzVFYz6fJJD8n4kvzU7l2N0ocGAroHRPWbXegO2g2fLZ9jhew+WMq0KZJ8AX8HIonIeO9p9JVK0fcDzbh/9wrnYrMxsZ0GYfihn1DFbz2cUD+A299/oP3Yvwytfx0SLW+yPd+QbpJBxAQstlRl4ncOjjsnSarSFKkO8+6mDprz2D3zm55Om+f05plMSCElWmAvaNo6Q4quiruzv1d9q0G4H6BcHCS+uafsF5jvvaQ==

Webstorm v2024.1 EAP doesn't use the latest vue language server, imo

@so1ve thanks for the hint!
I looked into Webstorms changelog and found this article where they stated they enabled Vue Language server by default now, but not which version. So I checked and found that my Webstorm is bundled with 1.8.27, but the latest version is 2.0.16, so I installed @vue/language-server globally via npm and selected it in webstorm. Now everything works fine, thank you! :)
I'm considering requesting a feature to update it manually/automatically directly in WebStorm though. But I guess this issue can be closed as it's clear now that the issue didn't come from vue.


info = ydl.extract_info(url, download=False) RecursionError: maximum recursion depth exceeded while calling a Python object

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.09.24 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
maximum recursion depth exceeded while calling a Python object
Traceback (most recent call last):
File "fb_scraper.py", line 67, in reels
info = ydl.extract_info(url, download=False)
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1564, in extract_info
return self.__extract_info(url, self.get_info_extractor(key), download, extra_info, process)
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1575, in wrapper
return func(self, *args, **kwargs)
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1731, in __extract_info
return self.process_ie_result(ie_result, download, extra_info)
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1813, in process_ie_result
extra_info=extra_info)
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1564, in extract_info
return self.__extract_info(url, self.get_info_extractor(key), download, extra_info, process)
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1575, in wrapper
return func(self, *args, **kwargs)
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1710, in __extract_info
ie_result = ie.extract(url)
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/extractor/common.py", line 715, in extract
ie_result = self._real_extract(url)
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/extractor/generic.py", line 2563, in _real_extract
embeds = list(self._extract_embeds(original_url, webpage, urlh=full_response, info_dict=info_dict))
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/extractor/generic.py", line 2588, in _extract_embeds
gen = ie.extract_from_webpage(self._downloader, url, webpage)
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/extractor/lazy_extractors.py", line 26, in getattr
return getattr(cls.real_class, name)
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/extractor/lazy_extractors.py", line 26, in getattr
return getattr(cls.real_class, name)
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/extractor/lazy_extractors.py", line 26, in getattr
return getattr(cls.real_class, name)
[Previous line repeated 485 more times]
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/utils/_utils.py", line 5016, in get
return self.func(cls)
File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/extractor/lazy_extractors.py", line 33, in real_class
cls._real_class = getattr(importlib.import_module(cls._module), cls.name)
File "/usr/lib64/python3.7/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1006, in _gcd_import
File "", line 980, in _find_and_load
File "", line 148, in enter
File "", line 174, in _get_module_lock
File "", line 59, in init
RecursionError: maximum recursion depth exceeded while calling a Python object
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out UTF-8, error UTF-8, screen UTF-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] API
[debug] params: {'verbose': True, 'compat_opts': set(), 'http_headers': {'User-Agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/92.0.4515.131 Safari/537.36', 'Accept': 'text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8', 'Accept-Language': 'en-us,en;q=0.5', 'Sec-Fetch-Mode': 'navigate'}}
[debug] Python 3.7.16 (CPython x86_64 64bit) - Linux-5.10.192-182.736.amzn2.x86_64-x86_64-with-glibc2.2.5 (OpenSSL 1.0.2k-fips  26 Jan 2017, glibc 2.2.5)
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.19.0, brotli-1.1.0, certifi-2023.07.22, mutagen-1.47.0, sqlite3-2.6.0, websockets-11.0.3
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
https://www.facebook.com/594872649014090
[generic] Extracting URL: https://www.facebook.com/594872649014090
[generic]<PHONE_NUMBER>14090: Downloading webpage
[redirect] Following redirect to https://www.facebook.com/login/?next=https%3A%2F%2Fwww.facebook.com%2F594872649014090
[generic] Extracting URL: https://www.facebook.com/login/?next=https%3A%2F%2Fwww.facebook.com%2F594872649014090
[generic] ?next=https://www.facebook: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] ?next=https://www.facebook: Extracting information
[debug] Looking for embeds
WARNING: Falling back to normal extractor since lazy extractor VLiveWebArchiveIE does not have attribute real_class; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
maximum recursion depth exceeded while calling a Python object

Traceback (most recent call last):
  File "fb_scraper.py", line 67, in reels
    info = ydl.extract_info(url, download=False)
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1564, in extract_info
    return self.__extract_info(url, self.get_info_extractor(key), download, extra_info, process)
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1575, in wrapper
    return func(self, *args, **kwargs)
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1731, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1813, in process_ie_result
    extra_info=extra_info)
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1564, in extract_info
    return self.__extract_info(url, self.get_info_extractor(key), download, extra_info, process)
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1575, in wrapper
    return func(self, *args, **kwargs)
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/YoutubeDL.py", line 1710, in __extract_info
    ie_result = ie.extract(url)
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/extractor/common.py", line 715, in extract
    ie_result = self._real_extract(url)
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/extractor/generic.py", line 2563, in _real_extract
    embeds = list(self._extract_embeds(original_url, webpage, urlh=full_response, info_dict=info_dict))
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/extractor/generic.py", line 2588, in _extract_embeds
    gen = ie.extract_from_webpage(self._downloader, url, webpage)
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/extractor/lazy_extractors.py", line 26, in __getattr__
    return getattr(cls.real_class, name)
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/extractor/lazy_extractors.py", line 26, in __getattr__
    return getattr(cls.real_class, name)
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/extractor/lazy_extractors.py", line 26, in __getattr__
    return getattr(cls.real_class, name)
  [Previous line repeated 485 more times]
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/utils/_utils.py", line 5016, in __get__
    return self.func(cls)
  File "/home/datascience.apis/.local/lib/python3.7/site-packages/yt_dlp/extractor/lazy_extractors.py", line 33, in real_class
    cls._real_class = getattr(importlib.import_module(cls._module), cls.__name__)
  File "/usr/lib64/python3.7/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1006, in _gcd_import
  File "<frozen importlib._bootstrap>", line 980, in _find_and_load
  File "<frozen importlib._bootstrap>", line 148, in __enter__
  File "<frozen importlib._bootstrap>", line 174, in _get_module_lock
  File "<frozen importlib._bootstrap>", line 59, in __init__
RecursionError: maximum recursion depth exceeded while calling a Python object

Duplicate of #8168


Slowing down the action - CPU vs. CPU

I'm a weirdo who likes to run AI vs AI tournaments in my spare time. As this is the best tool atm for doing so in terms of sheer card pool, I was wondering if there was any way to slow down the game speed to make it easier to follow the action.

One way you could do this is by using cheat engine to slow down the game. vsync needs to be turned off though which requires a hook. I'll add the vsync hook in the future.

Gotcha! Should probably open a separate ticket for this request, but would it be possible to pause the duel mid-game or is that a cheat engine situation as well?
Also like, adding new icons for duelists? Is that something one can do, or does it have to replace old icons? Pretty much all I wanted to know about!

The current issue is fine.
Fully pausing the game might break rendering. It might be possible to hack in the pause duel button, but the button might appear on top of / under something else as it wasn't designed for this.
It's possible to add new icons but multiple things need to be edited. I'll make a guide in the future.


Add brcmfmac43456-sdio for Pi400 and CM4

Pi400 and CM4 use brcmfmac43456-sdio

- What I did
Pi 400 and CM4 use the Broadcom BCM43456 WiFi chip instead of the BCM43455 in the Pi 4. This adds support.
- How I did it
Pull in the firmware files for brcmfmac43456-sdio from the RPi-Distro repo

Can one of the admins verify this patch?

@agherzan should hopefully be in the correct format now?

@agherzan-ci ok to test

@agherzan CI failed - then I realised there was an extra character lingering in the first patch.
Fixed now though


Add support for linux

The linux objdump (at least on ubuntu) uses <> instead of _.
Example output from objdump -D on ubuntu -<PHONE_NUMBER>402bf0 <runtime.c64hash>:
This way it works for compiled binaries on linux (checked on Ubuntu only) with CGO_ENABLE=1 and CGO_ENABLE=0

Is this patch all that's necessary for it to work on Linux? You don't need to trim off the ">" at the end?

At 149 you cut "tail" from dot.
So  it's up to you - but this particular func dont care - if you say so I'll update pr


AClient will not work on Windows

TLDR: Unless you're a Windows expert and know how to fix this, explicitly document that we do not support Windows.
AClient needs to open a COM port, but on Windows you need Administrator privilege to do that. And apparently even when you run a python script under Admin, it still refuses to open a com port, with error "Permission denied".
To test this, open a command prompt as admin (right click on command prompt, open as administrator), run python, and:
import os
os.open("\\.\COM", os.O_RDWR)
(Yes, you need all that backslash magic because Windows)
That will actually work. However, running otl866 will not, because otl866 is an exe, and I guess Windows trashes the permissions.

The core functions should abstract pexepect under the hood pretty well. I imagine we could write a generic pexpect replacement pretty easily just using pyserial read() and buffering it ourselves. The biggest drawback would be that we'd loose select and consume more CPU (at least on Windows)

Can we consolidate the other issues into this one? Seems like the core issue, and having the others open may make this difficult to track

Well, I think this is really the only issue -- opening a COM port on Windows in Python via a script installed through setup.py -- and it's a bit of a tough nut to crack. Apparently even my Google-fu isn't good enough to find a workable solution to this one.
I've closed the other issue #67 .


error when install megatron

🐛 Bug
hi,
I want to training language model with transformer_lm by multi GPU, and program show error
ModuleNotFoundError: No module named 'fairseq.model_parallel.megatron.mpu'
after that error show suggest install megatron:
Please install the megatron submodule:
  git submodule update --init fairseq/model_parallel/megatron

but when i am running this command, another error occurr:
fatal: reference is not a tree: adb23324c222aad0aad89308e70302d996a5eaeb
Unable to checkout 'adb23324c222aad0aad89308e70302d996a5eaeb' in submodule path 'fairseq/model_parallel/megatron'

I clone and using the lastest version of fairseq. How can i fix this problem, thankyou

Following-up here; I received the same error when attempting to install Graphormer  (i.e., git clone --recursive https://github.com/microsoft/Graphormer.git)
fatal: reference is not a tree: adb23324c222aad0aad89308e70302d996a5eaeb
Unable to checkout 'adb23324c222aad0aad89308e70302d996a5eaeb' in submodule path 'fairseq/fairseq/model_parallel/megatron'
Failed to recurse into submodule path 'fairseq'

I too am running into this issue when attempting to install fairseq as instructed in the current README

@trangtv57  did you find a solution to this?

@IanMGriff  @atosystem any solution?

@ayushnoori  I also encountered the same problem, which will lead to the problem of reproducing the grapher later. How do you solve it？


Unsynchronized bounded mpsc channel

I've attempted to implement a variant of sync::mpsc specialized for communication between tasks on the same thread. Atomicity overhead is avoided and message storage is pre-allocated, so sending messages through the channel should be fast. Sending a message will only produce a dynamic allocation when the number of blocked sender tasks increases substantially over the previous high watermark, which should only happen a small number of times per lifetime in typical use-cases.
This is my first attempt at inter-task synchronization code, so please review carefully. I'm not sure whether the unit tests adequately test inter-task behavior, though I was able to successfully test some simple multitasking network code I have using this channel.
To support the introduction of this and future additional channel types, I moved SendError out of sync::mpsc and into sink for easy reuse, pub useing the old location with a deprecation attribute to maintain backwards compatibility.

I've implemented all of your suggestions that made sense to me, and fixed a potential deadlock when the receiver is dropped while any senders are blocked. I can't imagine why SendError would need to be different between any channel implementations, but I suppose it doesn't cost much to be conservative here. There were a few suggestions I didn't implement:

store a weak pointer in the Receiver

This would cause in-flight messages to be silently lost when all senders are dropped. I instead simplified the ownership story with Rc::get_mut, which preserves property of never losing messages so long as a receiver exists.

once all senders are dropped it should unpark the receiver, and that could be handled naturally with a Drop implementation if the receiver stores a weak pointer.

Because a weak pointer wasn't used, this doesn't fall out very well. I opted instead to unpark the receiver on every Sender drop for simplicity, since spurious wakeups should be tolerated; I can change this to manually reference-count senders if that's preferred. If the rc_counts feature was stabilized, Rc::strong_count would mitigate the need for additional manual reference-counting.

could a test be added for dropping all senders waking up the receiver

I'm not sure how to write a test that models task (un)parking. Can you point me at an example?

On further consideration, I think it actually makes sense for the senders to hold weak pointers. I've made that change, resulting in some slight simplification.

Applied the second batch of comments. Any tips on testing task dynamics?

Oh right sorry I missed that. In this test suite the easiest thing historically has been to spawn threads and call .wait(), but given how this is not a synchronized version that may not be easy to do so.
To write a test with multiple tasks you may have to cook up a local executor which uses rc handles and such to emulate a mini event loop.

Pushed an attempt at that. I'm slightly suspicious because at no point did I actually implement Executor, but it seems to work. I'm not sure why Unpark implementations need to be Sync--surely they're only ever seen by one thread at a time in idiomatic code?

Moved local_executor into tests/support and tweaked its API to be a bit closer to tokio-core's as suggested. I've also got most of a oneshot implementation written and will open a separate PR for it once it has tests and this one's been merged.

I feel like something like local_executor might be useful to have publicly available for e.g. handling large numbers of sync streams, but I suppose such a thing would be better off in another crate, seeing as it only needs public APIs anyway.

Awesome, thanks @Ralith!
I'll look to merging soon


Add a way to get the app instance

In my app I make extensive use of the app instance that I can get anywhere because the NestFactory returns an instance. The use case is DB models and similar classes that aren't part of the dependency injection directly. But the CommandFactory does not return the instance, and I understand why - nothing can be returned until the command is done running. So I figured we could do it with a callback instead.

This is wrong pls don't merge. I'll redo the PR at a later date better. I just tested it lol


implemented extensions

Hi @Furkanzmc , sorry for the 3 month(!) delay. Life.
(It makes me all the more impressive that open source software happens and is supported at all, thank you.)
This is the implementation of items 1-3 in the spec issue #23.

Ok, thanks for the feedback.
I have checked teh spelling, run stylua and removed the ids argument.
I have also merged the changes you made in master and updated the example of the contains function.
Finally I have improved teh parent_id documentation.

Also sorry for the poluted commit history. It got tangled when I merged in master, but didn't realise and then redid it...

Hi @Furkanzmc , is there anything more that I need to do for this pull request?

Hi, thank you! Sorry for my impatience. I know what you mean about work being crazy!


Test Extractor short feedback of what was identified

Description of the new feature / enhancement
Text Extractor should just for a brief moment show the text that has been extracted, so the user can ensure it's actually correct. As of right now Text Extractor isn't 100% reliable
Scenario when this would be used?
This feature would be useful every time one uses Text Extractor
Supporting information
The extracted text could be shown right below the pop-up where the language can be changed

/dup #20305


feature request: functions to retrieve previous/next photo

I'd like to bind left/right arrow key presses to functions which move to previous/next photos on a map. Currently, browsing through all the photos on a map is a little tedious (requires a lot of clicking; close current photos, re-expand cluster group, pick next photo, etc).
The biggest challenge seems to be figuring out which photo is the correct previous or next photo from the currently-displayed popup. If you can suggest a way you'd approach this problem, I'm willing to try creating something. Do you think this can be implemented in one way for maps with and without markercluster, or will these require separate implementations?
I see that L.layerGroup has a getLayer function, which seems like it would be useful, but it's not clear what kinds of id values it's expecting or how those are generated.

I agree that feature can be a real + ;)

@turban Any ideas on how you'd approach this?


Feedback FlexibleAdapter Problem

How to use the FlexibleAdapter in an Activity, showing multiple different layouts?

@TCQQ, as explained you in fb, 1 item has 1 layout for 1 view holder.
The demoApp I made is done exactly in that way, so if you don't know precisely, please, study the example items and also read the documentation in the Wiki page and the javaDoc too.

Thank you, I understand.
        val adapter: FlexibleAdapter<IFlexible<*>> = FlexibleAdapter(getItems(), this, true)
        recycler_view.layoutManager = SmoothScrollLinearLayoutManager(this)
        recycler_view.adapter = adapter
        recycler_view.setHasFixedSize(true)

    private fun getItems(): List<IFlexible<*>> {
        val items = ArrayList<IFlexible<*>>()
        items.add(TestOneItem("1", "a1"))
        items.add(TestOneItem("2", "a2"))
        items.add(TestTwoItem("3", "b1"))
        return items
    }


Update payout rate

Update payout_rate to 3 cents per character on 1 Aug.

@jhirschkorn Should this change apply to PRs created or merged after 1 Aug?

Will I get the old rate for already written articles?

@jhirschkorn Should this change apply to PRs created or merged after 1 Aug?

created after 1 aug

Will I get the old rate for already written articles?

yes

@kishoian, note the latest comment from @jhirschkorn in the issue. We are raising the rates on specific entities we are interested in.


Skip object detector unit tests in Travis CI for now

We need to get our Travis CI build under two hours more reliably. We can skip the OD tests for now, since I'll likely be running then manually a bunch anyway in the near future (aside from any other CI coverage).

If this doesn't suffice, next up are probably the recommender tests and the image similarity tests, just in terms of shaving the most minutes off with one test suite. Meanwhile, we need to start investing in a more sustainable test design strategy.

Good idea @nickjong. When this is merged please update #1031.
Making the image classification and image similarity test run much faster is on my list of things to do. Hoping to have time to get to that next week.


live data crash on mouse wheel scroll

exception: "System.ArgumentException: 'mouse wheel was scrolled but neither direction is set'" on live data example when mouse wheel is excessively zoomed in and out. Have tried render locking, forcing zoom in only one direction, and slowing down mouse wheel scroll. Found bug in my own code and replicated in live data demo. If mouse wheel delta value is low, did not see issue or if slowly moved. But if control is used by another user, they won't know to avoid issue so looking for a fix.

Hi @Jordant190, thanks for reporting this!
I fixed this issue (#2600 #2626) and will publish an updated package to NuGet tonight, but let me know if it doesn't work as expected after the upgrade 👍


Update: AKS public standard load balancer outbound port calculation

Type of issue
Other (describe below)
Feedback
Original calucaltion:
64,000 ports per IP / <outbound ports per node> * <number of outbound IPs> = <maximum number of nodes in the cluster>.

Propose change:
The maximum number of nodes in the cluster = (sum of the maximum or fixed node counts across all agent pools) + (one buffer node for each agent pool)

To figure out ports/IPs: (64,000 ports per IP / <outbound ports per node>) * <number of outbound IPs> >= <maximum number of nodes in the cluster>

Page URL
https://learn.microsoft.com/en-us/azure/aks/load-balancer-standard#configure-the-allocated-outbound-ports
Content source URL
https://github.com/MicrosoftDocs/azure-aks-docs/blob/main/articles/aks/load-balancer-standard.md
Author
@asudbring
Document Id
6e30ff27-5ab1-75a8-23a0-13e3927d888a

Hi, is there any further details require for this update? I can provide more context, just let me know.


Isomorphic mapped types and unions

A mapped type of the form { [P in keyof T]: X }, where T is some type parameter, is said to be an isomorphic mapped type because it produces a type with the same shape as T. With this PR we make instantiations of isomorphic mapped types distributive over union types. For example, when { [P in keyof T]: X } is instantiated with A | B for T, we produce { [P in keyof A]: X } | { [P in keyof B]: X }. Furthermore, when a primitive type is substituted for T in an isomorphic mapped type, we simply produce that primitive type. For example, when { [P in keyof T]: X } is instantiated with A | undefined for T, we produce { [P in keyof A]: X } | undefined.
type A = { a: string };
type B = { b: string };
type C = { c: string };

type T1 = Partial<A | B | C>;  // Partial<A> | Partial<B> | Partial<C>
type T2 = Readonly<A | B | null | undefined>;  // Readonly<A> | Readonly<B> | null | undefined
type T3 = Readonly<A | B[] | string>;  // Readonly<A> | Readonly<B[]> | string
type T4 = Readonly<string>;  // string
type T5 = Readonly<string | number | boolean>;  // string | number | boolean

This PR also includes a minor fix that makes any isomorphic mapped type assignable to an empty object type.

@mhegazy Ping!

//cc: @DanielRosenwasser and @RyanCavanaugh


[extra metadata loader] Make downloading video's for only steam / PC Games optional

Extension
Extra Metadata Loader
Describe the solution you'd like
Make downloading video's for only steam / PC Games optional, i used the functionality also for xbox and playstation (console) games. As for a lot of games on those platforms the games also exists on steam and the video / microtrailer can be used
Screenshots

Done in latest version.


Feature: $comment for fail2ban::filter

I often need a comment in config files that I deploy.
It allows to keep track of why things have been done in that specific way.
If you agree with this feature, I can do the same for fail2ban::jail defined type.
Regards,

Thank you for contributing. I don't know why NodeSecurity got involved, I must have something misconfigured. I will not be able to look at it until tonight, about 10 hours from now.

NodeSecurity problem should be fixed.

If you could fix the error, verify that all tests run to completion, and re-submit, I would like to accept this PR. Thanks!

Here is the fix against the rake test failure !
I know nothing about node security platform, I'll try to have a look soon.
Regards,

Node security was my problem, not yours. It is a service that examines javascript files. I don’t know how I accidentally configured it to look at this puppet code.
From: Cédric DEFORTIS<EMAIL_ADDRESS>Sent: 03/25/2017 5:49 AM
To: calmenergy/calmenergy-fail2ban<EMAIL_ADDRESS>Cc: chrisowensboston<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [calmenergy/calmenergy-fail2ban] Feature: $comment for fail2ban::filter (#1)
Here is the fix against the rake test failure !
I know nothing about node security platform, I'll try to have a look soon.
Regards,
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/calmenergy/calmenergy-fail2ban/pull/1#issuecomment-289201402 , or mute the thread https://github.com/notifications/unsubscribe-auth/AQKb5_ljk2pVZYfG9FQgQ6wvzeDYJ-dBks5rpOMegaJpZM4MlBbl .  https://github.com/notifications/beacon/AQKb59NKR85MXkpOuuY-RrIZAXaWvypbks5rpOMegaJpZM4MlBbl.gif