function-parsing does not match grammar (probably docs issue?)

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
There is one drift between the defined parsing syntax and the implemented parser. We probably should consider this a docs issue (as it is probably way easier to fix the doc than the parser, esp. because there are currently (almost?) no 3rd party parser implementations). But as technically the parser definition is the source of truth for most implementations it makes it a parser issue. So I'm opening it in here for discussion.
The function definition says:
function-statement:
    function new-lines~opt~ function-name function-parameter-declaration~opt~ { script-block }
    filter new-lines~opt~ function-name function-parameter-declaration~opt~ { script-block }
    workflow new-lines~opt~ function-name function-parameter-declaration~opt~ { script-block }

And when resolved first contains the issue of referencing the command-name-expr that is missing a definition (probably a typo, see below). As well as missing a new-lines~opt~ after the function-name token. This omission of the new-lines~opt causes the drift (See expected behavior section for examples why this makes a difference).
Additionally, I also noticed a (probably just) a typing error related to the command-name-expr which is part of the function-name definition. Either it is not defined at all, or it lacks a reference to https://learn.microsoft.com/en-us/powershell/scripting/lang-spec/chapter-08?view=powershell-7.3#82-pipeline-statements
AND contains a typing error as it is written as command-namecommand-name-expr, which itself doesn't appear to be referenced by anything.
Expected behavior
# Documentation would mean parsing error for, as there is no new-lines~opt~ between function-name and the curved open bracket:
function
name
{
}

# Also parsing error for this (new line after the parenthesis of function-parameter-declaration, as this makes that token non optional anymore and it doesn't specify any new-lines~opt~ between it and the curved open bracket:
function
name
()
{
}

# Also technically omitting the white space before the curved open bracket should be a syntax error but currently isn't. Same for omitting the space between function-name and the round open bracket of function-parameter-declaration. At least if we consider the spaces in the definition to be part of the definition itself and not just for readability within the documentation.
# Otherwise, if we would consider the spaces to be NOT part of the syntax it would mean that this would have to be valid (which it currently isn't). That's because it doesn't contain anything about mandatory spaces then.
functionname(){}

Actual behavior
All of these are considered valid by the parser, except for `functionname(){}` which is currently considered invalid.

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.4.0
PSEdition                      Core
GitCommitId                    7.4.0
OS                             Arch Linux
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

