Asking to authenticate after every launch.

After adding the authentication code, it asks to authenticate after every launch. Authentication the second time results in an error.

It is expected that it will authenticate on every launch, the sample does no implement any token caching mechanisms. What's the error?

By what I understand, it is getting an authentication error, then failing with an internal error. But, I am only getting the error while running the project after downloading the project (after adding my app code).

I think the error might be because I reused an appid.

Assuming your app registration matches what this app needs, it should be fine. The screenshot you're showing me looks off though: you shouldn't be seeing repeated prompts to open your browser. This should only happen once per run of the app. I see it's saying "service is unavailable to process the request". Are you by chance logging into this with a personal account (like an outlook.com user)?

Yes. The app I built from the "Build your first .Net Core app in 30 Minutes" works just fine. It also uses the same appid.
Sent from my T-Mobile 4G LTE Device
Get Outlook for Androidhttps://aka.ms/ghei36

From: Jason Johnston<EMAIL_ADDRESS>Sent: Wednesday, July 1, 2020 9:09:10 AM
To: microsoftgraph/msgraph-training-dotnet-core<EMAIL_ADDRESS>Cc: minecraftchest1<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoftgraph/msgraph-training-dotnet-core] Asking to authencate after every launch. (#13)
Assuming your app registration matches what this app needs, it should be fine. The screenshot you're showing me looks off though: you shouldn't be seeing repeated prompts to open your browser. This should only happen once per run of the app. I see it's saying "service is unavailable to process the request". Are you by chance logging into this with a personal account (like an outlook.com user)?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoftgraph/msgraph-training-dotnet-core/issues/13#issuecomment-652440940, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKIAII6REUVNXEKJNFZAT3TRZM7QNANCNFSM4OLXAOLA.

Ok. I reproduced the "service is unavailable" error using my personal account, but then I ran the code a second time and it authenticated just fine. There may be intermittent issues with the Microsoft identity platform, support for this login flow for personal accounts is relatively new.
So you do not have any of these problems with the version you built from the tutorial, but the code in this repo does have the problem? Can you diff the code to see if there's anything different? (or if not, can you post your tutorial-built project somewhere I can access it, like GitHub?)

Here is the link to the repro with the totural built code. I'll push my latest code to it later today, as well as as branching and adding the repro code to it. I don't have acess to my dev machine right know or I would  have it done already.
Sent from my T-Mobile 4G LTE Device
Get Outlook for Androidhttps://aka.ms/ghei36

From: Jason Johnston<EMAIL_ADDRESS>Sent: Wednesday, July 1, 2020 9:37:21 AM
To: microsoftgraph/msgraph-training-dotnet-core<EMAIL_ADDRESS>Cc: minecraftchest1<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoftgraph/msgraph-training-dotnet-core] Asking to authenticate after every launch. (#13)
Ok. I reproduced the "service is unavailable" error using my personal account, but then I ran the code a second time and it authenticated just fine. There may be intermittent issues with the Microsoft identity platform, support for this login flow for personal accounts is relatively new.
So you do not have any of these problems with the version you built from the tutorial, but the code in this repo does have the problem? Can you diff the code to see if there's anything different? (or if not, can you post your tutorial-built project somewhere I can access it, like GitHub?)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoftgraph/msgraph-training-dotnet-core/issues/13#issuecomment-652457432, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKIAII73E4TEQKVKTCC5VMLRZNC2DANCNFSM4OLXAOLA.

Forgot to post the link.
https://github.com/minecraftchest1/MicrosoftGraphTestProjects

Doesn't look like that's complete:
using System;

namespace MsGraphApp1
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("Hello World!");
        }
    }
}

I thought I synced it.
Sent from my T-Mobile 4G LTE Device
Get Outlook for Androidhttps://aka.ms/ghei36

From: Jason Johnston<EMAIL_ADDRESS>Sent: Monday, July 6, 2020 11:07:49 AM
To: microsoftgraph/msgraph-training-dotnet-core<EMAIL_ADDRESS>Cc: minecraftchest1<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoftgraph/msgraph-training-dotnet-core] Asking to authenticate after every launch. (#13)
Doesn't look like that's complete:
using System;
namespace MsGraphApp1
{
class Program
{
static void Main(string[] args)
{
Console.WriteLine("Hello World!");
}
}
}
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoftgraph/msgraph-training-dotnet-core/issues/13#issuecomment-654328649, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKIAII5VRKCW2ZLAS6K6CMTR2HZFLANCNFSM4OLXAOLA.

Its pushed now. I forgot to commit the changes last time.

Thanks. Which project is the one you built from the tutorial that doesn't have any problems?

I think it is project1.

From: Jason Johnston<EMAIL_ADDRESS>Sent: Monday, July 6, 2020 11:07:49 AM
To: microsoftgraph/msgraph-training-dotnet-core<EMAIL_ADDRESS>Cc: minecraftchest1<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoftgraph/msgraph-training-dotnet-core] Asking to authenticate after every launch. (#13)
Doesn't look like that's complete:
using System;
namespace MsGraphApp1
{
class Program
{
static void Main(string[] args)
{
Console.WriteLine("Hello World!");
}
}
}
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoftgraph/msgraph-training-dotnet-core/issues/13#issuecomment-654328649, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKIAII5VRKCW2ZLAS6K6CMTR2HZFLANCNFSM4OLXAOLA.

It's project1
Sent from my T-Mobile 4G LTE Device
Get Outlook for Androidhttps://aka.ms/ghei36

From: Wyatt C Jackson<EMAIL_ADDRESS>Sent: Monday, July 6, 2020 11:35:44 AM
To: microsoftgraph/msgraph-training-dotnet-core<EMAIL_ADDRESS>Subject: Re: [microsoftgraph/msgraph-training-dotnet-core] Asking to authenticate after every launch. (#13)
I think it is project1.

From: Jason Johnston<EMAIL_ADDRESS>Sent: Monday, July 6, 2020 11:07:49 AM
To: microsoftgraph/msgraph-training-dotnet-core<EMAIL_ADDRESS>Cc: minecraftchest1<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoftgraph/msgraph-training-dotnet-core] Asking to authenticate after every launch. (#13)
Doesn't look like that's complete:
using System;
namespace MsGraphApp1
{
class Program
{
static void Main(string[] args)
{
Console.WriteLine("Hello World!");
}
}
}
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoftgraph/msgraph-training-dotnet-core/issues/13#issuecomment-654328649, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKIAII5VRKCW2ZLAS6K6CMTR2HZFLANCNFSM4OLXAOLA.

Ok. After diffing that with the repo, there's no significant difference. Your copy doesn't have the code to get the user's events, but the auth stuff is all the same. I'm not sure that there's any action I can take on this repo based on the information I have available.

I'm not to concerned sense that is a test anyway. I have other projects to work on.
Sent from my T-Mobile 4G LTE Device
Get Outlook for Androidhttps://aka.ms/ghei36

From: Jason Johnston<EMAIL_ADDRESS>Sent: Monday, July 6, 2020 11:39:11 AM
To: microsoftgraph/msgraph-training-dotnet-core<EMAIL_ADDRESS>Cc: minecraftchest1<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoftgraph/msgraph-training-dotnet-core] Asking to authenticate after every launch. (#13)
Ok. After diffing that with the repo, there's no significant difference. Your copy doesn't have the code to get the user's events, but the auth stuff is all the same. I'm not sure that there's any action I can take on this repo based on the information I have available.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoftgraph/msgraph-training-dotnet-core/issues/13#issuecomment-654344786, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKIAII5FVKWYL2LRJKTRMPDR2H427ANCNFSM4OLXAOLA.


helm lint error

Output of helm version: version.BuildInfo{Version:"v3.7.1", GitCommit:"1d11fcb5d3f3bf00dbe6fe31b8412839a96b3dc4", GitTreeState:"clean", GoVersion:"go1.16.9"}
Issue: With v3.7.1 version of helm, the helm lint command throws error if the template has multiple objects separated with YAML separator like below
==> Linting .
[INFO] Chart.yaml: icon is recommended
[ERROR] templates/sample.yaml: unable to parse YAML: invalid Yaml document separator: apiVersion: v1

Error: 1 chart(s) linted, 1 chart(s) failed

Steps to reproduce:
helm create lint-issue
vi lint-issue/templates/sample.yaml with below content
{{- if .Values.serviceAccount.create -}}
apiVersion: v1
kind: ServiceAccount
metadata:
  name: serviceaccount1
  labels:
    {{- include "lint-issue.labels" . | nindent 4 }}
  {{- with .Values.serviceAccount.annotations }}
  annotations:
    {{- toYaml . | nindent 4 }}
  {{- end }}
{{- end }}
---
{{- if .Values.serviceAccount.create -}}
apiVersion: v1
kind: ServiceAccount
metadata:
  name: serviceaccount2
  labels:
    {{- include "lint-issue.labels" . | nindent 4 }}
  {{- with .Values.serviceAccount.annotations }}
  annotations:
    {{- toYaml . | nindent 4 }}
  {{- end }}
{{- end }}

helm lint lint-issue
Above command will throw the error.
NOTE:
Everything works fine without error with helm version v3.6.x.
Also the issue seems to be limited to helm lint. helm template/install seems to be working fine with v3.7.1

can I take on this issue if no one else is working on it?

Can you try to remove the leading dash of the second document? I made a similar mistake before: https://github.com/helm/helm/issues/10149#issuecomment-929205040
{{- if .Values.serviceAccount.create -}}
apiVersion: v1
kind: ServiceAccount
metadata:
  name: serviceaccount1
  labels:
    {{- include "lint-issue.labels" . | nindent 4 }}
  {{- with .Values.serviceAccount.annotations }}
  annotations:
    {{- toYaml . | nindent 4 }}
  {{- end }}
{{- end }}
---
- {{- if .Values.serviceAccount.create -}}
+ {{ if .Values.serviceAccount.create }}
apiVersion: v1
kind: ServiceAccount
metadata:
  name: serviceaccount2
  labels:
    {{- include "lint-issue.labels" . | nindent 4 }}
  {{- with .Values.serviceAccount.annotations }}
  annotations:
    {{- toYaml . | nindent 4 }}
  {{- end }}
{{- end }}

Hope this helps!

Can you try to remove the leading dash of the second document?

When working with chart dependencies, patching the template to work with Helm 3.7 isn't often possible.
For instance, we have a chart with dependencies that lints at 3.6.3 but fails on 3.7.x:
$ helm version
version.BuildInfo{Version:"v3.7.2", GitCommit:"663a896f4a815053445eec4153677ddc24a0a361", GitTreeState:"clean", GoVersion:"go1.17.3"}

$ helm lint deploy/
==> Linting deploy/
[INFO] Chart.yaml: icon is recommended
[ERROR] templates/secrets.yaml: unable to parse YAML: invalid Yaml document separator: apiVersion: kubernetes-client.io/v1

Error: 1 chart(s) linted, 1 chart(s) failed

$ helm363 version
version.BuildInfo{Version:"v3.6.3", GitCommit:"d506314abfb5d21419df8c7e7e68012379db2354", GitTreeState:"clean", GoVersion:"go1.16.5"}

$ helm363 lint deploy/
==> Linting deploy/
[INFO] Chart.yaml: icon is recommended

1 chart(s) linted, 0 chart(s) failed

I think the conclusion is correct: it's a template issue. But unfortunately it manifests while upgrading to Helm 3.7.x and will require upstream changes for some charts.


Bluetooth: controller: mark new LLCP as experimental

Add text in Kconfig to notify users that the refactored
LLCP is experimental
Signed-off-by: Andries Kruithof<EMAIL_ADDRESS>

superseded by #39736

Superceded by #39736


fix(region,host): backup guest

What this PR does / why we need it:
backup guest bugfix.

Does this PR need to be backport to the previous release branch?:
release/3.9

/hold cancel


Make sure built script forces Java 1.7 options to javac

The shake/Build.hs build script should pass etlas options -optjava=“-source 1.7 -target 1.7” because the eta compiler doesn't really use Java 8 features yet and it helps out the people trying to use Eta for Android.

Just experimenting with this. I've added the following in bold to the buildLibrary function in shake/Build.hs:
buildLibrary :: Bool -> (String -> String) -> String -> [String] -> Action ()
buildLibrary debug binPathArg lib _deps = do
let dir = library lib
installFlags = ["--allow-boot-library-installs"]
++ nonNullString (binPathArg "../../")
configureFlags = if debug
then ["--enable-optimization=0"
,"--eta-options=-ddump-to-file -ddump-stg -dumpdir=dump"]
else ["--enable-optimization=2"
,"--eta-options=-optjavac=\"-source 1.7 -target 1.7\""]
No luck though, the libraries are still compiled to java 1.8. Any suggestions?

Yes, that's in the right direction. Also add a -v3 flag into the mix and watch out for the call to javac - it should display all the arguments that it passed. If it didn't, let me know.

Thanks. I got most of the libs to compile to java 1.7 but a few still get compiled to 1.8. Specifically the following ones:
./eta/integer/Utils.class
major version: 52
./eta/base/Utils.class
major version: 52
./eta/base/HSIConv.class
major version: 52
Are these some special cases?

Those are referenced via the "java-sources" Cabal field. I guess etlas doesn't use eta-options for that. We might not be able to reuse eta-options and might need a more specific flag instead.

maybe with etlas --javac-option(s)?

It turns out that the value of opt-javac was getting added to the wrong part of the javac options (at the end after the source file) so it wasn't taking effect. The eta compiler now forcefully sends the options -source 1.7 -target 1.7 which can be overrided with opt-javac if desired.

I'll be cutting the 0.0.9b2 release shortly which will have binaries with the correct versions.

The 0.0.9b2 binary release is available now and this should be fixed. Let me know if it works.


[Samples] Principal propagation to SuccessFactors

Target
Prepare a sample application showing extensibility of SFSF with principal propagation

This issue is closed as the project is no longer maintained


Dendron note parents are not displayed in trail/grid view or in graph view

Apologies if this is due to misconfiguration. I am brand new to obsidian!
Observation
"This note has no real implied parents" is displayed in the trail view for a note that has a Dendron style parent. It is evident that the parent exists when the matrix view is inspected:

The dendron hierarchy also does not appear in graph view:

Expectation
The trail view should show the parents/up hierarchy which are already displayed in the matrix view. The graph view should show both parents and children.
Vault files

Config
All defaults + Alternative Hierarchies -> Dendron Notes -> Add Dendron notes to graph is enabled. As per the defaults a standard hierarchy is configured "(up) (same) (down) (next) (prev)"

Update (29/01/2022)
I have found that ticking up in Limit Trail View to only show certain fields (as shown below) makes the the trail show a note's parent. This was previously off by default in my newly created vault. Graph view still does not represent the dendron hierarchies however.

However I would have expected that if I didn't have it limited to any fields that it would show ALL parent notes by default. i.e. set as this:

From the settings in https://github.com/SkepticMystic/breadcrumbs/wiki/Settings it would appear that the intended behaviour is all up fields should be shown by default. Not sure whether that means all fields should have been ticked by default or if having none ticked implies there is no limiting enabled, but I do think showing all parents by default would be most intuitive.

Limit Trail to only show certain parents
From a list of your different up fields, choose which should be shown in the trail view. All are shown by default.

Hey @Kamholtz :)
I see that the changes made in your edit resolved the issue.
So that settings works as "If a field is ticked, show it in the trail. If not, don't show it".
All of the fields should be ticked by default, therefore showing all fields by default - as the wiki says.
But given that the up field was not ticked, and you didn't untick it, then there must be a bug there

There is definitely a bug here, thanks!
I've added a fix which I believe should resolve it. But it doesn't work retroactively. So if the checkboxes were already unticked, it won't go and tick them for you. It will, however, tick new fields

@andrey-jef @Kamholtz alright, 2.45.0 is out, this issue should be resolved. Be sure to tick all the appropriate checkboxes first (you'll only need to do this once)

Thank you for such a fast fix!

Just a side note, would you expect Dendron hierarchies to be reflected in the graph view as well? I don't seem to see this happening even with the Limit Trail View to only show certain fields setting applied.

@Kamholtz Breadcrumbs' Dendron features don't add "real" links to Obsidian. So they definitely won't show up in the Graph View, no

Do you think adding real links from Dendron hierarchies would be within the scope of your plugin? I am happy to write up a feature request if so even if it's not a priority.

Have a look at this FR on the Obsidian API repo, we added it for this reason :)
https://github.com/obsidianmd/obsidian-api/issues/33

Have a look at this FR on the Obsidian API repo, we added it for this reason :)
https://github.com/obsidianmd/obsidian-api/issues/33

