chore(go): support get/set kernel API in go runtime

Fixes https://github.com/aws/jsii/issues/2128

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 875683c2c0ecb08e87d241e7a2a13afc31bc7bf5
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: d6f5473d8986c0ec45b194a2ec566eaca0c529f9
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: d03cb53ba5ee3fb6b3f383732723e141b6238abc
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 1fd2d4e8ab1364ca4cb8da55b3518474451a0a61
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 7eff4717a448853a784c2775d1e8537d8e6778fa
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 81763e6b4f54a74cad79c4697b2562d953c71c6d
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


How to push model into demo.goharbor.io from machine behind proxy

Is this a BUG REPORT or FEATURE REQUEST?:
/kind feature
What happened:
I am trying out ormb commands to push models into demo.goharbor.io

Executing ormb commands from Linux VM behind a proxy
ormb login was failing before setting https_proxy but after setting the https_proxy, ormb login is successful
ormb push is failing with connection timeout(even after setting proxy)

What you expected to happen:
ormb push should be successful after setting https_proxy
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
ormb push demo.goharbor.io/ormb/xgboostsample:v1
The push refers to repository [demo.goharbor.io/ormb/xgboostsample]
ref:       demo.goharbor.io/ormb/xgboostsample:v1
digest:    b57ea86dceff41196d633005bbaeb70af71ad2a7b3e8a8f490ad06757d1436fa
size:      905 B
format:    XGBoost
Error: failed to do request: Head "https://demo.goharbor.io/v2/ormb/xgboostsample/blobs/sha256:e56034e222179843e08f702fe87e6e9804bef82aea085cc8d8620ac51487282c": dial tcp <IP_ADDRESS>:443: connect: connection timed out
Usage:
  ormb push [flags]

Flags:
  -h, --help         help for push
      --insecure     allow connections to TLS registry without certs (default true)
      --plain-http   use plain http and not https

Global Flags:
      --config string      config file (default is $HOME/.ormb/config.yaml)
      --log-level uint32   Log level (default 4)

PANI[2021-01-05T15:21:51Z] Failed to run the  command                    error="failed to do request: Head \"https://demo.goharbor.io/v2/ormb/xgboostsample/blobs/sha256:e56034e222179843e08f702fe87e6e9804bef82aea085cc8d8620ac51487282c\": dial tcp <IP_ADDRESS>:443: connect: connection timed out"
panic: (*logrus.Entry) (0xb9acc0,0xc0003693b0)

