Stop installing unwanted deps when precompiling

closes https://github.com/membraneframework/membrane_core/issues/653

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/membraneframework/elixir?version=dev:52d620d89e7451d4772470b60c79ab61e16006fa

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/membraneframework/elixir?version=dev:7763cd1a4b436e1ade6d1aaf7d1705f960c06eb0

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/membraneframework/elixir?version=dev:b650aa37c7473aa3ccf5714a78de9c7cea4b9b6d

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/membraneframework/elixir?version=dev:4960918efd4ddd9c5744dfbf38af825cf3a04f46

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/membraneframework/elixir?version=dev:7423bc620d8d66fd45f09b7525ab11d2cc90b5d2

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/membraneframework/elixir?version=dev:b9331e40cd8834f2f19500b25cbd82600e64f20b

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/membraneframework/elixir?version=dev:b9331e40cd8834f2f19500b25cbd82600e64f20b

Your orb has been published to the CircleCI Orb Registry.
You can view your published orb on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/membraneframework/elixir?version=1.4.3


Function Send() not working - hardware seral atmega2560

Sorry for my not good English.
I migrate from old version of library to new so I can use software serial.
During migration I found that function Send() not working.
This is on hardware serial Serial1 on arduino mega.
Code stop at this function and waits until infinity.
I research proglem and function
waitForTransmitToComplete();
This function does flush of serial and here code "freezes"
void ICSC::waitForTransmitToComplete()
{
Serial.println("w 1");
if (_dePin == -1)  //Skip flush
return;
Serial.println("w 2");
#if defined(ARDUINO) && ARDUINO >= 100
#if ARDUINO >= 104
// Arduino 1.0.4 and upwards does it right
Serial.println("w 3");
_dev->flush();
Serial.println("w 4");
#else
// Between 1.0.0 and 1.0.3 it almost does it - need to compensate
// for the hardware buffer. Delay for 2 bytes worth of transmission.
Serial.println("w 5");
_dev->flush();
Serial.println("w 6");
delayMicroseconds((20000000UL/9600)+1);

Serial.println("w 7");
I can see control print Serial.println("w 5");
and not see Serial.println("w 6");
I think there is problem in line
_dev->flush();
Can I comment this line?
What is right way to continue use ICSC library?
Thank you.

Are you calling send from within an interrupt by any chance?

I've found that I have to add to my code the following line
Serial1. Begin (9600);
Before
ICSC. Begin ();
That's because ICSC. Begin does not initialize serial port
Now everything works fine.
Thanks


Why do some works have thumbnails, but others don't?

Quoting Branwen in Slack:

I was wondering why a thumbnail image sometimes appears on Works and other times not? We recently ingested a set of 10 Royal Botanic Gardens prints, but only one has a thumbnail - https://wellcomecollection.org/works/xfq9r7ym  (b32890783)
The others appear like this: https://wellcomecollection.org/works/u6snm6q6  (b32890345)

Okay, so I went to have a look at this.
b32890345 is a combination of Sierra physical bib and a METS work, which has two items:

a physical item, which has the access status "by appointment"
a digitised item, which has the access status "open"

The merger suppresses thumbnails if any of the items on the work have restrictions, but in this case I think it's being overzealous – although the physical item has restrictions, we should be fine to display a digitised thumbnail.
I think we should suppress thumbnails only if the digitised item (i.e. the METS work) has restrictions.
I'm going to leave this ticket for now, with two notes:

@jtweed should confirm if that's the correct behaviour
if it is the correct behaviour, this would be a good first merger patch for somebody

I think that's the correct behaviour, ie the thumbnail restrictions should come from the digitised work not the physical work.


Coap option tidy

Mbed-client-c has been refactored to be more memory efficient.
Changes in mbed-client-c has been updated to mbed-client side in this PR

rebuild

rebuild

rebuild

rebuild

rebuild

prkl


Adding showHours prop adds an extra letter in the time stamp

The showDuration prop works perfectly.

When I add the showHours prop, it shows this:

I could reproduce this.
Here is where we calculate the time: https://github.com/criszz77/react-native-media-console/blob/master/src/utils.ts#L45-L76
Should be an easy fix.

Well I will give it a shot! :)

It would be this right here, right? Should be a zero?

After you fork the repo and make your changes you will have to run yarn prepare. You can test it by running yarn add [path to the forked repo] in your app project (be sure to delete node_modules from forked repo). An easy way to get the path is to right click on the forked repo folder, press Option and keep it pressed, then select Copy react-native-media-console as Pathname.
If you run into issues, let me know, I'll be more than happy to help.

It would be this right here, right? Should be a zero where the "a]" is?
I tested it and it seems to work, however, I don't have a video that long to test 🙈

@criszz77 Awesome!!!

Haha thank you! I am very happy as well :D


Doesn't work with Linux, only Ubuntu Linux.

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{ R200 / F200 / SR300 / ZR300 / D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win (8.1/10) / Linux (Ubuntu 14/16/17) / MacOS

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC/Raspberry Pi/ NVIDIA Jetson / etc..

SDK Version
{ legacy / 2.. }

Language
{C/C#/labview/nodejs/opencv/pcl/python/unity }

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
<Describe your issue / question / feature request / etc..>
I've replaced the instances of generic with my kernel version for linux-headers in scripts/patch-realsense-ubuntu-lts.sh and it still reports Unsupported kernel version 4.19.0-8-amd64 . The patches are maintained for Ubuntu16 (Xenial) with LTS kernel versions 4.4, 4.8, 4.10, 4.13 and 4.15
Is there some reason for this, not supporting Linux and only supporting Ubuntu Linux?
I hate to install a distribution of Linux based on the one I normally use, Debian.

Dug deeper in #6651 will create a patch for Ubuntu once patch for 4.16.1 is fixed.

Hi @andrewhodel   I see that you closed the case, but I will share the answer that I was working on anyway.  :)
There is a "backend" installation method that requires an internet connection but is not dependent on Linux versions or kernel versions, and does not require patching.
First, download the source code of an SDK version by going to the 'Releases' page and downloading the 'Source code' zip file from the "Assets" file list at the bottom of each version's details.
https://github.com/IntelRealSense/librealsense/releases/

Once the source code zip for the SDK version that you want to install has been extracted from the ip folder, follow the instructions in the link below to build Librealsense without dependence on Linux, kernels or patches.
https://github.com/IntelRealSense/librealsense/issues/6368#issuecomment-636316279

https://github.com/IntelRealSense/librealsense/issues/6652
On Fri, Jun 19, 2020 at 11:35 AM MartyG-RealSense<EMAIL_ADDRESS>wrote:

Hi @andrewhodel https://github.com/andrewhodel I see that you closed
the case, but I will share the answer that I was working on anyway. :)
There is a "backend" installation method that requires an internet
connection but is not dependent on Linux versions or kernel versions, and
does not require patching.
First, download the source code of an SDK version by going to the
'Releases' page and downloading the 'Source code' zip file from the
"Assets" file list at the bottom of each version's details.
https://github.com/IntelRealSense/librealsense/releases/
[image: image]
https://user-images.githubusercontent.com/41145062/85163798-53727700-b25b-11ea-9a2b-bdb9506230b2.png
Once the source code zip for the SDK version that you want to install has
been extracted from the ip folder, follow the instructions in the link
below to build Librealsense without dependence on Linux, kernels or patches.
#6368 (comment)
https://github.com/IntelRealSense/librealsense/issues/6368#issuecomment-636316279
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/6650#issuecomment-646779983,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFVCSNGLFVISM4JRMCYT23RXOOW7ANCNFSM4OC36CYA
.

I just don't understand why your instructions on Github and on the Intel
SDK site are so terrible.
Why don't you just tell people to rebuild the kernel, apply the patches
needed and use the kernel they built?
Then it works with any Linux distribution or version that supports the
library versions (so 4.18.1+ or whatever).
Andrew
On Fri, Jun 19, 2020 at 12:05 PM Andrew Hodel<EMAIL_ADDRESS>wrote:

https://github.com/IntelRealSense/librealsense/issues/6652
On Fri, Jun 19, 2020 at 11:35 AM MartyG-RealSense <
<EMAIL_ADDRESS>wrote:

Hi @andrewhodel https://github.com/andrewhodel I see that you closed
the case, but I will share the answer that I was working on anyway. :)
There is a "backend" installation method that requires an internet
connection but is not dependent on Linux versions or kernel versions, and
does not require patching.
First, download the source code of an SDK version by going to the
'Releases' page and downloading the 'Source code' zip file from the
"Assets" file list at the bottom of each version's details.
https://github.com/IntelRealSense/librealsense/releases/
[image: image]
https://user-images.githubusercontent.com/41145062/85163798-53727700-b25b-11ea-9a2b-bdb9506230b2.png
Once the source code zip for the SDK version that you want to install has
been extracted from the ip folder, follow the instructions in the link
below to build Librealsense without dependence on Linux, kernels or patches.
#6368 (comment)
https://github.com/IntelRealSense/librealsense/issues/6368#issuecomment-636316279
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/6650#issuecomment-646779983,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFVCSNGLFVISM4JRMCYT23RXOOW7ANCNFSM4OC36CYA
.

You say this, Once the source code zip for the SDK version that you want to
install has been extracted from the ip folder, follow the instructions in
the link below to build Librealsense without dependence on Linux, kernels
or patches.
But this is not true.
The DKMS file for the kernel drivers that are patched (uvcvideo) don't
support Debian 10 for example, the kernel version is excluded in DKMS.conf
If you told people to just build a new kernel, there's no problem running
4.18 or 5.0+ on Debian 10
Your instructions are too high, they need to get back to understanding what
a kernel module is.
Andrew
On Fri, Jun 19, 2020 at 12:06 PM Andrew Hodel<EMAIL_ADDRESS>wrote:

I just don't understand why your instructions on Github and on the Intel
SDK site are so terrible.
Why don't you just tell people to rebuild the kernel, apply the patches
needed and use the kernel they built?
Then it works with any Linux distribution or version that supports the
library versions (so 4.18.1+ or whatever).
Andrew
On Fri, Jun 19, 2020 at 12:05 PM Andrew Hodel<EMAIL_ADDRESS>wrote:

https://github.com/IntelRealSense/librealsense/issues/6652
On Fri, Jun 19, 2020 at 11:35 AM MartyG-RealSense <
<EMAIL_ADDRESS>wrote:

Hi @andrewhodel https://github.com/andrewhodel I see that you closed
the case, but I will share the answer that I was working on anyway. :)
There is a "backend" installation method that requires an internet
connection but is not dependent on Linux versions or kernel versions, and
does not require patching.
First, download the source code of an SDK version by going to the
'Releases' page and downloading the 'Source code' zip file from the
"Assets" file list at the bottom of each version's details.
https://github.com/IntelRealSense/librealsense/releases/
[image: image]
https://user-images.githubusercontent.com/41145062/85163798-53727700-b25b-11ea-9a2b-bdb9506230b2.png
Once the source code zip for the SDK version that you want to install
has been extracted from the ip folder, follow the instructions in the link
below to build Librealsense without dependence on Linux, kernels or patches.
#6368 (comment)
https://github.com/IntelRealSense/librealsense/issues/6368#issuecomment-636316279
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/6650#issuecomment-646779983,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFVCSNGLFVISM4JRMCYT23RXOOW7ANCNFSM4OC36CYA
.

Yes, the process that I specified is designed for source code installation.  DKMS packages have their own installation process.
I do understand your frustration.  The instructions try to meet the needs of as wide an audience as possible on multiple types of computer / computing device, though there are cases where further support may be required to successfully complete an installation.

You are giving a wide audience instructions for effectively a single
platform, Ubuntu LTS.
Why don't you document how to do this the right way?
Andrew
On Fri, Jun 19, 2020 at 12:15 PM MartyG-RealSense<EMAIL_ADDRESS>wrote:

Yes, the process that I specified is designed for source code
installation. DKMS packages have their own installation process.
I do understand your frustration. The instructions try to meet the needs
of as wide an audience as possible on multiple types of computer /
computing device, though there are cases where further support may be
required to successfully complete an installation.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/6650#issuecomment-646801349,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFVCSISXZHNBA3N4TYQUPDRXOTLFANCNFSM4OC36CYA
.

Ubuntu is the most widely used platform, though we do provide options for other forms of Linux and create specialist solutions for platforms such as Raspberry Pi and Nvidia Jetson.
I cannot speak for the creators of the documentation, but I can assure that they try to do their best for everyone and give individual specialist support on the forum where necessary.
Thanks for your patience, it really is appreciated!

Can you at least list the patches in the scripts/ directory by kernel
versions, not distribution specific ones?
Andrew
On Fri, Jun 19, 2020 at 12:41 PM MartyG-RealSense<EMAIL_ADDRESS>wrote:

Ubuntu is the most widely used Linux platform, though we do provide
options for other forms of Linux and create specialist solutions for
platforms such as Raspberry Pi and Nvidia Jetson.
I cannot speak for the creators of the documentation, but I can assure
that they try to do their best for everyone and give individual specialist
support on the forum where necessary.
Thanks for your patience, it really is appreciated!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/6650#issuecomment-646811661,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFVCSJENZ76X73OZN7WNHDRXOWPJANCNFSM4OC36CYA
.

I am on the Support team and are not involved with development decisions, so I cannot comment on that point.  I recommend raising the points in your open case.  Thanks.

@andrewhodel hello,
I've followed the PR you submitted as I wasn't sure about its purpose or to whom it was addressed.
Here it seem to be the proper context for reply -

I've replaced the instances of generic with my kernel version for linux-headers in scripts/patch-realsense-ubuntu-lts.sh and it still reports Unsupported kernel version 4.19.0-8-amd64 . The patches are maintained for Ubuntu16 (Xenial) with LTS kernel versions 4.4, 4.8, 4.10, 4.13 and 4.15
Is there some reason for this, not supporting Linux and only supporting Ubuntu Linux?

That is not correct. Librealsense2 SDK and RealSense Depth cameras and is in no way limited for Ubuntu.They can and be deployed on any Linux distribution and platform that satisfy the following minimal requirements: Kernel 4.4+/ GCC ~4.8 and USB2/3 port according to the camera model.
Most of the camera and SDK functions become available immediately upon installation without patches  - Device enumeration, controls, data streaming.
The kernel patches are intended to enable advanced capabilities unique to Realsense devices, and also to fill kernel gaps for functionalities that are yet-to-be implemented or buggy. Still, the core feature that justifies the existence of those patches is the frame metadata.
As for the selected distribution - we maintain patches for Ubuntu LTS kernels in a form of script and also as a precompiled DKMS module. Both methods are functionally-equivalent but for obvious reasons we recommend DKMS as the preferred installation method (Documentation).
In addition to the patches provided by Intel for Ubuntu there are multiple patches that were generated and  provided by community. Unfortunately those tend to become obsolete as in most cases community members do not update them after the initial submissions.
And this is the core distinction - the officially-"maintained" patches are being updated continuously and reflect the up-to-date state of all the supported devices/required functionalities. So when the need arise they are used as a baseline for a new port by maintainers or community.

I just don't understand why your instructions on Github and on the Intel
SDK site are so terrible.

The online documentation is being checked and updated periodically. Can you provide specific examples?

Why don't you just tell people to rebuild the kernel, apply the patches
needed and use the kernel they built?
Then it works with any Linux distribution or version that supports the
library versions (so 4.18.1+ or whatever).

Because not only it is not precise, It is actually the is opposite of the message we convey:

We do not require the users to apply kernel patches. The users are free to experiment with the default settings, and for many users this is sufficient.
Ubuntu users have a certain advantage as they have the kernel patches/installation package ready for them already. But, as pointed above by @MartyG-RealSense , for users that don't use Ubuntu
the SDK provides a "driver-less" build. This configuration delivers all the "extra" functionality (metadata) without touching the kernel. Currently this is the recommended method for RaspberryPi and Tegra boards. Additionally we are open for collaboration and ready to provide technical assistance for users that port the patches to other kernels/distros.
For most* distributions it is unnecessary - the patches apply to several loadable modules so it is technically inefficient to rebuild the whole kernel to replace those 6-7  *.ko modules.
**There are several exceptions for this case, such as Tegra or Yocto distros, but they also have dedicated handling/installation guidance.

They have to apply the kernel patches to get it to work on distributions
other than Ubuntu.
I built all the scripts for linux vanilla and debian 10 and submitted a
pull request to the github project for realsense with them.
https://github.com/IntelRealSense/librealsense/pull/6653
Andrew
On Sat, Jun 20, 2020 at 1:29 PM ev-mp<EMAIL_ADDRESS>wrote:

@andrewhodel https://github.com/andrewhodel hello,
I've followed the PR you submitted
https://github.com/IntelRealSense/librealsense/pull/6653 as I wasn't
sure about its purpose or to whom it was addressed.
Here it seem to be the proper context for reply -
I've replaced the instances of generic with my kernel version for
linux-headers in scripts/patch-realsense-ubuntu-lts.sh and it still reports
Unsupported kernel version 4.19.0-8-amd64 . The patches are maintained for
Ubuntu16 (Xenial) with LTS kernel versions 4.4, 4.8, 4.10, 4.13 and 4.15
Is there some reason for this, not supporting Linux and only supporting
Ubuntu Linux?
That is not correct. Librealsense2 SDK and RealSense Depth cameras and is
in no way limited for Ubuntu.They can and be deployed on any Linux
distribution and platform that satisfy the following minimal requirements:
Kernel 4.4+/ GCC ~4.8 and USB2/3 port according to the camera model.
Most of the camera and SDK functions become available immediately upon
installation without patches - Device enumeration, controls, data streaming.
The kernel patches are intended to enable advanced capabilities unique to
Realsense devices, and also to fill kernel gaps for functionalities that
are yet-to-be implemented or buggy. Still, the core feature that justifies
the existence of those patches is the frame metadata
https://github.com/IntelRealSense/librealsense/blob/master/doc/frame_metadata.md
.
As for the selected distribution - we maintain patches for Ubuntu LTS
kernels in a form of script and also as a precompiled DKMS module. Both
methods are functionally-equivalent but for obvious reasons we recommend
DKMS as the preferred installation method (Documentation
https://github.com/IntelRealSense/librealsense/blob/master/doc/distribution_linux.md#linux-distribution
).
In addition to the patches provided by Intel for Ubuntu there are multiple
patches that were generated and provided by community. Unfortunately those
tend to become obsolete as in most cases community members do not update
them after the initial submissions.
And this is the core distinction - the officially-"maintained" patches are
being updated continuously and reflect the up-to-date state of all the
supported devices/required functionalities. So when the need arise they are
used as a baseline for a new port by maintainers or community.
I just don't understand why your instructions on Github and on the Intel
SDK site are so terrible.
The online documentation is being checked and updated periodically. Can
you provide specific examples?
Why don't you just tell people to rebuild the kernel, apply the patches
needed and use the kernel they built?
Then it works with any Linux distribution or version that supports the
library versions (so 4.18.1+ or whatever).
Because not only it is not precise, It is actually the is opposite of the
message we convey:

We do not require the users to apply kernel patches. The users are
free to experiment with the default settings, and for many users this is
sufficient.
Ubuntu users have a certain advantage as they have the kernel
patches/installation package ready for them already. But, as pointed above
by @MartyG-RealSense https://github.com/MartyG-RealSense , for users
that don't use Ubuntu
the SDK provides a "driver-less" build. This configuration delivers
all the "extra" functionality (metadata) without touching the kernel.
Currently this is the recommended method for RaspberryPi and Tegra boards.
Additionally we are open for collaboration and ready to provide technical
assistance for users that port the patches to other kernels/distros.
For most* distributions it is unnecessary - the patches apply to
several loadable modules so it is technically inefficient to rebuild the
whole kernel to replace those 6-7 *.ko modules.
**There are several exceptions for this case, such as Tegra or Yocto
distros, but they also have dedicated handling/installation guidance.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/6650#issuecomment-647036612,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFVCSIWAHQQWLX7AK6QTNDRXUE2LANCNFSM4OC36CYA
.


fix: change the winget package name

see https://github.com/microsoft/winget-pkgs/issues/67708

Checklist

[x] npm test passes
[x] documentation is changed or added (if applicable)
[ ] permission has been obtained to add new logo (if applicable)
[x] contribution guidelines followed here

Hi I am currently facing some issue on signing the ECA. How to sign with the GitHub inner noreply web email? I can see there is a disscussion discussing<EMAIL_ADDRESS>I have already linked my GitHub account to Eclipse account. However the system still not recognize.


Augment the plugin to support additional Nexus Staging REST APIs

Nexus supports several staging workflow scenarios as described here:
https://github.com/sonatype/nexus-maven-plugins/blob/master/staging/maven-plugin/WORKFLOWS.md
Currently, the plugin supports most of the Sonatype Nexus Staging Plugin REST APIs except for staging profile creation: /staging/profiles/{profileIdKey}/start

Merged #22


add int8 support for tf.tile

add int8 support for tf.tile otherwise there is an error after int8 quantization