Thanks for pointing to the obsidian FR. The explanation in details of nothingislost is great. TIL that obsidian also uses database called indexedb 🤓

I'm gonna close this out, as the original bug seems to be fixed


shader texscreen incorrectly documented

In the documentation it says color texscreen( vec2 ) but it should be vec3 texscreen(vec2) as this function returns a vec3. I think it should actually return color, even tho' alpha will always be 1. Makes things easier to work with it since you're looking for color and converting the vec3 to it anyway...

Fixed by https://github.com/godotengine/godot-docs/commit/bfd4549c807103abed4d516c5b6b7a806ae45084


Creating a contact us separate page.

Describe the bug
the contact us page is not yet created and it should be created with the similar UI with necessary information.
To Reproduce
Steps to reproduce the behavior:

Go to the website
Click on "contact us"
Not responsive and showing same page

Expected behavior
A separate page should get open where the information is provided
Screenshots

Desktop (please complete the following information):
Windows
Google Chrome
Smartphone (please complete the following information):
Redmi Note 9
Android
google chrome

@yatulearn can u assign this to me?


Add links to all Compilers

This PR adds links to each compiler, so you can click on them and it takes you to the homepage.
I tried to link to the project page where possible, but some things I couldn't find a good page so I just linked to the github, or wikipedia (only for Visual C/C++ and Java/JVM, as I don't think there is a good homepage for them).
I also added YASM as another assembler.

Thanks a lot for this! If you could open a pull request against the develop branch of mohd-akram/compilers with the URL changes that would be great. You can compile the CoffeeScript code using:
coffee -o lib -c src

I like the idea of clicking the links, but I'm not sure how well it would work on mobile. Once the URL changes are merged I can look into that.

I checked on an iPad and clicking the links doesn't seem to work. I will close this PR, but mohd-akram/compilers#1 still remains.


Hotfix/0.14.1

A quick release of 0.14.1 to fix some bugs.

Wemo: fix static configured wemo devices (@kk7ds)
Wemo: Fix known device tracking (@kk7ds)

We should pull in #1431 per @jaharkes as well.


wireguard: Add peers.*.publicKeyFile as an alternative to publicKey

Motivation for this change
When dealing with several servers, we might want a mechanism by which the servers share their public key with each other after being provisioned. Thus, we can't give the NixOS configuration the public keys directly because they are not known at that point in time. publicKeyFile solves the issue.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Ping...

Any thoughts on this? otherwise I'll close this PR and delete the branch.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2330


Any idea why I get this?

thx

@Rjrunner44 hey, it can be a syntax error in the .prisma file or the version mismatch with the prisma-engine.
Can you provide prettier output for the npx prettier --write packages/address-book/prisma/schema.prisma

not sure. I have 2 diff projects, one it formats .prisma just fine, another I get the error above. Same exact VS CODE env also. Same pretty much everything (same prisma, etc, etc)...and if I copy a xx.prisma file from one to the other, one formats and one doesn't (i.e. a good prisma file formats on one but not the other). Wish I had more to go on lol 🤷♂😅 In other words, it's not the .prisma file itself that is the issue.

ah, there is one diff between the 2 repo projects: 1 is a normal repo (package.json at root) and the other is a lerna monorepo. Maybe that is affecting this somehow

if I run npx prettier --write ./prisma/schema.prisma in the sub repo, I get:
prisma/schema.prisma
[error] prisma/schema.prisma: TypeError: Cannot create property 'tokens' on string 'generator client {
[error]   provider = "prisma-client-js"
[error] }
[error]
[error] datasource db {
[error]   provider = "postgresql"
[error]   url      = env("DATABASE_URL")
[error] }
[error]
[error] model Message {
[error]   id          Int      @id @default(autoincrement())
[error]   fqcn String
[error]   payload String
[error]   signature String
[error]   createdAt   String
[error]   defaultTimeout   String
[error] }'
[error]     at attachComments (/Users/rj/Projects/ew/ewai/messaging/packages/archive-manager/node_modules/prettier/bin-prettier.js:13244:14)
[error]     at coreFormat (/Users/rj/Projects/ew/ewai/messaging/packages/archive-manager/node_modules/prettier/bin-prettier.js:13270:21)
[error]     at format (/Users/rj/Projects/ew/ewai/messaging/packages/archive-manager/node_modules/prettier/bin-prettier.js:13516:73)
[error]     at formatWithCursor (/Users/rj/Projects/ew/ewai/messaging/packages/archive-manager/node_modules/prettier/bin-prettier.js:13532:12)
[error]     at Object.formatWithCursor (/Users/rj/Projects/ew/ewai/messaging/packages/archive-manager/node_modules/prettier/bin-prettier.js:44213:15)
[error]     at format$1 (/Users/rj/Projects/ew/ewai/messaging/packages/archive-manager/node_modules/prettier/bin-prettier.js:45598:19)
[error]     at /Users/rj/Projects/ew/ewai/messaging/packages/archive-manager/node_modules/prettier/bin-prettier.js:45792:16
[error]     at /Users/rj/Projects/ew/ewai/messaging/packages/archive-manager/node_modules/prettier/bin-prettier.js:45731:14
[error]     at Array.forEach ()
[error]     at eachFilename (/Users/rj/Projects/ew/ewai/messaging/packages/archive-manager/node_modules/prettier/bin-prettier.js:45730:15)

@Rjrunner44 should be fixed in v0.5.1

thx. however it is unchanged in behavior (in my monorepo), same error. Thanks for taking a look though, appreciated.
    "prettier": "^2.2.1",
    "prettier-plugin-prisma": "^0.5.1",