goroutine 1 [running]:
github.com/sirupsen/logrus.Entry.log(0xc0000d8000, 0xc0003758c0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
        /Users/bytedance/go/pkg/mod/github.com/sirupsen/logrus@v1.4.2/entry.go:239 +0x2da
github.com/sirupsen/logrus.(*Entry).Log(0xc000369340, 0x0, 0xc00025ff08, 0x1, 0x1)
        /Users/bytedance/go/pkg/mod/github.com/sirupsen/logrus@v1.4.2/entry.go:268 +0xeb
github.com/sirupsen/logrus.(*Entry).Logln(0xc000369340, 0x0, 0xc00025ff58, 0x1, 0x1)
        /Users/bytedance/go/pkg/mod/github.com/sirupsen/logrus@v1.4.2/entry.go:359 +0xd1
github.com/sirupsen/logrus.(*Entry).Panicln(...)
        /Users/bytedance/go/pkg/mod/github.com/sirupsen/logrus@v1.4.2/entry.go:397
github.com/kleveross/ormb/cmd/ormb/cmd.Execute()
        /Users/bytedance/go/src/github.com/kleveross/ormb/cmd/ormb/cmd/root.go:45 +0xcb
main.main()
        /Users/bytedance/go/src/github.com/kleveross/ormb/cmd/ormb/main.go:23 +0x20

Can you show us the https_proxy config? I am not sure if it’s a proxy issue.

Can you show us the https_proxy config? I am not sure if it’s a proxy issue.

Can you show us the https_proxy config? I am not sure if it’s a proxy issue.

I am setting an environment variable(https_proxy)  in the shell prompt before executing ormb push
export https_proxy=<proxyhost:proxyport>

Can you show us the https_proxy config? I am not sure if it’s a proxy issue.

I am setting an environment variable(https_proxy)  in the shell prompt before executing ormb push
export https_proxy=<proxyhost:proxyport>


what training algorithms is pointpillar using?

May I ask what type of traning algorithm is PointPillars using? is it Adam or SGD? If it is could you explain it to me?

Hello, it's here:
https://github.com/zhulf0804/PointPillars/blob/b9948e73505c8d6bfa631ffdf76c7148e82c5942/train.py#L46-L49

thank you


[ETL-659] Snowflake git integration

Deploys Snowflake objects to a specific environment as part of our CI/CD process. An environment is analogous to a database. We assume that account-level objects have already been created.
The following objects are created:

RECOVER_GIT_REPOSITORY git repository
RECOVER_{environment} database (I will refer to this merely as the RECOVER database from here on for simplicity sake).
PARQUET schema
PARQUET_FORMAT file format.
PARQUET_S3 external stage.
A table for each Parquet dataset in our external stage (manually defined)

Deployment logic and object DDL is organized as a hierarchy:
snowflake/objects
└── databases
    └── recover
        └── schemas
            └── parquet
                ├── file_format
                ├── stages
                └── tables

(Almost) every level in this hierarchy has a deploy.sql which will deploy all child objects with respect to the current directory.
snowflake/objects
├── databases
│   ├── deploy.sql
│   └── recover
│       ├── deploy.sql
│       └── schemas
│           ├── deploy.sql
│           └── parquet
│               ├── deploy.sql
│               ├── file_format
│               │   ├── deploy.sql
│               │   └── parquet_format.sql
│               ├── stages
│               │   ├── deploy.sql
│               │   └── parquet_s3.sql
│               └── tables
│                   ├── deploy.sql
│                   ├── enrolledparticipants_customfields_symptoms_parquet.sql
│                   ├── enrolledparticipants_customfields_treatments_parquet.sql
│                   ├── enrolledparticipants_parquet.sql
│                   ├── fitbitactivitylogs_parquet.sql
│                   ├── fitbitdailydata_parquet.sql
│                   ├── ...
└── deploy.sql

For example, the file located at snowflake/objects/databases/recover/deploy.sql will deploy all objects under the RECOVER database, and snowflake/objects/databases/recover/schemas/parquet/deploy.sql will deploy the file formats, stages, and tables under the PARQUET schema.
The child objects, that is, the schemas, file formats, stages, tables – anything that is not a database – are defined in such a way as to be agnostic to their parent context. For example, snowflake/objects/databases/recover/schemas/parquet/deploy.sql will deploy the PARQUET schema and all its child objects to whichever database your Snowflake user assumes. There is nothing in the SQL which restricts the PARQUET schema to be created within the RECOVER database. Likewise, the tables can be deployed to any schema, although their DDL is specific to the columns in our Parquet datasets.
The entrypoint for deployment is snowflake/objects/deploy.sql. When a commit is pushed to a branch, our CI/CD process will instantiate all Snowflake objects to an environment-specific database prefixed with your branch name, e.g., RECOVER_ETL_659. Details on how to manually deploy are provided in the file itself.
Implementation details

This PR is an incomplete solution to Snowflake deployment. In a production-ready deployment configuration, we would already have a production database with tables containing data from which we could easily derive tables to deploy to a dev environment. Lacking that, we define our tables from scratch and don't yet load any data into them. Having the ability to create tables from scratch is a critical component for doing reproducible deployments, but it's not an effective production-level deployment strategy.
Objects are created in a few different ways:

CREATE OR REPLACE will destroy any existing objects and create a new object. This is a powerful, declarative way of enforcing object configurations, but it's not always effective to recreate objects and sometimes existing objects ought to be reused.
CREATE ... IF NOT EXISTS will create an object if it doesn't yet already exist. In some cases this is to preserve the state of that object or its child objects. In other cases (when creating the GIT REPOSITORY) it's used a little more subjectively. The prototypical use case for this statement is to preserve tables containing data and their parent objects. I use this statement with DATABASE and SCHEMA objects because they are parents of table objects. Although, if this is a dev deployment, I instead CREATE OR REPLACE the database object in anticipation of cloning preexisting production tables while ensuring a completely reproducible dev environment with each new deployment.
CREATE OR ALTER is a mechanism for declaratively updating table objects. More information here.

Table data types are defined to be as large as possible without consuming extraneous storage or requiring additional processing. For more information, see String & binary data types and Numeric data types.

I wanted to touch on this "This PR is an incomplete solution to Snowflake deployment." I am still a bit confused by this, but could you outline what you think a "production" deployment looks like via CI/CD?

To summarize what the deployment does in this PR, it effectively creates all the objects necessary to create tables (plus a few extra – FILE FORMAT and STAGE objects – in anticipation of future work) and then creates tables for each of our Parquet datasets. It doesn't load any data, it merely creates the tables (or alters them, if they already exist).
This is less than ideal for a developer. They don't want to load any data and they definitely don't want to do it manually! So in the future, when we have Snowflake tables in production which already contain our data, we can modify the default behavior of this deployment so that we clone those tables, rather than create empty tables. That's what makes this an incomplete solution.
As for what a production-ready deployment system looks like, I believe we ought to have multiple ways of deploying tables:

By cloning from existing tables in production
By creating the table from scratch and loading data into the table via COPY INTO from an external stage.
By updating the table if there are changes, and otherwise doing nothing

(1) is our ideal dev environment deployment scenario. It requires minimal overhead and we get a production database in an isolated environment that we can muck around in.
(2) is our bootstrap scenario. If – for whatever reason – we need to redeploy our database entirely from scratch, we should be able to do that.
(3) is our production scenario. We want to update the table metadata if there have been any changes, but data loading happens via a separate mechanism not related to deployment.

when you push into main here, I think it creates a recover_main database?

No, RECOVER_MAIN is one of the databases which we already assume to exist as detailed in snowflake/objects/deploy.sql. I use Snowflake scripting to specify that we only create or replace a database if this is not the main or staging branch/environment.


Use page resources for OpenGraph and Twitter Card images

For my site I'm using page resources to transform and display cover images for my stories. It's working super well.
But the internal OpenGraph and Twitter Card templates use the "images" param:
https://github.com/gohugoio/hugo/blob/488631fe0abc3667355345c7eb98ba7a2204deb5/tpl/tplimpl/template_embedded.go#L194
Page resources seem like the ideal way to do OpenGraph and Twitter meta tags, too. Does it make sense to migrate the internal templates?
For now I have created my own partials. The image tag looks like this:
{{ with (.Resources.ByType "image").Match "cover" }}{{ range first 6 . }}
  <meta property="og:image" content="{{ .RelPermalink | absURL }}" />
{{ end }}{{ end }}

This depends on me having a "cover" resource, but a more generic version of this would just choose the first image resource on the page.
I think the only challenge here is making the change in a non-breaking way.

Would this be fixed by #5157?

I believe that right now Twitter card template already does what you want, and indeed #5157 will bring the same behavior to the OpenGraph template. The lookup order is:

Explicit images param in a frontmatter
Page bundle images which match *cover* or *thumbnail*
Global Site.Params.images

Seems like the Schema template is missing this too.

@spech66 I'll try to submit a PR with something similar for schema.html in a few days.

@spech66 Done!

Could you also update the OpenGraph template please ?
It would be nice to get some kind of cohesion between these 3 templates.

@TomPichaud I did exactly that! The #5378 will bring this behavior to OpenGraph template, and #5415 to Schema.
Hopefully, someone from maintainers will merge these PRs.

Please get this PR merged. (I really don't understand why this bug/PR is still open.)

Please merge this.  I've locally patched for now.
For me it's not convenience, but to workaround a bug.
absURL from images doesn't work on page-bundled images.
.Permalink from resources does work on page-bundled images

This would be really nice to have,
/cc @bep

bump

Opengraph / Twittercard and Schema still don't generate the propper url when an image is used in a page bundle.

@meredrica Could you provide an example of how you use this feature?
Here is how I tested it: https://github.com/gohugoio/hugo/pull/5157#issuecomment-419701217
And here is an example page bundle: https://github.com/max-arnold/testblog/tree/master/content/articles/post1

I've been running some tests on these since I wasn't getting the correct results (which turned out to be my fault), and it really seems to me that everything is working as intended at the moment according to that https://github.com/gohugoio/hugo/pull/5157#issuecomment-419701217.
@meredrica Here were my mistakes just in case you've done the same thing.

I had a cover.jpg in a page bundle, but it was inside a subfolder, so the path was something like mypost/img/cover.jpg. This doesn't work because this code
https://github.com/gohugoio/hugo/blob/25a6b33693992e8c6d9c35bc1e781ce3e2bca4be/tpl/tplimpl/embedded/templates/opengraph.html#L8
only gets resources from the folder where index.md is. (At least, it did that for me. I'm unclear on if this is what is intended.) So I had to move my image out of the subfolder for the cards templates to automatically find it.
I tried to use a relative URL in the images=[] front matter of my post, but it only accepts absolute URLS.

Perhaps either of my mistakes will help you out.
@max-arnold I actually find it rather frustrating that we can only use absolute URLs for the images array in post front matter. I'd really love to be able to use relative URLs as well since page bundles inherently encourage their use. What would it take to add that functionality? I saw someone on the forum had what looked like a decent suggestion for supporting it here.

I didn't change this behavior in my PRs, and I only brought the existing logic from the TwitterCard template to OpenGraph & Schema templates, so they behave in the same way.
The resulting links should be absolute - I've seen requests from broken bots that can't deal with relative links. Also this https://stackoverflow.com/a/9858694/65523
I'm not sure about the nested resource subfolders, but I believe that the ability to specify relative links in the front matter (and converting them to absolute ones) is useful, because it allows moving page bundles to a different folder without any modifications.
@kaushalmodi Is the approach you described still the best one? I'd be happy to add something linke this to OG/Twitter/Schema templates, but I hope that a more elegant approach does exist.

Is the approach you described still the best one?

Hello, it's been few years since when I suggested that approach! So now I cannot say if that's still the best way.
But it certainly still works because I am using that for my site.

The resulting links should be absolute

Agreed! I was just thinking about relative link options on the front matter creation side. 🙂
I would also counter myself and say that it may not be absolutely necessary to support relative link input in my subfolder case since the quick fix was just to move all of my cover.jpg files into the root of the page bundle where they would be automatically discovered during a build and then used as thumbnail sources. It was a very minor change in my page bundle organization.
However, for the general case of moving page bundles around, relative links would probably help a lot.
@max-arnold I may have a suggestion. I like how @kaushalmodi's / option in images is driven by user input, so it's a clear decision to switch from absolute to relative paths, but perhaps that small detail could be easily missed and cause confusion if users are unaware of its use. To make that more clear, I might suggest throwing in a new variable like relimages or something similar in order to decide on using absolute versus relative paths. Then the various card templates could easily be updated to check for relative URLs and build out the absolute paths using the data if the imagesvariable isn't present.
I'm unsure if this is what you mean by "a more elegant approach", though, since adding variables is not necessarily a good plan. It also seems a little redundant. Just brainstorming over here.

Not being able to use relative image paths in the front matter is a real pain.
It breaks scaffolding if you have a default image in your archetype - I have that, I publish a comic and need to replace the image, obviously.

@razonyang I'm pretty sure this was addressed with https://github.com/gohugoio/hugo/pull/11688. Please confirm.

@jmooring Yes, this can be achieved via the images parameter since v0.121.1.
$ tree content/foo
├── cover1.jpg
├── cover2.jpg
├── cover3.jpg
├── cover4.jpg
├── cover5.jpg
├── cover6.jpg
└── index.md

$ cat content/foo/index.md
---
images:
  - cover1.jpg
  - cover2.jpg
  - cover3.jpg
  - cover4.jpg
  - cover5.jpg
  - cover6.jpg
---

$ cat layouts/_default/single.html
{{ template "_internal/twitter_cards.html" . }}
{{ template "_internal/opengraph.html" . }}
{{ template "_internal/schema.html" . }}

$ hugo

$ cat public/foo/index.html | grep image
<meta name="twitter:card" content="summary_large_image" />
<meta name="twitter:image" content="https://example.org/foo/cover1.jpg" /><meta name="twitter:title" content="111"/>
<meta property="og:url" content="https://example.org/foo/" /><meta property="og:image" content="https://example.org/foo/cover1.jpg" /><meta property="og:image" content="https://example.org/foo/cover2.jpg" /><meta property="og:image" content="https://example.org/foo/cover3.jpg" /><meta property="og:image" content="https://example.org/foo/cover4.jpg" /><meta property="og:image" content="https://example.org/foo/cover5.jpg" /><meta property="og:image" content="https://example.org/foo/cover6.jpg" /><meta property="article:section" content="" />
<meta itemprop="wordCount" content="0"><meta itemprop="image" content="https://example.org/foo/cover1.jpg" /><meta itemprop="image" content="https://example.org/foo/cover2.jpg" /><meta itemprop="image" content="https://example.org/foo/cover3.jpg" /><meta itemprop="image" content="https://example.org/foo/cover4.jpg" /><meta itemprop="image" content="https://example.org/foo/cover5.jpg" /><meta itemprop="image" content="https://example.org/foo/cover6.jpg" />


Handle touch based devices

Handled in https://github.com/tschf/2016ADCEntry/commit/3247012c700f43706ccbd487ac3a3216be89c5d0


in survival server

Please add a mode that does not require op
After typing $play
Prompt number of note blocks
and the number of blocks to adjust the tone in chat room
I need to use it in servival servers

same.

I added this feature on my fork of SongPlayer if you want to give it a try

@Sk8kman thanks for your work ! ! !

@Sk8kman would you care to update the fork to 1.20.4? or either make a pull request??

@flaryx32 my fork should already work with 1.20.4

Added a "survival only" mode that requires you to place the required instruments for a song within reach distance manually, then tunes them automatically when you play the song.


chore: configure GitHub Codespaces for this project

Description
So: I was really bored yesterday and decided to learn the GitHub Codespaces mechanism to set up a Cloud dev IDE. These are pretty cool - a whole development environment set up automatically in a container, accessible from a browser. AFAIK works fine; the README badge will become active upon merge to dev branch (currently configuration files are not there yet). In order to test it yourself go to repo page, switch to codespaces branch click green button Code, then Codespaces and another green button to create a codespace.
Type of change

[x] Other

Checklist
Please carefully read these items and tick them off if the statements are true
or do not apply.

[x] I have performed a self-review of my own code
[ ] ~My code follows the existing coding style, lints and generates no new
warnings~
[ ] ~I have added type annotations to all function/method signatures, and I
have added type annotations for any local variables that are non-trivial,
potentially ambiguous or might otherwise benefit from explicit typing.~
[x] I have commented my code in hard-to-understand areas
[ ] ~I have added ["Google-style docstrings"] to all new modules, classes,
methods/functions or updated previously existing ones~
[ ] ~I have added tests that prove my fix is effective or that my feature
works~
[ ] ~New and existing unit tests pass locally with my changes and I have not
reduced the code coverage relative to the previous state~
[ ] ~I have updated any sections of the app's documentation that are affected
by the proposed changes~

If for some reason you are unable to tick off all boxes, please leave a
comment explaining the issue you are facing so that we can work on it
together.

Org admin needs to adjust the Codespace billing settings so that the org does not pay for everyone who creates Codespaces (see below), rather that you should enforce "Individual billing", i.e. every user pays up from their pool of free codespace minutes (similarly as for ci).


Docs 860 part1: max policy document size, webhook metrics

Docs updates for two items from MinIO RELEASE.2023-05-18T00-05-36Z:

Max policy size of 2KiB for service accounts
I think this change now means json policy documents have the same max size in all cases. The limit wasn't documented previously. This change adds max size info throughout, not only for service accounts.
Webhook usage metrics

Staged:
http://<IP_ADDRESS>:9000/staging/DOCS-860-part1/linux/html/reference/minio-mc-admin/mc-admin-user-svcacct-add.html
http://<IP_ADDRESS>:9000/staging/DOCS-860-part1/linux/html/reference/minio-mc-admin/mc-admin-user-svcacct-edit.html
http://<IP_ADDRESS>:9000/staging/DOCS-860-part1/linux/html/reference/minio-mc-admin/mc-admin-policy-create.html
http://<IP_ADDRESS>:9000/staging/DOCS-860-part1/linux/html/administration/identity-access-management/policy-based-access-control.html#policy-document-structure
Partly addresses https://github.com/minio/docs/issues/860

Nothing to add beyond what Daryl did, once he clears this is :shipit:


widget for Radio Station

maybe for Bose only:
create a widget for all Presets and add the radio station img from object.

maybe for Bose only:

Why?

Because I do not know, if other smart speakers offer similar funcionality.
I can implement it for any speaker, if somebody gives mit dem appropriate DP for them ;)

Because I do not know, if other smart speakers offer similar funcionality.
I can implement it for any speaker, if somebody gives mit dem appropriate DP for them ;)

{   "from": "system.adapter.musiccast.0",   "user": "system.user.admin",   "ts":<PHONE_NUMBER>550,   "common": {     "name": "netusb favourites",     "type": "array",     "role": "list",     "desc": "netusb favourites",     "read": true,     "write": false   },   "native": {},   "acl": {     "object": 1636,     "owner": "system.user.admin",     "ownerGroup": "system.group.administrator",     "state": 1636   },   "_id": "musiccast.0.RX-V481_08125F03.netusb.preset_info",   "type": "state" }

thx


[0.9] this.resolvePath() missing in 0.8/0.9

https://www.polymer-project.org/0.5/docs/polymer/polymer.html#resolvepath

@sorvell @sjmiles Is this or something like this destined to return?
There appears to be a resolveUrl in 0.9, but it doesn't appear to do the same thing.

The intent is that resolveUrl(url) is the replacement for 0.5's resolvePath.
@arthurevans Why do you say it does something different?

I reproduced the example described in the docs:
https://www.polymer-project.org/0.5/docs/polymer/polymer.html#resolvepath
index.html
bower_components/
my-thing/
my-thing.html
foo.png
index.html contains:
<doctype html>
<html>
  <head>
    <link rel="import" href="bower_components/my-thing/my-thing.html">
  </head>
  <body>
    <my-thing>
    </my-thing>
    <script>
      var thing = document.querySelector("my-thing");
      alert(thing.resolveUrl("foo.png"));
    </script>
  </body>
</html>

Alert says: "foo.png".
Expected alert to say: "bower_components/my-thing/foo.png".

Grr. GH won't let me edit my comment properly. Tree should look like this:
index.html
bower_components/
  my-thing/
    my-thing.html
    foo.png

Does my-thing have a dom-module? That's now a requirement.

Gah. It had a dom module ... with a typo.
Fixed typo, now get correct information.
Le sigh. Now just need to document this little hiccup.

This issue was moved to Polymer/docs#1162


Included Nerd4ever Kaya Stubs

Stubs to add Nerd4ever / Kaya proprietary extension support for php
#WI-64232 Inclusion of "non-standard" extensions Nerd4ever/Kaya
https://youtrack.jetbrains.com/issue/WI-64232

Are there any usage statistics  available for this extension? Any installation instructions?

Sorry, if this extension isn't publically available, then there's no point including it for everyone in PhpStorm stubs,
Please let us know the previously requested info about the extension.

I have to close this PR for now. We can reconsider it if you could provide us representative package usage statistics.


GH-41813: [C++] Fix avx2 gather offset larger than 2GB in CompareColumnsToRows

Rationale for this change
AVX2 intrinsics _mm256_i32gather_epi32/_mm256_i32gather_epi64 are used in CompareColumnsToRows API, and treat the vindex as signed integer. In our row table implementation, we use uint32_t to represent the offset within the row table. When a offset is larger than (0x80000000, or 2GB), the aforementioned intrinsics will treat it as negative offset and gather the data from undesired address. More details please see https://github.com/apache/arrow/issues/41813#issuecomment-2168286330.
Considering there is no unsigned-32bit-offset or 64bit-offset counterparts of those intrinsics in AVX2, this issue can be simply mitigated by translating the base address and the offset:
new_base = base + 0x80000000;
new_offset = offset - 0x80000000;

What changes are included in this PR?
Fix and UT that reproduces the issue.

Are these changes tested?
UT included.

Are there any user-facing changes?
None.

GitHub Issue: #41813

cc @pitrou @amoeba @mrd0ll4r

Hi @zanmato1984, thanks for your work on this. I'm hoping others can review the implementation but I did just check that the new test passes (it does) and also fixes the original issue (it does). :+1:

Hi @zanmato1984, thanks for your work on this. I'm hoping others can review the implementation but I did just check that the new test passes (it does) and also fixes the original issue (it does). 👍

Thank you @amoeba for verifying, and the help on reproducing the issue!

I can't give much feedback or test this out, unfortunately. But I'm very thankful for you all looking into this!

Ran into this issue when I was debugging my own issue where running a group_by/aggregate on a table with null columns was failing to group some keys, i.e. some group key value tuples were duplicated in the result.
Why I mention it here
:

My investigation also pointed to a problem with CompareColumnsToRows, but I don't fully understand the code.
Disabling AVX2 (i.e. `ARROW_USER_SIMD_LEVEL=AVX) solved the issue

However, the table I use is only 3.8MB - so perhaps there is some other bug around the AVX2 related code here as well, unrelated to the size, but related to nulls.

Hi @FreekPaans, can you please open a new issue for that? I think the issue will be fixed in the upcoming 17.x PyArrow release but it'd be good to make sure.

@amoeba sure thing https://github.com/apache/arrow/issues/42231
Any issue/PR you can point me to for where it's fixed?

@pitrou @felipecrv @ZhangHuiGui @mapleFU Would you please help to take a look? Thanks.

I've committed another change containing only code restructures (moving, renaming, etc.) to the test, to make the test logic more clear and readable. Hope it doesn't trouble your review @pitrou . Thanks.

Thanks a lot @zanmato1984 !


code CLI still opens duplicate VSCode instance in the macOS dock

Issue Type: Bug
This is a re-file of https://github.com/microsoft/vscode/issues/60579, which does not appear to be fixed for me in the November VSCode release.
VSCode is still opening processes with duplicate dock icons (and therefore slowing down the use of code to open files). Judging from the issue and the accompanying PR, it looks like I'm not the only one who is still experiencing this issue. Unfortunately, the other issues and PRs are closed and locked, so I'm filing this new one.
Simple repro: code new-file.txt
One workaround from the other issue was to use open -b com.microsoft.VSCode. However, this has some challenges:

The open command completely fails on macOS if passed any file that does not exist yet. This makes the raw command useless for creating new files from the commandline. 😔

open -b com.microsoft.VSCode new-file outputs: The file /Users/lgarron/new-file does not exist.

It is not obvious how to craft a shell function wrapper that properly opens files, can handle multiple paths (files or folders), and isn't subject to edge cases and injections (e.g. file names that will be confused for arguments to open).

open -b com.microsoft.VSCode --args new-file does nothing, outputs nothing, and exits without an error. Leaving out --args is unsafe (when the file name is not typed by hand and known not to conflict with an argument to open) but doesn't work at all.

Using open a wrapper for as a workaround for code breaks non-file arguments — even code --help.
In addition to setting an alias/function for code in your shell, you will need to set up $EDITOR separately (at least in some shells).

A proper fix would be really appreciated.
 May be duplicate of #138712

Do you see same issue if you disable all extensions? May be duplicate of #138712

Yes, code --disable-extensions successfully disables extensions but does not address this issue. 😢

I also just realized that opening the editor for things like git commit was super slow. set EDITOR "open -b com.microsoft.VSCode" has helped for that, but I'm worried it will break in other situations.

I just added set EDITOR "open -b com.microsoft.VSCode" and that's a lot faster, but I'm worried it will break in other situations.

... aaaaand:

😔

All extensions are disabled and the new icon is still bouncing when a file is opened with the terminal in the same workspace. disappears in a few seconds

I had the same problem and I use the "Extension Bisect" feature to find the culprit.
In my case it was the extension "Spell Right". I'm posting an issue on their repo.

What would be the best way to troubleshoot this? I just started seeing this behavior (double dock icons) the last few weeks, perhaps after an update to an extension or vs code itself. I tried code --disable-extensions . as in https://github.com/microsoft/vscode/issues/139634#issuecomment-999541635, but the issue persists.

What would be the best way to troubleshoot this? I just started seeing this behavior (double dock icons) the last few weeks, perhaps after an update to an extension or vs code itself. I tried code --disable-extensions . as in https://github.com/microsoft/vscode/issues/139634#issuecomment-999541635, but the issue persists.

As I said just before, try using the "Extension Bisect" feature. You can start it in the Extensions panel, with the "..." menu.
More information: https://code.visualstudio.com/blogs/2021/02/16/extension-bisect

Thanks @Drarig29 ! I had expected --disable-extensions to stop all extensions from running and rule out the possibility of being extension related, but that seems to not be the case. I ended up having the same extension as the culprit.

In addition to setting an alias/function for code in your shell, you will need to set up $EDITOR separately (at least in some shells).

I've tried to debug this a bunch of times, and concluded that it's impractical (or maybe even impossible) to maintain enough workarounds for this. I use fish, and all the combinations of abbreviations / $EDITOR / $VISUAL just don't seem to work for the "edit commandline with external editor "functionality.
If I want to use the workaround for the slow duplicate VSCode instance (which still happens for me), I inevitably have to leave other things broken right now. 😕

VSCode has been frustratingly slow to load[^1] for me, and I've been having issues pinning it down, since it occurs even for code --disable-extensions. I now believe the root cause is related to this issue. :-/
[^1] Slow to load, slow to create files, slow to open files. Once a file is open, things are fast, though.

This isn't a direct solution to this, but if you don't use the recent apps section of the dock, you can disable it: https://www.howtogeek.com/714379/how-to-hide-recent-apps-from-the-mac-dock/#:~:text=In System Preferences%2C select “Dock,will disappear from your Dock.

This isn't a direct solution to this, but if you don't use the recent apps section of the dock, you can disable it: https://www.howtogeek.com/714379/how-to-hide-recent-apps-from-the-mac-dock/#:~:text=In System Preferences%2C select “Dock,will disappear from your Dock.

For what it's worth, I've personally had the recent apps section disabled for a long time, and it seems to be unrelated to this issue (which is still happening, and I've just learned to live with the slight slowdown and extra visual distraction of using the original code binary as my editor so that I don't have to deal with the bugs and security issues of the workaround alternatives).

I can replicate this with zero extensions enabled, as long as Code is running.
In other words:

run code . from Terminal
Only pinned Dock icon bounces.
work on that window for a bit, then realize "oh I need to work on another project"
use the Terminal to navigate to the new project
run code .
duplicate icon appears

@plttn I'm starting seeing a similar issue.  Furthermore, the more times I execute code-insiders . the more VSCode icons I'll see in the Dock.  This issue always start with the editor window not resizing to the full size of the monitor.

Also seeing this issue.

I solved it on my machine with this:
alias code="open -b com.microsoft.VSCode"
Not really an actual solution. But this was annoying to me.

Here's a recent error report and a picture of what I'm seeing on the screen when I attempt to make the image full-screen.
error report:
https://gist.github.com/conradwt/4fcdc1f3447ea3306923cdccf7e98191

I confirm the issue persists using latest. Besides, the open workaround fails to jump to a specific line as --args is ignored:
/usr/bin/open -b com.microsoft.VSCode --args --goto myFile.ext:10

Which I need for Run Command on iTerm's Semantic History: /usr/bin/open -b com.microsoft.VSCode --args --goto \1:\2
Using:
Version: 1.85.1 (Universal)
Commit: 0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2
Date: 2023-12-13T09:48:06.308Z
Electron: 25.9.7
ElectronBuildId: 25551756
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 23.2.0

Yeah,the work around is lame, it broke my arguments from working also.
It would be nice to not need the work around so the regular flags worked.

After upgrading to macOS Sonoma, all the VSCode - Insider issues that I reported went away for me.

I'm using macOS Sonoma 14.2.1 and I confirm the issue persists using the latest regular (non-Insider) VS Code.
Besides, the open workaround fails to jump to a specific line as --args is ignored:
open -b com.microsoft.VSCode --args --goto myFile.ext:10

Which I need for Run Command on iTerm's Semantic History:
/usr/bin/open -b com.microsoft.VSCode --args --goto \1:\2

Using:
Version: 1.85.1 (Universal)
Commit: 0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2
Date: 2023-12-13T09:48:06.308Z
Electron: 25.9.7
ElectronBuildId: 25551756
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 23.2.0

On MacOS Sonoma 14.2.1, seeing multiple instances of VSCode when using the code cli.
The open workaround fails to parse arguments properly (you can check by trying: open -b com.microsoft.VSCode --args --new-window doesn't work at all.

In your Mac Dock Settings, disable the recent apps in the dock and it will work just fine 👍

In your Mac Dock Settings, disable the recent apps in the dock and it will work just fine 👍

I've had recent apps disabled in the dock for a long time. It didn't help for me a year ago and still doesn't help now, unfortunately. The bug still persists.


Cannot create a product without the "net" value, even if tax exists.

Description
When creating a product, the net-value is required even if there is a tax associated with it.
Possible Solution
Make the 'net' attribute optional if the tax is given.
Additional context
Assume this is the operation for a new product.

If the tax i in the payload, why does the net value needs to be set?

 $operation = new SyncOperation(
            'write',
            'product',
            SyncOperation::ACTION_UPSERT,
            [
                [
                    'name' => 'Testproduct',
                    'productNumber' => Uuid::randomHex(),
                    'stock' => 1,
                    'tax' => ['name' => 'tax reduced', 'taxRate' => 16],
                    'manufacturer' => ['name' => 'test'],
                    'price' => [['currencyId' => Defaults::CURRENCY, 'gross' => 10, 'net' => 0, 'linked' => true]],
                ],
            ],
            PlatformRequest::API_VERSION
        );

Hello,
Within our company roadmap and work capacity, we try to address each bug or improvement request but admit that not each one will be resolved. To continue our culture of honesty and openness, we are closing this issue to focus on our roadmap on behalf of all Shopware users.
If you feel like this issue is still impacting you, please create a new issue and let us know.
Thank you again for your valuable feedback.


Mixin apply for mod tooltipfix failed

Doesn't create a crash report, but here's the latest.log: https://gist.github.com/33903f1d2dba2a67fdfc4bcf91a1784d
Using tooltipfix-1.1.1-1.19.3

Looks like you're running Minecraft 1.19.2. The tooltipfix you are using is meant for 1.19.3. Please try the 1.19.2 version


Seek on LIVE seek-bar not updates

On LIVE Stream if I seek using this API then seek-bar is not updated, but video works as it should
player.core.mediaControl.container.playback.seekSeconds(35)

If I seek using
player.seek(5);

Then seek-bar updates correctly but problem that this accepts only percentage.

Probably related https://github.com/clappr/clappr/issues/668

https://youtu.be/JbOemVHrJh0

this should be fixed in https://github.com/clappr/clappr/commit/e663199c6f21802bd2e11557d583fa4bae6fe2b2

@FDiskas can you check this again? I think this is fixed

Now its good.


Consider adding a full example

Hello and thanks for this project.
I think it would be more interesting if potential users are shown an example (or use case). I was able to create simple link previews with a hover effect like this:

I made some tweaks to your talk.jekyllrb.com forum reply. This file is called _includes/link_preview.html
{% comment %}See https://talk.jekyllrb.com/t/embedding-opengraph-links-in-posts/8124/6{% endcomment %}
{% assign meta = include.url | metadata %}

  {% if meta['og:image'] and meta['og:image'] != blank %}

  {% else %}
  {% comment %}Add a fallback image here, if the opengraph image for given url does not exist. See https://upload.wikimedia.org/wikipedia/commons/f/f5/Antu_edit-image-face-recognize.svg Elegant Alternative Suite for Plasma 5. Antü Plasma-KDE Theme are licensed under the CREATIVE COMMONS BY-SA 3.0.{% endcomment %}

  {% endif %}

{{ meta['title'] | truncate: site.link_preview.title_max_characters }}

{{ meta['og:description'] | truncate: site.link_preview.description_max_characters }}

I created a separate CSS file called _sass/link-preview.scss that is imported by the main one
.link-preview-box {
    width: 100%;
    border: none;
    padding: 0px;
    margin-top: 8px;
    margin-bottom: 8px;
}

.link-preview-box a {
    font-family: sans-serif;
    border-radius: 4px;
    color: black;
    text-decoration: none;
    background-color: #dddddd;
    padding: 8px;
    display: block;
    top: 0;
    left: 0;
    bottom: 0;
    right: 0;
}

.link-preview-box a:hover {
    background-color: #555;
    color: white;
}

.link-preview-box img {
    width: auto;
    height: auto;
    max-width: 512px;
    max-height: 64px;
}

.link-preview-title {
    font-size: 18px;
}

.link-preview-description {
    font-size: 14px;
    margin: 0;
}

Finally, add this to the _config.yml file
link_preview:
  title_max_characters: 64
  description_max_characters: 128

Links are added in the same way you do
{% include link_preview.html url="https://git-lfs.com/" %}

Hi @frnmst, thanks for suggesting! It would be helpful if there were a working example in the example site.
Would you like to contribute your implementation on the example page? Else it would time some time to write something of my own because of other commitments.

Thank you! The example is now live on the site.


[offload][OMPT] Fix CMake build-dir residue by cleaning out-of-tree

Leftover files from former builds could lead to false test results.
This adds out-of-tree file removal using 'git clean -fdx'

Thanks @ddpagan for reporting and the investigation! :)


Pipeline JSON Format Change - Version Next

Describe the issue
These are some of the changes we might consider when bumping the pipeline version:
primary_pipeline[app_data][source] -> primary_pipeline[app_data][filename] -> Consistency with node format
"Notebooks" component id -> "notebook" component id

For the next version bump I think we should move all the node properties from directly in app_data to a properties object inside of app_data.
from this:
"app_data": {
  "filename": "load_data.ipynb",
  "runtime_image": "amancevice/pandas:1.0.3",
  "ui_data": {
    "label": "Load Data",
    "image": "data:image/xxx",
    "x_pos": 50,
    "y_pos": 100,
    "description": "Notebook file"
  }
}

to this:
"app_data": {
  "properties": {
    "filename": "load_data.ipynb",
    "runtime_image": "amancevice/pandas:1.0.3"
  },
  "ui_data": {
    "label": "Load Data",
    "image": "data:image/xxx",
    "x_pos": 50,
    "y_pos": 100,
    "description": "Notebook file"
  }
}

description should  be a child of properties because it's not a UI property.

@ptitzler label and description are required to be in ui_data by canvas. (This description is different than the user editable description)

Ah yes, okay. I thought this was was the user-provided value. Never mind.

@bourdakos1 suggestion is already being done as part of #1882


chore(main): release 1.2.1

:robot: I have created a release beep boop
1.2.1 (2024-09-05)
Bug Fixes

fix goreleaser brew stage (342ffec)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.2.1
:sunflower:


Bump operator-manifest-tools dependency

generate bundle now specifies default auth when pulling images

this means operator-sdk will default to whatever auth docker is using on the system already (i.e. if the user has run docker login, it will use those creds).
Closes #6027
Closes #6095

Closing and reopening to kick off the CI again and pull in the ansible CI fixes


Question: Keeping a constant speed

Hi,
I am trying to simulate a miles-in-trail scenario and would like the aircraft to hold a constant speed, but this doesn't seem to happen, as the speed decreases with time. Is there any way I could achieve this? I'm also attaching a simple scenario to recreate this.
`00:00:00>cre aclmg1 B744  45.815917 1.025583<PHONE_NUMBER>0930185 25000 476
00:00:00>cre actou1 B744  43.680833 1.309806 54.50122318892586 25000 476
00:00:00>schedule 110,cre aclmg2 B744  45.815917 1.025583<PHONE_NUMBER>0930185 25000 476
00:00:00>schedule 110,cre actou2 B744  43.680833 1.309806 54.50122318892586 25000 476
00:00:00> trail on
00:00:00> asas off`
Thank you!

Hi!
You are creating an aircraft at an airspeed that lies outside of its flight envelope (476 kts CAS at FL250 is > 1 Mach!)
If you use a reasonable speed it should remain constant after creation.


bug: preact watch with --esm flag will crash to shell if syntax error is saved in any file being watched

Do you want to request a feature or report a bug?
Report a bug.
What is the current behaviour?
local build being watched errors out and preact watch command has to be run again
If the current behaviour is a bug, please provide the steps to reproduce.

Open any file in the project being watched.
Add a syntax error to the file.
Save / autosave the change.

What is the expected behaviour?
local build being watched displays stack, but doesn't error out of build, and then recovers after syntax error is fixed.
If this is a feature request, what is motivation or use case for changing the behaviour?
Please mention other relevant information.

Node version
npm version
Operating system
CLI version
Browser

@shoojx please fill out relevant information, after that @prateekbh could you please have a look?

I am out till Sunday, will look once m back

I can see the error stack with --esm https://pastebin.com/Z074d4bS
Fixing these in my editor also brings everything back on its track.
Is there a specific kind of bug which crashes it?

unable to repro on 3.0.0-rc.1, was happening on an older version

closing this for now


Top-up feature

Hi, I wonder if it would be possible to have an ability to top-up (charge to 100%), but still keep the original maintenance setting. Currently, if I set battery maintain 80 and before business trip I do battery charge 100, it removes the 80% maintenance level. It'd be ideal if that maintenance level is kept, so that I don't forget about it.

This can be achieved with bash:
battery charge 100 && battery maintain 80

The second happens after the first.


Fix return value of subscribed_task_stats

The method subscribed_task_stats wraps almost all methods in RakeHelper but it has a bug.
For instance, the upgrade method in the same module is supposed to return a list of changed indexes in https://github.com/toptal/chewy/blob/7d567542c7685cf44a52050b8d16e61c373cd186/lib/chewy/rake_helper.rb#L65-L85
However, the wrapping method isn't returning to the caller the value of changed_indexes. Instead, it returns the result of the method output.puts which is nil.
I'm simply moving output.puts to be called within ensure so the output of the method correctly returns the result of:
ActiveSupport::Notifications.subscribed(Chewy::RakeHelper::JOURNAL_CALLBACK.curry[output], "apply_journal.chewy") do
    ActiveSupport::Notifications.subscribed(Chewy::RakeHelper::IMPORT_CALLBACK.curry[output], "import_objects.chewy", &block)
end

that would resolve to the correct value in the end when &block is evaluated.

Before submitting the PR make sure the following are checked:

[ ] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[ ] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[ ] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the changelog if the new code introduces user-observable changes. See changelog entry format for details.

hey, @fabiormoura thanks for the fix, PR looks good, but, could you please provide a spec for the subscribed_task_stats method? Thanks!

Also, please provide a changelog entry, so that will be easier to generate a release (example)

Also, please provide a changelog entry, so that will be easier to generate a release (example)

Thanks for the suggestions @konalegi. I made the changes which have been amended.

@fabiormoura could you please fix rubocop offenses as well? Thanks

@fabiormoura could you please fix rubocop offenses as well? Thanks

it's been fixed.

Merged, going to release it on Monday. Thanks for PR :)


Fix code scanning alert - Multiplication result converted to larger type

Tracking issue for:

[ ] https://github.com/n-west/bliss/security/code-scanning/1

I think the right fix for this is to not worry about the types but implement the kahan sum for cpu (we do that for gpu and it makes a nice difference)

Fixed in #80


MacOS M1, Python 3.9: fails as unsupported

Describe the bug*
A clear and concise description of what the bug is.
It seems like your platform or Python version are not supported yet.
Windows, Linux, macOS and Python >= 3.7 (3.9 for M1) are currently supported.
Please submit a github issue if you believe Reloadium should be working on your system at

To Reproduce
Steps to reproduce the behavior:

Click on Debug of python configuration for test run

Expected behavior
Should work?
Desktop or remote (please complete the following information):**

OS: MacOS
OS version: Darwin Kernel Version 22.2.0: Fri Nov 11 02:03:51 PST 2022; root:xnu-8792.61.2~4/RELEASE_ARM64_T6000 arm64
M1 chip: yes
PyCharm plugin version: 0.8.11
Editor: Build #PY-223.8214.51, built on December 20, 2022
Python Version: Python 3.9.16 (main, Jan 12 2023, 23:52:36)  [Clang 14.0.0 (clang-14<IP_ADDRESS>)] on darwin
Python Architecture: x86_64
Run mode: Debug

Additional context
Just noticed I actually installed https://plugins.jetbrains.com/plugin/20302-you-com-with-reloadium--hot-reloading--profiling-for-python, not clear https://plugins.jetbrains.com/plugin/18509-reloadium--hot-reloading--profiling-for-python if it matters.

Please install https://plugins.jetbrains.com/plugin/18509-reloadium--hot-reloading--profiling-for-python You.com with reloadium is a fork which is not maintained by me.


Feature request: Support for "Next January"...

I tried "@next January" but it returned January 1st of the current year instead of January 1st of next year.
It would be great if you could add this type of dates.

hm I just tried this and it worked as expected - returns 1 Jan 2023.

I just tried it as well and I also get the right answer.
Could it be related to date math ? I posted my previous message on June 11th which is closer to January 2022 than January 2023.

Hi, I have the same issue. Today on june 1, if i input "dec 1", i get december 22. If i input "nov 1" i get november 23. If i input jan-jun 1 i also get current year. even if i put "next january" i get jan 22. I am in germany with correct time set on windows. i have tried changing the start of the week and reinstalling the plugin.
I get that some people need to refer to dates in the past but i only want it to output dates in the future. is there something that i can do?


Highlight AS where not all peering sessions are configured

Issue type
[x] Feature request
[ ] Bug report
[ ] Documentation

Environment

Python version:  3.6
Peering Manager version:  master

Description
Right now it is complicated to see which ASN has not configured all to the IX connected sessions.

Therefore, I would recommend to add some kind of notification or state attribute which can be filtered and provides you a fast view that there is work to do for this AS.
I think the Autonomous System table would fit good for that information.

We could either highlight the row in the AS table using a special color or add a new column with an indicator.
Either way, it's likely that there will be a performance impact if the function to check if sessions are missing is not well designed. So we will need to write a function that will need to make the less SQL queries possible.

What about adding a filter.
This way the check only runs in case you really want to see it.

We cannot look for this in a filter because filters use table fields for matching data. This kind of details are always computed and thus not stored in the database. The only way is to work on an optimized algorithm or find a way to run the algorithm only on demand.
I made a proof of concept patch (see below). With a list of about 25 autonomous systems, the time to check every AS is acceptable, but it would require some testing with more AS objects.
diff --git a/peering/models.py b/peering/models.py
index 2d5cf21..1b24648 100644
--- a/peering/models.py
+++ b/peering/models.py
@@ -11,30 +11,7 @@ from django.urls import reverse
 from django.utils import timezone
 from django.utils.safestring import mark_safe

-from .constants import (
-    BGP_RELATIONSHIP_CHOICES,
-    BGP_RELATIONSHIP_CUSTOMER,
-    BGP_RELATIONSHIP_PRIVATE_PEERING,
-    BGP_RELATIONSHIP_TRANSIT_PROVIDER,
-    BGP_STATE_CHOICES,
-    BGP_STATE_IDLE,
-    BGP_STATE_CONNECT,
-    BGP_STATE_ACTIVE,
-    BGP_STATE_OPENSENT,
-    BGP_STATE_OPENCONFIRM,
-    BGP_STATE_ESTABLISHED,
-    COMMUNITY_TYPE_CHOICES,
-    COMMUNITY_TYPE_EGRESS,
-    COMMUNITY_TYPE_INGRESS,
-    PLATFORM_CHOICES,
-    PLATFORM_EOS,
-    PLATFORM_IOS,
-    PLATFORM_IOSXR,
-    PLATFORM_JUNOS,
-    ROUTING_POLICY_TYPE_CHOICES,
-    ROUTING_POLICY_TYPE_EXPORT,
-    ROUTING_POLICY_TYPE_IMPORT,
-)
+from .constants import *
 from .fields import ASNField, CommunityField
 from peeringdb.api import PeeringDB
 from peeringdb.models import NetworkIXLAN, PeerRecord
@@ -133,42 +110,59 @@ class AutonomousSystem(ChangeLoggedModel):

         # Get common IX networks between us and this AS
         common = PeeringDB().get_common_ix_networks_for_asns(settings.MY_ASN, self.asn)
-
-        # Divide sessions based on the IP versions
-        ipv6_sessions = []
-        ipv4_sessions = []
+        missing_peering_sessions = []

         # For each common networks take a look at it
         for us, peer in common:
             # We only care about networks matching the IX we want
             if us.id == internet_exchange.peeringdb_id:
-                # Check on the IPv6 address
+                peering_sessions = []
+
                 if peer.ipaddr6:
                     try:
-                        ip_address = ipaddress.IPv6Address(peer.ipaddr6)
-                        if not InternetExchangePeeringSession.does_exist(
-                            internet_exchange=internet_exchange,
-                            autonomous_system=self,
-                            ip_address=str(ip_address),
-                        ):
-                            ipv6_sessions.append(ip_address)
+                        peering_sessions.append(
+                            str(ipaddress.IPv6Address(peer.ipaddr6))
+                        )
                     except ipaddress.AddressValueError:
                         continue
-
-                # Check on the IPv4 address
                 if peer.ipaddr4:
                     try:
-                        ip_address = ipaddress.IPv4Address(peer.ipaddr4)
-                        if not InternetExchangePeeringSession.does_exist(
-                            internet_exchange=internet_exchange,
-                            autonomous_system=self,
-                            ip_address=str(ip_address),
-                        ):
-                            ipv4_sessions.append(ip_address)
+                        peering_sessions.append(
+                            str(ipaddress.IPv4Address(peer.ipaddr4))
+                        )
                     except ipaddress.AddressValueError:
                         continue

-        return (ipv6_sessions, ipv4_sessions)
+                # Get all known sessions for this AS on the given IX
+                known_sessions = InternetExchangePeeringSession.objects.filter(
+                    internet_exchange=internet_exchange,
+                    autonomous_system=self,
+                    ip_address__in=peering_sessions,
+                )
+                # Check if peer IP addresses are known sessions
+                for peering_session in peering_sessions:
+                    # Consider the IP as not known at first
+                    known = False
+                    for known_session in known_sessions:
+                        if peering_session == known_session.ip_address:
+                            # If the IP is found, stop looking for the info and mark
+                            # it as known
+                            known = True
+                            break
+                    # If the IP address is used in any peering sessions append it,
+                    # keep an eye on it
+                    if not known:
+                        missing_peering_sessions.append(peering_session)
+
+        return missing_peering_sessions
+
+    def has_missing_peering_sessions(self):
+        # Get missing peering sessions and count them
+        for internet_exchange in self.get_common_internet_exchanges():
+            # Return true as soon as we found a missing sessions
+            if len(self.get_missing_peering_sessions(internet_exchange)) > 0:
+                return True
+        return False

     def sync_with_peeringdb(self):
         """
diff --git a/peering/tables.py b/peering/tables.py
index 0d579f5..f34f0f1 100644
--- a/peering/tables.py
+++ b/peering/tables.py
@@ -64,6 +64,11 @@ class AutonomousSystemTable(BaseTable):
             "ipv4_max_prefixes",
             "actions",
         )
+        row_attrs = {
+            "class": lambda record: "table-warning"
+            if record.has_missing_peering_sessions()
+            else ""
+        }

 class CommunityTable(BaseTable):


Base 64 image in html editor Field

trying to embed a base 64 image in HtmleditorField causes the page to hang on save.

Fixed in ss4 with tinymce 4.


Give an example how to give user rights with MDM Policy CSP

With Windows 10 1803 you can grant user rights with MDM Policy CSP. The documentation is missing what is the correct syntax to grant user rights to specific users/groups. Because MDM policy's are used in cloud-only environments, the examples should include how you can grant the user rights to Azure AD users/groups.

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

ID: 3b62d452-1910-d962-0ccc-9ac3f8135f15
Version Independent ID: 5a4e6f0c-bcd4-8d8b-93f7-5bda7135b931
Content: Policy CSP - UserRights
Content Source: windows/client-management/mdm/policy-csp-userrights.md
Service: unspecified
GitHub Login: @nickbrower
Microsoft Alias: maricia

Would really like some examples here, what is the CSP expecting User SIDs? in what format. I cannot consume these CSPs with the information provided.

Added examples and link to well known SIDs.

Your examples seem to be flawed.  I can't get this to work using the SIDs.  Are you sure this is the correct syntax?

Hi Patti,
What should I do to find the correct syntax?
Thanks,
Heidi Lohr
Writer
Studio C
Phone (Skype): +1<PHONE_NUMBER> (Ext. 34140)

From: Patti Short<EMAIL_ADDRESS>Sent: Wednesday, September 5, 2018 11:22:05 AM
To: MicrosoftDocs/windows-itpro-docs
Cc: Heidi Lohr; Assign
Subject: Re: [MicrosoftDocs/windows-itpro-docs] Give an example how to give user rights with MDM Policy CSP (#856)
I will help resolve this one 👍
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-itpro-docs%2Fissues%2F856%23issuecomment-418831039&data=02|01|Heidi.Lohr%40microsoft.com|ac82874db2ed41919c2808d6135c7c42|72f988bf86f141af91ab2d7cd011db47|1|0|636717685276931468&sdata=O2IQbraYNCEb0ekDJf8FyvZdnTT9wSn%2FUpUC3my2oe0%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAfioBarX2fXjB-2TC8PdlosUNEQ5dWGOks5uYBZNgaJpZM4TyjPH&data=02|01|Heidi.Lohr%40microsoft.com|ac82874db2ed41919c2808d6135c7c42|72f988bf86f141af91ab2d7cd011db47|1|0|636717685276941482&sdata=xOlVg36Twn7ZsYVo5eHpOT7dvhyIK3NFXOUdN95Z62M%3D&reserved=0.

@dan529 The dev team informed me that this feature is only supported in the next major Windows release, and you won't be able to use it before then unless you're using an Insider build.

So you saying this will only work with 1809??  Its working on 1803 currently but only one group added as a String (example Administrators)  but if 2 groups are needed on the same Settings it doesn't work.  Already tried with "name1;name2" "name1; name2" "name1,name2" "name1, name2" "Name1Name2" "Name1Name2"  can someone provide more info on how to get the these settings working?? Thanks

Anyone following this issue?

Hey MS, just tell these people the truth that everything in this article doesn't work!
Here's what I did to get it to work.

The separator for the values like "Remote Desktop Users" and "Administrators" is 

Will that work as "Remote Desktop UsersAdministrator" ?
-Absolutely NOT!

Go to https://coderstoolbox.net/string/#!encoding=xml&action=decode&charset=us_ascii
-Set string conversion to XML, Decode, and US-ASCII
-Input = 
-This will spit back a rectangle box in the Output
-Copy that rectangle box and put it in the middle as the separator -  Remote Desktop UsersAdministrators

DO NOT COPY FROM HERE; IT WILL NOT WORK!!!

So here's the kicker!
-USE EDGE Browser.  It WILL NOT WORK in Chrome

Ok so it actually converts the code here!  Putting spaces to help:

Input = & # 6 1 4 4 0 ;

@Illnasty, Thank you for your feedback.  I'll look into this policy and make the necessary changes.

@Illnasty thanks for that mate.  Mine is still not working.  I'm decoding the code provided at the website and pasting in between AdministratorsRemote Desktop Users as a simple string and the OMA is ./Device/Vendor/MSFT/Policy/Config/UserRights/AccessFromNetwork but still not working.  Are you adding any more coding as per the page?? ie  ??  thanks

So that I can get a better understanding, why doesn't  work as the separator?

Hi patti.  it give s catastrophic failure. i got it working too following @Illnasty solution.  the code &#xF000 ; is not encoded, so if you use it between the groups (like admin &#xF000 ; Authenticated users) it wont work. you need to go to the website and encoded it to xmk (then you are left with that square symbol)  that is what you copy and paste in between the groups. Funny thing is that if you encode, copy the square and paste on the input line and decode, if are left with  &#61440 ; and not &#xF000 ; anymore

@schenardie, thank you for the response.  Now that I understand it better, and your response makes sense, it sounds like the code in the doc needs to be replaced and a note added about the &#xF000 code not being encoded :)

Well to be honest it says on the documentation
"Here are examples of data fields. The encoded 0xF000 is the standard delimiter/separator" but that is not clear how you encode it. And also i got that working via GUI, if you try to import that via Graph API passing either encoded or decoded it doest work. Once it reaches Intune if you check via the GUI its something like ??and a square.  So it cannot currently be automated.
If you could check that it would be great.  It makes all the difference on fully automated environments.  Thanks @shortpatti

@schenardie, you bet, I'll check with the engineering team about this one.

Also since we're at it. The empty input () works but it gives a catastrophic error as well on the event logs.  If you can dig the right input which doesn't generate errors would be very helpful. Thanks again

@Heidilohr - Do you have any updates on this? Thanks.

Pay attention if you use the "Recommended value" of the "1809 MDM Security Baseline Preview" Excel sheet for the AllowLocalLogon CSP. "Users Administrators" will lock out everybody of his device because of the wrong separator. If deleting the rule doesn't work configure it to "Users".

Ok so it actually converts the code here! Putting spaces to help:

Input = & # 6 1 4 4 0 ;

Just to remind users of Github; MarkDown code block fences ( ``` ) are useful to show code and not letting the web page convert it into symbols:
* Input= 

@officedocsbot assign @jvsam

For me it works if I use the CDATA syntax and the separator as described in the documentation. The settings are applied successfully on the client but the verification in Intune fails. I get an "Error" for all user rights settings in the "per setting status" of my profile. Can somebody confirm this behavior?
Example of a setting:
OMA URI: ./Device/Vendor/MSFT/Policy/Config/UserRights/AllowLocalLogOn
Value:
<![CDATA[UsersAdministrators]]>

@Ricardo-Cz I can confirm this behavior. I also used OMA URI: ./Device/Vendor/MSFT/Policy/Config/UserRights/AllowLocalLogOn The policy is successfully applied, but in Intune it shows an error  -2016281112 (Remediation failed)

Is there a way to get the policy to be applied and the status on Intune show
Succeeded using custom profiles??

@ManikaDhiman : Sorry to ping you about this in your otherwise busy schedule,
but maybe you could answer the unanswered questions here?

After fighting over few months with Intune Premier support, here is the answer we got from them :

Essentially the issue is caused because of the encoding of the user groups used when adding the users into the policy.
When Intune does a GET on the value set, it expects it to match what was sent in an ADD or REPLACE, in this case due to the encoding, it doesn’t which is why we see the remediation failed message:
This is documented in the following article - https://docs.microsoft.com/en-us/intune/custom-settings-windows-10#find-the-policies-you-can-configure
To work with Intune, the setting must support the Add, Replace, and Get operations. If the value returned by the Get operation doesn't match the value supplied by the Add or Replace operations, then Intune reports a compliance error.
This error can be safely ignored as it is expected.

While this is "understandable" from technical point of view, I am still surprised that they did not implemented some additional code to handle this.
From a customer point of view, I did nothing wrong and errors are reported anyway in Intune... For me, I still see this as a bug.

Beware of browsers for coding/decoding

Has anyone found a way to deploy the decoded character via graph api? (not using the CDATA, i dont really want to see the profile failing for settings which i know are implemented).
Thanks

@PanuSaukko - This is something that is not yet available in a way that can be deployed easily. I would suggest you put this as an idea in the Azure User Voice portal. This is the only best way to highlight and get this sorted out in a way this should be.
@nam31 - Please guide the user on how an idea can be submitted on the Azure User Voice channel.
Thanks.
Imran.

@PanuSaukko thank you for engaging with the community here for the docs.
I know the product teams are always looking for user feedback and how to make Azure better . They keep a close tab on user feedback and it would be awesome if you could add your voice in User Voice

I entered the boilerplate MSFT recommendation:

2

chr
text/plain

./Device/Vendor/MSFT/Policy/Config/UserRights/BackupFilesAndDirectories

Authenticated UsersAdministrators

https://docs.microsoft.com/en-us/windows/client-management/mdm/policy-csp-userrights#userrights-accessfromnetwork
Doesn't work, registry entries are 'Dirty'
Still looking for a way to manage these settings for compliance.

Idk if this has been worked on or not but after a long fight I've found how to do this...
What I've done is that I've pasted this into value: S-1-5-32-544S-1-5-32-555
That's without any CDATA's, rectangle characters, and so on. Just use  as a separator.

Yeah that does work indeed. Been using like that for a year now. The issue is that you cant programatically write that from graph api. For someone doing it for one tenant its easy enough to do it on the web browser. But 3 dozen tenants it get tricky. If anyone ever find a way to pass unicode characters via graph api please share here.

@schenardie do you also happen to know how to pass an empty value? Like - I'd like a right not to be assigned to any user role, how do I pass a string, which is empty. That's the bit I'm missing too... Theoretically they've mentioned  but I think what they meant was   but that doesn't seem to work :(
Thanks in advance

Does it.
Just that will empty any groups that already exist.


Header file to accompany the C API?

Is there a header file I could use to use libwinit from other languages?

@feenkcom if there doesn't exist one, is there any example on how to use libwinit?

Hi @VarLad
winit.h is now available as part of the release. See:
https://github.com/feenkcom/libwinit/releases/latest


Generic type initialization

The initialization of ObservedValue without parameters may cause NullPointerException when initializating.
https://github.com/FerMod/EventDispatcher/blob/f8155266c756bea7c03e6dc049bb3f3b773b6886/src/main/java/com/fermod/observer/ObservedValue.java#L21-L26

One solution could be the one give in "Instantiating a generic class in Java".

It could need one of the patterns described in "Creational patterns".


Fix tests / CI in pxt-blockly

Describe the bug
Just tracking for future reference / when we next pull blockly; as of https://github.com/microsoft/pxt-blockly/pull/381/files#diff-e39ea64563ca0b82835282545d8fc6e01b54cbdec1a7ad9f9072e16b45f6abdbR87-R90 we're ignoring the results of the CI tests in pxt-blockly; we don't typically modify things that will cause existing tests to break but we were hitting an error (without changes to existing block) that doesn't seem to warrant investigation / appears to just be a test dependency break.
To Reproduce
Steps to reproduce the behavior:

Go to pxt blockly tests/run_all_tests.sh
See that it's disabled

Expected behavior
fix tests and make sure they don't break due to passage of time / chrome version changes. Pulling in blockly updates typically fixes this
Additional context
Add any other context about the problem here.

Not planning to track this as a bug.


video looping

Script version
the newest as of 6/21/2024
Describe the bug
video resets and it plays 2x times (needed 3 or 2 refreshes)
Desktop (please complete the following information):

all platforms

also i forgor to add no auto play

hey


BufferUnderflowException when attempting to install from snapshot

Noticed the following exception
2016-02-19 22:26:47,757 | INFO  | <IP_ADDRESS>:9876 | ServerStateMachine  | <IP_ADDRESS>/<IP_ADDRESS>:9876 - Installing snapshot 3902239
2016-02-19 22:26:47,759 | ERROR | 203:9876-state-1 | SingleThreadContext  | An uncaught exception occurred
java.nio.BufferUnderflowException
	at io.atomix.catalyst.buffer.AbstractBuffer.checkRead(AbstractBuffer.java:371)[68:org.onosproject.onlab-thirdparty:1.5.0.SNAPSHOT]
	at io.atomix.catalyst.buffer.AbstractBuffer.checkRead(AbstractBuffer.java:351)[68:org.onosproject.onlab-thirdparty:1.5.0.SNAPSHOT]
	at io.atomix.catalyst.buffer.AbstractBuffer.readLong(AbstractBuffer.java:578)[68:org.onosproject.onlab-thirdparty:1.5.0.SNAPSHOT]
	at io.atomix.copycat.server.storage.snapshot.SnapshotReader.readLong(SnapshotReader.java:151)[68:org.onosproject.onlab-thirdparty:1.5.0.SNAPSHOT]
	at io.atomix.variables.state.LongState.install(LongState.java:46)[68:org.onosproject.onlab-thirdparty:1.5.0.SNAPSHOT]
	at io.atomix.manager.state.ResourceManagerState.install(ResourceManagerState.java:81)[68:org.onosproject.onlab-thirdparty:1.5.0.SNAPSHOT]
	at io.atomix.copycat.server.state.ServerStateMachine.lambda$installSnapshot$82(ServerStateMachine.java:124)[68:org.onosproject.onlab-thirdparty:1.5.0.SNAPSHOT]
	at io.atomix.copycat.server.state.ServerStateMachine$$Lambda$587/1905889655.run(Unknown Source)[68:org.onosproject.onlab-thirdparty:1.5.0.SNAPSHOT]
	at io.atomix.catalyst.util.concurrent.Runnables.lambda$logFailure$17(Runnables.java:20)[68:org.onosproject.onlab-thirdparty:1.5.0.SNAPSHOT]
	at io.atomix.catalyst.util.concurrent.Runnables$$Lambda$49/1217411637.run(Unknown Source)[68:org.onosproject.onlab-thirdparty:1.5.0.SNAPSHOT]
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)[:1.8.0_31]
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)[:1.8.0_31]
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:180)[:1.8.0_31]
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:293)[:1.8.0_31]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)[:1.8.0_31]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)[:1.8.0_31]
	at java.lang.Thread.run(Thread.java:745)[:1.8.0_31]

