Add Ballerina language

This PR adds Ballerina language support. Ballerina is a general purpose, concurrent and strongly typed programming language with both textual and graphical syntaxes, optimized for integration.

@Shan1024
First of all, truly sorry for the long delay in anyone responding to this PR.
Thank you so much for desiring to contribute and writing a language grammar.  Unfortunately, we currently are no longer accepting new languages into the core repository.  The existing maintainers simply don't have time to maintain new languages (fixing bugs, dealing with issues, etc).  We'd still love to host your language as a 3rd party module though - if you're willing to help maintain it.
It's easy to turn your language into a module that anyone who likes can easily integrate with Highlight.js using a single line of code:
<script src="/path/to/highlight.min.js"></script>
<!-- they'd simply add your language modules -->
<script src="/path/to/highlightjs-ballerina.js"></script>
<script>hljs.initHighlightingOnLoad();</script>

It's also possible for users to rebuild the highlight.js library to include specific 3rd party modules, all in a single file (vs loading a bunch of 3rd party languages one at a time).
If you'd like to become the maintainer for this language then we're happy to create a new repository in the highlightjs organization and host the repository there.  That assumes you'd be open to reviewing issues and fixing bugs as they arise.  You're also welcome to host the repo yourself and we'll link to it from our README.
There isn't much in the way of how to docs on this yet, but there are sample repositories and it's honestly pretty easy.  We're still fleshing out an official "spec" for 3rd party modules, but even after that's done it should require minimal changes from the examples we have already.  IE, if you wanted to get started now there isn't any big reason to wait.
My own robots.txt grammar repository is a great, simple example:
https://github.com/highlightjs/highlightjs-robots-txt
If you have any questions feel free to ask.

Closing for inactivity; (also finally cleaning all these old PRs out for the new year).
As mentioned above if you're interested in becoming the maintainer of a 3rd party module, please respond [whenever you find time] and we'd be happy to help you get started.


Files

Missing that a user can add files to the website
Can netlifycms handle files ?
netlifycms
- label: "Manual PDF"
  name: "manual_pdf"
  widget: "file"
  default: "/uploads/general-manual.pdf"
  media_library:
    config:
      multiple: true

done


Windows 8.1/WinRT/XAML platforms do not use the search paths to load .TTF fonts.

The Font Loading routines are not using the CCContentManager to search and load .TTF files.  This should be changed to use the CCContentManager to allow this functionality and bring in line with other platforms.

Closed via commit https://github.com/mono/CocosSharp/commit/b31ede5ca786cd22c73c7b60aea316fcffd1b872


only return visible elements as focusable

This PR fixes #14519
This PR fixes #14520
This PR fixes #14318
This PR fixes #14317
the elements returned by the query contains invisible elements and caused these 2 issues. the fix is to further filter the list to only return visible elements.
also I am moving this function out so that it can be reused (I will reuse it for the query result filter dialog).

Pull Request Test Coverage Report for Build 685153261

9 of 10 (90.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 43.273%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/browser/modal/modal.ts
1
2
50.0%

Totals

Change from base Build 685073699:
0.01%

Covered Lines:
25468

Relevant Lines:
53697

💛  - Coveralls

seeing this late... thanks @alanrenmsft  for this :)


graphql seems to have an issue with vite-node

I apologize if this bug is invalid. I tried searching the internet for similar issues but could not find one.
I believe this has something to do with how libraries are bundled in vite-node.
Here is a reproduction
https://stackblitz.com/edit/stackblitz-starters-se6shc
node src/server.js works but running yarn dev fails
If this is a vite-node or graphql issue, let me know.
Many things can be wrong or it all may just be an incompatibility issue. It would be nice if someone could explain what is going on and let others know the solution if they are encountering the same issue.

The error hints:

Ensure that there is only one instance of "graphql" in the node_modules
directory. If different versions of "graphql" are the dependencies of other
relied on modules, use "resolutions" to ensure only one version is installed.

Which is weird because you do only have one graphql module.
This issue is not caused by graphql-http though, it belongs more to yarn or vite-node.


** (ArgumentError) invalid option :null for field/3

After initial setup when trying to migrate i get the following error:
Compiling 22 files (.ex)

== Compilation error in file lib/oko_shipping/shopify_shops/shopify_shop.ex ==
** (ArgumentError) invalid option :null for field/3
    (ecto 3.8.4) lib/ecto/schema.ex:2208: Ecto.Schema.check_options!/3
    (ecto 3.8.4) lib/ecto/schema.ex:1917: Ecto.Schema.__field__/4
    lib/oko_shipping/shopify_shops/shopify_shop.ex:6: (module)

This error comes after i have recompiled where i got a warning saying:
warning: OkoShipping.ShopifyPaymentGuard.list_available_plans_for_guard/2 is undefined (module OkoShipping.ShopifyPaymentGuard is not available or is yet to be defined)
  lib/shopifex_web/views/payment_view.ex:8: ShopifexWeb.PaymentView.available_plans/2

@imrying Thanks for being a guinea pig for the mix shopifex.install mix tool. I haven't used it in a while, so I guess it was no longer working with new Phoenix versions. I have updated the package with version 2.1.5, which should fix the first issue. Let me know if the second issue still persists, I didn't try to fix that one.

Cool, i will test it tomorow ;-)


[bazel] Convert the build system to bazel

This is a tracking issue for converting the build system to bazel.
The conversion will be performed in such a way that the meson build system remains intact until we have the entire software build converted.  At that time, we'll request feedback and comments on the conversion, address the feedback and then switch the software build from meson to bazel, and delete meson from the project.
The initial conversion will cover only the software build.  Eventually, we'd like to convert the hardware build over to bazel as well, but our first priority is addressing the shortcomings of meson in the software portion of the project.
The conversion will be structured as follows:

Establish the basic bazel configuration.
Create rules for auto-generated resources (ie: register definition headers).
Write rules for DIFs and DIF unittests.
Write rules for base software libraries (sw/device/lib).
Write rules for boot_rom and demo programs.
Write rules for mask_rom and rom_ext.
Write rules for host-based tooling (ie: opentitantool, etc)
Integrate on-target tests into the build system.
Configure the toolchain to automatically define appropriate preprocessor symbols (e.g. OT_TARGET_PLATFORM, OT_HOST_PLATFORM).
Select appropriate mmio/csr preprocessor symbols based on the toolchain.

Longer term:

Convert the //hw/... hierarchy over to bazel, including suitable rules for verilog "libraries" and test benches.

The Mask ROM currently builds with Bazel. The major remaining gap is clang build support.

We discussed and approved this RFC in the TC meeting today.
See also RFC doc.


remove profile prod from pull instructions

Pulling dev and prod images has been causing some confusion, and in at least one instance it caused someone to be unable to pull the images.
The only difference seems to be that the dev profile pulls the IDE image, since otherwise the images are the same between dev and prod, so this should still pull the same set of images.

Thank you Josh! Just to add some extra context, the pull failure was happening on a Windows laptop. When --profile prod was removed from that command all the images were able to be pulled with no issue.


Support NET6 combined with NET5 and netcore3.1

On a system without dotnet:

curl -sSL https://dot.net/v1/dotnet-install.sh | sudo bash /dev/stdin -c 3.1 --install-dir /usr/share/dotnet
sudo ln -sf /usr/share/dotnet/dotnet /usr/bin/dotnet
dotnet build (result OK)
curl -sSL https://dot.net/v1/dotnet-install.sh | sudo bash /dev/stdin -c 5 --install-dir /usr/share/dotnet
dotnet build (result OK)
curl -sSL https://dot.net/v1/dotnet-install.sh | sudo bash /dev/stdin -c LTS --install-dir /usr/share/dotnet
dotnet build (result NOK)

/home/user/.nuget/packages/msbuild.sdk.sqlproj/1.11.4/Sdk/Sdk.targets(97,5): error : The current SDK bundles target framework 'netcoreapp6.0' which is not supported by MSBuild.Sdk.SqlProj. Either install a .NET Core SDK that supports 2.1 or 3.1, or file an issue at https://github.com/jmezach/MSBuild.Sdk.SqlProj/ to add support. [/home/user/Projects/customer/apps/api/db/db.csproj]

It looks like you were using version 1.11.4 which doesn't support .NET 6. We've added initial support for .NET 6 in our preview release 2.0.0-beta.1 which is up on NuGet.org for you to try. If you're still running into issues with that release feel free to post them here.

Yeah sorry, I noticed that too and then closed this issue, but forgot to comment on it.
Thanks for your reply. I noticed that there is a beta release that supports NET6.
How can I try that out?

You should be able to update your project file to read:
<Project Sdk="MSBuild.Sdk.SqlProj/2.0.0-beta.1">

Thanks will try that! Awesome work.

When DacFX 16 hits RTM

@jmezach DacFx already reached version 160.5400.1, waiting for 2.0.0 release :)

You're right, not sure why we didn't get a dependabot PR updating that package though. But anyway, 2.0.0 is up on NuGet.org now :).


Collect a History

If you know an activist and you would like to make an interview please:

Read our Collecting History Guideline
Comment here  ⬇    we'll get in touch with you

@EducationalVideoCenter we should have a video call, it sounds really interesting. As we saw you are based in New York and we in Berlin(Berlin GMT +2). We have 6 hours of difference but that is no impediment for us to talk! 😃  🎉  ☎

@dannyrayman that would be great! Shoot us an email and we can begin to coordinate!
Mary Grueser, EVC Director of Professional Development<EMAIL_ADDRESS>Emmanuel Garcia, EVC Senior Media Coach<EMAIL_ADDRESS>

(I was answering you msg and I run out of buttery...sorry)
@EducationalVideoCenter Hi! how great to hear that. I´d really like to talk with you if possible before you implemented, I had some feedback on the guideline today...and i´m still processing it. I´ll be probably doing some changes on it on the next days...but anyway is there for you to use it. But I really like to get some feedback on your use as well so we can improve
How about talking ? you can Write me to<EMAIL_ADDRESS>and we can make a call .

Hey BA, Yes!  We'd love to set up a time to chat with you!!
What does this coming Tuesday, May 15 look like for you?  Could you do a
zoom/google hangouts/skype call around 6pm your time 12pm our time?
I'll also write you from my email<EMAIL_ADDRESS>On Fri, May 11, 2018 at 6:02 PM, triaslucia<EMAIL_ADDRESS>wrote:

(I was answering you msg and I run out of buttery...sorry)
@EducationalVideoCenter https://github.com/EducationalVideoCenter Hi!
how great to hear that. I´d really like to talk with you if possible before
you implemented, I had some feedback on the guideline today...and i´m still
processing it. I´ll be probably doing some changes on it on the next
days...but anyway is there for you to use it. But I really like to get some
feedback on your use as well so we can improve
How about talking ? you can Write me to<EMAIL_ADDRESS>and we can
make a call .
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Beyondactivismo/Beyondactivismo/issues/39#issuecomment-388496807,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjuMUhiiwkihI-5ef4NWmvvIAmcXqjqtks5txgpmgaJpZM4TjAI6
.