[Bug]: Preview mode - widget text overlaps with the tab area (with page names) on scrolling

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I scroll on the preview mode, the widget texts overlap with the tab area (with page names).
Steps To Reproduce
Drag and drop a few widgets, say text/checkbox group/checkbox/switch etc.
Now scroll up and down.
Note that the text overlaps over the page tab area

Public Sample App
No response
Issue video log
No response
Version
Production

Seems to be working now


[autoparallel] refactor metainfo forward memory calculation

What's New?
In this PR, I refactor the forward memory calculation of former patched operations. After this PR, in SPMD solver, we will use the conservative way to estimate the memory, i.e. always believing that shape consistency will create a new tensor for TP.

Build and Test failed because the latest hugging face GPT2 model could not be traced


Add image IO examples

You commit history as a merge commit which should be in a PR. Perhaps if you rebase to master it will be removed.

Can I just squash them all into one commit?

Dave

You commit history as a merge commit which should be in a PR. Perhaps if you rebase to master it will be removed.

Can I just squash them all into one commit?

I think you need to remove the merge first or all the commits from the merge will be in there.

Dave

You commit history as a merge commit which should be in a PR. Perhaps if you rebase to master it will be removed.

Can I just squash them all into one commit?

I think you need to remove the merge first or all the commits from the merge will be in there.

Does it look OK now?  I rebased to master and squashed all the commits into one.


Services Add: SiriusXM

Services Add:
SiriusXM, since you have Apple Music; https://www.siriusxm.com/player/home
The video is in the,
iPhone, iPad, AppleTV  https://apps.apple.com/us/app/siriusxm-music-sports-news/id317951436
APK Google Play https://play.google.com/store/apps/details?id=com.sirius&hl=en_US&gl=US
Microsoft Store is Windows 11 Only https://apps.microsoft.com/detail/XPFCXHBKZW156L?hl=en-us&gl=US
macOS App Store have no SiriusXM App.

@big610 Added!

@big610 WDYM?

Test with Subscription.

@big610 OK closing now


nginx template has a wrong config in proxy_backend.conf.template

nginx template has a wrong config in proxy_backend.conf.template, you need to change "proxy_set_header Upgrade $connection_upgrade;" to "proxy_set_header Upgrade $http_upgrade;".Otherwise the websockets can not be used.

Thanks!

@sunyboy are you sure that's the case?
We use it like this: https://github.com/coreruleset/modsecurity-docker/blob/ea97f48fdb98cf28a22d902a83839ec015ccdcaa/v3-nginx/templates/conf.d/default.conf.template#L3

@sunyboy Did you check this one? ☝

No feedback from @sunyboy, closing.

Hello,
I've noticed the same: impossible to establish a webocket with the default configuration.
Websocket can be established successfully with the suggested change, i.e. proxy_set_header Upgrade $http_upgrade instead of  proxy_set_header Upgrade $connection_upgrade.
The 'map' block in default.conf is the one suggested by http://nginx.org/en/docs/http/websocket.html.

Ok, thanks for confirming. Will take a look then.

Yes, according to the docs, we accidentally use $connection_upgrade there: https://github.com/coreruleset/modsecurity-docker/blob/ea97f48fdb98cf28a22d902a83839ec015ccdcaa/v3-nginx/templates/includes/proxy_backend.conf.template#L3


[Feature]Add cpu usage policy to limit big query

Add last n second cpu usage metric for Workload Policy.
It's used to kill some query which cpu usage exceeds some value in the last period of time.
For example:
1 create a cpu usage policy.
create workload policy cpu_policy2 conditions(last_10s_cpu_usage_percent > 4) actions(cancel_query);

2 show this policy.
mysql [information_schema]>select * from workload_policy;
+-------+-------------+--------------------------------+--------------+----------+---------+---------+----------------+
| ID    | NAME        | CONDITION                      | ACTION       | PRIORITY | ENABLED | VERSION | WORKLOAD_GROUP |
+-------+-------------+--------------------------------+--------------+----------+---------+---------+----------------+
| 43033 | cpu_policy2 | last_10s_cpu_usage_percent > 4 | cancel_query |        0 |       1 |       0 |                |
+-------+-------------+--------------------------------+--------------+----------+---------+---------+----------------+
1 row in set (0.01 sec)

3 execute a insert select query, cpu usage could increase.
mysql [information_schema]>select CPU_USAGE_PERCENT from backend_active_tasks where QUERY_TYPE='LOAD';
+-------------------+
| CPU_USAGE_PERCENT |
+-------------------+
|              3.78 |
+-------------------+
1 row in set (0.01 sec)

mysql [information_schema]>select CPU_USAGE_PERCENT from backend_active_tasks where QUERY_TYPE='LOAD';
+-------------------+
| CPU_USAGE_PERCENT |
+-------------------+
|               4.8 |
+-------------------+
1 row in set (0.02 sec)

4 query is killed

mysql [hits]>insert into hits2 select * from hits;
ERROR 1105 (HY000): errCode = 2, detailMessage = (<IP_ADDRESS>)[INTERNAL_ERROR]query c2bffbd4474143d5-ae9346b7343b191d cancelled by workload policy: cpu_policy2, id:43033

Check List (For Author)

Test

[x] Regression test
[x] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No colde files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[x] Yes.

Does this need documentation?

[ ] No.
[x] Yes.

Release note

None

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall


Pull Request Template Added

closes #85
Pull Request Template Added. Please review the changes.
@Gaurav-Verma07 Thank you for this opportunity.

#85

@Gaurav-Verma07 sir, can you please give me your linkedin profile link as i need to add it in code of conduct file


OpenSearch: reestructure articles

There are some contents that started to repeat themselves in terms of titles:
Connection articles:

[ ] Python: https://developer.aiven.io/docs/products/opensearch/howto/connect-with-python.html
[ ] NodeJS: https://developer.aiven.io/docs/products/opensearch/howto/connect-with-nodejs.html

Sample dataset: recipes

[ ] Load the data with Python
[ ] Load the data with NodeJS

Sample dataset: recipes

[ ] Get data mapping with NodeJS
[ ] https://github.com/aiven/devportal/pull/586

Following what @anelook said here:
With data mapping examples for both JS and Python the article grew. Let's move "Get data mapping" (with examples of JS and Python) to a separate article?
Another way to organize those articles should be proposed to make the flow easier for the reader. Also, it would be nice to know what is your opinion in the Sample dataset: recipes article because from the title seems hard to know what it is about it.
But I understand it is linked in other places on the devportal.

@anelook we can discuss here how it can be structured the OpenSearch files

My concern is that you cannot do what "Make queries with HTTP client" section says before you upload the data. So putting it as the first section will be confusing to the readers: they will have to scroll down, follow the steps and then scroll back up.  I think it is safe to put it at the end. It gives a generic example to play with data once it is loaded.

Regarding "Data mapping", it is a concept on its own, so I think it can be safely moved to a separate article, explaining what it is and giving examples how it can be done.
Initially, it was put here as another example what you can do once the data is loaded. However, by putting extra python code, it becomes heavy for one article.

The current approach is right now a huge list of articles. I am bringing one example but the issue has more and we plan to bring more into those lists.
I would like that it is divided by category instead and the same approach has already been implemented for the Apache Kafka section before.
Check out here: https://github.com/aiven/devportal/pull/519
The reasons I feel the nested way is better:

Titles are cleaner and more organized
Avoid repetition of the same thing because we lack categories sections
Can be hard to find without using the search tool (If not placed alphabetically)
The emphasis in the title

Possible disadvantage:

search tools could work differently with categories and sub-categories kind of organization
It could break the original link because we are changing the link

Kafka is divided into groups of articles, covering similar topics and with 6-8 items in each subcategory, to avoid the navigation list getting very long. The approach proposed here of having one topic and then two sub articles is something quite different and I don't think it helps. We can re-organise the navigation without breaking links, and I think that using shorter article names for navigation titles could help here, without adding another layer of navigation between a user and the content they are looking for.


adding fabform.io to the template

Hello
Can you add https://fabform.io to your static site templates contact form

👋 Hi @fabformG
The forms on our templates are configured to use our own Form offering, so we aren't looking to integrate third parties into our templates directly.
We do have a contact form comparison on our ecosystem website here: JamStack Contact Forms. If you fill out the form on that page then the team that manages it can get you listed as an option.
Cheers!


add lib/domains/ph/edu/sacs.txt file to add Saint Alphonsus Catholic School

@ejkorg There are several files in the request, along with file.zip. It must not be there. Please remove file.zip and make pull request again.  I have to close this request without merging. Thank you for your understanding.


Improve TTML caption rendering

The captions in the following MPD should be presented in a region that starts 30% for the left and ends 10% to the right, and the font size should be 1/30 of the height of the video.
https://palemieux.com/public/foms2017/CEP150_512kb.mpd
Happy to provide additional information.
P.S.: consider using the W3C IMSC1 test suite to validate IMSC1 rendering, or using the imscJS polyfill, which is also used by dash.js.

Text display is currently handled by the browser or the application.  Browsers do not currently support regions, but we do parse them and make the information available to text display plugins.
Adding this to the backlog.  We will consider a complete DOM-based implementation of TextDisplayer.

In the mean time, feel free to write your own plugin for this.  We would be happy to review a pull request if you decide to work on this.

@joeyparrish Thanks for the feedback. Is the following possible for a plug-in to:

receive the media sample, i.e. TTML document, and a target rendering DOM element (e.g. div overlaying the video object)?
return the times at which it would like to be called back, and opaque state information to be passed at each call back?

At each callback, the TTML plugin would draw the subtitle/captions in the target rendering DOM element.

It's not a TTML plugin, but rather a text display plugin.  It receivers cues which it is responsible for rendering.  Here's the interface documentation:
https://shaka-player-demo.appspot.com/docs/api/shakaExtern.TextDisplayer.html
Here's the documentation for the cue objects:
https://shaka-player-demo.appspot.com/docs/api/shaka.text.Cue.html
This system is more general than TTML.  It will be used by the player for all subtitle/caption rendering, regardless of the input format.
You can pass whatever state you want into the constructor or into additional methods you have on your class.

Ok. Thanks for the details. As I understand it,

the media track is parsed and Text Cue instances are generated
these Text Cue instances are passed to a Text Displayer plugin for rendering

Is that right? If so, what happens if the current Text Cue interface does not support the full range of capabilities offered by TTML? Can the media track parser return any object as long as it implements the Text Cue interface? Will the full object be passed back to the plugin?
I am thinking the media track parser could return a TTML cue that implements the current interface (for minimal compatibility) but also contains the information (as private members) needed to fully render the TTML cue. Would that work?
Thanks for your help. Happy to continue the discussion offline. Feel free to DM me at<EMAIL_ADDRESS>

The shaka.text.Cue class is owned by us, and is independent of the browser.  If it is missing some TTML feature you need, we can extend it.  We intend it to be generic enough to support both TTML and WebVTT.
The exact Cue object (as output by the TTML parser) will be sent to the TextDisplayer plugin.  If there is something missing in that object, we would be happy to take either a feature request or pull request to add fields to Cue and add parser support for them in the TTML parser.