Not immediately clear what happened here.

Hmmm...

There appears to be an issue with how the data field (of type Buffer) in InstallRequest gets serialized and resurrected.
At LEADER:
Sent InstallRequest[term=2, leader=-1062706609, index=66421, offset=0, data=HeapBuffer[position=0, limit=176], complete=true] to <IP_ADDRESS>/<IP_ADDRESS>:9876

At FOLLOWER:
Received InstallRequest[term=2, leader=-1062706609, index=66421, offset=0, data=HeapBuffer[position=0, limit=0], complete=true]

At first glance it is not clear to me why it is happening. If I change the type of the data field from Buffer to byte[] in InstallRequest, it works fine.

Hmm... Maybe should use byte[] just to be safe and can look at the Buffer serializer separately

On Mar 7, 2016, at 8:27 PM, Madan Jampani<EMAIL_ADDRESS>wrote:
There appears to be an issue with how the data field (of type Buffer) in InstallRequest gets serialized and resurrected.
At LEADER:
Sent InstallRequest[term=2, leader=-1062706609, index=66421, offset=0, data=HeapBuffer[position=0, limit=176], complete=true] to <IP_ADDRESS>/<IP_ADDRESS>:9876
At FOLLOWER:
Received InstallRequest[term=2, leader=-1062706609, index=66421, offset=0, data=HeapBuffer[position=0, limit=0], complete=true]
At first glance it is not clear to me why it is happening. If I change the type of the data field from Buffer to byte[] in InstallRequest, it works fine.
—
Reply to this email directly or view it on GitHub.