Not transferring NFT smart contract to test opensea via setTokenURI and metadata file

Hi - thanks for this great resource it has been extremely useful in helping me build NFT smart contracts.
I have made it through every step but am having trouble at the last part - transferring to OpenSea.
scripts/set-token-uri.js works and produces the below output:
Let's set the tokenURI of your characters
{
  tx: '0xcc0bc3d1734b5af8d17cdb35db372e66b861933ba8c0531d4021655a4cdd5169',
  receipt: {
    blockHash: '0xc631dfd809ac2220649398301e77f79ecdafc9852695a38fce23479f63d6a1b1',
    blockNumber: 9107658,
    contractAddress: null,
    cumulativeGasUsed: 1761662,
    effectiveGasPrice: '0x3b9aca09',
    from: '0x03485453f0b0511dd6bd4173ad05093b1e0313de',
    gasUsed: 49245,
    logs: [],
    logsBloom: '0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000',
    status: true,
    to: '0xc2c90f5b3a757625c55635901542c3b488cc9788',
    transactionHash: '0xcc0bc3d1734b5af8d17cdb35db372e66b861933ba8c0531d4021655a4cdd5169',
    transactionIndex: 14,
    type: '0x0',
    rawLogs: []
  },
  logs: []
}
0xcc0bc3d1734b5af8d17cdb35db372e66b861933ba8c0531d4021655a4cdd5169
Truffle v5.4.0 (core: 5.4.0)
Node v12.22.4

However when I plugin what I think is the smart contract address for OpenSea test net at 'to:0xc2c90f5b3a757625c55635901542c3b488cc9788' above.. the page that results at OpenSea finally doesn't hold any content?[https://testnets.opensea.io/assets/0xc2c90f5b3a757625c55635901542c3b488cc9788/0](https://testnets.opensea.io/assets/0xc2c90f5b3a757625c55635901542c3b488cc9788/0)
Also the contractAddress field is null ?
I feel i may need to go back a step or two. The .json metadata file and the image file are both at IPFS and pinned.
Any ideas much appreciated. Thank you!

Hi!
Thanks for making the issue here.
https://rinkeby.etherscan.io/address/0xc2c90f5b3a757625c55635901542c3b488cc9788#readContract
I'm looking at the tokenURI and for index 0 I get this:
https://ipfs.io/ipfs/QmfVmgGFMc3vobFBhjXahaUxbripXMiK2SUA2q6sCJN9by?filename=DeusIrae-crop.json
I can't seem to be able to access this metadata though. Can you see the information if you pop this into your browser?

Seems the local IPFS was down. I've now also pinned to Pinata. Should be viewable now!

Got it working now. I think previous unlined IPFS data was due to either IPFS issue, or mismatch between the tokenID's set in set-token-uri.js - also metadata refreshing at OpenSea.
Though a few items seems to be delayed in updating such as 'description' and 'levels' - i.e. they've been updated in a new .json file at IPFS but don't display - the new image DOES display.. odd.


import (cachev1alpha1 "github.com/example/memcached-operator/api/v1alpha1") in memcached_controller.go for Memcached-operator not working

Type of question

General operator-related help

Question
What did you do?
Following the golang based operator tutorial https://sdk.operatorframework.io/docs/building-operators/golang/tutorial/. Followed it step by step. After making all the code changes went on to build the operator using following command :
$ make install

What did you expect to see?
for build to complete without any errors

What did you see instead? Under which circumstances?
controllers/memcached_controller.go:35:2: cannot find module providing package github.com/example/memcached-operator/api/v1alpha1

Environment
Ubuntu
Operator type:

language go

Kubernetes cluster type: minikube (though never reached this stage)

$ operator-sdk version
operator-sdk version: "v1.3.0", commit: "1abf57985b43bf6a59dcd18147b3c574fa57d3f6", kubernetes version: "1.19.4", go version: "go1.15.5", GOOS: "linux", GOARCH: "amd64"

$ go version (if language is Go)
go version go1.15.6 linux/amd64

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.1", GitCommit:"c4d752765b3bbac2237bf87cf0b1c2e307844666", GitTreeState:"clean", BuildDate:"2020-12-18T12:09:25Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}

Additional context
I was able to build and run the operator one time locally. Tried after about an hour or so and got the specified error. After getting this error, checked the github repo specified but got 404. I am fairly new to kube and operators, so any help would be appreciated.

Hi @Jitesh817
The Golang import paths are relative to your $GOPATH. Can you please check if your project is inside github.com/example/memcached-operator ? If you followed the tutorial, I suspect it should be in example-inc/memcached-operator, since the initial part of the tutorial directs to create project inside example-inc. The import statement should basically reflect the location of v1alpha1 package locally. Changing example to example-inc (the folder in which the project is present) should work.

Hi @Jitesh817
The Golang import paths are relative to your $GOPATH. Can you please check if your project is inside github.com/example/memcached-operator ? If you followed the tutorial, I suspect it should be in example-inc/memcached-operator, since the initial part of the tutorial directs to create project inside example-inc. The import statement should basically reflect the location of v1alpha1 package locally. Changing example to example-inc (the folder in which the project is present) should work.

Hi @varshaprasad96
Thank you... that did the trick. Apparently I copied the controller code from the specified repo which changed the package location.

Hi @varshaprasad96
Thank you... that did the trick. Apparently I copied the controller code from the specified repo which changed the package location.


Add TOML  formatting

The hack being present makes pre-commit hang infinitely on my system, so we need a better solution.

Some tests still not passing though?

Ah, I just checked the maintenance status and it’s actually not maintained. So we need a different solution…


breeze-plymouth: add optional nixos branding, enable for plasma5

Motivation for this change
I noticed that the breeze-plymouth theme has some options for customizing it for a distro. I thought it would be nice to do this for nixos.
I then enabled this for the version that's used by plasma5 by default.
Note that we need to set some of the arguments in an override directly in the nixos module, since we want to include the vesion.
The result is now quite a nice splash screen which includes the proper nixos version and logo. There are also some options to set a color gradient in the background, but the selection of colors is quite limited and I wasn't sure any of it looked better than just black, so I left it.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Thank you! Looking at the implementation of the breeze theme, I can see why you didn't make the branded variant independent from the breeze theme even though on the surface there is only the spinner in common. In the end though it may be better to extract it into a separate repository (if the breeze theme is independent from breeze-text.so) to make it the default NixOS theme (not restricting it to KDE users).
However, the current label under the logo is not useful or pretty. For me it reads NixOS 18.03.git.db3d846 which makes it wider than the logo, but this git part of the version is just noise. I'd suggest either using NixOS for all versions, or NixOS for unstable and NixOS 17.09 for the current release. (When the version is empty the theme adds a space after the name, which moves it off center with the logo. I guess this may be counteracted by prepending a space to the name.)
There is also a tiny issue that both commits add some spaces at the ends of the lines. It is better to configure your editor to automatically trim whitespace.