TTML supports capabilities beyond what shaka.text.Cue supports, including:

style variations within a cue (color, text decoration, font weight, etc.)
position of the cue within the video
etc...

Couple of options:

extend shaka.text.Cue to support these variations
allow shaka.text.Cue to carry an HTML fragment that would be generated by the TTML parser
allow shaka.text.Cue to carry an opaque TTML object that would be generated by the TTML parser

Thoughts?

Makes sense to allow the Cue object to support richer definition of payload content than it does now. Maybe even by allowing Cue.payload to be a Cue itself, or if you don't want to support infinite nesting, then by including some kind of markup within the payload that would override the appropriate properties like fontStyle, fontWeight, color etc. for a subsection of text within them.
Some approach like that will be required to support TTML properly since it allows style attributes to be specified on spans within what are considered "cues" here.

First, let me address using imscJS for TTML support:
Our current TTML parser, limited though it may be, compiles to about 7,939 bytes.  imscJS + its sax dependency is 159,787 bytes.  That's a 20x increase in size for the TTML parser itself.  Shaka Player as a whole is currently 185,141 bytes (in my working directory, anyway), so adding imscJS + sax would be an 86% increase in the size of Shaka Player itself.
That is way too big for a built-in subtitle parser.  If you want to use imscJS to handle TTML, you can always have a text-parsing plugin at the application level which replaces our default TTML parser.  See the API docs on TextParser if you want to pursue that: https://shaka-player-demo.appspot.com/docs/api/shakaExtern.TextParser.html
As for enriching our existing TTML parser, I'm completely open to that.  But since it sounds like our Cue interface is not up to the task of full TTML support, it will need to be redesigned.  If you want to contribute in this area, I would prefer to get a design proposal for Cue and iterate on that before you make a complete pull request.  If you don't want to do this, my team will look into it once we have taken care of some higher-priority features.  For now, this is still in the "backlog" milestone for us.
Thanks!

That is way too big for a built-in subtitle parser. If you want to use imscJS to handle TTML, you can always have a text-parsing plugin
at the application level which replaces our default TTML parser.

Seems a reasonable approach: have the application provide text parser and cue renderer implementations, e.g. using imscJS. Perhaps a simple example is all that is required.

@joeyparrish P.S.: I very much appreciate your taking the time to explore these various options.

This issue is quite old, and our TTML and rendering capabilities have grown a lot.  If there are still gaps in our support of TTML parsing or rendering, please feel free to open new issues for them.  Thanks!


[SPARK-16818] Exchange reuse incorrectly reuses scans over different sets of partitions

https://github.com/apache/spark/pull/14425 rebased for branch-2.0

Test build #63053 has started for PR 14427 at commit ef60367.

LGTM pending Jenkins.

Test build #63053 has finished for PR 14427 at commit ef60367.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63053/
Test PASSed.

Merging in branch-2.0.

@ericl can you close this?

Done


optionFormat and eitherFormat should be typed JsonFormat[T]

unfortunately these two implicit methods return very specific types, making them very troublesome to deal with when overriding or doing anything funky with shapless to automatically derive product/trait types.

actually, either might be ok. It's just optionFormat that's killing me.

I can workaround this with a non-implicit method of the same name, and introducing an implicit one with the correct return type.

This fix broke compatibility with version 1.3.1 - optionFormat has a different return type from 1.3.1, and anything compiled against it will now throw a runtime exception like:
Exception in thread "main" java.lang.NoSuchMethodError: spray.json.DefaultJsonProtocol$.optionFormat(Lspray/json/JsonFormat;)Lspray/json/StandardFormats$OptionFormat;

I'll open a separate issue.

ouch

the workaround is to create an implicit override def that will return the specialised type instead of JsonFormat[Option[T]]


Avalonia.Android nuget abandoned

Describe the bug
Updates for the Avalonia.Android package on nuget have not been released for over a year (since Avalonia version 0.8.4)
Expected behavior
Package updates are published along with updates to the main Avalonia packages.
Screenshots

@jp2masa is it stable enough to be published on nuget.org again?

Not sure how stable it needs to be for publishing on nuget.org. It's much better than it was a few months ago (it was completely unusable) but there are still a few bugs (IIRC changing orientation breaks in some cases and similar problems). If you consider that kind of bugs is acceptable (in my opinion it is, as the backend is still experimental), then I think it makes sense to publish again.

I think that even in spite of the bugs, you need to publish the package (even as a prerelease or some other mark that it is unstable)
It takes a time to build all this from the sources yourself
If you don't want to publish this on nuget, then maybe you should consider publishing this on the nightly feed?

Nightly packages are available. Stable nuget will be there with next major version.
Also templates
https://github.com/AvaloniaUI/avalonia-dotnet-templates/pull/79
And sample app
https://github.com/AvaloniaUI/Wordle-onia


custom_deps add-on's pypi option naming confusing

Hi, I've just come across this awesome and helpful add-on via https://github.com/zigpy/zigpy-zigate/pull/29.
The custom_deps' pypi option is really a list of pip packages which is simply passed to pip install: https://github.com/home-assistant/hassio-addons-development/blob/aa8c7a416c73aec5b4e41aac31880770b6f1c586/custom_deps/run.sh#L28
So packages can not only come from pypi but also from other sources such as git repositories which is helpful to testing branches before merging PRs and releasing new versions of packages.
For example, the following config works great:
pypi:
  - 'git+https://github.com/pdecat/zha-device-handlers@dm/legrand-devices'
apk: []

Maybe it should be renamed from pypi to pip to avoid confusion or, to avoid breaking changes, be documented to make this more obvious.
I'd be happy to submit a PR if this makes sense to you. Thanks!

Hi @pvizeli @frenck, looks like this went under your radar which I would totally understand.
Happy to close if you feel it's a non issue.
Best regards!

I don't think for an add-on which has a very specific use-case, this needs to be fixed or adjusted.
Changing the config is also a breaking change.

Ok then :)


Rocket structure

In this repo

[ ] Add a rocket image on the structure
[ ]  Add dummy text for the paragraph
[ ] confirmed that the  works appropriately.

Thanks


Clarification on the interpretation of inputed matrices

I was wondering if you could clarify to me what is the interpretation of values in imputed matrices with Higashi. I see that the value of the entries ranges from -1 to +1, shall I interpret them as the result of Pearson correlation? Is there any way to recover values that are compatible with a raw Hi-C matrix or a normalized one?
Thank you for any tip

Hum. The output should always be >0, if mode=='classification' it should be a value between 0 - 1 as a probability of observing a contacts, otherwise it should be a value >= 0 as a regression contacts.
Could you provide screenshot / code of how you fetch the imputed matrices?


Update Netlify build image from Ubuntu 16.04 to Ubuntu 20.04 (depcration: November 15, 2022)

Description
We need to update our build image to prevent the Netlify build to start failing in November.
We should follow https://answers.netlify.com/t/please-read-end-of-support-for-xenial-build-image-everything-you-need-to-know/68239#how-can-i-run-a-test-build-with-the-new-image-6 to do this without trouble for our users.

I’ve just switched the build image, locked site deployments, triggered a manual deployment, checked the preview, and re-enabled site deployments. This is done.


Thank you for this great library

I just wanted to let you know that we've been using CombineCoreBluetooth for a few months now, and it makes working with iOS BLE a breeze!
The API is very well designed, it's super easy to add to a project and integrates perfectly with SwiftUI and our own code base. It hides complicated implementation details but at the same times gives us a lot of flexibility.
Keep up the good work! :)

Thank you for the kind words! I very much appreciate it, and I'm glad it's been a useful tool for you!


[do not merge] More code generation tooling

add swagger types doc generation
add verify-* scripts
add update-all.sh

Don't merge yet... there are some issues.

Updated. Should be good to go.

Working on protobuf...

@deads2k fyi the UPSTREAM commit has lgtm but hasn't merged yet


question 4-storage Postgresql inheritance for event types

I know the current support for Postgresql is just plain event storage (ie unstruct is not supported).
I am adding unstruct event support to our ETL pipeline which uses Postgresql for intermediate storage. I do what I guess you guys call "shredding" in our own pipeline (ie use Java + Jackson to do this).
My idea is to use Postgresql inheritance for each one of the event types (page_view, unstruct, struct, etc). This is to make updating the atomic event schema less messy as our unstruct table will probably be pretty custom as well as faster filtering on event types (apparently postgres does smart things with inheritance and check constraints).
I have written some of the SQL migration scripts for this and could put a pull request or a simple gist. Or I could be completely wrong in my approach 😨  hence the question. The other option of course is joining tables but this complicates things for us.
I'm curious on how (or if) you guys plan on accomplishing supporting other events with Postgresql.

Hey @agentgt - we definitely plan on adding full nested entity support to Postgres once we've done a full refactoring of our Redshift load code.
Our working assumption is that our Postgres approach will very closely mirror our Redshift approach - but this isn't something we've devoted any R&D time to, so it's all still to explore.
Postgres table inheritance sounds interesting - not something we'd really had on our radar, I'd love to see a PR and then continue the conversation  :coffee:

I will see if I can do a PR next week. This is roughly what our stuff does (we have different names for some stuff events -> event).
ALTER TABLE event RENAME TO event_v04;

-- Copy schema
CREATE TABLE event AS (SELECT * from event_v04 limit 0);

-- FORCE programmatic dispatch to table (v9.2).
-- A trigger could be used to route but this forces correct
-- handling of struct and unstruct
ALTER TABLE event ADD CONSTRAINT event_no_insert CHECK (false) no inherit;

-- CREATE sub tables
CREATE table event_page_view (
       CHECK ( event = 'page_view' )
) INHERITS (event);

CREATE table event_page_ping (
       CHECK ( event = 'page_ping' )
) INHERITS (event);

CREATE table event_unstruct (
       CHECK ( event = 'unstruct' )
) INHERITS (event);

CREATE table event_struct (
       CHECK ( event = 'struct' )
) INHERITS (event);

INSERT INTO event_page_view (SELECT * from event_v04 where event='page_view');
INSERT INTO event_page_ping (SELECT * from event_v04 where event='page_ping');
INSERT INTO event_unstruct (SELECT * from event_v04 where event='unstruct');
INSERT INTO event_struct (SELECT * from event_v04 where event='struct');

Now when you do something like:
SELECT * from event where event.event = 'page_view'

Postgresql will only look at the event_page_view table.
We may not use the check constraint on the event field but rather the event_name and event_vendor in the future now that the schema 4-tuple info is now in the events (vendor,name,format,version).
If you did that approach you would create a table per schema for unstructs and have them inherit.
The table name would be something like:
tableName = event.eventVendor.replaceAll("\\.", "_") + "_" event.eventName

But that might get you some unwieldy table names :)

Thanks for sharing @agentgt! Super interesting. In this approach, where do the event-specific columns get created and live?

All the basic columns still live logically in atomic.events (ie the structured ones). Physically though I think postgresql duplicates them but I don't know the internals well enough.
The event_unstruct could contain additional columns perhaps flattened from schemas. For example we do something like this for the link_click schema:
ALTER TABLE event_unstruct
ADD COLUMN link_click_element_id character varying(256);