That sounds good to me. I'll open a pull request for that shortly


kedge: Add support for advanced TLS configuration for backends

Right now, there is no possibility to configure correct TLS. The only possibility is to use insecure option.

Hello
Can you please provide more details on the TLS configuration and the backend system where it needs to be configured ?


How to troubleshoot .torrent file not getting downloaded?

Hello, i am trying to use the browser version of WebTorrent, All worked fine with the sample .torrent in the docs, but if i use for example a yify torrent ( https://yts.ag/torrent/download/423EED0C53F3857F66FCFB7522FB8C3829D1DCF4.torrent ) I am able to get the torrent data, but the player does not start downloading the file.

WebTorrent version: Last
Node.js version: Last
Browser name/version (if using WebTorrent in the browser): Chrome last

The torrent surely isn't available in the WebRTC network. And please, do not use WebTorrent for copyright infringement.


AutomaticKeepAliveClientMixin does not work with BottomNavigationBar

I have a BottomNavigationBar with 3 tabs, one of which is a stateful widget with the AutomaticKeepAliveClientMixin, wantKeepAlive getter returning true and the build method calling super.build(context) at the beginning.
However, every time I switch tabs, the Widget that is supposed to be kept alive is always re-created.
Others have the same issue too. Does anyone have a solution to this problem?

To investigate we would need a minimal runnable reproduction
as a single file so that we can just copy your code into lib/main.dart of a new project and run to reproduce.
Please add the output of flutter doctor -v.

Hi, here's a dummy example. I usually have a Map but I simplified it with just a text box. Write something in the text box, then switch tabs. You'll see that This should only be called once will be called multiple times.
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return new MaterialApp(
      title: 'Keep alive test',
      theme: new ThemeData(
        primarySwatch: Colors.indigo,
      ),
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key}) : super(key: key);

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _tabSelection = 0;
  var _tabs = [
    MapScreen(),
    ProfileScreen(),
  ];

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return new Scaffold(
      body: Center(
        child: _tabs.elementAt(_tabSelection),
      ),
      bottomNavigationBar: BottomNavigationBar(
        items: <BottomNavigationBarItem>[
          BottomNavigationBarItem(
              icon: Icon(Icons.pin_drop), title: Text('Map')),
          BottomNavigationBarItem(
              icon: Icon(Icons.announcement), title: Text('Submissions')),
        ],
        currentIndex: _tabSelection,
        fixedColor: Colors.indigo,
        onTap: _onItemTapped,
      ),
    );
  }

  void _onItemTapped(int index) {
    setState(() {
      _tabSelection = index;
    });
  }
}

@override
class MapScreen extends StatefulWidget {
  State createState() => _MapScreenState();
}

class _MapScreenState extends State<MapScreen>
    with AutomaticKeepAliveClientMixin {
  @override
  bool get wantKeepAlive => true;

  @override
  void initState() {
    super.initState();
    print('This should only be called once.');
  }

  @override
  Widget build(BuildContext context) {
    super.build(context);
    return Stack(
      overflow: Overflow.clip,
      children: [
        TextField(
          decoration: InputDecoration(
            labelText: 'Test',
          ),
        )
      ],
    );
  }
}

class ProfileScreen extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Text('profile screen');
  }
}

Flutter doctor:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, v1.2.2-pre.44, on Mac OS X 10.14.3 18D109, locale en-CA)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
[✓] Android Studio (version 3.3)
[!] IntelliJ IDEA Ultimate Edition (version 2016.3.5)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    ✗ This install is older than the minimum recommended version of 2017.1.0.
[✓] Connected device (1 available)

! Doctor found issues in 1 category.

Sounds like https://github.com/flutter/flutter/issues/11655#issuecomment-348287396

I understand that the build method will get called again, but it shouldn’t be the case of initState with that mixin, no?

On Feb 22, 2019, at 3:27 PM, Günter Zöchbauer<EMAIL_ADDRESS>wrote:
Sounds like #11655 (comment)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I see. Sorry, didn't pay attention.

