Python: Update SK Python to be compatible with openai-python 1.0

Motivation and Context
Need to update SK Python to integrate the breaking changes from openai-python 1.0
https://github.com/microsoft/semantic-kernel/issues/3330

Could we merge this into a separate branch so that multiple people can contribute to these changes, lot's of larger and smaller issues we will need to iron out for this! Might be good to consider combining with this: #2843 because there I made some improvements in the classes which will reduce the amount of doubled code.

Could we merge this into a separate branch so that multiple people can contribute to these changes, lot's of larger and smaller issues we will need to iron out for this! Might be good to consider combining with this: #2843 because there I made some improvements in the classes which will reduce the amount of doubled code.

Just created https://github.com/microsoft/semantic-kernel/pull/3470 as its own separate branch! @eavanvalkenburg can you merge over the code that you have in #2843 and we can iterate fully on that branch? I have merge rights for it, so we can move more quickly

Closing this PR because we will be moving this work over to this branch instead https://github.com/microsoft/semantic-kernel/pull/3470


Build failed: Crunching Cruncher failed

I'm having issues with app:mergeDebugResources task.
error:
* What went wrong:
Execution failed for task ':app:mergeDebugResources'.
> Crunching Cruncher abc_tab_indicator_mtrl_alpha.9.png failed, see logs

I just created a new react native project with this command. No plugins and other code added yet.
react-native init MyProject

From package.json version 0.24.1 of React is installed:
{
  "name": "TestReact",
  "version": "0.0.1",
  "private": true,
  "scripts": {
    "start": "node node_modules/react-native/local-cli/cli.js start"
  },
  "dependencies": {
    "react": "^0.14.8",
    "react-native": "^0.24.1"
  }
}

The error occurs for android, not really sure if the same problem exists with iOS.
I'm using Ubuntu for development.
Gradle is at version 2.4 according to the gradle.properties file:
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-2.4-all.zip

for anyone having the same problem. this solved the issue for me:
http://stackoverflow.com/questions/22701405/aapt-ioexception-error-2-no-such-file-or-directory-why-cant-i-build-my-grad
It's an issue with 64-bit OS, because aapt isn't available you have to install these 2 packages:
sudo apt-get install lib32stdc++6 lib32z1

thanks,that works for me.

Thanks! It worked for me too. :+1:


Cannot find module<EMAIL_ADDRESS>

I have an Angular 4+ project setted up with angular cli. I installed the modules with yarn but importing<EMAIL_ADDRESS>in my app.module is not working.
I see that the name of the module has been changed recently, maybe something is wrong there?

I have the same problem too... what can i do?

@lironta
I just figured out you don't need to specify the whole node_modules path.
This works: import { LeafletModule } from '@asymmetrik/ngx-leaflet'
The documentation needs to be updated.

Can you clarify what the issue was and what the resolution was? Or if you are still having issues?
The typescript import should not typically reference the node_modules path (unless you have a weird custom ts config).
I'm using the new renamed module successfully in an ng cli project.
If there's an issue with the documentation, can you provide the specific issue? Thanks!

@reblace the problem is in the README file at line 121
this line exactly import { LeafletModule } from<EMAIL_ADDRESS>It does'nt work like that but it works by removing .module.
So the README needs to be updated to:
import { LeafletModule } from '@asymmetrik/ngx-leaflet';

Ahhhh. I see, yeah I know exactly how that typo ended up in there. I'll fix is ASAP. Thanks!

Fixed the README. Sorry for the confusion!


bt_connect: Unable to allocate buffer

This bug sometimes gets triggered when attempting a date and time syncronization. It seems to have been fixed upstream here.
From the RTT log:
[00:02:05.666,015] <wrn> bt_conn: Unable to allocate buffer: timeout 30000
[00:02:05.666,015] <err> bt_att: Unable to allocate buffer for op 0x05
[00:02:05.714,141] <err> bt_att: ATT Timeout
[00:02:05.714,172] <err> bt_gatt: Invalid data len 176
[00:02:05.714,202] <err> os: ***** Reserved Exception ( -16) *****
[00:02:05.714,202] <err> os: r0/a1:  0x2000ba38  r1/a2:  0x00010bf9  r2/a3:  0x00026b9f
[00:02:05.714,202] <err> os: r3/a4:  0x00010acd r12/ip:  0x00010acd r14/lr:  0x00010acd
[00:02:05.714,202] <err> os:  xpsr:  0x00010a00
[00:02:05.714,233] <err> os: Faulting instruction address (r15/pc): 0x00010acd
[00:02:05.714,233] <err> os: >>> ZEPHYR FATAL ERROR 0: CPU exception on CPU 0
[00:02:05.714,233] <err> os: Current thread: 0x20005ee8 (unknown)
[00:02:06.078,979] <err> fatal_error: Resetting system

Updating Zephyr seems to have fixed this. I'll close this when the Zephyr rebase has been completed.

The bug i still there, unfortunately:
[00:02:09.486,022] <wrn> bt_conn: Unable to allocate buffer: timeout 30000
Transfer rate: 0 KByte/s Bytes written: 24 KByte  bt_att: Unable to allocate buffer for op 0x11
[00:02:09.582,672] <err> bt_att: ATT Timeout
[00:02:09.582,702] <err> bt_gatt: Invalid data len 176
[00:02:09.582,702] <err> os: ***** MPU FAULT *****
[00:02:09.582,702] <err> os:   Instruction Access Violation
[00:02:09.582,733] <err> os: r0/a1:  0x00000000  r1/a2:  0x00000000  r2/a3:  0x20005fec
[00:02:09.582,733] <err> os: r3/a4:  0x20005fc8 r12/ip:  0x00000000 r14/lr:  0x0000a581
[00:02:09.582,733] <err> os:  xpsr:  0x40000000
[00:02:09.582,733] <err> os: Faulting instruction address (r15/pc): 0x20005fc8
[00:02:09.582,733] <err> os: >>> ZEPHYR FATAL ERROR 0: CPU exception on CPU 0
[00:02:09.582,763] <err> os: Current thread: 0x20005fec (unknown)
[00:02:09.955,474] <err> os: Halting system


chore: add outDir to compiler options in tsconfig.json

Description
Add outDir to compiler options in tsconfig.json.
Motivation and Context
Fixes #1986
How has this been tested?
Error message disappeared in VSCode on macos.
Screenshots (if appropriate)
Types of changes

Internal changes:

[x] Infrastructure (prefix: chore - examples include GitHub Actions, issue templates)

Checklist

[ ] My code follows the code style of this project and passes yarn run lint.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] My change has adequate Unit Test coverage.

Terms

[x] My contribution follow this project's contributing guide
[x] I agree to follow this project's Code of Conduct

Oh it’s interesting that you set it to an empty value. I had seen examples that set it to non-empty.

Oh it’s interesting that you set it to an empty value. I had seen examples that set it to non-empty.

well, our output dir is the root of the repo since this is where main.js appears, so I've set it to an empty value.


DTO run with Nginx,  dto only run 10ms and dto destructor function cleanup_dto executed, this cause DSA unmap and close wq portal.

1 Run Nginx separately, it works fine:

2 Run DTO with Nginx,
2.1 Configure DTO:
➜  DTO git:(main) ✗ accel-config disable-device dsa0
accel-config disable-device dsa2
accel-config disable-device dsa4
accel-config disable-device dsa6
accel-config load-config -c ./dto-4-dsa.conf
accel-config enable-device dsa0
accel-config enable-device dsa2
accel-config enable-device dsa4
accel-config enable-device dsa6
accel-config enable-wq dsa0/wq0.0
accel-config enable-wq dsa2/wq2.0
accel-config enable-wq dsa4/wq4.0
accel-config enable-wq dsa6/wq6.0
export DTO_USESTDC_CALLS=0
export DTO_COLLECT_STATS=1
export DTO_WAIT_METHOD=yield
export DTO_MIN_BYTES=4096
export DTO_CPU_SIZE_FRACTION=0.33
export DTO_AUTO_ADJUST_KNOBS=1
disabled 1 device(s) out of 1
disabled 1 device(s) out of 1
dsa4 device not found
disabled 0 device(s) out of 1
dsa6 device not found
disabled 0 device(s) out of 1
device is not available
Parse json and set device fail: -1
enabled 1 device(s) out of 1
enabled 1 device(s) out of 1
dsa4 device not found
enabled 0 device(s) out of 1
dsa6 device not found
enabled 0 device(s) out of 1
enabled 1 wq(s) out of 1
enabled 1 wq(s) out of 1
dsa4 device not found
enabled 0 wq(s) out of 1
dsa6 device not found
enabled 0 wq(s) out of 1
2.2 Run DTO with Nginx:

3 Expect: when Nginx keep running, dto need keep running too.

Hi, DTO logs the above output when the process terminates. There must have been one nginx process that terminated and thats why DTO printed above logs.
I reproduced the issue on my system and I can see that the process (pid) no longer exists for which DTO printed above logs. There are other nginx processes that continue to keep running.
I think it is an expected behavior. Please let me know if you think otherwise.


I have some questions about tasty file inspectation

This is just some questions, not an issue.
Question 1:
/** This class is needed for runtime inspection--so we can get a Reflection object to pass to unwindType
 */
class TastyInspection(clazz: Class[_]) extends Inspector:

  var inspected: RType = UnknownInfo(clazz.getName)

  // Note: Class arg to TastyInspection drives inspection, NOT tastys List.
  def inspect(using Quotes)(tastys: List[Tasty[quotes.type]]): Unit =
    import quotes.reflect._
    val tastyType = quotes.reflect.TypeRepr.typeConstructorOf(clazz)
    inspected = RType.unwindType(quotes)( tastyType, false )

As we could see, the tasty file is not used in this method, as is described in the note.
What the method do is just to enter a scope where Quotes exists, is it?
That is to say, if I have a class A with A.tasty  and a B.tasty which has nothing to do with A.
I want to reflect the fields in A. If I give the path to B.tasty, the TastyInspection also works, and could also return the correct fields in class A, is it?
Questions 2:
What is the difference between TastyInspection and withQuotes?   They could both obain "Quotes" in their scope.
@gzoller
Thank you very much for this library which helped me a lot.
I have much interest in scala reflection and macro programming, but the official reference could not explain everything such as questions above and I cound not find any better reference. I am very pleased if you could give me some suggestions

I tested the performance of TastyInspection and withQuotes in Question 2,  and the latter one has a better performance, even 5 times better.
So I am wondering if there is any behavior difference beween the two solution?
If there is no difference, I recommand to use withQuotes as the default implementation

···
val tc = new TastyInspection(clazz)
val tastyPathOpt = PathResolver.findPathForClass(classOf[ArrayInfo])
···
I tried to fix the path of tasty to ArrayInfo instead of clazz, and it is still working fine.

I'm not 100% sure about your question, but I think you're wondering why sometimes Quotes exists and other times not.  It's been some time since I looked at this code... but I think the difference is this:   This library was designed to work in 2 distinct modes.  You can enable it as a compiler plug-in to get macro support (which has Quotes available).  Understanding that maybe some folks won't want to do that--they prefer a "stock" compiler--this library still works, but will parse the .tasty files.  This is much slower due to the file IO, however the actual logic is the same, so I think what you're seeing there is provision for handling the same work with/without the Quotes.
You're absolutely right about metaprogramming documentation.  It's... not super.  99% of this library was due to banging my head against a wall and the amazing Gitter support of several key members of the Scala 3 team.  In a few cases I even had to add PRs into Scala 3 itself to plug a hole or two in the Tasty handling.  I use reflection a lot, so I didn't want to crawl through compiler internals each time--wanted a higher-level abstraction like the old reflection in Scala 2, which is what this library attempts to provide.

Thanks very much


Creating a "Resources" page to feature books & videos

Hi,
As Bootstrap is been gaining popularity in Web development and has empowered millions of websites, Packt is proud to publish books on Bootstrap.
This year Packt has published 4 titles on Bootstrap. 3 books and a video
Books:
Bootstrap Site Blueprints Volume
Bootstrap Site Blueprints
Learning Bootstrap 4 - Second Edition
Video:
Building a Responsive Application with Bootstrap [Video]
We would appreciate if you could create a Resources page to feature such content. I think it would really help the Bootstrap community and budding developers.
Let me know what you think.
Many thanks

The Bootstrap Expo already has a Resources page: http://expo.getbootstrap.com/resources/
Feel free to file an issue in the Expo's repository: https://github.com/twbs/bootstrap-expo

Thanks @cvrebert alot


Docker image issue

Hello,
when I type in the terminal, the below shown
docker run -tiv pwd:pwd -w pwd --rm cloudv/fault:latest fault -V
I am getting the following error
docker: Error response from daemon: OCI runtime create failed: container_linux.go:345: starting container process caused "process_linux.go:299: copying bootstrap data to pipe caused "write init-p: broken pipe"": unknown.

Hi: That's usually indicative of the fact you're probably on Linux, using an older kernel than is supported by the Docker image.
Would recommend trying Ubuntu 18.04.

Additionally, you can also update Docker to the latest possible version if you would like to keep your current operating system for the time being.
Please let me know how it goes.

Hi: That's usually indicative of the fact you're probably on Linux, using an older kernel than is supported by the Docker image.
Would recommend trying Ubuntu 18.04.

Yes, I am using Ubuntu 18.04. Still facing the problem!!

Alright, I'm going to need way more details.
Please show me the outputs of lsb_release -a and docker version.

It prints the following:
Client: Docker Engine - Community
Version:           19.03.2
API version:       1.40
Go version:        go1.12.8
Git commit:        6a30dfc
Built:             Thu Aug 29 05:29:11 2019
OS/Arch:           linux/amd64
Experimental:      false
Server: Docker Engine - Community
Engine:
Version:          19.03.2
API version:      1.40 (minimum version 1.12)
Go version:       go1.12.8
Git commit:       6a30dfc
Built:            Thu Aug 29 05:27:45 2019
OS/Arch:          linux/amd64
Experimental:     false
containerd:
Version:          1.2.6
GitCommit:        894b81a4b802e4eb2a91d1ce216b8817763c29fb
runc:
Version:          1.0.0-rc8
GitCommit:        425e105d5a03fabd737a126ad93d62a9eeede87f
docker-init:
Version:          0.18.0
GitCommit:        fec3683

After several tries,now this is happening
when I type in the terminal> fault -V
I am getting this below shown
/usr/bin/fault: 7: [: -V: unexpected operator
Fault 0.2.0. ©Cloud V 2019. All rights reserved.

Thank you!
First, I recommend updating Docker to the latest version using this guide: https://docs.docker.com/install/linux/docker-ce/ubuntu/
The second issue is a bug I have introduced by accident in the latest version and I will push a fix posthaste. I apologize.

Alright, I think fixed the issue. Try now, please. 😅

I am clueless what really happening ??? I personally feel, if there is one script file which may look-into auto installation of all the required things will possibly be the best solution.

We're using Docker for that precise reason. Which, you can install from here: https://docs.docker.com/install/linux/docker-ce/ubuntu/ or by invoking this command: curl -fsSL https://get.docker.com | sh.
Docker runs a full environment with all dependencies preinstalled. It is like a virtual machine.

After typing in the terminal, the following
docker run -tiv pwd:pwd -w pwd --rm cloudv/fault:latest fault -V
it then starts downloading something and finally gives this following error
docker: failed to register layer: symlink ../c6331cd098b1793002eb5bf2b9f52b5807c55f763fb145ca9a621f4f5aab2541/diff /var/lib/docker/overlay2/l/FWJD4SB3EEXD453WLUC5VLI3FL: no such file or directory.
See 'docker run --help'.

If type the following in the terminal
fault -V
then I am getting this below error
/usr/bin/fault: 7: [: -V: unexpected operator
Fault 0.2.0. ©Cloud V 2019. All rights reserved.

my request is

script file which installs all the following,( this script file user uses it and runs in the terminal )
A. docker
B. fault
C. any other that is part of the application

end-user should able to only type the commands related to fault-application to use it.

My docker version
Docker version 19.03.5, build 633a0ea838
which is latest

Given the variance in operating systems, a script file that installs everything and ensures functionality is nigh on impossible, given that even containerized approaches such as Docker are prone to misconfiguration errors on part of the end user. We do not have the resources to commit to such a feature. Docker is the tool we elected to bypass this problem and we stand by it.
When a docker is image is updated, you have to manually update it by typing docker pull cloudv/fault.
Please, once again try these commands specifically:
docker pull cloudv/fault:latest
docker run -tiv `pwd`:`pwd` -w `pwd` --rm cloudv/fault:latest fault -V

I have tested and found these commands to work just now on all of:

Apple macOS
Amazon Linux (our build server)
Lubuntu 18.04.

Given the variance in operating systems, a script file that installs everything and ensures functionality is nigh on impossible, given that even containerized approaches such as Docker are prone to misconfiguration errors on part of the end user. We do not have the resources to commit to such a feature. Docker is the tool we elected to bypass this problem and we stand by it.
When a docker is image is updated, you have to manually update it by typing docker pull cloudv/fault.
Please, once again try these commands specifically:
docker pull cloudv/fault:latest
docker run -tiv `pwd`:`pwd` -w `pwd` --rm cloudv/fault:latest fault -V

I have tested and found these commands to work just now on all of:
* Apple macOS

* Amazon Linux (our build server)

* Lubuntu 18.04.

I am getting the following error
docker: Error response from daemon: invalid volume specification: '/:/': invalid mount config for type "bind": invalid specification: destination can't be '/'.
See 'docker run --help'.

Given the variance in operating systems, a script file that installs everything and ensures functionality is nigh on impossible, given that even containerized approaches such as Docker are prone to misconfiguration errors on part of the end user. We do not have the resources to commit to such a feature. Docker is the tool we elected to bypass this problem and we stand by it.
When a docker is image is updated, you have to manually update it by typing docker pull cloudv/fault.
Please, once again try these commands specifically:
docker pull cloudv/fault:latest
docker run -tiv `pwd`:`pwd` -w `pwd` --rm cloudv/fault:latest fault -V

I have tested and found these commands to work just now on all of:
* Apple macOS

* Amazon Linux (our build server)

* Lubuntu 18.04.

I am getting the following error
docker: Error response from daemon: invalid volume specification: '/:/': invalid mount config for type "bind": invalid specification: destination can't be '/'.
See 'docker run --help'.

upon typing the commands, I am getting the following
zakir@zakir-virtual-machine:~$ sudo docker run -tiv pwd:pwd -w pwd --rm cloudv/fault:latest fault -V
[sudo] password for zakir:
Fault 0.2.1. ©Cloud V 2019. All rights reserved.
zakir@zakir-virtual-machine:~$ fault -V
Command 'fault' not found, did you mean:
command 'faust' from deb faust
Try: sudo apt install

This is another problem
zakir@zakir-virtual-machine:~$ EXEC_PREFIX=</usr/local/bin> sudo ./iverilog_build.swift
bash: ./iverilog_build.swift: No such file or directory

I am getting the following error
docker: Error response from daemon: invalid volume specification: '/:/': invalid mount config for type "bind": invalid specification: destination can't be '/'.
See 'docker run --help'.

upon typing the commands, I am getting the following
zakir@zakir-virtual-machine:~$ sudo docker run -tiv pwd:pwd -w pwd --rm cloudv/fault:latest fault -V [sudo] password for zakir: Fault 0.2.1. ©Cloud V 2019. All rights reserved. zakir@zakir-virtual-machine:~$ fault -V
Command 'fault' not found, did you mean:
command 'faust' from deb faust
Try: sudo apt install

It seems there is a misconception. Docker is not a universal installer. Docker is a containerized environment, again, akin to a virtual machine.
You can run Fault commands by invoking:
sudo docker run -tiv `pwd`:`pwd` -w `pwd` --rm cloudv/fault:latest <command here>

Considering you got the intended output for fault -V, which is Fault 0.2.1. ©Cloud V 2019. All rights reserved., means that this issue is particular issue is resolved.

This is another problem
zakir@zakir-virtual-machine:~$ EXEC_PREFIX=</usr/local/bin> sudo ./iverilog_build.swift
bash: ./iverilog_build.swift: No such file or directory

This has nothing to do with this particular issue. This is the IcarusVerilog build script, which you should not be running if you're using a containerized environment. The containerized environment prepares everything for you.
Nevertheless:
A. You are not in the correct folder. Your home folder has no file called iverilog_build.swift. You need to navigate to the repository first.
B. </usr/local/bin> is not a valid path. By UNIX convention, angled brackets typically indicate that this is a configurable path with a default value of /usr/local/bin. You can elect to leave it blank and just run sudo ./iverilog_build.swift assuming you are satisfied with /usr/local/bin.


[Question] Language support for an existing extension

I was told to ask this here by the github support team:
I'm currently in development of a programming language called "Nuua" (https://nuua.io). Suprisingly, the extension I've chosen to use (.nu) has already been chosen by a lisp-like language called Nu.
Could a new programming language have the same extension and be recognized as Nuua instead of Nu in the github repository depending on it's content?

Could a new programming language have the same extension and be recognized as Nuua instead of Nu in the github repository depending on it's content?

Yup, but Linguist would need a heuristic (regex) that can be used to clearly differentiate between the two languages as detailed in Adding a Language.
Pay special attention to the very first line under the heading... we won't accept a PR if the language doesn't meet this requirement.

Thanks for the response, I'll take it into consideration.


az acr pack build

Is it possible to use other cloud native buildpack pack build cli options with az acr?  Specifically, I would like to specify environment variables on the pack build command like --env NAME=VALUE or is there another way to provide environment variables?
Other useful pack build cli options would be --buildpack and --volume, can they be used with az acr pack build?
https://learn.microsoft.com/en-us/azure/container-registry/container-registry-tasks-pack-build
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 73ff7047-440a-0262-6ea5-e06f0092f576
Version Independent ID: 4df6a6d2-ed03-5207-8a66-50db513c6363
Content: Build image with Cloud Native Buildpack - Azure Container Registry
Content Source: articles/container-registry/container-registry-tasks-pack-build.md
Service: container-registry
GitHub Login: @tejaswikolli-web
Microsoft Alias: tejaswikolli

@kevin-ortega
Thanks for your feedback! We will investigate and update as appropriate.

@kevin-ortega
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly
@tejaswikolli-web
Can you please check and add your comments on this doc update request as applicable.

Thank you for your dedication to our documentation.
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue and creating an internal work item to incorporate your suggestion as time allows. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


42 influx integration

Add basic influx integration. Only sample data is saved.

Many thanks for the good suggestions. I have updated the code accordingly.


rviz: 1.12.4-0 in 'lunar/distribution.yaml' [bloom]

Increasing version of package(s) in repository rviz to 1.12.4-0:

upstream repository: https://github.com/ros-visualization/rviz.git
release repository: https://github.com/ros-gbp/rviz-release.git
distro file: lunar/distribution.yaml
bloom version: 0.5.26
previous version for package: null

rviz
* Restored "Use ``urdf::*ShredPtr`` instead of ``boost::shared_ptr``" (#1064 <https://github.com/ros-visualization/rviz/issues/1064>)
  Now supports ``urdfdom`` 0.3 and 0.4 through a compatibility header in ``urdf``.
* You can now visualize joint axis and display type and limits (#1029 <https://github.com/ros-visualization/rviz/issues/1029>)
* Contributors: Lucas Walter, Robert Haschke, William Woodall

This is just an initial release to unblock other developers. I'll probably have an updated version soon.

thanks @wjwwood !


docs(examples): update list example and list.tape

Issue: https://github.com/ratatui-org/ratatui/issues/846
This PR adds:
for list.rs

subjectively better-looking front
change list example to a todo list example
status of a TODO can be changed, further info can be seen under the list.

for list.tape

change to get better output from the new code

Here is the new output:

I'll take a look at the code a little bit later, just wanted to say that the gif looks awesome!

I'll take a look at the code a little bit later, just wanted to say that the gif looks awesome!

Thanks a lot! I'm not sure should we remove the events list or not. It is irrelevant with the TODO list but kinda useful to have in an example.

I think go ahead and remove the events list. A single good example is worth more than multiple of the same

I think we should also enable the option that always allocates space for the selection to avoid the display jumping.

This is how the example looks after removing events list

I think some of the contrast can be improved (the greens on darker colors particularly) but that's probably just me (I have color issues).

Great job improving all the examples! Excited for all the PRs!

This is the final GIF output.

The only thing that is not addressed is removing the unselecting option. It is cool to have tbh but simplicity is something important. I'm ready to make the needed changes depending on your opinion on this.

I think some of the contrast can be improved (the greens on darker colors particularly) but that's probably just me (I have color issues).

Here are some different greens.
c400:

c500:

c600 (currently used):

c700:

I personally liked the c600 most. An improvement could be also changing the selected text to green if it's completed. Which one do you find best among these?

Great job improving all the examples! Excited for all the PRs!

Thanks! First time contributing to open open-source. This is a big win for me because now I'm able to talk with and learn more from experienced developers. Happy to hear that you're also happy with my work :)

Thank you @Valentin271!

Here are some different greens.

Thanks for experimenting! c400 is the highest contrast and easiest to read for me, but c500 also seems good to me (hard to say exactly without seeing it in while using it in the terminal / seeing the colors change etc). c600 seems a little too washed out for me.
I did put these numbers into a contrast checker and was experimenting with a few values: e.g. https://coolors.co/contrast-checker/16a34a-0f172a
Here's c400:

And c600:

The website says this at the bottom:

Regarding colors, the standard defines two levels of contrast ratio: AA (minimum contrast) and AAA (enhanced contrast). The level AA requires a contrast ratio of at least 4.5:1 for normal text and 3:1 for large text (at least 18pt) or bold text. The level AAA requires a contrast ratio of at least 7:1 for normal text and 4.5:1 for large text or bold text.

It seems like the colors you have are good, it's just a question of how much better it can be in terms of contrast. Especially in the terminal with small text, if we wanted to optimize for readability, I'd recommend a contrast ratio of 7:1 or higher.

It seems like the colors you have are good, it's just a question of how much better it can be in terms of contrast. Especially in the terminal with small text, if we wanted to optimize for readability, I'd recommend a contrast ratio of 7:1 or higher.

I like darker colors in general so that might be the reason why I liked the c600 more. But let's meet in the middle and go with c500 then 👍🏻

Hmm @Emivvvvv I kind of messed up the rebase, could you push your branch again?

I force pushed and that messed up the whole PR I guess 💀

Do you still have the changes on your computer? If so just pushing should be fine. Otherwise I'm really sorry that's my bad ... :/

Should I open a new PR?

Do you still have the changes on your computer? If so just pushing should be fine. Otherwise I'm really sorry that's my bad ... :/

Yes I have the changes. I can open a new PR

Do you still have the changes on your computer? If so just pushing should be fine. Otherwise I'm really sorry that's my bad ... :/

I force pushed from the beginning so I think that messed up this PR. The changes are on the branch but nothing happening here

Probably because its closed, open a new one its fine (I can't reopen it)

Here is the new PR: https://github.com/ratatui-org/ratatui/pull/864


Location tracking do not work in background

I'm using Android 9 and Huawei P20 Light.
App integration works fine, but even after changing power setting for HA app I still cannot get properly working tracking system.
App update location only when I turn it on manually, do not work in background.

I can work in background but it could send last known location due to #490
You can be sure it works in background by checking battery level reported to the same device_tracker entity.


Removed Step 3 as it was no longer applicable

Spoke to dliao who wrote the changeset for this, step #3 no longer works for 2019.  Checking to make sure everything is sync'd is fine. Removed step 3, applicable for SQL 2017 and 2019 which support read-scale AGs.

@SeanGallardy : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#signoff


:slotted() styles for css-modules

Version
3.2.20
Reproduction link
codesandbox.io
Steps to reproduce
:slotted() work with <style scoped>
how to make it work in css-module <style module>
What is expected?
:slotted() works well with scoped styles
expected that work with module styles too
What is actually happening?
how to use :slotted() with

https://v3.vuejs.org/api/sfc-style.html#style-scoped
in vue3 documentation mentioned :slotted() with
it's a question how to do slotted styles with css-module style :class="$style.red"

yeah I know, in a component both scoped and module work
but when using the slot, for styling slotted content with :slotted() only scoped works

it seems that it can not take effect when use :slotted() in <style module>
:slotted in <style scoped> will let content of slot add data-v-xxxx-s and the corresponding css will add [data-v-xxxx-s].
but <style module> just generate the unique classname.
maybe we can generate a unique slot classname and let all of nodes in slot add this classname. and the corresponding css will transform .classname1.uniqueSlotClassname {}
it is just my guess. 😂

This question came to my mind when the slotted module was not mentioned in the vue documentation
so if I cant implement this I will add style module with parent component style module
I want to know more about whether this method can be used or not
YES - No answer can be helpful for me and others that came up with this issue


Develop derived table po_purchase_orders_instances

Join table po_purchase_orders to tables po_lines and inventory_instances.

In progress (MM?)

Natalya has completed this derived table and will submit it shortly.

Closed by #232


bug: Multiple API calls happening on window focus

Provide environment information
version: 0.1.0
Describe the bug
I'm using the scaffolded app which makes the initial API call, but whenever I move focus (click) outside the browser window and then focus back (click) on the browser window a subsequent API call is made. Is this expected behaviour? See image below for multiple API calls in the Network Tab.

Reproduction repo
https://github.com/t3-oss/create-t3-app
To reproduce
Move focus (click) outside the browser window and then focus back (click) on the browser window and a subsequent API call will be made.
Additional information
No response

It's not a bug, its the default behavior of react-query.
You can disabled that: https://tanstack.com/query/v4/docs/react/guides/window-focus-refetching

Thanks @FeMaffezzolli. Will close.


Add Deno support

Description
I know this feature has been asked previously, but Deno recently got an update where you can use NPM specifies. So, I was wondering if this feature is possible now..
Suggested solution
N/A
Alternative
No response
Additional context
No response
Validations

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

It's possible. You can try it with npm create vite-extra@latest --template deno-vanilla


TypeError: Keras symbolic inputs/outputs do not implement __len__

How to solve the following problem？
[tensorflow-gpu==2.10.1, keras==2.10.0]
Traceback (most recent call last):
File "targeted_UAP_CIFAR10/generate_noise.py", line 90, in
classifier = KerasClassifier(model=model)
File "adversarial-robustness-toolbox/art/classifiers/keras.py", line 84, in init
self._initialize_params(model, use_logits, input_layer, output_layer)
File "adversarial-robustness-toolbox/art/classifiers/keras.py", line 171, in _initialize_params
if predictions == self._input:
File "miniconda3/envs/python3.7/lib/python3.7/site-packages/keras/engine/keras_tensor.py", line 245, in len
"Keras symbolic inputs/outputs do not "
TypeError: Keras symbolic inputs/outputs do not implement __len__. You may be trying to pass Keras symbolic inputs/outputs to a TF API that does not register dispatching, preventing Keras from automatically converting the API call to a lambda layer in the Functional Model. This error will also get raised if you try asserting a symbolic input/output directly.

Sorry for late reply.
Our repository assumes tensorflow-gpu==1.15.5 and Keras==2.2.5.
For TF2, ART library, in which our targeted UAP is available, needs to be used.
Please see the following notebook for details.
https://colab.research.google.com/drive/1ay_4otbY1qHyWvYdR5vqMMUk4tABCz7D?usp=sharing


Fix copy operator of KeyboardManipulator

to be compatible with osg 3.2.0 and 3.4.0

@pFernback

I don't really know, I write it this way because it was the same in nearly all CameraManipulator.
But I confirm that your pull request still work with osg 3.4.


More request options and powerful jq commands

This PR adds:

transparent parameters for request (e.g. method: "POST")
jq processing of incoming data
multiple MMM-json instances support

This allows me, and others of course, to do really powerful JSON conversions.
I will be using it to display next episodes per TV show available on my Kodi. Example below.
NOTE: The sample provided in the README is dependent on my PR for node-jq.
Kodi example
{
  module: "MMM-json",
  position: "bottom_left",
  header: "TV Shows",
  config: {
    url: "http://user:password@kodi/jsonrpc",
    request: {
      method: "POST",
      json: {
        "jsonrpc":"2.0",
        "method": "VideoLibrary.GetEpisodes",
        "id": "neweps",
        "params": {
          "properties": [
            "title",
            "playcount",
            "showtitle",
            "season",
            "episode"
          ],
          "filter": {
            "field": "playcount",
            "operator": "is",
            "value": "0"
          },
          "sort": {
            "order": "ascending",
            "method": "tvshowtitle",
            "ignorearticle": true
          }
        }
      }
    }
    jq: ".result.episodes | [group_by(.showtitle)[] | {(.[0].showtitle): .[0].label}] | add"
  }
}

This uses the Kodi JSON-RPC to search for unwatched episodes. The result (see below) is converted using a jq command to select only the last episode per show and convert it to a JSON object. Which I don't think would be possible with JSONPath.
{
  "id": "neweps",
  "jsonrpc": "2.0",
  "result": {
    "episodes": [
      {
        "episode": 1,
        "episodeid": 3715,
        "label": "9x01. Episode 1",
        "playcount": 0,
        "season": 9,
        "showtitle": "Call the Midwife",
        "title": "Episode 1"
      },
      {
        "episode": 2,
        ...
        "showtitle": "Call the Midwife",
        ...
      }
      ...
      {
        "episode": 8,
        "episodeid": 3890,
        "label": "3x08. Crisis Theory",
        "playcount": 0,
        "season": 3,
        "showtitle": "Westworld",
        "title": "Crisis Theory"
      }
    ],
    "limits": {
      "end": 100,
      "start": 0,
      "total": 100
    }
  }
}

{
  "Call the Midwife": "9x01. Episode 1",
  "Westworld": "2x01. Journey into Night"
}

Nice! I will have a look in the evening.

 Which makes node-jq not work on a Pi...
So this is going to take some more work.
I'll keep you posted

Maybe use https://github.com/FGRibreau/jq.node ?

jq.node is indeed quite a nice alternative.
I wasn't familiar with it, but it seems to integrate into NodeJS a lot better than jq.
PR updated to use jq.node.


adding all files

I have added all files

@sahilsingh7735  after login the page why is showing me the home screen not the chat

check the code


Check submissions

check if a bot has already been submitted (and has a type).

Check if it exists on twitch.


Use the created System.Func rather than through the FSharpFunc where we know the specified Type.

If no one has an issue with this, I'll merge it over the next few days.

Insufficient performance improvement. (and I've lost my own momentum in tuning..._


Wadm 0.4

This is an administrative PR to mark us merging wadm 0.4 into the main branch!

Please note some tests will fail because they'll be running the old elixir tests on main


extradata column

new functions for improving and managing 'extradata' column
update admin picture for managing and saving it on 'extradata' column
update profileController and views for file uploading and showing admin picture

good job


Fix issue from broken PR links

When the Azure DevOps Services still uses the old URL (account.visualstudio.com), the links to the PRs are breaking.

Fixed #47


Add support for Foreach statement in C#

Hello @olafurpg,
I would be happy to get a review for this PR when you have some time.

Conflict resolved.
Thanks a lot for your reviews!

Thank you! https://github.com/sourcegraph/scip-dotnet/releases/tag/v0.2.3


query biolink model within ubergraph

biolink-model was recently added directly to ubergraph. It should be faster to use in queries this way instead of populating VALUES lists.

This is done.


macos native input method, press caps lock to switch Chinese to English in terminal, it will show English content twice

VSCode Version:1.51.1
OS Version: Darwin x64 20.1.0

Steps to Reproduce:

Does this issue occur when all extensions are disabled?: Yes

I encountered the same problem as this issue(https://github.com/microsoft/vscode/issues/112349), is there any way to solve it now?
It is normal to use the same version under win10.

Vscode Version: 1.52.1
Os Version: 5.9.16-1-MANJARO(arch)

I encountered the same problem as this issue(https://github.com/microsoft/vscode/issues/112349), is there any way to solve it now?
It is normal to use the same version under win10.

Vscode Version: 1.52.1
Os Version: 5.9.16-1-MANJARO(arch)

This issue should be specific to macOS, #92415 is the same issue.
@jinnzy For Linux distros we have #96041 and #76542

#87499 might also be the same.
The key marked 中/英 (Chinese/English) in mac Chinese keyboard layout is the same key as CapsLock in US layout.

/duplicate  #92415


Bazel Basic Tests for Python (Local) failing

Bazel Basic Tests for Python (Local) is failing for the PR: #26471. The failure seems to be unrelated to the changes in the PR.
Log: https://source.cloud.google.com/results/invocations/ccd03a11-1686-48b1-9f22-48a383d75fee/targets/grpc%2Fcore%2Fpull_request%2Flinux%2Fgrpc_python_bazel_test/log

Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/3326/execroot/com_github_grpc_grpc/bazel-out/k8-py2-fastbuild/bin/src/python/grpcio_tests/tests/interop/_insecure_intraop_test.python2.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests/interop/_intraop_test_case.py", line 36, in testClientStreaming
    methods.TestCase.CLIENT_STREAMING.test_interoperability(self.stub, None)
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/3326/execroot/com_github_grpc_grpc/bazel-out/k8-py2-fastbuild/bin/src/python/grpcio_tests/tests/interop/_insecure_intraop_test.python2.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests/interop/methods.py", line 451, in test_interoperability
    _client_streaming(stub)
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/3326/execroot/com_github_grpc_grpc/bazel-out/k8-py2-fastbuild/bin/src/python/grpcio_tests/tests/interop/_insecure_intraop_test.python2.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests/interop/methods.py", line 107, in _client_streaming
    response = stub.StreamingInputCall(requests)
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/3326/execroot/com_github_grpc_grpc/bazel-out/k8-py2-fastbuild/bin/src/python/grpcio_tests/tests/interop/_insecure_intraop_test.python2.runfiles/com_github_grpc_grpc/src/python/grpcio/grpc/_channel.py", line 1131, in __call__
    return _end_unary_response_blocking(state, call, False, None)
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/3326/execroot/com_github_grpc_grpc/bazel-out/k8-py2-fastbuild/bin/src/python/grpcio_tests/tests/interop/_insecure_intraop_test.python2.runfiles/com_github_grpc_grpc/src/python/grpcio/grpc/_channel.py", line 849, in _end_unary_response_blocking
    raise _InactiveRpcError(state)
_InactiveRpcError: <_InactiveRpcError of RPC that terminated with:
	status = StatusCode.UNAVAILABLE
	details = "Broken pipe"
	debug_error_string = "{"created":"@1625848679.987405821","description":"Error received from peer ipv4:<IP_ADDRESS>:33901","file":"src/core/lib/surface/call.cc","file_line":1070,"grpc_message":"Broken pipe","grpc_status":14}"
>

Looks like a one-off broken pipe. This test is still green on master. Waiting until we see this again.


Cabrillo - export needs to be fixed to be compatible

Today I joined the DARC FT4 contest and tried to export the QSO to a Cabrillo file.
Unfortunately, the created file did not comply with the requirements of the DARC contest hub (https://dxhf2.darc.de/~ft4log/upload.cgi?form=referat&lang=en)
The following issues appeared and needed to be fixed manually:

Mode must not be "MFSK" but "DG" when logged QSO was in FT4. As far as I know MFSK is never allowed.
QSO must contain exch with value of locator(S) & RST (S)/locator(R) & RST (R)

I attached the original file and my adjusted file
adjusted.cbr.txt
original.cbr.txt
Any chance to get this changed or at least make the export customizable?
My used Wavelog version: 1.8.7
Thanks

Hello @205er
You're right. Related to the official Cabrillo Documentation it should be DG and not MFSK
https://wwrof.org/cabrillo/cabrillo-qso-data/

wow, that was fast!
Thanks for your great support!


Updating readme sajidmahamud835

The readme file became backdated as GitHub changed its default to main. I've fixed this to help new contributors.

replacing master with main on readme
replacing m the license URL with the correct one

The branch name and other README related updates are usually automatically synchronized with the main reactjs.org repo by @react-translations-bot .
Closing this for now as the change mentioned in the PR is currently already synchronized thorough @react-translations-bot .
Really appreciate the effort you put in. It would be great if you could pick up any pending documents in #1 if you are looking to contribute more @sajidmahamud835 . :smiley:


StoreService::getShopwareVersion removed in v<IP_ADDRESS>

PHP Version
7.4
Shopware Version
<IP_ADDRESS>
Expected behaviour
Based on the deprecation annotation in the function signature, the expected behavior is that this was going to be removed in v6.5.0
Actual behaviour
Function was removed in v<IP_ADDRESS>.
How to reproduce
After installing the Adyen plugin (or any other plugin that calls this function), the following error will be thrown on the checkout page:
Uncaught PHP Exception Symfony\Component\ErrorHandler\Error\UndefinedMethodError: "Attempted to call an undefined method named "getShopwareVersion"

We are aware that we should not rely on this function anymore (based on the deprecation annotation), but it seems that this is a breaking change introduced in a feature release.
Related issue
Best regards,
Jean
Adyen

Hi @shyim,
Thanks for the quick response!
Best regards,
Jean
Adyen


this.$inertia.put not working. Laravel update method does not getting model and request data

this.$inertia.put(this.route('users.update', this.user.id), formData).then(() => {
this.sending = false
})
Laravel :
public function update(Request $request, User $user)
{
return  $request->all(); // $request->name // $user
}
did not get any data to request or $user model...
Form Data sending to the exact route

response from update method

***Store method working fine. I am sending data exact same way  ****

What does formData look like? Where is it populated?
I think this is something related to the browser sending a FormData object instead of JSON (not sure, just a hunch)

submit method >>>
       var formData = new FormData();
        formData.append("name", this.form.name || "");
        formData.append("email", this.form.email || "");
        formData.append("password", this.form.password || "");
        formData.append("password_confirmation", this.form.passwordConfirmation || "");
        formData.append("status", this.form.status || false);
        formData.append("phoneNo", this.form.phoneNo || "");
        formData.append("role", this.form.role || null);
        formData.append("profilePhoto", this.form.profilePhoto || "");
        this.$inertia.put(this.route('users.update', this.user.id), formData)
            .then(() => {
            this.sending = false
            })

submit method >>>
       var formData = new FormData();
        formData.append("name", this.form.name || "");
        formData.append("email", this.form.email || "");
        formData.append("password", this.form.password || "");
        formData.append("password_confirmation", this.form.passwordConfirmation || "");
        formData.append("status", this.form.status || false);
        formData.append("phoneNo", this.form.phoneNo || "");
        formData.append("role", this.form.role || null);
        formData.append("profilePhoto", this.form.profilePhoto || "");
        this.$inertia.put(this.route('users.update', this.user.id), formData)
            .then(() => {
            this.sending = false
            })

And the same for store method and that's working as expected.

Append a method field to the formData and send a post request instead and it should be fine.
Check how it's done in PingCRM below.
https://github.com/inertiajs/pingcrm/blob/master/resources/js/Pages/Users/Edit.vue#L86-L95

What @Juhlinus said!
Fyi, here's the underlying issue: https://github.com/laravel/framework/issues/13457

Append a _method field to the formData and send a post request instead and it should be fine.
Check how it's done in PingCRM below.
https://github.com/inertiajs/pingcrm/blob/master/resources/js/Pages/Users/Edit.vue#L86-L95

Thank you very much. I had same issue but that worked for me.

Append a _method field to the formData and send a post request instead and it should be fine.
Check how it's done in PingCRM below.
https://github.com/inertiajs/pingcrm/blob/master/resources/js/Pages/Users/Edit.vue#L86-L95

Thanks!


Add 'currrency' to 'shipping_discount' method

The added currency parameter for shipping_discount can have multiple use cases such as possible multi-currency feature integration by dependant projects.

This looks good.
Remember to use a descriptive (as to the intention of the patch) commit message when merging to master.


Adjust Class hierarchy add synonyms to caudal ganglionic eminence derived interneuron

I think this term caudal ganglionic eminence derived GABAergic cortical interneuron should just be a synonym of caudal ganglionic eminence derived interneuron and not a different term.
I had a look at a few papers, and they do not really describe CGE-derived interneurons that are not GABAergic. I can look a bit further if a discussion is needed.
cell label:
caudal ganglionic eminence derived interneuron
parent terms
interneuron
downstream terms:
vip GABAergic cortical interneuron
lamp5 GABAergic cortical interneuron
Lamp5 Lhx6 [NTR - #2345 ]
PAX6 neuron [NTR]
sncg GABAergic cortical interneuron
vip GABAergic cortical interneuron
anatomical location
neocortex
Text definition (already present):
A GABAergic cortical interneuron that develops from the caudal ganglionic eminence and has migrated to the cerebral cortex.
database_cross_reference
PMID:20130169
PMID:37824663
PMID:37824655
PMID:20130169
has_a_related_synonym
CGE-derived cell
CGE-derived GABAergic INs
has_a_synonym_type
abbreviation
database_cross_reference
PMID:27034423
has_a_related_synonym
CGE-derived interneuron
has_a_synonym_type
abbreviation
database_cross_reference
PMID:20130169
PMID:37824655
has_a_related_synonym
CGE cortical interneuron
database_cross_reference
PMID:20130169
has_a_related_synonym
caudal ganglionic eminence derived GABAergic cortical interneuron
database_cross_reference
PMID:20130169
has_a_related_synonym
CGE interneuron
has_a_synonym_type
abbreviation
database_cross_reference
PMID:37824663
symbol
CGE interneuron
@aleixpuigb @Caroline-99 @JABelfiore to review, once it is good to go, I will turn this into a PR

Decision:

Merge.
Use term name from general class (as all, or almost all mammalian interneurons are GABAergic, we don't need this in the name)
Use EquivalentTo axiom from more specific class


Files cannot be accessed by shareFilesFromDirectory method

The code is
server["/"] = shareFilesFromDirectory(MyPath)    server.start(8080)
Then open browser and type "http://<IP_ADDRESS>", nothing happened?

Did you try http:<IP_ADDRESS>:8080 ? or localhost:8080.

@SmileAppDev Yes, I tried.

Did you include the port number in the address?
Were you connecting from the same machine that you were running the code on?
If it was on iOS, was the app in the foreground when you attempted to connect?


Backups documentation not clear enough

URL for relevant page?
https://docs.influxdata.com/influxdb/v0.13/administration/backup_and_restore/
What products and version are you using?
InfluxDB 0.13
Where did you look before opening the issue?
The docs
The current backup docs dont answer a lot of questions such as:
Scenario: My dedicated server burns down, I only have backups and I am planning to restore them on a new server.

Does metastore backup continues queries as well? Or I will need to re-create them?
Lets say it took me 1 hour to setup the new server. After restoring I insert all the missing data for the 1 hour the DB was down. Will CQ process the data correctly since its technically from the past?
In the above scenario do I need to do anything else to get a new server with 1:1 data as the old one except from the commands below?

S1: influxd backup /path/to/metadata
S1: influxd backup -database mydatabase /path/to/backup
S2: influxd restore /tmp/backup
S2: influxd restore -database mydatabase /tmp/backup

Bonus: What stops me from doing rsync on the directories used by InfluxDB? I could keep a second server in sync and if something happens to master I could simply do service start influxdb on the second server to get up and running without doing any backup restores.
Thank you!

It seems that there are certain requirements the backups have with certain flags that the docs do not inform customers of.
For example, the –shard flag needs the –db flag unless there is only one shard per db.
Also, the –full flag timed out restoring the same shards as the incrementals without it, hard to say given the same conditions that the –full flag should be tamed with –db and/or –shard as well.
The –full flag is either broken or simply not working as its supposed to based on the docs.
So, just running without the flag, an incremental of each db shards seems to do the trick.
When trying to restore each shard without the –db flag, it resulted in a conflict when it tried to restore shards within the same db, so it seems that the db flag must be required to restore each shard individually. Most of this is not indicated in the docs.

Closing as legacy backup documentation has been deprecated. New enterprise-compatible backup and restore is documented and was reviewed.


Cleanup and simplify EL builds.

Remove EL6 builds.
Remove MBR builds which are deprecated.
Revamp EL7 kickstart configs to get rid of old EL6 constructs and cleanup to mimic EL8 config.
EL7 uses chrony instead of ntpd - remove ntpd.
Add CentOS Stream 8 config.
Simplify kickstart file generator.

/assign @gaohannk

/assign @gaohannk

/lgtm

/lgtm


SQL Migration Extension Version bump up

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.1%) to 41.758%

Totals

Change from base Build<PHONE_NUMBER>:
-3.1%

Covered Lines:
30827

Relevant Lines:
69081

💛  - Coveralls


fix cap error message

fix #1028

@wonderflow Thanks for approve.
And how to trigger a test ?
Maybe like add some rebot to /test Go / unit-tests (pull_request)  ?

@wangkai1994  I triggered, you can change your commit message and force push if you want to manually trigger it from your side.


improve event alert

Problem:

Alert resolved time is 5 mins, alert get deleted in alertmanager if the same event did not happen in 5 mins. Event alert is special, an event will trigger only one time, extend the repeat interval then the event alert will be resolved after resolve time.
Event for the same resource with the different message will be placed to the same group, alertmanager will send notifications when new alert adds to this group, this notification includes all alerts in this group. If want user only sees current happen event without others in the same group, we need to add event message to the group key.

Solution:

Add default repeat time for event
Add message to event group key

Issue:
https://github.com/rancher/rancher/issues/19302
https://github.com/rancher/rancher/issues/19058

LGTM

LGTM


Fix the project code style, as it is not consistent.

Fixes #.
This PR fixes code style for the project file.
The Problem
Different files in the project use different code style. That is not consistent and leads to harder maintenance of the project.
Description of the changes:

Have investigated and determined the most used code style across the given codebase
Have configured IDE and applied code style to all project files.
Some cases were fixed manually

How changes were validated:
manual/ad-hoc testing, automated testing

All tests pass as before because these are only code style changes.

Additional
Please review, and let me know if I have any mistake in the code style. If any mistake I will change the configuration and re-apply it to the project.

that was training :)


 I've bumped it in #19 already (during some other changes today).
I didn't mean to ignore this PR at all, just noticed it after the fact.
Sorry about that 🙏
Nevertheless, thanks for being willing to contribute 👍 ❤
../Frenck


Employee Call Out List Self Sign Up

Every week, a call out list is generated for weekend call outs.  A paper sign up sheet is posted.  Employees sign up.  The list is distributed to supervisors before the weekend.
A simple kiosk page could be made to allow employees to select an "open" call out list they are eligible to be on, and add themselves to it.

To be released in v1.0.0-alpha.5


CI: Test go-ceph with unreleased versions of Ceph

As the needs of the projects that use go-ceph evolve we're finding more and more desire to implement things that are on the bleeding edge of Ceph development, rather than going back and filling in long standing gaps in the API coverage.
This means that people are requesting APIs and contributing patches for features that don't exist in released versions of ceph.  This might include a change that has been backported to ceph pacific or ceph quincy but not included in a release version of pacific/quincy/etc. This may even entail changes being contributed to ceph's main branch that is not being planned for backport and is intended for reef or some other future version.
We would like to extend the kinds of coverage we have for leading edge versions of ceph but also make sure that we make no "dangerous" promises to consumers of our library in case APIs change in these development versions.  Running unreleased versions of ceph may even help us find bugs in ceph that we can then report and get fixed prior to a release.
My suggestion would be to add three new jobs to the ci: prerelease pacific, prerelease quincy and, latest main. These are where ceph development is active. Older named versions shouldn't need to track pre release builds. We'll need to make changes to the build scripts/ci jobs to be able to consume container images from the Ceph CI rather than the release builds only.
In addition, we'll need to establish a pattern similar to ceph_preview and the recent "hack" ceph_ci_untested tags. To distinguish features that have been tested but are still only tried on unreleased versions of ceph. I'm thinking something like ceph_development_<name>. I think having the ceph codename as part of the tag here will be necessary as we can't guarantee simultaneous backports to all unreleased branches and so we need to indicate what new APIs should be paired with what unreleased version.
Additionally, we may need to think about how this affects our merge rules. Right now the pacific run is very flaky and test flakes interfere with our ability to get things merged. Increasing the number of jobs, especially those of unreleased ceph, will increase the likelyhood that something in the batch will fail. We may need to consider using something like githhub labels to change what jobs must pass for a given PR to be merged.

@ansiwen - It's long but I'd like your feedback. It's also long because I am hoping that maybe someone wants to volunteer to help with this can use it as a rough plan of action.

It looks like quay.ceph.io/ceph-ci/ceph:main, quay.ceph.io/ceph-ci/ceph:quincy and quay.ceph.io/ceph-ci/ceph:pacific are built and updated post any merge to corresponding branches(main, quincy and pacific) in ceph repository. ceph-dev-build job's configuration responsible for this process are maintained at ceph-build repo which ultimately consumes ceph-container to build container images.

:+1:

So we are good to go with the above mentioned images as pre-release versions. As soon as we have go build tags(ceph_development_quincy, ceph_development_pacific etc) distinguishing features from unreleased branches we can start with adding BUILD_TAGS to Makefile.

We can start by updating the existing set of "untested" from PRs: #698 #691
Convert them to the new planned build tags and use them to test the dev ci jobs


[BUG] Breadcrumb Not Rendering Dynamic Segments, e.g. :projectId

Describe the bug
The breadcrumb component is not rendering dynamic segments. See the screenshot below:

Steps To Reproduce
N/A
Expected behavior
Should render dynamic segment.
Packages
All to latest.
Additional Context
No response

Just to be clear, by breadcrumb components you mean the Projects / Applications part? What specific dynamic segments are you referring to?

I also wanted to clarify as to how you got to the projects route, did you make that route yourself or load it from somewhere else because i don't see it listed as one of the routes in the repo.

Just to be clear, by breadcrumb components you mean the Projects / Applications part? What specific dynamic segments are you referring to?

That is correct I mean the Projects / Application part - the missing part is the Projects / :projectId / Applications as shown in the URL in the screenshot. See the resources:
  createResource("dashboard", {
    meta: {
      hide: true,
      parent: "projects",
    },
  }),
  createResource("applications", {
    meta: {
      --- SNIP ---
      hide: true,
      parent: "projects"
    },
  }),

In short, the  don't seem to parse dynamic segments like :projectId and the documentation has no information about that and there is no example with dynamic path - because it is not supported.

@alicanerdurmaz exactly what I want and what I would expect the breadcrumb to do. :)
Is there a way to make it work as it is now? Haven't found a solution in the doc and digging the code, it seems that this is not supported as it is not parsing url params to replace placeholders like :projectId.

There is no way to do that right now. I wrote hard-coded code to explain the feature.
We need to implement this on useBreadcrumb hook.
Our hands are full with other tasks, but we're always open to PRs 🙌

Is there a data stored in relation to the unique uuid in the URL? Eg: some readable name to be shown in the breadcrumb area? from where we can extract the readable name from the uuid and show it there in the breadcrumb

@darshan-mundra-08 should be define in resource routes? Can we make parent accept strings like ":projectId"?

@aress31 No, Its not important that it should be in the source resource routes? the data should be defined anywhere in the project whose variable are accessible to the breadcrumb component.

Can I work on this guys?

Hello, @darshan-mundra-08 or @aress31 Do you want to work on this? If you don't I will assigned to @ShashankGupta10

@alicanerdurmaz  I would love to contribute to it if you want.

@alicanerdurmaz feel free to assign @ShashankGupta10 or @darshan-mundra-08 for this ticket.

I assigned to @darshan-mundra-08 because @darshan-mundra-08 asked first. Thank you for improving Refine 🚀

Thanks @alicanerdurmaz @aress31


Rapdily cycling through commits with down arrow results in application crash.

Version: v1.3.0
OS: Ubuntu 22.04
Package: Flatpak
Steps to reproduce:
-Open a repo with many commits (mine has about 75).
-on the left hand pane, focus the first commit with the mouse.
-press and hold the down arrow key, to start cycling down through the commits.
-after cycling down through about 4-5 commits, the application crashes.
I launch the app with flatpak run [full gittyup name], and no log messages were logged on crash.

Hi,
perhabs currently the log messages are only available in debug mode
https://github.com/Murmele/Gittyup/blob/8a0dd90f37ade7cf1bff583732721f64e5543aa9/CMakeLists.txt#L97-L101
Can you try the build comming out from this MR?
https://github.com/Murmele/Gittyup/pull/555
There I enabled Debug Messages


Error Logging to a file

add functionality to ODM to log if there are any errors to make it easier for the  development team to debug and make it so users don't have to be given the runaround to get them the information they need.

Putting this on the first milestone. @sreeder will evaluate the time commitment to implement.

@AmberSJones the log file should be located in the default log location on your system- the same as the config file location. let me know if you need any more info for testing this

I've tested this. Appears to be working well.


chore: update referral form details

Review Checklist

Summary*

[ ] fix: Bug fix for Add the items tackled here as checklists
[x] feat: Completed task
[ ] chore: Incomplete task

[ ] test: Sub-task 1

Structure*

[ ] The Pull Request has a proper title that conforms to our MR title standards
[ ] The Pull Request has one commit, and if there are more than one, they should be squashed
[ ] The commit should have a proper title and a short description
[ ] The commit must be signed off
[ ] Unused imports are not present
[ ] Dead code is not present
[ ] Ensure dry running library to confirm changes

Tests

[ ] Proper and high quality unit, integration and acceptance(if applicable) tests have been written
[ ] The coverage threshold should not be lowered

Sign off*

[ ] All comments have been resolved by the reviewers
[ ] Approved by Czar {replace_with_czar_name}
[ ] Signed off by second reviewer {replace_with_name}
[ ] Ensure all checks are done before merging :warning:
[ ] All PRs needs to be signed before merging :warning:

N/B:

Add a checklist if more than one item is done.
Add screenshots and/or images where necessary
Indicate any breakages caused in the UI :exclamation:
Where necessary, indicate which issue the Pull Request solves (Closes #)
Any new files are updated in the folder structure in the README

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

29 of 29 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 82.229%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
7542

Relevant Lines:
9172

💛  - Coveralls


Canonical order for Z* extensions

Table  27.1  has listed Zicsr, Zifencei, Zam, Ztso and Zx*, and said it's also canonical order, does it mean IMADZifencei_C is correct and IMADCZifencei is incorrect order ? if later one is correct, table 27.1 might need to put all Z* in later row?
And further order question:

Order between Zxm* and Z*? I assume in general Zxm* appear after Z* because alphabet order, but in case we might have Zz* extension in future?
Order between Z* and others (S, H and X)

This is a bug in the table, or perhaps a bug in the caption.  IMADCZifencei is valid; IMADZifencei_C is not.
I think we should clearly state that Z extensions follow all other standard extensions (i.e. after all other letters except X), and X extensions follow all standard extensions (i.e. after all other letters).
Let's keep the rules simple and have Zxm ordered alphabetically.  It's OK if Zz extensions come after Zxm extensions.

Got it, there is some people begin implement -march parser for Z* on B taskgroup[1], so I think it's time to implement that on binutils/gcc.
[1]https://github.com/riscv/riscv-bitmanip/issues/38

Agreed.
I've fixed the table, and I fixed the text to indicate X-extensions must come last.  This is not a complete fix, but it will help avoid some confusion.  Can you check this commit and see if you agree with it? e87173489d7cf3818aecaf397b7e54f787cc2d31

Hmm, order between Zicsr, Zifencei, Zam and Ztso should in alphabet order in table 27.1?
Zam, Zicsr, Zifencei, Ztso

I forgot that Z extensions are ordered the same way as base extensions:
that is, Zi Zm Za Zf Zd Zc...
So yes, this issue is still not resolved.
On Tue, Jul 23, 2019 at 8:46 PM Kito Cheng<EMAIL_ADDRESS>wrote:

Hmm, order between Zicsr, Zifencei, Zam and Ztso should in alphabet order
in table 27.1?
Zam, Zicsr, Zifencei, Ztso
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/riscv/riscv-isa-manual/issues/415?email_source=notifications&email_token=AAH3XQXEA6MBVDQMYSLG5ETQA7GCNA5CNFSM4IGEKLTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2VCP7Q#issuecomment-514467838,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAH3XQRU6CUAA55MIVLM7V3QA7GCNANCNFSM4IGEKLTA
.

Alphabetical is simple and predictable, allows coding now for any future extension. One option is to be a bit disruptive and apply this to all including base extensions. Short of that, what about only specifying ordering of currently commonly used and ratified base extensions (IEMAFDC, E is not common but takes the place of I) and leaving everything else alphabetical?  Including among Z*, and possibly X* ordered last.
The simplicity in understanding and coding seems worth a slight bit of disruption.

There are already many upstream tools that enforce the current IMAFD...
ordering, so it would be extremely disruptive to change that.
Since nothing depends on the current Z ordering AFAIK (unless @asb knows
otherwise?), now is a good time to make sure we are getting that part
right. Making it fully alphabetized seems like a good simplification to me.
On Wed, Jul 24, 2019 at 6:47 AM Marc Gauthier<EMAIL_ADDRESS>wrote:

Alphabetical is simple and predictable, allows coding now for any future
extension. One option is to be a bit disruptive and apply this to all
including base extensions. Short of that, what about only specifying
ordering of currently commonly used and ratified base extensions (IEMAFDC,
E is not common but takes the place of I) and leaving everything else
alphabetical? Including among Z*, and possibly X* ordered last.
The simplicity in understanding and coding seems worth a slight bit of
disruption.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/riscv/riscv-isa-manual/issues/415?email_source=notifications&email_token=AAH3XQWQVV446OTMFBIWZZDQBBMQNA5CNFSM4IGEKLTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2WMNOY#issuecomment-514639547,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAH3XQTXKQPD2KC2T367JA3QBBMQNANCNFSM4IGEKLTA
.

For conciseness, the letters are context-sensitive and the canonical
ordering is meant to avoid having to contemplate all partial orders.
E.g., RV32IDC - what is in C, depends on whether D is present, whereas
D does not depend on C.  Or more simply, RV32ID and RV64ID -what D
means varies depending on prefix.
My original thought was that Z followed this model, and separately we
have fully disambiguated strings in a feature map (similar to what
Nikhil has been working on in context of compliance).  The fully
disambiguated strings would look something like:
"rv32i-f-fadds"
Krste

On Wed, 24 Jul 2019 10:40:52 -0700, Andrew Waterman<EMAIL_ADDRESS>said:

| There are already many upstream tools that enforce the current IMAFD...
| ordering, so it would be extremely disruptive to change that.
| Since nothing depends on the current Z ordering AFAIK (unless @asb knows
| otherwise?), now is a good time to make sure we are getting that part
| right. Making it fully alphabetized seems like a good simplification to me.
| On Wed, Jul 24, 2019 at 6:47 AM Marc Gauthier<EMAIL_ADDRESS>| wrote:
|| Alphabetical is simple and predictable, allows coding now for any future
|| extension. One option is to be a bit disruptive and apply this to all
|| including base extensions. Short of that, what about only specifying
|| ordering of currently commonly used and ratified base extensions (IEMAFDC,
|| E is not common but takes the place of I) and leaving everything else
|| alphabetical? Including among Z*, and possibly X* ordered last.
||
|| The simplicity in understanding and coding seems worth a slight bit of
|| disruption.
||
|| —
|| You are receiving this because you commented.
||
||
|| Reply to this email directly, view it on GitHub
||
| https://github.com/riscv/riscv-isa-manual/issues/415?email_source=notifications&email_token=AAH3XQWQVV446OTMFBIWZZDQBBMQNA5CNFSM4IGEKLTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2WMNOY#issuecomment-514639547
| ;,
|| or mute the thread
|| https://github.com/notifications/unsubscribe-auth/AAH3XQTXKQPD2KC2T367JA3QBBMQNANCNFSM4IGEKLTA
|| .
||
| —
| You are receiving this because you are subscribed to this thread.
| Reply to this email directly, view it on GitHub, or mute the thread.*

I believe this issue is clarified and both implemented on GNU and LLVM toolchain.


add order as bill classification choice

Coverage remained the same when pulling 99d62b24a8c6197accfef8650681a82689ee4bea on fgregg:patch-1 into 12624b2de81bf75bda55120023599195088091e0 on opencivicdata:master.


[Bug] Players with an applied HumanoidDescription do not fire player events

Hey! I recently spent a while debugging why my code wouldn't react to the playerEntered and playerExited events, and after a bit of digging, I found out that when I applied a HumanoidDescription to the player, the player was not recognized by any zones on the server (possibly the client too).
A repro game is linked below:
https://www.roblox.com/games/8699903286/ZonePlus-Repro
Comment out line 15 in CharacterSetup in ServerScriptService to make the zone work again.
This bug is reproducible on the latest version of ZonePlus, v3.2.0.

Hi sixfalls, thanks for the repo place! I've played around at the place for a bit and even when trying different start packages, delaying the morph, repeating the applying of different descriptions, it still appears to work fine for me. Are you happy to provide a video or some images with further details? Thanks!

Would it be alright for you to try the same thing at the repo place but on an alt or new account? You might be right about it being a beta feature or a certain studio configuration.

Looks like it's just the headless head that makes it not fire; I did some testing and it works fine when I don't have headless equipped.

Bug is reproducible with all beta features disabled, here's a video:
https://streamable.com/54of83

What you mean specifically by headless head? The Headless Horseman head body part for example, or literally no Head instance within the character?
https://thumbs.gfycat.com/SeparatePointlessHomalocephale-mobile.mp4

The headless horseman head, which you can equip on the avatar shop. I'll try to update the repo place to use a custom avatar with headless.

Ah found it, turns out this bug was being created from a race condition between AncestryChanged and the new replacement Head being swapped in on the same frame.
To fix, simply hop into the 'Tracker' module (under ZoneController) and change line 208 from self:update() to task.defer(self.update, self)``.
Thanks for your help finding this bug, I'll officially publish this fix alongside the next ZonePlus release.
If this covers everything, feel free to close the issue.

Thanks for the help! The fix worked fine.

@1ForeverHD Sorry for the bump; this issue still occurs when accuracy is set to High. Either keeping the accuracy on default fixes it, or not applying the HumanoidDescription at all.


[storage-file-datalake] Migrate to core-rest-pipeline

Packages impacted by this PR
@azure/storage-file-datalake
Issues associated with this PR
#15813
Describe the problem that is addressed by this PR
This PR migrates storage-file-datalake to the new core pipeline in the same fashion that storage-blob was recently migrated. There are no changes to the public surface and existing recorded tests still pass.
Provide a list of related PRs (if any)
#24141

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-storage-file-datalake

/azp run js - storage-file-datalake - tests

/azp run js - storage-file-datalake - tests

@xirzec , I'm looking into this PR and may need to run some case manually.  We have several cases which requires manually setting environments.
About the SAS token case failures, will you look into them? Or do you need me to look into the them?

@EmmaZhu I have an update for the failing tests and I have been looking into an issue with the browser in live mode. I hope to have an updated version pushed soon.
I'll ping you when the live tests are passing and it's in a good state for manual testing. I appreciate you helping validate! 👍

/azp run js - storage-file-datalake - tests

@EmmaZhu I think it should be good for testing now! 👍

Hi @xirzec , I'm working on the validation.
Seems there are some required file are not tracking by git, "git clean -dxf" would delete them and cause failures.
I also see some failure in one of the large file uploading case. I think I can send a PR to your branch later.

Seems there are some required files not tracking by git, "git clean -dxf" would delete them and cause failures.

How strange. Do you know which files are having issues? I'm surprised it built in CI with untracked changes.

I also see some failure in one of the large file uploading case. I think I can send a PR to your branch later.

Sure! Also happy to take a look at a repro if you have a consistent one. 👍

The failure is in test case upload with chunkSize = FILE_UPLOAD_MAX_CHUNK_SIZE should succeed but it's not caused by migration, the migration should be fine.
The 'git clean -dxf' issue might be caused by my local environment, the issue has gone on a totally new clone environment.
I left one comment. We'd need to run some perf/resource consumption testing before releasing, but I don't think it's a blocking for this PR.

I left one comment. We'd need to run some perf/resource consumption testing before releasing, but I don't think it's a blocking for this PR.

Yes, thank you for this! It was a good catch. @HarshaNalluru do we have existing perf tests for this package?

@xirzec triggered the pipeline https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2596390&view=results


Two Errors in One Row

Fibble Unlimited 8787 X/8 2/2🏴
⬛⬛🟩⬛⬛
🟨⬛⬛⬛⬛
🟩⬛⬛⬛🟨
🟨🟩🟨⬛⬛
⬛⬛⬛⬛🟨
⬛⬛⬛⬛⬛
⬛🟨⬛🟩🟨
🟨⬛⬛🟨🟨
⬛🟨🟨🟨⬛
https://fibble.xyz/?unlimited=8787

Two lies in one row. (two e's) - row 5.

I see. Thanks for the explanation. I get it now.


README references obsolete APIs.

Right now several methods that use .exponent() have demo images that document their usage with a second argument specifying the exponent as an argument.

I think this refers to an older API of this module, and we'll need to update the images.

Yes, we should migrate this documentation to Observable so it’s easier to update.

This notebook generates all the images.

with the new github page design the images are enlarged to ugly. A solution is to remove the size hints in the img tags (currently width=100% height=240).


upload_date pulls wrong date for afreecaTV

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2021.12.01. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
no geo-block
Description
When I try to download a VOD from afreecaTV with the upload date as output name, it gives me 2021 January 02 or NA even if the VOD wasn't uploaded on that date. The upload_date works few months ago when I last used it.
Here are some sample links:
https://vod.afreecatv.com/PLAYER/STATION/81558503
https://vod.afreecatv.com/PLAYER/STATION/79406380
Verbose log
K:\youtube-dl>yt-dlp -Uv https://vod.afreecatv.com/PLAYER/STATION/81558503 -o h:\%(upload_date)s-%(uploader)s-%(title)s.%(ext)s
[debug] Command-line config: ['-Uv', 'https://vod.afreecatv.com/PLAYER/STATION/81558503', '-o', 'h:\\%(upload_date)s-%(uploader)s-%(title)s.%(ext)s']
[debug] Encodings: locale cp950, fs utf-8, out utf-8, err utf-8, pref cp950
[debug] yt-dlp version 2021.12.01 [91f071a] (win_exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.22000-SP0
[debug] exe versions: ffmpeg 4.2, ffprobe 4.2
[debug] Optional libraries: Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
Latest version: 2021.12.01, Current version: 2021.12.01
yt-dlp is up to date (2021.12.01)
[debug] [afreecatv] Extracting URL: https://vod.afreecatv.com/PLAYER/STATION/81558503
[afreecatv] 81558503: Downloading webpage
[afreecatv] 81558503: Downloading video info XML
[afreecatv] 81558503: Downloading part 1 m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[download] Downloading playlist: 엘린 세비지가쏘아올린공 허그가 당했누..'-'ㅎ
[afreecatv] playlist 엘린 세비지가쏘아올린공 허그가 당했누..'-'ㅎ: Collected 1 videos; downloading 1 of them
[download] Downloading video 1 of 1
[debug] Default format spec: bestvideo*+bestaudio/best
[info] 20211221_10C22F41_237809232_3: Downloading 1 format(s): hls-original
[debug] Invoking downloader on "http://vod-archive-global-cdn-z02.afreecatv.com/v101/hls/vod/20211221/232/237809232/REGL_10C22F41_237809232_3.smil/original/both/playlist.m3u8"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 100
[download] Destination: h:\20210102-크레용팝엘린-엘린 세비지가쏘아올린공 허그가 당했누..'-'ㅎ.mp4
[download] 100% of 188.63MiB in 00:03
[debug] ffprobe command line: ffprobe -hide_banner -show_format -show_streams -print_format json "file:h:\20210102-크레 용팝엘린-엘린 세비지가쏘아올린공 허그가 당했누..'-'ㅎ.mp4"
[FixupM3u8] Fixing MPEG-TS in MP4 container of "h:\20210102-크레용팝엘린-엘린 세비지가쏘아올린공 허그가 당했누..'-'ㅎ.mp4"
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:h:\20210102-크레용팝엘린-엘린 세비지가쏘아올린공 허그가 당했누..'-'ㅎ.mp4" -c copy -map 0 -dn -f mp4 "-bsf:a" aac_adtstoasc "file:h:\20210102-크레용팝엘린-엘린 세비지가쏘아올린공 허그가 당했누..'-'ㅎ.temp.mp4"
[download] Finished downloading playlist: 엘린 세비지가쏘아올린공 허그가 당했누..'-'ㅎ

The extractor currently guesses at the upload date by looking for it at the beginning of the file key. Apparently that heuristic used to be reliable, but some time ago I discovered a small number of videos for which that heuristic didn't work. In this commit, I made the extractor reject values that were obviously invalid. In making that change, I updated the date parsing to pass through the unified_strdate utility function. It appears that unified_strdate is buggy; passing in a string of 20211221 results in 20210102 as output.
Conclusion: this bug for the AfreecaTV extractor was introduced in version 2021.09.01. However, the underlying cause is a problem in the common utilities, not the extractor itself.

Looks like (surprisingly, to me) the date format %Y%m%d%H%M will successfully match against one-digit month, day, hour, and minute strings (even though %m et. al. are documented as being zero-padded). For most of the year, the leading or trailing zero on the two-digit month prevents it from matching, but for dates in November and December, 11 can parse as January 1st, and 12 as January 2nd (with the actual day being interpreted as hour and minute).
I think a valid fix would be to add a format string of %Y%m%d to our supported format strings, with higher priority. It's worth noting that the parsing function doesn't exit the loop on the first successful match, so higher priority is at the bottom of the format list, not the top.


Support evaluating resource IDs from symbolic names within the same resource

Some resources, such as load balancers, require referencing resource IDs as part of the properties of the resource definition, but those resource IDs are child resources of the resource itself. This means that Bicep detects this as a circular reference. For example, if I have a load balancer (simplified definition) like this:
resource loadBalancer 'Microsoft.Network/loadBalancers@2020-06-01' = {
  name: loadBalancerName
  location: location
  sku: {
    name: 'Standard'
  }
  properties: {
    frontendIPConfigurations: [
      {
        name: frontendIPConfigurationName
        properties: {
          privateIPAllocationMethod: 'Dynamic'
          subnet: {
            id: subnetResourceId
          }
        }
      }
    ]
    loadBalancingRules: [
      {
        name: 'HttpRule'
        properties: {
          frontendIPConfiguration: {
            id: loadBalancer.properties.frontendIPConfigurations[0].id
        }
      }
    ]
  }
}

then this line causes a circular reference:
            id: loadBalancer.properties.frontendIPConfigurations[0].id

The workaround is to use the resourceId() function directly. However, it would be nicer if Bicep could allow this scenario using the symbolic reference instead.

Related issues: #724, #1470 & #1787

I've found a potential workaround - however it's not currently possible in bicep due to - I think - a bug:
resource vmsslb 'Microsoft.Network/loadBalancers@2021-05-01' = {
  name: '${vmssName}-lb'
  location: location
  properties: {
    frontendIPConfigurations: [
      {
        name: frontendIpConfig.name //ERROR: The expression is involved in a cycle ("frontendIpConfig" -> "fwlb"). bicep(BCP080)
        properties: {
          privateIPAddress: lbPrivateIp
        }
      }
    ]
  }

  resource frontendIpConfig 'frontendIPConfigurations' existing = {
    name: 'frontend'
  }
}

This in fact should be possible as frontendIpConfig is an existing resource and therefore should be (IMHO) excluded from the cycle detection.

I found a working workaround but quite ugly:
resource _vmsslb 'Microsoft.Network/loadBalancers@2021-05-01' existing = {
  name: '${vmssName}-lb'

   resource frontendIpConfig 'frontendIPConfigurations' existing = {
    name: 'frontend'
  }
}

resource vmsslb 'Microsoft.Network/loadBalancers@2021-05-01' = {
  name: _vmsslb.name
  location: location
  properties: {
    frontendIPConfigurations: [
      {
        name: _vmss.frontendIpConfig.name //this works
        properties: {
          privateIPAddress: lbPrivateIp
        }
      }
    ]
  }
}

If we proceed with excluding the existing resource dependency, we should also ensure that it works correctly with symbolic name codegen because we emit existing resources to JSON in that case.

Assigning this to myself, however if someone is willing to pick this up - please write down a comment

this is mental

this issue is open for over a year, whereas this problem exists since the inception of bicep and its not yet fixes.

@4c74356b41 one of the reasons this hasn't really been prioritized is that we haven't seen any great need for this feature outside of the Microsoft.Network/loadBalancers resource; it is hard to justify building a new feature to address quirks in a single resource API.
Do you have any example use-cases outside of the loadbalancer resource? If you're able to share, that would help us triage & prioritize this.

appgw - the only one that pops up right now

Working with appgw is a nightmare already in arm/bicep.  This would help.  Fixing the model to not work this way might help more though...

honestly i dont understand why do they need to refer to each other by id? its not like you can reference a listener from another appgw, right?

I came here because of app gateway but I have circular id calls all over the place converting our existing infrastructure to code. This is indeed mental.

Also stuck on app gateway config..

@Ro3A
I have assume examples here:

https://github.com/Azure/bicep/issues/10268

@brwilkinson Thanks for the reply.
If I'm understanding correctly, in that example, what you're demonstrating is the creation of the resource id ahead of time. If so, that's where I ended up yesterday as well.
That said, even if this was resolved from a resource perspective, I suspect further work would need to be done to support this same objective when using a module, such as the app gateway module in the CARML project. The App Gateway module in the Azure/ResourceModules (CARML) project provides inputs for many of the nested resources required for app gateway. Backend pools, ssl, userAssignIdentity, etc. The networking aspects, especially, require the resourceId of the parent app gateway.
However, using resourceId() at all, with a module, is a code smell due to cycle detection.

Since https://github.com/Azure/bicep/issues/10268 was a duplicate, I will just bring that over here
Consider allowing a resource to reference itself for the values that are required at build time e.g. Name, Id, api, scope
Many network resources use the fully qualified resourceId for properties that reference itself.

Since there are some properties that are required at build, consider allowing self-reference for these properties.

What we do now (workaround)
param AppGWName string = 'mywaf123'

// Note I put this is as a variable - example 1
var AppGWID = resourceId('Microsoft.Network/applicationGateways', AppGWName )

//  it's more common for people to use resourceid()  - example 2
// in many places throughout the template, which does not represent a clean 'Bicep worthy' solution

resource applicationGateway 'Microsoft.Network/applicationGateways@2022-09-01' = {
  name: AppGWName
  location: resourceGroup().location
  properties: {
    frontendIPConfigurations: [
      {
        name: 'public' // we are cross referencing this property ____________________________________________\/
        properties: {
          publicIPAddress: {
            id: 'id'
          }
        }
      }
    ]
    httpListeners: [
      {
        name: 'name'
        properties: {
          frontendIPConfiguration: {
            // id requires the resourceid use AppGWID as substitute
            id: '${AppGWID}/frontendIPConfigurations/public' // cross reference to frontendIPConfiguration ___/\
          }
          frontendPort: {
            // a second example of cross reference, which is the more common pattern
            id: resourceId('Microsoft.Network/applicationGateways/frontendPorts', AppGWName, 'fe-80')
          }
          protocol: 'Http'
          sslCertificate: null
        }
      }
    ]

The proposal - unblock the following self-reference, only allow access to: name, id, apiversion?

This expression is referencing its own declaration, which is not allowed.
[{
"resource": "Untitled-2",
"owner": "generated_diagnostic_collection_name#0",
"code": "BCP079",
"severity": 8,
"message": "This expression is referencing its own declaration, which is not allowed.",
"source": "bicep",
"startLineNumber": 20,
"startColumn": 21,
"endLineNumber": 20,
"endColumn": 39
}]

param AppGWName string = 'mywaf123'

resource applicationGateway 'Microsoft.Network/applicationGateways@2022-09-01' = {
  name: AppGWName
  location: resourceGroup().location
  properties: {
    frontendIPConfigurations: [
      {
        name: 'public' // we are cross referencing this property
        properties: {
          publicIPAddress: {
            id: 'id'
          }
        }
      }
    ]
    httpListeners: [
      {
        name: 'name'
        properties: {
          frontendIPConfiguration: {
            id: '${applicationGateway.id}/frontendIPConfigurations/public' // use-self for id - example 1
          }
          frontendPort: {
            id: '${applicationGateway.id}/frontendPorts/fe-80' // use-self for id - example 2
          }
          protocol: 'Http'
          sslCertificate: null
        }
      }
    ]

There is a snippet for the Application Gateway to try this yourself res-app-gateway

There are other resources similar to this e.g. Virtual Network Gateway.

Any additional movement on this story?  The workarounds made by @brwilkinson and @miqm still the leading solutions?  Anyone else find a creative way to resolve this while waiting for a feature change in bicep?

@ckittel no update at this time.
Team has reviewed and did not schedule this work, so far.
So still sitting in the needs up vote stage... So the more comments and feedback will be useful to prioritize this for the future.

Tangent: Maybe we can get the AppGW team to add a new property called name to each of those in a future API version, and we only need to provide 'myListener.  Or id` is built to support either the full ID or just the name part. :)  It's not like we need to be referencing the listeners from a different app gateway. :)

+1. Not sure why this isn't prioritized. A new property called name or nameOrId should suffice.

I am using the work around for the frontendIPConfiguration on my app gateway however I am getting the error messaage:
Resource /subscriptions/a2bee89e-23fc-4c75-9a78-79d0aed1c958/resourceGroups/pxise-dmx-dbh-trial-rg/providers/Microsoft.Network/applicationGateways/pxise-dmx-dbh-trial-appgw/frontendIPConfigurations/appGwPublicFrontendIp referenced by resource /subscriptions/a2bee89e-23fc-4c75-9a78-79d0aed1c958/resourceGroups/pxise-dmx-dbh-trial-rg/providers/Microsoft.Network/applicationGateways/pxise-dmx-dbh-trial-appgw/httpListeners/pxise-dmx-dbh-trial-appgw-lstner-20305 was not found. Please make sure that the referenced resource exists, and that both resources are in the same region. (Code: InvalidResourceReference)
Is there another workaround that I can try?  Guess I could just do a separate module to run after the app gateway gets created to address these child references.

I'm again back to setting up a nat gateway + nat gateway ip + vnet w/ subnets, and this would be useful functionality.

I don't understand why this is in a state of needs: upvote. Without this functionality, we have to use half-functional workarounds to even do things such as deploy app gateway via bicep. I'd think core functionality would be scheduled regarless of number of users.

I disagree that this is core functionality. The feature ask here would only be to accommodate a strange API design that only AppGW (and maybe other networking resources?) implemented. That's why we need a lot of customer demand to justify implementing a fix for a specific case like this.
Ideally, the AppGW team implements somethign like what @ckittel is describing here: https://github.com/Azure/bicep/issues/1852#issuecomment-1604307397

I disagree that this is core functionality. The feature ask here would only be to accommodate a strange API design that only AppGW (and maybe other networking resources?) implemented. That's why we need a lot of customer demand to justify implementing a fix for a specific case like this.
Ideally, the AppGW team implements somethign like what @ckittel is describing here: #1852 (comment)

I challenge anyone to implement the well-architected framework via IaC and not run into this issue. It's not just AppGW and a few networking components few people use. It's almost all networking related components which are foundational to well architected framework.

@ro3a - is there an inventory of the different networking resources that have this API design? I probably don't have a full understanding of the scope of the problem, but my point still stands if it is only the Networking RP that has APIs designed in this way.
It is not to say that there is not justification for implementing the fix on our side, but from a layering perspective it is not the right place to fix this problem.

I disagree that this is core functionality. The feature ask here would only be to accommodate a strange API design that only AppGW (and maybe other networking resources?) implemented. That's why we need a lot of customer demand to justify implementing a fix for a specific case like this.
Ideally, the AppGW team implements somethign like what @ckittel is describing here: #1852 (comment)

I'm willing to agree the API/impelmentation is strange, though I'd argue from a customer perspective we see functionality as usability in the end-result, not who owns layers in the middle.
Is there a way to flag/bring this to the NRP team? I'm more than happy to have them fix the backend instead, though that would only work for API versions going forward and not fix the issue that already exists.
Our company has a call with PMs for Bicep and a few other teams, I'll bring this up there as well, but any guidance on how to push for this would be really useful.

I agree with the assessment. Ultimately, an end user shouldn't have to care why something is not working as expected, but I am trying to give context on how best to route the issue. If you have a call with a Microsoft Product Group, definitely bring this up and they can help route this to the NRP team. I will also put this on their radar.

Facing same issues with AppGateway module, with trustedRootCertificates neededing to be referenced inside sslProfiles. trustedRootCertificates  itself is a array, so difficult to reference inside sslProfiles>trustedClientCertificates>id field.

@johndowns any info about this issue?

@alex-frankel Are you able to provide any info for @Xyloto91 please?

@miqm -- I think I hear you on the checkIfExists thread, but I am not following the ask on this thread. As far as I know, App Gateway is the only resource that requires this self-referencing, which is why we haven't been able to prioritize this one.
The parent property/child resource issue is separate and the networking team is actually working on fixing these on a case-by-case basis. You can follow the latest here: https://github.com/Azure/azure-quickstart-templates/issues/2786#issuecomment-1846145291


fix(plugins/plugin-kubectl): k get all with direct/get doesn't fail back to the old impl of get

Fixes #6643

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[x] 🐛 Bug fix
[ ] 💅 Enhancement
[ ] 🚀 New feature

Please confirm that your PR fulfills these requirements

[x] Multiple commits are squashed into one commit.
[x] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[x] All npm dependencies are pinned.

This one isn't right. E.g. normal 404s from direct/get now result in a re-invoke of the CLI

This one isn't right. E.g. normal 404s from direct/get now result in a re-invoke of the CLI


Mask graphics holes in CanvasRenderer

Look at polygon direction, draw a hole to counter it.
Bad: https://www.pixiplayground.com/#/edit/W2Dm4fZVm9ygrRYqiVDXJ
Fixed: https://www.pixiplayground.com/#/edit/X8tnNzuRIXgPJBBiJTi7x
Build: http://pixijs.download/dev-canvas-mask-holes/pixi.js

#4177
#4183

Hero!!!

I think we should port this to GraphicsRenderer too. Working on it.

Done, please review again.

Lets refactor it in v5, I dont want to add complexity in this PR

@ivanpopelyshev awesome work. Do you think we can port this to v5?


DOC: Fix example in OSSL_PARAM_int.pod

This fixes an incorrect NULL check.
Fixes #11162

This pull request is ready to merge

Merged
3800cc6f4d285699210da496a42434bd2b42e065 DOC: Fix example in OSSL_PARAM_int.pod