ALTER TABLE event_unstruct
ADD COLUMN link_click_element_target character varying(128);

ALTER TABLE event_unstruct
ADD COLUMN link_click_target_url character varying(2048);

ALTER TABLE event_unstruct
ADD COLUMN link_click_element_content character varying(1024);

You won't see these fields though unless you directly query event_unstruct (I think) but event_unstruct would contain all the atomic.events columns. You can think of this as partitioning by event type (in fact I think Postgresql calls it partitioning).
Now the advantage to this approach besides the aforementioned performance is you guys can continuously make updates to the events table and all the inheriting schemas will pick up the updates. Obviously they are some caveats that are in the postgresql documentation so it would be wise to check those out.
You can of course partition out later on and migrate. The only fairly tricky thing is that because we are adding columns you must insert into the tables directly (Many examples don't do this because they are partitioning by time for cube analysis. The sub tables are only for performance.). Thus the ETL storage process would have to know how to dispatch to the correct table (we do this on our own but I believe you guys have a mechanism for this with jsonpath and what not).

closing


Support directory recursing instead of relying on all the files to be passed on the command line

Being able to do zigimport . (like zig fmt) is a much better UX, especially for large projects (also, passing all the files as args doen't scale as you will eventually hit ARG_MAX)

Does . imply ./**/*.zig, i.e. zig files in arbitrarily deep folders?

This has been implemented in https://github.com/tusharsadhwani/zigimports/commit/2cf7d52acda465d3fc42a04af9eb69b94106d7b6

The way I'm recursing right now, all files and directories starting with . are ignored.
Can you provide the repository that you're testing on? The couple that I'm using to test don't reproduce the memory leak.

Alright, the leak is happening due to a bug caused by symlinks to paths that don't exist. Fixing...

Fixed with https://github.com/tusharsadhwani/zigimports/commit/b3e9bb2a25b03bfbfd767eb7d0cdce1b334fda0c, can you check?

In hindsight I should have been less confident and made a PR.

$ ls -lah ./examples/zig/legacy/lib
total 0
drwxr-xr-x@ 3 tusharsadhwani  staff    96B Sep  6 02:05 .
drwxr-xr-x@ 5 tusharsadhwani  staff   160B Sep  6 02:05 ..
lrwxr-xr-x@ 1 tusharsadhwani  staff    11B Sep  6 02:05 pkmn -> ../../../..

Infinite recursion. Of course. Disabling symlink following...

Symlinks have been handled, linting the pkmn/engine project works on my machine. Can you confirm?
P.S. Thanks for this project, I found a bunch of edge cases.

$ git checkout 6be9a77596ce8b2474b90f5269880992aa292fa2
[...]

$ zigimports .
./src/tools/transitions.zig:6:0: swtch is unused
./src/lib/gen2/mechanics.zig:37:0: PointerType is unused
[ ... 40 lines hidden ... ]
./src/lib/gen1/test.zig:6:0: DEBUG is unused

$ git checkout 745843a652c717c2d7f74570d10e44574cde4c53
Previous HEAD position was 6be9a775 change sleep mask onstant name after ziglang/zig#21231
HEAD is now at 745843a6 rm unused imports

$ zigimports .
./src/tools/transitions.zig:6:0: swtch is unused
./src/lib/gen2/mechanics.zig:6:0: DEBUG is unused
[ ... 23 lines hidden ... ]
./src/lib/gen1/test.zig:6:0: DEBUG is unused

https://github.com/tusharsadhwani/zigimports/commit/10dfc7c6b982c33b9968939e7888f58a6bb247ce seems to work for me. Can you check once again and confirm it's broken?

Running zigimports without any arguments will throw an error now, fixed in https://github.com/tusharsadhwani/zigimports/commit/3a1bf3ea98ab25cbd1444e15d5dc024e3f679f83

I am running this locally on an M1 mac and it works. This is very weird
I'll add debug logging and a --debug flag, we'll figure it out then

Added debug logging, can you run zigimports . --debug and send the output?
And run git rev-parse HEAD and git status and send the output as well, just in case.

Actually wait. Let me add debug logs to the recursive find files part too.

Okay, done.

Yeah, a message should be printed in case of errors. Thanks for the help!

Very weird that I'm unable to reproduce this at all. I re-created node modules and ran zigimports, in your output it seems like it starts failing after ~1400 files opened, but in my case it opened all 19000+ files with no errors.
$ uname -a
Darwin Tushars-MacBook-Air.local 22.6.0 Darwin Kernel Version 22.6.0: Wed Jul  5 22:22:52 PDT 2023; root:xnu-8796.141.3~6/RELEASE_ARM64_T8103 arm64

output.txt
No ProcessFdQuotaExceeded for me, runs just fine. So the point at which ProcessFdQuotaExceeded happens is a bit unpredictable. I'll look into not having this happening entirely.

Found the culprit, _get_zig_files was opening files recursively, but was doing defer file.close() so directories stay open until the entire directory contents have been recursed into.
Should be fixed now.

Also the reason why it doesn't reproduce for me is probably because of ulimit:
$ ulimit -n
8192

Regarding skipping node_modules, even with node_modules present, zigimports . seems to finish in 1 second for me, so I think it's fine to not do for now.


i18n(zh-tw): add new astro-components.mdx translation

Description

Adds src/content/docs/zh-tw/basics/astro-components.mdx
Fixes broken links

Related issues & labels

Closes #
Suggested label: i18n

Some broken links are detected. I'll mark this PR as draft and try to fix them 🤔

Broken links fixed 🙌

Thanks for your update @mingjunlu
Note that when PR is merged, the pages will be marked as not outdated in translate status.
So my advice is to try to keep page changes to a minimum in a PR. Such as why-astro，you can make a new PR. For some pages with broken links, we need to update it to the latest in this PR.

Thanks for your feedback, @liruifengv. I'll separate 0815060 and create a new PR for it.

Great. For layouts and astro-pages, aslo need to update then to the newest

Finally done! Here are the PRs:

#6980
#7044


Webview not responding to goBack() or injectJavascript()

Hello, I've noticed that the expo webview is not responding to goBack() or injectJavascript even if the reference is not null.
Considering the below code sample,  console.log("backb"); prints correctly a reference, but javascript is not injected and webview won't go back in any way
import { StatusBar } from 'expo-status-bar';
import React, { useState, useRef, useEffect } from 'react';
import { ActivityIndicator, Linking, SafeAreaView,StyleSheet, BackHandler, Button} from 'react-native';
import { WebView } from 'react-native-webview';

export default function App() {
    const webViewRef = useRef()
    const [isLoading, setLoading] = useState(false);

    const handleBackButtonPress = () => {
      console.log("backb");
      const run = `document.getElementsByClassName('gLFyf')[0].value = 'World';`;
        try {
          webViewRef.current.injectJavaScript(run);
          console.log(webViewRef.current)
            webViewRef.current?.goBack()
        } catch (err) {
            console.log("[handleBackButtonPress] Error : ", err.message)
        }

        return false;
    }

    useEffect(() => {
        BackHandler.addEventListener("hardwareBackPress", handleBackButtonPress)
        return () => {
            BackHandler.removeEventListener("hardwareBackPress", handleBackButtonPress)
        };
    }, []);

    return (
      <SafeAreaView style={styles.safeArea}>
<WebView
            originWhiteList={['*']}
            source={{ uri: 'https://google.com' }}
            style={styles.container}
            ref={webViewRef}
            onLoadStart={(syntheticEvent) => {
                setLoading(true);
            }}
            onShouldStartLoadWithRequest={(event)=>{
                if (event.navigationType === 'click') {
                    if (!event.url.match(/(google\.com\/*)/) ) {
                        Linking.openURL(event.url)
                        return false
                    }
                    return true
                }
                else{
                    return true;
                }
            }}
            onLoadEnd={(syntheticEvent) => {
                setLoading(false);
            }}
        /><Button
        onPress={handleBackButtonPress}
        title="GoBack"
        color="#841584"
      />
        {isLoading && (
            <ActivityIndicator
            color="#234356"
            size="large"
            style={styles.loading}
            />
        )}
        </SafeAreaView>
      );
  }
const styles = StyleSheet.create({
  safeArea: {
  flex: 1,
  backgroundColor: '#234356'
},
loading: {
  position: 'absolute',
  left: 0,
  right: 0,
  top: 0,
  bottom: 0,
  alignItems: 'center',
  justifyContent: 'center'
},
container: {
  flex: 1,
  backgroundColor: '#fff',
  alignItems: 'center',
  justifyContent: 'center',
},
});

I'm testing it with expo app in Android

react-native version:  "0.68.2",
react-native-webview version: "^11.23.0"

How about try this - note this last true there
const run = `
      document.body.style.backgroundColor = 'blue';
      true;
    `;

@tero-paananen seems crazy but nothing happens. The super-weird thing is that I've checked the other functions like "canGoBack" an they return true, but the 'reactive-native-webview' is untouchable in any way.
This is my package.json:
{
  "name": "webviewproject",
  "version": "1.0.0",
  "main": "node_modules/expo/AppEntry.js",
  "scripts": {
    "start": "expo start",
    "android": "expo start --android",
    "ios": "expo start --ios",
    "web": "expo start --web",
    "eject": "expo eject"
  },
  "dependencies": {
    "@react-native-community/netinfo": "^9.3.0",
    "expo": "~45.0.0",
    "expo-status-bar": "~1.3.0",
    "expo-updates": "~0.13.4",
    "react": "17.0.2",
    "react-dom": "17.0.2",
    "react-native": "0.68.2",
    "react-native-web": "0.17.7",
    "react-native-webview": "^11.23.0"
  },
  "devDependencies": {
    "@babel/core": "^7.12.9"
  },
  "private": true
}

let js =  "document.getElementById("topnav").remove();
document.getElementById("footer").remove();
";
This script was not working on IOS

How about try this - note this last true there
const run = `
      document.body.style.backgroundColor = 'blue';
      true;
    `;

Still not working on ios

Try property below
const injectedScriptBeforeLoaded = //your js code wrapped in string``
then put it in prop
      injectedJavaScriptBeforeContentLoaded={injectedScriptBeforeLoaded}

That problem happens for the react native webview version you are on...
Kindly downgrade the version
react-native-webview version: "^11.23.0" to react-native-webview version: "^11.18.1"
I think their upgrade broke a code.... I also had similar issue

Had the same issue on the latest version, downgrading fixed it

Hmm, downgraded to 11.18.1 and still doesnt seem to be working. Anyone else having issues?

Hmm, nevermind, latest version works I just had to disable enableApplePay is this expected?
In iOS 13, webpages loaded in WKWebView can now accept Apple Pay. In order to protect the security of Apple Pay transactions in WKWebView, Apple Pay cannot be used alongside of script injection APIs such as WKUserScript or evaluateJavaScript(_:completionHandler:).

If these APIs are invoked before a webpage uses Apple Pay, Apple Pay will be disabled. If a webpage uses Apple Pay before evaluateJavaScript(_:completionHandler:) is invoked, the completion handler will be called with a non-nil NSError. These restrictions are reset every time the top frame is navigated.

Pretty sad how bad things are maintained in the react native ecosystem. Everyone is always just looking for workarounds instead of fixes because the maintainers don't seem to care.

is there any progress to fix the issue?

is there any progress to fix the issue?

I do not have currently issues with this. How about testing example app that is attached with this module.

That problem happens for the react native webview version you are on...
Kindly downgrade the version
react-native-webview version: "^11.23.0" to react-native-webview version: "^11.18.1"
I think their upgrade broke a code.... I also had similar issue

I am now using version 11.23.1

I have found this issue only in android exists in 11.23.1. iOS is working fine. Don't know what about the expo.

I have the same issue, also with the latest version 11.26.0. The method injectJavascript() is not working on Android.

I created new version about Messaging.tsx in the modules example. This works on iOS. I have some problems to run example on Android. Let see...
import React, {useCallback, useRef} from 'react';
import {View} from 'react-native';

import WebView from 'react-native-webview';

const HTML = `<!DOCTYPE html>\n

    <title>Messaging</title>
    <meta http-equiv="content-type" content="text/html; charset=utf-8">
    <meta name="viewport" content="width=320, user-scalable=no">

    <button onclick="sendPostMessage()">Send post message from JS to WebView</button>

`;

/*
Note 'window' in all these places:
- window.postMessage
- window.addEventListener
- window.dispatchEvent
*/

const POST_MESSAGE_JS_INTERFACE = `window.postMessage = function(data, origin) { window.ReactNativeWebView.postMessage(data); };`;

function getInjectableJSMessage(message) {
  return `(function() {window.dispatchEvent(new MessageEvent('message', {data: ${JSON.stringify(
    message
  )}}));})();`
}

const Messaging = () => {
  const webViewRef = useRef<WebView>(null);

  const handleLoadEnd = useCallback(()=>{
    console.log("handleLoadEnd");
    setTimeout(()=>{
      console.log("injectJavaScript / postMessage");
      webViewRef.current.injectJavaScript(getInjectableJSMessage('hello!'));
    },1000)
  },[])

  const handleOnMessage = useCallback((e)=>{
    console.log("onMessage: ",e.nativeEvent.data)
    // I don't know why Alert.alert('message') does not show anything here
  },[]);

  return (
    <View style={{height: 120}}>
      <WebView
        ref={webViewRef}
        source={{html: HTML}}
        onLoadEnd={handleLoadEnd}
        automaticallyAdjustContentInsets={false}
        injectedJavaScript={POST_MESSAGE_JS_INTERFACE}
        onMessage={handleOnMessage}
      />
    </View>
  );
}

export default Messaging;

Just to follow the ticket, all has been focus on "injectJavaScript()" but I'm having problems with goBack() on android.
In short, it doesn't work
I'm working on a refactor of old a RN app, so my new packages are
"react": "18.1.0",
"react-native": "0.70.6",
"react-native-webview": "11.25.0",

My webView component:
const webView = useRef<WebView>(null);
   return (
      <View style={{flex: 1}}>
        <WebView
          style={clearWebView ? null : {marginBottom: 50}}
          ref={webView}
          ...
          />

const goback = () => {
    webView.current?.goBack()
}

The same code works with the old versions:
"react": "17.0.2",
"react-native": "0.65.2",
"react-native-webview": "^11.2.3",

Any ideas to make it work on Android?
P.S. I already tried to use webview v. 11.2.3 with the newest react native version (0.70.x) and it doesn't install
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react-native
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS><0.64" from<EMAIL_ADDRESS>npm ERR! node_modules/react-native-webview
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!


It would be nice if there was significant information added to the info message

https://github.com/doanduyhai/Achilles/blob/81d6174f7245a9730e81d35fa1c9ff899ca8b925/achilles-core/src/main/java/info/archinnov/achilles/internals/context/ConfigurationContext.java#L294
...
[..] INFO - Start injecting dependencies to meta classes
[..] INFO - Start injecting dependencies to meta classes
[..] INFO - Start injecting dependencies to meta classes
[..] INFO - Start injecting dependencies to meta classes
...
This produces quite a number of log entries (50+ in our app) that doesn't really provide any useful information.
I'd like to suggest either reducing it to debug, or adding the entity information.

Agree to reduce it to DEBUG. Indeed this log message is only there to inform the users on which stage the bootstrapping is currently

+1


Choose Hotkey for FancyZones Instead of Only Shift Key

Description of the new feature / enhancement
Currently PowerToys FancyZones only has the option to "Hold Shift Key to activate zones by dragging" to quickly snap windows to zones. I propose a feature to choose a custom hotkey other than just shift such that you select the custom option and then press a key on the keyboard or a mouse button to assign that button. I specify "or a mouse button" because I am personally interested in using the middle mouse click to snap windows so it does not take two hands to re-tile, and I imagine those with programmable mouse buttons would be interested in using them for this feature.
Scenario when this would be used?
Different users may find different hotkeys more comfortable, or they may wish to be able to re-tile windows with a single click and one hand only (using just the mouse) versus with two hands and two clicks (holding shift and left-clicking).
Supporting information
No response

/dup #8184, #2830, #6810


Call Pinkman on computation thread

Operations createPin, changePin, isValidPin, etc may take some time.
I think these operations shouldn't be run on the main thread. A developer can manage which thread to use himself but it would be great to have Rx wrapper around Pinkman or something like.

I haven't been planning to add Rx support to this library, but if there is "community request" I'll consider this ability. You can use a defer operator to deal with it now.

It easily can be done in a separated library. So people who don't need Rx can avoid it as a dependency.


Source Code build not finishing

VSCode Version:None
OS Version:Win 10 64bit

Steps to Reproduce:
1.yarn run watch
2.
It has been 20 mins since the terminal has outputted anything after [16:11:18] Finished compilation with 0 errors after 48040 ms finished. This also occurs on my mac on the exact same step.
C:\Users\jledb\vscode>yarn run watch yarn run v1.5.1 $ gulp watch --max_old_space_size=4096 [16:10:26] Node flags detected: --max_old_space_size=4096 [16:10:26] Respawned to PID: 6648 [16:10:29] Using gulpfile ~\vscode\gulpfile.js [16:10:29] Starting 'clean-client'... [16:10:29] Starting 'clean-extension:configuration-editing'... [16:10:29] Starting 'clean-extension:css-language-features-client'... [16:10:29] Starting 'clean-extension:css-language-features-server'... [16:10:29] Starting 'clean-extension:emmet'... [16:10:29] Starting 'clean-extension:extension-editing'... [16:10:29] Starting 'clean-extension:git'... [16:10:29] Starting 'clean-extension:grunt'... [16:10:29] Starting 'clean-extension:gulp'... [16:10:29] Starting 'clean-extension:html-language-features-client'... [16:10:29] Starting 'clean-extension:html-language-features-server'... [16:10:29] Starting 'clean-extension:jake'... [16:10:29] Starting 'clean-extension:json-language-features-client'... [16:10:29] Starting 'clean-extension:json-language-features-server'... [16:10:29] Starting 'clean-extension:markdown-language-features-preview-src'... [16:10:29] Starting 'clean-extension:markdown-language-features'... [16:10:29] Starting 'clean-extension:merge-conflict'... [16:10:29] Starting 'clean-extension:npm'... [16:10:29] Starting 'clean-extension:php-language-features'... [16:10:29] Starting 'clean-extension:python'... [16:10:29] Starting 'clean-extension:typescript-basics-test-colorize-fixtures'... [16:10:29] Starting 'clean-extension:typescript-language-features'... [16:10:29] Starting 'clean-extension:vscode-api-tests'... [16:10:29] Starting 'clean-extension:vscode-colorize-tests'... [16:10:29] Finished 'clean-client' after 10 ms [16:10:29] Starting 'watch-client'... [16:10:29] Starting compilation... [16:10:29] Finished 'clean-extension:configuration-editing' after 30 ms [16:10:29] Starting 'watch-extension:configuration-editing'... [16:10:29] Finished 'clean-extension:css-language-features-client' after 32 ms [16:10:29] Starting 'watch-extension:css-language-features-client'... [16:10:29] Finished 'clean-extension:css-language-features-server' after 33 ms [16:10:29] Starting 'watch-extension:css-language-features-server'... [16:10:29] Finished 'clean-extension:emmet' after 35 ms [16:10:29] Starting 'watch-extension:emmet'... [16:10:29] Finished 'clean-extension:extension-editing' after 37 ms [16:10:29] Starting 'watch-extension:extension-editing'... [16:10:29] Finished 'clean-extension:git' after 40 ms [16:10:29] Starting 'watch-extension:git'... [16:10:29] Finished 'clean-extension:grunt' after 42 ms [16:10:29] Starting 'watch-extension:grunt'... [16:10:29] Finished 'clean-extension:gulp' after 44 ms [16:10:29] Starting 'watch-extension:gulp'... [16:10:29] Finished 'clean-extension:html-language-features-client' after 45 ms [16:10:29] Starting 'watch-extension:html-language-features-client'... [16:10:29] Finished 'clean-extension:html-language-features-server' after 46 ms [16:10:29] Starting 'watch-extension:html-language-features-server'... [16:10:29] Finished 'clean-extension:jake' after 48 ms [16:10:29] Starting 'watch-extension:jake'... [16:10:29] Finished 'clean-extension:json-language-features-client' after 49 ms [16:10:29] Starting 'watch-extension:json-language-features-client'... [16:10:29] Finished 'clean-extension:json-language-features-server' after 50 ms [16:10:29] Starting 'watch-extension:json-language-features-server'... [16:10:29] Finished 'clean-extension:markdown-language-features-preview-src' after 52 ms [16:10:29] Starting 'watch-extension:markdown-language-features-preview-src'... [16:10:29] Finished 'clean-extension:markdown-language-features' after 53 ms [16:10:29] Starting 'watch-extension:markdown-language-features'... [16:10:29] Finished 'clean-extension:merge-conflict' after 54 ms [16:10:29] Starting 'watch-extension:merge-conflict'... [16:10:29] Finished 'clean-extension:npm' after 54 ms [16:10:29] Starting 'watch-extension:npm'... [16:10:29] Finished 'clean-extension:php-language-features' after 55 ms [16:10:29] Starting 'watch-extension:php-language-features'... [16:10:29] Finished 'clean-extension:python' after 56 ms [16:10:29] Starting 'watch-extension:python'... [16:10:29] Finished 'clean-extension:typescript-basics-test-colorize-fixtures' after 57 ms [16:10:30] Starting 'watch-extension:typescript-basics-test-colorize-fixtures'... [16:10:30] Finished 'clean-extension:typescript-language-features' after 61 ms [16:10:30] Starting 'watch-extension:typescript-language-features'... [16:10:30] Finished 'clean-extension:vscode-api-tests' after 63 ms [16:10:30] Starting 'watch-extension:vscode-api-tests'... [16:10:30] Finished 'clean-extension:vscode-colorize-tests' after 64 ms [16:10:30] Starting 'watch-extension:vscode-colorize-tests'... [16:11:15] [monaco.d.ts] Starting monaco.d.ts generation [16:11:17] [monaco.d.ts] Finished monaco.d.ts generation [16:11:18] Finished compilation with 0 errors after 48040 ms

[monaco.d.ts] Finished monaco.d.ts generation
[16:11:18] Finished compilation with 0 errors after 48040 ms

The build finished. It's a watch task so it doesn't end.


From version Node v8.9.0, require.resolve() accepts custom paths

See the new require.resolve.paths(request) and the updated require.resolve(request[, options]).
resolve-from can use these new exposed functions instead of accessing private functions of Module.
Of course, only when they are available.

Awesome to see this finally being added to core.

I tried this out using require.resolve with the paths option but couldn't make it work given this structure:
unicorn
    node_modules
        foo.js
    bar
        cat.js
index.js

const path = require('path');

require.resolve('foo', {paths: [path.join(__dirname, 'unicorn', 'bar')]});

I'd expect it to use the regular module resolution and just traverse upwards.

It should be
const path = require('path');

require.resolve('foo', {
  paths: require.resolve.paths(path.join(__dirname, 'unicorn', 'bar'))
})

It should work anyway if it follows the usual module resolution which the documentation states it does. Anyway, it doesn't work using your example either.

require.resolve doesn't work with custom paths (no idea why), but this module works fine

but it can't work with exports access the esm entry

Now that NodeJS have paths option built in, then i suppose that there is no point in upgrading this pkg to ESM and we could just deprecate this package and say something like: just use native function, here is how...


Color and size per glyph

This is great!
Feature request for scatter plots: would be fantastic if you could set the color and size of each glyph individually, for visualization of multidimensional scientific data.

I agree and started to implement it.

Thank you, as @jeammimi is saying, he's working on it in #10

Fixed by #10 thanks to @jeammimi

Hey, I'm trying to plot a 3D scatter with colorscale. Something like this :

I am using ipyvolume.pylab.scatter but I don;t understand how I can have colors in such fashion, I looked at #10 and tried it using the color option.
A trivial code which follows the idea that I'm using to achieve the above shown image.
import ipyvolume.pylab as p3
import numpy as np

#points
x = np.arange(10)
y = np.arange(10)
z = np.arange(10)

#colors
#For each point x,y,z the color corresponding to the same index in c should be used
c = np.arange(10).astype('float64')/10

p3.clear()
p3.scatter(x, y, z, color=np.meshgrid(range(3),c)[1], size=3)
p3.show()

The output :

Can I directly use c without using np.meshgrid() or something similar? Note that I used np.meshgrid just to create an array with required shape. I'm sure there is a cleaner way to do this.
How do I set a colorscale as shown in the first image? And how to add a colorbar legend?
Any help is appreciated, thanks :-)

Hi,
maybe https://github.com/maartenbreddels/ipyvolume/issues/12 would have been useful, but otherwise the code below might help. The idea is to use a matplotlib colormap, that maps to rgb(a) values.
import ipyvolume.pylab as p3
import numpy as np
import matplotlib.cm
#points
x = np.arange(10)
y = np.arange(10)
z = np.arange(10)

#colors
colormap = matplotlib.cm.viridis
c = colormap(np.linspace(0, 1, len(x)))
p3.clear()
p3.scatter(x, y, z, color=c, size=3)
p3.show()

Hope this helps, let me know!
cheers,
Maarten

Hey! This works just fine :)
Yes, I should have looked at #12
Since I probably cannot use pyplot.colorbar for this, I'm still trying to find a way to create legends for multiple scatters on the same cell output display. Anyway, the matplotlib colormap technique works well for the colors.
Thanks a lot!
Although I'm new to this, it's quite simple to use and integrates really well with ipywidgets as compared to matplotlib. Awesome stuff :-)

Hi,
thanks!
Indeed, there isn't any colorbar support yet in ipyvolume. You can try to plot that in the same cell using matplotlib, make sure you call pylab.show(), although I've never plotted just a single colorbar it should be possible.
cheers,
Maarten


As a cook, I can upload new recipes

so that I can refer to them in the future.

classify under add command, almost the same as issue #2


19c Installer complains of low memory

Seems default memory for VM must be increased from current 2048M, to a higher value, in the Vagrantfile.
Warning:
oracle-19c-vagrant: [WARNING] [DBT-11209] Current available memory is less than the required available memory (1,536MB) for creating the database.
oracle-19c-vagrant:    CAUSE: Following nodes do not have required available memory :
oracle-19c-vagrant:  Node:oracle-19c-vagrant                Available memory:1.3859GB (1453176.0KB)

@yogiboy This warning occurs with both the 18.3.0 and 19.3.0 single-instance database boxes. It's fairly harmless; the boxes build and run fine as is. Increasing the VM memory to 3072 MB in the Vagrantfile eliminates the warning.
@gvenzl If you agree that this is a legitimate issue, and that increasing the VM memory to 3072 MB for the 18c and 19c boxes is the correct solution, I'd be happy to submit a PR.  (Or, if you'd prefer, I could add a commit to #230, since the affected files are already being updated there.)

@yogiboy This warning occurs with both the 18.3.0 and 19.3.0 single-instance database boxes. It's fairly harmless; the boxes build and run fine as is. Increasing the VM memory to 3072 MB in the Vagrantfile eliminates the warning.
@gvenzl If you agree that this is a legitimate issue, and that increasing the VM memory to 3072 MB for the 18c and 19c boxes is the correct solution, I'd be happy to submit a PR. (Or, if you'd prefer, I could add a commit to #230, since the affected files are already being updated there.)

Nonethless, I recently came across this amazing repo. Thank you putting this together. I have just started learning vagrant etc.

Hi @PaulNeumann, I'm hesitant to increase the memory too much as still many laptops have only 8GB overall and it's unlikely that people will ever decrease the memory of the VM (why would they, things may break). It's handy to have multiple VMs running alongside, and users can always increase the memory if they wish to.
So, instead of increasing it to 3GB straight, why not just add the 200 (256) MBs that are required to get the installer not to raise that warning? And to round it up, just make it 2300MB, as 2304 looks just odd to me.

@gvenzl That makes sense. I'll test to make sure 2300 MB works for both 18c and 19c.  Do you want a separate PR for this, or do you want me to add it to #230? Right now, I think #230 is ready to go.

Hi @PaulNeumann, now that PR #230 is merged, mind if you provide a separate one for this one?

@gvenzl Will do.


ENH: Add sample weight test LinearModel

Adds **kwargs to fit ala scikit-learn to make it compatible with sample_weight
e.g.
LinearModel().fit(X, y, sample_weight=sample_weight)

LGTM. I can merge once it's green.

travis failure is unrelate.


while updating flutter gettting this error.

shaileshmishra@Shaileshs-MacBook-Pro appliction_poc % flutter doctor -v
Downloading Dart SDK from Flutter engine <<<<<<< HEAD
5babba6c4d25fa237bbf755ab85c9a0c50b3c6ec
=======
6ac856380fa4f66469552ce986edf59f65f7cd68
>>>>>>> 3595343e20a61ff16d14e8ecc25f364276bb1b8b...
curl: (3) URL using bad/illegal format or missing URL

Failed to retrieve the Dart SDK from: https://storage.googleapis.com/flutter_infra_release/flutter/<<<<<<< HEAD
5babba6c4d25fa237bbf755ab85c9a0c50b3c6ec
=======
6ac856380fa4f66469552ce986edf59f65f7cd68
>>>>>>> 3595343e20a61ff16d14e8ecc25f364276bb1b8b/dart-sdk-darwin-x64.zip
If you're located in China, please see this page:
  https://flutter.dev/community/china

Note: I am not from china

Hi @ishaileshmishra
Which Flutter version you're upgrading from? Please provide flutter doctor -v
Did you modify Flutter SDK? Please try git reset --hard then upgrade

Got solution. Tried to update flutter but the patch file downloaded in flutter sdk was broken.
Deleted and reinstalled the flutter sdk. Worked fine.
Thanks.

Given your last message I feel safe to close this issue, if you disagree please write in the comments and I will reopen it.
Thank you


Add note about formatting on save to README

This actually doesn't work as I thought it would. It format's the code, but then you have unsaved changes caused by the formatting. Not sure what why though, the same code works with Prettier.


AKS Private DNS Permissions error

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
We're getting a permission issue similar to this one AKS Private DNS while creating the AKS private cluster. We enable private dns zone and assign the private DNS zone ID as below.
private_cluster_enabled = true
private_dns_zone_id     = data.azurerm_private_dns_zone.azmk8s.id

However, upon running the Terraform code to create the private cluster, we get the customPrivateDNSZoneMissingPermission error, as shown below.
Service principal or user-assigned identity must be given permission to read and write to custom private dns zone /subscriptions/{sub-id}/resourceGroups/rg-dns-connectivity/providers/Microsoft.Network/privateDnsZones/privatelink.westeurope.azmk8s.io. Check access result not allowed for action Microsoft.Network/privateDnsZones/write
One way to solve this is to assign the "Private DNS Contributor" permissions to the AKS user-assigned identity, but that doesn't seem to be a good solution following the hub-spoke approach, as in this case, we will be allowing landing zone corp subscriptions services to get permissions over the private DNS zones in the connectivity subscription.
Another way I am thinking of solving this is to delegate the private DNS zones to the land-zones subscriptions, but the CAF module doesn't have any such feature to enable this.
Following the CAF best practices, any suggestions or help for solving this issue would be appreciated.

Hi I think this is the wrong repo - you want https://github.com/aztfmod/terraform-azurerm-caf

@matt-FFFFFF We are not using https://github.com/aztfmod/terraform-azurerm-caf
We use this terraform-azurerm-caf-enterprise-scale module to deploy the management group hierarchy, including the governance, policies, and access controls. The module also deploys private DNS zones in the connectivity subscription.
The issue is that whenever we provision a private AKS cluster or enable a private endpoint for any of the CORP landing zone subscription services, it asks for the Private DNS Contributor permissions on the private DNS zones in the connectivity subscriptions.

fixed by #919


Manifest V3 adaptation?

Hey! I'm wondering if you're aware that Google is going to turn off all Manifest V2 extensions at the beginning of 2023?
https://developer.chrome.com/docs/extensions/mv3/mv2-sunset/
Because it doesn't look like you've worked on an adaptation to MV3 yet?
Most significantly (in terms of Roll Together), in MV3 you can no longer have a persistent background page. Instead you have ephemeral service workers that can't persist any global constants/variables. This presents an issue in particular for the backend socket connection which is currently maintained there.
I forked Roll Together a while back (before you switched to typescript) to experiment with making a party watch extension (crosswatch) that would work across several services. I had to drop it half a year ago as I moved to a new home. Then I picked it up again a few weeks ago, and then it was announced today that Wakanim, Funimation and VRV would have their stuff moved to CR, which now maybe kind of makes the idea a bit obsolete :P
In any case, I had actually started restructuring it for MV3. So if you want an idea for how to do it, my approach was to make a pinned "status" page tab instead of a popup when you activate the extension. The status page tab then remains open while watching synced videos, and can run a script with persistent variables and constants (including socket connections). Then you can move most of what was previously the background script over there, while maybe showing some status information on the page. It shouldn't be too hard to adapt to MV3 this way, and it has the added benefit of being backwards-compatible with Manifest V2, which is useful if you intend to maintain a Firefox version as well. The Firefox implementation of MV3 will probably not be ready for a while yet.
Thank you for your work on Roll Together, and best regards!

Thanks for sharing your adaptation idea for Manifest V3.
I read about it before but I am waiting to see what kind of solutions will be used for that kind of extension.
One possibility that I think was to move the WebSocket connection to the page itself. And one solution that I saw people using was to have some kind of system to keep the background page always alive.


:sparkles: The AboutScreen header image animation ON/OFF function and the Fallback function for devices with low memory capacity have been implemented.

Issue

None
This PR will be opened after the following PRs are merged

#637

Overview (Required)

Links

Screenshot (Optional if screenshot test is present or unrelated to UI)

Before
After

Movie (Optional)

Before
After

@takahirom
This PR is designed to

animation ON/OFF
turning FallbackMode ON/OFF

I would like to close and reopen the PR since it handles both of the above at once. 🙇
Please let me check before closing. ✋
I am thinking of using the contents of No. 2 of this PR to address the #650 problem.
What do you think about the above?

@takahirom
Sorry, am I correct that this post is in response to this PR reference? 🙇
If so, is it correct to say that both of the above two functions are unnecessary? 🙇

I apologize for the unclear answer. I believe we don't have settings for animation or image display. Having these settings would be difficult for users to manage.

@takahirom
I understand.
I will discard this implementation. 🫡
We will implement it in a way that judges only by API Level, not by the amount of memory installed.

I think the problem lies in the OS implementation of Vector Drawable.

Discard this PR to be handled by #673.


Eventing TLS: Add AddressableFromDestination method on the resolver

After https://github.com/knative/pkg/issues/2711 is implemented, as the Eventing TLS feature track describes we should add a new method on the resolver as follow:
AddressableFromDestination(ctx Context, dest Destination, parent interface{}) (*Addressable, error)

The implementation should be similar to URIFromDestination
https://github.com/knative/pkg/blob/main/resolver/addressable_resolver.go#L75-L76C10
but with the following additional algorithm/logic:

If status.addresses of the resolved destination is set (has at least 1 element)

if dest.ref.address is not specified:

Select the first (in order) address from all addresses
If no address is found use status.address of the resolved destination

else

Select the first (in order) address from all addresses with name == dest.ref.address
If no address is found return an error explaining that the address with name <address_value> cannot be found/resolved

else return status.address of the resolved destination

Since the new AddressableFromDestination is a superset of URIFromDestination, the URIFromDestination should make use of the AddressableFromDestination method but return the  resolved URI
Additional Info

https://docs.google.com/document/d/1H-x_oji8LqkCyd7tlsSyclmUe7FAmEJPgRxOU_0pkn8/edit?resourcekey=0-lzDIPJsZOP3G17QE_g1lHw#

/good-first-issue
/kind feature
/area API

/assign


Need multi platform find and replace

Neither find nor sed are available on windows. Need a multi-platform approach. Furthermore, windows doesn't include any native solution.
It appears that only 5 files need to be altered:

/demoapp/gitops/app-of-apps.yaml
/demoapp/gitops/applications/argo-config.yaml
/demoapp/gitops/applications/demo-application.yaml
/demoapp/gitops/applications/ingress-nginx.yaml
/demoapp/gitops/applications/cert-manager.yaml

Windows
set YOURHANDLE=YourGitHubUserNameHere
set YOURREPOSITORY=YourGitHubRepoHere

docker run --rm -v %cd%:/files ubuntu:kinetic sed -i "s_github.com/AloisReitbauer/progressiveDelivery-masterclass_github.com/%YOURHANDLE%/%YOURREPOSITORY%_g" /files/demoapp/gitops/app-of-apps.yaml
docker run --rm -v %cd%:/files ubuntu:kinetic sed -i "s_github.com/AloisReitbauer/progressiveDelivery-masterclass_github.com/%YOURHANDLE%/%YOURREPOSITORY%_g" /demoapp/gitops/applications/argo-config.yaml
docker run --rm -v %cd%:/files ubuntu:kinetic sed -i "s_github.com/AloisReitbauer/progressiveDelivery-masterclass_github.com/%YOURHANDLE%/%YOURREPOSITORY%_g" /demoapp/gitops/applications/demo-application.yaml
docker run --rm -v %cd%:/files ubuntu:kinetic sed -i "s_github.com/AloisReitbauer/progressiveDelivery-masterclass_github.com/%YOURHANDLE%/%YOURREPOSITORY%_g" /demoapp/gitops/applications/ingress-nginx.yaml

#60 will close this


Issue when the anchor is not on the same page

Hello,
So i'm trying to use Scrollable Anchor on a link in the navbar to scroll down to an anchor on the homepage. It's working perfectly when i'm on the homepage but if i'm on another page it redirects me to the homepage but doesn't scroll down enough, it stops halfway.
Can you help me please ?
I've tried to use the goToAnchor method but i'ts not working either
Thanks !

same

Same problem, would love a solution.


Add support for multiple range selectors

Text animators can have multiple range selectors, but BM currently only exports one selector per animator.
Update to handle all selectors.
For official players compatibility, only export the full selector array when
a) more than one selector is present
and
b) "Include non supported properties" is checked

thanks! I'll try to support multiple selectors in the player soon


V2 Separation for Followed Sites and Personal Calendar and personal email

Category

[x] New Feature

What's in this Pull Request?
Moved V1 web parts Followed Sites and Personal Calendar & Email into V2 solution
Updated libraries (pnpjs,pnp/reusablecontrols, pnp/propertypanecontrols, etc..)
Updated Package-Solution to be version <IP_ADDRESS>
I noticed the uifabric icon was different for the web part. I used the Icon from V1. But this can be changed back to SiteScan later.

Noticed my commit messages and some of the changes were out of sync... but all changes should be there... just some commit comments, don't exactly match the actual change.

@bcameron1231 Followed sites tested great. Personal calendar though did not as webApiPermissionRequests is missing from package-solution.json.

Would you please update package-solution.json to ensure the proper scope is requested for the Microsoft Graph resource?
Also, it would be easier to merge one webpart per PR, I am going to close this PR. Please submit two new PR's, one each for followed sites, and the updated personal calendar.
Lastly, would you please update the two readme's to migrate over the original solution author and version history, adding your name to the author list as well as add a new version entry? I suggest a version date of the date the PR is made.

Thank you for being the first to pave the way for the rest of us here.


Cannot get application to install . . . get error message every time

No doubt it's something I'm failing to recognize or do. But I cannot get this program to install on my system.
Any suggestions on what I should change to get the program to install? I'd sure like to try it out. [Windows 10 Professional]

I have the same issue. [Windows 10 Professional 21H1] Can't find any clue. 😔

Restart the pc, then the installer worked. 🤣


PlotFASToutput MATLAB\Simulink

Dear all,
i am trying to run OpenFast V3.2 with Matlab\Simulink 2021b. i aa running the "Test01" i have a problem with plotting the results. i dont Know what to do to fix this error, can any body help me?
may Matlab dont Know where to find the the Data to plot, but i still dont Know what to write in the Command Prompt.

Bug description

To Reproduce

Steps to reproduce the behavior:

Compile with 'MATLAB 2021b' as instruction from Kumara Raja E

Expected behavior

Screenshots, if applicable

OpenFAST Version 3.2.0

**************************************************************************************************
 OpenFAST

 Copyright (C)  National Renewable Energy Laboratory
 Copyright (C)  Envision Energy USA LTD

 This program is licensed under Apache License Version 2.0 and comes with ABSOLUTELY NO WARRANTY.
 See the "LICENSE" file distributed with this software for details.
 **************************************************************************************************

 OpenFAST-v2.0.0
 Compile Info:
  - Architecture: 64 bit
  - Precision: double
  - Date: Nov 27 2018
  - Time: 17:19:38
 Execution Info:
  - Date: 11/29/2018
  - Time: 10:52:28-0700

System Information (please complete the following information):

OS: <e.g. Ubuntu 14.04 or macOS 10.12>
Compiler: <e.g. GFortran 4.4>
Compiler settings: <e.g. CMake flags or other settings>

Additional context

The OpenFAST-Simulink simulation appears to have finished running, but it can't find the routine that you are calling to plot the data.
PlotFASToutput is a function in the OpenFAST/matlab-toolbox repository: https://github.com/OpenFAST/matlab-toolbox. You will need to download that toolbox/function and make sure that the routine is on your Matlab path.

thank you for your Response. i think it is done Know.
I still have one Question: I am preparing myself to write my Bachelor Thesis around the dynamic Response of the Foundation of tow arts of floating Wind turbine. the Simulation musst be with OpenFAST.
Which modules of OpenFAST should I focussed on?
Is there any training materials to understand how to work with Openfast?
Which compiler is the best in my situation to work with? MATLAB, VS, or others?
i would be thankful for any Tip

Dear @SaeedHAMB,
I don't know what you mean when you say, "foundation of tow arts".  Please clarify.
If you are running Windows, you shouldn't need a compiler unless you need to modify the source code.  NREL has already provided precompiled binaries of the OpenFAST executable and OpenFAST interface to MATLAB/Simulink.
Regarding training material, please see my response to a similar question asked here: https://github.com/OpenFAST/openfast/issues/626.  Moreover, we recently released the presentations and sample models from our most recent workshop held in conjunction with NAWEA / WindTech 2022: https://drive.google.com/drive/folders/1Z6hqCxGP57nZTgyTIMi6AkYd83rJuZmh?usp=sharing.
Best regards,

Dear @SaeedHAMB,
To model the dynamics of floating offshore wind turbines in OpenFAST, requires the use of ElastoDyn (for structural dynamics), AeroDyn (for aerodynamics), InflowWind (for ambient inflow wind), ServoDyn (for the controller), HydroDyn (for hydrodynamics), and a mooring module (e.g., MAP++ or MoorDyn) for response of the stationkeeping system.   Modules such as BeamDyn can optionally be used for high-fidelity blade structural dynamics modeling and SubDyn can be optionally be used for hydro-elastic effects and internal member-level loads of the substructure, if desired.
Best regards,


Add github actions for testing MRs

[X] Renames pages action
[X] Set up towncrier action for changelog
[ ] Run pre-commit tasks on ubuntu + py3.10

Expanding to nox environment tests to be done in separate PR

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


DST bugs.

This is nothing too specific, just a question on DST issues in a more generic sense: it seems to me that moment/moment-timezone have some long-lasting issues related to DST, like #4152, #3132, #327, #2749, etc.
I wonder if fixing these is of high priority on your list?
Great great projects, BTW.

Yeah, we'd love to get fixes for those. But no, it's not anyone on the team's priority. So it would have to come as a PR.

A PR to fix DST issues has been submitted here: https://github.com/moment/moment/pull/4164 It would be great if someone could have a look and merge. We use this fix in a widely used calendar application and it works well for us.