If you want AutomaticKeepAliveClientMixin to continue working, please use PageView to wrap the body , another simple way is using IndexedStack, like same problem

Ok, thanks. I'll try and get back to you.

Same for me. Don't work

see  https://stackoverflow.com/questions/53011686/flutter-automatickeepaliveclientmixin-is-not-working-with-bottomnavigationbar

Same for me.

Same for me.

Same for me.

Same for me.

Any news on that?

It also does not work even on common Tabs

Reproducible in 1.21.0-1.0.pre base on https://github.com/flutter/flutter/issues/28345#issuecomment-466525654.

 flutter doctor -v
[✓] Flutter (Channel dev, 1.21.0-1.0.pre, on Mac OS X 10.15.5 19F101, locale
    en-AO)
    • Flutter version 1.21.0-1.0.pre at /Users/pedro/dev/SDKs/flutter_dev
    • Framework revision f25bd9c55c (9 days ago), 2020-07-14 20:26:01 -0400
    • Engine revision 99c2b3a245
    • Dart version 2.9.0 (build 2.9.0-21.0.dev 20bf2fcf56)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/pedro/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Users/pedro/Library/Application
      Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/193.6514223/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.3

[✗] Chrome - develop for the web (Cannot find Chrome executable at
    /Applications/Google Chrome.app/Contents/MacOS/google-chrome-unsafe)
    ! /Applications/Google Chrome.app/Contents/MacOS/google-chrome-unsafe is not
      executable.

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] Android Studio (version 4.0)
    • Android Studio at /Users/pedro/Library/Application
      Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/193.6514223/Android
      Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.47.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.12.1