The problem with making it a separate derivation is that if you want to use the nixos branded one with a version, you need to be in a nixos context (like a module) where you have access to the version. So it seems like it has to be defined by an override in a module.
I did think about making it the default plymouth theme (IMO it's a lot nicer than the current default one), do you think I should just go ahead and do that?
I think I thought that the git suffix on the version was just if you'd built it from a git commit yourself, but I see that it's actually always there. In that case I agree, probably best to use the value of nixosRelease instead. I think it's not obvious when the user is on unstable - I could try and match on the string value of defaultChannel, but I'd rather not.

Updated to use nixosRelease and strip whitespace.
I think I'll open a separate PR to suggest making this the default, and possibly also enabling it on the live CD to make a good first impression.

Gentle ping @orivej . Are you happy with this now?

Thanks!
On 18 Nov 2017 10:37, "Orivej Desh"<EMAIL_ADDRESS>wrote:

Merged #31378 https://github.com/NixOS/nixpkgs/pull/31378.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/31378#event-1348347891, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABnuioeBHuk_N_Ie5jGr_T2udipMm2Hgks5s3rNpgaJpZM4QVot-
.

Thank you too!

This is probably for another issue, but when I enable this theme, my laptop hangs on boot to the point when it's completely unresponsive (ESC key doesn't work so I can't figure out what went wrong). If I hit ESC a bit earlier (before it hangs) and hide Plymouth, everything is fine.


Fixed sync dispatch

CheckAccess works all the way around. Access check should be after receiving the message as we don't know from which source we got it.

@JaggerJo Hi Josua, please take a look at this PR.

Thanks for the contribution!
Do you need a published version ?
BTW. What are you using FuncUI for (if you can/want to tell) ?

Do you need a published version ?
No, thank you, I have this fix in my code.

BTW. What are you using FuncUI for (if you can/want to tell) ?
It's a small application, where user logs in, scans several barcodes, application moves him through the steps of scanning different products, then shows a summary and send scanned data to a processing.
I wanted to write an MVU GUI app, stumbled upon FuncUI and liked it.


The development image is using Visual Studio 2015

Instructions are out of date. It says:
"Start Visual Studio 2013 as Administrator.
Click Project > Options > Source Control > Plug-in Selection."
VS 2015 is installed - and the there is no Options under Project.

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

ID: a1486907-b279-d1c2-9cac-acf009a20d26
Version Independent ID: debe77cc-b887-0033-9c68-92c1d5ba1b39
Content: Version control, metadata search, and navigation - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/dev-tools/version-control-metadata-navigation.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @RobinARH
Microsoft Alias: jorisde

@mores, thanks for the feedback.
@jorisdg, please review. Thank you!

We updated the topic and the changes should go live tonight. - Robin


Enable bottom container style and behavior customization

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #1842
Description

Add onScrollListener to edit and review recycler view, so people can override and customize the bottom container behavior when scrolling
Add a divider above bottom container, gone by default
Add attrs and default xml style

Alternative(s) considered
Tried to use BottomAppBar and lift on scroll, elevation won't show and no logic to show the divider.
Type
Choose one: Feature
Screenshots (if applicable)
device-2023-02-13-193814.webm
Checklist

[x] I have read and acknowledged the Code of conduct.
[x] I have read the Contributing page.
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
[x] I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

@shelaghm fyi

@FikriMilano Great work!
One suggestion is to center the button inside the bottom container with 8px padding on top and bottom. See screenshot below.

@shelaghm thanks! I'll add that as the default style

@shelaghm padding fixed!

Excellent, thank you @FikriMilano

Reviewing pr, thanks!


Replace campaign blade with coming soon template

Describe the task
Replace the current campaign blade in HR with a coming soon template.

This issue is completed. Closing it.


[doc, hw] Generated documentation for flash_ctrl's prog_fifo and rd_fifo is wrong.

Looking at the generated docs, the prog_fifo and rd_fifo show very strange layouts, including self-overlap, cross-region overlap, and even overlap with previous registers. Given these are one-element FIFOs shouldn't they just be one 32-bit entry as rendered?

@lenary @sjgitty for visibility.

Triaging: The problem is here: https://github.com/lowrisc/opentitan/blob/0bd7566655a0d18b044a0acb749110273c36bee2/util/reggen/gen_html.py#L277-L278 If win['items'] is 1, then the generated table is too long.

This seems to be fixed now:


Disable motd

Hello,
could you please add an option like "--disable-analytics" for disabling the Message of the Day (motd)? (also this would be more privacy friendly).
Thanks!
Best regards,

Sorry, we wont be allowing the disablement of MOTD.
Sent from my iPhone
On 6/01/2019, at 5:42 AM, muuuh <notifications<EMAIL_ADDRESS>wrote:
Hello,
could you please add an option like "--disable-analytics" for disabling the Message of the Day (motd)?
Thanks!
Best regards,
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_issues_2599&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=T9nzcUry89gAYWdMSJsoFBs2ggTTG0yd1K3WjFSwXug&s=FGW8GsQX4kON2nOg69E-FtJKXZi3ebogMF4S8zu-7JY&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWGrlVWVgAeFflISyZ1lw8MZkRPWDrZNks5vANXWgaJpZM4ZxnpQ&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=T9nzcUry89gAYWdMSJsoFBs2ggTTG0yd1K3WjFSwXug&s=FYtdipewAImJ-pms-iQEnpuwsDOzQW3qPGoPJcZ8wzo&e=.


Update README.md

See

https://github.com/react-native-community/react-native-template-typescript/blob/master/template/ios/HelloWorld/AppDelegate.m
https://github.com/flatlogic/react-native-starter/blob/master/ios/ReactNativeStarter/AppDelegate.m
https://github.com/BranchMetrics/react-native-branch-deep-linking-attribution/tree/master/examples
etc.

This change updates documentation from Swift to Objective-C. We are ok with the documentation remaining in Swift.


Remove/disallow the WaitForARMFeedback label when the BreakingChangeReviewRequired label is added

When adding the BreakingChangeReviewRequired label, first check if the WaitForARMFeedback label is present. If so, remove it.

When adding the WaitForARMFeedback label, first check if the BreakingChangeReviewRequired label is present. If so, do not add the WaitForARMFeedback label.

ARM API reviewers do not review PRs with breaking changes until the breaking changes are fixed or approved, so the WaitForARMFeedback should not be on a PR while a breaking change review is required.

Currently, our pipeline will not add WaitForARMFeedback and ARMReview label when there is breakingChangeReview label.  This function is already existed. Could you give me an PR example?

Was this a recent change?

Eg: https://github.com/Azure/azure-rest-api-specs/pull/21861

It has been running for a time.  It's a bug in our pipeline, our pipeline don't take account the breakingChangeReviewRequired label which added bot. I will fix it and keep updated with you.

I have fixed this issue and deployed it to prod pipeline. Pls monitor other PRs


Firebase Auth crashes with TrustKit

[REQUIRED] Step 2: Describe your environment

Xcode version: 10.3 (10G8)
Firebase SDK version: 5.20.2, 6.2.2, 6.7.0
Firebase Component: Auth (Auth, Core, Database, Firestore, Messaging, Storage, etc)
Component version:  5.20.2, 6.2.2, 6.7.0
TrustKit 1.6.2

[REQUIRED] Step 3: Describe the problem
Firebase Auth crashes when using TrustKit in the project with enabled swizzling
Steps to reproduce:
What happened? How can we make the problem occur?
This could be a description, log/console output, etc.

Install Firebase/Auth and TrustKit using Cocoa Pods
Init TrustKit with swizzling enabled
Exec PhoneAuthProvider.provider().verifyPhoneNumber(...)
App crash

View my sample project
https://github.com/thetminko/FirebaseAuthTrustKit
Console Logs:
2019-08-28 16:20:47.499384+0800 FirebaseAuthTrustKit[21828:4910670] === TrustKit: Proxy-ing NSURLSessionDelegate: GTMSessionFetcher 2019-08-28 16:20:47.499766+0800 FirebaseAuthTrustKit[21828:4910670] Fetcher delegate class: TSKNSURLSessionDelegateProxy 2019-08-28 16:20:47.500185+0800 FirebaseAuthTrustKit[21828:4910670] Fetcher made an extra session: GTMSessionFetcher 0x137a03400 2019-08-28 16:20:47.500357+0800 FirebaseAuthTrustKit[21828:4910670] Couldn't assign delegate. 2019-08-28 16:20:47.504906+0800 FirebaseAuthTrustKit[21828:4910670] Fetcher delegate class: TSKNSURLSessionDelegateProxy 2019-08-28 16:20:47.505126+0800 FirebaseAuthTrustKit[21828:4910670] -[GTMSessionFetcher setFetcher:forTask:]: unrecognized selector sent to instance 0x137a03400 2019-08-28 16:20:47.505945+0800 FirebaseAuthTrustKit[21828:4910670] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[GTMSessionFetcher setFetcher:forTask:]: unrecognized selector sent to instance 0x137a03400' *** First throw call stack: (0x193e5a98c 0x1930339f8 0x193d771c8 0x193e601d4 0x193e61e6c 0x104e630c0 0x104e40df8 0x104e3d634 0x104defb80 0x104df2848 0x104df1eb4 0x104def444 0x104e13d5c 0x104dec9a8 0x104dec748 0x104de6324 0x105058c78 0x105067480 0x104de625c 0x104de5d34 0x193e62010 0x193d43be0 0x104e7b2e4 0x104e79f40 0x193d463c8 0x104e79d48 0x104e7b130 0x1050576f4 0x105058c78 0x1050666fc 0x193dec32c 0x193de7264 0x193de67c0 0x195fe779c 0x1c08e5c38 0x104dd9de4 0x1938aa8e0) libc++abi.dylib: terminating with uncaught exception of type NSException

Stack Trace
thread #1, queue = 'com.google.firebase.auth.globalWorkQueue', stop reason = signal SIGABRT frame #0: 0x00000001939f70dc libsystem_kernel.dylib__pthread_kill + 8
frame #1: 0x0000000193a749b0 libsystem_pthread.dylibpthread_kill$VARIANT$armv81 + 296
frame #2: 0x000000019394fea8 libsystem_c.dylibabort + 140
frame #3: 0x000000019301c788 libc++abi.dylibabort_message + 132
frame #4: 0x000000019301c934 libc++abi.dylibdefault_terminate_handler() + 308
frame #5: 0x0000000193033e00 libobjc.A.dylib_objc_terminate() + 124
frame #6: 0x0000000193028838 libc++abi.dylibstd::__terminate(void (*)()) + 16
frame #7: 0x00000001930288c4 libc++abi.dylibstd::terminate() + 84
frame #8: 0x0000000193033d5c libobjc.A.dylibobjc_terminate + 12
frame #9: 0x0000000105058c8c libdispatch.dylib_dispatch_client_callout + 36
frame #10: 0x0000000105067480 libdispatch.dylib`_dispatch_lane_barrier_sync_invoke_and_complete + 124
frame #11: 0x0000000104de625c FirebaseAuthTrustKit-[FIRAuth setAPNSToken:type:](self=0x0000000282735360, _cmd="setAPNSToken:type:", token=32 bytes, type=FIRAuthAPNSTokenTypeUnknown) at FIRAuth.m:1371:3
frame #12: 0x0000000104de5d34 FirebaseAuthTrustKit-[FIRAuth application:didRegisterForRemoteNotificationsWithDeviceToken:](self=0x0000000282735360, _cmd="application:didRegisterForRemoteNotificationsWithDeviceToken:", application=0x0000000135d02910, deviceToken=32 bytes) at FIRAuth.m:1336:3
frame #13: 0x0000000193e62010 CoreFoundation__invoking___ + 144
frame #14: 0x0000000193d43be0 CoreFoundation-[NSInvocation invoke] + 292
frame #15: 0x0000000104e7b2e4 FirebaseAuthTrustKit__93-[GULAppDelegateSwizzler application:donor_didRegisterForRemoteNotificationsWithDeviceToken:]_block_invoke(.block_descriptor=0x000000016b02e8d0, interceptor=0x0000000282735360) at GULAppDelegateSwizzler.m:849:37
frame #16: 0x0000000104e79f40 FirebaseAuthTrustKit__72+[GULAppDelegateSwizzler notifyInterceptorsWithMethodSelector:callback:]_block_invoke(.block_descriptor=0x000000016b02e830, key=@"GUL_0x282735360", obj=0x000000028142ef30, stop=NO) at GULAppDelegateSwizzler.m:680:7
frame #17: 0x0000000193d463c8 CoreFoundation-[__NSFrozenDictionaryM enumerateKeysAndObjectsWithOptions:usingBlock:] + 176
frame #18: 0x0000000104e79d48 FirebaseAuthTrustKit+[GULAppDelegateSwizzler notifyInterceptorsWithMethodSelector:callback:](self=GULAppDelegateSwizzler, _cmd="notifyInterceptorsWithMethodSelector:callback:", methodSelector="application:didRegisterForRemoteNotificationsWithDeviceToken:", callback=0x0000000104e7b1c4) at GULAppDelegateSwizzler.m:667:3
frame #19: 0x0000000104e7b130 FirebaseAuthTrustKit-[GULAppDelegateSwizzler application:donor_didRegisterForRemoteNotificationsWithDeviceToken:](self=0x00000002816286e0, _cmd="application:didRegisterForRemoteNotificationsWithDeviceToken:", application=0x0000000135d02910, deviceToken=32 bytes) at GULAppDelegateSwizzler.m:840:3
frame #20: 0x00000001050576f4 libdispatch.dylib_dispatch_call_block_and_release + 24
frame #21: 0x0000000105058c78 libdispatch.dylib_dispatch_client_callout + 16
frame #22: 0x00000001050666fc libdispatch.dylib_dispatch_main_queue_callback_4CF + 1360
frame #23: 0x0000000193dec32c CoreFoundation__CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE__ + 12
frame #24: 0x0000000193de7264 CoreFoundation__CFRunLoopRun + 1924
frame #25: 0x0000000193de67c0 CoreFoundationCFRunLoopRunSpecific + 436
frame #26: 0x0000000195fe779c GraphicsServicesGSEventRunModal + 104
frame #27: 0x00000001c08e5c38 UIKitCoreUIApplicationMain + 212
frame #28: 0x0000000104dd9de4 FirebaseAuthTrustKitmain at AppDelegate.swift:14:7
frame #29: 0x00000001938aa8e0 libdyld.dylib`start + 4

Podfile
platform :ios, '10.0'
target 'FirebaseAuthTrustKit' do
  pod 'Firebase/Auth', '~> 6.2.0'
  pod 'TrustKit', '~> 1.6.2'
end

Do you have -ObjC in your Other Linker Flags under the Build Settings tab of your target in Xcode?
What version of GTMSessionFetcher are you using?

Hi @morganchen12

I do have -ObjC in Other Linker Flags under Build Settings for the target

I do not explicitly install GTMSessionFetcher. I only have Firebase/Auth and TrustKit in my Podfile.
See my Podfile: https://github.com/thetminko/FirebaseAuthTrustKit/blob/master/Podfile#L7

However, there is an internal dependency, GTMSessionFetcher, under Firebase/Auth.
See here: https://github.com/thetminko/FirebaseAuthTrustKit/blob/master/Podfile.lock#L12

Can you try updating the version of GTMSessionFetcher?

@morganchen12 u mean updating the internal dependency of FirebaseAuth module?

It looks like GTMSessionFetcher version the latest from the Podfile.lock - https://github.com/thetminko/FirebaseAuthTrustKit/blob/master/Podfile.lock#L31
Something else is going on.

I'm able to reproduce the crash. Thanks for providing the sample!
Based on the type of delegateDispatcher in this screen shot, it looks like a problem triggered by TrustKit's swizzling.

@paulb777 Thanks and appreciate for helping! May I ask if I am logging this issue at the right place or should I log this issue to TrustKit?

@thetminko I don't fully understand the issue, and it's still possible there's an issue with Firebase's swizzling. However, based on the data we have so far, it seems more likely to be a TrustKit issue. I recommend logging it to TrustKit.

@paulb777 Thanks for the recommendation. I forgot to mention one thing. Firebase Messaging is actually working well with TrustKit swizzling. So, does it mean that the issue probably lie in the Firebase Auth module?

I suspect it's because FirebaseAuth uses GTMSessionFetcher, but FirebaseMessaging does not. And the it looks like TrustKit's swizzling is breaking the functionality of GTMSessionFetcher.

Hi @paulb777, thanks for the explanation. I have logged the issue in TrustKit as well. Let me open this issue till I get a confirmation from TrustKit about a fix. Thanks!

@paulb777 Sadly, TrustKit is not going to fix this issue and their suggestion is to turn off the TrustKit swizzling. I had a use case where I could not so I think I have to look for some alternatives. Thanks for the support!

@thetminko Thanks for sharing the investigation.  Good luck solving your use case!


issue #3

Get the md5 has of a file

Added code to complete md5 file hashing : 946df5fe2b6220944c8be6093839d1d54c21e869


Jenkins host is not selected by jq from Route

https://github.com/openshift/jenkins/blob/master/2/contrib/s2i/run#L581
This line does not work, because jq cannot escape JENKINS_SERVICE_NAME  in double quotes.
Replace this:
JENKINS_URL="$JENKINS_SCHEME$(oc get routes -o json | jq -c -r '.items[] | select (.spec.to.name == env.JENKINS_SERVICE_NAME ) | .spec.host ' | head -n 1)/"

to this:
JENKINS_URL="$JENKINS_SCHEME$(oc get routes -o json | jq -c -r --arg JENKINS_SERVICE_NAME "$JENKINS_SERVICE_NAME" '.items[] | select (.spec.to.name == $JENKINS_SERVICE_NAME) | .spec.host ' | head -n 1)/"

Thanks for letting us know,
Please feel free to submit a pull request and we will review it.

OK, I'll submit when I have free time. I would add there (to support Kubernetes as well):
  JENKINS_SCHEME="https://"
  if [[ $(oc get routes -o json | jq -c -r '.items[0].spec.tls')  == "null" ]]; then
    JENKINS_SCHEME="http://"
  fi
  JENKINS_URL="$JENKINS_SCHEME$(oc get routes -o json | jq -c -r --arg JENKINS_SERVICE_NAME "$JENKINS_SERVICE_NAME" '.items[] | select (.spec.to.name == $JENKINS_SERVICE_NAME) | .spec.host ' | head -n 1)/"
  if [ -z "${JENKINS_URL}" ]; then
      JENKINS_URL="$JENKINS_SCHEME$(kubectl get ingresses -o json | jq -c -r --arg JENKINS_SERVICE_NAME "$JENKINS_SERVICE_NAME" '.items[] | select (.spec.rules[].http.paths[].backend.service.name == $JENKINS_SERVICE_NAME) | .spec.rules[].host ' | head -n 1)/"
  fi


fix for shifted buttons if devconfig is enabled

If you have created a file named „tweaks“ under /etc/config/ which contains CP_DEVCONFIG=1  the devconfig button will shown in the webui under settings > system control but it shifted a few other buttons.
how to fix:
repleace the following code in control_panel.cgi under /www/config/
if { "[read_var /etc/config/tweaks CP_DEVCONFIG]" != "" } {
puts ""
puts ""
puts "
devconfig
"
puts "		"
incr i
}

by this code:
if { "[read_var /etc/config/tweaks CP_DEVCONFIG]" != "" } {
puts "
"
puts "
devconfig
"
puts "

	"
incr i
}

if you also want to supplement the devconfig menu with a link to the known devicetypes repleace the following code in devconfig.cgi under /www/tools/
proc cmd_main {} {
global env urlsid
title "HomeMatic Configuration"
body {
h2 "HomeMatic Configuration"
h3 [url "Channel List" $env(SCRIPT_NAME)?cmd=list_channels&$urlsid]
h3 [url "Device List" $env(SCRIPT_NAME)?cmd=list_devices&$urlsid]
h3 [url "Interface List" $env(SCRIPT_NAME)?cmd=list_interfaces&$urlsid]
h3 [url "Device Status" $env(SCRIPT_NAME)?cmd=device_status&$urlsid]
h3 [url "Link List" $env(SCRIPT_NAME)?cmd=list_links&$urlsid]
h3 [url "Service Messages" $env(SCRIPT_NAME)?cmd=service_messages&$urlsid]
h3 [url "RSSI" $env(SCRIPT_NAME)?cmd=show_rssi&$urlsid]
}
}

by this code:
proc cmd_main {} {
global env urlsid
title "HomeMatic Configuration"
body {
h2 "HomeMatic Configuration"
h3 [url "Channel List" $env(SCRIPT_NAME)?cmd=list_channels&$urlsid]
h3 [url "Device List" $env(SCRIPT_NAME)?cmd=list_devices&$urlsid]
h3 [url "Interface List" $env(SCRIPT_NAME)?cmd=list_interfaces&$urlsid]
h3 [url "Device Status" $env(SCRIPT_NAME)?cmd=device_status&$urlsid]
h3 [url "Link List" $env(SCRIPT_NAME)?cmd=list_links&$urlsid]
h3 [url "Service Messages" $env(SCRIPT_NAME)?cmd=service_messages&$urlsid]
h3 [url "RSSI" $env(SCRIPT_NAME)?cmd=show_rssi&$urlsid]
h3 [url "Devicetypes" /tools/devicetypes.html]
}
}

@HMside Thanks for this kind information. I will try to integrate your suggested changes into the next version of RaspberryMatic


Update README.md

Added MobaXterm

Unsubscribe Me.
Thank you

Il 04.06.2024 03:46, Piju ha scritto:

  Added MobaXterm

  You can view, comment on, or merge this pull request online
    at:
    https://github.com/trimstray/the-book-of-secret-knowledge/pull/432
  Commit Summary

    9d0eebb
      Update README.md

  File Changes
  (1 file)

     M README.md (1)

  Patch Links:

    https://github.com/trimstray/the-book-of-secret-knowledge/pull/432.patch
    https://github.com/trimstray/the-book-of-secret-knowledge/pull/432.diff

  —
    Reply to this email directly, view it on GitHub, or unsubscribe.
    You are receiving this because you are subscribed to this
    thread.Message
      ID<EMAIL_ADDRESS>


ADD: unofficial repository.

@qianzhangxa Like we talked, here my PR to add a unofficial community repository to provide mesos binaries.

Thanks @andreaspeters!


Arbitrary stack overflow in SSLSocketChannel2.read()

Every so often I get a StackOverflowError in SSLSocketChannel2.read().  I don't see any correlation between when they happen; seems random.  Here's my stack trace:
0   java.lang.StackOverflowError
1       at org.apache.harmony.xnet.provider.jsse.SSLEngineImpl.unwrap(SSLEngineImpl.java:412)
2       at javax.net.ssl.SSLEngine.unwrap(SSLEngine.java:385)
3       at org.java_websocket.SSLSocketChannel2.unwrap(SSLSocketChannel2.java:150)
4       at org.java_websocket.SSLSocketChannel2.readRemaining(SSLSocketChannel2.java:254)
5       at org.java_websocket.SSLSocketChannel2.read(SSLSocketChannel2.java:220)
6       at org.java_websocket.SSLSocketChannel2.read(SSLSocketChannel2.java:241)
7       at org.java_websocket.SSLSocketChannel2.read(SSLSocketChannel2.java:241)
8       at org.java_websocket.SSLSocketChannel2.read(SSLSocketChannel2.java:241)
........
254     at org.java_websocket.SSLSocketChannel2.read(SSLSocketChannel2.java:241)
255     at org.java_websocket.SSLSocketChannel2.read(SSLSocketChannel2.java:241)
256     at org.java_websocket.SSLSocketChannel2.read(SSLSocketChannel2.java:241)
257     at org.java_websocket.SocketChannelIOHelper.read(SocketChannelIOHelper.java:12)
258     at org.java_websocket.client.WebSocketClient.interruptableRun(WebSocketClient.java:229)
259     at org.java_websocket.client.WebSocketClient.run(WebSocketClient.java:188)
260     at java.lang.Thread.run(Thread.java:1019)
As near as I can tell, it looks like for some reason no bytes are getting transferred from inData to dst, repeatedly.  Not sure why, though.

@Erhannis Thanks man, your solution fixed the issue for me. Cheers!

Issue was fixed by @Erhannis.
Thx for your help.
Closing issue!
Greetings
marci4


Add ability to upload new commuting data.

It would be nice to be able to:

[ ] Do this on the fly while the server is running.
[ ] Persist the uploaded data between sessions.

It is at best possible to persist the data only until the app goes to ~sleep on ShinyApps.  To do better one would need to utilize AWS or equivalent.
Could likely set recurring task on work PC to redeploy each night.
Still worth doing upload functionality to demonstrate it can work.  Might be useful as internal modeling tool when coupled with a download.


Implement Austria 🇦🇹

No solid leads or good official information was found on how to validate the Austrian Personal Identification Number.
Help is needed to implement it, if possible.

I will pick this one 🙂


Terminal z-index prompt seems to go on top of scrollbar

Can you pls provide repro steps? Not sure what that image is indicating

Shrink the panel to make the pwsh terminal really small
Shrink the VS code window to reduce its width so there's a scrollbar at the bottom
:bug: The prompt seems to cover the scrollbar

Can repro, pretty minor issue but solving this could fix some other layout edge cases:

This only happens when tabs are showing

It also only starts when the terminal is < 180px

Caused by this minimum:
https://github.com/microsoft/vscode/blob/5f1fec188f6b90f8fbffe17ddb9f5c4854e7af1e/src/vs/workbench/contrib/terminal/browser/terminalGroup.ts#L19-L22

I can still get the issue to occur on the latest Insiders on Windows.
It is a small issue though, so feel free to mark it as a wontfix.

@rzhao271 that's the overall window's scroll bar, not the terminal view's scroll bar. 👍 as won't fix as things are expected to get a little wonky when the window is too small for the workbench.


check incorrect endpoint configurations

Describe the feature
Validate the configuration of existing VPC endpoints for private subnets even it has route to the NAT gateway / instance.
Use Case
Sometimes the pipeline is deployed in the subnets with NAT gateway / instances. However, the VPC endpoints might be configured for the subnets, it might cause the connectivity issues due to the endpoints's configuration.
Proposed Solution - Optional

Other Information

related issues #87
Notes:

[ ] this feature has an UI update
[ ] this feature contains an implementation guide update

This is a :rocket: Feature Request

closed by #446


Add Youtube Video Series

•Adds youtube video series into tutorials

not sure why these jobs failed, the url works.

Maybe better category could be Videos? :thinking:


Latest builds failing with RegEx parsing issue

See example job on OpenSearch:
Pattern syntax error near position 0: recursive wildcards must form a single path component

I've noticed the same problem on my own blog (private repository). Any thoughts on what might be causing it?
The job I am running is mapped to the following YAML file:
name: Link Checker

on:
  repository_dispatch:
  workflow_dispatch:
  schedule:
    - cron: "00 18 * * *"

jobs:
  linkChecker:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2

      - name: Load Excludes
        run: |
          LYCHEE_EXCLUDE=$(sed -e :a  -e 'N;s/\n/ /;ta' .lycheeexclude)
          echo "LYCHEE_EXCLUDE=$LYCHEE_EXCLUDE" >> $GITHUB_ENV

      - name: Link Checker
        uses: lycheeverse/lychee-action@master
        with:
          args: --accept=200,429 --exclude ${{ env.LYCHEE_EXCLUDE }} --exclude-mail "**/*.md"
        env:
          GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}

      - name: Create Issue From File
        uses: peter-evans/create-issue-from-file@v3
        with:
          title: Link Checker Report
          content-filepath: ./lychee/out.md
          labels: report, automated issue

