Missing translations fallback

We need graceful translation fallback when the translation is missing. Right now the UI shows the ID/key of the string. What's the least useful is to fallback to the English string instead.
Edit: The most prominent examples are the eta_* strings.

One way of doing it is to:

INSERT the English locale in the empty table;
UPDATE the locale field with the target locale value (say en-GB);
UPDATE the available in the target locale value fields matched by key;
repeat for all locales;
INSERT the English locale last.

I think the easiest way to default to 'en-US' is to just set it in code - I think I did something like that, and when I look in the code I see this; https://github.com/picotorrent/picotorrent/blob/master/src/picotorrent/ui/translator.cpp#L116. So, I made a bad assumption but the fix is easy 😃

Isn't line 119 the right place?

No, that is what should happen if we don't find the English translation using the default key - which is misspelled as 'en' when it should be 'en-US' 🙄

I tried your suggestion in #1051 but is not what it should be. Since I can't wrap my head around the code I'm leaving it to you for now.


Test failures resulting from env.py not being imported with the app_context()

First off - love the project :)
I believe this is an issue with using pytest-alembic out-of-the-box with Flask-SQLAlchemy / Flask-Migrate project.
I encountered a test_upgrade error ending in:
E               pytest_alembic.plugin.error.AlembicTestFailure: Failed to upgrade through each revision individually.

The relevant parts of the stacktrace above it seem to be:
    from __future__ import with_statement

    import logging
    from logging.config import fileConfig

    from flask import current_app

    from alembic import context
    from curation.workspace import INPUT_DATA_TABLE_PREFIX

    # this is the Alembic Config object, which provides
    # access to the values within the .ini file in use.
    config = context.config

    # Interpret the config file for Python logging.
    # This line sets up loggers basically.
    fileConfig(config.config_file_name)
    logger = logging.getLogger("alembic.env")

    # add your model's MetaData object here
    # for 'autogenerate' support
    # from myapp import mymodel
    # target_metadata = mymodel.Base.metadata
    config.set_main_option(
        "sqlalchemy.url",
>       str(current_app.extensions["migrate"].db.get_engine().url).replace("%", "%%"),
    )

migrations/env.py:26:

and below that:
    def _find_app():
        top = _app_ctx_stack.top
        if top is None:
>           raise RuntimeError(_app_ctx_err_msg)
E           RuntimeError: Working outside of application context.
E
E           This typically means that you attempted to use functionality that needed
E           to interface with the current application object in some way. To solve
E           this, set up an application context with app.app_context().  See the
E           documentation for more information.

venv/lib/python3.9/site-packages/flask/globals.py:47: RuntimeError

It looks like my migrations/env.py file needs to be imported with an app_context(). Providing this context myself like below got me past the issue:
from pytest_alembic import tests

def test_upgrade(app, alembic_runner):
    with app.app_context():
        tests.test_upgrade(alembic_runner)

For reference, I'm using Flask-Migrate, and I had to override the alembic_config fixture to successfully find my alembic.ini:
@pytest.fixture
def alembic_config():
    return {"file": "migrations/alembic.ini"}

I'll also note that on this project using pytest --test-alembic always matched all of my tests, and none of the built-in tests (had the same effect as simply pytest. I had to import from pytest_alembic.tests right from the get-go to get anything running.
Thanks!

    config.set_main_option(
        "sqlalchemy.url",
>       str(current_app.extensions["migrate"].db.get_engine().url).replace("%", "%%"),
    )

For this, I would recommend following the suggestion here: https://pytest-alembic.readthedocs.io/en/latest/setup.html#env-py.
Specifically this makes the default alembic_runner fixture used by the default test impls use the engine generated by your tests, rather than the ones your default env.py might naturally pick up.
This may or may not solve your problem automatically, since it will not be using that set_main_option option, and therefore wont be using current_app. Happy (imo) side-effect is you'd not be using the real-world env.py settings and risk actually running these migrations against a live database.
If you're using flask-sqlalchemy or something else which im pretty sure forces you to still have a live app context in these sort of situations, you could then probably do something like suggested lower in that page
@pytest.fixture
def alembic_engine(app):
    with app.app_context():
        return ...however you get your engine...

I'll also note that on this project using pytest --test-alembic always matched all of my tests, and none of the built-in tests (had the same effect as simply pytest. I had to import from pytest_alembic.tests right from the get-go to get anything running.

 The --test-alembic flag only causes the default tests to be automatically included (by inserting them at [pytest_alembic_tests_folder](https://pytest-alembic.readthedocs.io/en/latest/running.html#config (default tests/).
So the default assumption is that your tests live at tests/, and you'd run tests like pytest --test-alembic tests (or configure your pytest default folders). In your particular case currently, where you're redeclaring your own test_upgrade, it wont have any effect (with that being said, my above suggestion would mean you shouldn't need to define your own versions of the tests in the first place.
If you want to specifically run only the alembic tests, you can use the alembic mark. So pytest -m alembic if you have your own custom tests, or pytest --test-alembic -m alembic tests if you use the default tests (to include them, then select only them).

Let me know if any of this ends up working out, I can add a specific note about compatibility with flask-migrate

Assuming everything worked out, at best there's some documentation improvements to extract out of this, but I'm not actually sure what you landed on or what would be the most useful information to include.
So i'm gonna close this issue out for now, but (anyone!) let me know otherwise and I can re-open

Hi! Sorry for the long delay in getting back to you.
run_migrations_online() update
Per your first suggestion to follow the docs and update run_migrations_online(), for reference here's what gets placed in migrations/env.py by default by Flask-Migrate's init command, which is a bit different than what non-Flask-Migrate Alembic usage seems to put in:
def run_migrations_online():
    """Run migrations in 'online' mode.

    In this scenario we need to create an Engine
    and associate a connection with the context.

    """

    # this callback is used to prevent an auto-migration from being generated
    # when there are no changes to the schema
    # reference: http://alembic.zzzcomputing.com/en/latest/cookbook.html
    def process_revision_directives(context, revision, directives):
        if getattr(config.cmd_opts, 'autogenerate', False):
            script = directives[0]
            if script.upgrade_ops.is_empty():
                directives[:] = []
                logger.info('No changes in schema detected.')

    connectable = current_app.extensions['migrate'].db.get_engine()

    with connectable.connect() as connection:
        context.configure(
            connection=connection,
            target_metadata=target_metadata,
            process_revision_directives=process_revision_directives,
            **current_app.extensions['migrate'].configure_args
        )

        with context.begin_transaction():
            context.run_migrations()

I believe this means that alembic is already set up by Flask-Migrate to get the database engine from the flask app's configuration, which (if run in an app_context()) should yield the correct engine in both prod and testing.
alembic_engine fixture
I tried your suggestion for this, and it worked! I added this fixture to conftest:
@pytest.fixture
def alembic_engine(app):
    with app.app_context():
        yield db.get_engine()

where db is the SQLAlchemy object that I run db.init_app(app) in my create_app() factory. Now, I don't need to wrap my custom pytest test invocations in an app_context(), so, eg., this just works:
def test_single_head_revision(alembic_runner):
    tests.test_single_head_revision(alembic_runner)

test invocation
Your suggestions about the test invocations also worked! I think I just had an incorrect understanding of --test-alembic, like I thought it would run those tests exclusively, and since in my previous configuration I had to wrap the test runs in app_context() anyway I stopped investigating it. So on top of the alembic_engine() fixture I can now remove all of those custom test invocations entirely.
So it looks like you're right, there's likely no code changes here. At best we could consider a new docs page specifically targeting people using Flask-SQLAlchemy and Flask-Migrate, since I'd imagine that's not an uncommon vector. I know I was confused and a little discouraged looking at the run_migrations_online() part in the docs and not seeing it line up with my own env.py.


Flexbile control of the regions of interest to be sampled

Sampling can be implemented in both "sides":

Tracer: tracer decides if a particular execution of the region of interest needs to be sampled.

Pros: Less overhead if we decide to not sample.
Cons: Less information. Tracer does not have information about previous sampled regions. As a result, only random or simple (e.g. first 10) sampling methods can be implemented (no intelligent sampling)

Tracee: tracee decides if a particular execution of the regions of interest needs to be sampled.

Cons: Some overhead if we decide to not sample.
Pros: tracee can use existing samples (e.g. historical data) to decide or not to sample a particular region.

I do think that having both would be beneficial. One one side, we can use the Tracer sampling to do a first level pruning of the executions (e.g. from 1 million to 1 thousand) and then use the Tracee extra information available to do a second and more intelligent filtering.

With the new infrastructure, this "on-the-fly" tracing approach can be implemented to avoid the full profiling of an entire application.


[TF Lite] Add support for dilated convolution with valid padding.

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

uname -a
Linux archlinux 4.19.2-arch1-1-ARCH #1 SMP PREEMPT Tue Nov 13 21:16:19 UTC 2018 x86_64 GNU/Linux

TensorFlow installed from (source or binary):

pip install tensorflow

TensorFlow version (or github SHA if from source):

1.12.0

Provide the text output from tflite_convert
RuntimeError: TOCO failed see console for info.
2019-01-22 11:50:43.257264: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before Removing unused ops: 14 operators, 27 arrays (0 quantized)
2019-01-22 11:50:43.257374: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before general graph transformations: 14 operators, 27 arrays (0 quantized)
2019-01-22 11:50:43.257409: I tensorflow/contrib/lite/toco/graph_transformations/identify_dilated_conv.cc:217] Replaced sub-network with Dilated Conv2D op outputting "conv2d/Conv2D".
2019-01-22 11:50:43.257458: F tensorflow/contrib/lite/toco/graph_transformations/propagate_fixed_sizes.cc:991] Check failed: height_with_paddings % block_height == 0 (12 vs. 0)

Also, please include a link to a GraphDef or the model if possible: keras_model.zip
Any other info / logs
Running this in jupyter notebook:
import numpy as np
import tensorflow as tf
tf.keras.backend.clear_session()
print(tf.__version__)
print(tf.keras.__version__)
keras_model_savepath = '/tmp/keras_model.h5'

x_input = x = tf.keras.layers.Input(shape=(725, 725, 1))
x = tf.keras.layers.Conv2D(6, 3, padding='valid', dilation_rate=25)(x)
x = tf.keras.layers.Conv2D(6, 3, padding='valid', dilation_rate=19)(x)
keras_model = tf.keras.models.Model(x_input, x)
print(keras_model.summary())
pred = keras_model.predict(np.ones([1,725,725,1]))
print(pred[0,0:5,0,0])
keras_model.save(keras_model_savepath)
toco = tf.contrib.lite.TFLiteConverter.from_keras_model_file(keras_model_savepath)
model_tflite = toco.convert()

gives:
1.12.0
2.1.6-tf
_________________________________________________________________
Layer (type)                 Output Shape              Param #
=================================================================
input_1 (InputLayer)         (None, 725, 725, 1)       0
_________________________________________________________________
conv2d (Conv2D)              (None, 675, 675, 6)       60
_________________________________________________________________
conv2d_1 (Conv2D)            (None, 637, 637, 6)       330
=================================================================
Total params: 390
Trainable params: 390
Non-trainable params: 0
_________________________________________________________________
None
[-1.9924886 -1.9924886 -1.9924886 -1.9924886 -1.9924886]
WARNING:tensorflow:No training configuration found in save file: the model was *not* compiled. Compile it manually.
INFO:tensorflow:Froze 4 variables.
INFO:tensorflow:Converted 4 variables to const ops.

---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
<ipython-input-48-0a572848505b> in <module>
     15 keras_model.save(keras_model_savepath)
     16 toco = tf.contrib.lite.TFLiteConverter.from_keras_model_file(keras_model_savepath)
---> 17 model_tflite = toco.convert()

~/.pyenv/versions/3.6.7/envs/tensorflow-1.12/lib/python3.6/site-packages/tensorflow/contrib/lite/python/lite.py in convert(self)
    451           input_tensors=self._input_tensors,
    452           output_tensors=self._output_tensors,
--> 453           **converter_kwargs)
    454     else:
    455       # Graphs without valid tensors cannot be loaded into tf.Session since they

