Funimation fails to parse JSON

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2021.04.26
[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

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.funimation.com/shows/talentless-nana/healing/']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.04.26
[debug] Python version 3.9.4 (CPython) - Linux-4.4.0-19041-Microsoft-x86_64-with-glibc2.31
[debug] exe versions: ffmpeg 4.2.4, ffprobe 4.2.4, rtmpdump 2.4
[debug] Proxy map: {}
[Funimation] healing: Downloading webpage
WARNING: [Funimation] healing: Failed to parse JSON Expecting value: line 1 column 1 (char 0)
ERROR: Unable to extract al:web:url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 806, in wrapper
    return func(self, *args, **kwargs)
  File<EMAIL_ADDRESS>line 827, in __extract_info
    ie_result = ie.extract(url)
  File<EMAIL_ADDRESS>line 534, in extract
    ie_result = self._real_extract(url)
  File<EMAIL_ADDRESS>line 96, in _real_extract
    player_url = self._html_search_meta([
  File<EMAIL_ADDRESS>line 1140, in _html_search_meta
    return self._html_search_regex(
  File<EMAIL_ADDRESS>line 1021, in _html_search_regex
    res = self._search_regex(pattern, string, name, default, fatal, flags, group)
  File<EMAIL_ADDRESS>line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract al:web:url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

Funimation extractor fails to parse JSON.

Same thing happening on the Windows side as well:
[debug] System config: [] [debug] User config: [] [debug] Custom config: [] [debug] Command-line args: ['-u', 'PRIVATE', '-p', 'PRIVATE', '--cookies', 'cookies.txt', 'https://www.funimation.com/shows/dr-stone/secret-mission', '--verbose'] [debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252 [debug] youtube-dl version 2021.04.26 [debug] Python version 3.4.4 (CPython) - Windows-10-10.0.21364 [debug] exe versions: ffmpeg 2021-04-14-git-c5ca18fd1b-full_build-www.gyan.dev, ffprobe 2021-04-14-git-c5ca18fd1b-full_build-www.gyan.dev [debug] Proxy map: {} [Funimation] Logging in [Funimation] secret-mission: Downloading webpage WARNING: [Funimation] secret-mission: Failed to parse JSON Expecting value: line 1 column 1 (char 0) ERROR: Unable to extract al:web:url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output. Traceback (most recent call last):   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpeav8wi2f\build\youtube_dl\YoutubeDL.py", line 806, in wrapper   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpeav8wi2f\build\youtube_dl\YoutubeDL.py", line 827, in __extract_info   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpeav8wi2f\build\youtube_dl\extractor\common.py", line 534, in extract   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpeav8wi2f\build\youtube_dl\extractor\funimation.py", line 100, in _real_extract   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpeav8wi2f\build\youtube_dl\extractor\common.py", line 1142, in _html_search_meta   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpeav8wi2f\build\youtube_dl\extractor\common.py", line 1021, in _html_search_regex   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpeav8wi2f\build\youtube_dl\extractor\common.py", line 1012, in _search_regex youtube_dl.utils.RegexNotFoundError: Unable to extract al:web:url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Regex is not matching the URL for the video player to be used in finding the video ID. I think the extractor is outdated and needs to be updated.

The URL for the video player is stored in the src attribute of the iframe element with the id "player". Updating the Regex to match this could fix the issue.

the new video player is very buggy right now I don't know how long it will be for someone of figure out what to do to be able to download from Funimation I want to know when this will start to work again because I can't stand funimations video player

The unit test for the Funimation extractor fails, I think the extractor needs to be updated.

how long do you think it will take for Funimation to work again

I'm not sure; it'll be after someone fixes the extractor. After that the updated extractor will be included in the next release.

I am having the same issue. Hope it gets fixed soon

From what I can tell, funimation switched over to using brightcove as a hosting service for their videos. I looked and there is an extractor for brightcove already in youtube-dl so hopefully it's as simple as making a few tweaks to the brightcove extractor and using it for funimation. But I don't know how to do that. So I hope someone can look into this soon.

The file is less outdated than I thought; I believe the only problems are the HTML metadata searches and bad Regex.

So we will have a patch pretty soon than we thought right

Eh probably not, as the entire layout of the website has changed. The urls are now wrong and it just says that the website is unsupported as the urls the extractor used to look for are now gone.

Not everything has changed, KANE and TITLE_DATA (the major parts) will still work once the regex is fixed.

Well I think the new layout is going to be something like this look "https://www.funimation.com/shows/magikano/?qid=None" but it doesn't look like they have done this to all shows yet. I don't know how much everything else has changed just noticed a layout change.

but it does seem that for now the actual episode pages don't seem to have changed, so hopefully you are right and it just needs some updating to the regex. :-)

there are changes to the episode page but the changes are with how the video is laid out like you can't pick dubbed or subbed when you pick the episode it would edit the url of the to be applied and the way youtube-dl downloaded it was through the url of the episode meaning you can't make the switch from dubbed or subbed anymore

@dstftw What is this a duplicate of? Can you link it so I can follow that one? Because to me it looks like you just closed all of the issues pertaining to this problem without any fixes being issued


The categories of patterns from the previous version have disappeared

Previously, it was possible to select patterns by category: structural, behavioral, and so on

You are right, this is not possible (yet) on the current website. We will add it.


Refactor leaky abstraction of query vs array of queries for serialized queries

Noticed while working on #897 that serializeInvoiceable threw this error if only a query was passed which wouldn't return an array:

TypeError: results.flat is not a function

Example call:
return await serializeInvoicable(
  models.sub.update({ where: { name }, data: { userId: me.id, status: 'ACTIVE' } }),
  { models, hash, hmac, me }
)

Seems like this was never the case so far. We either always used an array of queries or raw queries.

Is this command invoicable though? I don't see a reason to serialize it even.
  models.sub.update({ where: { name }, data: { userId: me.id, status: 'ACTIVE' } }),
  { models, hash, hmac, me }
)```

Is this command invoicable though? I don't see a reason to serialize it even.
return await serializeInvoicable(
  models.sub.update({ where: { name }, data: { userId: me.id, status: 'ACTIVE' } }),
  { models, hash, hmac, me }
)

Yeah that was just a query that triggered the bug, the actual query in #897 is billing the user now (see 9a57af2d).

I think making these consistent is a good idea. I think it might be best to just make them the same function if it makes sense to ... and if something is meant to be invoiceable, additional options are passed.

closing cause #1040


Acknowledge user input cannot be skipped

As of version 0.13.0.post1, it's not possible to skip user input for interactions of type Acknowledge.
This makes it difficult to unit test interactions where no user interaction is possible. In fact when attempting to unit test such a scenario, the test will fail due to a time out waiting for a user input.
Would it be possible to update the Acknowledge interaction so it honours the no_user_input  property like the rest of the questions? I'm happy to submit a PR for this.

Thanks for raising this issue to my attention, that was an oversight. I think it would make sense to add a no_user_input argument to display() like is done with ask().
I'd appreciate a PR to address this.


x/website: Documentation supports code highlighting

What is the URL of the page with the issue?
https://go.dev/doc/tutorial/database-access
https://go.dev/doc/effective_go
etc.
What did you expect to see?
code highlighting for Docs and Tutorials

Also https://github.com/golang/go/issues/40952 and https://github.com/golang/go/issues/52198. I think we should have a single umbrella issue for all of these.

The Go Playground and the website front page are also need code highlighting

Closing. Let's centralize the discussion in the umbrella issue.


fix: roll back schema reg client version to 5.4.2 for compat

Tested this. It's solid.

Solid like a teapot


database setup

Steps to reproduce
MIMIKATZ     <IP_ADDRESS>                      [+] Found credentials in Mimikatz output (domain\username:password)
Exception happened during processing of request from ('<IP_ADDRESS>', 64044)
Traceback (most recent call last):
File "/usr/lib/python2.7/SocketServer.py", line 290, in _handle_request_noblock
self.process_request(request, client_address)
File "/usr/lib/python2.7/SocketServer.py", line 318, in process_request
self.finish_request(request, client_address)
File "/usr/lib/python2.7/SocketServer.py", line 331, in finish_request
self.RequestHandlerClass(request, client_address, self)
File "/usr/lib/python2.7/SocketServer.py", line 652, in init
self.handle()
File "/usr/lib/python2.7/BaseHTTPServer.py", line 340, in handle
self.handle_one_request()
File "/usr/lib/python2.7/BaseHTTPServer.py", line 328, in handle_one_request
method()
File "/usr/local/lib/python2.7/dist-packages/crackmapexec-3.1.5.dev0-py2.7.egg/cme/cmeserver.py", line 33, in do_POST
self.server.module.on_response(self.server.context, self)
File "/usr/local/lib/python2.7/dist-packages/crackmapexec-3.1.5.dev0-py2.7.egg/cme/modules/mimikatz.py", line 219, in on_response
context.db.add_credential(credtype, domain, username, password, hostid)
File "/usr/local/lib/python2.7/dist-packages/crackmapexec-3.1.5.dev0-py2.7.egg/cme/database.py", line 30, in add_credential
cur.execute("INSERT INTO credentials (credtype, domain, username, password, pillagedfrom) VALUES (?,?,?,?,?)", [credtype, domain, username, password, pillaged_from] )
OperationalError: table credentials has no column named pillagedfrom
Command string used
crackmapexec <IP_ADDRESS>/24 -d test -u test -p password1 -M mimikatz
OS
Linux kali 4.7.0-kali1-amd64 #1 SMP Debian 4.7.6-1kali1 (2016-10-17) x86_64 GNU/Linux
Detailed issue explanation
Seems to be database setup error; cme installed with both methods, same error.

@9andrea1 you seem to be using the bleeding edge version from github, bugs are expected since it is not stable.
I'd suggest using the stable version which you can install by running pip install crackmapexec
Cheers

@byt3bl33d3r it was installed running pip, anyway it works properly now in new kali Linux kali 4.8.0-kali1-amd64 #1 SMP Debian 4.8.5-1kali1 (2016-11-04) x86_64 GNU/Linux
thanks

Just delete the .cme directory from directory where you were running crackmapexec, if you don't need previous info obtained.


<IP_ADDRESS> IBM Java SDK for MACOSX?

https://ibm.github.io/spm-kubernetes/01-prereq/3rdparty-sw states to install <IP_ADDRESS> IBM Java SDK - however following the link there is no version for Mac? Thanks

Mac isn't an officially supported platform for SPM development and/or deployment. The middleware part of the setup must be performed on Linux or Windows.
The Kubernetes/Docker part of the setup can be done on any of the platforms supporting installation of Docker/Minikube.

Since there is no recent update to the ticket I'm going to close it. Feel free to reopen.


Build flags to specify extra import paths

Hi,
I'm using the crystal-linter plugin for atom. When I'm editing a file with imports only from standard library it works fine but it seems it does not work when I'm importing something from a shard.
I think the compiler is looking for .shards directory in the current workdir. Is this correct ?
Are there any build flags to specify extra import paths ? Something like:
crystal build --import-path=/path/to/my-project/.shards file.cr

The default search path (env variable CRYSTAL_PATH) is set to src, libs and the standard library. I guess crystal-linter should invoke crystal build with a similar CRYSTAL_PATH. I guess this is an issue to report to crystal-linter.

Thank you @asterite
I asked on the crystal-linter issue tracker and the author does not have a proper solution, as well. I asked here about the compile flags so I can point it here in the original issue.
I think we can close this issue now.


Fixes #1064. Fixed error related to NewSeg logic

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Trivial change (affects only documentation or cleanup)

Checklist:

[x] I have tested this change with a run of GEOSgcm (if non-trivial)
[x] I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
[x] I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

We're using 2.6.3. Is this an easy patch on 2.6?
(FYI we're planning to upgrade to the latest MAPL in GCHP 13.4 or 13.5 -- soonish, but far enough away that it would be ideal if we didn't have the timestamp bug unitl then)

@LiamBindle Well, the next in the 2.6 would be 2.6.10, but I think it should be possible. Let me see...

Thanks @mathomp4 - testing it now


Fixes #1505

Fixes #1505

got referred to this PR by @mraocuahsi
tested locally and changes look good to me


drogon_ctl create model does not generate updateId (sqlite3)

Describe the bug
Not sure if it is a bug or not, but seems like it. Creating model does not create it's primary key updater function in sqlite3 on Windows 10
To Reproduce
Steps to reproduce the behavior:

sqlite3 C:/path/to/db.sqlite3
CREATE TABLE A (id INTEGER PRIMARY KEY, val TEXT NOT NULL); (I use backticks for names for some reason)
drogon_ctl.exe create model test\models

We have following output:
Create model
Connect...
Source files in the test\models folder will be overwritten, continue(y/n)?
yes
table name:A

open A.cc

void A::updateId(const uint64_t id)
{
}

Expected behavior
void A::updateId(const uint64_t id)
{
    id_ = std::make_shared<uint64_t>(id);
}

Desktop (please complete the following information):

OS: Windows 10

Additional context
model.json:
  "rdbms": "sqlite3",
  "filename": "db.sqlite3",
  "host": "<IP_ADDRESS>",
  "port": 5432,
  "dbname": "",
  "schema": "public",
  "user": "",
  "password": "",

config.json:
  "db_clients": [
    {
      "rdbms": "sqlite3",
      "filename": "../../../db.sqlite3",
      "host": "<IP_ADDRESS>",
      "port": 5432,
      "dbname": "",
      "user": "",
      "passwd": "",
      "is_fast": false,
      "client_encoding": "UTF-8",
      "connection_number": 1
    }
  ],

drogon_ctl -v:
Version: 1.3.0
Git commit:
Compilation:
  Compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.28.29333/bin/Hostx64/x86/cl.exe
  Compiler ID: MSVC
  Compilation flags: /MD /O2 /Oi /Gy /DNDEBUG /Z7  -std=c++17 -IC:/Users/User/source/vcpkg/installed/x86-windows/include -IC:/Users/Petr/source/vcpkg/packages/drogon_x86-windows/include
Libraries:
  postgresql: no  (batch mode: no)
  mariadb: yes
  sqlite3: yes
  openssl: yes
  brotli: yes
  boost: no
  c-ares: yes

@teplinsky-maxim Hi, this method is an internal method for tables whoes primary key is automatic type, when a new record insert into the table, orm::Mapper use the method to set the id. in your case, the primary key is not automatic, so this method is empty.
Please create table as follows and try again.
CREATE TABLE A (id INTEGER PRIMARY KEY AUTOINCREMENT, val TEXT NOT NULL);

BTW: if you need a method to modify the id, please use the public setId(...) method.

Yep, you are right! Thanks for the explanation

@an-tao Hi! This is a feature of SQLite -  SQLite Autoincrement:

INTEGER PRIMARY KEY is an alias for the ROWID

so in this case, the primary key is actually automatic

@Vinges541 Thanks for the info. I'll make a patch for this.


Inconsistent deserialization-serialization with root structs

When deserializing a struct that we got by serializing a toml, it might not include global fields, depending on the way they were described in the original toml.
Given toml:
[machine]
    name = "Sample text"

    [[activity]]
        name = "TOML FTW"
        property = 1

and definitions in rust:
#[derive(Debug, Serialize, Deserialize)]
pub struct Substruct {
    name: Option<String>,
    property: Option<i8>,
}

#[derive(Debug, Serialize, Deserialize)]
pub struct Machine {
    name: Option<String>,
    activity: Option<Vec<Substruct>>
}

I get an instance of Machine using toml::from_str and then serializing it back using toml::to_string (or toml::to_string_pretty).
Got result with missing Machine fields:
"[[activity]]\nname = \"TOML FTW\"\nproperty = 1\n"

Expected result (with Machine fields present):
"name = \"Sample text\"\n\n[[activity]]\nname = \"TOML FTW\"\nproperty = 1\n"

Workaround
It works as expected if I drop [machine] definition in source toml.
Link to playground

I think it's an interesting issue specifically because modifying source toml fixes deserialization. No idea why, though

(Please let me know if I misunderstand anything. I’m new to Rust and to TOML.)
It looks like that example is confusing the [machine] table and the root table of the TOML document.
To make sure we’re on the same page, here is my understanding of TOML structure.
# This `name` key is in the root table (root node) of the
# TOML document.
name = "foo"

# This TOML document has a single `machine` element in its root table.
# That element’s key is `machine` and its value is a table that contains a
# `name` element.
[machine]
name = "foo"

# Here we have both names.
name = "foo"

[machine]
name = "bar"

# This is equivalent to the previous snippet.
name = "foo"
machine.name = "bar"

Consider this modified code. Does that make sense?
In the original playground example, the code was expecting a name element in the root table but didn’t find any. The code was also ignoring the [machine] table entirely during parsing (the Machine struct actually corresponded to the root table, not the [machine] table). So the whole [machine] table couldn’t appear in the output later when the internal data structure was serialized.

Maintenance of this crate has moved to the https://github.com/toml-rs/toml repo.  As a heads up, we plan to move toml to be on top of toml_edit, see https://github.com/toml-rs/toml/issues/340.
Closing this out.  If this is still a problem, feel free to recreate this issue in the new repo.


html character encoding being applied to text only output s()

the s() function is defined as being a text only output, however the output is being put through html encoding, even though the variable type has the correct string length.
$testString = 'Hello & Goodbye';

\s($testTring)

Output is
string (15) "Hello & Goodbye

should be string (15) "Hello & Goodbye"

Hey, if you're using the wip branch:
https://github.com/raveren/kint/tree/1.0.0-wip
there are four different output types. There's the default rich output, and the text-only one - however the latter has three flavors. Kint autodetects whether you're running in HTML or CLI environment and furthermore if you're in CLI - does your console support colors.
For HTML the output is applied minimal formatting (similar to xdebug) and html-escaping. CLI output is purely whitespace formatted - with minor escaping of command line escape codes.
And finally - and this is what you want - is pure whitespace output. No formatting, no escaping. You can only invoke it manually in one of two ways (as s() applies detection too).
First, you can use a new modifier ~: ~\Kint::dump($testTring) it will change the current output - or just change all subsequent Kint outputs to be whitespace:
		\Kint::enabled( Kint::MODE_WHITESPACE );

		$testString = 'Hello & Goodbye';

		\dd($testTring);

Sorry for not documenting it properly elsewhere, I'll get to it eventually :)

thanks for the update. For reference, I'm using the v0.9.1 via composer.
You said that s() should auto-detect whether it's running in HTML or CLI. I'm running it via CLI, but it's still outputting the above even through CLI. My project is based on the Zend Framework 2 if that makes any difference.

the .9.1 is really old, I need to switch branches, can you switch to the wip version meanwhile? It has all the bells and whistles I talked about.

hm... I switched to dev-1.0.0-wip via composer and now Kint is causing fatal errors.
Warning: reset() expects parameter 1 to be array, null given in /var/www/tantornet/vendor/zendframework/zendframework/library/Zend/Db/ResultSet/AbstractResultSet.php on line 238

Warning: key() expects parameter 1 to be array, null given in /var/www/tantornet/vendor/zendframework/zendframework/library/Zend/Db/ResultSet/AbstractResultSet.php on line 222

Warning: reset() expects parameter 1 to be array, null given in /var/www/tantornet/vendor/zendframework/zendframework/library/Zend/Db/ResultSet/AbstractResultSet.php on line 238

Warning: key() expects parameter 1 to be array, null given in /var/www/tantornet/vendor/zendframework/zendframework/library/Zend/Db/ResultSet/AbstractResultSet.php on line 222

Warning: reset() expects parameter 1 to be array, null given in /var/www/tantornet/vendor/zendframework/zendframework/library/Zend/Db/ResultSet/AbstractResultSet.php on line 238

Warning: key() expects parameter 1 to be array, null given in /var/www/tantornet/vendor/zendframework/zendframework/library/Zend/Db/ResultSet/AbstractResultSet.php on line 222

Fatal error: Call to a member function fetch() on null in /var/www/tantornet/vendor/zendframework/zendframework/library/Zend/Db/Adapter/Driver/Pdo/Result.php on line 158

woah, this is weird - and I don't see how's it related to kint - can you verify it's Kint's fault? Could you switch back to .9.1 and see if the problems persist?

yup. All I do is switch composer from v0.9.1 to dev-1.0.0-wip and I get fatal errors. If I comment out my single \Kint::dump() statement while on dev-1.0.0-wip, everything works fine again, so unfortunately looks like Kint having issues with something in my model that I'm trying to dump.
I can help diagnose / compare later today or tomorrow if that will help.

are you dumping a Zend object? Are you using Kint shorthand that halts execution (sd,dd,ddd)? It seems a query was run by Zend but it did not connect properly to database...

I'm dumping an Object that has a reference to Zend PDO object yes, and just using straight \Kint::dump(), nothing fancy. Only happens when I run the \Kint::dump() function.

could you please provide the code you are calling having removed any personal details, I'll install ZF2 and try your code.
Thanks for your help!

I don't need any databases you're using, I'll just mock my own, I just need the general code to go on - thanks.

I'm closing this for inactivity.


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v4.12 to before v5.12.10",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=1769152ac64b0b07583f696b621624df2ca4c840",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=5a5357cd2cd9a38944594eb5cd74377932e176bf"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "1769152ac64b0b07583f696b621624df2ca4c840",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "5a5357cd2cd9a38944594eb5cd74377932e176bf",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "amdgpu: fix GEM obj leak in amdgpu_display_user_framebuffer_create\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.12.10 by commit 5a5357cd2cd9a38944594eb5cd74377932e176bf, it was introduced in version v4.12 by commit 1769152ac64b0b07583f696b621624df2ca4c840. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1000803


Set-CsOnlineDialInConferencingUser Can't Modify Conference ID in manual mode with " -ConferenceId 8271964" Parameter

Set-CsOnlineDialInConferencingUser
[Can't Modify Conference ID in manual mode with " -ConferenceId 8271964" Parameter]
When i modified conference ID in manual mode , conference ID stay at '0' value .
Have you an idea for this ?
Thanks
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d1346667-4316-343c-df7e-62bd41182d0a
Version Independent ID: 626fd6f0-1a91-cae2-b5ca-083688a2ce39
Content: Set-CsOnlineDialInConferencingUser (SkypeForBusiness)
Content Source: skype/skype-ps/skype/Set-CsOnlineDialInConferencingUser.md
Service: skypeforbusiness-powershell
GitHub Login: @tomkau
Microsoft Alias: tomkau

@platinarock - thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome, let me know if it can be called out in the docs.
Please follow this link to contact support for business products:  https://docs.microsoft.com/office365/admin/contact-support-for-business-products.
Please keep us posted here on the resolution to feed whatever information you discover into the content.

@dariomws  Thank you very much for the contribution and sharing this explanation. @platinarock Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. We proceed to close it as here we are limited to documentation fixes and improvements only.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also


stage 0.18.2

We're currently processing your upload.  This comment will be updated when the results are available.


docs: update theme 1.8.3

Related issue https://github.com/scylladb/sphinx-scylladb-theme/issues/1229
ScyllaDB Sphinx Theme 1.8.3 is out 🥳!

This release addresses some issues introduced in version 1.8. The key updates include:

Dark theme settings are now preserved between projects.
Tooltip functionality now supports glossary entries.
Dependabot configuration was edited to allow theme-related packages updates.

You can read more about all the notable changes here (1.8.2 and 1.8.3)

cc/ @annastuchlik


[rancher-dns] environment names with space cause dns update requests to fail

Rancher Version:

1.1.4

Docker Version:

1.12.1

OS and where are the hosts located? (cloud, bare metal, etc):

centOS 7, virtualized (ESXi), hosts are on-premise

Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)

6 node rancher, internal DB

Environment Type: (Cattle/Kubernetes/Swarm/Mesos)

Cattle only

Steps to Reproduce:
docker-compose.yml for dnsupdate-rfc2136
rfc2136dns:
  environment:
    RFC2136_HOST: <IP_ADDRESS>
    RFC2136_PORT: '53'
    RFC2136_TSIG_KEYNAME: ielab.local.
    RFC2136_TSIG_SECRET: **redacted**
    ROOT_DOMAIN: ielab.local
    TTL: '299'
  labels:
    io.rancher.container.create_agent: 'true'
    io.rancher.container.agent.role: external-dns
  command:
  - -provider
  - rfc2136
  image: rancher/external-dns:v0.5.0

rancher-compose.yml for dnsupdate-rfc2136
rfc2136dns:
  scale: 1
  health_check:
    port: 1000
    interval: 2000
    unhealthy_threshold: 3
    strategy: recreate
    response_timeout: 2000
    request_line: GET / HTTP/1.0
    healthy_threshold: 2

For purposes of testing, I used IE Lab as my environment name.
rancher-dns service appeared to be up OK, so I went to the catalog in Rancher and provision a load balancer to my jenkins-ci stack to test the update. Example is shown below.
Results:
10/17/2016 4:52:00 PMtime="2016-10-17T20:52:00Z" level=info msg="Adding dns record: {jenkinslb.jenkins.ie lab.ielab.local. [<IP_ADDRESS>] A 299}"
10/17/2016 4:52:00 PMtime="2016-10-17T20:52:00Z" level=error msg="Failed to add DNS record to provider {jenkinslb.jenkins.ie lab.ielab.local. [<IP_ADDRESS>] A 299}: Failed to build RR: dns: not a TTL: \"lab.ielab.local.\" at line: 1:38"

What I believe happens is that external-dns creates a record based off this spec -
<service_name>.<stack_name>.<environment_name>.<root_domain> {record set}

Since there was a space in the <environment_name> variable, it led to a parsing error in the RR.
Expected:
When I changed my environment name to westeros, the update request worked.
edit: clear up reproduction steps

Duplicate of #3972


QR Code Generation Script

Changes made:

Made the logo path a parameter with a default value
Combined the logo resizing logic to improve readability
Used os.path.join to create the logo path.

Did you try running it? Seems like os.path.join part is removed and the readability doesn't seem to be effected that much with the changes

btw, the solana pay docs itself doesn't allow changing the logo path since it represents it is made by Solana Pay.

Did you try running it? Seems like os.path.join part is removed and the readability doesn't seem to be effected that much with the changes

it must be fixed now

@GitBolt it should be working fine now

Well, there were plenty of commits there just to have the changes introduced to be reversed back. The formatting is a very minor change left only. Merging won't make sense here.


first notebook for distributed rock, paper and scissors

Why are these changes needed?
A sample to help users use distributed runtime
Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

@ekzhu  can you please let me know what do you think about this simple implementation? If you think it's ok, I'll add the documentation and check for it's compliance and make the PR ready.

I think looks good. We can improve readability by splitting the agents into their own cells. Also having just one type of player agent (RandomPlayer) should be enough.


Removed the call to conn.maintain()

since it makes it impossible to connect with any user except admin

Thank you.


fix(config): avoid rewriting the config file if it's unnecessary

The server start will rewrite the config file to persist namespace/token pairs
if the namespace replication is enabled. But it's unnecessary if the namespace replication
is disabled or no tokens were loaded from the configuration file because the purpose of
this rewrite is to remove tokens from the config file and write to the Database.

It can fix the problem when there's no namespace in kvrocks except the default one.
But what if there are some namespaces? We also need to consider this situation

But what if there are some namespaces? We also need to consider this situation.

Rewrite() will persist them if there are any namespaces(except the default), and then remove them from the config file. So it should be good for this scenario.


Re-enable the T and C linter prefix selectors

Summary
In one of the prior refactors around lint rule prefixes, we removed the T and C selectors.
To take T as an example, the issue is that we have two "linters" that both use the T prefix -- flake8-debugger, whose rules are (e.g.) T100, and flake8-print, whose rules are (e.g.) T201. In the refactored model, flake8-debugger was represented as having a prefix of T10, and a rule under that prefix with code 0, which combines to an overall selector of T100.
Under this scheme, if the user provides the T prefix, we don't know where to map that code. Prior to this PR, it was redirected to T10, so if a user provided --select T, that aliased to --select T10, which enabled rule T100, thus ignoring the flake8-print rules entirely. I believe this was unintentional.
The same problem exists with the C prefix, where we have both the McCabe complexity and flake8-comprehensions rules. The McCabe rules no longer run under --select C, because it was redirected to C40.
This PR special-cases those two prefixes, introducing custom selectors for T and C to re-enable the expected user behavior.
Closes #3052.

There are likely further refactors we can do here, but I'd like to get this out in the next release as it's probably surprising to users that certain codes are no longer enforced despite no change in their selectors (e.g., select = ["C"] no longer runs the complexity checks).


about the using way of ‘ SAGpooling’

I encountered the following error report when I was using this model.

edge_index needs to be a torch.LongTensor of shape [2, num_edges] instead of a scipy sparse matrix.

Thanks for your help.
But I encounter one problem again, such next.
I have completed the graph pooling layer with your help.
Then I put it to pass some  graph convolution layers , but I can't unsample the new graph to the original graph.
So,  how to unsample the new graph in order that we can make it back to the original graph?

You can upsample your node features by making use of the returned perm vector, e.g., by filling the blanks with zero vectors:
x_downsampled, edge_index_downsampled, _, _, perm, _ = self.pool(...)

x = torch.zeros(num_orig_nodes, x_downsampled.size(-1))
x[perm] = x_downsampled

Thank you very much！
The another question is that.
What is this perm vector supposed to be, I don't quite understand.

The perm vector is a index tensor, which contains the node indices of the nodes that are kept after pooling.

Thank you very much ！！！！！！！Great help！！！！！


Operations Api shows RuntimeId, ProcessId, and ProcessName

This is for #2122. It updates the /operations and /operations/operationId api to show the RuntimeId, ProcessId, and ProcessName as a sub-object shown in the results.
Main question I wanted to bring up for discussion in this PR:
Most EgressOperations created can't really take in an IProcessInfo object since they pull an endpoint directly, which doesn't have the ProcessName stored. There are a few ways I'm thinking we can use to fix this -

Leave as-is and let most operations not have a processName that we can use.
If the processName isn't that relevant we can remove it from the API update so we only show RuntimeId and ProcessId.
We can query for the processInfo object using the found RuntimeId and get the name from that, similar to what's done in DiagController.GetProcessInfo(). Not sure how much I like this.
Update IEndpointInfo to have the processName inside it. Not sure how doable it is but I'm investigating.

Let me know what you think would be best, I'm not entirely sure myself but I'm leaning towards the 4th option, if possible.

/backport to release/6.x


Changing NamedUser Identifier Doesn't Call API

We are associating NamedUser with an internal identifier for targeting specific users, however, once we've set a NamedUser identifier it appears to be unchangeable until the app is closed and reopened. We would like to set this at log in, and swap it out with another user should they switch accounts.
Furthermore, messages sent to a NamedUser, once on the device, don't appear to have a way of validating that it is the NamedUser fetching said messages. I don't see any info on the message either indicating that it's a NamedUser and to which NamedUser it should go. This means if a user gets messages, logs out, logs into another account, they will see the messages for the first NamedUser

@benfrye Its fully expected for the named user to change mid run. Could you provide logs of your app changing the named user?
For the inbox being tied to the named user, thats currently not a supported feature. Writing into support requesting that feature is your best bet to get it prioritized. For a workaround you could add the named user as an extra for the message, then filter out the users messages on the device.


Open Question: Schema for geneset objects

This is for discussing the general structure of the schema and naming convention of the fields. As we add more data sources, we may find that we need to modify certain aspects of the model.
This is what what my current model looks like in YAML-like format:
_id: string
  is_public: boolean
  author: string       # Only for user-created datasets
  date: string         # Only for user-created datasets
  taxid: integer
  genes:
    - mygene_id: string      # mygene.info primary key
      uniprot: string or list         # Can be empty
      ncbigene: string or list        # Can be empty
      ensemblgene: string or list     # Can be empty
      symbol: string
      name: string                    # Can be empty
  [source_name]:           # Only for public genesets
    id: string             # Usually same as _id, unless _id contains id+taxid
    [source_specific_data]

For example, for a GO geneset:
{
  "_id": "GO:0007288_9606",
  "is_public": true,
  "taxid": "9606",
  "genes": [
    {
      "mygene_id": "286207"
      "uniprot": "Q5JU67",
      "symbol": "CFAP157",
      "ncbigene": "286207"
      "ensemblgene": "ENSG00000160401"
      "name": "cilia and flagella associated protein 157"
    },
    {
      "mygene_id": "79846"
      "uniprot": "A5D8W1",
      "symbol": "CFAP69",
      "ncbigene": "79846"
      "ensemblgene": "ENSG00000105792"
      "name": "cilia and flagella associated protein 69"
    },
  ],
  "go": {
    "id": "GO:0007288",
    "name": "sperm axoneme assembly",
    "type": "biological_process",
    "description": "The assembly and organization of the sperm flagellar axoneme, the bundle of microtubules and associated proteins that forms the core of the eukaryotic sperm flagellum, and is responsible for movement. [GOC:bf, GOC:cilia, ISBN:0198547684]"
  }
}

Question: Should we produce Uniprot IDs for datasets that do not specify them? (e.g. GO data has Uniprot IDs, but wikipathways comes with NCBI ids.)
Mapping from UniProt to gene is often 1:1 , but the reverse is usually 1:many. Furthermore, UniProt often comes with two sets of IDs: SwissProt and TrEMBL: https://www.uniprot.org/help/uniprotkb_sections

@ravila4: One question on source_name: so this field's name will vary depending on the exact data source? I was thinking of something like this:
source:
    id: str
    name: str
    description: str
    ...

}
...

@dongbohu  If we use the same field name, then all its sub-fields would be merged during the build process. I think keeping them separate allows us to ask something like: "Give me all the public genesets that come from GO".

@ravila4 Thank you for your reply. I am not familiar with the ElasticSearch-based API. What will be the syntax to get all GO public genesets? Something like .../query?go=*?

@dongbohu I believe it would be: .../query?q=go*

Good to know. Thanks!

@dongbohu  Correction: It would be /query?q=_exists_:go Learning about this too...
https://www.elastic.co/guide/en/elasticsearch/reference/current/query-dsl-query-string-query.html#query-string-syntax

💯  Thanks again!

@ravila4: I have been thinking of a better way to deal with the genes that are not found by mygene.info. A missed gene will look like this in the current schema:
...
genes: [
  {
    "mygene_id": null
    "uniprot": null
    "symbol": null,
    "ncbigene": null
    "ensemblgene": null
    "name": null
  },
  ...
]

which is a waste of bandwidth (and boring). Can we change it into:
...
genes: [
  {
    "source": <original_gene_name_in_data_source>
    "mygeneinfo": null
  },
  ...
]

in which <original_gene_name_in_data_source> is the original gene ID in data source (which is the query string that we use to search in mygene.info).
And a gene that is found in mygene.info would look like this:
"genes": [
  {
    "source": <original_gene_name_in_data_source>
    "mygeneinfo": {
      "_id": "286207"
      "uniprot": "Q5JU67",
      "symbol": "CFAP157",
      "ncbigene": "286207"
      "ensemblgene": "ENSG00000160401"
      "name": "cilia and flagella associated protein 157"
    },
  },
 ...
]

@dongbohu  I can see value in keeping the original source tag, for example KEGG's 'locus_tag', which is not in the current schema... but we also want to standardize the fields returned by the "mygeneinfo" object, otherwise the mapping will include all possible identifiers supported by mygene.info...
I think perhaps something like:
"genes": [
   {
     "source": {
       "identifier": <original_id_name e.g. 'locus_tag'>
       "value": <original value>
     }
    # MyGene.info ids
    "_id": "286207"
    "uniprot": "Q5JU67",
    "symbol": "CFAP157",
    "ncbigene": "286207"
    "ensemblgene": "ENSG00000160401"
    "name": "cilia and flagella associated protein 157"
  },
...
]

Also, we normally don't store any null values in the data. (You can use biothings.utils.dataload.dict_sweep() to drop them).
If we don't find a gene in mygene.info, we should simply exclude the fields, but perhaps add a tag? There should be an easy, on-the-fly way to filter out genes that were not found.
"source": {
       "identifier": <original_id_name e.g. 'locus_tag'>
       "value": <original value>
       "not_found": True
     }

Additionally, if we find conflicting information between mygene.info and the source, we should also correct it. For example, if a source NCBI id is retired or deprecated, we may add a flag:
"genes": [
   {
     "source": {
       "identifier": "ncbigene",
       "value": "100128403".
       "retired": "True"
     }
    # MyGene.info ids
    "_id": "9899"
    "uniprot": "Q7L1I2",
    "symbol": "SV2B",
    "ncbigene": "9899"
    "ensemblgene": "ENSG00000160401"
    "name": "synaptic vesicle glycoprotein 2B"
  },
...
]

Let me know what you think.

Tagging @newgene for input.

@ravila4 Thank you for your feedback. What you said makes sense. I got another idea: Since mygene.info is the domain where we did the query, we can make the field names more mygene-friendly. So for a retired gene, the fields would be:
"genes": [
  {
    "source": "9899",
    "query_scope": "retired",
    "_id": "9899"
    "uniprot": "Q7L1I2",
    "symbol": "SV2B",
    "ncbigene": "9899"
    "ensemblgene": "ENSG00000160401"
    "name": "synaptic vesicle glycoprotein 2B"
  },
  ...
]

And for missed genes, it could be:
"genes": [
  {
    "source": "foobar",
    "query_scope": "entrezgene, retired, symbol, locus_tag",
    "missed": true
  },
  ...
]

So query_scope field will tell us exactly which field in mygene.info matches the gene in data source. (I changed not_found to missed because that is the term that mygene.info uses. I also thought of using scopes instead of query_scope, but the word scopes seems a little too ambiguous.)

@dongbohu  I created a module that can be used for creating gene objects: https://github.com/ravila4/mygeneset.info/blob/master/src/utils/geneset_utils.py. My hope is that it would be an easy way to maintain all data plugins standardized and up to date. It is pending merge for need of more testing. Any  improvements are welcome. Do you think t it would be easy to incorporate it to your KEGG parser?
You can see an example in the Wikipathways parser: https://github.com/ravila4/wikipathways/blob/master/parser.py
I think for simplicity, missed genes should be simply be omitted. I haven't yet found any instance where an entrez id that  is missed and is not deprecated...


Remove scope=row attribute from non-th elements

In HTML5 scope attribute should only be used on <th> elements according to https://dequeuniversity.com/rules/axe/1.1/scope
This PR removes the wrongly placed scope attribute on <td>s
Fixes #526

will need to be updated with #528


⚠ Bofrost.fr has degraded performance

In 8738bf6, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1113 ms

Resolved: Bofrost.fr performance has improved in f629050.


Including or-tools in an existing C# project

Hi, I have an existing C# project in Visual Studio 2017. I'd like to include OR Tools in the project. I can't figure out how to include it as a dependency in my project.
I've looked at the installation guide here and was able to validate the code. However, there are no instructions on how to use the library from outside of the examples directory. https://developers.google.com/optimization/install/dotnet/windows
I've also tried to install OR Tools using the NuGet Package Manager in Visual Studio. The package successfully installs in my project. However, VS cannot find the package when I include using Google.OrTools.LinearSolver; at the top of a file.
Am I missing something basic?

Read the comments on this closed issue
https://github.com/google/or-tools/issues/900

@stuarthillary Thanks. Your suggestion here https://github.com/google/or-tools/issues/900#issuecomment-432944651 did the trick.


Fix wrong bound selection in histogram last bucket

Fixes:
https://github.com/CartoDB/support/issues/1871
Useful comments found in an older closed issue:
https://github.com/CartoDB/carto.js/issues/926

Let me know if you want me to check in Builder and give you the acceptance.

Let's see first is @VictorVelarde can have a look and make a CR and then yes, it will be very helpful if you could test it in Builder!

The fix LGTM @csubira, and it seems like there is no 'one duplicated bucket issue', so I think we can move on. I have just added 2 specs and made a little refactor to an if, and that's it.
Feel free to merge it

@csobier @VictorVelarde next time ask me for checking the change on staging!

I will Ramiro. Indeed I'll ask you for acceptance after merging another carto.js PR and releasing a new version


[Enhancement]: Rock Paper Scissors game

Do you want to have the enhancement of existing game ? 😀 Describe yourself..
UI can be better
problems in current game-
small logo for Rock Paper and Scissors
automatically selecting the choice
buttons are not proper due to color
bad front end
Describe the solution you'd like
https://dnyandeepchute.github.io/rockPaperScissorsGame/
you can refer this
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @dnyandeepchute , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#4236

Hey @dnyandeepchute ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey @dnyandeepchute !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

hey @kunjgit , please assign this to me, thank you.

/assign


centos image doesn't have ip command?

waaaat? that's a bummer. update image to a different one, or make sure it has ip command installed.

centos seems not to have 'ip' command, so you can install it by 'yum install -y iproute'.

Thanks for the pointer Tomo, appreciate it! Confused me that the centos:centos7 base image didn't have ip


Implement the new artifacts hashing algorithm

Also please update the PR description with more details on why this is happening.


updated for release of 2.1.1

Coverage increased (+67.5%) to 97.555% when pulling ffc5980c575d4e33d15facb3a1b564846d870bcb on release2.1.1 into eef3d3542992eb01741ac746c205bf759ec3ac96 on master.


Update README.md

added (follow Security settings running macros section to unblock file) to line 9 as per Customer/partner feedback

readme.md update


Support for ZSP-001

Hi,
I'm trying to use tuya-convert for a ZSP-001 socket. Since it is listed here
https://templates.blakadder.com/ZSP-001.html
I expect that it is possible. After unsuccessfully using tuya-convert I tried to flash it via serial but without any success. I soldered a serial connection and I'm getting logs:
< TUYA IOT SDK V:2.0.0 BS:30.01_PT:2.2_LAN:3.3_CAD:1.0.1_CD:1.0.0 >
[FW]: TC0084_TW03_PLUG_DLTJL2_P0 | [HW]: TW03 | [SW]: V1.4.2 (Nov 11 2019 11:55:38) | [PID]: tpfykupqyhsvhaqv
[01-01 00:00:00 TUYA Notice][hw_table.c:352] wifi status is :0
[01-01 00:00:00 TUYA Notice][wifi_hwl.c:1316]

I've checked the pinout of the wifi module and it seems to be an esp8266 but didn't open the can. All GPIO pins are correctly set but I can't enter flash mode (I've used this setup for other devices).
In smarthack-udp.log I find:
<IP_ADDRESS> {"ip":"<IP_ADDRESS>","gwId":"bf4a9c26922c28b30dkehp","active":0,"ablilty":0,"encrypt":true,"productKey":"tpfykupqyhsvhaqv","version":"3.3"}
WARNING: it appears this device does not use an ESP82xx and therefore cannot install ESP based firmware

The device loops at
[01-01 00:00:23 TUYA Err][iot_httpc.c:820] Post Data Return Fail.-1
[01-01 00:00:23 TUYA Err][iot_httpc.c:2045] httpc_gw_get_urlcfg_v41/httpc_gw_region_cfg_get_v10 error:-1
[01-01 00:00:25 TUYA Err][iot_httpc.c:820] Post Data Return Fail.-1
[01-01 00:00:25 TUYA Err][iot_httpc.c:2045] httpc_gw_get_urlcfg_v41/httpc_gw_region_cfg_get_v10 error:-1
[01-01 00:00:26 TUYA Err][iot_httpc.c:820] Post Data Return Fail.-1
[01-01 00:00:26 TUYA Err][iot_httpc.c:2045] httpc_gw_get_urlcfg_v41/httpc_gw_region_cfg_get_v10 error:-1
[01-01 00:01:14 TUYA Notice][gw_intf.c:4576] gw is long time(60s) disconnected to router/failed to connect mqtt,now go to reconnect!!
[01-01 00:01:14 TUYA Notice][gw_intf.c:4577] netstat:6
[01-01 00:01:14 TUYA Notice][hw_table.c:352] wifi status is :5
[01-01 00:01:15 TUYA Notice][hw_table.c:352] wifi status is :6
[01-01 00:02:15 TUYA Notice][gw_intf.c:4576] gw is long time(60s) disconnected to router/failed to connect mqtt,now go to reconnect!!

The registration server receives:
POST /d.json?a=tuya.device.config.get&et=1&other={"token":"00000000","region":"DE"}&t=20&uuid=c57ec9d8ecf21961&v=4.1&sign=79469f32ae29d7647d0529dcf887ed83
Host: a2.tuyaus.com
User-Agent: TUYA_IOT_SDK
Connection: keep-alive
Content-Type: application/x-www-form-urlencoded; charset=UTF-8
Content-Length: 37

payload 418D7F28A3DACB7E2EBD7783F15CC722

I've tried to play with the region settings in scripts/smartconfig/main.py but no success
Is this device ESP8266 based or is it just a waste of money?
Thanks Leif

Unfortunately the warning is accurate, this is not an ESP82xx which is also why you were unable to flash it with ESP firmware over serial.


IOS: entitlements of main target are overwritten

In your latest change "CODE_SIGN_ENTITLEMENTS" are set in the project for every build target of the project.
This is fine for the ShareExt target, but not for the other targets in the project.
It basically overwrties all previously stored settings for those targets
(for example push notification entitlements or associated domains)
It should inject the ShareExt app group setting into the entitlements and leave other settings intact.

I think it comes from running iosAddTarget.js on the after_prepare hook.
By fixing the problem you noticed here: https://github.com/j3k0/cordova-plugin-openwith/issues/95 I revert to the after_plugin_install hooks (with some other fixes).
Before PR, could you test my branch:
"dependencies": {
    "cc.fovea.cordova.openwith": "git+https://github.com/BenjaminPoncet/cordova-plugin-openwith.git#ios-fix-shareext-install"
  },

I hope this fixes both your problems at the same time!

@BenjaminPoncet i tested your branch, now no more errors in cordova platform add ios...
but..
after the command the project file  does not contain  the share extension. I think cordava modifies the project file in the prepare step and removes it.
during this breakpoint the project file does contain the share extension target..

at this breakpoint the project file the share extension target is gone

My bad! I checked that all hooks were well executed in all cases but I did not check again that the ShareExtention was present in the xcode project.
Thanks for the input, I think I'm very close.

@dickverweij, Got it!
Everything should work out in your case.
I also rewrote the addition of the entitlment to avoid overwriting.
It's all on my branch.

almost @BenjaminPoncet .. it is now without erors. but.. there was a reason you did a "after_prepare" step in the first place.
in the prepare step cordova overwrites the PRODUCT_BUNDLE_INDENTIFIER of the ShareExt target .. :-(

Really!? Good point! Cordova really doesn't like multi-target projects...
In a first one, I had switched to the prepare hooks but not especially for the PRODUCT_BUNDLE_INDENTIFIER but simply by thinking that those are the hooks that cover more usage.
By reworking the hooks, I tried to be less global and closer to the uses.
I just rebase my branch to keep the after_prepare hook!
However, during my tests, I noticed that by executing the cordova prepare command without ios, cordova rewrites the PRODUCT_BUNDLE_INDENTIFIER but doesn't execute the hooks specific to ios...
I have the feeling that this is a bug and that it would be enough to trigger the iosAddTarget.js (or a new hook) globally but well, it's starting to annoy me!
Any advice?

Advice ?  :-)
The cordava build system (next to the NPM package hell) is really a beast on its own. Now that we automated we have to be sure wat is going on under the hood. If a plugin fails it can fail complete build afterwards. Certain hooks can mess things up you just correct in another.
You're plugin also creates a multi target xcode project.. yep Cordova really does not like multi target stuff.. :-)

@dickverweij can you please share the patch that you did that worked for you?


Revert change to Transfer Acceleration from #178

what

Revert change to Transfer Acceleration from #178

why

Transfer Acceleration is not available in every region, and the change in #178 (meant to detect and correct drift) does not work (throws API errors) in regions where Transfer Acceleration is not supported

/test all


Correct install script for Ansible 2.8+

This PR corrects the install script for Ansible versions 2.8+, as the Netscaler module was removed in 2.8 (release notes).
The PR also clarifies the doc for installing using virtualenv and corrects a reference to the old repo name.
Resolves https://github.com/citrix/citrix-adc-ansible-modules/issues/117.

Fixed with #145


Feedback

Type of issue
Other (describe below)
Feedback
xv8yn;-mr2ct--PG88XQ-YCW72_MP2KM

No issue described


How should the loader impact

Forking from https://github.com/whatwg/html/pull/443#issuecomment-169161637 and the next two comments

Can you say more? Why?

src has an established meaning for web developers, having to do with URLs, not module specifiers. Breaking that mental model is not really acceptable. For example, <script src="jquery"></script> fetches http://example.com/base/url/jquery; <img src="jquery"> fetches http://example.com/base/url/jquery; and so <script type="module" src="jquery"> must also. But see below, maybe I didn't quite understand what you meant...

More concretely, an inline module (no src attribute) would go through the loader starting at the translate hook; an out-of-band module (with src attribute) would go through the loader starting at the fetch hook. IOW the point of my PS above was that both inline and out-of-band <script type="module">s should go through the loader.

HMMM. Overall this seems reasonable. A few things:

Did you intentionally omit resolve? So resolve does not apply to src? That is good. That means src at least uses URLs, not module specifiers, solving my top complaint. (If you want to intercept resolve, then my suggestion of a second attribute comes in. Or just use <script>System.loader.import(...)</script>; it seems about the same number of characters.)
The way of overriding the default fetch behavior in the browser is to use service worker. Adding a second way of intercepting fetches to the web platform---which runs on the main thread, no less---is going to need a lot more discussion. Maybe that is a separate issue for the browser loader spec to work out, but it's a big one that I haven't seen an issue for yet. Heck, maybe you set up the browser loader inside your service worker, using specialized APIs on the FetchEvent to pass it to a loader instance, or something.
Running translate and instantiate on all <script type="module">s makes sense. It's basically building a framework into the browser for allowing custom execution of <script type="text/x-transcode-me">...</script> to be built by supplying two functions (translate/instantiate). This framework is the high-level API that would normally have to be expressed using mutation observers + shadow DOM + some probably-complicated dependency management logic, as-is done today. If there's implementer interest in building a framework for this use case, then translate/instantiate on <script type="module"> seems like a reasonable path (although again, maybe service workers would fit more with the platform, since that's where people will be transcoding multimedia and other response bodies).

Let me try to say it a little differently: my principal concern with not allowing the loader to participate in the hooking of script/module tags is that <script type="module"> becomes an incomplete story for how to kick off an app. Devs would need to learn that if they want loader integration, they have to write a top-level wrapper script. And if that's the case, then <script type="module"> is less universally reliable than just always kicking off apps with dynamic APIs, à la:

Yeah, I get that, at least for apps which need to globally customize translate/instantiate behavior.
On the other hand, I'm not sure <script type="module"> is really aimed at developers who want custom loaders. I would think such developers would, well, use their custom loader. That is, you seem to be proposing that such developers will do:
<script>
System.loader = class JSXLoader extends System.Loader { ... };
</script>

<script type="module" src="foo.jsx"></script>
<!-- or is it loadfromspecifier="./foo.jsx"? -->

whereas it seems more likely to me that they will do:
<script type="module">
const jsxLoader = new class JSXLoader extends System.Loader { ... };
jsxLoader.import("./foo.jsx");
</script>

or perhaps
<script type="module" src="https://cdn.example.com/jsx-loader.js"
          data-start="./foo.jsx"></script>

which is just generic sugar for the above, given a sufficiently well-written jsx-loader.js. This seems more compositional, messing with less global state, and more transparent as to what's going on; it only affects the module trees you explicitly import that way. It seems more likely to work in a world of third-party scripts.
What do you think?

Did you intentionally omit resolve? So resolve does not apply to src?

Oh, absolutely -- that's right, there's no resolve since there's no module name, and src always means URL.

The way of overriding the default fetch behavior in the browser is to use service worker. Adding a second way of intercepting fetches to the web platform---which runs on the main thread, no less---is going to need a lot more discussion.

Sure, although that's inherent to having a fetch hook in the loader API at all. I agree that there's overlap, and SW is mostly more general. It may well be the case that the fetch hook is less critical than the others, esp. the translate hook, since you can hook fetches with SW. (I'd frankly even be up for considering eliminating the fetch hook from the entire loader API.) The important thing is for <script type="module"> to be a full participant in the language semantics, which is hooked by the loader.

This seems more compositional, messing with less global state, and more transparent as to what's going on; it only affects the module trees you explicitly import that way.

Yeah, but examples like polyfills actually depend on global state that everyone needs to share (for example, shared mutations to builtins, or a runtime shared between all compiled code). And these modifications want to be coalesced in one place so that other code can be written under the assumption that they are operating in the proper environment (for example, an ES7 environment, whether provided by the browser directly or by a polyfill). Such global mutations are for sure less compositional, which is why an app needs to install them in one single place at the top level. All the other code just operates under the assumption that the global heap is in the appropriate state.

or perhaps <script type="module" src="https://cdn.example.com/jsx-loader.js"          data-start="./foo.jsx"></script>

These examples are definitely plausible. The thing I'm aiming for is to try to get as close as possible to a programming model where you can pretend <script> without type="module" doesn't exist, or at least eschew it within individual apps or "house styles." (This is part of why I really would like us to work on a way to get to <module> some day as an alias for <script type="module">, so the ergonomics are as sweet as <script>. But I know there are major parser/security challenges with that.)
Now, I'll grant that installing a polyfill with <script src="babel-polyfill.js"> already violates that goal. :-/ I'm definitely open to alternatives. Your suggestions are interesting, especially because they don't use <script> at all, but the part that bothers me is that I don't want to disallow having multiple scripts in a page, or require putting all relevant script in separate files. Keep in mind that skipping the loader means you can never have, say, inline ES8 transpiled via Babel, or inline transpiled WebAssembly.

(Brief aside: @wycats and I have talked about this general space as "the Web's staging challenge" but had a hard time articulating it crisply. Systems like SW and module loaders are about reflectively modifying the semantics of a web app from within the app itself, so there needs to be a way to indicate that the modifications happen in a stage prior to the processing of the code that depends on those modifications. Throw in the performance pitfalls of blocking and it's a pretty subtle space. The nice thing about the module system and the loader is that they're designed to be thoroughly asynchronous. But the place where the rubber meets the road is exactly what we're talking about here: how, where, and when you get to insert your modifications into HTML.)

We might really want to think about SW integration, BTW. Given that it's made more headway on the "first boot" phenomenon than anything, maybe loader/SW integration could be a more fruitful tack.

@dherman @domenic I agree. I think that having the SW be responsible for early-stage extensible-web extensions (like new packaging formats, the loader, etc.) is the most likely way to let applications "set up the universe" before their app boots.
SW's first-boot story isn't amazing yet, but it's eminently abstractable, and once you abstract it once the same solution will work for all kinds of extensions.
(in this case, you can imagine an SW hook for "give me the loader configuration")

whereas it seems more likely to me that they will do:

<script type="module">
const jsxLoader = new class JSXLoader extends System.Loader { ... };
jsxLoader.import("./foo.jsx");
</script>
```

You think people are going to inline a loader into a script tag rather than use a prebuilt one?  I see this scenario happening very rarely, for page-specific needs only. Anything generic like a JSX loader will surely be distributed.

Which brings up a concern in regards to the defer semantics, consider this:

```
<script type="module" src="jsx-loader.js"></script>
<script type="module" src="app.js"></script>
```

Saying that `app.js` depends on loader hooks installed by `jsx-loader.js` this means the complete tree of `jsx-loader.js` must be executed before `app.js` can do anything at all.

This stinks, so I'm ok with saying that loader hooks must be installed before any `type=module` scripts; meaning that must be added via a sloppy script tag.

Oh, absolutely -- that's right, there's no resolve since there's no module name, and src always means URL.

Note that there are no module names in the WhatWG Loader spec, period, only module identifiers and urls (keys).
How are these "anonymous modules" registered in the Loader registry?
<script type="module">
  import $ from 'jquery';

  $(function() { ...
</script>

Or is this module not in the registry? If so, what is the value of referrer in the resolve hook for when jquery is imported?

@matthewp:

Oh, absolutely -- that's right, there's no resolve since there's no module name, and src always means URL.
Note that there are no module names in the WHATWG Loader spec, period, only module identifiers and urls (keys).
How are these "anonymous modules" registered in the Loader registry?

A module doesn't need to be part of the registry, in which case, the key is irrelevant (could be anything). In the case of inline modules, we can simply create a new source text module record bound to a loader instance (a loader back-pointer is required to load dependencies but the module will not be added to the corresponding registry automatically), and then evaluate it.

Or is this module not in the registry? If so, what is the value of referrer in the resolve hook for when jquery is imported?

This is TBD, but last time we spoke about this, the page base url was sounded, but since that value will not be an entry in the registry, undefined might be a better option. Again, this is TBD.

Although I think currently the Loader expects keys to be strings.

No, it is just Let keyString be ? ToString(key)..

@dherman

And these modifications want to be coalesced in one place so that other code can be written under the assumption that they are operating in the proper environment (for example, an ES7 environment, whether provided by the browser directly or by a polyfill). Such global mutations are for sure less compositional, which is why an app needs to install them in one single place at the top level. All the other code just operates under the assumption that the global heap is in the appropriate state.

I don't think this is necessarily true, at least for transpilation. There it seems better to explicitly annotate which code runs in the transpiled environment. I can't imagine a case in which this is not under your control; if you are including the script, you can include it within a given loader scope.
I agree that global modifications to the built-ins must be shared, but those don't have much to do with the loader---any old <script type="module">, without a loader at all, can perform those modifications.
Another idea I liked was
<script type="module" loader="path/to/loader.js">
import "./goes-through-loader.jsx";
import "./more/files/through-loader";
</script>

These examples are definitely plausible. The thing I'm aiming for is to try to get as close as possible to a programming model where you can pretend <script> without type="module" doesn't exist, or at least eschew it within individual apps or "house styles."

Definitely agreed. I don't think there's anything preventing that though.

but the part that bothers me is that I don't want to disallow having multiple scripts in a page, or require putting all relevant script in separate files. Keep in mind that skipping the loader means you can never have, say, inline ES8 transpiled via Babel, or inline transpiled WebAssembly.

I guess this is addressed by my <script type="module" loader="path/to/loader.js"> idea.
@matthewp

You think people are going to inline a loader into a script tag rather than use a prebuilt one? I see this scenario happening very rarely, for page-specific needs only. Anything generic like a JSX loader will surely be distributed.

No, of course not; that's why I gave a second example. Or do import JSXLoader from "https://cdn.example.com/jsx-loader.js".

Which brings up a concern in regards to the defer semantics, consider this:

Yes, that seems clear. See https://github.com/whatwg/html/pull/443#issuecomment-169147982 and https://github.com/whatwg/html/pull/443#issuecomment-169402223. I am thinking you can opt-in to allowing out-of-order nondetermnistic execution by adding the async attribute.

Note that there are no module names in the WHATWG Loader spec, period, only module identifiers and urls (keys).

It keeps changing. At one point it was identifiers; these days I think it's specifiers. I think we can excuse @dherman for not using the exact right noun.

Just to comment with respect to Service Worker integration, in browsers that support Service Worker (with a good "first boot" story), surely the fetch and translate hooks are then effectively already provided by the service worker hooks itself?
That is, if the <script type="module"> tag is not going to respect fetch and translate from the loader, then there would be no need for them to be included in the loader spec surely?

Service Workers provide a way to globally override a page's fetching. In multiple loader scenarios you need to know the loader that is performing the fetch in order to apply correct overrides (for example our JSXLoader might use more aggressive caching than the default System.loader would).
Aside from that it would be good to keep this specification portable to other runtimes than just the web browser.


Indentation of args in function defs

Rustfmt doesn't fix that:
fn rewrite_call(context: &RewriteContext,
                     callee: &ast::Expr,
                     args: &[ptr::P<ast::Expr>],
                     width: usize,
                     offset: usize)
                     -> Option<String>

I hope this is easy, just hunting down the bug, in theory...

Actually, it seems to be a problem when dealing with a multiple-file crate, it seems that only the main file is fixed for that. I'm investigating...

Actually, we reformat only the main file in a multiple-file crate. The only test of multiple-file crate was the codebase of rustfmt itself, and it's just an idempotent check...
Closing, see #118.


Adds query filters sample.

Do we want samples to build during CI? (If so, they should be added to EFCore.sln)

Poor EFCore.sln 😞


how to get the precision and recall from the Evaluation Script  URGENT !!!

I want to know which part of the code is calculating the precision and recall in the evaluation script.
is this  line of code gives the precision and recall, and i especially want the recall values, the below code gives values for 1000 points. does that mean it is for 1000 images ?? URGENT !!
propose = pr_curve[:, 0]
recall = pr_curve[:, 1]

@Hiteshsaai 1000 points for evaluation, not 1000 images.

@wondervictor what do you mean by points and can you tell me whether this line of code below is the right one to get the precision and recall, if not can you help me with getting them from the evaluation script?
propose = pr_curve[:, 0]
recall = pr_curve[:, 1]

@wondervictor @Hiteshsaai I want to calculate detection accuracy. Can you explain the logic for calculating precision and recall in your code ?

@SURABHI-GUPTA you can get the recall & precision from the evaluation function of the evaluation.py script.
Declare a new array for precision and recall and Append propose and recall values for every iteration into the respective array in the evaluation function at the end after line number 260, and return those two arrays.
Each array will have three nested array inside it like ---> [[],[],[]]
using that you can calculate precision and recall.

@wondervictor I don’t understand why I set 1000 points for calculation, and I don’t know how to calculate the reasonable rate and value rate. What is the principle of this design?

Has anyone figured it out ?


JOSS Review: package version

Hi,
just a quick question: the package version stated for the JOSS review (https://github.com/openjournals/joss-reviews/issues/6746) is 1.1.7, which is quite off from the last version release (0.2.2).
Am I missing something here?
Thanks!

Hi,
the 1.1.7 version on pypi was the latest version at the time we were submitting the paper, but now we are at 1.1.9 (just some bug fixes em documentation updates). About the releases, we end up forgetting to update the github releases, only made the tags. I just made the github 1.1.9 release, so feel free to use it for review purposes.
Thanks for the feedback.

Thanks for resolving!


pyleus 0.2.4 bug: Num workers , Num executors in Strom UI

$ git clone https://github.com/Yelp/pyleus.git
$ pyleus build pyleus/examples/exclamation_topology/pyleus_topology.yaml
$ pyleus local exclamation_topology.jar

That works well. However, when
$ pyleus submit -n NIMBUS_HOST exclamation_topology.jar

I submit the jar to the storm cluster.
Num workers is 16, Num executors is 1.
That maybe a bug.

And plus,  I want to  roll back to 0.2.2, but  I specific  pyleus==0.2.2 in  requirements.txt . It won't work.
$  pyleus --verbose build  pyleus/examples/exclamation_topology/pyleus_topology.yaml

It shows that :
New python executable in /tmp/tmpuYS5z3/resources/pyleus_venv/bin/python
Installing setuptools, pip...done.
Downloading/unpacking pyleus==0.2.4
......

The version of pyleus is fixed! And I can't change it.
It occured to anyone?

how to uninstall pylesus? @William-Sang @poros

@wo142857  hava a try
pip uninstall pyleus


[Feature][UI] Added form input parser.

Purpose of the pull request
ref https://github.com/apache/dolphinscheduler/issues/12526

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report

Merging #12701 (86b7130) into dev (e702bec) will decrease coverage by 0.02%.
The diff coverage is n/a.

:exclamation: Current head 86b7130 differs from pull request most recent head 5efa850. Consider uploading reports for the commit 5efa850 to get more accurate results

@@             Coverage Diff              @@
##                dev   #12701      +/-   ##
============================================
- Coverage     39.14%   39.12%   -0.03%
+ Complexity     4207     4203       -4
============================================
  Files          1045     1045
  Lines         39685    39653      -32
  Branches       4565     4547      -18
============================================
- Hits          15536    15514      -22
+ Misses        22384    22378       -6
+ Partials       1765     1761       -4

Impacted Files
Coverage Δ

...erver/master/processor/queue/TaskEventService.java
69.64% <0.00%> (-10.72%)
:arrow_down:

...org/apache/dolphinscheduler/remote/utils/Host.java
42.55% <0.00%> (-2.13%)
:arrow_down:

...hinscheduler/plugin/alert/script/ScriptSender.java
70.58% <0.00%> (-1.64%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
51.38% <0.00%> (-1.39%)
:arrow_down:

...cheduler/plugin/alert/dingtalk/DingTalkSender.java
34.13% <0.00%> (-0.78%)
:arrow_down:

...pache/dolphinscheduler/common/utils/DateUtils.java
68.30% <0.00%> (-0.55%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.86% <0.00%> (-0.55%)
:arrow_down:

...inscheduler/plugin/registry/etcd/EtcdRegistry.java
50.69% <0.00%> (-0.35%)
:arrow_down:

.../dolphinscheduler/plugin/task/datax/DataxTask.java
36.62% <0.00%> (-0.26%)
:arrow_down:

.../dolphinscheduler/plugin/task/spark/SparkTask.java
72.56% <0.00%> (-0.25%)
:arrow_down:

... and 19 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Look into cloud storage options for targets

The current github action runs the targets pipeline once a day.  I think the way it works is it commits all the files generated by running the pipeline, including the .rds files in the data store, to the targets-runs branch.  This is necessary so those targets are available the next time the action runs so it doesn't have to start from scratch.  However, I think committing these non-diffable files every day is going to quickly balloon the size of this repo on GitHub (at least on that targets-runs branch).  An alternative would be to use cloud storage for targets.  @dlebauer and I think there is a way to get University-supported AWS S3 buckets, but we need to look into it.

I realized that it is only committing the forecast targets daily, which are likely quite small.  But if we want to use an S3 bucket, I should estimate the usage and contact Matt Rahr.

I no longer think this is necessary

I don't think it's actually going to use up that much space, and worst-case we can nuke the targets_runs branch along with it's git history and start fresh.  I also just never heard back from the person who was going to get me a S3 bucket to try.


Provide a means to hide proxy user and password in settings.josn

Issue Type: Feature Request
The http.proxy value provides a means to specify a proxy if behind a firewall, but since my corporate proxy requires Userid and Password to access the proxy I have to enter that into the http.proxy setting.  Doing this makes uerid and password viewable in the clear within the settings.json file.  It would be desirable to somehow specify the password as an encrypted value or have the proxy setting not be stored in the clear within the settings.json file.
VS Code version: Code 1.21.1 (79b44aa704ce542d8ca4a3cc44cfca566e7720f1, 2018-03-14T14:47:13.351Z)
OS version: Windows_NT ia32 10.0.14393

This won't happen any time soon, unfortunately. We don't have a safe way of doing so.


Enable wheelPropagation on Demo Page

As propagation is the default behavior in native scroll containers, I think
this will increase the perceived quality of the library when people try it out
on the Demo Page.

I think this is a valid point. Rather than setting it true only in demo page, it may be better to set it as default behaviour?

@utatti I like that even better!

I may still close this. Users may be confused when it works differently than how they see on demo page. Sorry again, but please wait for #732. Thanks!


updated deps and ran go mod tidy & vendor

Updating deps cause we have no automation now

@gab-satchi for the release 1.9


xcode sims now take the data from simctl

Codecov Report

Merging #297 into develop will decrease coverage by 0.15%.
The diff coverage is 3.12%.

@@            Coverage Diff             @@
##           develop    #297      +/-   ##
==========================================
- Coverage    29.56%   29.4%   -0.16%
==========================================
  Files           54      54
  Lines         5777    5801      +24
  Branches      1270    1280      +10
==========================================
- Hits          1708    1706       -2
- Misses        2949    2965      +16
- Partials      1120    1130      +10

Flag
Coverage Δ

#rnvunittests
29.4% <3.12%> (-0.16%)
:arrow_down:

Impacted Files
Coverage Δ

packages/rnv/src/platformTools/apple/index.js
13.54% <0%> (ø)
:arrow_up:

...kages/rnv/src/platformTools/apple/deviceManager.js
10.63% <3.17%> (-6.51%)
:arrow_down:

Continue to review full report at Codecov.

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


CASMINST-6503: Linting

Description
As always, I noticed minor doc problems when working on the latest runthrough of the disaster recovery procedure on wasp. This PR corrects them.
Backport PRs forthcoming.

Checklist

[X] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[X] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[X] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

/backport release/1.3


Azure/spring-cloud-deploy@v1 fails to find package

I have a GitHub workflow with 1 Build Job and 1 Deploy Job, so I had to upload artifacts after the Build to then be able to download them at deployment time.
I have debug traces that shows the artifact is indeed there at the expected location "/github/workspace/${{ env.DISCOVERY_SERVER }}/${{ env.DISCOVERY_SERVER_JAR_ARTIFACT }}" , the Azure Spring Cloud App deployment runs fine ONLY with CLI and ONLY when
setting 1 variable built on top of other variables
DISCOVERY_SERVER_ARTIFACT_FULL_PATH="/github/workspace/${{ env.DISCOVERY_SERVER }}/${{ env.DISCOVERY_SERVER_JAR_ARTIFACT }}"
az spring-cloud app deployment create --artifact-path $DISCOVERY_SERVER_ARTIFACT_FULL_PATH ......

This snippets fails :
az spring-cloud app deployment create --artifact-path "/github/workspace/${{ env.DISCOVERY_SERVER }}/${{ env.DISCOVERY_SERVER_JAR_ARTIFACT }}"

See my pipeline :
az spring-cloud app deployment create --name ${{ env.DEPLOYMENT_NAME }} --app ${{ env.DISCOVERY_SERVER }} -s ${{ env.SPRING_CLOUD_SERVICE }} --artifact-path $DISCOVERY_SERVER_ARTIFACT_FULL_PATH -g ${{ env.DEPLOYMENT_RG }} --version ${{ env.DEPLOYMENT_VERSION }} --runtime-version ${{ env.DEPLOYMENT_RUNTIME_VERSION }} --cpu ${{ env.DEPLOYMENT_CPU }} --memory ${{ env.DEPLOYMENT_MEMORY }} --instance-count ${{ env.DEPLOYMENT_INSTANCE_COUNT }} --disable-probe ${{ env.DEPLOYMENT_DISABLE_PROBE }}

See my pipeline
    - name: Deploy DISCOVERY-SERVER
      uses:  Azure/spring-cloud-deploy@v1
      with:
        azure-subscription: ${{ secrets.AZURE_SUBSCRIPTION }}
        action: deploy
        service-name: ${{ env.SPRING_CLOUD_SERVICE }}
        app-name: ${{ env.DISCOVERY_SERVER }}
        use-staging-deployment: ${{ env.DEPLOYMENT_STAGING }}
        package: /github/workspace/${{ env.DISCOVERY_SERVER }}/${{ env.DISCOVERY_SERVER_JAR_ARTIFACT }}
        jvm-options: ${{ env.DEPLOYMENT_JVM_OPTIONS }}
        deployment-name: ${{ env.DEPLOYMENT_NAME }}
        create-new-deployment: ${{ env.DEPLOYMENT_CREATE_NEW }}
        version: ${{ env.DEPLOYMENT_VERSION }}
        runtime-version: ${{ env.DEPLOYMENT_RUNTIME_VERSION }}

Pipeline run failure: https://github.com/ezYakaEagle442/azure-spring-cloud-petclinic-mic-srv/runs/5742240456?check_suite_focus=true
Action failed with error: No package found with specified pattern: /github/workspace/discovery-server/spring-petclinic-discovery-server-2.6.3.jar

What would be needed as well is a GitHub action task to do the quivalent of 'az spring-cloud app deployment create'
Reading the GitHub Action Reference for Azure Spring Cloud  helps as I have found this argument create-new-deployment: ==> If set to true and the deployment specified by deployment-name does not exist at execution time, it will be created.
However there is no option to combine create-new-deployment setting the parameters below using Azure/spring-cloud-deploy@v1 :
--cpu ${{ env.DEPLOYMENT_CPU }} --memory ${{ env.DEPLOYMENT_MEMORY }} --instance-count ${{ env.DEPLOYMENT_INSTANCE_COUNT }} --disable-probe ${{ env.DEPLOYMENT_DISABLE_PROBE }}
Also this fails as well of course as $DISCOVERY_SERVER_ARTIFACT_FULL_PATH  is not known in this context
package: $DISCOVERY_SERVER_ARTIFACT_FULL_PATH

Action failed with error: No package found with specified pattern: $DISCOVERY_SERVER_ARTIFACT_FULL_PATH

See also :

https://github.com/MicrosoftDocs/azure-docs/issues/90220
https://github.com/Azure/azure-cli-extensions/issues/4592

Update, I have fixed  the issue , the main cause was : the step that creates or updates the environment variable does not have access to the new value, but all subsequent steps in a job will have access.
  DEPLOYMENT_JVM_OPTIONS: -Dazure.keyvault.uri=https://kv-petcliasc42.vault.azure.net -Xms512m -Xmx1024m -Dspring.profiles.active=mysql,key-vault,cloud

  DEPLOYMENT_VERSION: 2.6.3
  DEPLOYMENT_RUNTIME_VERSION: Java_11
  DEPLOYMENT_CPU: 500m
  DEPLOYMENT_MEMORY: 512Mi
  DEPLOYMENT_INSTANCE_COUNT: 3
  DEPLOYMENT_DISABLE_PROBE: false

  # If set to true and the deployment specified by deployment-name does not exist at execution time, it will be created.
  DEPLOYMENT_CREATE_NEW: true

  # /!\ Deployment name can contain only lowercase letters, numbers and hyphens.
  DEPLOYMENT_NAME: blue
  DEPLOYMENT_STAGING: true
  DEPLOYMENT_ACTION: deploy

  # Choose either Blue or Green
  # DEPLOYMENT_NAME: green
  # DEPLOYMENT_STAGING: false
  # DEPLOYMENT_ACTION: set-production

jobs:
  deploy:
    runs-on: ubuntu-latest
    steps:
    - name: Set Base environment variables
      run: |
        echo "LOCAL_IP=$(curl whatismyip.akamai.com)" >> $GITHUB_ENV

        echo "API_GATEWAY_JAR_ARTIFACT=${{ env.PRJ_PREFIX }}-${{ env.API_GATEWAY }}-${{ env.DEPLOYMENT_VERSION }}.jar" >> $GITHUB_ENV
        echo "ADMIN_SERVER_JAR_ARTIFACT=${{ env.PRJ_PREFIX }}-${{ env.ADMIN_SERVER }}-${{ env.DEPLOYMENT_VERSION }}.jar" >> $GITHUB_ENV
        echo "CONFIG_SERVER_JAR_ARTIFACT=${{ env.PRJ_PREFIX }}-${{ env.CONFIG_SERVER }}-${{ env.DEPLOYMENT_VERSION }}.jar" >> $GITHUB_ENV
        echo "DISCOVERY_SERVER_JAR_ARTIFACT=${{ env.PRJ_PREFIX }}-${{ env.DISCOVERY_SERVER }}-${{ env.DEPLOYMENT_VERSION }}.jar" >> $GITHUB_ENV
        echo "CUSTOMERS_SERVICE_JAR_ARTIFACT=${{ env.PRJ_PREFIX }}-${{ env.CUSTOMERS_SERVICE }}-${{ env.DEPLOYMENT_VERSION }}.jar" >> $GITHUB_ENV
        echo "VETS_SERVICE_JAR_ARTIFACT=${{ env.PRJ_PREFIX }}-${{ env.VETS_SERVICE }}-${{ env.DEPLOYMENT_VERSION }}.jar" >> $GITHUB_ENV
        echo "VISITS_SERVICE_JAR_ARTIFACT=${{ env.PRJ_PREFIX }}-${{ env.VISITS_SERVICE }}-${{ env.DEPLOYMENT_VERSION }}.jar" >> $GITHUB_ENV

        echo "API_GATEWAY_DEPLOYMENT=${{ env.DEPLOYMENT_NAME }}-${{ env.API_GATEWAY }}" >> $GITHUB_ENV
        echo "ADMIN_SERVER_DEPLOYMENT=${{ env.DEPLOYMENT_NAME }}-${{ env.ADMIN_SERVER }}" >> $GITHUB_ENV
        echo "CONFIG_SERVER_DEPLOYMENT=${{ env.DEPLOYMENT_NAME }}-${{ env.CONFIG_SERVER }}" >> $GITHUB_ENV
        echo "CUSTOMERS_SERVICE_DEPLOYMENT=${{ env.DEPLOYMENT_NAME }}-${{ env.CUSTOMERS_SERVICE }}" >> $GITHUB_ENV
        echo "DISCOVERY_SERVER_DEPLOYMENT=${{ env.DEPLOYMENT_NAME }}-${{ env.DISCOVERY_SERVER }}" >> $GITHUB_ENV
        echo "VETS_SERVICE_DEPLOYMENT=${{ env.DEPLOYMENT_NAME }}-${{ env.VETS_SERVICE }}" >> $GITHUB_ENV
        echo "VISITS_SERVICE_DEPLOYMENT=${{ env.DEPLOYMENT_NAME }}-${{ env.VISITS_SERVICE }}" >> $GITHUB_ENV

        echo "GH_API=api.github.com/user/packages/maven" >> $GITHUB_ENV

        echo "ADMIN_SERVER_GROUP_ID=org.springframework.samples.petclinic.admin" >> $GITHUB_ENV
        echo "ADMIN_SERVER_ARTIFACT_ID=${{ env.PRJ_PREFIX }}-${{ env.ADMIN_SERVER }}" >> $GITHUB_ENV

        echo "API_GATEWAY_GROUP_ID=org.springframework.samples.petclinic.api" >> $GITHUB_ENV
        echo "API_GATEWAY_ARTIFACT_ID=${{ env.PRJ_PREFIX }}-${{ env.API_GATEWAY }}" >> $GITHUB_ENV

        echo "CONFIG_SERVER_GROUP_ID=org.springframework.samples.petclinic.config" >> $GITHUB_ENV
        echo "CONFIG_SERVER_ARTIFACT_ID=${{ env.PRJ_PREFIX }}-${{ env.CONFIG_SERVER }}" >> $GITHUB_ENV

        echo "CUSTOMERS_SERVICE_GROUP_ID=org.springframework.samples.petclinic.client" >> $GITHUB_ENV
        echo "CUSTOMERS_SERVICE_ARTIFACT_ID=${{ env.PRJ_PREFIX }}-${{ env.CUSTOMERS_SERVICE }}" >> $GITHUB_ENV

        echo "DISCOVERY_SERVER_GROUP_ID=org.springframework.samples.petclinic.discovery" >> $GITHUB_ENV
        echo "DISCOVERY_SERVER_ARTIFACT_ID=${{ env.PRJ_PREFIX }}-${{ env.DISCOVERY_SERVER }}" >> $GITHUB_ENV

        echo "VETS_SERVICE_GROUP_ID=org.springframework.samples.petclinic.vets" >> $GITHUB_ENV
        echo "VETS_SERVICE_ARTIFACT_ID=${{ env.PRJ_PREFIX }}-${{ env.VETS_SERVICE }}" >> $GITHUB_ENV

        echo "VISITS_SERVICE_GROUP_ID=org.springframework.samples.petclinic.visits" >> $GITHUB_ENV
        echo "VISITS_SERVICE_ARTIFACT_ID=${{ env.PRJ_PREFIX }}-${{ env.VISITS_SERVICE }}" >> $GITHUB_ENV

        echo "GH_WORKSPACE=${{ github.workspace }}" >> $GITHUB_ENV # "/github/workspace"
      shell: bash

    # Variables in the env map cannot be defined in terms of other variables in the map
    - name: Set dependent environment variables
      run: |
        # ==== JARS ====: PWD already in /home/runner/work/azure-spring-cloud-petclinic-mic-srv >> $GITHUB_ENV
        echo "API_GATEWAY_JAR=${{ env.PRJ_PREFIX }}-${{ env.API_GATEWAY }}/target/$API_GATEWAY_JAR_ARTIFACT" >> $GITHUB_ENV
        echo "ADMIN_SERVER_JAR=${{ env.PRJ_PREFIX }}-${{ env.ADMIN_SERVER }}/target/$ADMIN_SERVER_JAR_ARTIFACT" >> $GITHUB_ENV
        echo "CONFIG_SERVER_JAR=${{ env.PRJ_PREFIX }}-${{ env.CONFIG_SERVER }}/target/$CONFIG_SERVER_JAR_ARTIFACT" >> $GITHUB_ENV
        echo "CUSTOMERS_SERVICE_JAR=${{ env.PRJ_PREFIX }}-${{ env.CUSTOMERS_SERVICE }}/target/$CUSTOMERS_SERVICE_JAR_ARTIFACT" >> $GITHUB_ENV
        echo "DISCOVERY_SERVER_JAR=${{ env.PRJ_PREFIX }}-${{ env.DISCOVERY_SERVER }}/target/$DISCOVERY_SERVER_JAR_ARTIFACT" >> $GITHUB_ENV
        echo "VETS_SERVICE_JAR=${{ env.PRJ_PREFIX }}-${{ env.VETS_SERVICE }}/target/$VETS_SERVICE_JAR_ARTIFACT" >> $GITHUB_ENV
        echo "VISITS_SERVICE_JAR=${{ env.PRJ_PREFIX }}-${{ env.VISITS_SERVICE }}/target/$VISITS_SERVICE_JAR_ARTIFACT" >> $GITHUB_ENV

        echo "API_GATEWAY_PACKAGE_PATH=$GH_WORKSPACE/${{ env.API_GATEWAY }}/${{ env.API_GATEWAY_JAR_ARTIFACT }}" >> $GITHUB_ENV
        echo "ADMIN_SERVER_PACKAGE_PATH=$GH_WORKSPACE/${{ env.ADMIN_SERVER }}/${{ env.ADMIN_SERVER_JAR_ARTIFACT }}" >> $GITHUB_ENV
        echo "CONFIG_SERVER_PACKAGE_PATH=$GH_WORKSPACE/${{ env.CONFIG_SERVER }}/${{ env.CONFIG_SERVER_JAR_ARTIFACT }}" >> $GITHUB_ENV
        echo "CUSTOMERS_SERVICE_PACKAGE_PATH=$GH_WORKSPACE/${{ env.CUSTOMERS_SERVICE }}/${{ env.CUSTOMERS_SERVICE_JAR_ARTIFACT }}" >> $GITHUB_ENV
        echo "DISCOVERY_SERVER_PACKAGE_PATH=$GH_WORKSPACE/${{ env.DISCOVERY_SERVER }}/${{ env.DISCOVERY_SERVER_JAR_ARTIFACT }}" >> $GITHUB_ENV
        echo "VETS_SERVICE_PACKAGE_PATH=$GH_WORKSPACE/${{ env.VETS_SERVICE }}/${{ env.VETS_SERVICE_JAR_ARTIFACT }}" >> $GITHUB_ENV
        echo "VISITS_SERVICE_PACKAGE_PATH=$GH_WORKSPACE/${{ env.VISITS_SERVICE }}/${{ env.VISITS_SERVICE_JAR_ARTIFACT }}" >> $GITHUB_ENV

        echo "ADMIN_SERVER_API_URL=https://$GH_API/$ADMIN_SERVER_GROUP_ID.$ADMIN_SERVER_ARTIFACT_ID/versions" >> $GITHUB_ENV
        echo "API_GATEWAY_API_URL=https://$GH_API/$API_GATEWAY_GROUP_ID.$API_GATEWAY_ARTIFACT_ID/versions" >> $GITHUB_ENV
        echo "CONFIG_SERVER_API_URL=https://$GH_API/$CONFIG_SERVER_GROUP_ID.$CONFIG_SERVER_ARTIFACT_ID/versions" >> $GITHUB_ENV
        echo "CUSTOMERS_SERVICE_API_URL=https://$GH_API/$CUSTOMERS_SERVICE_GROUP_ID.$CUSTOMERS_SERVICE_ARTIFACT_ID/versions" >> $GITHUB_ENV
        echo "DISCOVERY_SERVER_API_URL=https://$GH_API/$DISCOVERY_SERVER_GROUP_ID.$DISCOVERY_SERVER_ARTIFACT_ID/versions" >> $GITHUB_ENV
        echo "VETS_SERVICE_API_URL=https://$GH_API/$VETS_SERVICE_GROUP_ID.$VETS_SERVICE_ARTIFACT_ID/versions" >> $GITHUB_ENV
        echo "VISITS_SERVICE_API_URL=https://$GH_API/$VISITS_SERVICE_GROUP_ID.$VISITS_SERVICE_ARTIFACT_ID/versions" >> $GITHUB_ENV
      shell: bash

I now have 1 Build Job in 1 Workflow + 1 Deploy Job in another Workflow.
The Build Job uses Maven to deploy the Artifacts to GitHub packages
    - name:  Publish the package
      run: |
        mkdir -p ~/.m2
        echo "<settings><servers><server><id>github-asc</id><username>$(echo "$GITHUB_REPOSITORY" | awk -F / '{print $1}')</username><password>\${env.GITHUB_TOKEN}</password></server></servers></settings>" > ~/.m2/settings.xml
        REPO="github-asc::default::https://maven.pkg.github.com/${GITHUB_REPOSITORY}"

        mvn deploy -DaltReleaseDeploymentRepository="${REPO}" -DaltSnapshotDeploymentRepository="${REPO}" --batch-mode -DskipTests -Dmaven.install.skip=true
      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

Now in the deploy Workflow
    - name: Deploy DISCOVERY-SERVER
      uses:  Azure/spring-cloud-deploy@v1
      with:
        azure-subscription: ${{ secrets.AZURE_SUBSCRIPTION }}
        action: deploy
        service-name: ${{ env.SPRING_CLOUD_SERVICE }}
        app-name: ${{ env.DISCOVERY_SERVER }}
        use-staging-deployment: ${{ env.DEPLOYMENT_STAGING }}
        package: ${{ env.DISCOVERY_SERVER_PACKAGE_PATH }}
        jvm-options: ${{ env.DEPLOYMENT_JVM_OPTIONS }}
        deployment-name:  ${{ env.DISCOVERY_SERVER_DEPLOYMENT }}
        create-new-deployment: ${{ env.DEPLOYMENT_CREATE_NEW }}
        version: ${{ env.DEPLOYMENT_VERSION }}
        runtime-version: ${{ env.DEPLOYMENT_RUNTIME_VERSION }}

this argument create-new-deployment: ==> If set to true and the deployment specified by deployment-name does not exist at execution time, it will be created. However it does not provide all the options offered by the CLI :
az spring-cloud app deployment create  ** --cpu 500m --memory 512Mi --instance-count 3 --disable-probe false**
CONCLUSION: bascially we need a GitHub Action to create a deployment with all the options offered by the CLI , including --cpu  --memory --instance-count  --disable-probe


fix demo site script to render props correctly

General summary
A little fix to render the components props on the demo site
Screenshots
Before

After

Check list

[ ] (if has style) Add need: review-integration label
[ ] (if has textual documentation) Add need: review-doc label
[ ] (if has code) Add need: review-frontend label
[ ] (if has react) Check through the react dev check list
[x] (if fix or feature) Add need: test label

Check out the contribution guide for general guidelines for submissions to the design system.

Nice


GDScript: indexing PackedFloat32Array fails

Godot version:
4.0 edccc0b
OS/device including version:
Microsoft Windows [Version 10.0.19041.685]
Issue description:
I created a packed array with multiple elements. indexing any value such as 0 causes a gdscript exception:
Invalid get index '0' (on base: 'PackedFloat32Array').

Steps to reproduce:

open attached project.
open packedarraytest.tscn
play scene (2nd button from the right on top bar)
it will get an error

workaround: use for loop and break after index iterations.
Minimal reproduction project:
gdscriptbugs.zip

I am even getting an error just doing this in gdscript:

var p: PackedFloat32Array = PackedFloat32Array.new();

With error: "Builtin type cannot be used as a name on its own."

I am even getting an error just doing this in gdscript:

var p: PackedFloat32Array = PackedFloat32Array.new();

With error: "Builtin type cannot be used as a name on its own."

The correct syntax is:
var p: PackedFloat32Array = PackedFloat32Array.new();

Using new() on built-in types isn't supported.

The correct syntax is:
var p: PackedFloat32Array = PackedFloat32Array.new();

Using new() on built-in types isn't supported.

It looks like it is because REGISTER_INDEXED_MEMBER(PackedFloat32Array); is missing in the following code:
https://github.com/godotengine/godot/blob/7d70cdc6ccbea0cca2619a8790e097724e8e5f18/core/variant/variant_setget.cpp#L1034-L1056
There is one for PackedInt32Array but not PackedFloat32Array.
Adding it fix the issue for me.


Cleanup unused stuff

Removes some unused imports. Also removes gltf2_io_color_management.py. The functions there have been replaced by numpy code (importer, exporter).

@julienduroure Ping.

Done. I was waiting 2.91 release to avoid potential merge issue


Add missing RAC Checkbox/CheckboxGroup story files

I've realized there's no story for both RAC Checkbox and CheckboxGroup.
So I added them with the first examples from the docs:
Checkbox

https://react-spectrum.adobe.com/react-aria/Checkbox.html#example

CheckboxGroup

https://react-spectrum.adobe.com/react-aria/CheckboxGroup.html#example

✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

GET_BUILD


Clarify muliti-line chain elements

Say (and show) that a multi-line element causes all surrounding elements to be on their own line, not just later elements. But also maintain the caveat where multiple elements may be on the first line.
I added a closure the example so that it is more certainly a multi-line element.
See #151

Thank you for the review. I finally understand what was wrong with my understanding regarding "surrounding elements". I updated the PR with another attempt to clarify, hopefully without changing the rules this time.

Closing this as the guide doesn't live here anymore (it's in r-l/rust), discussion remains tracked in #151 in the event there are any additional examples or clarifications (including specific ones proposed in this PR) that we'd like to cherry pick over to the new guide location