[✓] Connected device (3 available)
    • AOSP on IA Emulator (mobile) • emulator-5554 • android-x86    • Android 9
      (API 28) (emulator)
    • macOS (desktop)              • macos         • darwin-x64     • Mac OS X
      10.15.5 19F101
    • Web Server (web)             • web-server    • web-javascript • Flutter
      Tools

! Doctor found issues in 1 category.

Same for me. It don't work with ConvexAppBar, Flutter version: 1.20.0 Stable

Verified on latest versions and observed that the issue is replicable.

 runnable code sample

import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return new MaterialApp(
      title: 'Keep alive test',
      theme: new ThemeData(
        primarySwatch: Colors.indigo,
      ),
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key? key}) : super(key: key);

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _tabSelection = 0;
  var _tabs = [
    MapScreen(),
    ProfileScreen(),
  ];

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return new Scaffold(
      body: Center(
        child: _tabs.elementAt(_tabSelection),
      ),
      bottomNavigationBar: BottomNavigationBar(
        items: <BottomNavigationBarItem>[
          BottomNavigationBarItem(
              icon: Icon(Icons.pin_drop), label: 'Map'),
          BottomNavigationBarItem(
              icon: Icon(Icons.announcement), label: 'Submissions'),
        ],
        currentIndex: _tabSelection,
        fixedColor: Colors.indigo,
        onTap: _onItemTapped,
      ),
    );
  }

  void _onItemTapped(int index) {
    setState(() {
      _tabSelection = index;
    });
  }
}