~/.pyenv/versions/3.6.7/envs/tensorflow-1.12/lib/python3.6/site-packages/tensorflow/contrib/lite/python/convert.py in toco_convert_impl(input_data, input_tensors, output_tensors, *args, **kwargs)
    340   data = toco_convert_protos(model_flags.SerializeToString(),
    341                              toco_flags.SerializeToString(),
--> 342                              input_data.SerializeToString())
    343   return data
    344

~/.pyenv/versions/3.6.7/envs/tensorflow-1.12/lib/python3.6/site-packages/tensorflow/contrib/lite/python/convert.py in toco_convert_protos(model_flags_str, toco_flags_str, input_data_str)
    133     else:
    134       raise RuntimeError("TOCO failed see console for info.\n%s\n%s\n" %
--> 135                          (stdout, stderr))
    136
    137

RuntimeError: TOCO failed see console for info.
b'2019-01-22 11:50:43.257264: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before Removing unused ops: 14 operators, 27 arrays (0 quantized)\n2019-01-22 11:50:43.257374: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before general graph transformations: 14 operators, 27 arrays (0 quantized)\n2019-01-22 11:50:43.257409: I tensorflow/contrib/lite/toco/graph_transformations/identify_dilated_conv.cc:217] Replaced sub-network with Dilated Conv2D op outputting "conv2d/Conv2D".\n2019-01-22 11:50:43.257458: F tensorflow/contrib/lite/toco/graph_transformations/propagate_fixed_sizes.cc:991] Check failed: height_with_paddings % block_height == 0 (12 vs. 0)\n'
None

Any updates?

This should be supported for float models now, can you give it another try?

@ltrottier,
With respect to this comment, this issue should be resolved. Can you please confirm it so that we can close this issue? Thanks!

I can confirm that this issue is resolved.
However, I had to use
toco = tf.lite.TFLiteConverter.from_keras_model(keras_model)

instead of
keras_model.save(keras_model_savepath)
toco = tf.lite.TFLiteConverter.from_keras_model_file(keras_model_savepath)

because of a change in the interface of TFLiteConverter:
AttributeError: type object 'TFLiteConverterV2' has no attribute 'from_keras_model_file'

Thank you !


Add possibility to make GIFs

Some kind of functionality where you could give a deterministic filename-scheme and make GIFs out of them would be cool!

Done. ex: c3pp plot res_030_c0001_k00000*.fits -gif -auto -fwhm 120
Will add more functionality later.


cmake Failed to Build On Fedora 22

From @Blue-Dog-Archolite on August 28, 2015 19:16
Fail to install NeoVim on Fedora 22.
1.9.3-p551::(master) $ brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry and just ignore them. Thanks!

Warning: "config" scripts exist outside your system or Homebrew directories.
`./configure` scripts often look for *-config scripts to determine if
software packages are installed, and what additional flags to use when
compiling and linking.

Having additional scripts in your path can confuse software installed via
Homebrew if the config script overrides a system or Homebrew provided
script of the same name. We found the following "config" scripts:

  /bin/libgcrypt-config
  /bin/krb5-config
  /bin/python-config
  /bin/python2-config
  /bin/xslt-config
  /bin/ncurses5-config
  /bin/curl-config
  /bin/ncursesw5-config
  /bin/python2.7-config
  /bin/x86_64-redhat-linux-gnu-pkg-config
  /bin/brltty-config
  /bin/pkg-config
  /bin/gpg-error-config
  /bin/xml2-config
  /bin/pcre-config

Warning: Some directories in your path end in a slash.
Directories in your path should not end in a slash. This can break other
doctor checks. The following directories should be edited:
    /home/thief/tools/BashProfile/git_flow_utility/

==> Installing neovim from neovim/homebrew-neovim
==> Installing dependencies for neovim/neovim/neovim: cmake, xz, libtool, homebrew/dupes/m4, autoconf, automake, gettext
==> Installing neovim/neovim/neovim dependency: cmake
==> Downloading http://www.cmake.org/files/v3.3/cmake-3.3.1.tar.gz
Already downloaded: /home/thief/.cache/Homebrew/cmake-3.3.1.tar.gz
==> Downloading https://pypi.python.org/packages/source/s/sphinx_rtd_theme/sphinx_rtd_theme-0.1.8.tar.gz
Already downloaded: /home/thief/.cache/Homebrew/cmake--sphinx_rtd_theme-0.1.8.tar.gz
==> python -c import setuptools... --no-user-cfg install --prefix=/tmp/cmake20150828-4146-bty7wj/cmake-3.3.1/sphinx --single-version-externally-managed --record=installed.txt
==> Downloading https://pypi.python.org/packages/source/s/snowballstemmer/snowballstemmer-1.2.0.tar.gz
Already downloaded: /home/thief/.cache/Homebrew/cmake--snowballstemmer-1.2.0.tar.gz
==> python -c import setuptools... --no-user-cfg install --prefix=/tmp/cmake20150828-4146-bty7wj/cmake-3.3.1/sphinx --single-version-externally-managed --record=installed.txt
==> Downloading https://pypi.python.org/packages/source/s/six/six-1.9.0.tar.gz
Already downloaded: /home/thief/.cache/Homebrew/cmake--six-1.9.0.tar.gz
==> python -c import setuptools... --no-user-cfg install --prefix=/tmp/cmake20150828-4146-bty7wj/cmake-3.3.1/sphinx --single-version-externally-managed --record=installed.txt
==> Downloading https://pypi.python.org/packages/source/P/Pygments/Pygments-2.0.2.tar.gz
Already downloaded: /home/thief/.cache/Homebrew/cmake--pygments-2.0.2.tar.gz
==> python -c import setuptools... --no-user-cfg install --prefix=/tmp/cmake20150828-4146-bty7wj/cmake-3.3.1/sphinx --single-version-externally-managed --record=installed.txt
==> Downloading https://pypi.python.org/packages/source/d/docutils/docutils-0.12.tar.gz
Already downloaded: /home/thief/.cache/Homebrew/cmake--docutils-0.12.tar.gz
==> python -c import setuptools... --no-user-cfg install --prefix=/tmp/cmake20150828-4146-bty7wj/cmake-3.3.1/sphinx --single-version-externally-managed --record=installed.txt
==> Downloading https://pypi.python.org/packages/source/p/pytz/pytz-2015.4.tar.bz2
Already downloaded: /home/thief/.cache/Homebrew/cmake--pytz-2015.4.tar.bz2
==> python -c import setuptools... --no-user-cfg install --prefix=/tmp/cmake20150828-4146-bty7wj/cmake-3.3.1/sphinx --single-version-externally-managed --record=installed.txt
==> Downloading https://pypi.python.org/packages/source/B/Babel/Babel-2.0.tar.gz
Already downloaded: /home/thief/.cache/Homebrew/cmake--babel-2.0.tar.gz
==> python -c import setuptools... --no-user-cfg install --prefix=/tmp/cmake20150828-4146-bty7wj/cmake-3.3.1/sphinx --single-version-externally-managed --record=installed.txt
==> Downloading https://pypi.python.org/packages/source/M/MarkupSafe/MarkupSafe-0.23.tar.gz
Already downloaded: /home/thief/.cache/Homebrew/cmake--markupsafe-0.23.tar.gz
==> python -c import setuptools... --no-user-cfg install --prefix=/tmp/cmake20150828-4146-bty7wj/cmake-3.3.1/sphinx --single-version-externally-managed --record=installed.txt
==> Downloading https://pypi.python.org/packages/source/J/Jinja2/Jinja2-2.8.tar.gz
Already downloaded: /home/thief/.cache/Homebrew/cmake--jinja2-2.8.tar.gz
==> python -c import setuptools... --no-user-cfg install --prefix=/tmp/cmake20150828-4146-bty7wj/cmake-3.3.1/sphinx --single-version-externally-managed --record=installed.txt
==> Downloading https://pypi.python.org/packages/source/a/alabaster/alabaster-0.7.6.tar.gz
Already downloaded: /home/thief/.cache/Homebrew/cmake--alabaster-0.7.6.tar.gz
==> python -c import setuptools... --no-user-cfg install --prefix=/tmp/cmake20150828-4146-bty7wj/cmake-3.3.1/sphinx --single-version-externally-managed --record=installed.txt
==> Downloading https://pypi.python.org/packages/source/S/Sphinx/Sphinx-1.3.1.tar.gz
Already downloaded: /home/thief/.cache/Homebrew/cmake--sphinx-1.3.1.tar.gz
==> python -c import setuptools... --no-user-cfg install --prefix=/tmp/cmake20150828-4146-bty7wj/cmake-3.3.1/sphinx --single-version-externally-managed --record=installed.txt
==> ./bootstrap --prefix=/home/thief/.linuxbrew/Cellar/cmake/3.3.1 --no-system-libs --parallel=4 --datadir=/share/cmake --docdir=/share/doc/cmake --mandir=/share/man --system-
==> make
make[2]: *** [Utilities/Sphinx/doc_format_man] Error 1
CMakeFiles/Makefile2:2119: recipe for target 'Utilities/Sphinx/CMakeFiles/documentation.dir/all' failed
make[1]: *** [Utilities/Sphinx/CMakeFiles/documentation.dir/all] Error 2
Makefile:160: recipe for target 'all' failed
make: *** [all] Error 2

READ THIS: https://github.com/Homebrew/linuxbrew/blob/master/share/doc/homebrew/Troubleshooting.md#troubleshooting

These open issues may also help:
cmake builds fail on CLT-only with --env=std https://github.com/Homebrew/homebrew/issues/29101

Copied from original issue: Homebrew/homebrew#43367

You should be able to work around this by running brew install cmake --without-docs first.

Great Call. Odd that the dependency resolution can't sort this out.

It's unfortunately not a dependency resolution issue; Red Hat flavors ship a patched Python that interferes with Linuxbrew's Python handling. --without-docs disables the parts of the cmake build that require python.


[C++][CMake] handle Protobuf_PROTOC_EXECUTABLE correctly in cross-compilation

