Fails to build with Clang in MSYS2

Hello,
I tried to build python-winsdk with Clang on MSYS2, but it failed with:
FAILED: CMakeFiles/_winrt.dir/cmake_pch.hxx.pch
C:\msys64\clang64\bin\c++.exe -D_winrt_EXPORTS -IC:/Users/Michel/git/python-winsdk/cppwinrt -IC:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src -isystem C:/msys64/clang64/include/python3.10 -O3 -DNDEBUG -std=gnu++20 -Winvalid-pch -fpch-instantiate-templates -Xclang -emit-pch -Xclang -include -Xclang C:/Users/Michel/git/python-winsdk/_skbuild/mingw_x86_64_clang-3.10/cmake-build/CMakeFiles/_winrt.dir/cmake_pch.hxx -x c++-header -MD -MT CMakeFiles/_winrt.dir/cmake_pch.hxx.pch -MF CMakeFiles\_winrt.dir\cmake_pch.hxx.pch.d -o CMakeFiles/_winrt.dir/cmake_pch.hxx.pch -c C:/Users/Michel/git/python-winsdk/_skbuild/mingw_x86_64_clang-3.10/cmake-build/CMakeFiles/_winrt.dir/cmake_pch.hxx.cxx
In file included from <built-in>:1:
In file included from C:/Users/Michel/git/python-winsdk/_skbuild/mingw_x86_64_clang-3.10/cmake-build/CMakeFiles/_winrt.dir/cmake_pch.hxx:5:
In file included from C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/py.Windows.AI.MachineLearning.Preview.h:5:
C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/pybase.h:802:28: error: expected a qualified name after 'typename'
    struct buffer<typename T>
                           ^
C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/pybase.h:2140:27: error: missing 'typename' prior to dependent type name 'winrt::com_array<IInspectable>::value_type'
            return sizeof(winrt::com_array<T>::value_type);
                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
C:/msys64/clang64/include/c++/v1/__memory/unique_ptr.h:714:32: note: in instantiation of member function 'py::ComArray<winrt::Windows::Foundation::IInspectable>::ValueSize' requested here
    return unique_ptr<_Tp>(new _Tp(_VSTD::forward<_Args>(__args)...));
                               ^
C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/pybase.h:1819:31: note: in instantiation of function template specialization 'std::make_unique<py::ComArray<winrt::Windows::Foundation::IInspectable>>' requested here
            auto array = std::make_unique<ComArray<T>>();
                              ^
C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/pybase.h:1913:30: note: in instantiation of member function 'py::converter<winrt::com_array<winrt::Windows::Foundation::IInspectable>>::convert' requested here
        return converter<T>::convert(instance);
                             ^
C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/py.Windows.Foundation.Collections.h:2114:24: note: in instantiation of function template specialization 'py::convert<winrt::com_array<winrt::Windows::Foundation::IInspectable>>' requested here
                return convert(items);}
                       ^
C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/py.Windows.Foundation.Collections.h:1870:9: note: in instantiation of member function 'py::impl::Windows::Foundation::Collections::IVectorView<winrt::Windows::Foundation::IInspectable>::seq_subscript' requested here
        IVectorView(winrt::Windows::Foundation::Collections::IVectorView<T> o) : _obj(o) {}
        ^
C:/msys64/clang64/include/c++/v1/__memory/unique_ptr.h:714:32: note: (skipping 5 contexts in backtrace; use -ftemplate-backtrace-limit=0 to see all)
    return unique_ptr<_Tp>(new _Tp(_VSTD::forward<_Args>(__args)...));
                               ^
C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/py.Windows.Security.Isolation.h:33:49: note: in instantiation of function template specialization 'py::convert<winrt::Windows::Foundation::Collections::IVectorView<winrt::Windows::Foundation::IInspectable>>' requested here
                py::pyobj_handle py_param1{ py::convert(param1) };
                                                ^
C:/Users/Michel/git/python-winsdk/cppwinrt/winrt/Windows.Security.Isolation.h:546:9: note: in instantiation of member function 'winrt::impl::delegate<winrt::Windows::Security::Isolation::HostMessageReceivedCallback, (lambda at C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/py.Windows.Security.Isolation.h:28:20)>::Invoke' requested here
        delegate(H&& handler) : implements_delegate<winrt::Windows::Security::Isolation::HostMessageReceivedCallback, H>(std::forward<H>(handler)) {}
        ^
C:/Users/Michel/git/python-winsdk/cppwinrt/winrt/base.h:5385:64: note: in instantiation of member function 'winrt::impl::delegate<winrt::Windows::Security::Isolation::HostMessageReceivedCallback, (lambda at C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/py.Windows.Security.Isolation.h:28:20)>::delegate' requested here
        return { static_cast<void*>(static_cast<abi_t<T>*>(new delegate<T, H>(std::forward<H>(handler)))), take_ownership_from_abi };
                                                               ^
C:/Users/Michel/git/python-winsdk/cppwinrt/winrt/Windows.Security.Isolation.h:1630:43: note: in instantiation of function template specialization 'winrt::impl::make_delegate<winrt::Windows::Security::Isolation::HostMessageReceivedCallback, (lambda at C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/py.Windows.Security.Isolation.h:28:20)>' requested here
        HostMessageReceivedCallback(impl::make_delegate<HostMessageReceivedCallback>(std::forward<L>(handler)))
                                          ^
C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/py.Windows.Security.Isolation.h:28:20: note: in instantiation of function template specialization 'winrt::Windows::Security::Isolation::HostMessageReceivedCallback::HostMessageReceivedCallback<(lambda at C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/py.Windows.Security.Isolation.h:28:20)>' requested here
            return [delegate = std::move(_delegate)](auto param0, auto param1)
                   ^
2 errors generated.

Can you give the exact steps to reproduce the problem?
I found and fixed some other errors when trying to compile with clang-cl that ships with VS2022, but did not see this error.

Install MSYS2
Start the MSYS2 CLANG64 env
Install the required dependencies. I don't remember which ones exactly, but here is a list of packages that I have installed, that might be useful:

mingw-w64-clang-x86_64-clang
mingw-w64-clang-x86_64-cmake
mingw-w64-clang-x86_64-ninja
mingw-w64-clang-x86_64-python-scikit-build

Clone the repo
Apply the following changes:

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 752a0c17..2dad77e8 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -9,12 +9,14 @@ file(GLOB headers RELATIVE "${CMAKE_CURRENT_SOURCE_DIR}"
     "pywinrt/${CMAKE_PROJECT_NAME}/src/*.h"
 )

-set(CMAKE_CXX_STANDARD 17)
+set(CMAKE_CXX_STANDARD 20)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 set(CMAKE_MSVC_RUNTIME_LIBRARY "MultiThreaded$<$<CONFIG:Debug>:Debug>")

-# TODO: re-enable /GL and /LTCG when we get a bigger build machine
-add_compile_options(/await /bigobj /GR- /permissive-)
+if (CMAKE_CXX_COMPILER_ID STREQUAL "MSVC")
+  # TODO: re-enable /GL and /LTCG when we get a bigger build machine
+  add_compile_options(/await /bigobj /GR- /permissive-)
+endif()

 # strip suffix to allow support for prerelease version
 string(REGEX MATCH "^[0-9]\\.[0-9]+(\\.[0-9]+)?" PYTHON_BASE_VERSION ${PYTHON_VERSION_STRING})
diff --git a/setup.py b/setup.py
index 26aea3fb..3cc7175c 100644
--- a/setup.py
+++ b/setup.py
@@ -29,7 +29,7 @@ setup(
         ],
     packages=find_namespace_packages(where=("pywinrt")),
     package_dir={"": "pywinrt"},
-    cmake_args=['-DCMAKE_BUILD_TYPE=Release', '-DCMAKE_C_COMPILER=cl', '-DCMAKE_CXX_COMPILER=cl'],
+    cmake_args=['-DCMAKE_BUILD_TYPE=Release', '-DCMAKE_C_COMPILER=cc', '-DCMAKE_CXX_COMPILER=c++'],
     # recursive glob (**) doesn't seem to work here
     package_data={"winsdk": ["py.typed", "*.pyi", "*/*.pyi", "*/*/*.pyi", "*/*/*/*.pyi", "*/*/*/*/*.pyi"]},
 )

Run python.exe setup.py build

Thanks for fixing those errors in pywinrt. Now both Clang and GCC are failing on PyObject to void conversions:
Clang:
C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/py.Windows.ApplicationModel.Appointments.DataProvider.cpp:266:22: error: cannot initialize a member subobject of type 'void *' with an lvalue of type 'PyObject *(PyTypeObject *, PyObject *, PyObject *) noexcept' (aka '_object *(_typeobject *, _object *, _object *) noexcept')
        { Py_tp_new, _new_AppointmentCalendarCancelMeetingRequest },
                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

[and many similar]
GCC:
C:/Users/Michel/git/python-winsdk/pywinrt/winsdk/src/py.Windows.ApplicationModel.AppService.cpp:173: error: invalid conversion from 'PyObject* (*)(PyTypeObject*, PyObject*, PyObject*) noexcept' {aka '_object* (*)(_typeobject*, _object*, _object*) noexcept'} to 'void*' [-fpermissive]
  173 |         { Py_tp_new, _new_AppServiceCatalog },
      |