@goexle could you provide some contexts to understand why this op is needed?

Fyi @thaink @teijeong

@goexle We are having an internal commit for this as well.
@abattery our internal cl is in better state since it also add quantization support and increase the op version.

@thaink thanks for mentioning. Do you know when your internal commit will be pushed to master?

Not sure about that yet but It is not a complicated one so I hope it will be landed soon.
I'll let you know then.

@goexle Can you describe a bit about your use-case?
Ex: What model are you trying to run and why does it need int8?

Yes sure. I want to quantize a model which is using the tf.tile operation. And I want to quantize it to int8.
Am 17. August 2021 02:22:59 MESZ schrieb Thai Nguyen @.>:
@. Can you describe a bit about your use-case?

Ex: What model are you trying to run and why does it need int8?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/tensorflow/tensorflow/pull/51480#issuecomment-899902563

@goexle  Any update on this PR? Please. Thanks!

@gbaned do you need further explanation about the use case or the model type beyond the mentioned use case?
The model is a modified version of TDNN for speech recognition.
If you need anything else, please let me know

@gbaned do you need further explanation about the use case or the model type beyond the mentioned use case? The model is a modified version of TDNN for speech recognition. If you need anything else, please let me know

@goexle Thank you for the update.


How To Upgrade Netty into a VPS

How can I upgrade the latest version of netty into my vps so I can try to fix an old "issue"?
Didn't find any tutorial on internet

Probably because Netty isn't something you install or upgrade on a VPS, it is an event-driven asynchronous network application framework used in other applications, based on your issue history I am going to go out on a limb and guess you are referring to BungeeCord.. If your VPS provider installed BungeeCord for you, you will need to contact them for support wrt updating BungeeCord or the BungeeCord forums etc.

I installed BungeeCord into the VPS by myself. So what can I do to upgrade it?

Sorry first time for me and I dunno what should I do.. I didn't found any information about it around the web

It entirely depends on how Netty was bundled with the application, if it's shaded, you'd probably need to fork the project and build it yourself from source after bumping the Netty version, if it's in a "libs" or other folder which is on the application classpath you might be able to monkey patch it by dropping in replacement jars.
Before going to the trouble of trying to update Netty you should first verify what version of Netty the application has and also check if there is a newer release of the software you want to run (some tutorials might have links to old version).

@KiritoKun90 if its part of BungeeCord you should ask them how to upgrade it (and if its even possible).

Ok I'll do it for sure and if I see that I still have a problem I'll let you know.
Cheers

@normanmaurer Already done and they told me it's a netty issue and to write here lel :)
Np I'll find it out by myself. Thx


Issue with running the script

I am trying to run this command "snakemake --cores 1 --use-conda -s workflow/fold_tree --config folder=myfam filter=False" but it is giving me an error. Looks like there is an issue with the script.
The error message:
(foldtree) [gsetia@turing fold_tree]$ snakemake --cores 1 --use-conda -s workflow/fold_tree --config folder=myfam filter=False
/home/gsetia/fold_tree/
Config file /home/gsetia/fold_tree/workflow/config/config_vars.yaml is extended by additional config specified via the command line.
Building DAG of jobs...
Using shell: /usr/bin/bash
Provided cores: 1 (use --cores to define parallelism)
Rules claiming more threads will be scaled down.
Job stats:
job                   count

all                       1
foldseek2distmat          1
foldseek_allvall_1        1
mad_root_post             3
postprocess               3
quicktree                 3
total                    12
Select jobs to execute...
[Fri Sep 29 14:08:12 2023]
rule foldseek_allvall_1:
input: myfam/finalset.csv
output: myfam/allvall_1.csv
log: myfam/logs/foldseekallvall.log
jobid: 8
reason: Missing output files: myfam/allvall_1.csv
wildcards: folder=myfam
resources: tmpdir=/tmp
Activating conda environment: foldtree
/usr/bin/bash: line 1:  1542 Illegal instruction     (core dumped) /home/gsetia/fold_tree/foldseek/foldseek easy-search myfam/structs/ myfam/structs/ myfam/allvall_1.csv myfam/tmp --format-output 'query,target,fident,alnlen,mismatch,gapopen,qstart,qend,tstart,tend,evalue,bits,lddt,lddtfull,alntmscore' --exhaustive-search --alignment-type 2 -e inf
[Fri Sep 29 14:08:14 2023]
Error in rule foldseek_allvall_1:
jobid: 8
input: myfam/finalset.csv
output: myfam/allvall_1.csv
log: myfam/logs/foldseekallvall.log (check log file(s) for error details)
conda-env: foldtree
shell:
/home/gsetia/fold_tree/foldseek/foldseek easy-search myfam/structs/ myfam/structs/ myfam/allvall_1.csv myfam/tmp --format-output 'query,target,fident,alnlen,mismatch,gapopen,qstart,qend,tstart,tend,evalue,bits,lddt,lddtfull,alntmscore' --exhaustive-search --alignment-type 2 -e inf
(one of the commands exited with non-zero exit code; note that snakemake uses bash strict mode!)
Shutting down, this might take some time.
Exiting because a job execution failed. Look above for error message
Complete log: .snakemake/log/2023-09-29T140812.273556.snakemake.log
(foldtree) [gsetia@turing fold_tree]$

Cloning the git again fixed this issue for me

is this still an issue @garima-setia ? Have you gotten everything to work?

Hi @cactuskid yes it worked now.


init_system check fails if system has a failed service

$(pgrep systemd > /dev/null && systemctl status > /dev/null);
has a return code of 3 when systemd is running but has a failed service.

Checking for the status of network would be good.

PR #84 can address this issue.

thank you @galthaus, i'test&merge asap. :)

PR #84


feat: enable data grid in Chatbot

Description

Issues Resolved
#1343
Check List

[x] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[x] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@joshuali925 Could we have one more approval to get this PR merged? The registerContentRenderer method has been changed to registerMessageRenderer and we need this PR to get merged.

@YANG-DB @joshuali925 Could you please help to merge this PR?

@joshuali925 @joshuali925 , could you please add backport 2.x label after backport of #1304 get merged?

sure #1374

@joshuali925 @YANG-DB , as backport failed, I create backport PR manually here. https://github.com/opensearch-project/dashboards-observability/pull/1383. Please help to review.
BTW, a minor change(1 line change) in parser, could you please help to review?
https://github.com/opensearch-project/dashboards-observability/pull/1382


DOC Document correct method signature for DB::require_table

The method signature for DB::require_table is wrong and should match the one from DBSchemaManager::requireTable.
I've documented the return type for both method while I was in there.
Parent issue

https://github.com/silverstripe/silverstripe-framework/issues/11424

@maxime-rainville Do you think you'll have a chance to resolve the outstanding requested changes?


InvalidOperationException: Response Content-Length mismatch when streaming a bitmap

Hi there,
If one tries to return an image in a FileStreamResult, the application will throw an exception along the lines of:
InvalidOperationException: Response Content-Length mismatch: too few bytes written (0 of 1234567)
My code was as follows:
private static IActionResult ImageResult(Bitmap image, ImageFormat format, string mimeType)
{
    var stream = new MemoryStream();
    image.Save(stream, format);
    return new FileStreamResult(stream, mimeType);
}

However, if I changed it to the following,  the image is returned without complaint:
private static IActionResult ImageResult(Bitmap image, ImageFormat format, string mimeType)
{
    var stream = new MemoryStream();
    image.Save(stream, format);
    return new FileContentResult(stream.ToArray(), mimeType);
}

It appears that doing the reading manually fixes the issue.
Thanks!

Did you forget to Seek the stream back to the beginning?

Yep that would do it!
Sorry to take up your time!

I get the same exception one out of two requests. It only happens when I deploy my application to Azure app service. Unfortunately, I can't reproduce it locally.
public class HomeController : Controller
{
    public IActionResult Spa()
    {
        return File("~/index.html", "text/html");
    }
}

@HWouters that looks like a different issue. See https://github.com/aspnet/Mvc/issues/6875

@Tratcher thank you for pointing me to the right issue. The workaround solves my problem until the patch is released.


Apply default user modes just before registration.

Previously, we were applying defaults before the user had completed registration. This meant that the number of invisible users was incremented when the user connected, and then the total was incremented when registration was completed.
Now both counters are updated at the same time. If a user disconnects prior to registration, +i has not yet been applied so it would not be decremented.
Fixes #1077.

I tested this with pre-registration users connected, and with always-on and multi-session.

Looks great, thanks!


Redirects: set up ibm.com redirects for methods

Part of https://github.com/Qiskit/documentation/issues/542. As explained there, we were getting a lot of 404s due to methods previously living on dedicated HTML pages with qiskit.org but now being inlined.
We already fixed the redirects from qiskit.org -> ibm.com to now redirect to the appropriate place. That fixed the problem with Google SEO having lots of old Qiskit versions that would redirect to 404s.
However, we are still getting 404s in our logs for dedicated method pages for historical versions of the Qiskit docs. That means that there are other origins of those bad URLs, such as if the search indexes maybe saved the bad URLs and are reusing them. Either way, we want to fix this. We need to add redirects to ibm.com when you are on a method page.
Here's our current logic:
const QISKIT_HOST = '^(?:www\\.)?qiskit\\.org'
const IBM_DOCS = 'docs.quantum.ibm.com'
const IBM_DOCS_QISKIT = `${IBM_DOCS}/api/qiskit`
const IBM_DOCS_PROVIDER = `${IBM_DOCS}/api/qiskit-ibm-provider`
const IBM_DOCS_RUNTIME = `${IBM_DOCS}/api/qiskit-ibm-runtime`

// Qiskit API methods moved from dedicated pages to instead live on their class pages.
// We explicitly made this change in Qiskit 0.44 and it applied to qiskit.org
// since then. But we also ended up making this change when migrating
// from qiskit.org to ibm.com because the API generation script for ibm.com
// would inline methods and atrributes to live on their parent class's page.
//
// We assume that classes use TitleCase, methods use snake_case, and attributes use ALL_CAPS. So,
// this regex looks for where the second-to-last component is TitleCase, and the last component
// before `.html` is snake_case or ALL_CAPS.
//
// We also need to avoid redirecting 9 classes in the Pulse library that end in `.rst.html`. We were
// incorrectly redirecting them before because the below regex had been too general.
const REDIRECTS__METHODS_TO_CLASS_PAGES = () => {
    const parentClass = '(.*\\.[A-Z][A-Za-z0-9_]*)'
    const classMember = '([a-z0-9_]+|[A-Z0-9_]+)'
    const fileExtension = '(?<!rst)\\.html'
    const stubsPortion = `stubs/${parentClass}\\.${classMember}${fileExtension}`
    const stableVersion = 'stable/(0\\.[0-9]{2})'
    return [
        // Qiskit
        [new RegExp(`${QISKIT_HOST}/documentation/${stubsPortion}`), `${IBM_DOCS_QISKIT}/$1#$2`],
        [new RegExp(`${QISKIT_HOST}/documentation/${stableVersion}/${stubsPortion}`), `${IBM_DOCS_QISKIT}/$1/$2#$3`],
        // Provider
        [new RegExp(`${QISKIT_HOST}/ecosystem/ibm-provider/${stubsPortion}`), `${IBM_DOCS_PROVIDER}/$1#$2`],
        // Runtime
        [new RegExp(`${QISKIT_HOST}/ecosystem/ibm-runtime/${stubsPortion}`), `${IBM_DOCS_RUNTIME}/$1#$2`],
        [new RegExp(`${QISKIT_HOST}/ecosystem/ibm-runtime/${stableVersion}/${stubsPortion}`), `${IBM_DOCS_RUNTIME}/$1/$2#$3`]
    ]
}

PR to set up the redirects created!

Note, we cannot land this until https://github.com/Qiskit/documentation/issues/542 is fully fixed because some class pages are still linking to non-existent method pages in ibm.com. We need to fix those broken links from showing up before we have them redirect to the class pages.
Also, we should wait a few days probably in-between deploying the fix to https://github.com/Qiskit/documentation/issues/542 and setting up these redirects to see if they're actually still happening in the wild, according to logs.

With https://github.com/Qiskit/documentation/issues/542 landed, we're going to look at the 404 logs over the next few days to see if we should add these.

Fixed by @arnaucasau. https://docs.quantum.ibm.com/api/qiskit/qiskit.algorithms.optimizers.BOBYQA.wrap_function now takes you to https://docs.quantum.ibm.com/api/qiskit/qiskit.algorithms.optimizers.BOBYQA#wrap_function. Great work, @arnaucasau!


Support finer grained system counters

Currently useful counters like: 'NAKs sent', 'NAKs recevied', 'Sender flow control limits applied' are only available at media driver level.  However it would be helpful if is possible to see these counters at a per-channel or per-stream level where appropriate.

We could add an option to add a number of optional counters per channel/stream so this could be enabled when needed and not crowd out the counters when not. Not sure if this is best but worth considering.

A more holistic approach to detecting loss is now available with the LossReport.
https://github.com/real-logic/Aeron/wiki/Monitoring-and-Debugging#loss-reporting


BC Public Preview: Page 810 Web Service takes very long to open

Describe the issue
In BC Public Preview with BcContainerHelper the Page 810 Web Service takes very long to open. In the BC Public Preview on SaaS this is not the case.
Scripts used to create container and cause the issue
New-NavContainer -accept_eula -containername mydocker -artifactUrl "https://bcpublicpreview.azureedge.net/sandbox/18.0.23013.23434/de" -PublicDnsName dockerhub.mycompany.com -useSSL -useTraefik -shortcuts None -auth NavUserPassword -Credential $credential -assignPremiumPlan -licenseFile "C:\dockerFiles\dev.flf" -sqlMemoryLimit "1G"

Full output of scripts
BcContainerHelper version 2.0.2
BcContainerHelper is version 2.0.2
BcContainerHelper is running as administrator
Host is Microsoft Windows Server 2019 Standard - ltsc2019
Docker Client Version is 19.03.14
Docker Server Version is 19.03.14
Fetching all docker images
Enabling SSL as otherwise all clients will see mixed HTTP / HTTPS request, which will cause problems e.g. on the mobile and modern windows clients
Using image mcr.microsoft.com/businesscentral:10.0.17763.1817
PublicDnsName is dockerhub.mycompany.com
Creating Container mydocker
Version: 18.0.23013.23434-DE
Style: sandbox
Multitenant: Yes
Platform: 18.0.22893.23390
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.1817 (ltsc2019)
Host OS Version: 10.0.17763.1817 (ltsc2019)
Using process isolation
Using locale de-DE
Adding special CheckHealth.ps1 to enable Traefik support
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Using license file C:\dockerFiles\dev.flf
--hostname dockerhub
-e webserverinstance=mydocker
-e publicdnsname=dockerhub.mycompany.com
-l "traefik.protocol=https"
-l "traefik.web.frontend.rule=PathPrefix:/mydocker"
-l "traefik.web.port=443"
-l "traefik.soap.frontend.rule=PathPrefix:/mydockersoap;ReplacePathRegex: ^/mydockersoap(.*) /BC$1"
-l "traefik.soap.port=7047"
-l "traefik.rest.frontend.rule=PathPrefix:/mydockerrest;ReplacePathRegex: ^/mydockerrest(.*) /BC$1"
-l "traefik.rest.port=7048"
-l "traefik.dev.frontend.rule=PathPrefix:/mydockerdev;ReplacePathRegex: ^/mydockerdev(.*) /BC$1"
-l "traefik.dev.port=7049"
-l "traefik.dl.frontend.rule=PathPrefixStrip:/mydockerdl"
-l "traefik.dl.port=8080"
-l "traefik.dl.protocol=http"
-l "traefik.enable=true"
-l "traefik.frontend.entryPoints=https"
--env customNavSettings=PublicODataBaseUrl=https://dockerhub.mycompany.com/mydockerrest/odata,PublicSOAPBaseUrl=https://dockerhub.mycompany.com/mydockersoap/ws,PublicWebBaseUrl=https://dockerhub.mycompany.com/mydocker
Files in C:\ProgramData\BcContainerHelper\Extensions\mydocker\my:
- AdditionalOutput.ps1
- CheckHealth.ps1
- license.flf
- MainLoop.ps1
- prismaSSL2020.PFX
- SetupCertificate.ps1
- SetupNavUsers.ps1
- SetupVariables.ps1
- updatecontainerhosts.ps1
Creating container mydocker from image mcr.microsoft.com/businesscentral:10.0.17763.1817
87e5ef8165cc7fb40fde351ebf7ce3d9118ff73ca639342d4bee32b2fe870be5
Waiting for container mydocker to be ready
Using artifactUrl https://bcpublicpreview.azureedge.net/sandbox/18.0.23013.23434/de
Using installer from C:\Run\150-new
Installing Business Central
Installing from artifacts
Starting Local SQL Server

Starting Internet Information Server

Copying Service Tier Files

Copying PowerShell Scripts
Copying dependencies
Copying ReportBuilder
Importing PowerShell Modules

Determining Database Collation from c:\dl\sandbox\18.0.23013.23434\de\BusinessCentral-DE.bak
Restoring CRONUS Demo Database

Setting CompatibilityLevel for tenant on localhost\SQLEXPRESS

Exporting Application to CRONUS

Removing Application from tenant
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Copying Web Client Files
Copying Client Files
Copying ModernDev Files
Copying additional files
Copying ConfigurationPackages
Copying Test Assemblies
Copying Extensions
Copying Applications
Copying Applications.DE
Starting Business Central Service Tier

Importing license file
Copying Database on localhost\SQLEXPRESS from tenant to default
Taking database tenant offline
Copying database files

Attaching files as new Database default

Putting database tenant back online
Mounting tenant database
Mounting Database for default on server localhost\SQLEXPRESS with AllowAppDatabaseWrite = False

Sync'ing Tenant
Tenant is Operational
Stopping Business Central Service Tier
Installation took 370 seconds
Installation complete
Initializing...
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is dockerhub
PublicDnsName is dockerhub.mycompany.com
Using NavUserPassword Authentication
Certificate File Thumbprint 6ACA6C1529F27493A8915CD0215D9DA9339EB27E
Import Certificate to LocalMachine\my
Modifying Service Tier Config File with Instance Specific Settings
Modifying Service Tier Config File with settings from environment variable
Setting PublicODataBaseUrl to https://dockerhub.mycompany.com/mydockerrest/odata
Setting PublicSOAPBaseUrl to https://dockerhub.mycompany.com/mydockersoap/ws
Setting PublicWebBaseUrl to https://dockerhub.mycompany.com/mydocker

Starting Service Tier

Registering event sources
Creating DotNetCore Web Server Instance
Enabling Financials User Experience
Enabling rewrite rule: Don't rewrite system files
Enabling rewrite rule: Already have tenant specified
Enabling rewrite rule: Hostname (without port) to tenant
Using license file 'c:\run\my\license.flf'
Import License

Dismounting Tenant

Mounting Tenant
Mounting Database for default on server localhost\SQLEXPRESS with AllowAppDatabaseWrite = False

Sync'ing Tenant
Tenant is Operational
Creating http download site

Setting SA Password and enabling SA
Creating admin as SQL User and add to sysadmin
WARNING: This license is not compatible with this version of Business Central.

Creating SUPER user
WARNING: This license is not compatible with this version of Business Central.

WARNING: This license is not compatible with this version of Business Central.
WARNING: This license is not compatible with this version of Business Central.
Assign Premium plan for ADMIN
Container IP Address: <IP_ADDRESS>
Container Hostname  : dockerhub
Container Dns Name  : dockerhub.dockerhub.mycompany.com
Web Client          : https://dockerhub.dockerhub.mycompany.com/mydocker/?tenant=default
Dev. Server         : https://dockerhub.dockerhub.mycompany.com
Dev. ServerInstance : BC
Dev. Server Tenant  : default

Files:
http://dockerhub.dockerhub.mycompany.com:8080/ALLanguage.vsix

Container Total Physical Memory is 16.0Gb
Container Free Physical Memory is 9.4Gb

Initialization took 77 seconds
Ready for connections!

Reading CustomSettings.config from mydocker
Set SQL Server memory limit to 1024 MB
Container mydocker successfully created
Because of Traefik, the following URLs need to be used when accessing the container from outside your Docker host:
Web Client:        https://dockerhub.mycompany.com/mydocker
SOAP WebServices:  https://dockerhub.mycompany.com/mydockersoap
OData WebServices: https://dockerhub.mycompany.com/mydockerrest
Dev Service:       https://dockerhub.mycompany.com/mydockerdev
File downloads:    https://dockerhub.mycompany.com/mydockerdl

Use:
Get-BcContainerEventLog -containerName mydocker
 to retrieve a snapshot of the event log from the container
Get-BcContainerDebugInfo -containerName mydocker
 to get debug information about the container
Enter-BcContainer -containerName mydocker
 to open a PowerShell prompt inside the container
Remove-BcContainer -containerName mydocker
 to remove the container again
docker logs mydocker
 to retrieve information about URL's again
...

Screenshots

The reason for this is something in the Web Services page, which when you are running Sandbox containers, assumes that you are running Saas.
I created an app to hide the controls that are time consuming:
https://businesscentralapps.blob.core.windows.net/hidestuff/latest/apps.zip
You can publish+install that to the container and the problem goes away.
I did file a bug on the application team earlier - will check on the status of that.

By looking into the code inside the Service Tier, I found that the reason for this is DNS lookups.
I will add DNS entries inside the container (to the hosts file) to avoid these delays.

Shipped in 2.0.9


Clarify org perms

Salesforce must enable org perms for onboarded orgs. Clarifying that.

I created https://github.com/salesforce/offline-app-developer-starter-kit/issues/10 to consider README updates. Heads up @alderete-sfdc , @agarwasg .


Support stripe-ruby 5.0+ Stripe::CardError.new arguments

Updating the mock error system to support stripe-ruby 5+ which was released a few days ago.
@see https://github.com/stripe/stripe-ruby/pull/816 for the specific changes
I made it backward-compatible but I don't know what is you policy about it. Otherwise maybe you should bump a major version of stripe-ruby-mock.
Otherwise stripe-ruby 5.0 seems to works OK for most of my use cases.

Would be good to get this merged even though the test suite is very broken.

Yeah the test suite is in a pretty bad shape, I didn't bother to add a test, I still feel bad about it ahah. This project is pretty useful but requires a lot of maintenance from non-paid Stripe contributors

Would be great to get this merged, it's effectively blocking me from testing anything other than the happy path for charges!

In my case I was able to fix a failing test by replacing
StripeMock.prepare_card_error(:card_declined)
with
error = Stripe::CardError.new('Test Msg', 'param_name', code: 'bad_code', http_status: 444, http_body: 'body')
StripeMock.prepare_error(error)
I was able to rescue from Stripe::CardError which allowed my code to still work

PS this should probably only be merged when we have a CI testing for stripe-ruby > 5 but quite a few other specs also fail with 5 (mostly because all methods were renamed list).

@j15e #658 has been merged. I don't think that we have to care about stripe version < 5. Maybe that's why this PR should be closed?

Yep seems legit to close! Thanks for the update.


Add icons for all the tutorials that are missing one

What is the documentation modification?
Add icons for all the tutorials that are missing one

None of these have graphical output.  The cards require graphical output otherwise it will use the default qiskit logo.  The exception is 4.14 which doesn't execute due to a bug that will be corrected in #648.

Nice. We can use some of the visual from them or icons. For example, for the transmon analytics we can use some of the resulting plots from nicks notebook or just a schematic image of the transmon.
For wavefunction for example we can show one of the images of a wavefunction etc.


WIP: New C core

This starts to implement a new C-based "core" for OpenEXR.
Current status:

most things are being written with doxygen-style comments, although some functions will need to be expanded upon and a cleanup pass made once we like the style to be made consistent.
base, lighter-weight attributes initial implementation sufficient for use, initial unit tests written
core file I/O / error reporting / memory allocation defined
initial file reading / parsing done, unit tests are in progress

only enough of the compression techniques to test have been ported, b44, pxr24, etc. are not difficult once we are happy with the core design, but I want to make that system a bit more ... replaceable

writing is largely incomplete at this stage
NB: This does not replace any C++ code at all currently (in fact, the current performance test relies on that!)

This is a massive PR, if it would be easier to re-construct it as individual layers, I can probably do that. But what I'm looking for is feedback on whether the "namespace" mechanism I've implemented degrades from the readability of the code too much to be worth it, or if we should eliminate that before there is too much inertia. Also looking for feedback on how I've chosen to break up the headers into component chunks - I dislike massive files. It is also written in a slightly different style, favoring underscores and such to try to differentiate it more from the C++.

I get having a C based API in libraries, but why should the implementation itself be C instead of C++? If there are parts of C++ that aren't wanted, such as exceptions, one can always avoid them, so it's not clear to me what forcing C accomplishes. Since it's not apparent to me, I suppose it would be worth having that outlined in case someone in the future wants to port this to C++.

I get having a C based API in libraries, but why should the implementation itself be C instead of C++? If there are parts of C++ that aren't wanted, such as exceptions, one can always avoid them, so it's not clear to me what forcing C accomplishes. Since it's not apparent to me, I suppose it would be worth having that outlined in case someone in the future wants to port this to C++.

I get having a C based API in libraries, but why should the implementation itself be C instead of C++? If there are parts of C++ that aren't wanted, such as exceptions, one can always avoid them, so it's not clear to me what forcing C accomplishes. Since it's not apparent to me, I suppose it would be worth having that outlined in case someone in the future wants to port this to C++.

@ThiagoIze - the idea is that this is C only because then it can be used in context where a c++ library is not desired (which has been requested numerous times), and I believe is a hindrance to adoption in things like the ffmpeg project, instead they have implemented a subset of the file format, which is problematic. Further, the roadmap (which is certainly not clear from what has been done so far) is to then re-implement the existing C++ library in terms of this library, although some API will have to be added to do so in a thread-safe manner. So C++ is not going away, just not part of this subset of the overall library. So, if you prefer C++, please continue to use the C++ API, as that will continue to be developed, and in future versions will gain the advantages this C layer was written to address (thread safety where multiple threads can read from the same file being the primary one)

I get having a C based API in libraries, but why should the implementation itself be C instead of C++? If there are parts of C++ that aren't wanted, such as exceptions, one can always avoid them, so it's not clear to me what forcing C accomplishes. Since it's not apparent to me, I suppose it would be worth having that outlined in case someone in the future wants to port this to C++.

@ThiagoIze - the idea is that this is C only because then it can be used in context where a c++ library is not desired (which has been requested numerous times), and I believe is a hindrance to adoption in things like the ffmpeg project, instead they have implemented a subset of the file format, which is problematic. Further, the roadmap (which is certainly not clear from what has been done so far) is to then re-implement the existing C++ library in terms of this library, although some API will have to be added to do so in a thread-safe manner. So C++ is not going away, just not part of this subset of the overall library. So, if you prefer C++, please continue to use the C++ API, as that will continue to be developed, and in future versions will gain the advantages this C layer was written to address (thread safety where multiple threads can read from the same file being the primary one)

I have removed the pseudo namespace layer. If anyone was following along, I used the following script to convert the code, then touched up a couple of things in the tests which were working by luck only.
`#! /bin/bash
echo 's/EXR_TYPE[ ]?[(][ ]?file[ ]?[)]/priv_exr_file_t/g' > /tmp/fixup.$$
echo 's/EXR_TYPE[ ]?[(][ ]?FILE[ ]?[)]/exr_file_t/g' >> /tmp/fixup.$$
echo 's/EXR_TYPE[ ]?[(][ ]?([A-Za-z][A-Za-z_0-9])[ ]?[)]/exr_\1_t/g' >> /tmp/fixup.$$
echo 's/EXR_DEF[ ]?[(][ ]?([A-Za-z_][A-Za-z_0-9])[ ]?[)]/EXR_\1/g' >> /tmp/fixup.$$
echo 's/EXR_FUN[ ]?[(][ ]?([A-Za-z_][A-Za-z_0-9]*)[ ]?[)]/exr_\1/g' >> /tmp/fixup.$$
find src/test/OpenEXRCoreTest -type f -exec sed -r -i -f /tmp/fixup.$$ {} ;
find src/lib/OpenEXRCore -type f -exec sed -r -i -f /tmp/fixup.$$ {} ;
find src/bin/exrinfo -type f -exec sed -r -i -f /tmp/fixup.$$ {} ;
rm /tmp/fixup.$$
`

I have removed the pseudo namespace layer. If anyone was following along, I used the following script to convert the code, then touched up a couple of things in the tests which were working by luck only.
`#! /bin/bash
echo 's/EXR_TYPE[ ]?[(][ ]?file[ ]?[)]/priv_exr_file_t/g' > /tmp/fixup.$$
echo 's/EXR_TYPE[ ]?[(][ ]?FILE[ ]?[)]/exr_file_t/g' >> /tmp/fixup.$$
echo 's/EXR_TYPE[ ]?[(][ ]?([A-Za-z][A-Za-z_0-9])[ ]?[)]/exr_\1_t/g' >> /tmp/fixup.$$
echo 's/EXR_DEF[ ]?[(][ ]?([A-Za-z_][A-Za-z_0-9])[ ]?[)]/EXR_\1/g' >> /tmp/fixup.$$
echo 's/EXR_FUN[ ]?[(][ ]?([A-Za-z_][A-Za-z_0-9]*)[ ]?[)]/exr_\1/g' >> /tmp/fixup.$$
find src/test/OpenEXRCoreTest -type f -exec sed -r -i -f /tmp/fixup.$$ {} ;
find src/lib/OpenEXRCore -type f -exec sed -r -i -f /tmp/fixup.$$ {} ;
find src/bin/exrinfo -type f -exec sed -r -i -f /tmp/fixup.$$ {} ;
rm /tmp/fixup.$$
`

Removal of the pseudo-namespaces has made the code infinitely easier to read.

Removal of the pseudo-namespaces has made the code infinitely easier to read.

Hey, can I make a request? Somewhere in one of the main headers (maybe OpenEXRConfig.h) can you please
#define HAS_OPENEXR_CORE 1

This will make it easier for downstream packages to know if they're dealing with a build of OpenEXR that has the new headers and C core functions.

This will make it easier for downstream packages to know if they're dealing with a build of OpenEXR that has the new headers and C core functions.

of course, we should do something like this. Although I don't actually like the name "core" - but I guess I will have to see how the rest of this goes for a little while


After feedback updates

Coverage remained the same at 89.848% when pulling 241f860c948c4699a3a0d2f7a98ba3b8c7f99d0b on V0.3-bernard into 51d82dea5841985b015b84f3d0dcb3c2d34f67d8 on master.


Close button not working properly on Product search Page

What
The close button on the product scan page is not closing the dialog
Steps to reproduce the behavior

Go to Search
Search for any product
Scroll down and click on Close

Screen Recording
https://user-images.githubusercontent.com/33085535/183059612-2bbb6fa9-3bb7-4de6-987d-bd60d8d12cef.mp4
Smartphone model

Device: Realme 3 Pro (RMX1851), Pixel 4 Emulator, Windows Subsystem Android
OS: Lineage OS 19.1 (Android 12), Android 13, Android 12

Thank you @abughalib for having detected this bug. Strange, it didn't happen when I coded it.
That said, I think I'll get rid of the "close" button: no error then ;)
And put the banner in a sliver instead.
Because anyway, it was a bit strange (assuming that the close button works, yes it did), that when you close the banner and then say "hey, finally I would like world data" you could not anymore as the banner was dismissed.