The language spec was written for version 3.0 and is mostly still available for historical reasons. There are no plans to update it (and at this point we'd likely need to write a new spec anyway)

It contains notes about features that were added later on though. That's why I thought that it was still up to date.
Can we somehow just compile one from the code base? Having the syntax within the documentation can be helpful for debugging when things appear to be parsed incorrectly as well as for many other things. So loosing it (or not maintaining it) would be a undesirable...

The way I read that is doesn't allow for  line break before the { for the script block .
function SayHello
{
    "Hello World"
}

Should not parse and needs to be
function SayHello  {
    "Hello World"
}

Quite a lot of people - including some on the PowerShell team - put their function braces on a new line, although because of the way script block parameters work (where-object and foreach-object don't like a line break before { ) this is discouraged in style guides.  But the spec isn't a style guide.
function

SayHello

($myParam)  {
    "Hello $myparam"
}

Is legal and really shouldn't be encouraged, but according to the spec would break with { on its own line
@SeeminglyScience If it's not practical to produce a current one, it might be wisest to add something to the doc saying it was written for V3 and isn't maintained.


[Ubuntu] Correct variable to print only tool name

it was broken in #3701
It can be seen here: https://github.visualstudio.com/virtual-environments/_build/results?buildId=113761&view=logs&j=011e1ec8-6569-5e69-4f06-baf193d1351e&t=5431112d-2b61-5a5f-7042-ef698f761043&l=19972
    azure-arm: Successfully installed pip-21.1.3
    azure-arm: Create complete file
==> azure-arm: /bin/chown: cannot access '/opt/hostedtoolcache/@{name=Python; url=https://raw.githubusercontent.com/actions/python-versions/main/versions-manifest.json; platform=linux; platform_version=18.04; arch=x64; versions=System.Object[]}': No such file or directory
    azure-arm: Installing node 10.* x64...
    azure-arm: Download node-10.24.1-linux-x64.tar.gz

Check list

[x] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run ubuntu1804, ubuntu2004

@catthehacker thanks for your contribution!


Failed all time

I'm using Window10, after deploying by executing powserscript (by "unblock-file"). It works fine until downloading Failed All Time at about 11%~13%.
I have not installed .NET CORE 3.0 so the project can't be debug. If you have some idea about the issue, please send me the instructions.
Thanks very much

I'm using Window10, after deploying by executing powserscript (by "unblock-file").

What exactly do you mean by "deploying by executing powserscript (by "unblock-file")"?

Hello,
There is a Powershell script in Deploy folder. Since the execution policy on my computer, I used Unblock-File comlet to open the gate to the script. The installation has been done successfully. That’s all.
The application works but the download never succeeds.

Le 29 sept. 2019 à 00:26, Alexey Golub<EMAIL_ADDRESS>a écrit :
I'm using Window10, after deploying by executing powserscript (by "unblock-file").
What exactly do you mean by "deploying by executing powserscript (by "unblock-file")"?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Can you simply download the latest release here?
https://github.com/Tyrrrz/YoutubeDownloader/releases

Thanks very much.
The release version works on my machine.
Le 29 septembre 2019 à 10:51, Alexey Golub<EMAIL_ADDRESS>a écrit :
Can you simply download the latest release here?
https://github.com/Tyrrrz/YoutubeDownloader/releases
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Great


[Bug]: Knowledge Based Not Included When Doing Self RAG

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
5ec19b5
Other environment information
additional context from self rag:
<IP_ADDRESS> - - [19/Jun/2024:17:20:18 -0400] "POST /api/chat HTTP/1.1" 200 935 "-" "ollama-python/0.1.8 (x86_64 linux) Python/3.10.12" "{\x22model\x22: \x22llama3:70b-instruct-fp16\x22, \x22messages\x22: [{\x22role\x22: \x22system\x22, \x22content\x22: \x22\x5Cn        You are an expert at query expansion to generate a paraphrasing of a question.\x5Cn        I can't retrieval relevant information from the knowledge base by using user's question directly.     \x5Cn        You need to expand or paraphrase user's question by multiple ways such as using synonyms words/phrase, \x5Cn        writing the abbreviation in its entirety, adding some extra descriptions or explanations, \x5Cn        changing the way of expression, translating the original question into another language (English/Chinese), etc. \x5Cn        And return 5 versions of question and one is from translation.\x5Cn        Just list the question. No other words are needed.\x5Cn    \x22}, {\x22role\x22: \x22user\x22, \x22content\x22: \x22list cyber security problems that are evidence of a latent security breach\x22}], \x22stream\x22: false, \x22format\x22: \x22\x22, \x22options\x22: {\x22temperature\x22: 0.8}, \x22keep_alive\x22: -1}"
<IP_ADDRESS> - - [19/Jun/2024:17:20:20 -0400] "POST /api/embeddings HTTP/1.1" 200 20625 "-" "ollama-python/0.1.8 (x86_64 linux) Python/3.10.12" "{\x22model\x22: \x22mxbai-embed-large\x22, \x22prompt\x22: \x22Here are five paraphrased questions:\x5Cn\x5Cn1. What are the signs or indicators of an underlying cybersecurity threat?\x5Cn2. List potential warning signals of a previously undetected security compromise\x5Cn3. Identify common manifestations of hidden network vulnerabilities\x5Cn4. What are the typical red flags of an unreported cyber attack?\x5Cn5. \x5Cu00bfCu\x5Cu00e1les son los problemas de seguridad inform\x5Cu00e1tica que sugieren una violaci\x5Cu00f3n de seguridad latente?\x22, \x22options\x22: {}, \x22keep_alive\x22: null}"
<IP_ADDRESS> - - [19/Jun/2024:17:23:17 -0400] "POST /api/chat HTTP/1.1" 200 61892 "-" "ollama-python/0

Actual behavior
I have proxy-ed traffic between ollama and ragflow to detect the issue.  it appears when doing self rag, and ollama, questions are generated and submitted, but the knowledge base is empty (I added bold to make it easier to see):
<IP_ADDRESS> - - [19/Jun/2024:17:27:39 -0400] "POST /api/chat HTTP/1.1" 200 10517 "-" "ollama-python/0.1.8 (x86_64 linux) Python/3.10.12" "{\x22model\x22: \x22llama3:70b-instruct-fp16\x22, \x22messages\x22: [{\x22role\x22: \x22system\x22, \x22content\x22: \x22You are an intelligent assistant. Please summarize the content of the knowledge base to answer the question. Please list the data in the knowledge base and answer in detail. When all knowledge base content is irrelevant to the question, your answer must include the sentence \x5C\x22The answer you are looking for is not found in the knowledge base!\x5C\x22 Answers need to consider chat history.\x5Cn      Here is the knowledge base:\x5Cn      \x5Cn      The above is the knowledge base.\x22}, {\x22role\x22: \x22user\x22, \x22content\x22: \x22list cyber security problems that are evidence of a latent security breach\x22}], \x22stream\x22: true, \x22format\x22: \x22\x22, \x22options\x22: {\x22temperature\x22: 0.1, \x22num_predict\x22: 933, \x22top_k\x22: 0.3, \x22presence_penalty\x22: 0.4, \x22frequency_penalty\x22: 0.7}, \x22keep_alive\x22: -1}"
Expected behavior
I would expect there to be knowledge
Steps to reproduce
use llama3
configure with self-rag
ask a question of knowledge base where you know you have relevant content

Additional information
No response

Retievaled nothing even using self-rag.
Could you list the user question and the relevant content in knowledge base in order to analyze the reason of nothing retrievaled?

Here is some additional detailed entries from when simply trying to index the data using RAG to summarize content (it appears none of the content is being passed back).
<IP_ADDRESS> - - [20/Jun/2024:10:59:27 -0400] "POST /api/embeddings HTTP/1.1" 200 20585 "-" "ollama-python/0.1.8 (x86_64 linux) Python/3.10.12" "{\x22model\x22: \x22mxbai-embed-large\x22, \x22prompt\x22: \x22There are no paragraphs provided for me to summarize. Please provide the paragraphs you would like me to summarize, and I'll be happy to assist you!\x22, \x22options\x22: {}, \x22keep_alive\x22: null}"
<IP_ADDRESS> - - [20/Jun/2024:10:59:27 -0400] "POST /api/embeddings HTTP/1.1" 200 20585 "-" "ollama-python/0.1.8 (x86_64 linux) Python/3.10.12" "{\x22model\x22: \x22mxbai-embed-large\x22, \x22prompt\x22: \x22There are no paragraphs provided for me to summarize. Please provide the paragraphs you would like me to summarize, and I'll be happy to assist you!\x22, \x22options\x22: {}, \x22keep_alive\x22: null}"
<IP_ADDRESS> - - [20/Jun/2024:10:59:27 -0400] "POST /api/embeddings HTTP/1.1" 200 20684 "-" "ollama-python/0.1.8 (x86_64 linux) Python/3.10.12" "{\x22model\x22: \x22mxbai-embed-large\x22, \x22prompt\x22: \x22There are no paragraphs provided for me to summarize. Please provide the actual text, and I'll be happy to help!\x22, \x22options\x22: {}, \x22keep_alive\x22: null}"
<IP_ADDRESS> - - [20/Jun/2024:11:00:15 -0400] "POST /api/chat HTTP/1.1" 200 745 "-" "ollama-python/0.1.8 (x86_64 linux) Python/3.10.12" "{\x22model\x22: \x22llama3:70b-instruct-fp16\x22, \x22messages\x22: [{\x22role\x22: \x22system\x22, \x22content\x22: \x22You're a helpful assistant.\x22}, {\x22role\x22: \x22user\x22, \x22content\x22: \x22Please summarize the following paragraphs. Be careful with the numbers, do not make things up. Paragraphs as following:\x5Cn      There\x5CnThere\x5CnThe above is the content you need to summarize.\x22}], \x22stream\x22: false, \x22format\x22: \x22\x22, \x22options\x22: {\x22temperature\x22: 0.3, \x22num_predict\x22: 1000}, \x22keep_alive\x22: -1}"
<IP_ADDRESS> - - [20/Jun/2024:11:00:24 -0400] "POST /api/chat HTTP/1.1" 200 708 "-" "ollama-python/0.1.8 (x86_64 linux) Python/3.10.12" "{\x22model\x22: \x22llama3:70b-instruct-fp16\x22, \x22messages\x22: [{\x22role\x22: \x22system\x22, \x22content\x22: \x22You're a helpful assistant.\x22}, {\x22role\x22: \x22user\x22, \x22content\x22: \x22Please summarize the following paragraphs. Be careful with the numbers, do not make things up. Paragraphs as following:\x5Cn      There\x5CnThe above is the content you need to summarize.\x22}], \x22stream\x22: false, \x22format\x22: \x22\x22, \x22options\x22: {\x22temperature\x22: 0.3, \x22num_predict\x22: 1000}, \x22keep_alive\x22: -1}"
<IP_ADDRESS> - - [20/Jun/2024:11:00:26 -0400] "POST /api/embeddings HTTP/1.1" 200 20627 "-" "ollama-python/0.1.8 (x86_64 linux) Python/3.10.12" "{\x22model\x22: \x22mxbai-embed-large\x22, \x22prompt\x22: \x22I apologize, but there is no content provided for me to summarize. The text only contains three instances of \x5C\x22There\x5C\x22 without any actual paragraphs or information. If you could provide the actual content, I'd be happy to help you with summarizing it!\x22, \x22options\x22: {}, \x22keep_alive\x22: null}"
<IP_ADDRESS> - - [20/Jun/2024:11:00:26 -0400] "POST /api/embeddings HTTP/1.1" 200 20635 "-" "ollama-python/0.1.8 (x86_64 linux) Python/3.10.12" "{\x22model\x22: \x22mxbai-embed-large\x22, \x22prompt\x22: \x22I apologize, but there are no paragraphs provided for me to summarize. The text only says \x5C\x22There\x5C\x22 and then nothing else. If you could provide the actual paragraphs, I'd be happy to help you with summarizing them!\x22, \x22options\x22: {}, \x22keep_alive\x22: null}"


Plugin-descriptor not found during plugin installation

Hi I tried to install armor for 2.0 Elastic search and i am getting below error and as per elasticsearch 2.0 plugin documentation keeping plugin-descriptor in root directory of the plugin is mandatory but some how plugin tool is not taking the descriptor even if i manually place and again build the zip.
./plugin install file:///armor2.1/target/releases/armor-2.0.0-SNAPSHOT.zip --verbose
-> Installing from file:///armor2.1/target/releases/armor-2.0.0-SNAPSHOT.zip...
Trying file:///armor2.1/target/releases/armor-2.0.0-SNAPSHOT.zip ...
Downloading ........................................................................DONE
Verifying file:///armor2.1/target/releases/armor-2.0.0-SNAPSHOT.zip checksums if available ...
NOTE: Unable to verify checksum for downloaded plugin (unable to find .sha1 or .md5 file to verify)
ERROR: Could not find plugin descriptor 'plugin-descriptor.properties' in plugin zip

Fix on the last build (https://oss.sonatype.org/content/repositories/snapshots/com/petalmd/armor/2.0.0-SNAPSHOT/armor-2.0.0-20151216.195914-11.zip)
Thanks for the report!


Mono: Rename margin to side

Rename margin to side that was missed in #44605

Superseded by #44751, but thanks for the contribution nevertheless!


Fix spacing when using latex

Sometimes, svgs that were correctly created from latex were displayed with very small gaps in their structures.
See for example here:

Setting    "background_stroke_width": 0 in tex_mobject solves this problem.
See here:

The corresponding issue is here: https://github.com/ManimCommunity/manim/issues/566

It is fascinating that this change would fix the issue. Can anyone explain why? Tagging SVG and LaTeX pros @yoshiask @PhilippImhof @cobordism

I have encountered similar issues in the past.
It's strange, the way manim renders SVGs with 'normal' and 'background' stroke and fill. I don't know what they are exactly.
I know that we usually have to deal with stroke and fill values in svgs; and I've had to manually futz around with the stroke to get some fonts looking like they should:
https://discordapp.com/channels/581738731934056449/581739090773540874/757178775787012096
In that case I chose "background_stroke_width": 0.25 for aesthetic reasons, but "background_stroke_width": 0 would have rendered the font "correctly.
I suggest we look closer at what all the background stuff in svg rendering in manim is all about.

Can anyone explain why?

I was only guessing, so unfortunately no.

I do not know how much you all are into LaTeX, but the \xrightarrow is internally composed of several parts: the left end, the right end with the arrow tip and as many intermediate line segments as needed. That's how it is written into the DVI file. As a consequence, that's also how the SVG file is constructed. If you open it in Inkscape, you get one group of 4 elements. To illustrate this, I split the group up and shifted the pieces vertically.

Now we see that they overlap. This does not disturb, because they have just black fill, but no stroke. When we set background_stroke to 1, we introduce a stroke that should not be there and thus make the individual segments visible.

Excellent. Now the question is, why does manim import with "background stroke" at all?
We should disable it by default in all Tex classes, no?

That's beyond my sphere of competence .... probably yes. At least, we should not have a stroke on TeX objects, as they are not supposed to have one.

probably yes.
but let me experiment a bit first...

Ok. we can set background_stroke_width to 0 by default.
When you just write letters, the extra stroke adds a nice outline to each character, but in composite symbols it creates the error (above).

let's check it this messes with the write animation. As I recall, that animation draws stroke and then fill.

Now that we're after the v0.1 release, I want to come back to this one. Did anyone try this with the Write animation?

I would love to find a way to automatically merge overlapping paths in the svg.
That way we solve our problem, but also can keep the black edge if we want it...

Write animation still looks AOK.
proceed.


changing php_fpm_pool_user and php_fpm_pool_group to "nginx" kills the build process

Issue Type
Bug Report / Support Request
Your Environment
Vagrant 1.9.1
VirtualBox 5.1.10r112026
ansible <IP_ADDRESS>
  config file = /etc/ansible/ansible.cfg
  configured module search path = Default w/o overrides

Your OS
macOS (10.12.3)
Full console output
fatal: [wondrous-intern]: FAILED! => {"cache_update_time":<PHONE_NUMBER>, "cache_updated": false, "changed": false, "failed": true, "msg": "'/usr/bin/apt-get -y -o \"Dpkg::Options::=--force-confdef\" -o \"Dpkg::Options::=--force-confold\"     install 'php7.1-mysql'' failed: E: Sub-process /usr/bin/dpkg returned an error code (1)\n", "stderr": "E: Sub-process /usr/bin/dpkg returned an error code (1)\n", "stdout": "Reading package lists...\nBuilding dependency tree...\nReading state information...\nThe following packages were automatically installed and are no longer required:\n  python-crypto python-ecdsa python-httplib2 python-jinja2 python-markupsafe\n  python-paramiko python-pkg-resources python-setuptools python-six\n  python-yaml sshpass\nUse 'sudo apt autoremove' to remove them.\nThe following NEW packages will be installed:\n  php7.1-mysql\n0 upgraded, 1 newly installed, 0 to remove and 56 not upgraded.\nNeed to get 119 kB of archives.\nAfter this operation, 453 kB of additional disk space will be used.\nGet:1 http://ppa.launchpad.net/ondrej/php/ubuntu xenial/main amd64 php7.1-mysql amd64 7.1.1-1+deb.sury.org~xenial+1 [119 kB]\nFetched 119 kB in 0s (667 kB/s)\nSelecting previously unselected package php7.1-mysql.\r\n(Reading database ... \r(Reading database ... 5%\r(Reading database ... 10%\r(Reading database ... 15%\r(Reading database ... 20%\r(Reading database ... 25%\r(Reading database ... 30%\r(Reading database ... 35%\r(Reading database ... 40%\r(Reading database ... 45%\r(Reading database ... 50%\r(Reading database ... 55%\r(Reading database ... 60%\r(Reading database ... 65%\r(Reading database ... 70%\r(Reading database ... 75%\r(Reading database ... 80%\r(Reading database ... 85%\r(Reading database ... 90%\r(Reading database ... 95%\r(Reading database ... 100%\r(Reading database ... 69478 files and directories currently installed.)\r\nPreparing to unpack .../php7.1-mysql_7.1.1-1+deb.sury.org~xenial+1_amd64.deb ...\r\nUnpacking php7.1-mysql (7.1.1-1+deb.sury.org~xenial+1) ...\r\nProcessing triggers for php7.1-fpm (7.1.1-1+deb.sury.org~xenial+1) ...\r\nJob for php7.1-fpm.service failed because the control process exited with error code. See \"systemctl status php7.1-fpm.service\" and \"journalctl -xe\" for details.\r\ninvoke-rc.d: initscript php7.1-fpm, action \"restart\" failed.\r\ndpkg: error processing package php7.1-fpm (--unpack):\r\n subprocess installed post-installation script returned error exit status 1\r\nErrors were encountered while processing:\r\n php7.1-fpm\r\n", "stdout_lines": ["Reading package lists...", "Building dependency tree...", "Reading state information...", "The following packages were automatically installed and are no longer required:", "  python-crypto python-ecdsa python-httplib2 python-jinja2 python-markupsafe", "  python-paramiko python-pkg-resources python-setuptools python-six", "  python-yaml sshpass", "Use 'sudo apt autoremove' to remove them.", "The following NEW packages will be installed:", "  php7.1-mysql", "0 upgraded, 1 newly installed, 0 to remove and 56 not upgraded.", "Need to get 119 kB of archives.", "After this operation, 453 kB of additional disk space will be used.", "Get:1 http://ppa.launchpad.net/ondrej/php/ubuntu xenial/main amd64 php7.1-mysql amd64 7.1.1-1+deb.sury.org~xenial+1 [119 kB]", "Fetched 119 kB in 0s (667 kB/s)", "Selecting previously unselected package php7.1-mysql.", "(Reading database ... ", "(Reading database ... 5%", "(Reading database ... 10%", "(Reading database ... 15%", "(Reading database ... 20%", "(Reading database ... 25%", "(Reading database ... 30%", "(Reading database ... 35%", "(Reading database ... 40%", "(Reading database ... 45%", "(Reading database ... 50%", "(Reading database ... 55%", "(Reading database ... 60%", "(Reading database ... 65%", "(Reading database ... 70%", "(Reading database ... 75%", "(Reading database ... 80%", "(Reading database ... 85%", "(Reading database ... 90%", "(Reading database ... 95%", "(Reading database ... 100%", "(Reading database ... 69478 files and directories currently installed.)", "Preparing to unpack .../php7.1-mysql_7.1.1-1+deb.sury.org~xenial+1_amd64.deb ...", "Unpacking php7.1-mysql (7.1.1-1+deb.sury.org~xenial+1) ...", "Processing triggers for php7.1-fpm (7.1.1-1+deb.sury.org~xenial+1) ...", "Job for php7.1-fpm.service failed because the control process exited with error code. See \"systemctl status php7.1-fpm.service\" and \"journalctl -xe\" for details.", "invoke-rc.d: initscript php7.1-fpm, action \"restart\" failed.", "dpkg: error processing package php7.1-fpm (--unpack):", " subprocess installed post-installation script returned error exit status 1", "Errors were encountered while processing:", " php7.1-fpm"]}

Summary
Like stated here http://docs.drupalvm.com/en/latest/configurations/webservers-nginx/
I changed the php_fpm_pool_user and php_fpm_pool_group to nginx to reflect the use of nginx. If i create the vm the above error is given.
If i do not do this everything is fine.

It looks like the failure is here:
Job for php7.1-fpm.service failed because the control process exited with error code.
See "systemctl status php7.1-fpm.service" and "journalctl -xe" for details.

After that failure, can you run systemctl status php7.1-fpm.service inside the VM and see what it says?

Also, I presume you're using Ubuntu 16.04 (any other config.yml changes?).

I use drupalvm as a compose dependency and a local.config.yml as overwrite.
# Composer settings
build_composer_project: false

#drush alias fix
configure_drush_aliases: false

# Base settings for vagrant box
vagrant_hostname: wondrous-intern.dev
vagrant_machine_name: wondrous-intern
vagrant_ip: <IP_ADDRESS>

# Correct folders
drupal_composer_install_dir: "/var/www/drupalvm"

# webserver
drupalvm_webserver: nginx
#php_fpm_pool_user: nginx
#php_fpm_pool_group: nginx

#solr
solr_version: "6.4.1"

post_provision_scripts:
  - "{{ config_dir }}/../scripts/vagrant/configure-solr.sh"

# Use mariaDB instead of mysql
mysql_packages:
  - mariadb-client
  - mariadb-server
  - python-mysqldb

#php
php_version: "7.1"
php_memory_limit: "256M"

# xdebug
php_xdebug_default_enable: 1
php_xdebug_coverage_enable: 0
php_xdebug_max_nesting_level: 512
php_xdebug_idekey: PHPSTORM

# Site install false
install_site: false
drupal_enable_modules: []

installed_extras:
 # - adminer
 # - blackfire
 # - drupalconsole
 # - memcached
 # - newrelic
 # - nodejs
 # - ruby
 # - selenium
 # - varnish
 # - xhprof
 # - drush
 - solr
 - redis
 - mailhog
 - xdebug
 - pimpmylog

The task where it fails is this one:
TASK [geerlingguy.php-mysql : Install PHP MySQL dependencies (Debian).] ********
here the output:
agrant@wondrous-intern:~$ systemctl status php7.1-fpm.service
● php7.1-fpm.service - The PHP 7.1 FastCGI Process Manager
   Loaded: loaded (/lib/systemd/system/php7.1-fpm.service; enabled; vendor preset: enabled)
   Active: failed (Result: exit-code) since Sat 2017-02-11 07:54:48 UTC; 46s ago
     Docs: man:php-fpm7.1(8)
  Process: 28801 ExecStart=/usr/sbin/php-fpm7.1 --nodaemonize --fpm-config /etc/php/7.1/fpm/php-fpm.conf (code=exited, statu
 Main PID: 28801 (code=exited, status=78)

I can reproduce this, it's caused because there is no user called nginx:
vagrant@drupalvm-project:~$ systemctl status php7.1-fpm.service
* php7.1-fpm.service - The PHP 7.1 FastCGI Process Manager
   Loaded: loaded (/lib/systemd/system/php7.1-fpm.service; enabled; vendor preset: enabled)
   Active: failed (Result: exit-code) since Tue 2017-02-28 23:46:58 UTC; 32min ago
     Docs: man:php-fpm7.1(8)
  Process: 28892 ExecStart=/usr/sbin/php-fpm7.1 --nodaemonize --fpm-config /etc/php/7.1/fpm/php-fpm.conf (code=exited, status=78)
 Main PID: 28892 (code=exited, status=78)
vagrant@drupalvm-project:~$ sudo /usr/sbin/php-fpm7.1 --nodaemonize --fpm-config
sudo /usr/sbin/php-fpm7.1 --nodaemonize --fpm-config /etc/php/7.1/fpm/php-fpm.conf
[01-Mar-2017 00:19:55] ERROR: [pool www] cannot get uid for user 'nginx'
[01-Mar-2017 00:19:55] ERROR: FPM initialization failed

I'll close this. To fix it you can add a preprovision script/task that creates the nginx user if you need it.


A little bug in vlpencoder.py

code: https://github.com/microsoft/X-Decoder/blob/a85d2f4e8aa62c732866a0b9f1aa1c43e4c60be6/xdecoder/language/vlpencoder.py#L119
I think this should be:
if prompt:
    for clss in class_names:
        txts = [template.format(clss.replace('-other','').replace('-merged','').replace('-stuff','')) for template in templates]
        clss_embeddings.append(extract_mean_emb(txts))
else:
    for clss in class_names:
        clss_embeddings.append(extract_mean_emb([clss]))

Thanks for your careful! Just fix this bug : )


Transition plan away from Code Climate

The team has agreed to migrate away from Code Climate due to costs. We need to replace with a set of similar services that can integrate into our Circle CI/PR workflow.
Among the obvious (free for Open Source) candidates are:

[ ] Coveralls (test coverage, we're already using for Code Climate)
[ ] Hound (PR integration for Rubocop linters)

But, this is an opportunity to review our options before committing.
Let's keep the entire @ucsdlib/developers crew in the loop on this.

@mcritchlow Can you see  "hound — No violations found. Woof!" from the pull request https://github.com/ucsdlib/billing-ill/pull/100 ?

@hweng - Yep, I see it listed in the list of passed Checks.

@mcritchlow Great! Do we want to try .jshintrc configuration? Hound uses JSHint to review JavaScript code.

@hweng maybe we could discuss that as part of a separate ticket? I think linting JS is a great idea, but brings up the question of what flavor of JS we're writing. In many newer apps I'm increasingly seeing the usage of ES6, as opposed to say CoffeeScript or older native JS. What do you think?

@mcritchlow I agree that we could discuss it separately, as there are several options for JS lint. Or maybe we could choose ESlint rather than JSHint.   Mainly in Hound config, the RuboCop will be enabled to review Ruby code.  haml-lint will be enabled only to billing-ill app. Others lints, such as  scss-lint,  CoffeeLint, might not be necessary to enable because we don't use them in our projects. Let me know if you have any other suggestions. Thanks!

Sounds good. I'll setup a ticket, perhaps in our DevelopmentGuides repo for us all to discuss? Then we can implement any documentation changes there and perhaps setup another ticket here in the billing repo so you can pilot out the additional linters?

@mcritchlow Sounds good to me.

@mcritchlow It looks good to me for testing the hound and coverage/coveralls on billing repo: https://github.com/ucsdlib/billing-ill/pull/102

@hweng - agreed. that looks perfect 👍

@hweng - one question, the commits look like they're still reference Code Climate. Are you removing that as a separate step/task?

@mcritchlow Yes, I tried removing code climate gem and config from billing-ill project, but it got errors in circleCI. I think it need to completely remove the repo from codelimate dashboard. Do you want me to remove the billing-ill from codelimate dashboard?

@hweng - yeah i think that would be great. Then we'll know everything involved with transitioning a project away from CodeClimate. Thanks.

@mcritchlow Yep, completely removed CodeClimate, see the pull request: https://github.com/ucsdlib/billing-ill/pull/103

@hweng - just noticed this ticket is still open for some reason. We can safely close, yes?

@mcritchlow Yes, the billing-ill was done migrating away from code climate.


Update processing-recommendations.rst

Added documentation for MetaT sample processing guidelines: Sortmerna for ribosomal read filtering

Closing as @smruthi98 will open a new PR based on dev so tests pass.


Alignment and 2 GiB overflow issues in Span.TryCopyTo

Only affects the IsPrimitiveValueType<T> case;

overflow for large src.Length and non-trivial T; might need to loop, since cpblk is limited to uint sizes (thus is not quite the same issue as #846)
CopyBlock assumes that the pointers are aligned; otherwise, cpblk needs the unaligned prefix

(also impacts ReadOnlySpan<T>)

Raised issue for unaligned https://github.com/dotnet/corefx/issues/12243


Missing Tag & GitHub release for RC5-5

As the title says we're missing the git tag for RC5-5 and the releases page also doesn't mention the new release.

Thanks for the remainder!


[Tabs extended vertical] Web Component: update focus state - DPO Consulting

Engineering info:

Innovation Team engineer: @andy-blum
Innovation Team JIRA Ticket: https://jsw.ibm.com/browse/HC-2396
DPO consulting engineer: @emyarod

User Story

As a [user role below]:
Carbon for ibm.com developer

I need to:
create/change the Tabs extended vertical

so that I can:
provide the ibm.com adopter developers components they can use to build ibm.com web pages

Specific update needed
When Tooltip is visible (on hover & on focus), the tooltip's little triangle should be on the top, and aligned left.

Additional information

Story within Storybook with corresponding knobs
Utilize Carbon
Create with Shadow DOM and Custom Elements standards
See the Epic (#7372 ) for the Design and Functional specs information
No E2E testing update is needed.
Web Components Visual QA testing issue (#7485)
Web Components Prod QA testing issue (#7486)

Acceptance criteria

[ ] Include README for the web component and corresponding styles
[ ] Create Web Components styles in styles package
[ ] Do not create knobs in Storybook that include JSON objects
[ ] Break out Storybook stories into multiple variation stories, if applicable
[ ] Create codesandbox example under /packages/web-components/examples/codesandbox and include in README
[ ] Minimum 80% unit test coverage
[ ] Update the Carbon for ibm.com website component data file to be sure the web site Component Status and Storybook links are correct
[ ] Use the Visual QA checklist to verify design quality
[ ] Update the e2e test scripts, if needed.
[ ] Confirm e2e test results have no major errors.
[ ] If a design is provided, the Designer is included as a Reviewer in the Pull Request
[ ] Provide a direct link to the deploy preview for the designer in the Pull Request description
[ ] A comment is posted in the Prod QA issue, tagging Praveen when development is finished

Just wanted to double check the design on this. The carbon tooltip component we're using here doesn't allow for changing the placement of the caret, just the text bubble. The caret is always center-aligned to the element that triggers it

.

cc: @oliviaflory and/or @RichKummer

@oliviaflory The implementation of the tooltip with the tabs is already matching this, then. Currently the tooltip is left-aligned with the trigger element (the tab in this case), and takes the width of its text content up to 13 rems.

The tooltip is left aligned to it's parent

The tooltip is set to takes up the width of max-content, so long as that value falls between 1.5 & 13 rems.

The only thing that this issue would change is the placement of the caret in relation to the parent item. Is that something we actually want to change away from carbon core?

@andy-blum  cc: @jeffchew

The only thing that this issue would change is the placement of the caret in relation to the parent item. Is that something we actually want to change away from carbon core?

No, we don't want to change away from how the Carbon tooltip is done. I think in an earlier version of the Carbon tool tips the caret was more like the "expected" example above, but it seems like design didn't catch the update to always be center aligned to the parent.
If this is already implemented there is no further action needed and we can close out this issue.

@oliviaflory Thanks for getting back to us! Feels like there's no action that's needed for this particular issue. I'll go ahead and close this out for now. Feel free to re-open if there is more action needed. cc: @ljcarot @jeffchew


feat: add Helm-based install process

Description
Add and document the procedure for creating a cluster and installing Karpenter using published Helm charts.

Add configure-values.sh script, which interrogates the cluster and generates karpenter-values.yaml based on karpenter-values-template.yaml. These values are used when installing Karpenter using Helm chart. Skaffold configuration is shifted to use this file as well.
Add create-cluster.sh script which can be used for creating clusters compatible with Kaprenter provider
Update readme with the procedure of creating cluster and installing Karpenter using published Helm chart and image

How was this change tested?

make az-mkaks-user and make az-helm-install-snapshot

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.546%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
35741

Relevant Lines:
36640

💛  - Coveralls


Segmentation fault when trying to parse custom dates

I'm trying to make valinor map DateTimeInterface properties to cakephp/chronos objects, by following this documentation, but when configure it to infer the type, it halts the script with a segmentation fault when trying to map objects.
It can be reproduced with valinor 0.15, using this script under PHP 8.1
<?php

declare(strict_types=1);

use Cake\Chronos\Chronos;
use CuyZ\Valinor\Mapper\Source\Source;
use CuyZ\Valinor\Mapper\Tree\Message\MessageBuilder;
use CuyZ\Valinor\MapperBuilder;

require __DIR__ . '/../vendor/autoload.php';

class User {
    public function __construct(
        public readonly string $email,
        public readonly DateTimeInterface $dateOfBirth,
        /** @var string[] */
        public readonly array $favoriteTopics,
        public readonly ?string $domain = null,
    ) {
    }
}

$data = [
    'email' =><EMAIL_ADDRESS>    'dateOfBirth' => Chronos::now()->startOfDay()->subYears(20)->toAtomString(),
    'favoriteTopics' => ['foo', 'bar'],
];

$mapper = (new MapperBuilder())
    ->infer(DateTimeInterface::class, static fn () => Chronos::class)
    ->registerConstructor(static fn ($date): Chronos => Chronos::parse($date))
    ->filterExceptions(static fn (Throwable $e) => MessageBuilder::from($e))
    ->mapper();

try {
    var_dump($mapper->map(User::class, Source::array($data)));
} catch (Throwable $e) {
    var_dump($e);
}

If the infer, registerConstructor and filterExceptions lines are commented out, then it works by mapping the date as a DateTimeImmutable object.

Nice catch!
Should be fixed with #237 — any chance you can test it out on your side and confirm if it's ok?

I sure can

Yep, I can confirm it fixes it :)

Thank you for the confirmation!
Now released as 0.16.0. 😉


Airlock function private endpoint for internal storage

What is being addressed
The Airlock Function uses a storage account internally but does it over public network.
This is part of #2518
How is this addressed

Add Private Endpoint(s) for the storage account
Block public internet access to the storage account

/test-extended

@tamirkamara
why does the airlock function need a private endpoint?
isn't a firewall rule in the storage account enough?

@tamirkamara why does the airlock function need a private endpoint? isn't a firewall rule in the storage account enough?

@yuvalyaron I imagine you'd define a rule with the subnet/vnet internal address, right? A private endpoint is essencially a way to give the storage account an ip address in the vnet and by such we make sure all communication is done over private network. IIRC, you need this anyway for the firewall rule mentioned above.


Connection Strings in User Secret no longer shows up in the Update Database dropdown

Describe the bug
Connection Strings in User Secret (secrets.json) no longer show up in the Update Database dropdown when I want to effect my migration changes to the DB.
This used to work in previous versions, was this change intentional?
To Reproduce
Steps to reproduce the behavior:

Go to 'Entity Framework Core Tool'
Click on 'Update Database'
Scroll down to 'Additional Options'
Uncheck the "Use default connection of startup project"
Select the Connection dropdown
connection strings from secrets.json no longer show there, however, those from appsettings.json still show

Expected behavior
connection strings from secrets.json should be populated in the dropdown as they used to be.
Screenshots

Environment (please complete the following information):

OS: Windows 10 Home Version 22H2 (OS Build 19045.3324)
.NET SDK info (installed ones and version used within Rider):
Installed version:  7.0.400
Used in Rider: .NET Core v7.0.7 x64 (Server GC)
Plugin version: 232.1.0
Rider version: JetBrains Rider 2023.2.1 Build #RD-232.9559.61, built on August 22, 2023

Additional context
This used to work in previous versions of the plugin, at least 2 versions ago.

By the way, thanks for highlighting the problem with the dialog width when long connection string is presented. It should not shrink so long.

Hi @Xor-el, Thanks for the ticket. I'm not able to reproduce the issue:
Are you maybe made some changes to the project structure? Can you locate your secrets.json file using Tools | .NET User Secrets action?

@seclerp Thanks for the quick response.
I did not make any changes to the project structure recently.
The only thing I did recently on the project was update it to .NET 7.
No idea if that might have caused it.
also, yes I can access my user secrets file (secrets.json) via the method you mentioned above.
here is a screenshot to that effect

Yes, it's an option. You could create a private issue on YouTrack shared only for JetBrains employees with the project attached in one of the 2 ways:

.zip file as an attachment to the issue itself
Using https://uploads.jetbrains.com

Please use "Rider" as project and "Entity Framework Core" as a subsystem.

@seclerp Created the issue and attached the project on Youtrack.
added you to the list of people who can act on it.
Can you please confirm that you can access the ticket on YouTrack?

Yes, I have access to it, thanks for all the details and files.

Done in #184


update user story - find

This change is

Coverage remained the same at 85.714% when pulling c0872fdc328332e5048e510e75d9d8803ce39965 on Update-Developer-Guide into 9dd3b69bbbbc3d45af01fe39a72a5bf31b5b71ff on master.


Testing plugin in example project.

Hi. Today, I tried to work on making test codes for my plugin and I can't make the call to plugin methods in test code. Is there any guideline make test code for plugins?
import 'package:flutter/material.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:flutter_sound/flutter_sound.dart';

import 'package:flutter_sound_example/main.dart';

void main() {
  test('startRecorder', () async {
    FlutterSound flutterSound = new FlutterSound();
    String path = await flutterSound.startRecorder(null);
    print('path: $path');
  });
}

Error code below.
Exception: MissingPluginException(No implementation found for method startRecorder on channel flutter_sound)
  package:flutter_sound/flutter_sound.dart 100:7  FlutterSound.startRecorder
  ===== asynchronous gap ===========================
  dart:async                                      _AsyncAwaitCompleter.completeError
  package:flutter_sound/flutter_sound.dart        FlutterSound.startRecorder
  ===== asynchronous gap ===========================
  dart:async                                      _asyncErrorWrapperHelper
  package:flutter_sound/flutter_sound.dart        FlutterSound.startRecorder
  test/widget_test.dart 28:38                     main.<fn>

flutter doctor.
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel dev, v0.5.7, on Mac OS X 10.13.6 17G65, locale en-KR)
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
[✓] Android Studio (version 3.1)
[!] VS Code (version 1.26.1)
[✓] Connected devices (1 available)

! Doctor found issues in 1 category.

Also, tried mocking.
import 'package:flutter/material.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:flutter/services.dart';

import 'package:flutter_sound_example/main.dart';

void main() {
  final List<MethodCall> log = <MethodCall>[];
  MethodChannel channel = const MethodChannel('plugins.flutter_sound/flutter_sound');
  channel.setMockMethodCallHandler((MethodCall methodCall) async {
    log.add(methodCall);
  });

  test('Audio Recorder', () async {
    expect(log, equals(<MethodCall>[new MethodCall('startRecorder', "")]));
  });
}

Output
flutter test ./test/widget_test.dart
00:04 +1 -1: Audio Recorder [E]
  Expected: [MethodCall:MethodCall(startRecorder, )]
    Actual: []
     Which: shorter than expected at location [0]

  package:test                                        expect
  package:flutter_test/src/widget_tester.dart 153:16  expect
  test/widget_test.dart 35:5                          main.<fn>
  ===== asynchronous gap ===========================
  dart:async                                          _AsyncAwaitCompleter.completeError
  test/widget_test.dart                               main.<fn>

00:04 +1 -1: Some tests failed.


Tutorial Steps

Hi John,
Step 1.1 - "Add inject and change nodes to enable the following drone commands:"
May I know where do I add the commands after importing the "commands starter flow"
thanks,
Ashish

Hi Ashish - Take a look at the solution flow.  It gives you tips.
-- John

Thanks, I'll check it out.

On Jul 4, 2019 at 18:20,  <John Walicki (mailto:notifications@github.com)>  wrote:
Hi Ashish - Take a look at the solution flow. It gives you tips.
-- John
—
You are receiving this because you authored the thread.
Reply to this email directly,   view it on GitHub (https://github.com/johnwalicki/Node-RED-Tello-Control/issues/10?email_source=notifications&email_token=AGEDPXGSGWBN7H56GALT2ULP52HTBA5CNFSM4H4JUBPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZIIPPY#issuecomment-508594111), or   mute the thread (https://github.com/notifications/unsubscribe-auth/AGEDPXGBUV3JV2PUYCM5XIDP52HTBANCNFSM4H4JUBPA).


using urlParser instead of split to fix creation of longer signed urls

Updated the library in order to be able to sign url where the path is longer.
Added a specific test where module was not working properly
Ie. using url:
<EMAIL_ADDRESS>

Thanks for this!


Google Analytics Embed Example outdated.

https://ga-dev-tools.appspot.com/embed-api/server-side-authorization/ is outdatet.
# service-account.py

from oauth2client.service_account import ServiceAccountCredentials

# The scope for the OAuth2 request.
SCOPE = 'https://www.googleapis.com/auth/analytics.readonly'

# The location of the key file with the key data.
KEY_FILEPATH = 'path/to/json-key.json'

# Defines a method to get an access token from the ServiceAccount object.
def get_access_token():
  return ServiceAccountCredentials.from_json_keyfile_name(
      KEY_FILEPATH, SCOPE).get_access_token().access_token

I have trouble making this work with google-auth:
My try:
# service-account.py

from google.oauth2.service_account import Credentials

# The scope for the OAuth2 request.
SCOPE = 'https://www.googleapis.com/auth/analytics.readonly'

# The location of the key file with the key data.
KEY_FILEPATH = 'path/to/json-key.json'

# Defines a method to get an access token from the ServiceAccount object.
def get_access_token():
  return Credentials.from_service_account_file(KEY_FILEPATH, scopes=[SCOPE]).get_access_token().access_token

Note that the docs you link above aren't maintained in this repository.   Please report the issue here.

@tseaver No problem, I will do that, just one question: Are you employed for google? I know its an open-source project, but to use payed google services...

@julian-r I am paid by Google as a contractor, versus being an employee.


Puncture and shorten ClassicalCodes

These methods will be used in code-search experiments.
Also: added equality testing between ClassicalCodes, and added the ClassicalCode.from_generator constructor.
@mittaltushant no rush, but if you get the chance can you please take a quick look to make sure this looks correct?

@mittaltushant I'm going to merge but nbd if we need to revise after.

For the equality test function, it seems that the check would not work if one uses a different (but equivalent) parity check matrix/generator matrix. A more foolproof method would be to check if H_1G_2^T = 0 (basically, if the generators of C_2 satisfy all parity checks we have C_2 \subseteq C_1; the other direction can be a simple dimension check).
The rest looks good.

Hmm that's a good point.  I guess that raises a tension in what it means for two ClassicalCode instances to be "equal", or what the expectations of a user would be if I told them that code_a == code_b.  Thinking about this more, I'm hesitant to say that code_a == code_b if the two code instances behave differently -- for example by giving rise to different hypergraph product codes.  That could be pretty confusing.
Certainly my current equality test is misguided.  I'm sort of leaning having __eq__ just test equality of parity check matrices (<--> equality between "code instances"), and adding a ClassicalCode.equiv method that checks (C_a ⊆ C_b) & (C_b ⊆ C_a).  Does that sound reasonable?

Yeah it sounds good! I added a comment to green flag the merge but "closed it” by mistake.
Sorry for that,
Tushant

On Mar 26, 2024, at 8:44 AM, Michael A. Perlin @.***> wrote:
Hmm that's a good point. I guess that raises a tension in what it means for two ClassicalCode instances to be "equal", or what the expectations of a user would be if I told them that code_a == code_b. Thinking about this more, I'm hesitant to say that code_a == code_b if the two code instances behave differently -- for example by giving rise to different hypergraph product codes. That could be pretty confusing.
Certainly my current equality test is misguided. I'm sort of leaning having eq just test equality of parity check matrices (<--> equality between "code instances"), and adding a ClassicalCode.equiv method that checks (C_a ⊆ C_b) & (C_b ⊆ C_a). Does that sound reasonable?
—
Reply to this email directly, view it on GitHub https://github.com/Infleqtion/qLDPC/pull/41#issuecomment-2020470622, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE6JBDIUOULEIA4PQ5T2GWDY2F3URAVCNFSM6AAAAABFF3OOLCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRQGQ3TANRSGI.
You are receiving this because you were mentioned.


vSphere post-install config link

https://github.com/openshift-assisted/assisted-installer-ui/blob/0b60c3bbfc98f4de0ca60b8c104c3375a92e4921/libs/ui-lib/lib/common/config/docs_links.ts#L50
https://docs.redhat.com/en/documentation/assisted_installer_for_openshift_container_platform/2024/html-single/installing_openshift_container_platform_with_the_assisted_installer/index#vsphere-post-installation-configuration_installing-on-vsphere

Jira ticket: https://issues.redhat.com/browse/MGMT-18265

Now in production: https://console.redhat.com/openshift/assisted-installer/clusters/~new


[Bug Report] el-switch problem

Element UI version
1.4.12
OS/Browsers version
Google Chrome 62.0.3202 (WebKit 537.36)
Vue version
2.5.9
Reproduction Link
https://jsfiddle.net/coderosg/4voyc4dn/
Steps to reproduce
When I submit my form el-switch send value as "on".
What is Expected?
I want that el-switch sent the value that I indicated in on-value or off-value.
What is actually happening?
Actually el-switch send default system values.

If you must use FormData, you need to do some conversion on your end. Form elements are designed for JSON.

Can you show this in the example?


now the shore works

Motivation
closes #64
What was changed/added
added drop zones to the Shore with new Group "shore_dropzone"
resolved branch conflicts from #103
Testing
https://github.com/mango-gremlin/arch-enemies/assets/44133213/672b9d7c-4c90-4631-96cb-4b36a83fe1a5

the Allignment is now fixed and the squirrel works correctly.
i merged main into my branch


Device information in patching, option to fallback on host

Scikit-learn patching messages reformulated to mention Intel(R) Extension for Scikit-learn* instead of oneDAL
SKLEARNEX INFO: <estimator.method>: running accelerated version on [CPU/GPU]
SKLEARNEX INFO: <estimator.method>: fallback to original Scikit-learn
SKLEARNEX INFO: <estimator.method>: failed to run accelerated version, fallback to original Scikit-learn
Logging information extended with basic device information where accelerated code runs.
New parameter host_offload_on_fail added to sycl_execution_context that allows to fallback on the host if GPU execution is failed.

@rlnx @PetrovKP @napetrov please share your opinions especially about extending the patching information message.

/intelci: run

/azp run

/intelci: run

/intelci: run

/intelci: run

add update message here https://github.com/intel/scikit-learn-intelex#-intelr-extension-for-scikit-learn-verbose, https://github.com/intel/scikit-learn-intelex/blob/master/doc/sources/verbose.rst and https://github.com/intel/scikit-learn-intelex/blob/master/doc/daal4py/sklearn.rst#scikit-learn-verbose

/intelci: run


Graphical glitches relating to mousing away from elements/controls.

Mousing over elements (listed items, buttons, etc) and moving away from them sometimes leaves the text stretched out of the bounds of the control itself. Also present in BSMA.

Same issue over here.


Restore using the needed DWO_IMG for e2e tests

What does this PR do?
Fix e2e and happy path PR checks. See commits for details.
What issues does this PR fix or reference?
one of the commit should fix https://github.com/devfile/devworkspace-operator/issues/484
Is it tested? How?

PR Checklist

[ ] E2E tests pass (when PR is ready, comment /test v8-devworkspaces-operator-e2e, v8-devworkspace-happy-path to trigger)

[ ] v8-devworkspaces-operator-e2e: DevWorkspace e2e test
[ ] v8-devworkspace-happy-path: DevWorkspace e2e test

/test v8-devworkspaces-operator-e2e, v8-devworkspace-happy-path

/test v8-devworkspaces-operator-e2e, v8-devworkspace-happy-path

merging to fix e2e tests. will check happy path later


Play URL uses internal docker IP address

When running Music Assistant in a Docker Deployment,  it provides URLs with the internal docker IP address.
Media players which are not deployed inside  the docker network are not able to play the media.
Please add a configuration option for the IP address that is reported.

It uses the internal IP from your HA config so if you correct that, this issue is fixed.

Thanks for the incredibly fast response.
I cannot change the internal IP address from the GUI. See screenshot below.

Since that is a HASS issue and not MASS, I will dig deeper on how to change it in HASS.
A configuration option would still make it easier for people with Docker deployments or people using setups with reverse proxies for security/availability

In that case there is a HA issue... other things in HA also work on base of that internal IP, such as TTS.
So key is to fix the root issue with the internal URL.
Did you restart HA after setting the URL ?

Yes, I did restart it. Still getting the address from the network interface.

I got around the problem by putting mopidy in the same docker network.
Thanks again for MASS. It is great :)

@neurove I am having the same issue and would like to better understand your workaround with mopidy.

Hi,
Music assistant creates a playlist inside mopidy with the http streaming link. That link had the internal IP address in my setup.
You can manually create a playlist inside mopidy with the correct http link (i.e. replace the IP address inside the original http link with the correct IP address).
Hope that helps

@neurove @Nilson I believe that #516 might fix the issue for you both. I was running into something similar but with kubernetes instead of docker.


Subdomain used by students

Trang web: https://portal.ptit.edu.vn

@tuanldas I am sorry, the domain stu.ptit.edu.vn is removed from our trusted domains list because the school allows to register an email address there even if you are not a student of the school. If you are the student of this school, please apply with official document (OFFICIAL DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).


Typing: staleTime not available as prop in static query

Hello!
I ran into a small issue when trying to use the library.
It seems that the staleTime is not available in types when using a static query without any queryKey, while it works just fine when the query is dynamic:

Is this behaviour intended? I would assume the props should be similar in both cases.
Thanks in advance for reply!

@Dammic not sure if you've figured this out by now, but the typings are misleading either way - helper functions of createQueryKeys only return queryFn and queryKey. All other react-query useQuery options you may pass (staleTime, gcTime etc will be ommitted).
This can be seen in the implementation:
https://github.com/lukemorales/query-key-factory/blob/e110092b6c736494483b5c1326a316f702fdeb92/src/create-query-keys.ts#L55-L58
@lukemorales I think this behavior should be documented, or support added for all other useQuery() options being passed through, as current typings wrongly indicate this is supported and one may lose a bit of sanity trying to figure out why e.g queries aren't always getting fresh data when using {gcTime: 0, staleTime: 0}

@gomesalexandre the behavior is documented, there are no examples with options other than queryKey and queryFn, and nowhere in the documentation it is told that every query option is supported, every attempt of using anything else other than queryKey and queryFn it's the user to blame.
With that said, the issue is that typescript widens the type of the object as long as some of them match the expected format (with queryKey and/or queryFn) and it allows extra keys not defined before. But I'm already working on a rewrite of the lib to allow all queryOptions to be defined and problems like this will not happen again

That was super fast @lukemorales, thank you! 👑


chore: find a logo idea

Two ideas

yeah let see what @jviotti think about it :)

Looks great! Feel free to start using it on the README and Doxygen

I think this is a bit more consistent with the org's logo, helps with product branding in general to have imagery consistency


Using Ghidxx

Using NSGAY software for reverse-engineering videogames, including Animal Crossing is gross and unethical. All of your work should be discredited and removed.
As a software developer, I find it highly unethical and will not support videogame de-compilation projects using such software.
Seriously, github needs to approve repositories and not just let anyone dump projects without vetting.

i'm gonna be really honest nobody cares

And that's the problem. "No one cares". That's not a good enough answer, honestly, this is why the world has a lot of problems.

if you have enough free time to care about people who are passionate about decompiling and figuring out the intricate clockwork of a comfort game many of them have been playing for over 20 years, then you need to find a hobby
also, the issues tab is for programming issues inside the repo itself, so this is technically a misuse of the issues tab

also thanks for the compliment, i know i'm shitty <3

Ghidxx? Ghidxx isn't any software I've ever heard of.


Regression: false positive in a function returning a function

For the following code:
def make_foo(...):
    def foo(...):
        pass
    return foo

the line with pass is now reported as Missing (it was not reported as such a month ago).
When I replace pass with raise Value("missing coverage"), I see many ValueError: missing coverage traces.
Version: version 4.5.2 with C extension.
Also present in version 4.5.1 with C extension.

Thanks.  What version were you using a month ago?  4.5.1 came out in February 2018.

I don't have a record of what I was using before, probably 4.5.1.
I am stumped too.

Can you create a complete runnable example?  Links to large projects are fine if you have specific instructions of how to run them, and where to see the problem.

Okay, I now have the commit in my project which introduced the weirdness.
How would you like me to debug it?

Can you share the commit diff?  Is it a requirements change, or a code change?

It is a code change.
What was done in-process before, is now done via joblib.Parallel(prefer="processes").
Here is the test example: code/foo.py:
def make_foo(a):
    def foo():
        print(a)
    return foo

tests/foo_test.py:
import unittest

import code.foo as f
import joblib

class TestFoo(unittest.TestCase):
    def test_foo(self):
        foo = f.make_foo("test_foo")
        self.assertTrue(hasattr(foo,"__call__"))
        with joblib.Parallel(n_jobs=3, prefer="processes") as pool:
            self.assertEqual([None],pool([joblib.delayed(foo)()]))

execute:
coverage3 run --source=code -m unittest discover --pattern *_test.py && coverage3 report -m
output:
test_foo
.
----------------------------------------------------------------------
Ran 1 test in 0.284s

OK
Name               Stmts   Miss  Cover   Missing
------------------------------------------------
code/__init__.py       0      0   100%
code/foo.py            4      1    75%   3
------------------------------------------------
TOTAL                  4      1    75%

note that print(a) in foo.py is reported as Missing while, in fact, it is hit because you can see test_foo printed in the output.

It sounds like the code is being run in a subprocess when it wasn't before?  Have you tried configuring subprocess measurement? https://coverage.readthedocs.io/en/v4.5.x/subprocess.html


Units in Grbl mode

I tried to fix the mm/in problem in Grbl mode and mostly succeeded, after a lot of effort.  The fundamental difficulty is that the units for Grbl's position reports depend on the $13 setting, and cncjs does not maintain up-to-date knowledge of that setting, nor does it synchronize that knowledge with its copy of that position state.
To make things work "mostly right", the pendant asks for the settings on startup and also tracks $13= commands that might be issued via the serial console.  That works except in a rare circumstance.  There is an additional complication that the pendant receives its initial state report prior to the settings response, which required some work to delay the handling of the initial state report.
There are still a couple of problems, which are related to one another:

cncjs itself does not handle Grbl units correctly if $13=1 (inch reporting mode).  It seems to assume implicitly that $13=0
If you issue $13=1 in the serial console, then immediately restart the pendant, its initial DROs will be wrong because cncjs has a stale copy of the positions in mm units and that is what it tells the pendant.

One possible solution would be to disallow $13=1 when used with cncjs.  That is a potential source of confusion for a user whose machine has that setting.  It would be possible for cncjs to force the $13=0 setting, but that has its own problem in the case where you use both cncjs and other apps to drive the machine.  But maybe that is okay considering how poorly most other Grbl senders handle units.
The best solution, from the standpoint of "just works", would be for the cncjs server app to keep track of $13 and synchronize its position state to the initial value of $13 and later changes.
Please feel free to reject this pull request if you think its too much of a can of worms.  I'm submitting it as an example of the scope of the problem.

Thank you for the PR. I think the $13 setting should be handled in CNCjs as well. Could you post an issue on https://github.com/cncjs/cncjs/issues so I can keep track of it?


neg_binomial errors refer to "phi" as "precision", creating confusion

When using a neg_binomial, the phi parameter must be above 0. If, in the modeling process, this parameter goes to 0, we get this error:
Exception thrown at line 118: stan::prob::neg_binomial_2_log_log: Precision parameter is 0, but must be > 0!
This error message calls phi "precision". That's not helpful! Not knowing what the error was about, I had assumed that the "precision" parameter had something to do with numerical precision. The negative binomial literature doesn't typically use the name "precision" to refer to phi. I only pieced together the error message was referring to phi from this old issue: https://github.com/stan-dev/stan/issues/786
I suggest the error message be changed. If there is some mathematical connection between phi and the concept of precision, and thus one wants to still refer to phi as "precision", perhaps replace the current error with Precision parameter phi is 0....
Current Version:
v2.9.0

The name "precision" is not applied to the variable phi in general;  it's the role played by the argument.
The doc in the manual says:
real neg_binomial_2_log(ints y, reals mu, reals phi)
The negative binomial probability mass of n given location mu and precision phi.

It may also be helpful to add the identifier for the parameter (here, "phi").

This issue was moved to stan-dev/math#297

This was moved to the math repo.


How do we use flash-attn?

Compiled it and there is no way, or is there, that this should be this slow.  2it/s on a 4090?

I'm getting 2.92 it/s at 1024x1024 with 4090 when using flash attention, so yeah it's bit slow. Without it you'd be looking at seconds per iteration instead, so it does seem to be working if you are using a higher resolution.

Alright, now the quality I just showed discord and I am stumped.

So this is why I was getting around 30-40 sec / it on rx 6600 :) we don't have a way to flash attention at all here so good luck to nvidia brethren.

I'm getting 2.92 it/s at 1024x1024 with 4090 when using flash attention, so yeah it's bit slow. Without it you'd be looking at seconds per iteration instead, so it does seem to be working if you are using a higher resolution.

Man, I just can't seem to get it installed. Every time I try "pip install <pasted link to one of the whl files" it just keeps saying ERROR: flash_attn-2.5.9.post1+cu122torch2.3.1cxx11abiFALSE-cp39-cp39-win_amd64.whl is not a supported wheel on this platform.".  I've tried them all.  I'm on windows 11 with (2.3.1+cu121) torch.

I'm getting 2.92 it/s at 1024x1024 with 4090 when using flash attention, so yeah it's bit slow. Without it you'd be looking at seconds per iteration instead, so it does seem to be working if you are using a higher resolution.

Man, I just can't seem to get it installed. Every time I try "pip install <pasted link to one of the whl files" it just keeps saying ERROR: flash_attn-2.5.9.post1+cu122torch2.3.1cxx11abiFALSE-cp39-cp39-win_amd64.whl is not a supported wheel on this platform.". I've tried them all. I'm on windows 11 with (2.3.1+cu121) torch.

What's your Python version? That file is "cp39" which would be for Python 3.9.

I'm getting 2.92 it/s at 1024x1024 with 4090 when using flash attention, so yeah it's bit slow. Without it you'd be looking at seconds per iteration instead, so it does seem to be working if you are using a higher resolution.

Man, I just can't seem to get it installed. Every time I try "pip install <pasted link to one of the whl files" it just keeps saying ERROR: flash_attn-2.5.9.post1+cu122torch2.3.1cxx11abiFALSE-cp39-cp39-win_amd64.whl is not a supported wheel on this platform.". I've tried them all. I'm on windows 11 with (2.3.1+cu121) torch.

What's your Python version? That file is "cp39" which would be for Python 3.9.

I'm on Python 3.10.11 (windows). so that nugget you mentioned helped a lot. That said I tried every cp10 with the right cuda 1.21 that I had, and every one blew up my various comfy nodes (extra samplers etc), so eventually I gave up. I found a relatively fast workflow at 2.40:1 aspect ratio that renders in about 21 seconds with sdxl refiner. Thanks for the work on these nodes. Enjoy a picture. :)

How can I check what is my python environment in my ComfyUi installation to install flahs-attn?

Ok, stupid question, how do I know if I've installed correct flash-attn and it works? I've ran pip install "correct_wheel", it said intalled succesfully, but I'm running comfyui through stability matrix and it has it's own python, so I don't understand if I installed it to my global installation or to the local python in matrix.
Getting pretty long gen times, but no errors, is there some message in the console which shows the mode it's working in? Don't see any errors.
Sampling time looks like this:
ODE Sampling: 100%|██████████| 48/48 [02:08<00:00,  2.67s/it]
2+ minutes for 25 steps on 1024x1024 image, on 8gb 3070. I'm not going into low ram mode, it sits at about 7.7gb of VRAM.
Maybe it's working as intended? Hard to say. Pixart, sd3 and sdxl are sampling about 2-3 times faster for the same steps and resolution.

That does sound like it's not working, I'll add console message to make it
clearer. I don't actually use stability matrix so I don't know how it has
the python environment setup, it definitely doesn't use your global python
though.
On Tue, 18 Jun 2024, 10:45 sdk401, @.***> wrote:

Ok, stupid question, how do I know if I've installed correct flash-attn
and it works? I've ran pip install "correct_wheel", it said intalled
succesfully, but I'm running comfyui through stability matrix and it has
it's own python, so I don't understand if I installed it to my global
installation or to the local python in matrix.
Getting pretty long gen times, but no errors, is there some message in the
console which shows the mode it's working in? Don't see any errors.
Sampling time looks like this:
ODE Sampling: 100%|██████████| 48/48 [02:08<00:00, 2.67s/it]
2+ minutes for 25 steps on 1024x1024 image, on 8gb 3070. I'm not going
into low ram mode, it sits at about 7.7gb of VRAM.
Maybe it's working as intended? Hard to say. Pixart, sd3 and sdxl are
sampling about 2-3 times faster for the same steps and resolution.
—
Reply to this email directly, view it on GitHub
https://github.com/kijai/ComfyUI-LuminaWrapper/issues/5#issuecomment-2175423010,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJXG5EZMYA3VHJGQFVJ33TTZH7QSDAVCNFSM6AAAAABJOWBPMSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZVGQZDGMBRGA
.
You are receiving this because you commented.Message ID:
@.***>

definitely doesn't use your global python though.

yeah, i've tried installing directly with python.exe in local python folder, it said "already intalled".
trying one more time with "--force reinstall", I hope it will not break verything else :)

ok, now I see I have cu121, and the file says cu122.
but I can't understand how to instal pytorch with cu122, their configurator does not have it:

Found the button to install the wheel directly in matrix interface, looks like it worked, time is down to 1+ minute, looks like it's as fast as I'll get on my gpu. Sorry for the comment spam, thanks a lot fot the node :)
ODE Sampling: 100%|██████████| 48/48 [01:13<00:00,  1.54s/it]

How can I check what is my python environment in my ComfyUi installation to install flahs-attn?

This depends what kind of comfy install you have, it generally should tell you in the console log when you start Comfy.
Manually without running comfy, if it's the portable you can run this to show pytorch version:
python_embeded\python.exe -m pip show torch
Python version:
python_embeded\python.exe --version

what i miss?

what i miss?

Probably missing one of the dependencies from the requirements.txt

Would I gain anything by going flash-attn2?

Would I gain anything by going flash-attn2?

As far as I understood "flash attention 2" is just flast_attn versioning, as in 2.5.9 is flash attention 2 already.

I only ask because I built it and some other package complained that it was 1.


Chimoney Github Bot

Based on #12

@uchibeke @phyleria
I can't implement the email feature beacuse i dont have a vps to test

/payout $25

/payout $25


User Facing Light Client Issues

Issues found while using light client:
After connecting to Hub (0x5257964Ef9b81fba7276AF2a97c111AaD7B840D6) it doesnt show how much WETH I have available to open a channel, it only shows available WETH once ive typed in a number.
After typing in a number I have to put a whole number (1,2,3,4...) before it allows me to put a decimal place. I.e. I cannot just put .5, i have to put 3.5 then delete the 3.
When opening a channel, there is a step guy on top showing the stages. The image is obscured, each process has a strange diamond/square on top to make the steps not readable.

Unable to open channel: Set total Deposit Failed

When retrying it goes straight to "oops something went wrong, channel open failed"

@MrCleanerPickerUpper Thanks for testing the light client :-)
For the first two paragraphs, I have created separate issues (see #299 and #300).
Could you provide us information

which browser you are using
the error shown in the browser console logs (if you are familiar with it)

@MrCleanerPickerUpper The current version uses the Red Eyes contracts. The Red Eyes contracts unfortunately only allow up to 0.15 as a deposit and everything bigger will actually fail. We will start a discussion to integrate the limit checking in the newer version.
In regards to the stepper, I have seen this in a few browsers and we should probably remove it temporarily until we find a cross-browser compatible way to implement that doesn't break. @roy-castrillo.

@kelsos remove the stepper completely or leave it, but remove the shapes behind them? See the example:

Just remove the shapes, yes that looks great :D
On Tue, Sep 3, 2019 at 1:58 PM Roy Castrillo<EMAIL_ADDRESS>wrote:

@kelsos https://github.com/kelsos remove the stepper completely or
leave it, but remove the shapes behind them? See the example:
[image: Screenshot 2019-09-03 at 13 57 29]
https://user-images.githubusercontent.com/47739420/64171385-d2283000-ce52-11e9-8587-db1612f3b757.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/raiden-network/light-client/issues/298?email_source=notifications&email_token=AARKEJBEY6M2NUSP35GSD5LQHZGMXA5CNFSM4ISUBVNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5X6MXI#issuecomment-527427165,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AARKEJDVYJO3I7YO7OLTQ2DQHZGMXANCNFSM4ISUBVNA
.

Approach

First try to fix the current solution.
If this is not doable implement the new approach

@MrCleanerPickerUpper Thanks again for your feedback
We have created two additional issues regarding your feedback:

#302
#313


use batch map operators

https://github.com/torvalds/linux/commit/990bca1fc8ad added support for batch operations. As something like i:s:1 { print(@); clear(@); } is quite common in tracing scripts this might be useful for us.  We can use lookup_batch and delete_batch to do the whole batch in one go.
Even better if we batch the two, i.e. we detect the print(@);clear(@); and merge it into print_and_clear(@) internally which does the lookup_and_delete_batch.

poc

Can't see any downsides. Seems like a pure win.


classifier.create should accept single feature

model = tc.classifier.create(train, 'type', features='title')
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-21-d671048debb5> in <module>()
----> 1 model = tc.classifier.create(train, 'type', features='title')

/home/toby/anaconda2/lib/python2.7/site-packages/turicreate/toolkits/classifier/_classifier.pyc in create(dataset, target, features, validation_set, verbose)
    101             model_selector = _turicreate.extensions._classifier_available_models,
    102             features = features, validation_set = validation_set, verbose =
--> 103             verbose)

/home/toby/anaconda2/lib/python2.7/site-packages/turicreate/toolkits/_supervised_learning.pyc in create_classification_with_model_selector(dataset, target, model_selector, features, validation_set, verbose)
    496             features.remove(target)
    497     if not hasattr(features, '__iter__'):
--> 498         raise TypeError("Input 'features' must be a list.")
    499     if not all([isinstance(x, str) for x in features]):
    500         raise TypeError("Invalid feature %s: Feature names must be of type str" % x)

TypeError: Input 'features' must be a list.

Unclear user value for allowing single-feature model creation. Typical models will use more than one feature. A list of a single feature is not hard to provide if that's what is desired. Won't Fixing this issue.

@znation - I agree this certainly isn't hard for the user to workaround. To me the case for fixing it is about API consistency; there are many places in Turi Create where you can provide a list of things, but if you only provide a single value, then it also just works.

Thanks @TobyRoseman - I'm sold on the API consistency argument. I'll reopen.

I've changed my mind on this one. After reviewing all of the relevant toolkits, I think it would be much more consistent not to implement this change.


Fix balances for tokens with less than 18 decimals

There has been mentions of incorrent displayed balances. It was caused by the token's decimals being less than 18.
I'm not sure if this is the best approach, but I tested it with ZIL and DOGE and seems to work fine.
ZapIn txn hash: 0x9d3f4edbf0ba5311d9d3d4d7657535d07d96a0d2c18ef39e100909f99454d40a
ZapOut txn hash: 0x23e6f6e843808e7bd914bb3e4228ae5fb60e95063d71dc0980f8f99490c42138
Also, in Line 57 of fetchBalances, ...tokens[balanceResult.symbol], was appended in newTokens instead of ...newTokens[balanceResult.symbol],, I'm not aware if  that was a typo or not.

I would suggest to fix the problem in the view and avoid manipulation of state data.

My next thought is that the problem might be in the conversion of tokenBalance to 8 decimalPoints, there is no check of how many decimal points the balance has in the first place.
What do you think of a change there? Maybe add a function .decimalPlacesFromTo(fromDecimals, toDecimals) that handles the convertion?

My next thought is that the problem might be in the conversion of tokenBalance to 8 decimalPoints, there is no check of how many decimal points the balance has in the first place.
What do you think of a change there? Maybe add a function .decimalPlacesFromTo(fromDecimals, toDecimals) that handles the convertion?

I think we need this method in our repo: https://github.com/Uniswap/uniswap-v2-sdk/blob/518ff797c62e28933c3a40548aa9fd70050f35a9/src/entities/fractions/fraction.ts#L52-L74


User should be able to purchase a coupon

A new User purchases a coupon.
A new Coupon will be added to the system with max-redemptions=1
A new Digital Asset will be added to the user's account with a text value equal to the coupon code.
*In the future we may link directly to the coupon. For now, making it a text value will make this simple.

This issue is replaced by #34 using Gift Cards.


add test cases

Add test cases for inline-module.
Why use puppeteer? In the begining, i use jest + jest-dom to mock script. However jest-dom not support script[type="module"], it cannot run ESM code.
Maybe have another light-weight test platform to choose.

Wonderful, thank you.


Backup window selection is deprecated

Hi.
Hetzner deprecated the backup_window field.

https://github.com/thlisym/hetznercloud-py/blob/5a6bac4bb3e476e28685e203396576f051b6bd8a/hetznercloud/servers.py#L171
API documentation: https://docs.hetzner.cloud/#server-actions-enable-and-configure-backups-for-a-server

Hi! You should look into using the official HCloud Python library as this one is deprecated (and has been for a while now).
https://github.com/hetznercloud/hcloud-python


Support ORDER BY xxx NULLS FIRST | LAST for Oracle

Please answer these questions before submitting your issue. Thanks!
Which version of Sharding-Jdbc do you using?
1.5.1
Expected behavior
support ORDER BY xxx NULLS FIRST | LAST for Oracle.

fixed at 1.5.2


How to convert tensorflow model to .wts

Hi @zhaozhixu,
How to convert tensorflow checkpoint model to sqdtrt.wts
Thanks

Hi @pribadihcr ,
You can use these two scripts:

Put them in the same folder and run python genwts_tf.py -h to see the usage. You have to keep the tensor names consistent with weight names in sqdtrt.cpp. (All scope separators '/' are replaced to '_'.)
Actually LightNet would be a better solution for general neural network models.

Hi @pribadihcr ,
You can use these two scripts:
https://github.com/zhaozhixu/LightNet/blob/master/tools/genwts_tf.py
https://github.com/zhaozhixu/LightNet/blob/master/tools/genwts.pl
Put them in the same folder and run python genwts_tf.py -h to see the usage. You have to keep the tensor names consistent with weight names in sqdtrt.cpp. (All scope separators '/' are replaced to '_'.)
Actually LightNet would be a better solution for general neural network models.

Hi @zhaozhixu ,
Thanks. for LightNet does it support object detection task. or just replace squeezenet backbone with it.

@pribadihcr Yes. Its design goal is to support the optimization and deployment on various backends for general models. Some demoes with TensorRT including squeezedet have been finished and will be committed soon.

Hi @zhaozhixu,
Why my converted model .wts has bigger size?.. its 36.9 MB. The checkpoint TF model is aorund 17 MB. But the converted model seems work. Thanks

@pribadihcr Because TensorRT uses literal hexadecimal strings to store weights. That means two chars for one byte, so it's about twice the size.


Add link to plantcv docs

Adds a link at the top of the TOC to link back to the PlantCV documentation homepage.

Pull Request Test Coverage Report for Build 55

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 33.762%

Totals

Change from base Build 54:
0.0%

Covered Lines:
105

Relevant Lines:
311

💛  - Coveralls


[bbc] world-europe-60630352: Unable to extract playlist data

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.02.04. (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
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
UK
Description
bbc says to report the issue here. --write-pages file attached.
Verbose log
$ ./yt-dlp -vU --write-pages -F https://www.bbc.co.uk/news/world-europe-60630352
[debug] Command-line config: ['-vU', '--write-pages', '-F', 'https://www.bbc.co.uk/news/world-europe-60630352']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, err UTF-8, pref UTF-8
[debug] yt-dlp version 2022.02.04 [c1653e9ef] (zip)
[debug] Python version 3.6.3 (CPython 64bit) - Linux-4.13.0-46-generic-x86_64-with-Ubuntu-17.10-artful
[debug] exe versions: ffmpeg 3.3.4-2, ffprobe 3.3.4-2, rtmpdump 2.4
[debug] Optional libraries: sqlite
[debug] Proxy map: {}
Latest version: 2022.02.04, Current version: 2022.02.04
yt-dlp is up to date (2022.02.04)
[debug] [bbc] Extracting URL: https://www.bbc.co.uk/news/world-europe-60630352
[bbc] world-europe-60630352: Downloading webpage
[bbc] Saving request to world-europe-60630352_https_-_www.bbc.co.uk_news_av_world-europe-60630352.dump
ERROR: [bbc] world-europe-60630352: Unable to extract playlist data; please report this issue on  https://github.com/yt-dlp/yt-dlp , filling out the "Broken site" issue template properly. Confirm you are on the latest version using -U; please report this issue on  https://github.com/yt-dlp/yt-dlp , filling out the "Broken site" issue template properly. Confirm you are on the latest version using -U
  File "./yt-dlp/yt_dlp/extractor/common.py", line 612, in extract
    ie_result = self._real_extract(url)
  File "./yt-dlp/yt_dlp/extractor/bbc.py", line 1264, in _real_extract
    webpage, 'playlist data'),
  File "./yt-dlp/yt_dlp/extractor/common.py", line 1186, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)