```
[and many similar]

That should already be fixed by https://github.com/pywinrt/pywinrt/commit/c646dbe4652e036f92aed5823f48a8152f088b9a.

Thanks. That was indeed fixed in pywinrt.
I also have an issue with the linker complaining about duplicate symbols:
ld.lld: error: duplicate symbol: py::buffer_format<bool, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<signed char, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<unsigned char, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<short, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<unsigned short, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<int, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<unsigned int, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<long long, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<unsigned long long, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<float, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<double, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<char16_t, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<winrt::guid, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<winrt::hstring, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<winrt::Windows::Foundation::IInspectable, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<std::__1::chrono::time_point<winrt::clock, std::__1::chrono::duration<long long, std::__1::ratio<1ll, 10000000ll>>>, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<std::__1::chrono::duration<long long, std::__1::ratio<1ll, 10000000ll>>, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<winrt::Windows::Foundation::Collections::CollectionChange, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<winrt::Windows::Foundation::AsyncStatus, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: duplicate symbol: py::buffer_format<winrt::Windows::Foundation::PropertyType, void>
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt.cpp.obj
>>> defined at CMakeFiles/_winrt.dir/pywinrt/winsdk/src/_winrt_array.cpp.obj

ld.lld: error: too many errors emitted, stopping now (use --error-limit=0 to see all errors)

Closing the issue since after https://github.com/pywinrt/pywinrt/commit/64d3e0706bf516334db90d7b2cd1e7827c42b0de I can finally build it 😃
Thanks a lot for your help!


Fix 1337x poster URL

In some of the domains of 1337x, the poster URL already starts with https:. so, adding https: if required only.
Before (Invalid):
"Poster": "https:https://lx1.dyncdn.cc/cdn/83/83d607a6e927e9a6f04d7ec8849f21a3.jpg"
After:
"Poster": "https://lx1.dyncdn.cc/cdn/83/83d607a6e927e9a6f04d7ec8849f21a3.jpg"
Thanks for the API, great work :)

@hemantapkh  thank you for noticing.


Possibility to use custom imagePullSecret in scan jobs (marvin, popeye, trivy)

Feature Description
We want to have a possibility to inject imagePullSecrets values into scanning CroJob definitions (marvin, popeye and trivy). Currently, helm values for imagePullSecrets filed is injected only in Deployment for zora-operator.
Is your feature request related to a problem?
We want to use a private registry for all images related to the Zora project. For that we need to use imagePullSecrets. In values file of Zora helm chart we found a field responsible for that - https://github.com/undistro/zora/blob/98e8e91efa170f7dfa7afc026785606608ac418a/charts/zora/values.yaml#L57
But apparently it works only for zora-operator deployment, not for scan jobs (marvin, popeye, trivy). We also found out that CRD for clusterscan object (from which render of cronjob happens from my understanding - https://github.com/undistro/zora/blob/v0.8.4/charts/zora/crds/zora.undistro.io_plugins.yaml ) doesn't even have imagePullSecrets field, so it's not possible to add it there manually without editing the codebase.
Therefore, cluster cannot pull images from private registry to run scans and corresponding pods go into CrashLoopBackOff state.
What alternatives have you considered?
We haven't found any other way to inject imagePullSecret field into scan definition (CronJob/Job/Pod) without external interventions.
We came up with temporary solution: We use mutations inside Zora namespace to inject imagePullSecrets values into CronJobs definition. It works well but it's still a workaround.

Hi @recroutgr,
Thank you for your detailed feature request.
It makes sense and we understand the need for this enhancement.
We'll take a closer look into it. Contributions are always welcome.


Visual studio code extension host teminated unexpectedly.

VSCode Version:1.15.1
OS Version:windows 10

Does this issue occur when all extensions are disabled?: Yes
Removing 'Git' folder from "C:\Program Files\Microsoft VS Code\resources\app\extensions" the problem is solved. So, there is an issue with the files in this folder.

Please update your VS Code to the current version: https://code.visualstudio.com/


Failure enabling Azure Monitor for Containers

I tried enabling Azure Monitor for Containers and received this error using my pay-as-you-go subscription:
"The subscription is not registered to use namespace 'Microsoft.OperationsManagement'".
Two points:

You may want to consider adding this is prerequisite to the docs.
I checked the portal and this namespace is registered for my subscription. Are there any other namespaces that need to be registered?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ac3c529f-fe57-558f-bd90-02162ff531e8
Version Independent ID: 8300d958-120e-16c1-09cc-29740e6fae00
Content: Overview of Azure Monitor for containers - Azure Monitor
Content Source: articles/azure-monitor/insights/container-insights-overview.md
Service: azure-monitor
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@dstampfli Thank you for your feedback, we will further look into it and get back to you at the earliest

@MGoedtel  - can you take a peek at the doc-update request to add Namespace registration to the pre-requisite section?

@dstampfli - Can you clarify what type of K8s (i.e. AKS, hybrid Kubernetes cluster, etc.) that you were attempting to enable AzMon for containers for? What rights are assigned to your account in the subscription and with Azure Monitor?

@dstampfli  - any update please?

Hi Folks, I used AKS to create an AAD enabled cluster.  I have owner rights on both the tenant the subscription.  The subscription is pay-as-you-go.   I plan on recreating the cluster and trying again in the next few days.  If there are specific checks that I should do, please let me know.  By the way, I have enabled AzMon for Containers many times on my AIRS subscription (I'm a Microsoft CSA) with no issues.

Hi Dave, mind if we continue this convo internally to debug (via email)? Will start a thread with you, Matt and I. we can circle back to this GH issue if we determine this is not an isolated incident.
Will now close this thread but expect a MS mail from me shortly.
Cheers.


tf.summary.image does not work with keras layers in tf 2.0 due to EagerExecution issues

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Official tf docker
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):No
TensorFlow version (use command below):2.0.0-alpha0
Python version: 3.5
CUDA/cuDNN version: 10.1
GPU model and memory:Titan V

I use tf.keras subclassing, but when I want to track the some images in my network, the error at the bottom of the post occurs. Initially,  I think the reason for the error is, that tf.summary.image becomes a normal Tensor as input, but needs an eagerTensor. If I create a random tensor in the summary function and evaluate the type it is an eagerTensor, but if I use a Tensor from a keras.layers.Layer class it evaluates to a normal Tensor and throws the error (you can try it in your code).
First I thought that the tf.dataset returns a non-eager tensor, but if I use a numpy array as input, the same error occurs as well. So the error might be caused by the keras.layer.Layer class.
The error occurs on GPU and CPU.
Working example
import tensorflow as tf
import numpy as np

inp_arr_without_ds = np.random.rand(2, 200, 200, 3)

inp_arr_4_ds = np.random.rand(2, 2, 200, 200, 3)
tf_ds = tf.data.Dataset.from_tensor_slices((inp_arr_4_ds, inp_arr_4_ds))
tf_ds = tf_ds.map(lambda x, y: (x, y)).repeat(10).shuffle(10)

class random_model(tf.keras.Model):

    def __init__(self, name):
        super(random_model, self).__init__(name=name)
        self.conv_1 = tf.keras.layers.Conv2D(3, [3, 3], padding="same")
        self.tf_board_writer = tf.summary.create_file_writer("test")
        self.img_callback = [tf.keras.callbacks.LambdaCallback(on_epoch_end=self.save_img)]

    def call(self, inputs):
        self.initialized_layer = self.conv_1(inputs)
        return self.initialized_layer

    def save_img(self, epochs, logs):
        with self.tf_board_writer.as_default():
            # Does not work: type: class 'tensorflow.python.framework.ops.Tensor'
            tf.summary.image("image", self.initialized_layer, step=epochs)

            # Does work type: class 'tensorflow.python.framework.ops.EagerTensor'
            #tf.summary.image("image", tf.random.uniform([2, 200, 200, 3]), step=epochs)

    def compile_model(self):
        self.compile(tf.optimizers.Adam(0.001), tf.losses.mean_absolute_error)

    def fit_model_with_ds(self, ds):
        self.fit(ds, callbacks=self.img_callback)

    def fit_model_with_array(self, x, y):
        self.fit(x, y, callbacks=self.img_callback)

print(tf.__version__)

# Both do not work
non_ds_model = random_model("non_ds")
non_ds_model.compile_model()
non_ds_model.fit_model_with_array(inp_arr_without_ds, inp_arr_without_ds)

tf_ds_model = random_model("tf_ds")
tf_ds_model.compile_model()
tf_ds_model.fit_model_with_ds(tf_ds)

Error Message

> 2.0.0-alpha0
>
> ---------------------------------------------------------------------------
> TypeError                                 Traceback (most recent call last)
> <ipython-input-25-0b5a4b30a43a> in <module>
>      39 non_ds_model = random_model("non_ds")
>      40 non_ds_model.compile_model()
> ---> 41 non_ds_model.fit_model_with_array(inp_arr_without_ds, inp_arr_without_ds)
>      42
>      43 tf_ds_model = random_model("tf_ds")
>
> <ipython-input-25-0b5a4b30a43a> in fit_model_with_array(self, x, y)
>      32
>      33     def fit_model_with_array(self, x, y):
> ---> 34         self.fit(x, y, callbacks=self.img_callback)
>      35
>      36 print(tf.__version__)
>
> /usr/local/lib/python3.5/dist-packages/tensorflow/python/keras/engine/training.py in fit(self, x, y, batch_size, epochs, verbose, callbacks, validation_split, validation_data, shuffle, class_weight, sample_weight, initial_epoch, steps_per_epoch, validation_steps, validation_freq, max_queue_size, workers, use_multiprocessing, **kwargs)
>     871           validation_steps=validation_steps,
>     872           validation_freq=validation_freq,
> --> 873           steps_name='steps_per_epoch')
>     874
>     875   def evaluate(self,
>
> /usr/local/lib/python3.5/dist-packages/tensorflow/python/keras/engine/training_arrays.py in model_iteration(model, inputs, targets, sample_weights, batch_size, epochs, verbose, callbacks, val_inputs, val_targets, val_sample_weights, shuffle, initial_epoch, steps_per_epoch, validation_steps, validation_freq, mode, validation_in_fit, prepared_feed_values_from_dataset, steps_name, **kwargs)
>     406     if mode == ModeKeys.TRAIN:
>     407       # Epochs only apply to `fit`.
> --> 408       callbacks.on_epoch_end(epoch, epoch_logs)
>     409     progbar.on_epoch_end(epoch, epoch_logs)
>     410
>
> /usr/local/lib/python3.5/dist-packages/tensorflow/python/keras/callbacks.py in on_epoch_end(self, epoch, logs)
>     288     logs = logs or {}
>     289     for callback in self.callbacks:
> --> 290       callback.on_epoch_end(epoch, logs)
>     291
>     292   def on_train_batch_begin(self, batch, logs=None):
>
> <ipython-input-25-0b5a4b30a43a> in save_img(self, epochs, logs)
>      20         with self.tf_board_writer.as_default():
>      21             # Does not work: type: class 'tensorflow.python.framework.ops.Tensor'
> ---> 22             tf.summary.image("image", self.initialized_layer, step=epochs)
>      23
>      24             # Does work type: class 'tensorflow.python.framework.ops.EagerTensor'
>
> /usr/local/lib/python3.5/dist-packages/tensorboard/plugins/image/summary_v2.py in image(name, data, step, max_outputs, description)
>      71     encoded_images = tf.map_fn(tf.image.encode_png, limited_images,
>      72                                dtype=tf.string,
> ---> 73                                name='encode_each_image')
>      74     # Workaround for map_fn returning float dtype for an empty elems input.
>      75     encoded_images = tf.cond(
>
> /usr/local/lib/python3.5/dist-packages/tensorflow/python/ops/map_fn.py in map_fn(fn, elems, dtype, parallel_iterations, back_prop, swap_memory, infer_shape, name)
>     226                                      dynamic_size=False,
>     227                                      infer_shape=True)
> --> 228         for elem in elems_flat]
>     229     # Unpack elements
>     230     elems_ta = [
>
> /usr/local/lib/python3.5/dist-packages/tensorflow/python/ops/map_fn.py in <listcomp>(.0)
>     226                                      dynamic_size=False,
>     227                                      infer_shape=True)
> --> 228         for elem in elems_flat]
>     229     # Unpack elements
>     230     elems_ta = [
>
> /usr/local/lib/python3.5/dist-packages/tensorflow/python/ops/tensor_array_ops.py in __init__(self, dtype, size, dynamic_size, clear_after_read, tensor_array_name, handle, flow, infer_shape, element_shape, colocate_with_first_write_call, name)
>    1036         element_shape=element_shape,
>    1037         colocate_with_first_write_call=colocate_with_first_write_call,
> -> 1038         name=name)
>    1039
>    1040     self._implementation.parent = weakref.ref(self)
>
> /usr/local/lib/python3.5/dist-packages/tensorflow/python/ops/tensor_array_ops.py in __init__(***failed resolving arguments***)
>     742     if isinstance(size, ops.EagerTensor):
>     743       size = size.numpy()
> --> 744     self._tensor_array = [None for _ in range(size)]
>     745
>     746   @property
>
> TypeError: 'Tensor' object cannot be interpreted as an integer
>

@2649 Ran the provided code, it says NameError: name 'map_ds' is not defined. Please provide full reproducible code to investigate it further.

@muddham Sorry, my bad. This was an old variable. Now you can run it.

@muddham investigated further the problem and when you evaluate tf.executing_eagerly() in every function, only the call() function evaluates to False. I am not sure if this is expected or not?

@2649 Able to reproduce the issue with th provided code.
TypeError: 'Tensor' object cannot be interpreted as an integer

Any ideas?

@2649 I think there is some issue with your callback code that is giving the error. If I remove the callback function from model.fit then there is no error. Please check the gist here. Thanks!

@jvishnuvardhan Yes this is correct. I already pointed it out in the first post. The tf 2.0 documentation recommends to use tf.summary.image to log image files. This function excepts an eager tensor. However, when you use keras to train your model,  it will build a static graph and the summary will throw the mentioned error.

Thanks for reporting.
This is by design. tf.keras in 2.0 automatically wrap the call in tf.function (to accelerate computation), so output is symbolic tensor, which then you used for callbacks.
To make this work, there are two approaches:

use the regular model fit with Tensorboard callback, or tf.print (whichever works in your case)
compile the model with run_eagerly=True, so that output is EagerTensor instead of symbolic tensor.

This issue is not resolved for me. I use the regular fit function with the tensorboard callback, but I still get the same error. Setting the run_eagerly parameter also does not help. I use the output from a convolutional layer as input to the image summary operation.
Is there anything else I should be aware of to make this work?

I just switched from the keras compile and fit method to a custom training loop with gradient tape and wrap the computational parts in a tf.function. These are only few more lines of code, but you are very flexible. I'm very satisfied with this functionality. In between the tf.function calls you can use the tf.summary api, because it returns eager tensors. I used this tutorial.

@mlippie If there is something not working with model.fit and callback, can you file another bug and include your code snippet?

Hi @tanzhenyu,
I tested with tensorflow-gpu 2.0.0-rc0 and still have this problem while using tf.summary.image with a tf.keras.layer of a model
TypeError: 'Tensor' object cannot be interpreted as an integer
I follow the Using TensorBoard in Notebooks . You can check this notebook file

@shaolinkhoa
Have you solved this problem?

@fengyang0317
I'm sorry, I couldn't, therefore, I'm using Pytorch now.

This issue should not be closed. Still experiencing this exact issue and the suggestions in [https://github.com/tensorflow/tensorflow/issues/28868#issuecomment-518076766](above comment) by @tanzhenyu are not fixes.

Still experiencing this issue in TF2.1 and TF2.2

Just ran into this issue, tf.summary.image seems to not work with symbolic tensors. I'm trying to display output from a layer in my network.

Just ran into this issue, tf.summary.image seems to not work with symbolic tensors. I'm trying to display output from a layer in my network.

@courageon Not sure If i just ran into the exactly same issue but I managed to fix mine with this:
output = myModel(input)
output = tf.convert_to_tensor(output.numpy().copy())
tf.summary.image("Preview", output, step=0)```


Problem: testnet docs don't mention supported architecture

the published binaries are for x86_64

Closed by #38.


URL was changes after open the lightbox

Hi,
i found a "bug?" on my Projekt. When i open a image the URL in Chrome and IE the url was changed.
The url was changed to the base url without any sub-pages.
In Firefox there are no Problem, it also works on Version 0.1.18.
I uses the slickslider and init so:
$('#slick-demo').slick();
$('#slick-demo').slickLightbox({
src: 'src',
itemSelector: '.item img'
});

Hey, are you running a single page app?

sorry its run on a local typo3-site.

Use an arbitrary web server, don't run it as file://.

If you seriously do need to run it like that, use the v0.1.18 version.


Only superusers can see SeAT Groups menu item

Summary
Nobody can see the SeAT Groups menu item, except the superuser, and therefore they can't apply for managed groups.
Environment

SeAT 3.0.1 in a Docker image. New installation, not an upgrade.
SeAT Groups (Jun 17, 2018 commit)
CentOS 7.5.1804
Apache/2.4.6 (CentOS) as front end.

Discussion
At first I thought, "Ah! They probably need permissions to see the groups." I checked and there was a permission for that. It's apparently not assigned to everyone by default.
"No problem! I'll make a role for it and a SeAT auto-group to have it auto-assigned to everyone."  Good theory, but it doesn't seem to work in practice. I tried creating the auto-group to include all the corporations and assign the SeAT role that only contains Seatgroups.view permission (which was bound to All Corporations).  Nobody was assigned the role by the auto-group, not immediately, not in a few hours, not by the next day.
If I manually added someone to the permissions role, then they suddenly could see the SeAT Groups menu option. I'm thinking needing to manually give everyone permission isn't working as intended, so I must be doing something wrong.

Your initial thinking was right. You need Seatgroups.view permission to be able to see the SeAT Groups. SeAT Groups is not respecting the Corp-Affiliations of roles, you need instead select in the Auto Group the corporation. Not selecting a Corporation (middle coloumn) will not result in receiving any SeAT Group Roles.
This is how it should look like. Make sure (1) is/are the corporations whos members should get the Seatgroups.view Permission:

if not after 15min the users are receiving the roles. Make sure you have no 'ghost' user-groups which were created before SeAT Beta 20. If you have any more questions, join slack/plugin channel i am happy to help there.
This is no Bug/Code error, this is why i close the issue.


Unable to get rid of warning: RealSense UDEV-Rules are missing! UDEV-Rules permissions configuration   for RealSense devices.

librealsense
2.29.0 RELEASE

OS
Linux

Name
Intel RealSense D415

Serial Number
830512060562

Firmware Version
<IP_ADDRESS>

Advanced Mode
YES

Camera Locked
YES

Usb Type Descriptor
3.2

Product Line
D400

Asic Serial Number
842213022371

Firmware Update Id
842213022371

I just newly installed the RealSense, but I always get this warning of, "RealSense UDEV-Rules are missing! UDEV-Rules permissions configuration   for RealSense devices."
I tried using "sudo apt-get upgrade/install librealsense2-udev-rules" but I get a "E: Invalid operation upgrade/install" in return.
I proceed on to key in "sudo cp ~/.99-realsense-libusb.rules /etc/udev/rules.d/99-realsense-libusb.rules && sudo udevadm control --reload-rules && udevadm trigger" nothing returns to me. But problem still persists.

@zhangmingcheng28 , this was a udev-rules discrepancy with the Debian package in the last release. Will be fixed in the upcoming version.


Add Alexa Settings APIs

I'm submitting a...

[ ] Bug report
[X] Feature request
[ ] Documentation issue or request
[ ] Other

Expected Behavior
Implement the Alexa Settings APIs to expose the device's:

timezone
temperature measurement unit
distance measurement unit

https://developer.amazon.com/docs/smapi/alexa-settings-api-reference.html
Current Behavior
Not implemented. New feature.
Error log
none
Your Environment

Jovo Framework version used: 1.4.4
Operating System: Windows 10 Pro

Thanks Mark! @aswetlow added it today, take a look at this example: https://github.com/jovotech/jovo-framework-nodejs/blob/master/examples/alexa_specific/appAlexaSettingsAPI.js
For now, we haven't integrated the getTimestampLocal() feature request (which is great!) because we don't want to add another dependency for just one feature. We're open for suggestions though!

Thanks for #240, I think a simple code example like this is a great compromise 👍


state default is not permitted driver for network

Proposed changes
Explicitly state that default is not a valid driver for compose networks.
Related issues
https://github.com/docker/compose/issues/4905

@samuelcolvin thanks for the contribution, but I don't think we should merge this since it's negative documentation, in the sense that the docs should tell what is supported, and not the infinite number of things that are not supported.
So in this case, it's implied that if you don't specify a driver, the driver used is going to be the default one. Granted that there's a few samples in the same page showing
ipam:
  driver: default

So maybe we should update those samples and remove the driver:default.
ping @shin- for visibility about this inconsistency in the product.

I asked by one of the maintainers of docker-compose to make the change https://github.com/docker/compose/issues/4905#issuecomment-306941066
The ipam: driver: default is a different case: driver is a variable in the ipam dict not in the root of the network definition. At least that's my understanding, the whole section is fairly confused.

Thanks for the fast reply. In docker/compose#4905 you said

The docs here appear to give an example of using default, at the very least they don't make it clear that it's not allowed.

But I could not find any mention of this. As far as I can tell the docs say

The default driver depends on how the Docker Engine you’re using is configured, but in most instances it will be bridge on a single host and overlay on a Swarm.

This implies that you should use driver: bridge or driver:overlay. Everything else is not allowed.
Of course there's a usability issue with the error message that you got, and it seems we're already tracking that at moby/moby#29375
Sorry for making you bounce around multiple repos and spend time creating a PR to them close it. But I hope you understand.

@samuelcolvin @joaofnfernandes My bad, I skimmed through the docs and thought we had driver: default in some network definition examples. I realize now that these were IPAM definitions.

no problem, I got confused by it too.


Returning "Main Article" links as quotes

I noticed this when trying to get quotes from the Plato page on the English wikiquote.
Python 3.6.0 (default, Jan 12 2017, 01:20:20)
[GCC 4.2.1 Compatible FreeBSD Clang 3.8.0 (tags/RELEASE_380/final 262564)] on freebsd11
Type "help", "copyright", "credits" or "license" for more information.
>>> import wikiquote
>>> wikiquote.quotes('Plato', max_quotes=3)
['Main article: Euthyphro', 'Main article: Apology (Plato)', 'Main article: Crito']

Fixed in 0.1.5.


plex: <IP_ADDRESS>41-549599676 -> <IP_ADDRESS>62-0f739d462

Motivation for this change
A new (public & stable) version for the plex media server got released.
"Fixes: Issues starting up (database locked)".
Tested the version on my NixOS plex server. The update went smoothly!
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 131284 at 77e72c43 run on x86_64-linux 1

2 packages built successfully:

plex
plexRaw


problem running buid.sh

error when building tester after running build.sh
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.0.2:compile (default-compile) on project tester: Compilation failure: Compilation failure:
[ERROR] /home/hassan/open_source_workspace/asteroid_hunter/Source Code/tester/src/main/java/gov/nasa/asteroid/tester/ImageService.java:[7,11] error: generics are not supported in -source 1.3
[ERROR]
[ERROR] (use -source 5 or higher to enable generics)
[ERROR] /home/hassan/open_source_workspace/asteroid_hunter/Source Code/tester/src/main/java/gov/nasa/asteroid/tester/AsteroidDetectorTester.java:[217,17] error: generics are not supported in -source 1.3
[ERROR]

I had wrong version of maven


GetExtension should return null in case it didn't find the extension

This is not "mandatory" but would lead to a more friendly approach for a new user that comes across your package. Obviously, if the data is not found, you should never return a value (even if an empty string) as it may lead to strange behavior or misunderstanding.
I would highly recommend to introduce this breaking change in your library:
public static string GetExtension(string mimeType, bool throwErrorIfNotFound = true)
{
     ...

-    return string.Empty;
+    return null;
}

In addition to this, I would recommend anyone making checks against a string to use the extension methods provided to that use:

String.IsNullOrEmpty(String)
String.IsNullOrWhiteSpace(String)

I'm not going to introduce what could easily be a breaking change for everybody. Sorry. We are introducing a TryGetExtension method in another PR which should satisfy your needs.

Can you link that pull request ? I'd like to take a look at the code. I don't see anything related to that but it's a good idea to introduce a TryGetExtension too!

It's now been merged. Take a look at the updated code that includes TryGetExtension. NuGet also updated.


Update content-credentials.md

Edited information on Content Credentials to best align with Microsoft external communications as well as C2PA communications.

#sign-off


The Error at the Binance "More than 1 hours between startTime and endTime"

Hi, guys,
While trading, an error occurred below.
Do you know the reason?
I use ubuntu 16.04.
An error occurred { Error: binance GET https://api.binance.com/api/v1/aggTrades?symbol=XRPBTC&startTime=1514038924092&endTime=1514082124092 400 Bad Request {"code":-1127,"msg":"More than 1 hours between startTime and endTime."} (possible reasons: invalid API keys, bad or old nonce, exchange is down or offline, on maintenance, DDoS protection, rate-limiting)
at binance.defaultErrorHandler (/home/satoko/zenbot/node_modules/ccxt/js/base/Exchange.js:400:15)
at response.text.then.text (/home/satoko/zenbot/node_modules/ccxt/js/base/Exchange.js:413:25)
at
at process._tickCallback (internal/process/next_tick.js:188:7) constructor: [Function: ExchangeNotAvailable] }

This issue was moved to highvelocityspace/zenbot#71

I have the same problem, just appeared today from nowhere, there were no any changes in configuration

Change this in extensions/exchanges/binance/exchange.js
getTrades: function (opts, cb) {
var func_args = [].slice.call(arguments)
  var args = {};
  if (opts.from) {
    args.startTime = opts.from
  }
  if (opts.to) {
    args.endTime = opts.to
  }
  if (args.startTime && !args.endTime) {
    // add 1 hour instead of 12
    args.endTime = args.startTime + 3600000
  }
  else if (args.endTime && !args.startTime) {
    // subtract only 1 hour instead of 12
    args.startTime = args.endTime - 3600000
  }

// Credit to jadedgeek on discord

This is an issue w/ Binance servers where their servers are behind. Their devs had said to change the time server sync interval. I fixed it by modifying the module 'cctx' binance exchange code to send that 'recvWindow' parameter with a large value

Thanks @Jarel1337 🍺 - Potential duplicate at #513
@paroxysm, Please share your modifications as I was also looking into the changing cctx, updated package.json but never circled back.

Thank you so much, guys!
It works now.

I had the same issue, @Jarel1337 fix is quick to apply and works fine for now, but @paroxysm can fix the core issue in ccxt 👍

Well, the issue has to do w/ your local machine time syncing frequency. The fix in cctx is to bypass this by giving it a huge time "window" to mitigate the problem but it's a hack fix as well. Jarel's fix is just as good as trying to hackfix 'cctx' module.

@receptor, @paroxysm I did find the issues related to recvWindow being a potential fix, however seems this particular hack doesn't quite solve the issue when running via docker-compose - is anyone running this in a live scenario or found a workaround to:
binance GET https://api.binance.com/api/v3/account?timestamp=1514521618096&recvWindow=20000000&signature=0ae8bb40876593e1977836d3e5ac8dec89aea1c7debd0339148ea589670671eb
Request:
 { 'X-MBX-APIKEY': 'goonies' } undefined
An error occurred { Error: binance {"code":-1021,"msg":"Timestamp for this request was 1000ms ahead of the server's time."}
    at binance.handleErrors (/app/node_modules/ccxt/js/binance.js:752:27)
    at response.text.then.text (/app/node_modules/ccxt/js/base/Exchange.js:433:18)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:188:7) constructor: [Function: ExchangeError] }

Binance API is down! unable to call getBalance, retrying in 20s

Cheers,
@jadedgeek

@ramene https://github.com/ccxt/ccxt/issues/773
The above problem was caused by the exchange cutting down trade history limits from 24hrs to 1hr, this issue is now fixed in ccxt here: https://github.com/ccxt/ccxt/pull/995


Layers with size of around 2GB cause rce/Docker to exit with an out of memory error while updating.

Can be reproduced by creating a image which contains a big file generated with dd (from /dev/random) via a RUN command.

@imrehg is trying to reproduce this.

@agherzan currently testing this through the regular resin infrastructure does not work do to an open registry issue: https://github.com/resin-io/resin-registry2/issues/8
As an alternate test: trying resin local push, I've successfully deployed a 4000MB layer fine. Thus I would say:

local push suggests that this might not be an issue anymore
if it's still an issue, then it would be through deploying git push or resin deploy, and the registry's issue prevents such large images to reach the device at the moment anyways (ie. releases with such big layers cannot be created at the moment).

Thanks @imrehg for this investigation. I will close in that case this issue for now.

Just for reference, tried one more test based on comments by @shaunmulligan: push an image with a 2000MB layer to Docker HUB and then balena pull onto the device. That should be even closer to the regular deploy. Seemed to have gone through fine.


Stripping keras_learning

I am building an app for android with a keras model exported (through tenserflow).
Yet, it crashes at runtime because there are DT_BOOL node in the graph and the tensorflow library I use doesn't have KernelOp for DT_BOOL registered out of the box. So, I would like to try getting read of all DT_BOOL noes (if it is possible) from the model that I only use for inference on the device.
Is it possible to strip a keras model from all DT_BOOL nodes that depens on keras_learning_phase ?
How can it be done ?
I tried this :
K.set_learning_phase(0)  # all new operations will be in test mode from now on
serialize the model and get its weights, for quick re-building
config = model.get_config()
weights = model.get_weights()
re-build a model where the learning phase is now hard-coded to 0
new_model = Sequential.from_config(config)
new_model.set_weights(weights)
And I also tried a few things on the tensorflow side (optimize_for_inference, freeze_graph, (wrongly) turning Switch node into Identity (I corrupted my graph), ...)

I worked around this issue by exporting the model to pbtxt (protocol buffer text mode)
and then replacing
node {
name: "dropout_1/keras_learning_phase"
op: "Placeholder"
attr {
key: "dtype"
value {
type: DT_BOOL
}
}
attr {
key: "shape"
value {
shape {
unknown_rank: true
}
}
}
}
with a Const op
node {
name: "dropout_1/keras_learning_phase"
op: "Const"
attr {
key: "dtype"
value {
type: DT_BOOL
}
}
attr {
key: "value"
value {
tensor {
dtype: DT_BOOL
tensor_shape {
}
bool_val: false
}
}
}
}
and then I was able to freeze/transform the model and use it in an android application
Sadly, the nodes that depend from this boolean values aren't stripped though. So this isn't the best solution

I finally found the right way to do this:
In a completely new python process (ie. not in a process that I previously used for training), I use
K.set_learning_phase(0) model =  buildMyModel() model.load_weights('myWeights.hdf5') saver = tf.train.Saver() saver.save(K.get_session(), modelName)
and the model is saved without the nodes for learning/training


AoT and Tree shaking for angular 4

How to use AoT and tree shaking with Meteor app? Angular still have bad performance and big bundle size. AoT and tree shaking may solve that problems but i don't see any docs about this features

you can use ionic cli to write the app ,and use meteor as backen using https://github.com/Urigo/meteor-client-bundler ,see the tuturiol whatspp, and then
add
 import { enableProdMode } from '@angular/core';     enableProdMode(); to your main.ts
run
ionic cordova build  android --aot  --minifyjs --minifycss --optimizejs
it perform perfect on mobilephone,just 1.4mb, and load faster just need 3 sec to bootsup.

@blackjack0v0 thx, but i need build for browsers not for mobiles.
I'm personaly already migrating on React.js. Angular-meteor integration looks perspectiveless and unusable for now becouse of a lot of bugs and nobody want to fix them for more the half of year. Looks like this project is aboddoned.

@Deadly0  it also work for browser , you can use ngcli to build your project(don't use meteor-cli) ,just use meteor as backen . it does have many bugs, but I can't leave meteor./.

annnndddd it work!
thanks to the amazing @ardatan !
we've published new angular-compilers package that supports all the features the Angular CLI gives you:

Typescript compilation
Assets compilation (HTML, styles)
Angular's AoT compilation
Rollup's bundling with Tree Shaking

and added a lot of examples with for all those features plus an example with Angular CLI if you want to use it instead of the Meteor CLI:
https://github.com/Urigo/angular-meteor/tree/master/examples
We've also updated all the latest versions including the latest Angular 5 RC and Meteor 1.6.
Please try it out and open a new issue if you encountered one.


Cannot build the microprofile-conference

The microprofile-conference project cannot be build due to unsatisfiable dependency

[INFO] Reactor Summary:
[INFO]
[INFO] Conference ........................................ SUCCESS [0.262s]
[INFO] Conference :: Bootstrap Data ...................... SUCCESS [3.055s]
[INFO] Conference :: Session ............................. FAILURE [0.577s]
[INFO] Conference :: Vote ................................ SKIPPED
[INFO] Conference :: Vote :: Application ................. SKIPPED
[INFO] Conference :: Vote :: Config ...................... SKIPPED
[INFO] Conference :: Speaker ............................. SKIPPED
[INFO] Conference :: Speaker :: Model .................... SKIPPED
[INFO] Conference :: Speaker :: Web ...................... SKIPPED
[INFO] Conference :: Schedule ............................ SKIPPED
[INFO] Conference :: Web ................................. SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 4.285s
[INFO] Finished at: Tue Oct 25 15:12:43 EEST 2016
[INFO] Final Memory: 25M/124M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project microservice-session: Could not resolve dependencies for project io.microprofile.showcase:microservice-session:war:1.0.0-SNAPSHOT: Failure to find org.glassfish:javax.el-impl:jar:3.0.1-b08-jbossorg-1....

What version of maven are you using as I ran into some issues with Wildfly Swarm that were resolved by updating my maven version. Also what command are you using to build the app? mvn install or mvn package?

Apache Maven 3.0.5 (r01de14724cdef164cd33c7c8c2fe155faf9602da; 2013-02-19 15:51:28+0200)
The command is "mvn clean install"

I'm running with 3.2.5, can you try updating and see if it still occurs?

WildFly Swarm requires a minimum of Maven 3.2.5 unfortunately.

Thanks, with maven 3.3.9 the build finished successfully :) What do you think about adding a maven enforcer rule to check the minimum mvn version ? Then the error message would be something like: "Detected Maven Version: 3.3.9 is not in the allowed range [3.2.5,Whatever)."

Probably not a bad idea.
Could you create a new issue and close this one?
Thanks

Sure :)

Really only need this in parent pom:
<prerequisites>    <maven>3.2.3</maven> </prerequisites>

Have exactly this issue using Maven 3.3.9 and Nexus as artefacts repository. Any idea on how to fix it?

The versions of these dependencies seem really weird, why are they requiring versions with jbossorg-1? These are not available in the public Maven repositories. I cannot build any of the samples either because of this.


[RESTEASY-1698]:Add cause to ResteasyViolationException

@asoldano , My mistake,  thanks for spotting this. Add to all constructors and please review.

Thanks :-)


Auto ignore properties when Destination Class does not have that property

New behaviour to nested objects with defined classes. Auto mapping will remove missing properties in destination object that exists in source object
#19

Coverage decreased (-0.02%) to 94.332% when pulling f261296c3150f59d9719e8361f9898de7b4b6e4a on rmalveis:pullRequestBranch into 1355241bc430afc81bac35c156a0977c177487a9 on loedeman:master.


Allow the user to tell the conformance tests leave config in place

What type of PR is this?
/kind feature
What this PR does / why we need it:
With this small change you can run the conformance tests and leave the configuration in place at the end of the run:
 $ go test ./conformance/conformance_test.go --gateway-class={your-gw-class} --cleanup=false
This is helpful when a test fails because you can then explore what went wrong.
Does this PR introduce a user-facing change?:
To run the conformance tests and leave the configuration in place at the end of the run:
 $ go test ./conformance/conformance_test.go --gateway-class={your-gw-class} --cleanup=false

@acnodal-tc this duplicates https://github.com/kubernetes-sigs/gateway-api/pull/1078 which already has some conversation on it

Thanks for the tip! I added my $0.02 to that thread so I'll close this one.


Unable to find the selected font "Cascadia Mono".

Environment
Windows build number: Microsoft Windows NT 10.0.19042.0
Windows Terminal version: 1.6.10571.0

Steps to reproduce
This 'happens on my machine', not sure if easily reproducible.

Start Microsoft Terminal

Expected behavior
Microsoft terminal opens on the default terminal
Actual behavior
A pop-up is shown with the following error message:

Transcript:
Warning

Unable to find the selected font "Cascadia Mono".

"Consolas" has been selected instead.

Please either install the missing font or choose another one.

Additional information
Other applications, Microsoft Word, Paint.NET, show the Cascadia Mono font and allow usage without any issues.

Off course I think of a possible fix right after submitting the issue.
Starting Microsoft Terminal as Administrator once fixes the issue.
Subsequent runs as non-admin do not show the warning anymore.
Leaving the issue open so the maintainers can decide if this should be handled without the 'start as admin' workaround/fix.

After upgrading to 1.6.10571.0, this happens to me as well. But starting as administrator does not help for me. Any other workaround?

I'm experiencing the same behavior, this was not an issue until about a week ago. In my case, it is not finding the "Ubuntu Regular" font, and as you can see, the font is installed correctly.

Hey @miniksa can you take a look at this and #9157? We're seeing a pretty dramatic uptick in the number of these kinds of reports after adding the warning. Did we change anything about the font lookup when we added the dialog?

I have a Similar experience but with the font "Cascadia Code PL". The message does not go away if Started as Administrator.

I've also having an issue with Windows Terminal not being able to find Cascadia Code PL. I've been using Cascadia Code PL for several months with no issues, now Windows Terminal can't find it.

Hey @miniksa can you take a look at this and #9157? We're seeing a pretty dramatic uptick in the number of these kinds of reports after adding the warning. Did we change anything about the font lookup when we added the dialog?

I will not have the capacity to look at this for at least a week.

I am also suffering from this problem, after an automatic update via the Windows Store.

Same thing here, Source Code Pro.

Same issue, Licudia Console.  Starting as Administrator does not fix the issue, either.  Version 1.6.10571.0 updated through the Windows Store.

"Install for all users" fixed this trouble for me.

"Install for all users" fixed this trouble for me.

I've tried that and it didn't make a difference. I even rebooted the computer after reinstalling the fonts using the "Install for all users" option. On startup, Windows Terminal still complains about a missing font that's not missing and can be used anywhere else on the system ("Ubuntu Regular", in my case).

I had the same issue with the Store version, using "Cascadia Code PL Regular" as the font.
If I remove the "Regular" from the font string text, I do not get the error dialogue and the font loads correctly.