Working on a more simple version. Should be ready tomorrow.

And that gives me the opportunity to improve the "empty screen" too:


Conversion from darknet weights to h5 model issue in input shape

Hey,
I am trying to convert a trained darknet model for one class into a keras model. I am using the convert.py script and passed a valid cfg and weights file which work well in darknet testing. It converts the model into a keras model however the output shape that prints in the model is
(None, None.None, x) for most of the layers. Is there any way we can preserve the shape of the layers as this is causing further complications when I am working with the h5 model file in predictions.
Model summary from convert.py
Layer (type)                    Output Shape         Param #     Connected to
input_1 (InputLayer)            (None, None, None, 3 0

conv2d_1 (Conv2D)               (None, None, None, 3 864         input_1[0][0]

batch_normalization_1 (BatchNor (None, None, None, 3 128         conv2d_1[0][0]

...
conv2d_67 (Conv2D)              (None, None, None, 4 23085       leaky_re_lu_65[0][0]

conv2d_75 (Conv2D)              (None, None, None, 4 11565       leaky_re_lu_72[0][0]
Total params: 61,624,807
Trainable params: 61,572,199
Non-trainable params: 52,608

None
Saved Keras model to model_data/yolo.h5
Read 61624807 of 61624807.0 from Darknet weights.

Have you got any solution to these, I am also getting the same issue @reorder-cv


Upload a picture to url, and send

Welcome to be the task with the link to download the image and send. I think it is easier, but nothing happened happened.
Code:
  $image = $this->curl_get_contents("https://www.ru/data/resource_icons/0/{$id}.jpg");         file_put_contents("/var/www/www-root/data/www/www.ru/bots/test/Download/image.jpg" , $image);
try $this->telegram->getUploadPath(), and in other ways but failed to get a working effect it.
I will be very grateful for the example code

Hi @opkiler22789
Not sure I totally understand. Do you want users to be able to send a link to your bot, which then downloads it into the upload folder?

@noplanman In general, the bot is integrated with the site and information is displayed, I also try to download links from the site to the directory and send to the chat

Ok, so you're trying to send a picture to a chat via the bot?
In that case you can do the following in the command's execute method:
return Request::sendPhoto([
        'chat_id' => $chat_id,
    ],
    '/path/to/your/image.jpg'
);

@noplanman and it, therefore, but the picture itself does not appear in the directory

Have you set your upload path properly in your hook.php file?
You should be able to use this to save the file to the right location:
file_put_contents($this->telegram->getUploadPath() . '/image.jpg', $image);

Also, make sure the folder exists and that the permissions are set properly!

Thank you gave permissions 777 folder everything worked

Please don't use 777, especially for a folder that allows user uploads!
755 should work perfectly too.


Fix url for git clone

Types of PR

[ ] New Features
[ ] Upgrade of existing features
[x] Bugfix

Link to the issue
Description
URL for git clone looks wrong so I fixed it.
How to review this PR.
confirm you can clone this repository by running this command.
git clone https://github.com/tier4/scenario_simulator_v2.git
Others

Thank you for finding my mistake.


fix test plan

main purpose of thiese changes is now fixed renk method which now computee the full gauss (maybe we outsource it to a helper function for that)

thanks!


ohmyfetch package deprecated

use nuxt equivalent
or https://github.com/unjs/ofetch

use nuxt equivalent

this would be best?

use nuxt equivalent

this would be best?

Inside vue component scope, we could use usefetch from nuxt composable. While on the utils functions, we need a fetch suitable fetch package.

ofetch is auto-imported by Nuxt and used by the useFetch composable.
#ref: https://nuxt.com/docs/getting-started/data-fetching

ofetch should be a good choice for nuxt migrate.


Add code to inject js that make 'gp include' in 'gp include' works in reports

When a page include a page that include a page, the folding/unfolding javascript doesn't work well.
This PR include in each page of the report the correct javascript code that make the folding/unfolding works well.
The code is present but not called, so the task is not finished. I don't have any java framework on my computers (and don't know how to code java) so I can't test it...
Blind folded pull request here !

Hi,
Thank you for this functionality. Unfortunately, it can be integrated in greenpepper-core, as the JS depends on JQuery, which is not included in the specification output.
We can integrate it in the maven plugin report output though.

Closed by PR #96


Rewrite an Image from a Private Registry topic

ref: https://github.com/kubernetes/website/issues/30923
From @sftim:
https://kubernetes.io/docs/tasks/configure-pod-container/pull-image-private-registry/ clearly needs an update or even splitting into different pages. There are lots of Docker-specific details, some of which need to stay and some of which need to change.

We need an issue to update that page.
The example container registry is Docker Hub, so we might need to explain to readers what the different meanings of Docker could be.

I suggest the following:

Update mentions of Docker to specifically refer to Docker Hub
Update the introduction to this page to the following:

This page shows how to create a Pod that uses a Secret to pull an image from a private container image registry or repository. There are many private registries in use. This task uses Docker Hub as an example of the steps needed.

Update the Before You Begin section to the following:

* {{< include "task-tutorial-prereqs.md" >}}

* To do this exercise, you need the `docker` command line tool, and a
  [Docker Hub ID](https://docs.docker.com/docker-id/) for which you know the password.
* If you are using a different private container registry, you need the command line tool for that registry and any login information for the registry.

I would recommend this task for someone experienced with Secrets.
/triage accepted
/milestone 1.24

/assign

/retitle Rewrite “Pull an Image from a Private Registry” topic


Switching between Wikidata and Wikimedia Commons does not clear schema from Wikidata

When working with a dataset that combines Wikidata and Wikimedia Commons editing, and when a schema has been created in Wikidata, a switch to Wikimedia Commons will not clear the Wikidata schema, but will keep the 'wrong' schema in place.
I noticed this with the Wikidata-Commons switch but have not tried this with other Wikibases. This issue may also occur there.
Screenshot, showing that there has been a switch to Commons, but we still see the old Wikidata schema:

To Reproduce
Steps to reproduce the behavior:

Work with a dataset in which both Wikidata and Wikimedia Commons must be edited
First do edits on Wikidata with a Wikidata schema
Switch to Wikimedia Commons
Go to the schema tab and notice that the old schema is still in place

Expected Behavior
Schema should be cleared, OR the last saved schema from Wikimedia Commons would be expected to appear (if one has been saved before in this project)
Versions

Operating System: macOS 10.13.6
Browser Version: Chrome 103.0.5060.53
JRE or JDK Version: n/a
OpenRefine: 3.7 snapshot release of 30 June 2022

Datasets
public-art-Stuttgart.openrefine.tar.gz

I can also reproduce this.

This was solved by #5288.


KOGITO-9362: Fix operator version to 1.40.0 on kn-plugin-workflow

@caponetto i've updated the commit message to your suggestion.


Add minimal client for mips/mipsel/arm platform

Many embedded wireless platforms have enough resources to run a few tasks in the background. By building a minimal, single binary stand-alone version for MIPS and ARM, wireless routers could be used to gain passive and active data.

@DatanoiseTV Which minimal binary are you referring to? The CLI scanner?

@schollz Yes, the CLI scanner.

Yeah, so that can easily be built for ARM since Go cross compiles easily. The problem though is how to collect the Bluetooth/WiFi signals. In the scanner I'm using Linux-specific utilities (bluez and iw utilities). I think that these would depend on the specific device, unless there is a C-library maybe I can use to do this generically?

@schollz If you use the same API in Go which "iwlist scan" uses for scanning, you should have no problems to just use the Linux WiFi scanning API.


PSAR vs TV PSAR

Hello! Can you help me with the PSAR indicator? I can't get the same result as TradingView. I can't figure out what length the high and low input data should be?

Hi @anwebdevelopers 😊
I've double checked the indicator and TradingView the results indeed are different but the calculations are correct.
There are different because tradingview is implementing it differently.
here are the results:
{
  '0': 39789.3478992606,
  '1': 39779.286554734004,
  '2': 39768.10728303778,
  '3': 39755.685870041976,
  '4': 39741.88430004664,
  'name': 'technicalindicators'
}

{
  '0': 39789.3478992606,
  '1': 39779.286554734004,
  '2': 39768.10728303778,
  '3': 39755.685870041976,
  '4': 39741.88430004664,
  'name': '@ixjb94/indicators'
}

TradingView:

The difference ("error"?) between us and tradingview (which is negligible) :
0: 0.04%
1: 0.04%
2: 0.04%
3: 0.04%
4: 0.04%
But If you really want to achieve that number (or being really close to that number), you can use these parameters:
await ta.psar(highs, lows, 0.025, 0.2)

Results:
{
  '0': 39806.288131458794,
  '1': 39795.77215023862,
  '2': 39783.753885987,
  '3': 39770.01872684228,
  '4': 39754.32140210547,
  name: '@ixjb94/indicators'
}


CRUD Controller Does not Exist

Hey, was following the instructions on the website exactly and even created a brand new project that is still having the same issue. I looked to try and fix it to send a PR but can't find any source code for it o.O
When trying to create a new CRUD based Controller and add it to the routes file I am getting this.
ReflectionException in Container.php line 719:
Class Admin\NewsCommentCrudController does not exist
in Container.php line 719
at ReflectionClass->__construct('Admin\\NewsCommentCrudController') in Container.php line 719
at Container->build('Admin\\NewsCommentCrudController') in Container.php line 598
at Container->resolve('Admin\\NewsCommentCrudController') in Container.php line 567
at Container->make('Admin\\NewsCommentCrudController') in Application.php line 708
at Application->make('Admin\\NewsCommentCrudController') in Route.php line 217
at Route->getController() in Route.php line 736

Note: Admin\\NewsCommentCrudController
It seems to be adding double '' lines. I have tried be doing a group with a namespace, and directly coding the slash into the controller info.

add this to your app\Provider\RouteServiceProvider.php
public function map()
    {
    $this->mapAdminRoutes();
    ........
}
protected function mapAdminRoutes()
    {
        Route::middleware('admin')
            ->namespace($this->namespace.'\Admin')
            ->prefix('admin')
            ->group(base_path('routes/admin.php'));
    }

@AbbyJanke ,
Hmm... I can't seem to get the same result, it worked for me with Backpack\Generators 1.1.8:

I wonder - what's your setup?

operating system
server software
php version
?

I am not using the generators myself and doing it the longway. I prefer to
do that as it lets me see more of what is going on. I have yet to take a
look at my service provider as @IMokhles has suggested. As for what I am
running:

MacOS (10.12.4 beta)
Valet (with MySQL)
PHP 7

I can take a look further to see if the service provider is the isshe in a
few hours as I am just getting ready to head to work.
On Thu, Apr 20, 2017 at 9:45 PM Cristian Tabacitu<EMAIL_ADDRESS>wrote:

@AbbyJanke https://github.com/AbbyJanke ,
Hmm... I can't seem to get the same result, it worked for me with Backpack\Generators
1.1.8:
[image: screen shot 2017-04-21 at 07 43 47]
https://cloud.githubusercontent.com/assets/1032474/25263177/56966dee-2666-11e7-9610-39966db47c65.png
I wonder - what's your setup?

operating system
server software
php version
?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Laravel-Backpack/CRUD/issues/622#issuecomment-296061888,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdxdvVlpvZVV1kcX9Ws6JdLOFEZiQJAks5ryDSGgaJpZM4NDdOX
.

Oh, ok. So you've manually created the controller and route? In this case, please copy-paste the route and controller when you get to work. Cheers!

Update on this, been busy so sorry for taking forever. Adding the Namespace to the service provider map function like @iMokhles suggested worked.
Thanks.


[Feature] add model script, training configs and training weights of Volo

Thank you for your contribution to the MindCV repo.
Before submitting this PR, please make sure:

[ ] You have read the Contributing Guidelines on pull requests
[ ] Your code builds clean without any errors or warnings
[ ] You are using approved terminology
[ ] You have added unit tests

Motivation
(Write your motivation for proposed changes here.)
Test Plan
(How should this PR be tested? Do you require special setup to run the test or repro the fixed bug?)
Related Issues and PRs
(Is this PR part of a group of changes? Link the other relevant PRs and Issues here. Use https://help.github.com/en/articles/closing-issues-using-keywords for help on GitHub syntax)

Please format the code to pass pre-commit check

We need a futher discussion on val_amp_level


[App] Cannot select min/max values of log-scale parameters in parallel coordinates panel

Describe the bug
When I run a sweep over a parameter with discrete values, and I choose to view that value in a parallel coordinates panel using the log-scale, I cannot select the runs with the min/max value for that parameter.  This always happens when using log-scales, and never when using linear-scales.
To Reproduce
Steps to reproduce the behavior:

Create sweep with discrete parameter
In the sweep page, go to the parallel coordinates panel
Edit the panel and turn on "log-scale" for the parameter in question
Try to select the min or max value of that parameter by click and dragging over the appropriate section in the parallel coordinates panel.  Do this by dragging from inside the panel towards the outside of the panel, so that you exceed the actual min/max value.
The selection will fail and a "No runs selected" pop-up will appear.

Expected behavior
I expect that I would be able to select the runs which have the min or max value for that parameter.
Screenshots
This screenshot shows how I try to select the runs:  I draw from the inside towards the outside, and release when I am completely outside the panel.  As soon as I release the mouse, I get the "No runs selected" pop-up.

Operating System

OS: Debian
Browser:  Firefox
Version: 85.0.1

Hey @abaisero
I tried reproducing this issue. I did not find a problem when selecting the run in log scale as shown in the screenshot below 👇

System config:

OS: macOS Big Sur
Browser: Chrome
Version: 89.0.4389.90 (Official Build) (arm64)

Could this be a problem with the web-browser?

Same issue happens with me on

OS: Debian
Browser: Google-Chrome
Version: 88.0.4324.182 (Official Build) (64-bit).

In the past year we've majorly reworked the CLI and UI for Weights & Biases. We're closing issues older than 6 months. Please comment to reopen.


Fixed micronaut launch link

I think the zeebe feature of micronaut got renamed due to the other micronaut project.
So the link hasn't work until adding the camunda-zeebe dependency.

Thank you @lwluc !


Furkan adiguzel

can you please change your pr from phase-1 branch to main

and please specify what have you done in your pr header, (etc. http package initialized, http client created, ..)


Add eslint-plugin-no-secrets

I wrote this to prevent people from commiting things like OAuth tokens and AWS Keys. It was inspired by truffleHog, and it uses a similar design of using a combination of finding high entropy strings (with a different dictionary more aimed toward js codebases) and curated regular expressions.

Thanks for letting me know! I have updated my pull request.

Thanks!


Windows

Pretty amazing work, will windows be supported in the future?

Hey @dubing-bit, the project uses Anaconda and PyPI packages, so it should work on Windows. However, we have only run it on Ubuntu (Linux).
You could manually install the necessary packages from setup_conda.sh and try it.

Closing due to inactivity. Avneesh has addressed the concern.


package.json: add node-gyp as an optional peer dep

This way, the implicit dependency on node-gyp for local builds is explicit, and build tools like dream2nix don't need workarounds to add it :-)

You know what - I now see that node-gyp ships with npm by default (thanks for pointing that out) and it's just a matter of making sure it's in the path while building outside of npm. I was calling node-gyp directly and it was missing. Thanks again and sorry for the disturbance :+1:


how to support 'require' and 'exports' without MicroService

It depends on what you are trying to do. But for pure JS, you can use
liquidcore-cli to bundle your JS into a single file and load that file:
npm run bundler -- --platform=android

will start with liquid.js (you can change that with a command line
option) and bundle all required modules into a single file called
liquid.bundle. You can then load that as an asset and run it without a
micro service.
On Thu, Feb 21, 2019, 9:16 AM Heaven7<EMAIL_ADDRESS>wrote:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/LiquidPlayer/LiquidCore/issues/87, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABIuZBLOIj47u4spcwKuJSaEP6pBosyOks5vPgF8gaJpZM4bGgW2
.

i just want load another js module from local.

You can only use require with the bundler.
On Thu, Feb 21, 2019, 1:08 PM Heaven7<EMAIL_ADDRESS>wrote:

i just want load another js module from local.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/LiquidPlayer/LiquidCore/issues/87#issuecomment-465874384,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABIuZN3TRP_QMnyTWqMjqnhJP6jyyxSRks5vPjfmgaJpZM4bGgW2
.

bundler ?Can you tell me in detail?

npm install -g liquidcore-cli
liquidcore init myProject
cd myProject && npm install

Change index.js to be whatever you want it to be.  You can require anything you want (so long as it is pure javascript -- if you are not using a microservice, you can't use built-in node modules).
Then,
npm bundler -- --platform=android

Now you have a file called liquidcore.bundle.  This is a single, bundled version of all of your modules, starting with whatever you put in index.js.
In Android, drop that file in your assets folder and load it into a string (myJSCodeString).
Then,
JSContext context = new JSContext();
context.evaluateScript(myJSCodeString);

ok. even if this not support 'require'. I still want to thank you.


Exposing a cache type setting and integrating it with IndicesRequestC…

Description
Exposes a cache type setting for IndicesRequestCache which can be set by user. For now, integrating only onHeap. When tiered cache are changes are done, it will added later.
This setting is being as added like a feature flag as it is meant to be experimental in initial phase.
Related Issues
Resolves https://github.com/opensearch-project/OpenSearch/issues/11947
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[ ] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[x] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Looks like build is failing and please add changelog


Stage -> Main Aug 30 2024

Describe your specific features or fixes
Resolves: MWPW-NUMBER
Test URLs:

Before: https://main--events-milo--adobecom.hlx.page/
After: https://stage--events-milo--adobecom.hlx.page/

Codecov Report
Attention: Patch coverage is 83.60656% with 10 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@2ea9226). Learn more about missing BASE report.

Files with missing lines
Patch %
Lines

events/scripts/content-update.js
16.66%
10 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #139   +/-   ##
=======================================
  Coverage        ?   79.04%
=======================================
  Files           ?       13
  Lines           ?     2014
  Branches        ?        0
=======================================
  Hits            ?     1592
  Misses          ?      422
  Partials        ?        0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Remove reference to dead servers.

Don't really think this deserves notes.
Review by @eed3si9n

LGTM pending Travis.


fix: Change sdks linguist-vendored to linguist-generated

Changes

#662  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @AngelOnFira and the rest of your teammates on  Graphite


Update test cases

Codecov Report

Merging #353 (40f6a9a) into master (c081f81) will increase coverage by 2.63%.
The diff coverage is 75.00%.

@@             Coverage Diff              @@
##             master     #353      +/-   ##
============================================
+ Coverage     59.24%   61.87%   +2.63%
- Complexity      769      817      +48
============================================
  Files           138      136       -2
  Lines          3133     3140       +7
  Branches        409      412       +3
============================================
+ Hits           1856     1943      +87
+ Misses         1191     1120      -71
+ Partials         86       77       -9

Impacted Files
Coverage Δ

src/main/java/seedu/awe/model/group/GroupName.java
66.66% <0.00%> (-11.91%)
:arrow_down:

src/main/java/seedu/awe/model/person/Phone.java
100.00% <ø> (+15.38%)
:arrow_up:

src/main/java/seedu/awe/model/person/Name.java
93.33% <50.00%> (+7.61%)
:arrow_up:

.../logic/commands/ListTransactionSummaryCommand.java
100.00% <100.00%> (+100.00%)
:arrow_up:

...eedu/awe/logic/parser/FindGroupsCommandParser.java
100.00% <100.00%> (+42.85%)
:arrow_up:

...e/model/transactionsummary/TransactionSummary.java
95.00% <100.00%> (+47.63%)
:arrow_up:

.../java/seedu/awe/model/person/UniquePersonList.java
100.00% <0.00%> (+11.36%)
:arrow_up:

src/main/java/seedu/awe/AppParameters.java
95.00% <0.00%> (+20.00%)
:arrow_up:

... and 7 more

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 c081f81...40f6a9a. Read the comment docs.


Update shrinkwraps, and use pino-http directly.

nodejs-express: use pino-http directly
nodejs-*: update shrinkwraps

Checklist:

[ ] Read the Code of Conduct and Contributing Guidelines.

[ ] Followed the commit message guidelines.

[ ] Stack adheres to Appsody stack structure.

Modifying an existing stack:

[ ] Updated the stack version in stack.yaml

Contributing a new stack:

Describe how application dependencies are managed:

Explain how Appsody file watcher is utilized:

Describe other Appsody environment variables defined by the stack image:

Describe any limitations and known issues:

Related Issues:

@sam-github - please can you split this PR up (one for each stack) so that we can release? Also, the versions need bumping. Thanks!

Replaced with https://github.com/appsody/stacks/pull/786


Merge XML outputs into a consolidated report

Addresses #1295
Here are my own judgment calls that needs review:

Create a new gradle plugin for the root project. Because merging only makes sense for a multi-project setup. Configuring in any subproject will create dependencies across subprojects, which is not a good gradle practise.
Not depending on XML parsing library because we can safely assume that all the generated output is managed by detekt.
Not automatically apply the merge when running ./gradlew detekt or ./gradlew detektMain, this is because of incremental cacheability of gradle that does not guarantee each detekt task will be actually run / and or each individual gradle task may fail due to detected issues. The expectation for our users is to run ./gradlew detekt mergeXmlReport --continue or ./gradlew detekt --continue; ./gradlew mergeXmlReport

I'm not sure about this. Could we get encoding issues and problems with special characters and so on?

I have updated this PR by using Java XML API. There are some nuances that ending up with more configurations, and the output is not ideal either: attributes are sorted that column is shown before line.

I'm not sure about this. Could we get encoding issues and problems with special characters and so on?

I have updated this PR by using Java XML API. There are some nuances that ending up with more configurations, and the output is not ideal either: attributes are sorted that column is shown before line.

This is a highly requested feature for a long time. Please let me review this PR from a user perspective. Since this PR is a draft, I guess it needs more E2E testing and some refactoring.

The only missing part is documentation. If the code looks good, I can add documentation in the same PR or follow-up PR.
I have done e2e testing in https://github.com/chao2zhang/RequireNamedArgument/tree/testdetekt

This is a highly requested feature for a long time. Please let me review this PR from a user perspective. Since this PR is a draft, I guess it needs more E2E testing and some refactoring.

The only missing part is documentation. If the code looks good, I can add documentation in the same PR or follow-up PR.
I have done e2e testing in https://github.com/chao2zhang/RequireNamedArgument/tree/testdetekt

Okay, thanks for the detailed explanation.The codecov report shows a decrease in the code coverage. Is this only because of the newly added Gradle tasks?

Okay, thanks for the detailed explanation.The codecov report shows a decrease in the code coverage. Is this only because of the newly added Gradle tasks?

Okay, thanks for the detailed explanation.The codecov report shows a decrease in the code coverage. Is this only because of the newly added Gradle tasks?

Good catch.

codecov report won't be able to report gradle integration test coverage.
I can add integration test in this PR if possible.

Okay, thanks for the detailed explanation.The codecov report shows a decrease in the code coverage. Is this only because of the newly added Gradle tasks?

Good catch.

codecov report won't be able to report gradle integration test coverage.
I can add integration test in this PR if possible.

As a side note to my comments: great work @chao2zhang 🎉  This was a highly-requested feature, thanks for tackling it :)

As a side note to my comments: great work @chao2zhang 🎉  This was a highly-requested feature, thanks for tackling it :)

I'm afraid that having two Gradle plugins will add more complexity (double artifacts, double plugin markers, users confusions about which plugin should be applied where, etc.) than benefits.

I started with developing in the same plugin, and it didn't take long for me to realize that we need to deal with an exponential level of complexity.

Single module project: If using the same plugin, this should be easier. However, for the single module project, merging the XML output does not seem a problem necessary to solve.
Multi-module with detekt on both root project and subprojects: If using the same plugin, we should not expect any Detekt task to root module because of the issues reported in #3389, only subprojects should have Detekt task and the rootproject should have MergeTask only. Configuring DetektExtension on the root project is also erroneous.

The decision of creating a different gradle plugin is favoring a cleaner multi-module setup.

I'm afraid that having two Gradle plugins will add more complexity (double artifacts, double plugin markers, users confusions about which plugin should be applied where, etc.) than benefits.

I started with developing in the same plugin, and it didn't take long for me to realize that we need to deal with an exponential level of complexity.

Single module project: If using the same plugin, this should be easier. However, for the single module project, merging the XML output does not seem a problem necessary to solve.
Multi-module with detekt on both root project and subprojects: If using the same plugin, we should not expect any Detekt task to root module because of the issues reported in #3389, only subprojects should have Detekt task and the rootproject should have MergeTask only. Configuring DetektExtension on the root project is also erroneous.

The decision of creating a different gradle plugin is favoring a cleaner multi-module setup.

we should not expect any Detekt task to root module because of the issues reported in #3389, only subprojects should have Detekt task and the rootproject should have MergeTask only.

Yup you're right, thanks for clarifying this 👌

we should not expect any Detekt task to root module because of the issues reported in #3389, only subprojects should have Detekt task and the rootproject should have MergeTask only.

Yup you're right, thanks for clarifying this 👌

I think more or less like @cortinico here. Add an extra plugin seems to add too much complexity. Ideas:

I know that in gradle you can add a plugin in the root and it adds taks in subprojects. Can we do the opposite? Add a task in the parent?
Maybe, instead of providing this functionality in our own plugin we could create tasks and add them in our documentation. So anyone can copy&paste.

Right now I think that the second point is the best one. My reason is that detektMergeXmlReport doesn't have a deterministic output. It depends in your previous commands. And we can't make it deterministic because we have too much detekt tasks and we don't know which one the user wants to run.
If we implement #3364 this problem goes away. We could make detektMergeXmlReport dependOn detektCheck.
And each time that I think about it I'm more sure. detekt should output SARIF and then we should have other tasks to create the other reports using that file. And for this I think that just a task in the documentation is the best option, because we are not sure if we will keep that task in the future.

I think more or less like @cortinico here. Add an extra plugin seems to add too much complexity. Ideas:

I know that in gradle you can add a plugin in the root and it adds taks in subprojects. Can we do the opposite? Add a task in the parent?
Maybe, instead of providing this functionality in our own plugin we could create tasks and add them in our documentation. So anyone can copy&paste.

Right now I think that the second point is the best one. My reason is that detektMergeXmlReport doesn't have a deterministic output. It depends in your previous commands. And we can't make it deterministic because we have too much detekt tasks and we don't know which one the user wants to run.
If we implement #3364 this problem goes away. We could make detektMergeXmlReport dependOn detektCheck.
And each time that I think about it I'm more sure. detekt should output SARIF and then we should have other tasks to create the other reports using that file. And for this I think that just a task in the documentation is the best option, because we are not sure if we will keep that task in the future.

I know that in gradle you can add a plugin in the root and it adds taks in subprojects. Can we do the opposite? Add a task in the parent?

We can, but that does comprise the correctness:

We only need to add it once for a given subproject, so we need to perform a check to see if this is already added to the root project.
It is unclear which DetektExtension would be used to configure the behavior of the root project plugin either. There is one detekt extension in each subproject.
This may also cause conflict with the plugins already at the root project.

Maybe, instead of providing this functionality in our own plugin we could create tasks and add them in our documentation. So anyone can copy&paste.

Yes, we can expose Gradle tasks themselves and add examples. This is a good idea.

I know that in gradle you can add a plugin in the root and it adds taks in subprojects. Can we do the opposite? Add a task in the parent?

We can, but that does comprise the correctness:

We only need to add it once for a given subproject, so we need to perform a check to see if this is already added to the root project.
It is unclear which DetektExtension would be used to configure the behavior of the root project plugin either. There is one detekt extension in each subproject.
This may also cause conflict with the plugins already at the root project.

Maybe, instead of providing this functionality in our own plugin we could create tasks and add them in our documentation. So anyone can copy&paste.

Yes, we can expose Gradle tasks themselves and add examples. This is a good idea.

I am going to close this PR and publish a new one to expose Gradle Tasks to merge XML + SARIF with provided groovy/kts scripts.

I am going to close this PR and publish a new one to expose Gradle Tasks to merge XML + SARIF with provided groovy/kts scripts.


bug(MatAutocomplete): matAutocomplete with initial value does not filter options and does not highlight selected

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
A MatAutocomplete field initialized with a value does not filter the list of options are mark selected item.
Reproduction
StackBlitz link: https://stackblitz.com/edit/vemaxm-6ktpyr?file=src%2Fexample%2Fautocomplete-auto-active-first-option-example.html
Steps to reproduce:

In the stackblitz, the formControl for the autcomplet is initialized with 'Two'

Click on the field

if I click on the option

it is as expected.

after that if I click the 'clear' button

value is still selected

reselect a value, lets say 'Two'
delete the two last letters

Expected Behavior
The list of option should be filtered with the initial FormControl value
The actual value should be marked as selected.
Resetting the value should remove the selected
deleting last letter of input keeps last selected option as selected
Actual Behavior
All options are displayed, and nothing indicates that a value is selected.
Resetting the form still displays a value as selected.
if not option exactly match the value, there should not be any selected option.
Environment

Angular:
CDK/Material:
Browser(s):
Operating System (e.g. Windows, macOS, Ubuntu):

mat-autocomplete doesn't do any filtering, it just displays the mat-option that are projected and keeps track of the value. You can get it to work by changing startWith('') to startWith(this.myControl.value).

Ok that's fair, and resetting the selected is addressed with #27652
but what about the resetting selected when there is a partial value, or selecting when the value is the same, but was not selected in the dropdown?


Release notes for LM-134, LM-752 & LM-449

Release notes for changes that were made as of part of LM-134, LM-752 & LM-449

 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.

Hi @MarkvanMents,
Thank you for reviewing & fine tuning the releasing notes for us.
For the new graphs that we want to add, here is the MR https://github.com/mendix/docs/pull/4323 that is already open. I see that you need the images for the new graphs. I will add the images on your comments shortly.


Example projects

[ ] Todo MVC
[ ] Astroid game

Bumped to higher priority since more bugs will be resolved by working on other kinds of applications.


Mashup (orchestrator) - Implement node for persisting data in a relational database

There is no demand to implement it at this moment and if necessary it can be implemented external to the platform by remote processing nodes.


Simplify LogicExpr api and update DocId type to string

Simplify LogicExpr api
Update DocId type to string

 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.


Function missing required argument 'line1' (pos 2)

Bug report
Your Environment

Netflix add-on version: 1.2.0
Operating system version/name: Libreelec (Milhouse)
Device model:  Raspberrypi

Used Operating system:

[ ] Android
[ ] iOS
[ ] Linux
[ ] OSX
[x] Raspberry-Pi
[ ] Windows

Describe the bug
This error appears every time I try to watch smth.
log-2020-04-25-19.58.47.zip
Installation

[ ] I'm using other Netflix Repo
[x] I'm using a different source: Your repository for Kodi Matrix

This error not come from Netflix:
"/storage/.kodi/addons/script.module.inputstreamhelper/lib/inputstreamhelper/kodiutils.py", line 124, in yesno_dialog

@CastagnaIT What does that mean? Any idea on how I can fix it? Do I need a special version of that plugin? I cannot use Netflix plugin at all.

as you can read in my new post in the forum
Kodi 19 is implementing several API changes
that's going to break a lot of add-ons
you probably either installed too old a version
or they didn't full fixed their addon

So I must report it to inputstreamhelper if there is no new version?