world-europe-60630352_https_-_www.bbc.co.uk_news_av_world-europe-60630352.dump.txt

Fixed by 50e93e03a7ca6ae35a319ea310104f7d6d91eee3


Better error message for unparenthesized function/constructor type notation in union/intersection types

Fixes #39548.
With this PR, we parse unparenthesized function types in union/intersection types anyway, and emit a specialized error message for them.
Example
type Foo = string | () => string;

type InstOrConst<T> = T | new () => T;

Current Behavior (TS 4.0 beta)
src/index.ts:1:22 - error TS1110: Type expected.

1 type Foo = string | () => string;
                       ~

src/index.ts:1:24 - error TS1005: ';' expected.

1 type Foo = string | () => string;
                         ~~

src/index.ts:3:31 - error TS1005: ';' expected.

3 type InstOrConst<T> = T | new () => T;
                                ~

Found 3 errors.

New Behavior
test.ts:1:20 - error TS1385: Function type notation must be parenthesized when used in a union type.

1 type Foo = string | () => string;
                     ~~~~~~~~~~~~~

test.ts:3:26 - error TS1386: Constructor type notation must be parenthesized when used in a union type.

3 type InstOrConst<T> = T | new () => T;
                           ~~~~~~~~~~~~

Found 2 errors.

@typescript-bot pack this
@typescript-bot run dt
@typescript-bot perf test this