I never had "Cascadia Code PL Regular" as the font, only "Cascadia Code PL". And I still get that error dialogue.

I ran into a similar issue. I followed all sorts of instructions from various different posts online and I finally got powershell to recognize the right font but windows terminal would still not pick it up.
here is my powershell core rendering the font Space Mono for Powerline correctly.

A screenshot of the console's properties:

However, I tried the same in Windows Terminal and guess what?

settings file:

I got lucky and got help from smart people online by posting this question on twitter -https://twitter.com/eakangk/status/1369949792034562048?s=20
And someone responded that I was using the full name of the font and not the short name. I quickly opened the modern font setting window and searched for the font 'space' and that was when I realised, I couldn't find it.

However the same font would show up in the old schools c:\windows\fonts folder:

This prompted me to think that maybe, i didn't really install it for all users.
Luckily I had a clone of https://t.co/Xo6f5LikOV?amp=1 so I rightclicked the font and clicked install for all users.

and guess what happened next?

it appeared in Font Settings!
I couldn't wait to open up my windows terminal to check how it looked like and... wait for it....

it worked.
Thank you everyone for the valuable suggestions and directing me to this issue here. i had almost given up on this back in January, thinking I was taking too long to set up a prompt.

@lonelydev, I just checked my C:\Windows\Font folder. I don't have any font there named Space Mono for Powerline there. In fact, I don't have any font there that starts with Space Mono. Should I try to find that and install it?

@Rod-in-NM
seems like you need powerline fonts from https://github.com/powerline/fonts
once you clone the repo to your pc, you can then choose to install the fonts you wish to Windows right clicking the font image and then clicking install for all users.

In my case, I configured my Windows Terminal to use Space Mono, so I only selectively installed that font and it started rendering correctly.

@lonelydev I've cloned that font repo. The I installed for all users these fonts:

Space Mono Bold for Powerline
Space Mono Bold Italic for Powerline
Space Mono for Powerline
Space Mone Italic for Powerline

I didn't think I needed the others.
This hasn't helped - I'm still getting that error message whenever I get into Windows Terminal

you still getting
 ?
specifically Space Mono for Powerline?

No, I'm getting this error:

hey @Rod-in-NM
now that you have installed Space Mono for Powerline for all users, you should probably edit your Windows Terminal settings file to have Space Mono for PowerLine as the fontFace instead.

However, if you prefer Cascadia Code, download it from https://github.com/microsoft/cascadia-code/releases
and do the same "Install for all users" for that one too.
You can check if it has been installed properly by doing what I did, searching both c:\windows\fonts and also the font settings window as i posted here - https://github.com/microsoft/terminal/issues/9375#issuecomment-797136185

Install for all users FTW!

I'm having kind of the same problem but with another font not mentioned here. FiraCode Nerd Font.

It is both in the settings windows and the folder ... Although with different names BUT it doesn't finds it with neither.

I have the same problem (Windows Terminal v1.6.10571.0 and "Monospace Regular" font). Installing the font "for all users" doesn't solve the problem for me. Reinstalling windows terminal doesn't help either.
But rolling back to the previous version (v1.5.10411.0) helped me.

It helped  >
https://github.com/microsoft/cascadia-code/releases

As potentially helpful information, it worked when specifying a font family, but not when specifying a specific font -- in this case, Hasklig worked fine but Hasklig Light (or any Hasklig $variation) did not.

Okay.
Here's my plan:

A. I'm going to remove the string comparison test as a way of triggering this dialog box. Instead, I'm going to make the font resolution code return a bool or some sort of indicator when it decides to try the hardcoded font list (Consolas). The dialog will only pop if that flag is set. This should eliminate the scenario where the proposed name was close enough for DirectWrite to resolve it to ONE of the font's known names and we give a really dumb pop up because the string doesn't match in case (or otherwise exactly)
B. I'm going to add a small amount of retry resolution in the font fallback routine (prior to the previous flagging) that will trim off, word by word from the tail, the phrase that you've given us as the font and ask DirectWrite to try again. That will crudely resolve the case where people are specifying weight or other suffixes and we fall back to Consolas and show the annoying dialog.
C. In the future, we'll perhaps try to parse weight suffixes out of the specified font name and transform them into weight values or something more complicated. Backlog item.
D. For the cases where the font is somehow installed but is not turning up in the system font cache, I'm generally going to have to defer to the OS side bug. I'm not supposed to really be able to do anything if the system can't figure out the difference between per user and per system installs or mushes up the status of fonts registered by installed packages.
E. In the future, we'll try to have a fallback font loader that will at least attempt to get the Cascadia variants out of our own package by direct file open if the rest of the system font resolution chokes. Backlog item.

The first two (A and B) are what I want to send as a PR for this bug.
I'll file or find and link the backlog ones (C and E).
And I'll try to file or find and link the OS issue as well (D).
Finally, @DHowett and I have an alternative/interim solution:

@DHowett plans to pull the dialog from stable so it at least stops annoying people and is easy to service
@DHowett and I agree that if we reach the end of April and my proposed fixes get into people's hands and this is still a noisy issue... then we are going to yeet the entire dialog into space and note that it can only return once we have code that can better understand the text string for the font (like parsing out the weight as I said above.)

Questions, comments, concerns... let me know. Otherwise just give me a thumbs up and give me a chance to get it done. Thanks.

I also experienced this issue when installing Terminal with Scoop, but not with the MS store version.
I found a related open request on the scoop-extras bucket: [Request] Add default font Cascadia-Mono to the Windows Terminal manifest #5783

@Xbuduo, @matthewbarreiro, and to anyone who used scoop to install Terminal, I resolved this "Cascadia Mono" font issue by also installing the cascadia-code package. It requires an admin prompt and the nerd-fonts bucket.

Open terminal as Administrator
Run scoop bucket add nerd-fonts
Run scoop install cascadia-code

Hope that helps.

I have done A, B, and E in the linked PR #9734.
I will still generate a follow up for C.
On the OS side (part D), we have identified a bug with package installation and the font paths in the registry and have the relevant teams committed to fixing it.

open Windows Terminal Settings,  add/change fontFace setting
{
    "profiles": {
         "defaults":  {
                "fontFace": "Consolas"  //  edit here

It started happening to me today. Weeks no issues then all of WSL linux sessions closed, about 8 of them, while they were running tasks and without notifying me or giving me the option to delay/cancel/not update at this time!  This is a very very bad update procedure as it closed active SSH connections in all of these. Fortunately I started the critical jobs via screen.
Apps, especially terminal apps, that can be open for extended periods of time while running long-running jobs should not silently just close and update in the middle of the night.
As near as I can tell an update was applied and my terminal is now 1.9.1523. Not happy that it silently updated and closed my active sessions. And now I have the missing font error too. Which I did not have before. In fact everything had been working very well before. Now it's having problems. Very annoying.

As annoying as it was, I normally do not reboot because I might have things going for several days, I closed everything down and rebooted and it appears the error with the font has gone away.
So this "update" not only closed all my active terminal sessions without warning in the middle of the night, it also forced a reboot to fix this error.
Lest I only complain - I realize this is a preview release, so some hiccups are to be expected. I was caught off guard by the sudden closing and update without warning though. That's not good.
Other than this one issue, the preview version has been working great. My only issue is an apparent lack of a keyboard shortcut to start a new terminal window. The preview has otherwise been very useful. The terminal preview, along with WSL, has enabled my switch from Mac to Windows as I live in various unix/linux terminals most of the day with my job. Good job on it.

I finally solved the problem with losing active terminal sessions from an automatic update by disabling automatic updates for Windows Apps.   I just got the missing "Cascadia Mono" bug today also.

I finally solved the problem with losing active terminal sessions from an automatic update by disabling automatic updates for Windows Apps. I just got the missing "Cascadia Mono" bug today also.

Thanks @bikeoid , good idea. thuogh I'd prefer to just disable it for the terminal and not all apps. i'll look it up and see if the options are that granular.

Lest I only complain - I realize this is a preview release, so some hiccups are to be expected. I was caught off guard by the sudden closing and update without warning though. That's not good.

@softwrx I had the same issue and I'm running 1.8.1521.0 which, as far as I know, if not pre-release.   So I think the update process is borked.

Well, resetting the app, and unticking all fonts, chose cascadia code did it for me. You got limited selection ofc

Had to

Close Terminal
Install font from https://github.com/microsoft/cascadia-code/releases for all users
kill the explorer.exe process (from Task Manager)
start the explorer.exe process  (from Task Manager)
Start Terminal

And the error was gone.

I know the issue is closed, but i had it happen today for the very first time ever, couldnt find cascadia mono, on v1.8.1521.0
"repaired" the app from the windows 10 settings menu, restarted terminal and it worked

I just triggered the same issue, but sadly "repairing" the app did nothing for me. I'm using 1.8.1521.0.

I can also add that the font (Cascadia Mono) is installed and works fine in other applications.

Repairing the App did work for me

This is a "Me too" posts but adding that in Windows terminal "about" the version is "1.8.1521.0" but latest release is only "v1.8.1444.0". huh? Deployment issues? Time to repoen this issue?

Please refer to the body of the first post in this issue.


Change default button in review comment from "Add Comment" to "Start Review"

As the title states I'd really like to have the default action button in the comments dialog switched when adding a comment.
If you don't want to change this in a "hard coded" way, I'd also be fine with a new setting which allows me to adjust this.
Something like Default comment submit button when not in review: Add Comment | Start Review
Reason
I usually write the comment & submit with the keyboard without looking at the action buttons.
It already happened multiple times to me that I just left multiple "lose comments" whilst I actually wanted to add comments to a review since I simply forgot to actively use the Start Review button in the first comment.
Those lose comments all lead to single notifications for my co workers (the reviewees) which is rather unfortunate as I don't want to spam their inboxes.

I'm worried about changing the default for everyone, as the GitHub behavior around review comments can be confusing: the comment is "pending" until the review is submitted. This can result in users making comments and then not understanding why they are not visible to others. Because of this, the default for most people should still be "Add Comment".

A setting for changing the default perfectly fine! I'm really looking forward to this, thank you very much!


Limitations page says rack awareness not supported - but it now is

Minor edit required
https://github.com/aerospike/aerospike-kubernetes-operator/wiki/Limitations

Thanks @ken-tune for pointing it out. We will fix the doc.

Rack awareness is supported, and documentation updated on the wiki.


Polpatrol instrumentation

Modifications that expose additional information to the PolPatrol analysis tool.

What does it do?

Adds logging that tracks:

Calls to environment functions (tagged by category)
Wasm Memory consumption

What important points reviewers should know?

We think that as these are info-level logs they should not negatively influence performance.
Connected to this PR for polkadot: https://github.com/paritytech/polkadot/pull/700

As spoken in riot, they will look into the new tracing infrastructure of substrate to integrate their changes. :)

Will close it for now, feel free to reopen/recreate when you have something new.


Support for VTODO components

Hello!
I am trying to consume to do items from a CalDAV server but I am getting the following error:
unable to find matching component for VCALENDAR

I believe there is no support for VTODO components in the library, would it be possible to add it?

You’re right!
Since I’m developing an app that handles Todo, I will need that as well. It is on my free time though, so I won’t be able to dedicate a lot of time to it. But it’s clearly something of interest to me!
If you want to make a PR to get it faster, go for it! Otherwise, you will have to wait a little while.


I want Update Account but "Permission denied".

Provide loggined account's role and the request form you sent.

登入帳號是我自己建立的
<EMAIL_ADDRESS>pwd:asd

Provide loggined account's role and the request form you sent.

If you're updating your own account, there's no need to provide account_uuid; this parameter is for admins to update other accounts.

我關issue

登入帳號是我自己建立的<EMAIL_ADDRESS>pwd:asd

Accounts created through the backend API won't have admin privileges. To update their roles, you'll need to perform database operations or use an admin account to modify their account information.


Fixes and improvements to the MySQL database

Fixes:
Syntax error in name_data - missing semi-colon
Duplicate key errors on insert in name_data - this is due to data that varies only by case, resolved by making the name column case sensitive.
Improvement:
Set the character set and collation for the dms_sample database so that it uses utf8 and unicode rather than the latin1 swedish MySQL defaults.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Another change that can be wrapped into this PR is to correct "mysql/sampledb/v1/user/dms_user_privileges.sql" by removing the IDENTIFIED BY clause. It has been deprecated since MySQL 5.7


[docs] update zh_cn/faq.md

Hi @plutoyuxie ！First of all, we want to express our gratitude for your significant PR in the MMGen project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/UjgXkPWNqA
If you have WeChat account，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤


QUESTION: Future of Denizen

Since i am not around in times when mcmonkey or aufdemrand are there i wanted to ask, what the plans for denizen are now that bukkit is probably lost?
What will happen with denizen? Are you moving to Sponge? Is it realistic to move to Sponge?

@mcmonkey4eva Where is the core as of now in terms of being able to begin coding for other server brands? And now that we know the Spigot team is going to continue development, are we even still seriously considering this? What other brands that are already up and stable could we branch out to to possibly expand our community?
If we're unsure about any of these things, we should really get together and figure this stuff out. It's been months since the fall of Bukkit, and although it isn't completely crashed and burned we do have a system now to expand with. Why limit to Bukkit/Spigot if we're capable of more?

It's a fully functional core, it works fantastically. It still needs some upgrades (moving a few simple commands coreward) but for the most part it's ready to be implemented elsewhere.
and I still consider Sponge a good thing to build for, as it's quite popular even now.

Denizen-for-Voxalia!

That's called Frenetic

I lost track of the Sponge-development. Did they release a version you can work with now? Is Sponge still an option?

Sponge is still in heavy development, but we could theoretically start coding for it at any time.

It's pretty limited ATM, I don't want to bother with it yet. Maybe in 1.9 we'll start.
Until then, Spigot is best for now.

I'm going to close this since the Sponge Development was - in my opinion - way slower than expected and went a direction that I don't really like with Forge and I think I won't switch anytime soon.

I'd like this to be left open as information for anyone who wants to come reading about our plans.
Regardless of your personal interest in Sponge, we still intend to (Eventually) make a version for it.
I'd like to note that Sponge has a Vanilla version too, not just Forge, and ideally Denizen will work on both platforms.

Just a check to see if there's anything new re Sponge and Denizen ^_^

So for an update on this status.
We've analyzed Denizen's attack plan and there is a danger. Shall we have your shuttle on standby?
EHEM I mean, er, ...
Denizen 1 has some flaws weakening its beautiful core concepts.
As such, we are totally rewriting Denizen as Denizen2 on Sponge, specifically reoptimized for maximum speed/efficiency without in any way abandoning its ease of use.
Most of the changes will be core-side (IE, things you won't see while writing scripts), but since we're working with a different platform, the micro-details as well. By which I mean, some things like the names of tags may end up being different that they were on Bukkit.
AT THE SAME TIME, Denizen 1 (Bukkit edition) sails ever forward and will not be abandoned until we are 100% sure Bukkit/Spigot is no longer a viable platform.

Update:
Denizen2Sponge project is here https://github.com/DenizenScript/Denizen2Sponge and that readme links to the other components (Denizen2Core, etc).
If you want to run D2S right now, we highly recommend chatting with us on Discord: https://discord.gg/Q6pZGSR where we can help you get started - and you can help us shape development directions (building the D2 features you need as priority)
Also D1 on Bukkit is still really strong, especially with the most recent core upgrades.


[GitHub Desktop] Stashing changes feature

Anyone is welcome to open a PR!
Use this Desktop issue, Allow users to stash changes without switching branches, to understand the context of this change. Download the current beta version of Desktop to take any screenshots.
This feature allows users to right click on changes to stash them, instead of only having that option pop up when trying to change branches.
Audience: This is a feature that's really useful to new users, but they may not know about, so let's focus on new users for this change.
We use opinionated procedurals in our docs, so even though the issue above describes two separate ways to stash changes, we're only going to document one. For ease of use (and because it's the same for mac and windows users), we're only going to document the right click method.
We're going to create a new article titled, "Stashing changes"

[ ] Create a new file in the "Contributing and collaborating using GitHub Desktop" category
[ ] Name it stashing-changes.md
[ ] Copy the metadata from another article into your new file
[ ] Change the title in the metadata
[ ] Write an intro explaining in one sentence, why a user might want to stash changes.
[ ] Add the article to the index.md under contributing-and-collaborating-using-github-desktop, under "Committing and reviewing changes to your project"
[ ] Create a new conceptual section explaining why someone would want to stash changes
[ ] Create a new procedural explaining how to right click and stash changes.
[ ] Move the content from the "Switching between branches" and "Retrieving stashed changes" sections in the Managing branches, to the new article.

Hi, I'd like to work on this over the next few days. Thanks for opening the issue!

I would like to work on this issue . If no one is working over it!!!?

@poptire Are you still working on this issue?
@StarTrooper08 In the meantime, there are several more issues for GitHub Desktop updates that no one has commented on yet! https://github.com/github/docs/issues?q=is%3Aopen+is%3Aissue+label%3A"help+wanted"+Desktop

@janiceilene I have been out on assignment for the past week and unable to get as much work done on this as I would have liked. @StarTrooper08 might you be able to pick up my slack? Apologies, everyone.

Thanks for the update @poptire and definitely not a problem! there are always more issues to pick up if you're interested and less busy another time 💛

Ok I will head on with this issue .
As soon as I resolve the following issue.
I will update you with the same @janiceilene .
And also update you if I encounter with some error during resolving the issue.
Thank you🤗.

@poptire thanks for giving me opportunity . To work on this issue

@janiceilene what metadata should I add in stashing-changes.md file?


Environments Workgroup 1.2 Features and Code Muave Epic

These are features and code mauve wok planned in the Environments workgroup. Focusing on these items is our priority. Folks are free to submit work unrelated to these items. We can consider altering the Epic features over time.
(P0) Code Mauve:
stability testing for multi cluster and mesh expansion
Load balancing with locality (howardjohn)
More automated stability tests for whitebox - stability, performance, etc
Tests for environments WG related documentation
multicluster
mesh expansion: #13374
SRE-related documentation
metrics
alerts
selection of applications added to test cases
casandra
consul
kafka
nats
mysql
postgresql
redis
Will be added under modular installer, and be included by the test/stability.
MCP automated stability and install tests
Move the install/upgrade to KIND
Move bookinfo tests to KIND
Evaluate KIND for CNI tests.
Features:
(P?) Distroless images Issue #12122 (P?)
(P0) Decomposed Installation - repository: https://github.com/istio-ecosystem/istio-installer
(P1) Operator - repository:
(P1) Multicluster Gaps Issue #11731

zenhub spit out an error "Unknown error occured".  This is a duplicate so closing.


Fix Contact details

                 "persists please contact us at <TODO: "

Would also be nice if it were presented as a mailto: URI.

I added the matrix char URL

Worth closing then, right, @Murmele?

Not yet finished, I will close it when I merged my MR

@Murmele, you can link the issue to be closed when the MR is.


Add blog post announcing readonly feature

Add a blog post announcing the readonly feature from HHVM 4.139.0.
Not sure the best way to test this code as codespaces does not offer a port to forward. I ran all the examples locally to make sure they typecheck.

@fredemmott let me know if I did this right

Thanks, LGTM; how does aiming for tuesday sound? Historically we get readers on fridays and mondays.


Enable the build tag of rocksdb

This PR(https://github.com/line/tm-db/pull/41) ignored the build tag of rocksdb in lint/test once. Now, we can enable that again.

Codecov Report

Merging #42 (c520a23) into main (50290d8) will increase coverage by 14.83%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main      #42       +/-   ##
===========================================
+ Coverage   52.32%   67.16%   +14.83%
===========================================
  Files          25       30        +5
  Lines        2798     2494      -304
===========================================
+ Hits         1464     1675      +211
+ Misses       1230      754      -476
+ Partials      104       65       -39

Impacted Files
Coverage Δ

remotedb/proto/defs.pb.go

rdb_iterator.go
81.81% <0.00%> (ø)

rocksdb.go
68.78% <0.00%> (ø)

rdb.go
65.97% <0.00%> (ø)

rocksdb_iterator.go
100.00% <0.00%> (ø)

rocksdb_batch.go
70.49% <0.00%> (ø)

rdb_batch.go
68.18% <0.00%> (ø)

Impacted Files
Coverage Δ

remotedb/proto/defs.pb.go

rdb_iterator.go
81.81% <0.00%> (ø)

rocksdb.go
68.78% <0.00%> (ø)

rdb.go
65.97% <0.00%> (ø)

rocksdb_iterator.go
100.00% <0.00%> (ø)

rocksdb_batch.go
70.49% <0.00%> (ø)

rdb_batch.go
68.18% <0.00%> (ø)


Implement suggestions

Added suggestion implementation

fix tests
examples
readme

Please decline current PR. I'll prepare another one


chore(build): add support for multiarch build

Signed-off-by: shubham<EMAIL_ADDRESS>
added support for multiarch image with the following supported platforms (amd64, arm64, armv7, ppc64le)
refactored build script to add new targets to make use of docker buildx
added the following workflows into github actions

release
ci
build

@xUnholy  PTAL.


DOC: Improvements + added a Digital Clock to the project

Added a JavaScript File and and Html file to add a DIGITAL CLOCK to the project
Peace :)

Thanks! But there is no need to change the home page


add: hash passwords in the database

improved styling using eslint rules
refactored some weird parameter names
also changed the loginService to hash the incoming password before
comparison with the one stored in the database
some small overall improvements to the code
Closes #102

Test using the frontend - try creating a user via the UI and signing it in as well.

Please rebase with main - i need implemented logging in to test it

Please rebase with main - i need implemented logging in to test it

If that's the stopper for you, next time please merge the main into it on your own. Flag me only if there are conflicts. I try to keep the branches updated but without some kind of an automated process it's annoying to do it manually.


Network Controller Impl

For what purpose we need InboundClosedByPeer state? Why not set state to none?

For what purpose we need InboundClosedByPeer state? Why not set state to none?

Bc outbound stream is still alive, so the peer is still listening to us. Inbound stream can be re-opened.

pub struct ProtoNameWithUsize<T>(T, usize);
Please, name fields. It's easier to read with them
pub struct ProtoNameWithUsize<T>{ proto_name: T, proto_id: usize, }

Made it private. It's a detail of internal implementation.

pub struct ProtoNameWithUsize<T>(T, usize);
Please, name fields. It's easier to read with them
pub struct ProtoNameWithUsize<T>{ proto_name: T, proto_id: usize, }

Made it private. It's a detail of internal implementation.

internal implementation? It's just struct.
 self.0.remove(info.protocol_id).upgrade_inbound(sock, info.protocol_name); is better than
self.0.remove(info.1).upgrade_inbound(sock, info.0);

https://github.com/spectrum-finance/spectrum/blob/8db5a409d22e40b8fd4c138b3125cf6a744d136f/spectrum-network/src/peer_conn_handler.rs#L502
Opened -> None -> InboundClosedByPeer | Closed
Is it necessary to change state to None?

Why do you add "projection of states of enabled protocols" to network controller? You can use peer manager for this. And if so all the logic will be incapsulated only in pm

Why do you add "projection of states of enabled protocols" to network controller? You can use peer manager for this. And if so all the logic will be incapsulated only in pm

NC synchronizes ProtocolHandlers and protocols, while PM is only responsible for allocations.

https://github.com/spectrum-finance/spectrum/blob/8db5a409d22e40b8fd4c138b3125cf6a744d136f/spectrum-network/src/peer_conn_handler.rs#L502
Opened -> None -> InboundClosedByPeer | Closed
Is it necessary to change state to None?

Yes, we have to move data out of protocol.state when matching it (bc we consume substreams from it in some arms). But we can't leave protocol.state uninitialized at any point in time.

Why do you add "projection of states of enabled protocols" to network controller? You can use peer manager for this. And if so all the logic will be incapsulated only in pm

NC synchronizes ProtocolHandlers and protocols, while PM is only responsible for allocations.

If PM is "only responsible for allocations", why it can affect on connection by sending PeerManagerOut through poll_next?


x/build/cmd/pubsubhelper: use cloud pubsub for Gerrit instead of email

When I wrote pubsubhelper, Gerrit didn't support Cloud Pub Sub, so we had to parse emails to get live notifications out of Gerrit.
But @andybons pointed out that now it does, so we should move to it.
Low priority, but would be a nice cleanup.

For Google internal docs on how to set it up for our Gerrit instead, see http://go/golang-issue-28675-docs

Public docs: https://cloud.google.com/source-repositories/docs/quickstart-adding-pubsub-notifications

@andybons, IIRC, you said their pubsub topic today is still insufficient? It does't include all the NoteDB changes or something?

(For background for others, we currently run https://pubsubhelper.golang.org/ which is a fake email server that listens to Gerrit emails to know when things happen on Gerrit.)

CC @heschi.


Replace StringBuilder with String concatenation, as suggested by IntelliJ

Hi,
I replaced all usages of StringBuilder in child classes of Action by String concatenation.
This is a standard inspection by IntelliJ IDEA and simply makes the code shorter.
Therefore I simply applied the IntelliJ quick fix.
IntelliJ code inspection, cite:
"Reports any variables declared as or uses of java.lang.StringBuffer and java.lang.StringBuilder which can be replaced with a single java.lang.String concatenation. Using a String concatenation makes the code shorter and simpler. This inspection only reports when the resulting concatenation is at least as efficient or more efficient than the original StringBuffer or StringBuilder use."

Thanks for your contribution @nitram509 !


Add getters for Request

Simple PR. I was writing some tests and noticed that we don't have any getters in Request.

Keep those tests comin'!


waveform autoscale optional

!>> waveform autoscales the y-axis which makes it a little difficult to compare across amplitudes. maybe make that optional?

Our intention was that it is very easy to get sensible default plots. For full control, one can pass an axis object to any of the plotting functions and set limits, borders, spacing, etc. as desired:
_, axis = plt.subplots()
axis.set_ylim(-1, 1)
sig.waveform(axis=axis)

When comparing signals,  you'd probably make several axes in one figure and share dimensions. Passing pre-configured axis objects is more flexible than f.i. adding an argument for the y-axis limits.

aha^ would consider adding that to the docs if it wasn't already, but no objections from me since it's not a major point in the library & you have the bigger picture better than me!


Engagement unit test

fix #1133
implemented the following tests:

[x] Add
[x] Edit
[x] Close
[x] Delete
[x] Import scan results
[x] Add new CI/CD Engagement

When submitting a pull request, please make sure you have completed the following checklist:

[x] Your code is flake8 compliant (DefectDojo's code isn't currently flake8 compliant, but we're trying to correct that.)
[ ] If this is a new feature and not a bug fix, you've included the proper documentation in the ReadTheDocs documentation folder. https://github.com/DefectDojo/Documentation/tree/master/docs or provide feature documentation in the PR.
[ ] Model changes should include the necessary migrations in the dojo/dd_migrations folder.
[ ] Add applicable tests to the unit tests.

@Maffooch done!

@dr3dd589 Don't forget the delete product as well.

ohh! sorry about that. :)


[WIP] Maker fill metadata

Add the taker's OOA public key to the maker fill event (based on a new AccountFlag)
This makes it easier to do accounting for downstream protocols integrating with Serum that need to perform actions in response to trade events being processed

I think this was accidentally closed automatically because it built off the crank-auth branch, which was deleted.


Tests and Python Dependencies

Presently, the tests seem to be broken. The CI seems to be selectively testing only some parts, and hence seems to pass. Meanwhile, running ] test shows that tests don't pass. Is there any reason behind this? I can add a workflow that tests everything if required.
One of the reasons the tests seems to be broken is because -
NiPyC2 Tests: Error During Test at /home/runner/work/Xtals.jl/Xtals.jl/test/bonds.jl:139
  Got exception outside of a @test
  Python dependency scipy not loaded.

I'm a little confused here, I thought Xtals was a pure-Julia package (as mentioned in the README). Then why is not having a Python dependency breaking tests?

The CI runner builds the python deps first.  The CI is only selective, so far as I'm aware, when testing if Xtals has a breaking change for a couple of key projects within our group, PorousMaterials and PoreMatMod.  Failing that test doesn't mean we should block a merge, but we want a warning that those packages will need patching.
scipy enables the function infer_geometry_based_bonds!.  That code predates me on this project, so @SimonEnsemble should weigh in here. There is also the pymatgen-dependent primitive_cell.
Open to ideas on how to fix this up.  A few of my own to get started:

Split python-dependent code into a separate package.
Replace the python deps with Julia ones.
Change runtests.jl to skip python-dependent code by default.

To clarify: those python-dependent functions are not key elements of Xtals.  The "standard" bonding function infer_bonds! is not dependent on python, and primitive_cell is a convenient but not fundamental feature of the package.

Going with option 1 until option 2 can be done.  XtalsPyTools will contain python-dependent tools/interfaces for working with Xtals.