@override
class MapScreen extends StatefulWidget {
  State createState() => _MapScreenState();
}

class _MapScreenState extends State<MapScreen>
    with AutomaticKeepAliveClientMixin {
  @override
  bool get wantKeepAlive => true;

  @override
  void initState() {
    super.initState();
    print('This should only be called once.');
  }

  @override
  Widget build(BuildContext context) {
    super.build(context);
    return Stack(
      clipBehavior: Clip.hardEdge, children: [
        TextField(
          decoration: InputDecoration(
            labelText: 'Test',
          ),
        )
      ],
    );
  }
}

class ProfileScreen extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Text('profile screen');
  }
}

 console log

flutter: This should only be called once.
flutter: This should only be called once.
flutter: This should only be called once.
flutter: This should only be called once.
flutter: This should only be called once.
flutter: This should only be called once.

 stable, master flutter doctor -v

[✓] Flutter (Channel stable, 3.3.5, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.3.5 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d9111f6402 (10 hours ago), 2022-10-19 12:27:13 -0700
    • Engine revision 3ad69d7be3
    • Dart version 2.18.2
    • DevTools version 2.15.0

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.5.0-12.0.pre.45, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.5.0-12.0.pre.45 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c2edb20f3d (53 minutes ago), 2022-10-31 21:18:37 -0700
    • Engine revision f721db653d
    • Dart version 2.19.0 (build 2.19.0-356.0.dev)
    • DevTools version 2.18.0
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

这是来自QQ邮箱的假期自动回复邮件。 您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。


[ContainerRegistry] move to use bearerTokenChallengeAuthenticationPolicy from Core

move to depend on core CAE support
remove private copy.

Fixes #14760

/azp run js - container-registry - tests


Reduce Log Level

Is your feature request related to a problem? Please describe.

As a tester and administrator of DigiWF, I want to have a pleasant user experience by increasing the clarity of error messages. One way to achieve this goal is by reducing log levels to the necessary minimum. Currently, logs that are not very helpful are being written at millisecond intervals, which hides relevant information.
Describe the solution you'd like

Analyze which logging can be reduced.
Consider which logging is necessary for data protection reasons. It must be logged which user accesses the system.

Describe alternatives you've considered

Acceptance Criteria

[ ] Log level recommendations are available for the DigiWF components, including recommendations for classes and/or packages that require alternative log levels.
[ ] The log levels are adjusted in the property files and/or in the source code.

Additional context

Das bitte in digiwf-s3-integration-client-example mit korrigieren:
logging:
  level:
    io:
      muenchendigital: info


Make VS Code use DAP's "setExpression" request

In https://github.com/microsoft/debug-adapter-protocol/issues/188 we've discussed how to simplify the implementation of DAP's setVariable request in debug adapters. The proposal asked for passing an additional "l-value expression" to setVariable which can be easily used in an "evaluate"-based implementation.
One finding of this discussion was that DAP has already a setExpression request which is basically a variant of setVariable that takes a "l-value expression".
Based on that finding we came to the conclusion that only the following spec clarification is needed:

If a debug adapter implements both setVariable and setExpression, a client will only use setExpression if the variable has an evaluateName property.

A consequence of this is that VS Code will start to use setExpression if a variable has the evaluateName property and the debug adapter supports the setExpression request.

This makes good sense. Let me assign to June, so we change this next milestone once we finalise the DAP clarification.

@isidorn the proposed solution might not be ideal for @gregg-miskelly. Please see the discussion starting with https://github.com/microsoft/debug-adapter-protocol/issues/188#issuecomment-849757966

Thanks for the pointer, I see that it does not work for @gregg-miskelly
Should we move this to On-Deck until we land on a solution

I wonder, why VS Code is not using setExpression for watches.

@weinand we are not using setExpression for anything on the VS Code side. We simply have not adopted it.
So I think the reason is historical, we first added setVariable and we use that when chaning the value of the Variable in the VARIABLES view.
For the Watch expression, we do not allow to set expression from the Watch view. Only to edit, which would re-evaluate it.
So it seems like there is a missing piece that VS Code starts using setExpression for watches

@isidorn yes, I understand why we have setVariable. But setVariable only works if there is variable container. In the VARIABLEs view the top most container is the Scope.
But in the WATCHES view there is no scope (hence no container) and we are not using setVariable but evaluate.
If we would allow to change their values, we could use setExpression.

Hi
If you do your last sentence, it would close https://github.com/microsoft/vscode/issues/115063

Yeah good point, that feature request is what is missing for this scenario it seems.

I started investigating into this and created this PR which proves it is possible https://github.com/microsoft/vscode/pull/127162
Some open questions:

If the debugger does not support setExpression should we show the action as disabled or not in the menu
For variables that are children of watch expressions should we send the setVariable or setExpression

I am not aware of a debug adapter that supports the setExpression so I was not able to test this end to end right now. Also I am not super happy with the code thus I would prefer to not do this end of milestone. Thus moving to August since in July I am on vacation. If somebody is passionate about this feel free to continue the work on my PR.

Great ! Thanks a lot.
About your mentioned points, my feedbacks would be:

setExpression should not be available in the Menu (like VS Code does currently for Data Breakpoints, Set Variable and so on)
Here, can we just send setExpression if evaluateName is defined, setVariable instead ?

I am not aware of a debug adapter that supports the setExpression so I was not able to test this end to end right now.

In case this is useful to you: the C# (coreclr) and Windows C++ (cppvsdbg) debug adapters support setExpression.

@isidorn thanks for investigating the use of setExpression for the Watches view.
Since we have https://github.com/microsoft/vscode/issues/115063 for this investigation, it would be great if you could move your comment to that item. Then we can keep this item for the original (broader) discussion.

@weinand I have moved my comment to that issue. If you would like to keep this discussion cleaner feel free to delete my comments.

For the broader issue of DAs that would like to avoid needing to support setVariable to update the value of a child, I would like to repeat a suggestion I had earlier: a DA could indicate that it always wants to edit using evaluateName by:

Always provide evaluateName for any editable property
Return true for supportsSetExpression
Return false/null for supportsSetVariable.

@gregg-miskelly your suggestion makes sense and I will update the DAP doc as needed.
@isidorn we should support behavior.

@gregg-miskelly this suggestion makes good sense, I have implemented it via https://github.com/microsoft/vscode/pull/127162/commits/f1b1e3f2a090a5ece3129708e696d9620e480982
As suggested above:

If the extension does not support setExpression, then we will not show the Set Value action in the context menu
We send the setVariable for child element in the WATCH view (unless in the case that @gregg-miskelly pointed out)

I plan to merge this later today, so this will be in Insiders next week. I would really appreciate that you try it out and provide feedback. I have only tested it with Mock debug so far. I will also create a test plan item.

@connor4312 is it possible for js-debug to support setExpression this milestone? That would make it easier to test this out in the endgame. Let me know if you would like me to create an issue in js-debug. Thanks!

Created this test plan item https://github.com/microsoft/vscode/issues/130170
In case there are some cases that I did not cover feel free to comment on the test plan item. Thanks

Thanks, I'm adopting this in js-debug. One point of feedback is that I think the "watch" and "variables" view should be automatically invalidated after the set completes, to match the behavior Set Value in s the "variables" view.

Oh, I was behind on Insiders. The watch view is refreshed, but should the variables view similarly be refreshed? When evaluating an expression I think all debuggers will want this since it's probably not trivial to figure out what variables might have changed as a result of the evaluation of an arbitrary expression.

@connor4312 the variables view should also get refreshed. If that is not the case let me know and I can investigate. We basically refresh all views, since we never know how a change affect some other variables / watch expressions.

Nevermind, it was a caching error in my own code, works well :)

Hm, actually this is (also) caching in V8. Even after changing the variable, getting the stackframe "object" properties still returns the old value. I don't think I can work around this very efficiently 😕

@connor4312 it is a long standing issue I believe, since I think we were also hitting this a couple of years ago...