Hmm, parse time is frequently just baaarely significantly slower, and never significantly faster, so I'd call this a noticeable but very small slowdown. @DanielRosenwasser do you think this is still worth taking? I'd like to see a round or two of optimisation personally.

@sandersn Thank you for your reviews! I applied all your suggestions including refactoring isStartOfFunctionType to isStartOfFunctionTypeOrConstructorType.
I have no strong opinion on the performance concern; if an optimization is needed, maybe we could first try to parse normally and fallback to function type parsing after encountering a parse error.

@typescript-bot perf test this

@typescript-bot perf test this

These perf results seem weirdly suspicious. If the next one doesn't seem to have those same jumps, I think it's safe to call it a fluke.
@typescript-bot perf test this


host_api_call not emitted by gvki?

I need these to mimic the behaviour of the old interceptor.

@jeroenk Why do we actually need these? The current design doesn't really allow doing this very easily.

For the "macro library" it's possible but would require another level of indirection
For the "preload library" implementing this would be hard because this would require using an API to inspect the currently running binary and get a stracktrace which also requires debug symbols. Doing this is likely to be different on every platform or it might not even be possible on that platform.

There is also something in the JSON specification that isn't really very clear which is what calls should be logged in the case of multiple kernel invocations. For the second kernel invocation would we just list the host API calls in between the last kernel invocation?