So, as it turns out - something changes and the globs need not be wrapped in quotes. Not sure if this was documented/changed somewhere.

Hey @dend I never had quotes but it seems the parsing issue is still there but I don't see any errors on my run like you do. https://github.com/lycheeverse/lychee/issues/436

@vipulgupta2048 this worked in previous releases, and even affected OpenSearch, as linked below. Not saying that using quotation marks was the right approach, however something definitely changed in latest releases that resulted in using those breaking jobs 😄

This should be fixed in master now. Can you check again @dend or @vipulgupta2048?

The master version work. Looking forward to the release soon: https://github.com/lycheeverse/lychee-action/issues/68#issuecomment-1031422829


Bug in moving selection boxes

I've been getting an error where as soon as the event triggers to move a rect selection box, it jumps to some location off canvas with the handles in the top right corner, I've attached a vid showing the problem and the error that occurs on the stack is:
Uncaught TypeError: this.svg.viewBox.baseVal is null
    resizeObserver webpack:///./node_modules/@recogito/annotorious/dist/annotorious.min.js?:21
    yd webpack:///./node_modules/@recogito/annotorious/dist/annotorious.min.js?:21
    yd webpack:///./node_modules/@recogito/annotorious/dist/annotorious.min.js?:21
    Sd webpack:///./node_modules/@recogito/annotorious/dist/annotorious.min.js?:21
    Ad webpack:///./node_modules/@recogito/annotorious/dist/annotorious.min.js?:21
    Ld webpack:///./node_modules/@recogito/annotorious/dist/annotorious.min.js?:21
    emit webpack:///./node_modules/@recogito/annotorious/dist/annotorious.min.js?:1
    Fd webpack:///./node_modules/@recogito/annotorious/dist/annotorious.min.js?:21
    emit webpack:///./node_modules/@recogito/annotorious/dist/annotorious.min.js?:1
    Ad webpack:///./node_modules/@recogito/annotorious/dist/annotorious.min.js?:21
    mouseUp webpack:///./node_modules/@recogito/annotorious/dist/annotorious.min.js?:21