Describe the bug, including details regarding any error messages, version, and platform.
In the preparation for releasing arrow v16 in conda-forge (where we need to cross-compile for various targets), @raulcd ended up raising https://github.com/apache/arrow/issues/41333 which got fixed in https://github.com/apache/arrow/pull/41360. However, as I was working on the backports, I noticed that there's a hunk missing here in apache/arrow compared to what got committed in https://github.com/conda-forge/arrow-cpp-feedstock/commit/b80a399d39e04b09980d12ee3817b477b175de11, namely
diff --git a/cpp/cmake_modules/FindProtobufAlt.cmake b/cpp/cmake_modules/FindProtobufAlt.cmake
index f343b42f2..98a3281ee 100644
--- a/cpp/cmake_modules/FindProtobufAlt.cmake
+++ b/cpp/cmake_modules/FindProtobufAlt.cmake
@@ -31,6 +31,12 @@ endif()
 find_package(protobuf CONFIG ${find_package_args})
 set(ProtobufAlt_FOUND ${protobuf_FOUND})
 if(ProtobufAlt_FOUND)
-  if(Protobuf_PROTOC_EXECUTABLE)
-    set_target_properties(protobuf::protoc PROPERTIES
-    IMPORTED_LOCATION_RELEASE "${Protobuf_PROTOC_EXECUTABLE}"
-  )
-  endif()
   set(ProtobufAlt_VERSION ${protobuf_VERSION})
   set(ProtobufAlt_VERSION_MAJOR ${protobuf_VERSION_MAJOR})
   set(ProtobufAlt_VERSION_MINOR ${protobuf_VERSION_MINOR})

Component(s)
C++, Packaging

Issue resolved by pull request 41582
https://github.com/apache/arrow/pull/41582


Help! error when tweet with media

hello there, what's wrong with this code always got error, I wanna try tweet with media but there's appear error, help me please thanks in advance

fixed thanks!


[Feature] Choose different services than default

Hey,
Thanks for the awesome plugin!
It would be super cool if services would be able to change in the plugin config. in my case, using occupancy sensors instead of motion sensor does not represent well my home so I would love to change occupancy to motion sensors in HomeKit.
I also thought it would be nice to have it dynamically available for all sorts of properties and services.
for example, possible changes users would might like to use:
occupancy -> contact sensor
switch -> lock (I connected a normal switch that control a dry relay of a gate lock for example)
switch -> valve
and i'm sure there are many other changes users would want to make, mostly with boolean state like true/false or on/off
Thanks in advanced

Thanks for your feature request.
Would you mind to make a proposal for what the plugin configuration should look like for this feature in your opinion?

Thanks for your feature request.
Would you mind to make a proposal for what the plugin configuration should look like for this feature in your opinion?

I would do that all the binary accessories, divided to 2 groups, would be able to change the accessory type within their group:

Binary sensors that only report status but can't set/change state: motion, occupancy, button, contact, smoke, leak, and maybe more that I'm missing.

Binary accessories that can be controlled and set/change state: switch, lock, outlet, (maybe valve?).

Since all those devices report 0 or 1 we can change the type of accessory represented in the home app without a problem, only need to map the HAP accessories states and zigbee accessories state to 0 and 1.
Hope I made it clear

But what would the configuration in the JSON file look like?
Also consider that a single device can have multiple binary characteristics.

Hey, I am sorry for intervening. I used homebridge-mqttthing until yesterday. It worked fine, but I wanted to change it because I could not get a status update when I was manually pressed a plug button. The thing I like so much about it is that you can configure everything for a device.
Here is an example of configuring a Silvercrest Plug:

{
"type": "outlet",
"name": "Glob",
"logMqtt": true,
"topics": {
"getOn": {
"topic": "zigbee2mqtt/0xec1bbdfffeb6120d/get/",
"apply": "return JSON.stringify({state: null})"
},
"setOn": {
"topic": "zigbee2mqtt/0xec1bbdfffeb6120d/set/",
"apply": "return JSON.stringify({state: message})"
},
"getInUse": "zigbee2mqtt/0xec1bbdfffeb6120d/get/state"
},
"onValue": "ON",
"offValue": "OFF",
"onlineValue": "ON",
"offlineValue": "OFF",
"retryLimit": 3,
"history": false,
"turnOffAfterms": 0,
"manufacturer": "Tecnodidattica Ligure S.p.A.",
"serialNumber": "Andr3wSt3f",
"model": "25W, E14",
"firmwareRevision": "1.0",
"accessory": "mqttthing"
},

@AndrewStef7 The goal of this plugin is to have it working "out of the box" for most of the users, without any device configuration. The thing is that the example configuration can't be taken over "as is" within how this plugin is configured.
Personally I don't have time at the moment to work out what the plugin configuration should look like for this feature, so that's why I'm asking for others to contribute. 😄

Since it's already in the plugin settings to change settings for specific device I would just add another field named: accessoryType.
I would allow changing types between the groups I suggested, in any case you have more services you can add them to the newly created one, if they are binary too, and identical to the main service they should change too to the new type. If someone use a type which is not allow it can fallback to the original and show a message in the logs.
Hope it is clear

@nitaybz But that doesn't work. As mentioned, a single accessory can expose multiple services/characteristics (i.e. the Oujiabao CR701-YZ exposes both a binary sensor for Carbon Monoxide and for gas leaks).

I would just exclude those with specific destination like carbon and maybe leaks and smoke...
You can start with simple:
occupancy-contact-motion
switch-outlet-lock
And add more in the future if users request it specifically

But if we don't take that scenario into account now with how the configuration is setup, then it's likely that in the future there will be a breaking change when someone requests it for a different type of device.
By the way, you can't simply switch from a Switch service to a Lock Mechanism. A lock isn't just a single binary state on Zigbee nor on HomeKit.
Apparently the Outlet service no longer requires the Outlet In Use characteristic, so that would be feasible.

any progress on this matter?
can we at least make something as small as occupancy to motion sensor?

any progress on this matter?

No progress on it from my side.
It's not really on the top of my list (as there are a few feature request that more people have asked for and are also on my personal "wishlist").

can we at least make something as small as occupancy to motion sensor?

It can probably be done, but I want to have a uniform way of configuring this.
Perhaps it should be set based on the name of one of the MQTT properties (I haven't given this a lot of thought or gone over what this would mean for the code base, but it came to mind while writing this reply).

As it is long ago that someone wrote to this topic, I want to say that it would be really great to define an outlet as outlet in homekit, as it makes a difference in how to address it with Siri, as well as the really annoying look, that an outlet is displayed as a switch.
This would be really great.


Composer v2 compatibility

Resolution of #106 and #105
As on composer 2 installed.json comes with an new index [packages] instead of root
This code would not iterate between packages
foreach ($installed as $package) {

by adding this line before would make this composer 2 compatible and keep backward also:
$installed = is_array($installed['packages']) ? $installed['packages'] : $installed;

Resolution of #106 and #105
As on composer 2 installed.json comes with an new index [packages] instead of root
This code would not iterate between packages
foreach ($installed as $package) {

by adding this line before would make this composer 2 compatible and keep backward also:
$installed = is_array($installed['packages']) ? $installed['packages'] : $installed;

maybe this would be also a good aproach
$installed = empty($installed['packages']) ? $installed : $installed['packages'];

Looks like #107 was faster. Probably you both should merge your PRs?

Looks like #107 was faster. Probably you both should merge your PRs?

Yup only saw it after have done the pull request.
From what i see @thojou looks more complete solution because he did the tests.
Personally thinking, the real solution is one line code don't need to do another call to check if is v2 if that new method only has one line of code, but that's a point of view.
Just hoping that one or another solution gets in production because phar-composer is not working in composer v2

@nargotik Thanks for looking into this! I've just fixed this via #114. :shipit:


Added "I'm Batman" sound effects

I added a BATMAN SoundCollection, with "!batman" & "!imbatman" commands and 2 different audio files.

See #28

My bad, didn't see the other pull requests. I'll definitely try installing my modded version on a server separately, though.


[Scaler] Azure Event Hubs

For the type would be good to have both azure-eventhub or azure-iothub as alias as people may be approaching this from either #214

That means that we should also have an alias for metadata.eventHubName to avoid confusion.
Happy to review configurations in case there is a need.

Starting work on Monday, doing some validation around the capabilies of the Azure Go SDK

Working on validation/bug fixes.

Closed via PR: https://github.com/kedacore/keda/pull/276


#60 Add widgetName to capture downloads

Depends on https://github.com/MetaCell/geppetto-meta/pull/197 (please merge this first)

Updates download name of recording and screenshot to use widgetName

@afonsobspinto sorry to bother, I noticed the other day that the download and the preview are not working in staging, is it due to this PR that is missing from develop right now?
@zsinnema if that is the case could you review this and merge it please? Thx


industry field anomalies

https://github.com/JerBouma/FinanceDatabase/raw/master/Database/Equities/Countries/United States/United States.json
"industry" field contains the following anomaly:

"Banks\u2014Diversified",
"Banks\u2014Regional",
"Beverages\u2014Brewers",
"Beverages\u2014Non-Alcoholic",
"Beverages\u2014Wineries & Distilleries",

"Drug Manufacturers\u2014General",
"Drug Manufacturers\u2014Specialty & Generic",
"Insurance\u2014Diversified",
"Insurance\u2014Life",
"Insurance\u2014Property & Casualty",
"Insurance\u2014Reinsurance",
"Insurance\u2014Specialty",
"Real Estate\u2014Development",
"Real Estate\u2014Diversified",

these could use some normalization
"Aerospace & Defense",
"Aerospace/Defense - Major Diversified",
"Aerospace/Defense Products & Services",
It's a lot faster to work on offline database, cheers!

Ah, I fixed this within my search_products function. I will look into adjusting the JSON files as well!

Fixed the issue. The problem was that  \u2014\ referred to a special form of the dash ( - ) sign. This makes it very difficult to select that type of data. See the fix (for example): [Banks - Diversified]((https://github.com/JerBouma/FinanceDatabase/blob/master/Database/Equities/Industries/Banks - Diversified.json).
I also updated strings like Aerospace/Defense Products & Services to Aerospace Defense Products & Services because the forward slash usually gives problems with filenames.
Please let me know if you find more weird unicodes!


add composer.lock & update travis config

as per zendframework/zendframework#7660
also entails these changes:

make composer scripts entries consistent with the other components
fix travis.yml env variables to not be affected by travis-ci/travis-ci#1444

Thanks, @stefanotorresi. I cherry-picked to master.


lethe: 0.6.0 -> 0.6.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/kostassoid/lethe/releases.
meta.description for lethe is: "Tool to wipe drives in a secure way"
meta.homepage for lethe is: "https://github.com/kostassoid/lethe"
meta.changelog for lethe is: ""
Updates performed

Rust version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/lr4pyar97q66ac6vban3rljrwfsb5n10-lethe-0.6.1/bin/lethe passed the binary check.
1 of 1 passed binary check by having a zero exit code.
1 of 1 passed binary check by having the new version present in output.
found 0.6.1 with grep in /nix/store/lr4pyar97q66ac6vban3rljrwfsb5n10-lethe-0.6.1
found 0.6.1 in filename of file in /nix/store/lr4pyar97q66ac6vban3rljrwfsb5n10-lethe-0.6.1

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
lethe

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lr4pyar97q66ac6vban3rljrwfsb5n10-lethe-0.6.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A lethe https://github.com/r-ryantm/nixpkgs/archive/969cb864c919acffd15e17bf94a458392d040202.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/lr4pyar97q66ac6vban3rljrwfsb5n10-lethe-0.6.1
ls -la /nix/store/lr4pyar97q66ac6vban3rljrwfsb5n10-lethe-0.6.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

lethe

Maintainer pings
cc @fabaff for testing.

Result of nixpkgs-review pr 149314 run on x86_64-linux 1

1 package built:

lethe


NPM spectre.css version update to 0.1.5

You can update the NPM module to version 0.1.5.

Done. Thanks.

Thanks. Nice project :D

Thanks. :-D


[Gally]: master <- dev

Automatically created by Git-Ally

:tada: This PR is included in version 2.9.15 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Improve performance of leak tracker.

Move analysis of creation stack trace from start of tracking to leak filtering, to perform just for leaking objects.

consider adding a test to ensure that the test helper check does not run from shouldReport when this is false:
final shouldCheckCreator =
        phase.ignoredLeaks.createdByTestHelpers && result;

Added  TODO


Ignore components by name

It could be useful to ignore components given a pattern. e.g. src/routes/_not-a-page.vue, or .not-a-page or __not-a-page, etc but at the same time this can be achieved with the exclude option so is this really necessary?

I think it's worth adding a default value to exclude and let users override this if necessary. As for the convention, a file or folder starting with _ seems like an easy choice since URL usually do not start with a _. It still requires the option to be overridden for edge cases:
exclude: [
  '**/_*', // files
  '**/_*/**, // folders
  '!_not-ignored',  // _not-ignored should still be handled
]


Consume PALs from intended package

Currently, we link PALs for FX libraries from AppDep package. They should be eventually consumed from their own package.

#701 tracks this specifically for System.Native.a

We have Microsft.NETCore.Native for this purpose


CI is broken

It looks like the build has been broken since https://github.com/SymbiFlow/symbiflow-arch-defs/pull/1760

Simple designs such as counter and buttons are unroutable. It could be a problem with VPR.

@litghost / @acomodi - Can you please look into this?

@HackerFoo you have seen this for the vtr+Symbiflow CI, correct? In that case this PR solves the issue: https://github.com/SymbiFlow/symbiflow-arch-defs/pull/1711.
I have forced a Kokoro run as there were some infrastructure failures.

If you are speaking about the vtr+Symbiflow, that is a result of in progress master+wip updates.  To be clear, there are 3 CI's on arch-defs, Travis, vtr+Symbiflow and arch-defs.  Both Travis and arch-defs should both be green right now.

Then VtR+SymbiFlow doesn't need to pass?
The top level "Architecture Definitions (Presubmit)" is also failing on https://github.com/SymbiFlow/symbiflow-arch-defs/pull/1735, but I think it's because of the QuickLogic test.
Another strange thing is that several of the tests have not yet completed for the last 2 PRs.

The top level "Architecture Definitions (Presubmit)" is also failing on #1735, but I think it's because of the QuickLogic test.

If you rebase onto master, then the QuickLogic test will pass trivially.  Kokoro should be running as "presubmit as merged", but because it is part of the kokoro configuration it may be more restrictive.

Then VtR+SymbiFlow doesn't need to pass?

No.  VtR + SymbiFlow is a CI that is intended to show that the current master+wip on https://github.com/symbiflow/vtr-verilog-to-routing or master on https://github.com/verilog-to-routing/vtr-verilog-to-routing/ doesn't work with the particular revision of https://github.com/SymbiFlow/symbiflow-arch-defs/ .  Given that upstream master VTR and master+wip VTR can make changes that manifest issues with arch-defs (e.g. XML changes or https://github.com/verilog-to-routing/vtr-verilog-to-routing/issues/1571 ), it is possible for the VtR + SymbiFlow CI to be red as expected.  The signal is to investigate why the newer version of VTR no longer works.  In this particular case, changes in how the virtual RR graph was generated (because of upstream PR https://github.com/verilog-to-routing/vtr-verilog-to-routing/pull/1448) resulted in the latest master and master+wip needing changes.
This is why arch-defs uses environment.yml to decouple updates to yosys and VTR.
The key here is the what the various CI's are for.

Travis and kokoro arch-defs test whether the current revision of arch-defs works.  These are the CI's to pay attention too and make sure are green for the purposes of PR review and checking the health of master.
kokoro VtR master+wip + SymbiFlow and kokoro VtR master + SymbiFlow test whether the upcoming versions of VtR work with master arch-defs.  Ideally this stays green, but it will go red when an incompatible change arises on upstream.  They provide early signals of integration issues.  I believe @acomodi for example discovered https://github.com/verilog-to-routing/vtr-verilog-to-routing/issues/1571 from the VtR master+wip + SymbiFlow CI.

Another strange thing is that several of the tests have not yet completed for the last 2 PRs.

Do you mean that you don't see the kokoro arch-defs CI on the latest master commits?  That is because the previous continuous job is still running.  I investigated it, and it due to a routing job taking a while to completed.

When I check the merge details for #1762 or #1760, there are several either failed or "Tool failed" tests.

When I check the merge details for #1762 or #1760, there are several either failed or "Tool failed" tests.

I see that too, but if you look at the failing tests, are passing at the script level.  For example:
https://source.cloud.google.com/results/invocations/8ecd20df-54dc-4d60-820f-20a9b7224a24/log
[ID: 7673017] Build finished after 17563 secs, exit value: 0

Warning: Permanently added 'localhost' (ECDSA) to the list of known hosts.
[14:24:56] Collecting build artifacts from build VM
[14:27:05] Kokoro builder finished

We've been seeing some issues around artifact collection, so I suspect a Kokoro hiccup on those.

For reference, the master xc7 build for #1760 is green here: https://source.cloud.google.com/results/invocations/ba1043e9-80b6-4508-ac34-3bc488cc96f6/targets

When I check the merge details for #1762 or #1760, there are several either failed or "Tool failed" tests.

I see that too, but if you look at the failing tests, are passing at the script level. For example:
https://source.cloud.google.com/results/invocations/8ecd20df-54dc-4d60-820f-20a9b7224a24/log
[ID: 7673017] Build finished after 17563 secs, exit value: 0

Warning: Permanently added 'localhost' (ECDSA) to the list of known hosts.
[14:24:56] Collecting build artifacts from build VM
[14:27:05] Kokoro builder finished

We've been seeing some issues around artifact collection, so I suspect a Kokoro hiccup on those.

So the issue is likely that Kokoro hit out of disk when moving the workspace for artifact collection.  Because kokoro applies the collection filtering after copying all files from the workspace (don't ask me), the addition of RapidWright and some more third_party` submodules, we likely were scrapping the upper disk limit.

The "Xilinx Series 7 - Install (Presubmit)" test is now failing with:
ChecksumMismatchError: Conda detected a mismatch between the expected content and downloaded content
for url 'https://conda.anaconda.org/litex-hub/linux-64/gcc-riscv64-elf-nostdc-9.2.0-20200923_200922.tar.bz2'.
  download saved to: /tmpfs/src/github/symbiflow-arch-defs-presubmit-install/env/downloads/conda-pkgs/gcc-riscv64-elf-nostdc-9.2.0-20200923_200922.tar.bz2
  expected md5: 98bd7a66867ab138fdb104c53df1db44
  actual md5: 1e303a894909bad44b81fb1910adff22

I've just downloaded this file manually
# md5sum gcc-riscv64-elf-nostdc-9.2.0-20200923_200922.tar.bz2
98bd7a66867ab138fdb104c53df1db44  gcc-riscv64-elf-nostdc-9.2.0-20200923_200922.tar.bz2

It seems to have the expected md5

Now I get this from architecture presubmit tests:
Traceback (most recent call last):
  File "/tmpfs/src/github/symbiflow-arch-defs-presubmit-install/xc/common/utils/prjxray_routing_import.py", line 1566, in <module>
    main()
  File "/tmpfs/src/github/symbiflow-arch-defs-presubmit-install/xc/common/utils/prjxray_routing_import.py", line 1534, in main
    node_remap = create_node_remap(capnp_graph.graph.nodes, channels_obj)
  File "/tmpfs/src/github/symbiflow-arch-defs-presubmit-install/xc/common/utils/prjxray_routing_import.py", line 1333, in create_node_remap
    coord = tuple(hilbert_curve.coordinates_from_distance(h))
AttributeError: 'HilbertCurve' object has no attribute 'coordinates_from_distance'

It looks like that dependency just updated breaking compatibility: https://pypi.org/project/hilbertcurve/#history
Ugh.

The vendor tool test also fails with:
/tmpfs/src/github/symbiflow-arch-defs-presubmit-xc7-vendor/env/RapidWright/bin/rapidwright_classpath.sh was not found, check if RapidWright has been built.

link

Continuous CI has been green for the last 6 runs, closing.


StompClient hangs on [error] :closed

Hi,
I've been using the supervised stomp client example for a while already, and I've noticed that after a few activemq start/stop cycles I get an error message when calling StompClient.subscribe/3.
[error] :closed

To clarify, when this happens StompClient.connect/2 does return a pid, but no "connected" message shows that it was a successful connection. When I try to subscribe to a topic using that connection I get the message above.
Any ideas? What information should I add to the issue?
Thanks!

I have got it working by placing the call to StompClient.subscribe/3 in the on_connect callback handler.


[Docs]Update docs for Client libraries C++

Motivation

Update client-libraries-c++
Make the document easy to read
Correct inappropriate punctuation
Correction of incorrect document

Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[ ] no-need-doc
(Please explain why)

[x] doc
(If this PR contains doc changes)

@Anonymitaet @Jennifer88huang PTAL


Enable ability for smart contracts to be owners of metadata entries.

Summary
Allow metadata scopes, sessions, and/or records to have owners that are smart contracts.
Problem Definition
As a smart-contract creator, I want to be able to have my smart-contract own metadata entries so I can more easily manage updating information.
Proposal

In the x/metadata wasm encoders, add the smart contract address as the first signer of the message being sent.
In the x/metadata msg validation, make sure that the first signer is one of the required signers.
In our wasmd fork, update handleSdkMessage to only check the first address from GetSigners().

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

It looks like, for 3, we could instead use the WithMessageHandler option and define the handler in the provenance repo instead. Since you've already got it in our wasmd fork, though, we can probably use that just fine. It'd be nice to get off our fork at some point, and go back to using the standard repo. If we ever do that, we'll need to go with the WithMessageHandler way, but until then.... eh.


Second level cache stores identifier with correct type even if findById is called with wrong identifier type

Fixes http://www.doctrine-project.org/jira/browse/DDC-3967

Fixed through #1568. Thanks for the report and PR! =)


Log metrics/models etc... to studio directly with Live or DVCLiveCallbacks in training script

Hello, I was wondering if there is a way to directly log params, metrics, etc... to iterative ai studio within the training script using Live or DVCLiveCallbacks (hugging face) instances. If not, I don't really get how to do this tbh. is this through dvc exp push ?
Thanks a lot in advance.
Great work from all the iterative ai team btw !!

Hello, I was wondering if there is a way to directly log params, metrics, etc... to iterative ai studio within the training script using Live or DVCLiveCallbacks (hugging face) instances. If not, I don't really get how to do this tbh. is this through dvc exp push ?
Thanks a lot in advance.
Great work from all the iterative ai team btw !!

Yes @BaptisteLoquette, you need to include a Studio token locally to see them live in Studio. See https://dvc.org/doc/studio/user-guide/experiments/live-metrics-and-plots#set-up-an-access-token. We can better clarify this in the DVCLive docs.
Can you better describe how you got to this point? Were you already using DVC or started with DVCLive? Where did you go in the docs to try to find this info?

I already used dvc & cml. But new to the studio. I already added the studio token in the gitlab variables. And at each git trigger (here push) the commit appears on the studio though, except for the data size, nor the metrics, loss are models are logged/registered. I do think it's because I haven't configured a dvc.yaml file in my current project. Also, I configured an azure blob storage. When I log_artifact the model, it will be stored in the blob storage right ?
Thx for your answer

I mean it's logical that the data size is logged as I dvc pushed the datas, but I was thinking Using Live() or DVCLiveCallback would automatically log the metrics etc... to the studio

Huh, no sry didn't see this https://dvc.org/doc/user-guide/project-structure/configuration#studio I need to enable it in the config right ?

I mean it's logical that the data size is logged as I dvc pushed the datas, but I was thinking Using Live() or DVCLiveCallback would automatically log the metrics etc... to the studio. Only from the training script and no git action.

Yes, that's how it should look. It sounds like something isn't configured right for you.

Huh, no sry didn't see this https://dvc.org/doc/user-guide/project-structure/configuration#studio I need to enable it in the config right ?

I already added the studio token in the gitlab variables.

The token needs to be either in your dvc config under studio.token or in the environment variable DVC_STUDIO_TOKEN.

Fixed it, by setting with dvc config --global studio.token. But weird because it starts inside a gitlab CI script and DCC STUDIO TOKEN was configured in the variables

Thx a lot for your answers

If you want to post your CI script, we can try to debug. If not, I guess we can close the issue?

It's fixed. yes we can. Thx a lot again


Context dependent precedence

Implement context dependent precedence, similar to what is found in bison.

Done, for tech config mode


Allow the ability to specify user to run execute-command as

Hi,
When configuring webhooks to run as a service, it would be useful to allow the execute-command to be run as a particular user. Of course, this can be done within the execute-command string itself, i.e., sudo, but perhaps a more elegant approach could be something like:
[
    {
        "id": "push-push-push",
        "execute-command": "./post-something-witty-to-slack.sh",
        "execute-command-as": "foo",
        "command-working-directory": "/home/foo/bin"
    }
]

This would be very attractive from a security PoV.
Thank you.
-=david=-

All webhook is going to do is run su in the background.  This feels like feature creep to me.

I completely agree with @moorereason


not working when package not published

Situation

package is exist in monorepo
it's not published
only referenced in monorepo

Fix it

do not throw error when package not found
find repository package metadata from installed one

How to fix it?

read metadata from yarn workspaces


Improve grep in entrypoint.sh to only find .conf and .conflist files

the preivous grep expression will also find e.g. fooconffoo this one should only grab .conf and .conflist files ... I thought I had added this to my original PR but must have accidentally missed it.

@dougbtv ^^

Hey @dougbtv - I am working on a PR to add Multus to Kubespray (https://github.com/kubernetes-incubator/kubespray/pull/3166). I'd like to use the image from https://hub.docker.com/r/nfvpe/multus/tags/. Do you think you could update latest once this has been merged? Also, is there an ETA for Multus v3.2?

@Kusanagi9999 -- sounds awesome re: Kubespray! Let me think about it a bit -- I wanted to have :latest be the latest stable (so I don't have to update the daemonset yaml often), and the images/image tags gets updated when we tag in the git rpeo, however... I think maybe what we could do is update the .travis.yml to also build a nfvpe/multus-latest image.
OR: I can manually tag an image for you and push it up to say, nfvpe/multus:v3.1.autoconf which includes this PR? Would that work in the meantime?
No solid date on a 3.2, however, it shouldn't be too long. I would guess within 30 days, we have a number of features we'll want in and tagged shortly, I'm sure of it.
Also, merging here in a moment! Sorry so slow getting back to this PR, been on the road.

@dougbtv, OK got it, nfvpe/multus:v3.1.autoconf would totally work for now until v3.2 is out. Thanks a lot!


Added implementation of stack using two queues

Description of Change

Implementation of stack using two queues. The algorithm is implemented by making the pop() operation costly ( linear time).
Checklist

[x] Added description of change
[x] Added file name matches File name guidelines
[x] Added tests and example, test must pass
[x] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[x] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.

Notes:

Is this different from #953 ?

#953 is the algorithm of implementing a queue using 2 stacks while this one is the implementation of a stack using 2 queues.

#953 is the algorithm of implementing a queue using 2 stacks while this one is the implementation of a stack using 2 queues.
Got it. Can you please update the code with namespaces and descriptions as in #953 and also add more test cases. Thank you

can you suggest why compile checks(windows-latest) are failing? I am quite new to open source contribution. Thank you.

compile checks are failing

Please see the latest review comments, and apply the suggestions.

Thank you Panquesito, for suggesting the necessary changes.  Can you please explain why we used std:: move in case of string and not for double?

std::move of the variable book_price of the trivially-copyable type double has no effect.
If we remove std::move, the code will be more efficient; we're also fixing a clang-tidy warning. 🙂

can you merge this PR if it is fine?

It seems to be fine to me. 🙂
@kvedala, may you please review?

No response from author.
Closing this pull request.
If you are going to re-work on this, please ping me and I'll re-open the PR.


Base64 image encoding

Hello
I am trying to generate a simple test pdf to get started with the library which is working fine with text but I can not seem to add images.
The code below gives me the following error:
Uncaught Error: Supplied Data is not a valid base64-String jsPDF.convertStringToImageData
**function pdfBil() {
var doc = new jsPDF()

doc.addImage("./logo.jpg", 'JPG', 185, 5, 10, 15, 15, 10);
doc.text('test', 10, 10)
doc.save('a4.pdf')

}**
I have tried with both PNG and JPEG aswell as modifying the filepath(not sure if that makes a difference).
I have also tried both Chrome and Firefox.
Using the latest version of jsPDF
Sorry if this is a duplicate I am new to the github thing. I tried to search for similar errors but they all seemed to be resolved.

I cannot reproduce the error. Which version of jsPDF are you using? In theory, it's impossible that this error is thrown, because convertBase64ToBinaryString in the addimage plugin is always called with throwErrors = false.

I have tried both the cdns mentioned so
https://cdnjs.cloudflare.com/ajax/libs/jspdf/1.5.3/jspdf.debug.js
and
<EMAIL_ADDRESS>

Ah ok, I just checked: this was already fixed but not released. There will be a new release probably around Spring.


ActiveDirectoryDsc: Fix CHANGELOG.md

Pull Request (PR) description

Fixes the change log.

Also tests the new azurepipeline.yml configuraiton to see if it works.
This Pull Request (PR) fixes the following issues
None
Task list

[x] Added an entry to the change log under the Unreleased section of the
file CHANGELOG.md. Entry should say what was changed and how that
affects users (if applicable), and reference the issue being resolved
(if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Community Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
[ ] New/changed code adheres to DSC Community Style Guidelines.

This change is

I will merge this as soon as the tests finishes to test the CI pipeline.


Layers and views were not found from overlay directory

This will fix the existing feature to work with layers and views that were missed.

Thanks!


NPE when call queue.poll

      val bytes = queue.poll(10, TimeUnit.SECONDS)

java.lang.NullPointerException
at org.redisson.RedissonBlockingQueue$2.execute(RedissonBlockingQueue.java:74)
at org.redisson.connection.MasterSlaveConnectionManager.write(MasterSlaveConnectionManager.java:296)
at org.redisson.connection.MasterSlaveConnectionManager.write(MasterSlaveConnectionManager.java:285)
at org.redisson.RedissonBlockingQueue.poll(RedissonBlockingQueue.java:71)
at service.StatisticWorker$$anonfun$receive$1.applyOrElse(StatisticService.scala:143)
at akka.actor.Actor$class.aroundReceive(Actor.scala:467)
at service.StatisticWorker.aroundReceive(StatisticService.scala:103)
at akka.actor.ActorCell.receiveMessage(ActorCell.scala:516)
at akka.actor.ActorCell.invoke(ActorCell.scala:487)
at akka.dispatch.Mailbox.processMailbox(Mailbox.scala:238)
at akka.dispatch.Mailbox.run(Mailbox.scala:220)
at akka.dispatch.ForkJoinExecutorConfigurator$AkkaForkJoinTask.exec(AbstractDispatcher.scala:397)
at scala.concurrent.forkjoin.ForkJoinTask.doExec(ForkJoinTask.java:260)
at scala.concurrent.forkjoin.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1339)
at scala.concurrent.forkjoin.ForkJoinPool.runWorker(ForkJoinPool.java:1979)
at scala.concurrent.forkjoin.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:107)

Can't reproduce. Here is a testcase i have used:
        RBlockingQueue<Integer> queue1 = redisson.getBlockingQueue("queue1");
        queue1.put(1);

        queue1.poll(10, TimeUnit.SECONDS);

@mrniko I can replicate this consistently. Try running the following.
RBlockingQueue<Integer> queue1 = redisson.getBlockingQueue("queue1");
queue1.put(1);
queue1.poll(10, TimeUnit.SECONDS);
queue1.poll(10, TimeUnit.SECONDS);

The second call to poll will wait ten seconds, then you'll get a NullPointerException.
The issue is that io.netty.util.concurrent.Future#getNow returns null if the future is not done yet, which in this case happens when the queue is drained. That method is called by com.lambdaworks.redis.RedisAsyncConnection#awaitInterruptibly, which in turn is called by blpop, which is called in poll.

fixed

Thank you! :sparkles: :sparkles: :sparkles:

thanks too:P

and sorry I didn't come up with a producer.:( super busy.thanks again.


Add Yahoo! JAPAN as search engine

close #235
This PR adds Yahoo! JAPAN as a search engine.
For now, only web searches are supported.
Screenshots

Desktop
Mobile

Now it is ready for review 😎

@iorate Could you review this PR?


chore: allow override runtimeClassName

Hi,
A tiny PR to be able to force the runtimeClassName when I'm not using by default nvidia in my CRI config.
Thanks

Please sign off your commit as described here

Thanks @nikkon-dev ,changes done.

Can you please @nikkon-dev bump a minor release for this 🙏🏻


How does this bot perform compared to SaltBot by synkarius?

Hi, I'm interested in trying your bot on my account which has 30mil salt. How does your bot perform? How much does it gain on average per day, etc.? Any data on this at all would be very appreciated! :)

I have a very small dataset compared to the breath of characters in SaltyBet, this leads to most bets placed being coinflips. Further before today I only had a very basic "lets implement something at least" betting strategy.
The last few days of the New Bracket meant I had meaningful data on the characters involved but was probably held back by the actual betting logic. I think i peaked at slightly above 60% winrate then but I've not exactly done rigorous statistics on that.
At the moment it hovers at around 50% winrate. The nature of salty bet means that it still makes a tiny profit even if its breaking even on wins and tournaments help too.
Still if you were looking at trying RealSalt there is a reason there isn't a released version at this time. Even ignoring performance its not user friendly, its a console program and crashed on me twice today.
If you want a .net or c# betting that you can root around in the guts in this might be fore you.


[ConfigTransformer] Route is converted to accidently to service

The following route configuration:
bazinga_js_translation:
    resource<EMAIL_ADDRESS>
Is accidently converted to a service definition:
<?php

declare(strict_types=1);

use ConfigTransformer202207\Symfony\Component\Routing\Loader\Configurator\RoutingConfigurator;
use function ConfigTransformer202207\Symfony\Component\DependencyInjection\Loader\Configurator\service;

return static function (RoutingConfigurator $routingConfigurator): void {
    $routingConfigurator->import(service('BazingaJsTranslationBundle/Resources/config/routing/routing.yml'));
};

Duplicate of https://github.com/symplify/symplify/issues/4260
An algorythm that detects if config is routing must be improved.

@TomasVotruba looks like we are atleast since #4260 correctly detecting it that it is creating a RoutingConfigurator maybe depending on that it is now $routingConfigurator we can convert resource correctly.

Yes, it's covered here https://github.com/symplify/symplify/pull/4280/commits/03a46214b2b96a3f03a2f2e1a96870e796d48e3c#diff-fa4f1a230fe3e1c020939c290b47ac0bff8e88817bc5e8515a5687e762c2a09a


webrtc: restore compatibility when running behind a proxy, in a subpath

Hi.
Using v1.3.0, and I am not sure if it has been fixed.
Proxying with apache in a subfolder does not work : error 404 is returned. Direct access to the WebRTC stream thru the 8889 port is working.
Any hint/idea ?
Thanks.

@LeRatierBretonnien this is not the way to report bugs. Open an issue and provide details on how to replicate.


Why doesn't the index.wiki render?

Shouldn't this page look like a wiki?
https://github.com/vimwiki/testwikis/blob/master/vimwiki/index.wiki

Not sure, I understand you correctly. Github does not render Vimwiki – you'd have to render it using vimwiki to get a html-file.


Add basic version of Search

Builds on #3
Very basic implementation of a search. Right now, only searches through the titles of each page.
Todo:

[ ] Search the hash links of each page (or full text somehow?)

For more advanced use we can integrate Algolia DocSearch: https://docsearch.algolia.com.
But that needs to be added case by case, it’s great to have the basic search functionality (and styles!) so that we can easily switch to DocSearch.

Wow I really enjoy the styles! Might be better than DocSearch, thanks!


Build all jobs with matching name.

the MIT license from the license file is fine! Sorry I missed the notification!

No worries


[Backport] [2.x] Bump com.networknt:json-schema-validator from 1.1.0 to 1.2.0 (#11963)

Backport of https://github.com/opensearch-project/OpenSearch/pull/11963 to 2.x

❌ Gradle check result for 26d6bba: FAILURE

https://github.com/opensearch-project/OpenSearch/issues/11875
https://github.com/opensearch-project/OpenSearch/issues/9773
https://github.com/opensearch-project/OpenSearch/issues/8946

في اثنين، 22 يناير، 2024 في 11:27 م، كتب Andriy Redko <
@.***>:

❌ Gradle check result for 26d6bba
https://github.com/opensearch-project/OpenSearch/commit/26d6bba5d79973247b1ec034034b8ba70a9b9dc4:
FAILURE https://build.ci.opensearch.org/job/gradle-check/32450/
#11875 https://github.com/opensearch-project/OpenSearch/issues/11875
#9773 https://github.com/opensearch-project/OpenSearch/issues/9773
#8946 https://github.com/opensearch-project/OpenSearch/issues/8946
—
Reply to this email directly, view it on GitHub
https://github.com/opensearch-project/OpenSearch/pull/11973#issuecomment-1904750346,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BEINNMM2NCOJKVS3ONF5BODYP3DTLAVCNFSM6AAAAABCFXR7T2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBUG42TAMZUGY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

❌ Gradle check result for 26d6bba: FAILURE

https://github.com/opensearch-project/OpenSearch/issues/11875
https://github.com/opensearch-project/OpenSearch/issues/9773
https://github.com/opensearch-project/OpenSearch/issues/8946


Issue trying to install using go get -u github.com/lileio/lile/...

I tried to install the package but the terminal shows this:
# cd /home/user/go/src/google.golang.org/api; git pull --ff-only
From https://github.com/googleapis/google-api-go-client
 * [new branch]      master          -> origin/master
 * [new branch]      release-v0.38.0 -> origin/release-v0.38.0
 * [new tag]               v0              -> v0
 * [new tag]               v0.1.0          -> v0.1.0
 * [new tag]               v0.10.0         -> v0.10.0
 * [new tag]               v0.11.0         -> v0.11.0
 * [new tag]               v0.12.0         -> v0.12.0
 * [new tag]               v0.13.0         -> v0.13.0
 * [new tag]               v0.14.0         -> v0.14.0
 * [new tag]               v0.15.0         -> v0.15.0
 * [new tag]               v0.16.0         -> v0.16.0
 * [new tag]               v0.17.0         -> v0.17.0
 * [new tag]               v0.18.0         -> v0.18.0
 * [new tag]               v0.19.0         -> v0.19.0
 * [new tag]               v0.2.0          -> v0.2.0
 * [new tag]               v0.20.0         -> v0.20.0
 * [new tag]               v0.21.0         -> v0.21.0
 * [new tag]               v0.22.0         -> v0.22.0
 * [new tag]               v0.23.0         -> v0.23.0
 * [new tag]               v0.24.0         -> v0.24.0
 * [new tag]               v0.25.0         -> v0.25.0
 * [new tag]               v0.26.0         -> v0.26.0
 * [new tag]               v0.27.0         -> v0.27.0
 * [new tag]               v0.28.0         -> v0.28.0
 * [new tag]               v0.29.0         -> v0.29.0
 * [new tag]               v0.3.0          -> v0.3.0
 * [new tag]               v0.3.1          -> v0.3.1
 * [new tag]               v0.3.1+alpha    -> v0.3.1+alpha
 * [new tag]               v0.3.1-alpha    -> v0.3.1-alpha
 * [new tag]               v0.3.2          -> v0.3.2
 * [new tag]               v0.30.0         -> v0.30.0
 * [new tag]               v0.31.0         -> v0.31.0
 * [new tag]               v0.32.0         -> v0.32.0
 * [new tag]               v0.33.0         -> v0.33.0
 * [new tag]               v0.34.0         -> v0.34.0
 * [new tag]               v0.35.0         -> v0.35.0
 * [new tag]               v0.36.0         -> v0.36.0
 * [new tag]               v0.37.0         -> v0.37.0
 * [new tag]               v0.4.0          -> v0.4.0
 * [new tag]               v0.5.0          -> v0.5.0
 * [new tag]               v0.6.0          -> v0.6.0
 * [new tag]               v0.7.0          -> v0.7.0
 * [new tag]               v0.8.0          -> v0.8.0
 * [new tag]               v0.9.0          -> v0.9.0
There is no tracking information for the current branch.
Please specify which branch you want to merge with.
See git-pull(1) for details.

    git pull <remote> <branch>

If you wish to set tracking information for this branch you can do so with:

    git branch --set-upstream-to=origin/<branch> master

package google.golang.org/api/iterator: exit status 1
cannot find package "google.golang.org/api/option" in any of:
        /usr/local/go/src/google.golang.org/api/option (from $GOROOT)
        /home/user/go/src/google.golang.org/api/option (from $GOPATH)
cannot find package "google.golang.org/api/option/internaloption" in any of:
        /usr/local/go/src/google.golang.org/api/option/internaloption (from $GOROOT)
        /home/user/go/src/google.golang.org/api/option/internaloption (from $GOPATH)
cannot find package "google.golang.org/api/transport/grpc" in any of:
        /usr/local/go/src/google.golang.org/api/transport/grpc (from $GOROOT)
        /home/user/go/src/google.golang.org/api/transport/grpc (from $GOPATH)
cannot find package "google.golang.org/api/support/bundler" in any of:
        /usr/local/go/src/google.golang.org/api/support/bundler (from $GOROOT)
        /home/user/go/src/google.golang.org/api/support/bundler (from $GOPATH)```

Anything I can do?

Hi there,
I think you may have a couple of problems,
firstly, you're not using go modules, which means that you might run in to trouble using -u and with general version mis matches, so I recommend installing lile in a modules based project, if installing outside of homebrew, binary.
secondly, I think your cached version of google.golang.org/api, a library lile uses, has a general git checkout issue, so you try just doing rm -rf /home/user/go/src/google.golang.org/api and then doing a go get again?

Hi there,
I think you may have a couple of problems,
firstly, you're not using go modules, which means that you might run in to trouble using -u and with general version mis matches, so I recommend installing lile in a modules based project, if installing outside of homebrew, binary.
secondly, I think your cached version of google.golang.org/api, a library lile uses, has a general git checkout issue, so you try just doing rm -rf /home/user/go/src/google.golang.org/api and then doing a go get again?

Thanks

Thanks


Face mask detection

I am a GSSOC participant. I would like to work on a project that detects if a person is wearing a face mask and alerts the bystanders if there are a lot of people without face masks. Please assign this project to me.

Hey @akhil-debug, we are focusing more on the implementation of Image Processing techniques rather than its application.
This repo is not for the accumulation of projects/mini-projects.
Please reopen the issue if I missed out on something. Do let me know if there is a specific Image Processing technique you will be using here!


Delayqueue disorders the redis stream data during master restart

In my setup, where one application (producer) adds data into redis stream using XADD and another application (consumer) data through XREAD and latest ioredis client was used.
PROBLEM 1:
During a redis master restart case and producer side, we see below things:
ioredis throws error as "MOVED" and "CLUSTER IS DOWN". On such errors, it queues the events received in delayqueue in one order. On multiple retries during sendcommand, the error is throws and delayQueue is rearranged with events.
When redis connection is back within 3 secs, on a bulk the delayqueue events are sent to redis but in disorder fashion.
ioredis debug logs:
ioredis:delayqueue send 72 commands in clusterdown queue  -> In this 72 commands events are queued in different order
On consumer side: we notice events not received in the order.
After trying out couple of options, we still not yet solved this case.
What options were tried out so far:

enableOfflineQueue - set this to false
maxRedirections - set this to -1 (but those events were rejected during redis restart which is not expected here)
retryDelayOnClusterDown - we can't set this option because in production we don't know for how long cluster is down

Did we miss out anything else?
PROBLEM :2
Some of the events are not even sent to redis and we see the xadd response missing for few messages. At least, problem 1 was about events not in sequence being sent to redis. But in this problem, few events are even missing. And both problem 1 and 2, occurs on same run.
TL;DR:
Does ioredis handles delayqueue in effective way such that the events pushed into the queue are sent out of the queue in the same order? And also some events being missed to send to redis on redis restart.

Do we have any updates on this issue?

@luin, can you please help me solve this case? Have anyone faced this before?


Have a docker file that have all the needed software

I'll take a bite of it (would take a while I guess)

A first step into that direction would be a list of required programs.


Loading spinner when deleting collection and namespace content

No issue
Quick PR addition for loading spinner in a modal window when deleting collection and namespace, sometimes the request takes longer, so the user knows the request is still pending :)

Nice catch :+1:,
should we also do the same for container deletions?


minLength for non string type

When a property has mutliple allowed types (eg: string and null), but some of the validator options only apply to one of the types (eg: when it is a string), they get checked anyway, eg:
    "my_field" => {
      "type" => ["string", "null"],
      "required" => true,
      "minLength" => 1,
    },

will not validate when my_field is null.
I check in with the google group about this: http://groups.google.com/group/json-schema/browse_thread/thread/bc4bc01811d21bea/c2032f62b159d6eb .

Can you provide an example where this fails? I am attempting to test this with the latest release of json-schema (0.9.10), and I can't reproduce a failure in validation.
>> schema = {"type" => "object", "properties" => { "my_field" => {"type" => ["string","null"],"required" => true, "minLength" => 1 } } }
=> {"type"=>"object", "properties"=>{"my_field"=>{"required"=>true, "minLength"=>1, "type"=>["string", "null"]}}}
>> JSON::Validator.validate(schema, {"my_field" => "test"})
=> true
>> JSON::Validator.validate(schema, {"my_field" => nil})
=> true
>> JSON::Validator.validate(schema, {"my_field" => ""})
=> false

Sorry, I must have had an old version, I can't reproduce this now.


CASMINST-6543: Linting

Backport of https://github.com/Cray-HPE/docs-csm/pull/3989

/backport release/1.0


add build steps to readme and/or mavenize project

Can you please add the build steps to the readme? It would be great if you could mavenize, but either way.
Also, would you mind doing a release that includes your fix for https://github.com/idok/eslint-plugin/issues/19?
Thanks!
Ian

Currently, to build the project you need to use this menu:


Snapshot disappears on Undo

This is the first time I used snapshots, so I may have missed something.
If I create a snapshot and use Undo, the snapshot disappears.
It may happen "in real life" that you create a snapshot, do some edits and use Undo, but if you undo a little too far, you lose your snapshot.
This happened to me (I'm still not used to snapshots so I don't always think about recalling one), and it's pretty annoying.
So could you keep snapshot creation outside of the undo history, if possible ?
If it's an expected behavior, could you add an option ?

Hey killergege, sorry for the slow response.
Snapshots are stored in the RPP file.  Reaper manages the RPP when doing undo steps, and will delete any changes (like new snapshots) when you run undo.  There's not much I can do about that.
However, you can export snapshots by right clicking on them, or pasting to the clipboard.  Those exports of course will last forever.  I know this isn't a perfect solution, but that's really all I can do right now.

Ok, thanks !


drivers: gpio: sx1509b: convert to DT__ defines

Convert sx1509b sensor driver to use new defines.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@655d3cc). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #13101   +/-   ##
=========================================
  Coverage          ?   48.62%
=========================================
  Files             ?      313
  Lines             ?    46484
  Branches          ?    10728
=========================================
  Hits              ?    22604
  Misses            ?    19413
  Partials          ?     4467

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 655d3cc...e84df94. Read the comment docs.


detailed Content filter info

When completion is cancelnd in case of contentfilter Version one throes an Azure.RequestFailedException with detailed information as Json stored in Content:
_Azure.RequestFailedException: The response was filtered due to the prompt triggering Azure OpenAI's content management policy. Please modify your prompt and retry. To learn more about our content filtering policies please read our documentation: https://go.microsoft.com/fwlink/?linkid=2198766
Status: 400 (model_error)
ErrorCode: content_filter

Content:
{"error":{"message":"The response was filtered due to the prompt triggering Azure OpenAI's content management policy. Please modify your prompt and retry. To learn more about our content filtering policies please read our documentation: https://go.microsoft.com/fwlink/?linkid=2198766","type":null,"param":"prompt","code":"content_filter","status":400,"innererror":

{"code":"ResponsibleAIPolicyViolation","content_filter_result":{"hate":{"filtered":false,"severity":"safe"},"self_harm":

{"filtered":false,"severity":"safe"},"sexual":{"filtered":true,"severity":"high"},"violence":{"filtered":false,"severity":"safe"}}}}}_

Version 2 throws System.ClientModel.ClientResultException without this information
_System.ClientModel.ClientResultException: HTTP 400 (content_filter)
Parameter: prompt

The response was filtered due to the prompt triggering Azure OpenAI's content management policy.
Please modify your prompt and retry. To learn more about our content filtering policies please read our documentation: https://go.microsoft.com/fwlink/?linkid=2198766_

Is it possible to get the Content filter reason in one or an other way?

Hello, @wolfgangschneider. Thank you for reaching out! This repo is for the official OpenAI library for .NET. If you're working with Azure OpenAI and using the Azure.AI.OpenAI library, please open an issue in the azure-sdk-for-net repo instead.

I will do so


feat(CLI): Support envd images prune #976

Generally LGTM!
Thanks for your contribution! :tada: :+1:

The CI failure is not caused by your PR. The openvsx.org is down.
I will merge the PR manually if e2e-lang passes.
Thanks for your contribution! :tada: :+1:

@all-contributors Please add @enjoyliu for the code contribution


Update repo description

Update

A social coding experiment that updates its own code democratically

To mention autocracy now that #138 is merged

The comment

I'll revert it if it succeed

is strikingly similar to promises of real life dectators to step down after some "success".

I think we'll have a democracy again soon. In anticipation I'll go ahead and close this.


Removed Win32_Product from SPInstall

Quick fix to something that should have happened in 1.1 when we removed Win32_Product from SPInstallPrereqs. I also made style changes to bring SPInstall in line as well.

[X] Change details added to Unreleased section of changelog.md?
[n/a] Added/updated documentation and descriptions in .schema.mof files where appropriate?
[n/a] Examples updated for both the single server and small farm templates in the examples folder?
[X] New/changed code adheres to [Style Guidelines]?(https://github.com/PowerShell/DscResources/blob/master/StyleGuidelines.md)?
[X] Unit and Integration tests created/updated where possible?

This change is

Looks perfect

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

It just needs an update from Dev.

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 2 of 3 files at r1, 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Dragging/dropping is laggy on slower computers when using the dragOver event

If a function is called on the 'dragOver' event, depending on the function's complexity, it can make slower computers very laggy while dragging.
If there was a way to reduce the speed at which this executes, that would help a lot for performance.

You mean some kind of throttling/debouncing of the function itself? Maybe something like https://remysharp.com/2010/07/21/throttling-function-calls could be used so we wouldn't need to build it into knockout-dragdrop?

I was able to use underscore.js's _.throttle method to get the performance increased. This had some side effects (like if you let go before it would let the 'dragOver' event function execute) but they were pretty simple to resolve. It all works fantastically now, but built in throttling might be a good idea to keep in mind for feature enhancements. Especially if you could guarantee the first execution of the function call.


raft/rafttest/node.go: make send async in start

This is for TODO: make send async, more like real world....
And what does it mean by like real world? In real world, does
node wait until it sends out all messages, and then Advance
to return the next available Ready? This change would send out the
messages in parallel, not in a sequential order: this is asynchronous
not waiting all sends to return. But it still has synchronization step
to wait for all concurrent n.iface.send(m) to be done, right before
n.Advance().
It seems to pass the test even without waiting for sends
(just run n.iface.send(m) in the background with goroutine)
and just n.Advance right after launching goroutines, as below:
for _, m := range rd.Messages {
    go func(m raftpb.Message) {
        n.iface.send(m)
    }(m)
}
n.Advance()

Thanks,

And I wanted to run benchmarks to see if this introduces any performance difference, but
go test -bench . -benchmem -cpu 1,2,4,8,16; in my local ~/go/src/github.com/coreos/etcd/raft/rafttest directory keeps failing as below:
--- FAIL: TestNetworkDelay-16 (0.05s)
	network_test.go:70: total = 49.00659ms, want > 50ms

// Or

--- FAIL: TestBasicProgress-16 (0.63s)
	node_test.go:46: commit = 10007, want = 10006
	node_test.go:46: commit = 10007, want = 10006
	node_test.go:46: commit = 10007, want = 10006
	node_test.go:46: commit = 10007, want = 10006
	node_test.go:46: commit = 10007, want = 10006

Any suggestion?

https://travis-ci.org/coreos/etcd/jobs/81722485
shows it fails again with:
--- FAIL: TestKVHash (0.55s)
	kv_test.go:642: hash[0](1031971881) != hash[1](1321773765)
	kv_test.go:642: hash[1](1321773765) != hash[2](1031971881)

There's an issue on this: https://github.com/coreos/etcd/issues/3560

Closing this due to merge conflict and inactivity. Will revisit this later. Thanks,


unneccessary check for pool_grp/svcl being NULL in H5daos_init (breaks examples)

Hi,
in H5daos_init there are checks that the DAOS pool and SVCL aren't NULL. However, this appears to be inconsistent with that the connector is actually able to accept: https://github.com/HDFGroup/vol-daos/blob/95772a5c1134ebf671483c08a0f5d3ef1aafb69e/src/daos_vol.c#L384-L387
A little bit later, the (unmodified, since const) values of pool_grp and pool_svcl arguments are passed to H5_daos_set_pool_globals: https://github.com/HDFGroup/vol-daos/blob/95772a5c1134ebf671483c08a0f5d3ef1aafb69e/src/daos_vol.c#L405
However, H5_daos_set_pool_globals accepts the two parameters being NULL and the function uses the default values or values passed via environment variables in that case, which is quiet convenient.
The issue in H5daos_init actually breaks allmost if not all of the examples which initalize the group to NULL: https://github.com/HDFGroup/vol-daos/blob/95772a5c1134ebf671483c08a0f5d3ef1aafb69e/examples/h5dsm_file_open.c#L5
One ends up with an error like this:
DAOS VOL-DIAG: Error detected in DAOS VOL (1.0.0) MPI-process 0:
  #000: /home/centos/install/hdf5/vol-daos/src/daos_vol.c line 385 in H5daos_init(): not a valid service group
    major: Invalid arguments to routine
    minor: Bad value
*FAILED*
        at h5dsm_file_open.c:21 in main()...

I suggest to remove the checks mentioned above as the NULL values are actually accepted by the routine they are passed to.

Thanks for reporting this! As you noticed, there were indeed some changes to the way that the pool group name and pool service list were handled which allowed NULL internally so that they could be picked up from environment variables if set. Those changes hadn't made it to the public interface yet, but should be address by #18.


Feature/table backspace handler

Made a new backspace plugin to prevent the user from deleting the table cells

I made this PR based on the other PR I made so that one should be merged earlier

Nevermind, I saw the other PR was already approved so I merged it

Made a draft because I found a bug with the enter handler

I found a bug that when you press enter multiple times, and try to delete the new lines sometimes it jumps back to the line you deleted, but I found the same bug in an empty editor so I guess this PR shouldn't be in charge of fixing that


Mark optimized abs as being nogil-safe.

I'm not entirely sure how to get the single test file to run, so I'm not sure if they're written correctlyy, but at least it compiles.
Fixes #2746.

Thanks. Looks like your changes broke the int_abs() and long_abs() tests. To run the builtin_abs tests separately, just pass that name to runtests.py as an argument, e.g.
python3 runtests.py --debug -vv builtin_abs
Regarding the implementation, I think it would be nicer if nogil=True was just a new keyword argument to BuiltinFunction, which means adding it as a constructor argument to the Signature class in TypeSlots.py, and build the appropriate CFuncType in Signature.function_type().

To run the builtin_abs tests separately, just pass that name to runtests.py as an argument, e.g.

Ah, I tried passing the filename instead.

Perfect! Thanks!


[Designer] [Adaptive Card 'Only Show When' ]

I am trying to use "Only Show When" in an Adaptive Card posted to Teams.
"${@{outputs('Post_adaptive_card_and_wait_for_a_response')?['body/data/choiceFRATRating']} = 1}"
The "choiceFRATRating" is either 1,2, or 3.
The card shows the Textblock regardless of value.
Thoughts?

Please reopen using our Designer issue template. Thanks!

Can you please be more specific?

When you open a new issue in our repo, you should see a list of options that looks like the below. Please select the Designer bug template and fille in the required information. That helps us understand and triage the issue. Thanks!


This assignment

Fixes errors where lines like this.foo = bar don't parse.
To test: use
npx tree-sitter test

The this assignment test should pass.

closing this branch bc the git rebase is messy 😓 . opening a new branch soon!


Texecom2mqqt armed status in HA

Good morning
New to HA and the Texecom2Mqqt addon
Everything seems to be working fine so far with exception to the alarm state, when setting the alarm, either full or part set, always showing as arming
Log from the addon is as follows, I've appended the state HA is showing at the end of each line, guess it's a simple mapping issue but when I tried to do it, the state always came back as unknown
2022-11-07 080243 - INFO Area A status changed to Disarmed - HA shows disarmed
2022-11-07 080323 - INFO Area A status changed to In Exit - HA shows arming
2022-11-07 080333 - INFO Area A status changed to Part Armed 1 - HA shows arming
2022-11-07 080415 - INFO Area A status changed to Disarmed - HA shows disarmed
2022-11-07 080422 - INFO Area A status changed to In Exit - HA shows arming
2022-11-07 080432 - INFO Area A status changed to Armed - HA shows arming
Thanks in advance form any help
Simon

Same here!
Any solution, please?
Thank you!

No Sorry
I stopped using the add on shortly afterwards
On Fri, 10 Feb 2023 at 17:08, settercoly @.***> wrote:

Same here!
Any solution, please?
Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/dchesterton/texecom2mqtt-hassio/issues/75#issuecomment-1426098835,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATIMTPHZQI6P7V7COGXWZHLWWZYZBANCNFSM6AAAAAARY5H6BA
.
You are receiving this because you authored the thread.Message ID:
@.***>

have you added the areas into the config?
- id: house
  name: House Alarm
  full_arm: armed_away
  part_arm_1: armed_night
  part_arm_2: armed_home
  part_arm_3: armed_custom_bypass
  code_arm_required: false
  code_disarm_required: false
- id: garden
  name: Garden Alarm
  full_arm: armed_away
  part_arm_1: armed_night
  part_arm_2: armed_home
  part_arm_3: armed_custom_bypass
  code_arm_required: false
  code_disarm_required: false


Update set_spec_const_default_value_test.cpp

Work around warning of dangling else

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


UserLocation passed children not rendered

Describe the bug
Pass a child into UserLocation isn't rendered.
To Reproduce
I've tried several iterations, however I believe this is how the api is supposed to be used?
Correct me if I'm wrong:
<MapboxGL.UserLocation
  renderMode="custom"
  visible
  animated
  onUpdate={myUserLocationUpdateCallback}
>
  <View style={{ height: 10, width: 10, backgroundColor: 'red' }} />
</MapboxGL.UserLocation>

Expected behavior
Passed child is rendered
Versions:

Platfrom: Android
Device: LG Q7
OS: Android 8
React Native Version [0.61.2]

Additional context
Maybe I'm supposed to pass it in another way or set other props correctly?

Hey,
native components such as views can't be passed as children inside the MapView. Only mapbox-gl-native components can.
I'm guessing you want to pass in a symbol layer with an icon or something. If that is the case, you can do something like:
<MapboxGL.UserLocation renderMode={'custom'}>
  <MapboxGL.SymbolLayer
    id={'custom-user-symbol'}
    style={{
      iconImage: 'your-icon',
      iconRotationAlignment: 'map',
      iconAllowOverlap: true,
    }}
  />
</MapboxGL.UserLocation>

your-icon needs to be loaded like any other mapbox icon.

Ah, ok, I was confused by the types I guess: https://github.com/react-native-mapbox-gl/maps/blob/master/index.d.ts#L407
I will try that out and report back.
Thanks you kristfal

Can confirm, that it works like you mentioned 👍
My example:
<MapboxGL.UserLocation
  renderMode={'custom'}
  onUpdate={this.onUserLocationUpdate}
>
  <MapboxGL.SymbolLayer
    id="myMarker"
    style={myStyles}
    aboveLayerID="myAboveLayer"
  />
</MapboxGL.UserLocation>

Great. Btw, happy to see a tighter typescript definition for this if you
have time for a PR :)
On Fri, 8 Nov 2019 at 12:35, Kid Commit<EMAIL_ADDRESS>wrote:

Can confirm, that it works like you mentioned 👍
My example:
<MapboxGL.UserLocation
renderMode={'custom'}
onUpdate={this.onUserLocationUpdate}

<MapboxGL.SymbolLayer
id="myMarker"

style={myStyles}

aboveLayerID="myAboveLayer"

/>
</MapboxGL.UserLocation>
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/react-native-mapbox-gl/maps/issues/511?email_source=notifications&email_token=ABLAPW4GJNGJUXXDGV3VHJ3QSVFJFA5CNFSM4JJVGIO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDQUKXA#issuecomment-551634268,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABLAPW2FWTSN4FJWLUNXY6TQSVFJFANCNFSM4JJVGIOQ
.

yeah, that and I would love to update the docs for the new apis.
Just need to find the time :(

There is no way to make custom style with MapboxGL.SymbolLayer ? for example make circle view in special size and background image ?

Hate to beat a dead horse but...
Is 'custom' renderMode no longer supported? It is no longer in the docs and this recent issue only refers to two modes (nomal/native): https://github.com/react-native-mapbox-gl/maps/issues/869

Hate to beat a dead horse but...
Is 'custom' renderMode no longer supported? It is no longer in the docs and this recent issue only refers to two modes (nomal/native): #869

What exactly do you mean by "custom" - if you're talking about passing children to the UserLocation component, that's still supported.

What exactly do you mean by "custom" - if you're talking about passing children to the UserLocation component, that's still supported.

Yes, I am referring to an example like this:
<MapboxGL.UserLocation
  renderMode={'custom'}
  onUpdate={this.onUserLocationUpdate}
>
  <MapboxGL.SymbolLayer
    id="myMarker"
    style={myStyles}
    aboveLayerID="myAboveLayer"
  />
</MapboxGL.UserLocation>

When I try to implement a custom marker in version 8.2.0-beta1, I get no marker appearing (no crash or error thrown either), setting a symbolLayer with sourceLayer wrapping it instead of userLocation shows the image source just fine, also setting 'normal' for UserLocation with no children shows the default user location marker as expected.
The docs also do not state a 'custom' renderMode option: https://github.com/react-native-mapbox-gl/maps/blob/8.2.0-beta1/docs/UserLocation.md

The custom flag was removed as it's a bit superfluous - as soon as a child is passed it should take precedence over the default icon.
See relevant code:
https://github.com/react-native-mapbox-gl/maps/blob/a1a95722b4a7f3da35ef48e258785fc79a02b39a/javascript/components/UserLocation.js#L263-L275
So this:
<MapboxGL.UserLocation
  onUpdate={this.onUserLocationUpdate}
>
  <MapboxGL.SymbolLayer
    id="myMarker"
    style={myStyles}
    aboveLayerID="myAboveLayer"
  />
</MapboxGL.UserLocation>

"should" work - I can have a look, however earliest on Friday

Thanks in advance for verifying, I will take a look at my code again, perhaps it's on my end.
From the source of UserLocation it also looks like I cannot set my own coordinates in situations where I want to override the position (like it being passed via props, I suppose in this case I would need to extend the UserLocation component and overwrite the _onLocationUpdate function

This works for us to display custom location icons as children:
				<Mapbox.UserLocation
					visible={true}
					ref={mainMapUserLocationRef}
					children={[
						<Mapbox.SymbolLayer
							key="symbolLocationSymbol"
							id="symbolLocationSymbol"
							minZoomLevel={1}
							style={
								Platform.OS === 'ios' ? (
									{
										iconImage: locationIconTracking,
										iconAllowOverlap: true,
										iconRotate: userHeading - mapHeading,
										iconSize: 0.08
									}
								) : (
									{
										iconImage: locationIconTrackingAndroid,
										iconAllowOverlap: true,
										iconRotate: 0,
										iconSize: 0.1
									}
								)
							}
						/>
					]}
				/>

Just to tie this altogether. I now know why I was not getting the custom icon.
The culprit was including this prop: renderMode={'custom'}.
Basically I was following the old docs which expected 'custom'. However in the latest version of the library, this makes the userLocation marker simply disappear.
Changing renderMode to 'normal' or removing the prop completely solves the problem.
Thank you

Does anyone here know if it's possible to keep the blue user location dot but add a custom userHeadingIndicator?

@bryanboyko
<MapboxGL.UserLocation renderMode="native" showsUserHeadingIndicator />


nixos/grafana: unbreak and update to 2.6.0 (by cherry-picking from master)

This unbreaks the grafana service and updates grafana from 2.0.0-beta1 to 2.6.0 (on NixOS 15.09).
The unbreaking part can be made with a single line, but I figure we'd want to run latest stable by default. (Also, nobody seems to be running grafana on NixOS 15.09 except me, as it has been broken for a while. Since september last year?) I had a small dashboard defined with 2.0.0-beta1 that survived the upgrade. Grafana release notes mention several breaking changes between releases in the 2.x series though:
https://github.com/grafana/grafana/blob/master/CHANGELOG.md
Fixes https://github.com/NixOS/nixpkgs/issues/12970
CC @offlinehacker.

:+1:

merged


BetterEnd crashing 1.20.1 on startup

Mod Name
BetterEnd
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/betterend
Minecraft version
1.20.1 (LTS)
Describe the bug
Crash on startup with BetterEnd.
Steps to reproduce

Get the mods i have.
Latest of everything
The game crashes
...

Logs
https://gist.github.com/axtrough/82f61bc4825f0ed326976144eec56d61
Additional context
No response

As our LTS support target, 1.20.1 now only receives critical fixes related to Connector bugs. Please upgrade to the latest version for mod compatibility support.


Make Position implement Comparable

For different sorting purposes.
@lukas-h wdyt?

Maybe we do that for CoordinateType. The performance / memory overhead won't be too bad, because the class is already really huge imo.