I understand that this isn't very easy in the preload case. I was mostly interested in the macro case. In this case I'm trying to match the behaviour of the old interceptor. There we were interested in the line/file where the kernel was compiled and the line/file where the kernel was invoked. Those should be uniquely determined, right?

I understand that this isn't very easy in the preload case. I was mostly interested in the macro case.

I can have a go at implementing emission of host_api_call in the macro case. What should I emit in the case the "preload library" is being used. Just not emit the attribute or emit the attribute with it mapping to an empty array?

There we were interested in the line/file where the kernel was compiled and the line/file where the kernel was invoked. Those should be uniquely determined, right?

Yes.
I'm still trying to understand though. Why was this behaviour of the old kernel interceptor useful?

What should I emit in the case the "preload library" is being used.

Nothing, the attribute is optional

Why was this behaviour of the old kernel interceptor useful?

Suppose the verification of one of your kernels, and you have, say, 115499 kernel invocations, then any bit of help as to where the invocation happened would be useful.

Okay. I'll need to have a think about how to do this. This creates a difference between the JSON files that will be emitted when using the "preload library" and the "macro library" which will break the test suite.

I've implemented support for this the logHostCodeCalls branch. I'm not merging it into master right now because it breaks half the tests. Let me know if works okay for you.
I'll need to think of a way handle this. The file diff method isn't going to work because the "macro library" and "preload library" will behave differently and the "compilation unit" file path could be different on each machine.

