fix: manage BrokenPipeError thrown by MessageBus.connect()

With Home Assistant 2022.8.7 (and also 2022.8.2) I get:
hass[345]: 2022-08-27 18:00:52.289 ERROR (MainThread) [homeassistant.setup] Error during setup of component bluetooth
hass[345]: Traceback (most recent call last):
hass[345]:   File "/usr/lib/python3.10/site-packages/homeassistant/setup.py", line 235, in _async_setup_component
hass[345]:     result = await task
hass[345]:   File "/usr/lib/python3.10/site-packages/homeassistant/components/bluetooth/__init__.py", line 244, in async_setup
hass[345]:     elif await _async_has_bluetooth_adapter():
hass[345]:   File "/usr/lib/python3.10/site-packages/homeassistant/components/bluetooth/__init__.py", line 224, in _async_has_bluetooth_adapter
hass[345]:     return bool(await async_get_bluetooth_adapters())
hass[345]:   File "/usr/lib/python3.10/site-packages/homeassistant/components/bluetooth/util.py", line 19, in async_get_bluetooth_adapters
hass[345]:     adapters = await get_bluetooth_adapters()
hass[345]:   File "/var/lib/hass/deps/lib/python3.10/site-packages/bluetooth_adapters/__init__.py", line 21, in get_bluetooth_adapters
hass[345]:     bus = await MessageBus(
hass[345]:   File "/usr/lib/python3.10/site-packages/dbus_next/aio/message_bus.py", line 175, in connect
hass[345]:     self._stream.flush()
hass[345]:   File "/usr/lib/python3.10/socket.py", line 723, in write
hass[345]:     return self._sock.send(b)
hass[345]: BrokenPipeError: [Errno 32] Broken pipe
hass[345]: 2022-08-27 18:00:53.072 ERROR (SyncWorker_0) [homeassistant] Error doing job: Future exception was never retrieved
hass[345]: Traceback (most recent call last):
hass[345]:   File "/usr/lib/python3.10/site-packages/dbus_next/aio/message_bus.py", line 158, in on_hello
hass[345]:     raise err
hass[345]:   File "/usr/lib/python3.10/site-packages/dbus_next/aio/message_bus.py", line 365, in _message_reader
hass[345]:     if self._unmarshaller.unmarshall():
hass[345]:   File "/usr/lib/python3.10/site-packages/dbus_next/_private/unmarshaller.py", line 304, in unmarshall
hass[345]:     self._unmarshall()
hass[345]:   File "/usr/lib/python3.10/site-packages/dbus_next/_private/unmarshaller.py", line 243, in _unmarshall
hass[345]:     self.read(16, prefetch=True)
hass[345]:   File "/usr/lib/python3.10/site-packages/dbus_next/_private/unmarshaller.py", line 91, in read
hass[345]:     raise EOFError()
hass[345]: EOFError

This is on a Shenzhen Amediatech Technology Co., Ltd X96 Max running 5.18.1-1-aarch64-ARCH (Arch Linux ARM).
It has an AP6356S Bluetooth + wifi combo but I don't have bluetooth enabled (only bluez-libs are installed - none of the following is installed: bluez with the daemon, bluez-utils and bluez-tools).
Home Assistant still uses bluetooth-adapters version 0.1.3 but with the unit test I am able to reproduce the same issue on the latest version.
I think adding a fix in the bluetooth-adapters project is the right way. This is the first time I am troubleshooting (and trying to fix) Home Assistant though.

Thanks


filter_grep can't work with  to filter out records

I want to send nginx access log to elasticsearch and filter out which status is 200, so I configure the fluentd with configuration like:
<source>
  @type tail
  path /data/wwwlogs/net/access.log
  pos_file /var/log/td-agent/access.log.pos
  tag nginx.access
  format /^"(?<x_forward>[^\"]+)"\s-\s(?<user>\S+)\s\[(?<time>[^\]]*)\](?:\s+"(?<method>\S+))?(?:\s+(?<url>\S+))?\s+(?<http_version>\S+)\s+(?<status>[^\"]+)\s+(?<bytes>\d+)(?:\s"(?<rfc>.*)")?\s"(?<agent>.*)"\s(?<remote_addr>\S+)\s(?<time_spent>\S+).*$/
  time_format %d/%b/%Y:%H:%M:%S %z
</source>

<filter nginx.access>
  @type record_transformer
  remove_keys user,http_version,rfc,agent,bytes
  <record>
    setting_host "#{Socket.gethostname}"
  </record>
</filter>

<filter nginx.access>
  @type grep
  <exclude>
    key status
    pattern ^2\d\d$
  </exclude>
</filter>

omit match pattern...

it failed and send all records including which status is 200, logs like:
2017-08-30 16:39:37 +0800 [warn]: parameter 'key' in <exclude>
  key status
  pattern ^2\d\d$
</exclude> is not used.
2017-08-30 16:39:37 +0800 [warn]: parameter 'pattern' in <exclude>
  key status
  pattern ^2\d\d$

but if I use excludeN instead of <exclude> like:
<filter nginx.access>
  @type grep
  exclude1 status 200
</filter>

it works!
so why did <exclude> fail while excludeN worked?

td-agent version: 0.12.36
OS: amazon linux 2014.09

Because it is available since fluentd v0.12.38: https://www.fluentd.org/blog/fluentd-v0.14.19-has-been-released

If you want to use new syntax, please update bundled fluentd to latest one.


feat/build_without_gpg

When you cannot install GPG on the machine on which you want to contribute to datafaker, your build fails. By giving some details on how to manage this case in the documentation and by providing a maven profile, it can ease the process for newcomers to the project.

lgtm, thanks for contribution


Inconsequent and incorrect date format displayed for historic runs

Describe the Bug with repro steps
Incorrect date format is displayed under Run History on LA resource, under Overview.
The invented "fm/em" is not something I've ever seen nor is it correct, since a 24-hour clock would be the only correct here, like it is in the Runs History view, on the left hand side in the attached picture.
Region is set to Sweden, language is English as seen on the picture.
Best regards,
What type of Logic App Is this happening in?
Consumption (Portal)
Which operating system are you using?
Windows
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON

Screenshots or Videos

Browser
Chrome Version 131.0.6778.140 (Official Build) (64-bit)
Additional context
No response

Hi @amploid, I'll have to look into why this is showing these fm/em values; however you can switch to UTC time by clicking the colomn header: Start Time (Local Time)
By default I think in this coumn row we have to 12 hour clock, which I can also look to see if it can be moved to 24 hr just based on our other UI

That would be awesome!
Yeah, switching to UTC only toggles the time, not the format. So it would be really nice to keep it consequent between the two tables/lists. And for Sweden and most of Europe (UK excluded), that would be the 24-hour format.
Really impressed by the response time here, thank you :-)


v1.0.6

Add support for Vue 2

By leveraging vue-demi, this package is now usable by both Vue 2 & 3
The vue2-validus project, which specifically supports Vue 2, will be deprecated

Bump minimist from 1.2.5 to 1.2.6
Bump jsdom from 16.4.0 to 16.7.0
Bump terser from 5.5.1 to 5.14.2
Bump minimatch from 3.0.4 to 3.1.2
Bump loader-utils from 1.4.0 to 1.4.2
Bump decode-uri-component from 0.2.0 to 0.2.2
Bump vitepress from 0.9.1 to 0.22.4

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (ef77b9f) compared to base (2259583).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #36   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          212       212
  Branches        55        55
=========================================
  Hits           212       212

Impacted Files
Coverage Δ

src/field.ts
100.00% <100.00%> (ø)

src/fieldGroup.ts
100.00% <100.00%> (ø)

src/validators.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Layout non-convergence when using 2 counters

MRE:
#import "@preview/rich-counters:0.1.0": *

#set heading(numbering: "1.1")
#let counter1 = richcounter(
  identifier: "counter1",
  inherited_levels: 1,
  inherited_from: heading
)

#let counter2 = richcounter(
  identifier: "counter2",
  inherited_from: counter1
)

= Heading 1
#(counter1.step)() #context{(counter1.display)("1.1")}

#(counter2.step)() #context{(counter2.display)("1.1")}

Thanks for catching this.
I tried to debug it but I don't know what goes wrong. I'm rather getting more and more confused what the layout engine is doing... without understanding how it works, I can't really understand in detail what's happening and hence what goes wrong.
If anybody has a basic understanding of the layout engine please let me know.

Okay breakthrough. It DOES converge within 10 attemps. :D
Now, based on what I've seen what the layout engine produces after 1,2,3, etc. attempts, my current arrogant standpoint is that the layout engine just works relatively inefficient, and hence needs more than 5 rounds for this comparatively simple code construct.
I guess I will have to speak to Typst "experts" to find out why the layout engine needs so many attempts for this, and if there is a way to make the layout engine more efficient (-> a lot of work I guess).

Here is a Discord discussion about the this: https://discord.com/channels/1054443721975922748/1291866041022021756

Issue fixed! I re-implemented the package with a completely new algorithm and even more functionality!
Now your example compiles perfectly. You just need to change to version to 0.2.0 and richcounter to rich-counter.
#import "@preview/rich-counters:0.2.0": *

#set heading(numbering: "1.1")
#let counter1 = rich-counter(
  identifier: "counter1",
  inherited_levels: 1,
  inherited_from: heading
)

#let counter2 = rich-counter(
  identifier: "counter2",
  inherited_from: counter1
)

= Heading 1
#(counter1.step)() #context{(counter1.display)("1.1")}

#(counter2.step)() #context{(counter2.display)("1.1")}

I hope the package will be useful to you!
PS: If you only only want to inherit from the heading counter, I'm currently in the process of publishing a second package which is more tailored to exactly that purpose, and hence more efficient and simple.


Note internal route does not support paths

Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

A short explanation of the proposed change:

Add a note mentioning internal routes do not support paths in route documentation.

[x] I have reviewed the contributing guide

[x] I have viewed, signed, and submitted the Contributor License Agreement

[x] I have made this pull request to the master branch

[ ] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

Thanks for updating the docs!
Eli && @lisamcho


Support Request:

Describe Your Problem:
Hi there, firstly thank you for the hard work. I set everything up and it logs into my account and identifies my J9+ robots but then says "[iRobot] Failed to connect roomba: Arthur with error: No Roomba Found With Blid:*********". Any ideas what I am going wrong? Thanks again.
Expected behavior:
No response
Logs:
[9/7/2024, 5:06:04 PM] [iRobot] Initializing iRobotPlatform platform...
[9/7/2024, 5:06:04 PM] [Nest] Fetching Nest devices.
[9/7/2024, 5:06:04 PM] [iRobot] Logging into iRobot...
[9/7/2024, 5:06:06 PM] [iRobot] Configuring roomba: Robbie
[9/7/2024, 5:06:16 PM] [iRobot] Failed to connect roomba: Robbie with error: No Roomba Found With Blid: ***********

[9/7/2024, 5:06:16 PM] [iRobot] Configuring roomba: Jarvis
[9/7/2024, 5:06:26 PM] [iRobot] Failed to connect roomba: Jarvis with error: No Roomba Found With Blid: ***********

[9/7/2024, 5:06:26 PM] [iRobot] Configuring roomba: Winston
[9/7/2024, 5:06:37 PM] [iRobot] Failed to connect roomba: Winston with error: No Roomba Found With Blid: ***********

[9/7/2024, 5:06:37 PM] [iRobot] Configuring roomba: Arthur
[9/7/2024, 5:06:47 PM] [iRobot] Failed to connect roomba: Arthur with error: No Roomba Found With Blid: ***********

Config:
[9/7/2024, 5:06:04 PM] [iRobot] Initializing iRobotPlatform platform...
[9/7/2024, 5:06:04 PM] [Nest] Fetching Nest devices.
[9/7/2024, 5:06:04 PM] [iRobot] Logging into iRobot...
[9/7/2024, 5:06:06 PM] [iRobot] Configuring roomba: Robbie
[9/7/2024, 5:06:16 PM] [iRobot] Failed to connect roomba: Robbie with error: No Roomba Found With Blid: ***********

[9/7/2024, 5:06:16 PM] [iRobot] Configuring roomba: Jarvis
[9/7/2024, 5:06:26 PM] [iRobot] Failed to connect roomba: Jarvis with error: No Roomba Found With Blid: ***********

[9/7/2024, 5:06:26 PM] [iRobot] Configuring roomba: Winston
[9/7/2024, 5:06:37 PM] [iRobot] Failed to connect roomba: Winston with error: No Roomba Found With Blid: ***********

[9/7/2024, 5:06:37 PM] [iRobot] Configuring roomba: Arthur
[9/7/2024, 5:06:47 PM] [iRobot] Failed to connect roomba: Arthur with error: No Roomba Found With Blid: ***********

[9/7/2024, 5:06:47 PM] [iRobot] Not creating an accessory for unreachable Roomba: Robbie
[9/7/2024, 5:06:47 PM] [iRobot] Not creating an accessory for unreachable Roomba: Jarvis
[9/7/2024, 5:06:47 PM] [iRobot] Not creating an accessory for unreachable Roomba: Winston
[9/7/2024, 5:06:47 PM] [iRobot] Not creating an accessory for unreachable Roomba: Arthur
Setup Payload:
X-HM://00253G1G8ISG4

Node Version:
20.17.0
NPM Version:
npm -v
Homebridge Version:
Homebridge v1.8.4 (HAP v0.12.2) (Homebridge 2F83)
Plugin Version:
npm list -g homebridge-irobot-v2
Operating System:
MacOS

Is your Homebridge on a different network than the roombas?

Actually now that you mention it possibly as my Mac is on a VPN but the roomba are not. I take it that’s a deal breaker? ThanksStephen Sent from my iPhoneOn 7 Sep 2024, at 19:31, Thomas Theunen @.***> wrote:
Is your Homebridge on a different network than the roombas?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Yes, the connection is made over the local network using MQTT.

Thanks. I’ll change on this end. My Best, Stephen Sent from my iPhoneOn 8 Sep 2024, at 02:21, Thomas Theunen @.***> wrote:
Yes, the connection is made over the local network using MQTT.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


functional/prefer-readonly-type false positive on tuple types

Bug Report
Given the following code, ESLint reports functional/prefer-readonly-type rule:
type A = Readonly<[string]>;

Expected behavior
No error should be reported as the non-readonly tuple is never actually created since it is immediately being cast into a readonly tuple.
Actual behavior
Autofix replaces the above with:
type A = Readonly<readonly [string]>;

Proposed changes
Don't report the rule in this case, or disable auto fix, or remove Readonly<> when adding readonly

For reference, the Readonly<> syntax is preferred because it can be used for the following:
export interface Conformation extends Readonly<[number, ...Conformations]> {}

Where as equivalent using readonly [] is a syntax error:
export interface Conformation extends readonly [number, ...Conformations] {}

We currently don't have support for Readonly (see #51).
I've been working on the next release which will add support for this.


RenderDoc failed to launch a glProgramBinary sample.

Description
Attempt to use RenderDoc to capture an OpenGL sample but failed. The application is a sample of glProgramBinary, https://github.com/g-truc/ogl-samples/blob/master/samples/gl-410-program-binary.cpp. When launching with RenderDoc it always failed after glProgramBinary & glGetProgramiv, the "glGetProgramiv(ProgramName[program::VERT], GL_LINK_STATUS, &Success);" Success is GL_FALSE. It's fine if the application runs without RenderDoc.
Steps to reproduce

Clone and build https://github.com/g-truc/ogl-samples
Manually delete \ogl-samples\data\gl-410*.bin and run gl-410-program-binary.exe to re-generate the binary.
Launch gl-410-program-binary.exe with RenderDoc, the graphical window fails to show and can't be launched.

Environment

RenderDoc version: v1.24 (e5e68ef2)
Operating System: Win10
Graphics API: OpenGL
GPU: RTX2060
Driver: 527.86 (Nvidia Vulkan Beta driver from https://developer.nvidia.com/vulkan-driver)

This is not a RenderDoc bug - the program you've mentioned seems to fatal error if a program binary is rejected by GL, but it is legal for any GL implementation to reject a program binary for any reason at any time. That is the behaviour you're seeing.
I would recommend that any program using program binaries have a fallback to upload glsl if the program binary fails to upload.

Thank you for the explanation, so it's legal to fail to launch the program. I want to know if RenderDoc is able to view the shader source for glProgramBinary cases, could you tell me that directly?

The program launches fine, that doesn't fail. The program then has a fatal error when the program binary fails to load - that's fine but is entirely up to the program's logic and it would be quite possible to fall back to uploading shader source.
Program binaries are forced to fail precisely because there is no shader source to process, so the question doesn't really make sense.


pip installation from pypi throws a tensorflow dependency error

Describe the bug
Hi,
Following https://deepforest.readthedocs.io/en/latest/installation.html, I tried to install DeepForest with pip (pypi) on a fresh Ubuntu install, but I get this error message:
ERROR: Could not find a version that satisfies the requirement tensorflow==1.14.0 (from deepforest) (from versions: 2.2.0rc1, 2.2.0rc2, 2.2.0rc3, 2.2.0rc4, 2.2.0) ERROR: No matching distribution found for tensorflow==1.14.0 (from deepforest)
Even after installing Tensorflow with pip install tensorflow I get the same error.  It makes sense since the version numbers don't match.
Thank you for your help.
To Reproduce
Steps to reproduce the behavior:
pip install DeepForest
Expected behavior
Successful DeepForest installation with pip.
Desktop

OS: Ubuntu 20.04
Python: 3.8.2

Thanks for the well formatted report. I'll have a look today. I strongly
recommend using conda or a virtualenv and trying python 3.7. Too many
dependencies to manage otherwise. Tensorflow versions are getting released
fast these days and I'm guessing you are in front of them on linux 3.8.2.
conda create --name test python=3.7
conda activate test
pip install deepforest

works on the linux system I have right here.
This package is brand new, keep those issues coming on anything you see.
On Thu, Jun 18, 2020 at 12:45 AM jebins<EMAIL_ADDRESS>wrote:

Describe the bug
Following https://deepforest.readthedocs.io/en/latest/installation.html,
I tried to install DeepForest with pip (pypi) on a fresh Ubuntu install,
but I get this error message:
ERROR: Could not find a version that satisfies the requirement
tensorflow==1.14.0 (from deepforest) (from versions: 2.2.0rc1, 2.2.0rc2,
2.2.0rc3, 2.2.0rc4, 2.2.0) ERROR: No matching distribution found for
tensorflow==1.14.0 (from deepforest)
Even after installing Tensorflow with pip install tensorflow I get the
same error. It makes sense since the version numbers don't match.
Thank you for your help.
To Reproduce
Steps to reproduce the behavior:
pip install DeepForest
Expected behavior
Successful DeepForest installation with pip.
Desktop

OS: Ubuntu 20.04
Python: 3.8.2

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/weecology/DeepForest/issues/131, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJHBLDYU4IGTDDSEUMGQFLRXHA3LANCNFSM4OBJ7MRQ
.

--
Ben Weinstein, Ph.D.
Postdoctoral Fellow
University of Florida
http://benweinstein.weebly.com/

I can reproduce
(base) [b.weinstein@login3 ~]$ conda create --name test python=3.8.2
WARNING: A conda environment already exists at '/home/b.weinstein/miniconda3/envs/test'
Remove existing environment (y/[n])? y

Collecting package metadata (current_repodata.json): done
Solving environment: done

## Package Plan ##

  environment location: /home/b.weinstein/miniconda3/envs/test

  added / updated specs:
    - python=3.8.2

The following packages will be downloaded:

    package                    |            build
    ---------------------------|-----------------
    python-3.8.2               |he5300dc_7_cpython        70.8 MB  conda-forge
    ------------------------------------------------------------
                                           Total:        70.8 MB

The following NEW packages will be INSTALLED:

  _libgcc_mutex      conda-forge/linux-64::_libgcc_mutex-0.1-conda_forge
  _openmp_mutex      conda-forge/linux-64::_openmp_mutex-4.5-0_gnu
  ca-certificates    conda-forge/linux-64::ca-certificates-2<IP_ADDRESS>-hecda079_0
  certifi            conda-forge/linux-64::certifi-2<IP_ADDRESS>-py38h32f6830_0
  ld_impl_linux-64   conda-forge/linux-64::ld_impl_linux-64-2.34-h53a641e_5
  libffi             conda-forge/linux-64::libffi-3.2.1-he1b5a44_1007
  libgcc-ng          conda-forge/linux-64::libgcc-ng-9.2.0-h24d8f2e_2
  libgomp            conda-forge/linux-64::libgomp-9.2.0-h24d8f2e_2
  libstdcxx-ng       conda-forge/linux-64::libstdcxx-ng-9.2.0-hdf63c60_2
  ncurses            conda-forge/linux-64::ncurses-6.1-hf484d3e_1002
  openssl            conda-forge/linux-64::openssl-1.1.1g-h516909a_0
  pip                conda-forge/noarch::pip-20.1.1-py_1
  python             conda-forge/linux-64::python-3.8.2-he5300dc_7_cpython
  python_abi         conda-forge/linux-64::python_abi-3.8-1_cp38
  readline           conda-forge/linux-64::readline-8.0-hf8c457e_0
  setuptools         conda-forge/linux-64::setuptools-47.3.1-py38h32f6830_0
  sqlite             conda-forge/linux-64::sqlite-3.30.1-hcee41ef_0
  tk                 conda-forge/linux-64::tk-8.6.10-hed695b0_0
  wheel              conda-forge/noarch::wheel-0.34.2-py_1
  xz                 conda-forge/linux-64::xz-5.2.5-h516909a_0
  zlib               conda-forge/linux-64::zlib-1.2.11-h516909a_1006

Proceed ([y]/n)? y

Downloading and Extracting Packages
python-3.8.2         | 70.8 MB   | ############################################################################################################################################################### | 100%
Preparing transaction: done
Verifying transaction: done
Executing transaction: done
#
# To activate this environment, use
#
#     $ conda activate test
#
# To deactivate an active environment, use
#
#     $ conda deactivate

(base) [b.weinstein@login3 ~]$ conda activate test
(test) [b.weinstein@login3 ~]$ pip install deepforest
Collecting deepforest
  Downloading deepforest-0.2.19-cp38-cp38-manylinux2010_x86_64.whl (9.3 MB)
     |████████████████████████████████| 9.3 MB 4.3 MB/s
Collecting scipy
  Downloading scipy-1.4.1-cp38-cp38-manylinux1_x86_64.whl (26.0 MB)
     |████████████████████████████████| 26.0 MB 47.3 MB/s
Collecting pyyaml
  Using cached PyYAML-5.3.1.tar.gz (269 kB)
Collecting six
  Using cached six-1.15.0-py2.py3-none-any.whl (10 kB)
Collecting matplotlib
  Downloading matplotlib-3.2.2-cp38-cp38-manylinux1_x86_64.whl (12.4 MB)
     |████████████████████████████████| 12.4 MB 24.2 MB/s
Processing ./.cache/pip/wheels/80/dd/ac/842235b63dddac12faa4b48ebe58b8944e8c2e57c2e38dddb6/keras_resnet-0.1.0-py2.py3-none-any.whl
Collecting xmltodict
  Using cached xmltodict-0.12.0-py2.py3-none-any.whl (9.2 kB)
Collecting slidingwindow
  Using cached slidingwindow-0.0.13-py2.py3-none-any.whl (8.5 kB)
Collecting h5py
  Downloading h5py-2.10.0-cp38-cp38-manylinux1_x86_64.whl (2.9 MB)
     |████████████████████████████████| 2.9 MB 82.6 MB/s
Collecting keras==2.3.0
  Using cached Keras-2.3.0-py2.py3-none-any.whl (377 kB)
Collecting progressbar2
  Downloading progressbar2-3.51.3-py2.py3-none-any.whl (51 kB)
     |████████████████████████████████| 51 kB 32 kB/s
Collecting opencv-python
  Using cached opencv_python-<IP_ADDRESS>-cp38-cp38-manylinux1_x86_64.whl (28.2 MB)
ERROR: Could not find a version that satisfies the requirement tensorflow==1.14.0 (from deepforest) (from versions: 2.2.0rc1, 2.2.0rc2, 2.2.0rc3, 2.2.0rc4, 2.2.0)
ERROR: No matching distribution found for tensorflow==1.14.0 (from deepforest)

The good news is that our conda installation is soon to be up and running, but definitely we will add a ubuntu max python version to the setup.py. Thanks for finding this.

I think what is happening is that the current structure is just installation tensorflow to run tests.
  run:
    - python
    - matplotlib-base
    - sphinx
    - recommonmark
    - pyyaml
    - pytest
    - numpydoc
    - cython >=0.28
    - h5py
    - xmltodict
    - tqdm
    - {{ pin_compatible('numpy') }}
    - pillow
    - sphinx_rtd_theme
    - pandas
    - twine
    - rasterstats
    - psutil

test:
  imports:

  source_files:
    - tests/*
    - requirements.txt

The requirements.txt is being specified here as test requirement.
https://conda-forge.org/docs/maintainer/adding_pkgs.html#test-requirements
not as a build requirement. @henrykironde you mentioned it failed when moving to the build section, can you give any details?

Thanks for your feedback.
I successfully installed DeepForest with Python 3.7.7 both in conda and pyenv + venv .
I'll be happy to do further tests to help for the next DeepForest and Python versions.
Cheers

Great did you mean that you installed from Conda install deepforest  -c
conda-forge? If so what operating system, no problems? That’s new as of
last night. Or did you mean in a conda environment new pip install.
Thanks for letting me know, still trying to iron out all the different ways
we can make this better for a wide diversity of users.
On Sun, Jun 28, 2020 at 3:59 PM jebins<EMAIL_ADDRESS>wrote:

Thanks for your feedback.
I successfully installed DeepForest with Python 3.7.7 both in conda and
pyenv + venv .
I'll be happy to do further tests to help for the next DeepForest and
Python versions.
Cheers
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/weecology/DeepForest/issues/131#issuecomment-650833790,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJHBLCEGMWSQXFSAVUT4O3RY7DMPANCNFSM4OBJ7MRQ
.

--
Ben Weinstein, Ph.D.
Postdoctoral Fellow
University of Florida
http://benweinstein.weebly.com/

Great did you mean that you installed from Conda install deepforest -c conda-forge? If so what operating system, no problems? That’s new as of last night. Or did you mean in a conda environment new pip install. Thanks for letting me know, still trying to iron out all the different ways we can make this better for a wide diversity of users.
On Sun, Jun 28, 2020 at 3:59 PM jebins @.***> wrote: Thanks for your feedback. I successfully installed DeepForest with Python 3.7.7 both in conda and pyenv + venv . I'll be happy to do further tests to help for the next DeepForest and Python versions. Cheers — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#131 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJHBLCEGMWSQXFSAVUT4O3RY7DMPANCNFSM4OBJ7MRQ .
-- Ben Weinstein, Ph.D. Postdoctoral Fellow University of Florida http://benweinstein.weebly.com/

I installed using pip in a new conda environment.
Didn't know about conda install deepforest  -c conda-forge

Thanks.
Its a work in progress, still undocumented.


Expand the introduction section

Write more about me. For example:

My academic background,
My current projects I'm working on,
My career outlook,

FIxed by #8


Forward-merge branch-0.34 to branch-0.35

Forward-merge triggered by push to branch-0.34 that creates a PR to keep branch-0.35 up-to-date. If this PR is unable to be immediately merged due to conflicts, it will remain open for the team to manually merge.

SUCCESS - forward-merge complete.


removed deprecated bats

The used version of Bats is deprecated and no longer maintained. Also
there is an offical image of bats-core, bats/bats. Thus this image is no
longer needed.

@kr1sp1n Please review


fix from_hex method. The description of method say that the leading hash...

... is optional, and it was not. Now it is optional and return a valid color instance

wait for sass approval


Upload or Download Full video - Any Folder Destination - /saved/videos

Allow user to upload video, download full video by video URL or download by youtube-dl for any folder destination at /saved/videos

[ ] Upload video anywhere at folder directory - /saved/videos
[ ] Drop-down button - taking create folder button place - /saved/videos
[ ] Create folder button - available at Drop-down - /saved/videos
[ ] Input video URL for full download button - available at Drop-down - /saved/videos
[ ] Send directory plus video data to back-end
[ ] Download full video using downloadVideo function - ffmpeg-download-video.js
[ ] Download upload video using uploadVideoFile, downloadUploadedVideo functions - ffmpeg-upload-video.js
[ ] If specified download directory no longer exits after download is completed, post to main directory

Issue for another day


Fix separator regex to disallow empty strings

It looks like the current expression accepts an empty string as a valid separator.
Context: I was attempting to use this regex in a different language and it was hanging on certain inputs. I'm guessing that allowing empty string as a separator here causes pathological behaviors in some regex implementations. Digging around related context, it looks like the intention here was to allow for one or more dashes, and not to accept empty strings.
At the moment, separator is only used here:
pathComponent = alphanumeric + anyTimes(separator+alphanumeric)

But since alphanumeric is defined as [a-z0-9]+, anything that matched with the empty string behavior should still match.
Signed-off-by: Chris K. Wong<EMAIL_ADDRESS>

Codecov Report
Base: 56.56% // Head: 56.56% // No change to project coverage :thumbsup:

Coverage data is based on head (d1c1868) compared to base (3629105).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3818   +/-   ##
=======================================
  Coverage   56.56%   56.56%
=======================================
  Files         105      105
  Lines       10638    10638
=======================================
  Hits         6017     6017
  Misses       3948     3948
  Partials      673      673

Impacted Files
Coverage Δ

reference/regexp.go
100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@milosgajdos PTAL


t3c will ensure config files are owned by ats

Closes: #6879

Which Traffic Control components are affected by this PR?

Traffic Control Cache Config (t3c, formerly ORT)

What is the best way to verify this PR?

Build and install rpm, change file owner and group on some config files, queue updates on the cache, run t3c apply in syncds or badass, and verify owner and group have been set back to ats.
PR submission checklist

[ ] This PR has tests
[ ] This PR has documentation
[x] This PR has a CHANGELOG.md entry
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

To test this I made a copy of t3c-diff's README, but it doesn't seem to detect that it doesn't have the right owner:
$ stat ./README.md
  File: ./README.md
  Size: 3471      	Blocks: 8          IO Block: 4096   regular file
Device: 10303h/66307d	Inode: 17697181    Links: 1
Access: (0664/-rw-rw-r--)  Uid: ( 1000/ocket8888)   Gid: ( 1000/ocket8888)
Access: 2022-06-08 12:55:22.414520586 -0600
Modify: 2022-06-08 12:55:22.414520586 -0600
Change: 2022-06-08 12:55:22.414520586 -0600
 Birth: 2022-06-08 12:55:22.414520586 -0600
$ ./t3c-diff -l '<!--' -m 0664 -u 0 -g 0 -a ./README.md -b ./README.new.md
$ echo $?
0

the default value if left blank is 0
uid := getopt.IntLong("file-uid", 'u', 0, "User id the file being checked should have, default is running process's uid")
gid := getopt.IntLong("file-gid", 'g', 0, "Group id the file being checked should have, default is running process's gid")
if the value is 0 it will use the running process's uid/gid
if *uid == 0 { 		*uid = os.Geteuid() 	}
if *gid == 0 { 		*gid = os.Getgid() 	}
If you ran it as root it would find the difference. I could change the default value but it seemed logical to say that only the root user could say it should be owned by root.


Use binaries option to sign compiled node addons

How can the binaries option be used to sign compiled node addons? Right now, I have the option specified like this:
{
  binaries: "My App.app/Contents/Resources/app/node_modules/screen-recorder/build/Release/addon.node"
}

The signing and packaging process completes without errors, and can be uploaded to iTunes Connect, but it is later rejected because the addon.node is not signed.

Hi @jdp, please try to specify opts.binaries in an array of file paths like the following in current versions of this module. Thanks.
{
  binaries: ["My App.app/Contents/Resources/app/node_modules/screen-recorder/build/Release/addon.node"]
}

My mistake, I wrote that out by hand instead of copying and pasting. It is specified as an array in my script.

@jdp sorry for the late reply. It is better to have them in full paths, e.g. /Users/.... However, if doing so does not solve the problem, please try by adding a flag verbose: true to have some logs printed just to see if the listed binaries are signed during the automation.

@sethlu thanks for your help. I ended up making a small tweak to electron-osx-sign to automatically sign .node files in the same way .dylib files are automatically signed. Would you be open to a PR with that behavior included?

@kfranqueiro Looks like a handy enhancement. Actually I've also been thinking about signing the scripts,  they being recommended by Apple though not required. If you'd like to open a PR to https://github.com/sethlu/electron-osx-sign/tree/tweak-deep-sign and so I could then merge everything with https://github.com/electron-userland/electron-osx-sign/pull/26? Thanks.


Fix #53255 when a body gets stuck when it hits a descending platform in 2D/3D

fix #53255
This patch applies the vertical velocity of a downward platform when the body hit the ceiling. It also handles the option slide_on_ceiling by only applying the vertical velocity when it’s relevant.
Before:

After:

Does this PR also close https://github.com/godotengine/godot/issues/49387?


Introduce support for Reciprocal Rank Fusion (combining queries)

https://issues.apache.org/jira/browse/SOLR-17319
Description
Reciprocal Rank Fusion (RRF) is an algorithm that takes in input multiple ranked lists to produce a unified result set.
Examples of use cases where RRF can be used include hybrid search and multiple Knn vector queries executed concurrently.
RRF is based on the concept of reciprocal rank, which is the inverse of the rank of a document in a ranked list of search results.
The combination of search results happens taking into account the position of
the items in the original rankings, and giving higher score to items that are ranked higher in multiple lists. RRF was introduced the first time by Cormack et al. in [1].
The syntax proposed:
JSON Request
{
    "queries": {
        "lexical1": {
            "lucene": {
                "query": "id:(10^=2 OR 2^=1 OR 4^=0.5)"
            }
        },
        "lexical2": {
            "lucene": {
                "query": "id:(2^=2 OR 4^=1 OR 3^=0.5)"
            }
        }
    },
    "limit": 10,
    "fields": "[id,score]",
    "params": {
        "combiner": true,
        "combiner.upTo": 5,
        "facet": true,
        "facet.field": "id",
        "facet.mincount": 1
    }
}

[1] Cormack, Gordon V. et al. “Reciprocal rank fusion outperforms condorcet and individual rank learning methods.” Proceedings of the 32nd international ACM SIGIR conference on Research and development in information retrieval (2009)
Solution
The support has been introduced leveraging the "queries" support in the JSON request syntax.
The combination of results happen at QueryComponent, processing level.
The debug component has the responsibility of managing the explainability part.
Limitations:

grouping/field collapsing won't be supported in a first version
the distributed support is present but not super advanced: query combination happens at node level and then results are combined at shard level.

Tests
Tests have been added to cover the main use cases
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[x] I have added tests for my changes.
[ ] I have added documentation for the Reference Guide

As of today the solution is clean enough to be reviewed and tests are green.
The usage is intuitive and it should cover many use cases.
Pending activities:

add a new documentation page
add tests for the distributed scenario

I haven't looked at this in detail but I'm wondering could we instead have a single 'q' query parser, that is itself a "query combiner" of its children clauses? Ultimately this would mean not needing all this special code in QueryComponent. I'm trying to make recommendations that "fit in" well with Solr's architecture; less bolted / hacked on.
CC @hossman I think you'd have good inputs on this one too

In principle I agree, correct me if I'm wrong, your proposed solution means:

a Lucene contribution to create a new query
a Solr query parser to combine N queries into the new Lucene one?

I would need to think and design this possible solution, but I have no idea when this will happen as I am afraid it would take a significant amount of time. (of course anyone is happy to help).
Do you think it would make sense to start refining this contribution, bake it in so that users have this new possibility decently quickly and then iteratively replace it?
It's not because I have a strong opinion on this, I am just realistically think what could bring value from my side, in accordance with my future availabilty.
Happy to discuss!

I should have spent just another 10 minutes reading this to understand what's going on; I understand it better now. I had overlooked the query loop. I don't think a new Query would be that helpul for this use-case after-all. Even if one were to exist, somewhere there would be code to detect it and loop it so where would that go... either QueryComponent or SolrIndexSearcher which is another complexity minefield of greater magnitude. It's tempting to think of a QueryComponent subclass for this use-case (and for that matter grouping).
BTW I like the query syntax, especially the JSON variant.
Sorry for any distress; +1 overall.

No distress David! I'm not the kind of developer that get attached to its own solutions.
If there are better ways I'm always open to suggestions.
I also agree in principle with proposing ideas first, before spending days inplementing them, the only reason that I don't do that often is because I am not full time on Solr contributions right now, actually I end up at the moment with limited time to do them, so I prefer to produce something that could be valuable and iterate.
Having to wait for other volunteers time to discuss the idea could end up in exhausting the time I had allocated for the contrib and have to wait for the next availability window between clients.
Worst case scenario we end up with a discussion and we may ignore the Pull Request.
I remember at the beginning I tried query parsing and other approaches but the cleanest I got was the loop in the query component.
And I absolutely agree that component is messy, so after Berlin I'll spend a bit more time polishing the changes, and if in the meantime we come up with better ideas, the better!
No rush to merge, but at the same time if we get a decent consensus I would love to have this in as this feature will open many doors!
Thanks for the time you already spent on this, it's always very much appreciated!

I did the work of migrating to a QParser but don't have permissions to push to your PR branch.
Beware, I'm reverting most/all of the files that you modified of existing Solr source files.  If you would prefer I submit a separate PR, I'll do that.  Instead of combiner=true, you'd do q={!combine}.
Wouldn't the docSet (assuming it's needed, like for faceting) be computed on each subQuery, and if so how is that rolled up to the final QueryResult?  I see you tested that faceting works but I suspect there may be a very sad performance bug here.

I did the work of migrating to a QParser but don't have permissions to push to your PR branch.
Beware, I'm reverting most/all of the files that you modified of existing Solr source files. If you would prefer I submit a separate PR, I'll do that. Instead of combiner=true, you'd do q={!combine}.
I don't remember if this is the best way to invite you as a collaborator but I just added you on our repo, hopefully that works!

I wasn't entirely sure a Query Parser is the right place for this, given we are not really building a "new query" but then I thought : "hey we have the boolean query parser, that more or less does the same", so I have nothing against it!
You can go ahead and add your commit, I don't care much if you overwrote many of my changes, I just want a nice and clean solution, however we come up to it! Worst case scenario we go back in commits and re-use what's necessary!

Wouldn't the docSet (assuming it's needed, like for faceting) be computed on each subQuery, and if so how is that rolled up to the final QueryResult? I see you tested that faceting works but I suspect there may be a very sad performance bug here.

In regards to the docSet It was rolled up in:
SortedIntDocSet docSet = new SortedIntDocSet(combinedResultsDocIds, combinedResultsLength);     combinedRankedList.setDocSet(docSet);
combinedResultsDocIds is the merged array of doc Ids.
I admit I haven't spent the required time on this part and it was quickly done as a draft, so feel free to recommend a different way/correct me!

@cpoerschke see the parent JIRA issue for the real state of this controvertial PR

@cpoerschke see the parent JIRA issue for the real state of this controvertial PR

Thanks for the pointer! It inspired me to do some #2597 scribbling.


Address unused_qualifications lint overrides on JoinTestType enums in join_fuzz tests.

Describe the bug
When the unused_qualifications lint was added in Issue #13086 a lint allow was needed due to JoinTestType enums triggering the linter for datafusion/core/tests/fuzz_cases/join_fuzz.rs
It would be preferable to import JoinTestType in a way that does not need a linter override.
To Reproduce
Remove #[allow(unused_qualifications)] from datafusion/core/tests/fuzz_cases/join_fuzz.rs
Expected behavior
Linter should succeed without the override.
Additional context
No response

take


“GLTFQuickLook.qlgenerator” can’t be opened because Apple cannot check it for malicious software.

Get this error on Big Sur

I added an instruction to enable the plugin on Big Sur. Could you try it?

Run sudo xattr -r -d com.apple.quarantine /Library/QuickLook/GLTFQuickLook.qlgenerator or xattr -r -d com.apple.quarantine ~/Library/QuickLook/GLTFQuickLook.qlgenerator to allow GLTFQuickLook.qlgenerator to run.


The triggers section needs to be updated

There are triggers available for when an item is created or updated now. Would be nice to get some information on how logic apps is notified of changes. If it's query notifications, polling or other method.

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

ID: 5e190390-dc83-246e-3e69-189f33d67ed7
Version Independent ID: 66132130-9d94-fc33-7bc4-a54ee270efbc
Content: Add the Azure SQL Database connector in your Logic Apps
Content Source: articles/connectors/connectors-create-api-sqlazure.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan; ladocs

@Molotch Thanks for the feedback! We are currently investigating and will update you shortly.

Hi, were specifically interested in the SQL server trigger but the updated documentation shows it's polling based. Thanks!


TLS certificate for phpixie.com expired

The TLS certificate for phpixie.com expired Thu, 22 Dec 2022 07:43:18 GMT

Expired Again on (Dec 3rd 2023)


Pilot exposes metric pilot_virt_services but never sets it

Pilot has a metric pilot_virt_services, but it is never actually set. This is also in the dashboard so it is misleading.
I am fine with not having it if it isn't feasible due to some code changes or something, but we should remove it from the dashboard if that is the case.

Is there any downside in just reporting the number of virtual services known? For which gateway? The metric name does not say anything about that. I have used ISTIO_MESH_GATEWAY for now since it can be done with the existing code of the pushContext.
Any thoughts?


Incorporate the high-memory BWT implementation from fmlrc2

Hi @holtjma,
Sorry to bother you, and sorry if it is more issues and PR from me than from anyone in the recent days.
I believe this tool will greatly benefit from this addition.
I would like to try to see if we can incorporate the high-memory BWT implementation from fmlrc2 here. Before starting I prefer to have your recommendations and direction so that it will not be a waste of time for me and you. And if it is not something you would like for mswbt2, we can just leave it as it is.
Thanks!

It's no bother, though I am curious as to why the sudden interest in the crate.
Anyways, incorporating the exact implementation is likely tricky in the crate's current state; the main issue being that k-mer caching isn't really built into msbwt2 currently.  However, if you ignore caching, it should basically be a copy-paste-modify using a mixture of sources from msbwt2 and fmlrc2.  My suggestions for going about this:

Really familiarize yourself with what exactly the fmlrc2 BWT implementation is doing, both in code and concept.  You might also find useful info in the paper.
I suggest starting from the RLE BWT implementation for the initial copy-paste-modify.  That will get you the unit tests you need to pass.  It may also be worth reviewing the fmlrc2 implementation to see if any of those tests are worth adding.
Replace the corresponding RLE implementations with those from fmlrc2.  I note again that you'll likely need to remove/ignore anything related to caching for now.

That's the high level process I would use.  I just haven't had a need for it yet.

It's not so sudden interest for me :) This crate come to my interest in the last 4 months. I have tens of long and short reads transcriptomes and genomes to analyze and correct. It is really useful to have to type only one command to build the mswbt compared to the old version which is OS-dependent and more tedious in my opinion.
I had some free hours since yesterday so I tried to invest in the crate for my future works and hope to have at least help a bit. I don't know if I will have much free time in the coming days but we'll see how things are going.
For the incorporation of the high-memory BWT, I'll try to do my best when I can.
Thanks.
P.S. I'm also the person behind the Rust for Bioinformatics twitter profile and the community love your works!

Yea, that was initially the intent was to make building a bit more user friendly; granted it's also not as efficient as the ropebwt2 approach in its current state.  I just haven't had time to really theorize/implement faster versions given that this is currently a side-project (i.e. not directly work-related).
As for the high-mem impl., I'm happy to help review/iterate on that if you do decide to work on it!
Also, good to know the person behind the twitter handle haha


Bug: Yanked crate advisory gives false negative result if crate is in local cache

Describe the bug
Recently a url 2.5.3 crate was yanked and our CI rightfully started failing with:
error[yanked]: detected yanked crate (try `cargo update -p url`)

However running cargo deny check advisories locally was producing ok for all of our team members.
Only after rm -rf ~/.cargo/registry the problem became reproducible locally.
I suspect that code that check for crate presence finds it in the registry cache without checking the actual status of the crate in the source registry.
To reproduce
You could publish and yank a test crate in crates.io
cargo-deny version
0.16.2
What OS were you running cargo-deny on?
Linux
Additional context
No response

Thanks a lot for the explanation @Jake-Shadle. I think maximally relying on cargo makes total sense.
My fear was that if we add a more caching to our CI (which we plan on doing) it will also stop detecting yanked crates and dependency linting will give false negatives there too.
I wonder if adding cargo update --dry-run before cargo-deny would help, or the yanked crate will stay in the local cache.


changing render to children

Fixes #33
By naming the render prop 'children', allows both prop style and functional child style

Pull Request Test Coverage Report for Build 65

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

Totals

Change from base Build 64:
0.0%

Covered Lines:
175

Relevant Lines:
176

💛  - Coveralls

Pull Request Test Coverage Report for Build 65

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

Totals

Change from base Build 64:
0.0%

Covered Lines:
175

Relevant Lines:
176

💛  - Coveralls


add working tests

Workflow updated to use official ddev-add-on-test image (#7 )
Workflow disabled until meaning fulls added.


fix: use mint passive mode

closes #509

@ericmj @teamon all the tests are passing, so I am not sure if I should add a new test case or anything else to make sure this works.
Would appreciate some support over here! 🙏🏻

hah? all tests pass locally but failed in CI 😭

Might be a good to change these tests to use active mode since you changed the default: https://github.com/elixir-tesla/tesla/blob/master/test/tesla/adapter/mint_test.exs#L83.
Other than that I am not sure what I can help with?

hah, which leads me to a question, should I allow users to change this? Like should I use Map.put_new instead?

I don't see a reason why you would want active mode with with how tesla uses mint but I don't understand why you would want tesla to create the connection so there may be use cases I am missing.
But I don't think it hurts to leave it as an option with the default being passive mode.


Update non-windows.md

Removed DLP for the nonsupported section as macOS now supports Endpoint DLP - reference - https://docs.microsoft.com/en-us/microsoft-365/compliance/endpoint-dlp-getting-started?view=o365-worldwide

@JohanFreelancer9 Please copy review

@mjcaparas Please merge this PR. Thanks.

@msbemba Gentle reminder. Thanks

@msbemba please update to mac section not linux

@mjcaparas Sorry about that. I just fixed it.

Docs Build status updates of commit f203d60:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/non-windows.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit f203d60:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/non-windows.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


RDS validation check OPA

error creating RDS cluster: InvalidParameterValue: MasterUsername admin cannot be used as it is a reserved word used by the engine

Tracked here:
https://github.com/cds-snc/opa_checks/issues/12


Apply 2b patch updates

CI is expected to fail until patched base images are published.

@dleeapho - I added some readme and sample related changes to reference repos/images from MCR where applicable.


Remove _WIN32 guards on threadpool.c

The _WIN32 guards appear to be remnants of another time. Begone with them!
/cc @bnoordhuis @cjihrig @addaleax
Fixes: https://github.com/libuv/libuv/issues/2980
Refs: https://github.com/nodejs/node/pull/35021
Signed-off-by: James M Snell<EMAIL_ADDRESS>

I've added a guard, let's see how it fares: https://ci.nodejs.org/job/libuv-test-commit/2118/

I've added a guard, let's see how it fares: https://ci.nodejs.org/job/libuv-test-commit/2118/

Only known flakes.
@vtjnash Your feedback has been addressed (good feedback by the way!) so I'll take the liberty of dismissing the "request changes."

Only known flakes.
@vtjnash Your feedback has been addressed (good feedback by the way!) so I'll take the liberty of dismissing the "request changes."


Adding content and style to scale switcher

Type

[ ] Bug fix
[x] New feature
[ ] Improvement to existing feature

Related JIRA Issue(s)
ENSWBSITES-464
Description
This is to show the scale switcher which is a dummy component (no interaction at all and fixed for now). Also styling the element.
Views affected
Entity Viewer

Formatting issue:

I think our editors add an empty string to the end of the file by default. We probably should all do it.

All done, have a look @azangru


[KEP] Update KEP #993 - Add retry mechanism to AdmissionChecks

What type of PR is this?
/kind documentation

What this PR does / why we need it:
Propose changes to introduce retry mechanism for AdmissionChecks
Which issue(s) this PR fixes:

Part of #3258
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/assign @mimowo
cc @mwielgus @yaroslava-serdiuk

In my opinion we should let individual admission check controllers implement their own retry policy, not all ACCs need this and having another actor managing its ACStates makes room for conflicts or needs additional logic to avoid them.

In my opinion we should let individual admission check controllers implement their own retry policy, not all ACCs need this and having another actor managing its ACStates makes room for conflicts or needs additional logic to avoid them.

Thanks for the review Traian. I considered it, and change the proposal to cover only ProvisioningRequests

/cc

I took it a step further and moved the API to the ProvisioningRequestConfig API in #3278, so I'm closing this one.


Fix spelling of 'unofficial'

Hm, inofficial seems to be an acceptable adjective, isn't it? https://www.thefreedictionary.com/Inofficial
On the other hand we all speak German as first language, where the word is "Inoffiziell", so that might have influenced us :slightly_smiling_face:

@dbrgn I was surprised to find 'inofficial' in a dictionary online too. But most don't include it, such as this one.
Google doesn't know 'inofficial' either.

Thanks for all the pull requests @jeffwheeler! :smiley:


PLE1507 false positives

Hi :wave:
with latest version that adds PLE1507 I'm getting errors with a following usage. But I would say they are false positives.
import os

def dummy(env_prefix: str = ''):
    os.getenv(f'{env_prefix}NAME', '') # fail
    os.getenv(env_prefix + 'NAME', '') # fail

Yeah definitely.


Compute feature set scores by summing a low-rank representation.

Codecov Report

Merging #37 (e54b97c) into master (62df118) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master      #37      +/-   ##
==========================================
- Coverage   98.07%   98.06%   -0.01%
==========================================
  Files          85       85
  Lines        6530     6522       -8
  Branches      119      119
==========================================
- Hits         6404     6396       -8
  Misses         71       71
  Partials       55       55

Impacted Files
Coverage Δ

...clude/scran/dimensionality_reduction/pca_utils.hpp
95.83% <ø> (ø)

include/scran/dimensionality_reduction/RunPCA.hpp
94.11% <100.00%> (-0.62%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[Fix] Proper dot-annotation support for ValueMap#remove(name)

References how .set works
Needs review over behavior when value is not a map

CC @MineGame159 for review

Looks good but apart from one thing. Looks like you made it return this but in the context of a remove() method I would expect it to return the removed object or null if it couldnt remove anything. Any reason why you decided like this? Also the return value should be documented in javadoc.

As mentioned it was mostly based on how set was written so there isnt a need to have it return anything. In this case though, I can probably swap the fail cases to return null

As for the return value, again note the documentation was based on set which also doesnt document an @return value


Folder for stage

What is your question?
Is it possible to have a folder a stage because i need another stage for my mod and its getting messy

wait nvm i figured it out. you need to add "[folder name]/" to the image name.


Enhancement: Create a meanjs module manager

Will there be a way to submit modules and search through modules people have created. Plus maybe a doc for setting up minimum requirements(standard) for submission.

Unlikely happening, closing an old thread.