https://user-images.githubusercontent.com/7831621/127176921-e5384db3-f436-4a8b-be3a-1310065318c5.mp4

Weird. Are you modifying the Annotorious SVG element in some way? Is the element under Annotorious an image? I could imagine this happens when you attach it to a normal DOM node.

So I've only changed the css for it and have my own formatter (using Annotorious's interface for it), the image I am using is an empty image with my height and width set before anno starts, but it sits on top of a canvas that I am piping PDF data into

Aha, so it appears to lock the selection box to a 1:1 aspect when I use a 1x1 transparent png, and its doing the jump bug when the img has no src property

Annotorious uses the image naturalWidth and naturalHeight properties to size the annotation canvas. Just guessing that this might not work if the img tag had no src, or you're using eg a 1x1px image as a placeholder. Possible workaround, perhaps, would be to skip the img altogether, attach Annotorious to the canvas, and monkey-patch the canvas with fake naturalWidth/naturalHeight props equal to element width/height.

Thanks, I'll give that a try.

thanks again @rsimon for the lead, I got something pretty workable.
for posterity here is how to do it :
use the below code to generate an empty png of the right natural size using a canvas, in my case I use vue so my target img for annotations has its src value bound to the output of the generator, this keeps it consistent with the pdf pages I am pulling in (PDF pages can have independent sizes from the entire document)
this allows me to stream in the PDF in a canvas under the annotation layer in a performant layer, and I am tracking the annotation layers size and only running the generator if it changes size
I'll look at putting a pull request to add an option in annotorious to swap between using natural sizes and provided sizes for the annotation layer just to cover future cases like this (annotating non-images)
imageGenerator(pdfViewport){
    console.log('imageGen::pre', pdfViewport)

    let internalCanvas = document.createElement('canvas')
    internalCanvas.width = pdfViewport.width
    internalCanvas.height = pdfViewport.height

    const internalContext = internalCanvas.getContext('2d')

    $data.emptyOutput = internalCanvas.toDataURL('image/png')

    console.log('imageGen::post', $data.emptyOutput, internalCanvas, internalContext)
},

Thanks for this! I think another fix (inside Annotorious) would be to keep using naturalWidth/-height like before, but falling back in width/height of that doesn't exist. With this in place, you should be able to attach Annotorious to the canvas element directly I think.

Seems like a good plan, I did a load of googling and couldn't dig up ways to make natural sizing mutable


ARC-2372: Adding pathway that supports translation for subtitles - SRT

Looks generally OK, did you try doing the batch case in 1 LLM call (as opposed to calling the "single" pathway lots of times)?
Also, would suggest plural in naming: translate_subtitles vs. translate_subtitle for the batch pathway.


clear up master.lock

One customer report he failed to push content to his local git repo. after investigate we found the "master.lock" file was there causing the issue.
we have logic go clean other lock file, we may as well as to do the same to clean up master.lock.

Yes, I thought we were already doing that. Is master.lock something we create, or something git creates?

It is what git created.  Somehow it got left over.   We just need to add it to this list.    We are still investigating what led this.

One issue is, this lock is per branch. i think it is ok to add "master.lock" to our cleanup list, since most of git user will have master branch.
but if issue happen on a different branch, we will have to advice to manually remove it as work-around
or we clear all the locks under "refs/heads" when we perform cleanup
I suggest to clear all the locks under "refs/heads". what do you think? @davidebbo @suwatch

That sounds reasonable to me (with proper logging). We have our own lock, and there shouldn't be any operations going on that we're not aware of. Well, I guess it could happen if use does git commands using Kudu Console while a deployment is happening, but that's a bit of an edge case.


Make Commands and World apis consistent

Resolves #1253 #1562
This makes the Commands apis consistent with World apis. This moves to a "type state" pattern (like World) where the "current entity" is stored in an EntityCommands builder.
In general this tends to cuts down on indentation and line count. It comes at the cost of needing to type commands more and adding more semicolons to terminate expressions.
I also added spawn_bundle to Commands because this is a common enough operation that I think its worth providing a shorthand.

This is a pretty big change to user facing apis, so please weigh in if you have alternative ideas. I'd like to avoid touching these any more than we need to.

User-facing changes to Commands, collected for your convenience:

spawn: no longer takes a bundle. Now much easier to spawn entities with zero / one component.
spawn_batch: new convenience method to get back old behavior.
'entity:' Does... something? It's not documented, but I think it sets the current entity to use with the various EntityCommands
add:  replaces add_command
insert_bundle: replaces with_bundle
insert: replaces with
current_entity / with_current_entity: removed
insert / remove / despawn: now use commands.entity(entity).remove()

Now, thoughts. I love these changes. The API is much clearer, more consistent and more explicit, without making anything significantly harder to do.
The previous API often worked in... difficult to grasp and challenging to intuit ways as soon as you stepped beyond copying directly from the examples. The new EntityCommands seems much nicer to write custom commands for.
This API also seems like an excellent step towards batching commands due to the increased explicitness and clarity.
IMO these are the right changes to make, and I'd much rather break things all at once and earlier than do it later. The consistency with World in particular is really valuable for reducing mental overhead for intermediate and advanced users.

Is it worth solving #792 quickly before 0.5 to make this API really ergonomic? It fits in well with these changes and would be useful to folks trying to refactor their related code.

the new api looks nice 👍

i like the new API! thanks a lot for that.
however:

entity: Does... something? It's not documented, but I think it sets the current entity to use with the various EntityCommands
id: Seems to be the new replacement for current_entity?

i do agree with the confusion. i feel that they should contain a verb (eg: get_id) so that they consistent with being commands?

Is it worth solving #792 quickly before 0.5 to make this API really ergonomic? It fits in well with these changes and would be useful to folks trying to refactor their related code.

I'd prefer not to. Adding more features than we already are feels like to much risk at this point. Bundle merging also feels like a pretty superficial change from a user perspective (and non breaking). Folks are free to explore this and we can make a call when the code prior to release if the in front of us, but I think it would be a stretch.

i do agree with the confusion. i feel that they should contain a verb (eg: get_id) so that they consistent with being commands?

get_id() isn't a command, its a normal "getter". That means it should be in the x() form instead of get_x().
I do think the naming of world.entity(e1), commands.entity(e1), and world.entity_mut(e1) commands are worth discussing (especially in relationship to id()). These commands return "entity builder" structs for a specific entity (the same struct returned by spawn() and spawn_bundle()).
We could consider alternative names like world.entity_builder(e1) / world.entity_builder_mut(e1), but thats more of a mouth-full for a pretty common operation (especially when we account for the Option<EntityMut> variants: world.get_entity_builder_mut(e1)).

I'd prefer not to. Adding more features than we already are feels like to much risk at this point. Bundle merging also feels like a pretty superficial change from a user perspective (and non breaking). Folks are free to explore this and we can make a call when the code prior to release if the in front of us, but I think it would be a stretch.

I'm alright with this; we can explore it in another PR, probably after 0.5's release.

get_id() isn't a command, its a normal "getter". That means it should be in the x() form instead of get_x().

Your explanation was helpful, and I think that brevity is important to consider here. I think that good docstrings and examples should alleviate the confusion well enough here to stick with the shorter names.

Folks are free to explore this and we can make a call when the code prior to release if the in front of us, but I think it would be a stretch.

 I meant to say: "Folks are free to explore this and we can make a call prior to release if the code is front of us".

Your explanation was helpful, and I think that brevity is important to consider here. I think that good docstrings and examples should alleviate the confusion well enough here to stick with the shorter names.

Cool cool. I'll add the docs.

commands.entity(child).despawn();
commands.despawn_recursive(entity);

It seems like this may have been overlooked, or maybe the hierarchy/state/alien cake examples didn't get updated.

It seems like this inconsistency may have been overlooked, or maybe the hierarchy/state/alien cake examples didn't get updated.

Yup it was overlooked. Fixed!

bors r+

Little late to the party here, but insert would be initially confusing to me, since it seems like I'm adding the component to a list. Maybe attach is a better metaphor for associating a component with an entity? I understand that with doesn't make sense in a world context.


Returning 400: author https://fed.brid.gy/ is on Bridgy Fed, which is not supported

i am getting a 400 error message when I try to update my post

I 2023-07-22 10:31:32.600852+00:00 Returning 400: author https://fed.brid.gy/tomasparks.name is on Bridgy Fed, which is not supported
E 2023-07-22 10:31:32.601256+00:00 <class 'werkzeug.exceptions.BadRequest'>: 400 Bad Request: The browser (or proxy) sent a request that this server could not understand.

https://fed.brid.gy/log?path=%2F_ah%2Fqueue%2Fwebmention%2C%2Finbox&start_time=1690021892&key=https%3A%2F%2Fgallery.tomasparks.name%2Ftrips%2F2023%2F02%2F12%2F063545-MADV.html

Uh oh. Sorry for the trouble, thanks for reporting, will look!

Big refactoring recently, this subdomain bug (gallery.tomasparks.name vs tomasparks.name) evidently snuck Fixed now, will be deployed to prod in ~5m. Thanks again for reporting!


pglogical vs pgoutput

I was wondering what the reasons behind using the pglogical_output plugin over the standard pgoutput plugin were?

Ah, is it due to the desire to track DDL changes?

Although, that looks like it is done by the pgcapture extension

Hi, @judahrand
Yes, DDL capture is done by pgcapture extension.
The reason behind using pglogical is pgoutput was introduced in PostgreSQL 10 and it didn’t support binary representation until PostgreSQL 14.
By using pglogical, pgcapture can support binary logical replication with PostgreSQL 9.4 to 13.
Since now pgoutput supports binary representation in PostgreSQL 14, I will also make pgcapture support pgoutput in future versions.

This also looks like a potentially exciting development for the future!
https://postgrespro.com/list/thread-id/2590610

Hi @judahrand,
Thanks to @KennyChenFight, we can now use the standard pgoutput plugin to decode changes on PostgreSQL 14 and above!


Prevent building in GIL-less environment

This PR lets PyO3 checks Py_GIL_DISABLED build flag and prevents building against GIL-less Python, unless explicitly opt in the UNSAFE_PYO3_BUILD_FREE_THREADED flag.

Hi! the benches CI check fails because of SSL errors from curl. Could you schedule a rerun for it?

Thanks! I'll push a test in a moment!

Thanks 👍
I think easiest place to put a test would be in the test-version-limits job in noxfile.py, which overrides the build config and checks for success / failure. Could add one there to set the flag and check it fails to build (unless the env var is set).

Thank you!


Ability to customize fonts

allow

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @electronika in #requests.

forgot the closing quotes, the issue description should be as follows:
allow different font faces to be selected for headings, headlines, text, etc

Done


Add 白讀 to gwongzau.js

Currently, gwongzau.js only generates 文讀. Gwongzau 白讀 readings are also generally predictable from Middle Chinese readings, and so can be added as an option.

Thank you! But this seems incomplete?

Ah, you are right, I apologize! I forgot to push a few changes before I made this PR. I'll do so soon.

第276行有少少問題，白讀咁推會推埋落曾攝，但曾攝無咁樣白讀法，咁寫會將曾攝推錯音。


SBUS out should deliver the neutral position of servos

currently the SBUS output begins at the moment, when the vehicle is armed. Before there is just no output at all.
For SBUS Servos this works fine, And they are just quiet until armed. Thats ok.
But for ESC/Motors this is a real issue. And for other things like parachute and LED-Drivers as well.
Therefore; the correct handling on SBUS should be the same as for PWM-Out: Always the neutral Position (the trim from the calibration) should be delivered.

We can do that for the next release. In general it would be great to have a developer for S.BUS, do you feel you can maybe find someone from the RC hobby community with some programming skills to pick this up?

This is fixed in 615affd.


Can we make it an easier user experience when beforeContent is empty?

In this situation, the spacing at the top of the page is wrong.
The current solution is to add this to your page:
{% set mainClasses = "govuk-main-wrapper--l" %}
Wondering if we can do anything that makes this simpler/clearer or that, ideally, our users don't need to do anything

We could make the beforeContent block have default content that would provide the correct spacing:
{% block beforeContent %}

{% endblock %}

If we contributed the 'self' feature to the Nunjucks project we could do this in a non-hacky way:
https://github.com/mozilla/nunjucks/issues/164

I wonder if we could do this in CSS instead, perhaps by applying an additional margin to main if it's the :first-child (and thus has no preceding siblings)

Is this arguably a bug, if a user doesn't know to do the 'right' thing, it will be wrong?

We can definitely make it better and I hope we do, but the current feature works as we designed it to, so it's not a bug.

I believe this was addressed in #1371, so I'm going to close it.


Add system76 galp5-1650

Description of changes
Adds the system76 galp5 (1650 version in this case)
Things done

[x] Tested the changes in your own NixOS Configuration
[x] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

The NVIDIA driver is loaded with this configuration:
[sudo] password for aaronh:
  *-display
       product: i915drmfb
       physical id: 2
       bus info: pci@0000:00:02.0
       logical name: /dev/fb0
       version: 01
       width: 64 bits
       clock: 33MHz
       capabilities: pciexpress msi pm bus_master cap_list rom fb
       configuration: depth=32 driver=i915 latency=0 resolution=1920,1080
       resources: irq:184 memory:83000000-83ffffff memory:90000000-9fffffff ioport:1000(size=64) memory:c0000-dffff memory:892000000-898ffffff memory:8a0000000-97fffffff
  *-display
       physical id: 0
       bus info: pci@0000:23:00.0
       version: a1
       width: 64 bits
       clock: 33MHz
       capabilities: pm msi pciexpress bus_master cap_list rom
       configuration: driver=nvidia latency=0
       resources: iomemory:80-7f iomemory:80-7f irq:183 memory:81000000-81ffffff memory:880000000-88fffffff memory:890000000-891ffffff ioport:4000(size=128) memory:82000000-8207ffff

Though system76-power graphics does not seem to work:
[aaronh@garrus:~]$ system76-power graphics
daemon returned an error message: "The name com.system76.PowerDaemon was not provided by any .service files"
I have the hardware.system76.enableAll enabled in my configuration:
https://gitlab.com/ahoneybun/nix-configs/-/blob/main/hosts/x86_64/garrus/configuration.nix?ref_type=heads
I'll add add the system76 generic part as well of nixos-hardware to test.

Yep I needed that for the system76-power service!
[aaronh@garrus:~]$ system76-power graphics
nvidia

I think this might be that NixOS has version 1.1.23 of system76-power. I don't think this not working would be a blocking to add this system since the gaze18 was not blocked though.

@Mic92 I just copied the config from the gaze18 so if we need to address things here I can make a PR for that model as well.

@Mic92 I just copied the config from the gaze18 so if we need to address things here I can make a PR for that model as well.

Likely yes.

If I remove those options then rebuild and reboot nvidia-smi does not work so I think it is needed.

Alright @Mic92 I was incorrect and with the current changes nvidia-smi shows that the NVIDIA driver is loaded though system76-power graphics now reports this:
[aaronh@garrus:~]$ system76-power graphics
compute
so I think it is needed for NVIDIA mode at least but with Compute it should be similar to Hybrid.

Alright so I think compute is because I'm in the 6.6 kernel now and before I was in the 6.10 kernel series. I can't build 6.11 with the system76-module which is why I'm back in 6.6.

Alright with the current state of this PR puts the system in PRIME Offload Mode which I think is a good middle ground. You can use the NVIDIA GPU when needed and is pretty close to system76-power's Hybrid mode.

@mergify queue


Missing parameter - /l

[Enter feedback here]
The article is missing a parameter - /l . The command 'mstsc /l' (without the quotes), displays a list of monitors and their arrangement (offset from the center). This information is needed if a user wants to create a custom RDP file which uses more than one monitor, but not all monitors off the system (e.g not the full-screen option).
Output of the command from a sample machine with 3 monitors attached:

Tested on Windows Server 2019, Windows 10 and 11.

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

ID: eddd1822-7939-85d7-d010-25963b182434
Version Independent ID: 8597bd89-e0f0-1f73-8c3c-7215bbf8c0e8
Content: mstsc
Content Source: WindowsServerDocs/administration/windows-commands/mstsc.md
Product: windows-server
Technology: windows-commands
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

Hey @officedrone,
Thanks again for contacting Microsoft. We value your feedback.
A documentation change has been made adding the remaining parameters for mstsc and will be live after the next business day. We will now be closing this thread.
Thank you and have a most pleasant day. #please-close


feat(hubble): support highlighting and hints in query statements

Query statements support highlighting and hints

is this PR finished?

finished

Add gremlin keyword hints, basic function

Add gremlin keyword hints, basic function

@imbajin


Fix a false negartive for Performance/RegexpMatch

Follow up https://github.com/rubocop-hq/rubocop/pull/6965.
This PR fixes a false negartive for Performance/RegexpMatch when using RuboCop 0.68 or higher.
# example.rb
def foo
  if /re/.match(foo, 1)
    do_something
  end
end

RuboCop 0.67
RuboCop 0.67 warns.
% rubocop _0.67.2_ --only Performance/RegexpMatch
Inspecting 1 file
C

Offenses:

example.rb:2:6: C: Performance/RegexpMatch: Use match? instead of match
when MatchData is not used.
  if /re/.match(foo, 1)
       ^^^^^^^^^^^^^^^^^^

1 file inspected, 1 offense detected

RuboCop 0.68 (Before)
RuboCop 0.68 does not warn. This is a false negative.
% rubocop _0.68.1_ --only Performance/RegexpMatch --require
rubocop-performance
Inspecting 1 file
.

1 file inspected, no offenses detected

RuboCop 0.68 (After)
This PR fixes a false negative.
% rubocop --only Performance/RegexpMatch --require rubocop-performance
Inspecting 1 file
C

Offenses:

example.rb:2:6: C: Performance/RegexpMatch: Use match? instead of match
when MatchData is not used.
  if /re/.match(foo, 1)
       ^^^^^^^^^^^^^^^^^^

1 file inspected, 1 offense detected

This fix uses the following <> node pattern notation introduced in RuboCop 0.68, so it supports Ruby version 0.68 or higher.
https://github.com/rubocop-hq/rubocop/pull/6965

Before submitting the PR make sure the following are checked:

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

I will release RuboCop Performance 1.2.0 after being merged with this PR.

I think we should add a spec example that would catch this.

I'm lacking to explain. I noticed that existing tests are falling.
https://circleci.com/gh/rubocop-hq/rubocop-performance/419
So I think that there is no need to add a test to this PR because they are covered by existing tests.
I think that it is good to add a test of NodePattern to RuboCop Core if adding a test.


Move this resource to ComputerManagementDsc

In https://github.com/PowerShell/DscResources/issues/374 there are a proposal to move this resource over to the module ComputerManagementDsc, it would be a good opportunity to make breaking changes (fixing open issues) and making it more HQRM.
@bcwilhite or @nehrua what are your thoughts on this? Is this something already being worked on?

I don't think this is in the works, but I'll defer to Brian for confirmation.  I do think its a good fit for ComputerManagementDsc.

This is now in progress over in ComputerManagementDsc.

This has been completed. I'll raise a PR to this repo to mark it as deprecated (similar to xTimezone: https://github.com/PowerShell/xTimeZone).


Tweaks to Ibex to get core_ibex UVM to build with VCS

We're not quite there yet, but it's much closer*! I've put sensible comments on all the commits in the PR, but they are mostly to do with cleaving more carefully to the IEEE SystemVerilog spec and not assuming things that happen to be true for a different simulator.
(*) With the current code, running something like make IBEX_CONFIG=opentitan SIMULATOR=vcs ISS=spike ITERATIONS=1 SEED=1 TEST=riscv_arithmetic_basic_test WAVES=0 COV=0 gets as far as actually generating some binaries. Next: getting sensible trace logs!

Incidentally, I'm pleased to say that we were closer than I thought. I just saw a successful test run of riscv_arithmetic_basic_test (with seed 1). Phew!

Thanks for this @rswarbrick had been meaning to fix the VCS build for a while!


Library: habitat_sim_bindings.so -- Supported Python Version: Unknown

I have python2.7, python 3.5 and python 3.6 installed on my gpu laptop, ubuntu 16.04.
I tried to install api with python3.6, and ./build.sh as indicated in sim readme.
I tested the sim, it works fine.
Then I test on example/example.py, error occurs

Traceback
(most recent call last):
File "~/habitat-sim/habitat_sim/bindings/init.py", line 10, in
from habitat_sim._ext.habitat_sim_bindings import Simulator as SimulatorBackend
ModuleNotFoundError: No module named 'habitat_sim._ext'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "~/habitat-sim/habitat_sim/bindings/dev_bindings.py", line 72, in
import habitat_sim_bindings
ImportError: ~/habitat-sim/build/esp/bindings/habitat_sim_bindings.so: undefined symbol: _Py_ZeroStruct
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "examples/example.py", line 9, in
import demo_runner as dr
File "~/habitat-sim/examples/demo_runner.py", line 15, in
import habitat_sim
File "~/habitat-sim/habitat_sim/init.py", line 12, in
from .simulator import *
File "~/habitat-sim/habitat_sim/simulator.py", line 7, in
from habitat_sim.bindings import *
File "~/habitat-sim/habitat_sim/bindings/init.py", line 18, in
from .dev_bindings import *
File "~/habitat-sim/habitat_sim/bindings/dev_bindings.py", line 84, in
raise ImportError(msg)
ImportError:
Could not import habitat sim bindings
Please follow the building instructions in the README
Found the following libraries
Library: habitat_sim_bindings.so -- Supported Python Version: Unknown
Your python interpreter is version 36
Please re-build habitat sim with this version of python

I have tried to install api for 2 days. Any advice will be appreciated! Thanks!

We currently don't have a nice way to specify different versions of system python and will just generate bindings for whatever python is (which generally defaults to 2.7).  We highly recommend using conda environments to fix this (things should work as expected there, and python setup.py install will work).
If you want to stick with system python, there is a work-around for using different versions of system python here: https://github.com/facebookresearch/habitat-sim/issues/2

Thanks. Conda solved my problem.


README issues

Describe the issue:
The port is incorrect.

Suggestion: Update to 3100.

Describe the issue:
failed to read dockerfile error occurs when execute the command below, for the Dockerfile is actually in the src folder.

Suggestion:
Add a step cd src, or update the command to docker build --tag fastapi-app .\src .

Describe the issue:
Miss the option --template .

Suggestion:
Please provide the --template option, update to azd init --template pamelafox/simple-fastapi-container
@pamelafox for notification.

Thanks! Please see my PR at https://github.com/pamelafox/simple-fastapi-container/pull/8
I made the same change I discussed in the other review - removing azd init in favor of azd auth login, to match the flow of checking out the code first.

@pamelafox I think the modification in https://github.com/pamelafox/simple-fastapi-container/pull/8 is fine.

Great, merged


Enhance the codicon reference in treeview

Thank you for providing the codicon reference features to the TreeView. I have some question during using it:

Can the extension change the color of the codicon when assign it to the tree item? for example, I hope when I'm using ThemeIcon('error'), the icon could be red in the treeview.

Can the codicon in the treeview applies the spin style?

You can't control the color, but you can apply the spin style.

but you can apply the spin style.

Unfortunately not in the ThemeIcon - I am unsure if that should work implicitly or if it should be explicit, using another ctor-argument like modifier.
Also adding @aeschli to this because he has ideas of making this more open for extensions, e.g adding more icons

The ThemeIcon now allows to specify a color: https://github.com/microsoft/vscode/issues/103120

Thank you, @aeschli. Will VS Code team considering also adding the spin style to ThemeIcon?

@jdneo No plans at the moment as I'm not aware of much demand for that feature. But please file a separate issue just for that request.


moshi handling for lists and maps of wire messages

Previously we were relying on moshi's default handling for repeated and
map fields on wire messages; this is fine for primitives, but populates
the collection with the wrong types when dealing with repeated or maps
of other wire messages. This adds proper support for dealing with those
types of collections.

Is this something Moshi should support? If so, should we make that clear with an explicit package so it's not jumbled with other Misk-specific Moshi code?

It's specific to moshi handling of wire (like the other classes in this package)

Although Moshi is Java, so it wouldn't be a direct contribution to Moshi :/


Object.hasOwnProperty() should return true for accessors / variables / functions

In the attached test case, Object.hasOwnProperty() is returning false for everything in Royale (JS), but returns true in Flex for accessors / variables / functions (but not for non-existent names).
Royale:  TestObjectHasOwnProperty.mxml.txt
Flex:  TestObjectHasOwnPropertyFlex.mxml.txt
For this or other objects.

Tough one.
The ecma spec is that has hasOwnProperty does not return true for inherited properties. I think the Flash behavior is different. I'm loath to just mimic the Flash behavior and ignore the ecma spec.
Possibly we should add a hasOwnProperty method to Language which either uses for in (or Object.keys) or walks up the hierarchy (not sure which is most efficient) and only use that if a compiler option is enabled.
Thoughts?


[varLib] make build also accept DesignSpaceDocument objects

with source descriptors containing pre-loaded TTFont instances
Continuation of https://github.com/fonttools/fonttools/pull/1416

Can we use Python 3.x features yet? The whole path handling stuff makes my head hurt. I'd like to have all paths be pathlib.Path objects, no POSIX gymnastics, that are written to disk with as_posix(). Since filenames (not .path) must be relative according to the spec, we don't have to deal with drive roots on Windows.
I'm not super familiar with all the use-cases for designspace objects, but what about making filename a write-only property and always have path point to the absolute location of sources and instances? On writing the DS, the filename property is calulated as filename = SourceOrInstanceDescriptor.path.relative_to(Path(designspace_file)). Then there is also no need for a getSourcePath() method.
Let me think about what that would mean for client code.

A piece of code with an in-memory designspace object with in-memory UFOs or TTFs attached and no path for them set yet will write all font objects to disk somewhere, write the paths to the DS object and write it last.
Loading a designspace file would resolve the filename attributes on loading and instantiate absolute Path objects.
A piece of code moving font objects somewhere else on disk would overwrite the path attributes and write the DS object again.

Maybe I'm forgetting something, but path modification would mean simply overwriting the path attribute with the known new absolute path and writing the DS object to disk would take care of the rest.

Aside from this, the changes look mostly fine to me. One thing that has me scratching my head is the reload_font thing in varLib_test.py. Are we testing the __deepcopy__ changes properly? The TTX dumps we load from the test data are already "properly" serialized data structures with no temporary data, no? Shouldn't we test freshly compiled fonts that still have some working data in them that gets removed on save()?

what you suggest make sense, and i've also always been confused by this duality of path/filename attributes in designspace document. However, I don't want to break existing code for the sake of it. For now, let's keep both path and filename attributes as posix path strings, mutable, and nullable (with all the problems that this implies), and deal with them later in separate PRs.
About the reload_font thing in the test, that's only done because we are comparing TTX dumps for string equality, and there are a few redundant things which are only computed at compile time: e.g. count values (written out inside xml comments in the ttx dump), or in this particular case, the name records in the name table are sorted upon compile. No big deal, the data doesn't change.
I changed the test to load compiled binary test fonts instead of importing them as ttx to address your concern, and it actually caught a bug in that custom deepcoy implementation for SFNTReader.


{ "type": null } in profiles.schema.json violates JSON Schema draft 2019-09 meta-schema

https://aka.ms/terminal-profiles-schema redirects to https://raw.githubusercontent.com/microsoft/terminal/release-1.3/doc/cascadia/profiles.schema.json, i.e. the raw view of https://github.com/microsoft/terminal/blob/release-1.3/doc/cascadia/profiles.schema.json.  That schema uses the https://json-schema.org/draft/2019-09/schema# meta-schema:
https://github.com/microsoft/terminal/blob/327ddd702307952c315b07fc7a815d56fd68f7ba/doc/cascadia/profiles.schema.json#L3
The #/definitions/CloseOtherTabsAction and #/definitions/CloseTabsAfterAction schemas have a subschema  { "type": null }:
https://github.com/microsoft/terminal/blob/327ddd702307952c315b07fc7a815d56fd68f7ba/doc/cascadia/profiles.schema.json#L401
https://github.com/microsoft/terminal/blob/327ddd702307952c315b07fc7a815d56fd68f7ba/doc/cascadia/profiles.schema.json#L420
That is not valid according to https://tools.ietf.org/html/draft-handrews-json-schema-validation-02#section-6.1.1, referenced by http://json-schema.org/specification-links.html#2019-09-formerly-known-as-draft-8. The value of type "MUST be either a string or an array"; null is neither.
{ "type": "null" } would be valid according to the meta-schema, but it would not match the documentation https://docs.microsoft.com/windows/terminal/customize-settings/actions#close-tabs-after-index, which says that the index property is optional but does not say that an explicit { "index": null } is allowed.
The null literals were added in https://github.com/microsoft/terminal/pull/7390.  Meta-schemas were also discussed in https://github.com/microsoft/terminal/issues/7683.

"default": "" for the index property seems a bit dubious too, as it is not valid against the schema of that property. On the other hand, http://json-schema.org/draft/2019-09/json-schema-validation.html#rfc.section.9.2 only recommends validity and does not require it.

@KalleOlaviNiemitalo - I don't think that it is that important to forbid "index": null (and actually not sure how to do it). But it is definitely a time to make our schema compliant. Issuing a PR.

I don't think that it is that important to forbid "index": null (and actually not sure how to do it).

AFAIK, one would do it like this:
            "index": {
              "type": "integer",
              "description": "Close the tabs other than the one at this index. If no index is provided, use the focused tab's index."
            }

If the "index" property is not listed in "required", then the object can be valid without having that property. If the lack of property does not mean the same thing as any valid value, then don't specify a "default". That's how I understand JSON Schema, anyway.


FEAT-#6502: HDK: Add 'enable_multifrag_execution_result=1' HDK launch parameter

What do these changes do?

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #6502
[ ] tests added and passing
[ ] module layout described at docs/development/architecture.rst is up-to-date

LGTM! However won't that slow down any other cases?

We will see in the benchmark reports. According to my experiments, it gets ~8x-10x boost in some cases.

@AndreyPavlenko @anmyachev we need to refresh https://modin.readthedocs.io/en/stable/flow/modin/config.html# I guess


[Prefactoring] Recompute broker IngressHost every time - needed for future dynamic dataplanes

Fixes #

Proposed Changes

Recompute broker IngressHost every time - needed for future dynamic dataplanes; instead of computing it once and setting it as a struct field.
Computing: string building for the ingress service name
Prefactoring for KafkaBroker with dataplane-per-namespace

Release Note

Docs

/test integration-tests_eventing-kafka-broker_main

/hold
Blocked by https://github.com/knative-sandbox/eventing-kafka-broker/pull/2360

/unhold

/test integration-tests_eventing-kafka-broker_main

/test unit-tests_eventing-kafka-broker_main

/test unit-tests_eventing-kafka-broker_main

/test reconciler-tests_eventing-kafka-broker_main


Add filter-full icon

Implements filter-filled from codicons…

Hm... any use cases for this one? I've consulted the tags but couldn't find anything.

Hm... any use cases for this one? I've consulted the tags but couldn't find anything.

For example in VS Code (I think with git lens extension)—but I’ve seen in other places—the filter icon is either filled or outlined depending on whether active filters are applied…

Since Lucide doesn’t do fill, there is no other way to represent a difference.

Well, there's also that indicator dot. 😛
Dunno, just because we don't support filled icons out of the box, it doesn't mean people can't force fill them if they want to, anyhow, I'll let @ericfennis decide, personally, I'm not really a fan of this one. 😅

I agree with @karsa-mistmere. Looking at other icon libraries nobody has an icon like this one. Probably nobody will use it.
I close this one, If other people think differently about this, they can react to this PR. Maybe we will reconsider and reopen it.


Rules: Very basic implementation pending updates from Entity API.

This PR adds support for source_type based transformations. IE, when the source of a particular dataset is murine, the contains-pii value should always be False. This is pending work from @maxsibilla

@jpuerto-psc HuBMAP doesn't have Mouse source, all from Human. And SenNet has their own ingest-api. I've asked @shirey to take a look too.

@yuanzhou - yes I understand that sources, source, and source_type are available in HuBMAP. However, the code takes this into account by using the .get() function. If those fields do not exist, it will simply pass over this new code.
I did it this way so that we could keep SenNet & HuBMAP as aligned as possible.


[Garden] Fixing missing import

This pull request fixes a Python issue when customizing a WAM-V. When the yaml configuration isn't compliant, instead of showing the expected error, we were showing the following error:
[generate_wamv.py-1] NameError: name 'yaml' is not defined

This patch fixes the problem.
How to test it?
Follow this tutorial and you shouldn't get any unexpected error.

When I attempt to run through the part of the tutorial that demonstrates behavior for a non-compliant wam-v, I get the following message instead:
NameError: name 'rclpy' is not defined

This corresponds with the actual fix made to compliance.py, so maybe 'yaml' was a typo?


This is mostly a lot of code removal. None of these things are used by real people as far as I know. There wasn't even tests for most of it.

Travis passed it! Yay!

yay for cleanup! r+

893 lines GONE!

Thank you!


Disable elements in InlineLexer as well

Previously only Lexer checked the disabledElements option, now InlineLexer does as well.

I was also trying to get Renderer to recognize disabledElements (so that images, for example, could be disabled) but I haven't had any luck. I'm not sure how to overwrite the appropriate Renderer methods with noop.  Do you think there's a straightforward way to do it?

I don't even know how I never saw this PR before. Thanks


Question: How to set cookie for a gateway route while using as reverse proxy?

How can we extract and set cookie for a cloud gateway route? was looking at cookie route predicate factory but it sounds like a matcher

that is a matcher. Do you want to set a cookie to send to a downstream service or back to the client? Either way, a cookie is just a header.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Editor can search text within the document

Development of a custom UI for text searching. This is to prepare for the implementation of #20.

@jiayingy @tayjiehao
Consider exploring the search and replace addon suggested by CodeMirror http://codemirror.net/doc/manual.html#addons

Search is done by configuring Codemirror to use sublime text's key bindings


Added various new entries to names-generator.go

left = amazing, big, gigantic, small
right = bezos, dalzell, gates, jobs

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:Morgy93/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Please adjust your list to include as many women as you include men.


Horney smoking snail

Microsoft PowerToys version
<IP_ADDRESS>
Running as admin

[X] Yes

Area(s) with issue?
General
Steps to reproduce
Nsync
✔ Expected Behavior
Coherant behavior showing understanding of a tions and the reactions of actions. Relations in all areas are observied and scenarios non fictional and fictional situations. For correctional purposes and for unjustice.
❌ Actual Behavior
Normal  human reactions . Motivation brute force .short term use only.
Other Software
No response

The PowerToys version you submitted is not a valid PowerToys version. Please update your issue and clarify your issue.

@crutkas Please delete this as SPAM.


parser: separate off parser

We use CommonMark now, but maybe switch to wooorm/mdast.
We separate off parser(Markdown and plain text) from textlint to prepare for switching.
Blocker : implementing location on mdast.

I'v just implemet markdown-to-ast parser.

https://github.com/azu/markdown-to-ast/

It is dependent on jgm/CommonMark's parser.

Maybe we need re-define raw mead..
[label](http://example.com "title")

Is raw of theLink node [label](http://example.com "title") or  label?

landed https://github.com/azu/markdown-to-ast/ in 1.3.0!


⚠ Bofrost.be has degraded performance

In da8b99f, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1227 ms

Resolved: Bofrost.be performance has improved in 28f4f60.


Update App configuration, Cache, Cosmos, Event Hubs Kafka samples for Spring Boot 3

Purpose
Update App configuration, Cache, Cosmos, Event Hubs Kafka samples to provide a doc CONVERT_SAMPLE_TO_USE_SPRING_BOOT_3.md to convert the sample to run with Spring Boot 3.
Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test
Check the new doc to run with Spring Boot 3.
What to Check
N/A
Other Information

After discuss with @moarychan , I think it is no need to create this PR.
The project README illustrates how to convert to Spring Boot 3. If we need to do some code or dependency modifications in each sample, we should illustrate it in each sample.

PR is outdated, close it now.


Update/more ocr phish [dev]

Closed in favour of: https://github.com/CybercentreCanada/assemblyline-v4-service/pull/726


[stable/nginx-ingress] Fixes #7127 - conditionally define pdb

/assign @mattfarina

/assign @scottrigby

/lgtm

/lgtm

/lgtm
(change was just a merge)

@lachie83 could you have a new look - I had to merge with master so I "lost" the /lgtm

/lgtm


consider the browser as well as the driver in selecting the responseparser

Upon upgrading selenium to its latest version (3.9.1), the NoSuchElementError is thrown by the chromedriver instead of selenium. This change selects the correct requesthandler for parsing this error to an {:error, :no_such_element} tuple.
Is there a reason for still running the older 2.48 in the tests?

BTW: with regards to #199 -> if you move to circleCI for testing you can just start a docker container that hosts the latest version of selenium and run the tests against that...


vencord: 1.4.6 -> 1.4.7

Description of changes

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)
[ ] 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/)
23.11 Release Notes (or backporting 23.05 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

[ ] Fits CONTRIBUTING.md.

Welcome as a contributor to NixOS :purple_heart:

RE requested changes: I dont think changinf sha256 to hash is the scope of this PR.


WPT-77 Adding line-height attribute to display normally in Respondr

Fixing this CSS issue with viewed with Respondr: http://imgur.com/SLWiHLq

:+1:


tabnext doesn't update message line

Steps to reproduce
echo ONE > 1.txt
echo TWO > 2.txt
vim -p 1.txt 2.txt

vim now shows the contents of 1.txt with the message line showing
 "1.txt" 1L, 4B ...

Now type :nexttab (or gt) to go to the second tab. Now the message line still shows information for 1.txt, though we're now viewing 2.txt.
There's a similar issue with vim -o and vim -O except that the message line shows information for the last file rather than the first.
By contrast, if you run vim 1.txt 2.txt, the :next command update the message line to show information about the current file.
Expected behaviour
If you navigate to a new buffer, whether it's a tab, a window, or something else, the message line should be updated to show information about the new current file being edited.
Operating system
Ubuntu 20.04.3
Version of Vim
xterm, vim 8.2.3570
Logs and stack traces
No response

This is working as intended.  The message is just the last thing that happened, it is not a buffer title or status.
If you don't like it you could use a TabLeave autocommand to display an empty string.

I still think that visiting a new tab via :tabnext should behave more similarly to visiting a new file via :next, but thanks to your hint I can make it do so, so I'm content to leave this as closed.
I've added this to my .vimrc:
autocmd TabEnter * file
autocmd WinEnter * file

I still think that visiting a new tab via :tabnext should behave more similarly to visiting a new file via :next, but thanks to your hint I can make it do so, so I'm content to leave this as closed.
I've added this to my .vimrc:
autocmd TabEnter * file
autocmd WinEnter * file

The message comes from loading the file into the buffer.  :tabnext
doesn't do that, it only displays another tab, the buffer was already
loaded.
--
How To Keep A Healthy Level Of Insanity:
5. Put decaf in the coffee maker for 3 weeks. Once everyone has gotten
over their caffeine addictions, switch to espresso.
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///