Let me know if works okay for you.

This works perfectly, thanks.

I'll need to think of a way handle this.

Move the API calls to the end of the JSON file, run head and then diff?


Error when running the Quick start test, and would love some help!

Hi, I am a math teacher and would like to start making math videos.
But I ran into some problems when running the Quickstart test on the official website:
https://docs.manim.community/en/v0.9.0/tutorials/quickstart.html
This is my code return when I run the file (Using a Macbook air M1 chip):
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.9/bin/manim", line 5, in
from manim.main import main
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/manim/init.py", line 37, in
from .animation.composition import *
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/manim/animation/composition.py", line 12, in
from ..scene.scene import Scene
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/manim/scene/scene.py", line 31, in
from ..camera.camera import Camera
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/manim/camera/camera.py", line 14, in
import cairo
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/cairo/init.py", line 1, in
from ._cairo import *  # noqa: F401,F403
ImportError: dlopen(/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/cairo/_cairo.cpython-39-darwin.so, 2): Symbol not found: _cairo_append_path
Referenced from: /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/cairo/_cairo.cpython-39-darwin.so
Expected in: flat namespace
in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/cairo/_cairo.cpython-39-darwin.so
As mentioned, Im using Apple Silicon the M1 chip on my macbook air,
I checked that I've correctly installed Manim (community version, and by the instructions to use Rosetta2 to install brew and other dependencies), can't really figure out what the probelm is!
Hope you can enlighten me!

You are using the community version, so please ask this question on their discord channel or open an issue in ManimCommunity/manim instead of here.


docs: fix a broken search link

This PR
The link for "speaks your language" links to a wrong spelling of 'flagd' so make it actually work.
How to test
Follow the "speaks your language" link in the "README.md" document.

Updated title from "doc:" to "docs:"

Good catch, thanks!


types: fixing some type problems when compiling against other architectures

Filling in a sec just want a full CI run.
Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

I'd love to know what architecture you are referring too. I've been compiling on aarch64 / arm64 (without hot restart and Lua) for quite a while now. Originally Bazel, rules_go and then later rules_foreign_cc were issues but all have been resolved over time.

@moderation yep! Description updated. I wanted to run all of CI before doing that


Add support for the ext_mesh_shader extension

Changelog:
### Additions
- Support for the `ext_mesh_shader` extension.

This adds full mesh shader support for the EXT mesh shader extension. If we do want to support the NV version, that could be a little trickier, because it has its own draw_mesh_tasks* commands, which must be matched with the type of mesh and task shader. In other words, calling the EXT command requires EXT mesh shaders, and calling the NV command requires NV mesh shaders. Right now, the commands at least check to make sure you're using EXT mesh shaders. Other than the commands, NV mesh shaders are already handled properly so the commands are the only thing left, if we want to do it.
I haven't added an example that uses mesh shaders, as I don't really fully know how they're supposed to be used. This is something we definitely want. Perhaps a mesh-shaderfied version of the triangle example to demonstrate the differences, and then a more elaborate example using task shaders as well?

Seems to work great! Got a Task (just dispatches 4 meshes) + Mesh that outputs a couple rectangles. Yippee!
It required digging into the pipeline validation logic to see that when using Mesh shaders, input state and assembly state must be None. I think the documentation saying those fields must always be Some needs updating :D


progress mapped row count is mislabeled

The mapped row count is showing the total number of source rows, not the number of mapped rows. In this example, the Mapped Row count is 363, but it should be 350. It's okay to show 363, but it should be labeled as SourceRows

now looks like this: