wxSVG: 1.5.14 -> 1.5.15

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/wxsvg/versions.
meta.description for wxSVG is: '"A SVG manipulation library built with wxWidgets"'.

Checks done (click to expand)

built on NixOS
Warning: no invocation of /nix/store/m7cx7m5lzfcbx8qhqisw9ar5jxwmap70-wxSVG-1.5.15/bin/svgview had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 1.5.15 with grep in /nix/store/m7cx7m5lzfcbx8qhqisw9ar5jxwmap70-wxSVG-1.5.15
directory tree listing: https://gist.github.com/17a4f12d9c44ab8608cfbb250aec83b5
du listing: https://gist.github.com/6dac176fea3f1cda5f14e6a1a2639fc8

Instructions to test this update (click to expand)

One-time optional setup to skip building using Cachix:

Install cachix from https://cachix.org/
Use r-ryantm's cache:cachix use r-ryantm

Test this update by entering a nix shell, seeing what is inside the
result, and if applicable, running some binaries:
nix-shell --pure -I nixpkgs=https://github.com/r-ryantm/nixpkgs/archive/294cebc45bf54154a487e81b7126a7e961802951.tar.gz -p wxSVG
ls -la /nix/store/m7cx7m5lzfcbx8qhqisw9ar5jxwmap70-wxSVG-1.5.15

cc @AndersonTorres for testing.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: wxSVG
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: wxSVG
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/m7cx7m5lzfcbx8qhqisw9ar5jxwmap70-wxSVG-1.5.15
shrinking /nix/store/m7cx7m5lzfcbx8qhqisw9ar5jxwmap70-wxSVG-1.5.15/bin/svgview
shrinking /nix/store/m7cx7m5lzfcbx8qhqisw9ar5jxwmap70-wxSVG-1.5.15/lib/libwxsvg.so.3.11.0
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/m7cx7m5lzfcbx8qhqisw9ar5jxwmap70-wxSVG-1.5.15/lib  /nix/store/m7cx7m5lzfcbx8qhqisw9ar5jxwmap70-wxSVG-1.5.15/bin
patching script interpreter paths in /nix/store/m7cx7m5lzfcbx8qhqisw9ar5jxwmap70-wxSVG-1.5.15
checking for references to /build in /nix/store/m7cx7m5lzfcbx8qhqisw9ar5jxwmap70-wxSVG-1.5.15...
/nix/store/m7cx7m5lzfcbx8qhqisw9ar5jxwmap70-wxSVG-1.5.15


Pokerole 1e

Changes / Comments
This is a new character sheet for the Pokerole system. Built to offer a more aesthetically pleasing sheet than the current. This sheet offers additional functionality and a friendly design matching the rulebook's.
Pokerole System
I followed the Building character sheets standards and welcome any comments if I have failed to meet them.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[x] Does this add functional enhancements (new features or extending existing features) ?
[x] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Its for the same version, I've named it 1e since the developers are currently working on a 2nd.
The original sheet is messy and intimidating for new players. I wanted to offer a cleaner looking sheet with more features.
I opted to create a new sheet to avoid affecting any games made in the current.

Thanks for the update. I'll deprecate the old one so people currently using it will no be effected but yours will be the primary one now.


Waters DDA changes

The Waters DDA changes discussed on a call with Matt and Brendan.
Waters DDA data is now processed with the DDAProcessor in the MassLynxSDK. This handles:

Lock mass corrected precursor mass (in future this will be expanded to use peak picking, rather than lock mass correcting the set mass)
Merging of adjacent MS2 scans which relate to the same set mass. The id of such scans is expanded with merged=23-25, though for compatibiltiy the scan=23 is still present, which indicates the first scan in the group.
Omitting the lock mass function

The biggest difference is that in 'DDA mode' SpectrumList_Waters creates each index entry 'on the fly' as the scan is retrieved. This avoids looking up the values twice in the index.
Currently this uses a beta version of the MassLynx SDK, therefore I have made this a draft pull request, so not to be merged now. We plan to have a new release version in the next couple of weeks, at which point I'll update the PR.

Look to be a couple of issues with the build/checks. I think both are a result of the MassLynxSDK being built a different way to previous releases, presumably with different versions of the C++ runtime.

We no longer have an x86 build. Is there a way to disable Waters raw support for the x86 build? Out of interest, is anyone still using x86?
There are some tiny changes in floating point numbers in the expected test results. Would you be happy with updating the expected results, provided we confirm all the changes are very small (they look to be in the 6th significant figure or thereabouts)?

Yes we can disable Waters for x86.

Don't suppose you could give me a hint on where to do this?

Yes we can disable Waters for x86.

Don't suppose you could give me a hint on where to do this?

Don't worry about it, I'll take care of it if you remove the x86 binaries from the 7z. BTW did you add the password to encrypt the 7z?

Well, that was weird. I used GitHub's CLI to checkout this PR. I made my change, tested it, committed it, and then tried to push it. I figured it would either fail to push to your fork or maybe work if it realized it was for a PR to our repo. I did NOT expect it to push to pwiz's master and apparently count this PR as merged. Oops! I had to force reset master to what it was at. And I guess this PR needs to be reissued? Here's a patch for the 32-bit disabling code. Which GitHub requires me to zip for no good reason!
disable-32bit-test.zip


Concurreny with eUTXO model

How does ergo-dex address the concurrency problem with the eUTXO model? Is the DEX limited by one swap per block?
Maybe this information could be added to the readme.

https://github.com/ergoplatform/ergo/issues/279
It seems Ergo blockchain has enabled chaining transaction within one block. So it should be possible to do multiple swaps in one block.
This reddit thread has more discussions.
https://www.reddit.com/r/ergonauts/comments/nz6lk0/dex_eutxo_and_scaling_problems/


Huggingface Spaces Web Demo

Web Demo on Huggingface Spaces using Gradio

Did you accidentally put the Swin Transformer link in README?

@bowenc0221 yes sorry will update the correct link soon https://huggingface.co/spaces/akhaliq/Mask2Former

@bowenc0221 thanks, updated the link

Thanks for the demo.

@AK391 Hello, demo seems to be broken - there is some error with pip installation of fvcore. Could you have a look, Please? I would love to try out this demo

@niemiaszek Hi thanks for letting me know the demo should be back up now

May you share what model is used in the demo?


Remove example from published package

Describe the bug
I don't think the example should be included within the node_module. Perhaps this should be excluded when publishing?
Expected behavior
Examples should be tested when cloning the repo not via the package installed within node_modules.
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Context is I am trying to patch the package with patch-package this folder is giving me issues.

For more context, the pods within the example folder use a symlink for the various React Native pods. Since i think it's a relative path and patch-package downloads the package in a tmp folder. this breaks it.
``ts
Error: ENOENT: no such file or directory, stat<EMAIL_ADDRESS>

@thorsten-stripe  do you have a rough timeline on when this would be published? just so i can plan my next sprint on when to incorporate the public package

@michaelbayday I am working on it already

@michaelbayday hoping to publish later today (Fri May 14th AM PT time)


mediapipe facemesh orientation

Good morning,
I would like to know if a person looks at the camera or not. I can't get the orientation of the face.
Nor the position of the eyes and chin to somehow calculate the pitch, yaw, roll
Can someone help me with a simple example, which I can't find, they are all references to incomplete parts that cannot be seen.
Thank you

Hi @isdito,
Would you please let us know the complete details, platform, solution you are using? It's recommend to fill the template  to get max relevant info to understand the issue better. Thank you!


add check for circular dependency

@bipuladh @alfonsomthd there are other ways as well (like adding a analyser script and create a dependency visualisation etc etc), but webpack seems easiest and relatively fastest...

/test odf-console-e2e-aws

/lgtm


Roll Flutter Engine from b5a6b1c9cba5 to 98df113ca882 (1 revision)

https://github.com/flutter/engine/compare/b5a6b1c9cba5...98df113ca882
2023-07-21<EMAIL_ADDRESS>Roll Skia from 6f2b2e94ebbd to 049f389db75b (1 revision) (flutter/engine#43873)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC bdero@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Rename/move files in vscode and automatically update links

Motivation
Moving or naming a file in vscode file explorer breaks links within documents. It would be great if the links could be automatically updated.
For example renaming a image file would automatically update links within #image() to the new location across the project.
This is a feature included in vscode when editing markdown files. It is disabled by default, but can be set to prompt first or update files automatically.

The first PR updates import paths when renaming source files. We still don't support to handle rename image requests, as we don't have analyzer that identifies link in the document, which is not quite easy. Since we have made some progress on it, we can open anther feature request (issue) and update descriptions.


RefreshBrowserSource gone?

Description:
previously (4.9.....) I had obs.send("RefreshBrowserSource", { sourceName: "streamelements" }); to reload my browsers when they bugged sometimes...
5.x doesnt seem to have that event.... How do we go about it?
Toggeling its visibility seems to work...
but as I explained in #303 it requires lots of code... we go from 1 line to 20 lines of code:
obs.call("GetSceneItemId", {
	sceneName: "browsers",
	sourceName: "streamelements",
})
.then((data) => {
	console.log(data.sceneItemId);
	obs.call("SetSceneItemEnabled", {
		sceneName: "browsers",
		sceneItemId: data.sceneItemId,
		sceneItemEnabled: false,
	})
	.then(() => {
		obs.call("SetSceneItemEnabled", {
			sceneName: "browsers",
			sceneItemId: data.sceneItemId,
			sceneItemEnabled: true,
		});
	});
});

This feature was basically access to that last button Refresh cache of current page

Versions Used (if applicable):

obs-websocket-js version<EMAIL_ADDRESS>obs-websocket plugin version: none
obs-studio version: OBS Studio - 28.0.1 (linux)
node version: v17.9.1

https://github.com/obsproject/obs-websocket/issues/972


Change key combination for showing suggestions to SHIFT + SPACE

Pull Request Test Coverage Report for Build 89

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.316%

Totals

Change from base Build 88:
0.0%

Covered Lines:
1682

Relevant Lines:
1822

💛  - Coveralls


Please upgrade to Django 5.0

Django 5.0 has been out for a week now, and it's the last package that's preventing me from upgrading.

Version 3.4.0 is now on pypi with support for Django 5.0. There is one known issue that has an easy work-around: https://github.com/nshafer/django-hashid-field#known-issues

All the tests are green and everything works fine. Thank you @nshafer and happy new year!


Set a user agent with the version when talking to the registry

This can be used to track what versions of Cargo are being used in the
wild, which can be very useful.
r? @alexcrichton

@bors: r+

:pushpin: Commit ae2b598 has been approved by alexcrichton

:hourglass: Testing commit ae2b598 with merge 71e996e...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 71e996e47b59741c184d1db5937c77e9814e6a4d to master...


Make filters in bookmark scrollable

Idea Description
Make scroll available for the filters in book mark screen
Reference images and links
You can see the right side chip is being bloken from screen shot test
https://github.com/DroidKaigi/conference-app-2023/pull/692#issuecomment-1684035996

🙋♂

💪
assigned!!


Process freezes while trying to parse large binary file

As you may know, there is a Wakatime plugin for the hexrays product IDA Pro, and IDA Pro stores the project info in their custom binary format, and often this file could reach a very huge size. Lately, we've been getting system crashes because wakatime-cli process keeps running with 4gbs of allocated memory, each time you make a change in your project, the new instance of wakatime-cli spawns and starts to eat your memory, resulting in a system crash(and a project changes loss too).
So what I did is I grabbed the arguments that are getting passed to wakatime-cli and tried to just run it within my python script
from subprocess import Popen
from subprocess import PIPE
from pathlib import Path

def launch(*args):
    x = Popen(args, stdout=PIPE, stderr=PIPE)
    x.wait()
    print(x.stdout.read())

wakatime_cli_path = Path('C:\\Users\\retard\\.wakatime\\wakatime-cli-windows-amd64.exe')

while True:
    launch(
        str(wakatime_cli_path.absolute()),
        '--entity', 'SomeEntity',
        '--time', '1694564455.902440',
        '--plugin', 'ida-pro/7.7 ida-pro-wakatime/1.0',
        '--local-file', 'D:\\test.i64',
        '--alternate-language', 'IDA',
        '--category', '"code reviewing"',
        '--key', 'redacted',
        '--alternate-project', 'test.i64'
    )

That's how i confirmed that when i remove the local-file path it works just fine.
When the process froze I checked the opened handles and found the handle to the file, considering that i think you guys can add some flag that would disable all the logic related to this file. I uploaded a .i64 file that causes the process to allocate the memory for it and just freeze, you can try to pass it to the wakatime-cli and you'll see what i mean.
Expected behavior (what should have happened):  wakatime-cli should send the heartbeat and gracefully exit without eating your whole ram
Actual behavior (what went wrong): wakatime-cli keeps running resulting in many instances running at the same time and the system crashes with a blue screen on Windows :^)
Environment:

OS: tested only on windows64
Platform: amd64

Logs:
there's none

I can't download that file as it's broken for me. Is there another way to share with us? Maybe WeTransfer?

@es3n1n I can't download that file as it's broken for me. Is there another way to share with us? Maybe WeTransfer?

Reuploaded it to the WeTransfer

A new lexer is needed in order to avoid wakatime-cli detecting it as Objective C and hanging forever trying to read it. As soon as this PR gets merged and released you'll need to close the IDE and reopen to update the cli version.

Can confirm that everything works as expected on the latest preview release


Regular Expression Denial of Service (ReDoS)

hapi-auth-jwt requires an old version of jsonwebtoken (5.x), which depends on ms which has a ReDoS vulnerability. Please update jsonwebtoken to use at least 7.4.1 which has fixed this issue.
Thanks!
https://snyk.io/test/github/screwdriver-cd/screwdriver.git

This plugin seems to be largely abandoned, see https://www.npmjs.com/package/hapi-auth-jwt2 for a maintained version.

Thanks.


[ACR] Extend weekly pipeline timeout

/azp run python - containerregistry - tests-weekly

API change check
API changes are not detected in this pull request.

/azp run python - containerregistry - tests-weekly


Change copy to Cardano GovTool

Area
Other
Is there new design needed?
Yes
What?
Change copy from Voltaire GovTool to Cardano GovTool
Why?
To better reflect the narrative of the voting.
How?
Throughout the website, replace all occurrences of Voltaire GovTool to Cardano GovTool.
(Optional) User Story with acceptance criteria
No response

All references to Sanchonet changed to "Cardano" on the master Figma file

Verified on Sancho.


Add event operation definitions for known failure modes

Requires #187
Identify additional failure modes that exist during deployment. Then create event operations for each of those states. Modifying the existing error handling logic to utilize these event operations.
Additionally, pull error codes from the Connect logs when they exist. This work may be broken out depending on the scope.

We've done some of this already and tracked it elsewhere


Feature/newsletter form

newsetter modal added in html, click function added to js

newsetter modal added in html, click function added to js

ok - IS


respond-to mixin could be more robust

Sparked by an issue originally reported by @kevinsimper on https://github.com/sass/libsass/issues/891.
I am aware it comes with the below warning, but this doesn't stop people copy-pasting it.

Obviously, this is a fairly simplistic breakpoint manager that will not do the trick when dealing with custom and/or multiple-checks breakpoints.

As it stands this mixin could support more complex query lists if the breakpoint value where unquoted (if it's a string of cause). e.g. http://sassmeister.com/gist/9e8e20ffe4ca184321b7

I like that. I might update it. :)

Great to hear. I've closed the related Libsass issue.

Done for 1.0.1: https://github.com/HugoGiraudel/sass-guidelines/commit/87eb7c2eea03d70c964c416d5356291f89224451.


CTRL+LF is not supported in WSL and should not be used.

Issue Type: Bug
I am editing bash scripts and profiles that are breaking.
I have WSL supposrt installed I amd launching vs code from WSL to edit the files yet it is adding windows line endings and breaking everything.
This makes vscode unsuable.
I really do not understand why windows needs to add Ctrl+LF in everyhting when It understands using just LF.
Windows should follow the standard everyone else is using and stop breaking eveything and causing headaches for developers.
VS Code version: Code 1.39.2 (6ab598523be7a800d7f3eb4d92d7ab9a66069390, 2019-10-15T15:35:18.241Z)
OS version: Windows_NT x64 10.0.18362
Remote OS version: Linux x64 4.4.0-18362-Microsoft

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9700 CPU @ 3.00GHz (8 x 3000)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.93GB (4.92GB free)

Process Argv
--file-uri=vscode-remote://wsl+Ubuntu-18.04/home/brad/.zprofile --remote=wsl+Ubuntu-18.04

Screen Reader
no

VM
0%

Item
Value

Remote
WSL: Ubuntu-18.04

OS
Linux x64 4.4.0-18362-Microsoft

CPUs
Intel(R) Core(TM) i7-9700 CPU @ 3.00GHz (8 x 3000)

Memory (System)
15.93GB (4.92GB free)

VM
0%

Extensions (4)

Extension
Author (truncated)
Version

vscode-line-endings
jha
0.2.0

auto-close-tag
for
0.5.6

remote-wsl
ms-
0.39.9

vscode-icons
vsc
9.4.0

You can configure this with the files.eol setting, otherwise it detects what the file is using already.

Cool thanks where can I find this setting?
And how can I change the line endings on existing files?
I tried some extensions that show line endings but they cant change them.
Still think LF should be the default esp considering im launching it from wsl or at least offer some visible options for line endings.

https://github.com/microsoft/vscode/issues/49418 fixed that shellscripts now use LF by default.
If you start code out of WSL (using the Remote-WSL extension), we use LF as default for all files, unless you have changed the files.eol setting. The default is used for new files. For existing files we always detect what's already there and stay consistent. There are command to switch the line delimiters.


Polaris22Fixup do not fix the window bug in Bigsur 11.0.1？

Polaris22Fixup do not fix the window bug in Bigsur 11.0.1 when using Polaris22Fixup（ V1.3.1）
Also there is some bug here.Double border on the The menu of finder.
Please update it.Thanks very much.

Duplicate of #316


How to use this package with a separate backend

createTRPCClient only allows relative urls, so how can this package be used with a backend on a separate domain or port ?

According to https://github.com/icflorescu/trpc-sveltekit/issues/52#issuecomment-1407628849, it's not the use case meant for this library. You should just use the tRPC vanilla client.
https://trpc.io/docs/vanilla

As @Sazzo says, if your back-end is on a separate domain/port, a  should probably be using the tRPC vanilla client.
tRPC-SvelteKit is suited for monolithic SvelteKit apps.

I need it

As @Sazzo says, if your back-end is on a separate domain/port, a should probably be using the tRPC vanilla client. tRPC-SvelteKit is suited for monolithic SvelteKit apps.

I need it to work in both scenarios, one scenario is as a sveltekit app (monolithic) and therefore the url is local, and the other scenario is when it's running not at theserver.com, lets say it will run as an app through "webview", so i would like to point the URL to the sveltekit monolithic app.
and that way, both ways are covered.


full screen is not working in android mobile

Full screen is not working in android mobile source as youtube means,....

@sivanraj Closing old issues. If you'd like to reopen this, let us know. Thanks!


test(extension): tests lw-8432,lw-8433

Checklist

LW-8643
LW-8645
[ ] Proper tests implemented
[ ] Screenshots added.

coverage: 53.17%. remained the same
when pulling 5256da0d956325be359759e75d89ce08a8c6065d on lw-8643-e2e-tests
into a4c9ba85dc8f90d741f831c3a591161566b1037f on main.


fix: fix setting range for layerId suggestions

This fixes the suggestions for layerIds by adjusting the range property. Newer versions of monaco-editor require the range to be set and to be a single line range.
The range was adjusted so that suggestions replace the characters of a typed word, if there are any. Otherwise suggestions will just be added to the current cursor position.
This also adjusts the default value of the suggestions, so that they will be fetched when refreshing the application view.

Looks good to me! but maybe you can adjust the linting according to the warnings

Looks good to me! but maybe you can adjust the linting according to the warnings

Thanks. Since the warnings are not related to my changes, I would rather have it in a separate PR


Cookies

I dont get any roblox cookies anymore, used on my self, logged in once but still no cookies

The program verifies if the cookie is valid. If itbis invalid, it doesn't shown it. Can I ask which browser/client did you use to log in your roblox account and how long ago was it?

Hi, i used edge and opera gx. After login, i wont get any cookies anymore.

Edge and Opera gx are not implemented yet. The cookies are grabbed only from chrome.


Terragrunt command validation breaks some workflows

Since #2676 (answering #438), Terragrunt validates the Terraform commands and exits early if it is incorrect.
This breaks one, well, creative workflows I had that leverages hooks. Basically, I want to run semi-arbitrary commands in the terragrunt context, and I’m (ab)using before-hooks and non-existing commands.
My particular use-case is: in my team we have found convenient to be able to "lock" a Terragrunt environment − someone is doing some hairy work and no-one else should apply that layer in the meantime. We (ab)use the Terraform lock feature by placing in DynamoDB a lock object (with a nice message in there « locked by Alice because of issue #XYZ » ). But to do that I need to know the region, the DynamoDB table name etc − information that Terragrunt has (ie constructs from various variables etc, work which I certainly don't want to redo)
My solution was thus to use hooks (see below) on commands that do not exist. The before-hook does its job (locking or unlocking), and then Terragrunt fails because the command does not exist but that's fine.
That does not work anymore since #2676 : Terragrunt exits early.
Soooo, not sure what's the best way forward here. My use-case is a bit too hacky and specific to give it first-class support, I think ; How about an environment variable to be able to ignore the validation ?
[1] Full dump of hooks, in case anyone is interested:
  before_hook "lock_layer" {
    # Hook on fictional command to lock the layer
    commands = ["lock"]
    execute = [
      "python",
      "${get_repo_root()}/scripts/layer_locking.py",
      "create",
      "--region",
      local.aws_region,
      "--table",
      local.dynamodb_table,
      "--lock_id",
      "${local.bucket}/${local.key}"
    ]
  }

  before_hook "unlock_layer" {
    # Hook on fictional command to unlock the layer
    commands = ["force-unlock"]
    execute = [
      "python",
      "${get_repo_root()}/scripts/layer_locking.py",
      "delete",
      "--region",
      local.aws_region,
      "--table",
      local.dynamodb_table,
      "--lock_id",
      "${local.bucket}/${local.key}"
    ]
  }

  error_hook "check_layer_lock" {
    # The command failed as it could not acquire the state lock, let's see what's in the lock table
    commands = get_terraform_commands_that_need_locking()
    on_errors = [
      ".*Error locking state.*",
      ".*Error acquiring the state lock.*",
    ]
    execute = [
      "python",
      "${get_repo_root()}/scripts/layer_locking.py",
      "read",
      "--region",
      local.aws_region,
      "--table",
      local.dynamodb_table,
      "--lock_id",
      "${local.bucket}/${local.key}"
    ]
  }

(And yes, the similarity to https://xkcd.com/1172/ has not escaped me :) )

Resolved in v0.50.11 release.


Investigate if type-lift prevents tree-shaking

The PlainDateFactory interface requires a type-lift function (of) that is meant to be a reference back to the function itself.
I suspect that the self-reference prevents any such function (and it's dependencies) from being tree-shaken in a production build 🙀
Investigate if we can drop the type-lift from the interface and the implementations!

Yes, removing type-lift from the factory interface makes tree-shaking work fine, at least with vite.


[SPARK-49349][SQL] Improve error message for LCA with Generate

What changes were proposed in this pull request?
This PR add a new check in CheckAnalysis to improve error message for LCA with Generate.
Why are the changes needed?
Improve error message.
Before this PR:
[INTERNAL_ERROR] Invalid call to dataType on unresolved object SQLSTATE: XX000
org.apache.spark.sql.catalyst.analysis.UnresolvedException: [INTERNAL_ERROR] Invalid call to dataType on unresolved object SQLSTATE: XX000
	at org.apache.spark.sql.catalyst.analysis.UnresolvedAttribute.dataType(unresolved.scala:292)
	at org.apache.spark.sql.catalyst.expressions.LateralColumnAliasReference.dataType(namedExpressions.scala:472)

After this PR:
[UNSUPPORTED_FEATURE.LATERAL_COLUMN_ALIAS_IN_GENERATOR] The feature is not supported: Referencing a lateral column alias `new_name` in generator expression "unresolvedalias(lateralAliasReference(new_name) LIKE a%)". SQLSTATE: 0A000;
'Project [explode(split(name#21, ,, -1)) AS new_name#19, unresolvedalias(lateralAliasReference(new_name) LIKE a%)]
+- SubqueryAlias spark_catalog.default.employee
   +- Relation spark_catalog.default.employee[dept#20,name#21,salary#22,bonus#23,properties#24] orc

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Unit test.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @cloud-fan

+1, LGTM. Merging to master.
Thank you, @wangyum.


Helpy 2

Version 2 includes a number of awesome improvements in the open source edition, and even more in the pro and cloud hosted versions:

Refreshed Admin UI
New Helpcenter theme: Singular
HTML support when responding to tickets
Nicer HTML alert emails
Nicer HTML responses to customers
HTML emails now include the full ticket history
UI for replying to tickets re-imagined
Inline customer editing
Channel and source displayed for each ticket
New support for emoji's in ticket replies
Customize the colors of the admin UI
Ability to email customers from the create ticket dialogue
New internal ticket type
Set all ticket params from admin create ticket UI
Font Awesome 5 iconography
Improved support for CC and BCC recipients
Import/Export data in CSV
Comply with GDPR by deleting or anonymizing users

closes #805
closes #697

@scott the main problem our QA manager provided is that new /admin/topics inbox doesn't show closed, and as soon as ticket gets assigned to someone, you cannot see it. A manger would want to have a big picture view and be able to see ticket counts and filter tickets by status regardless of the assignee.


Usage of __BEGIN_DECL/__END_DECL is not correctly parsed

Describe the bug
A simple header with __BEGIN_DECL/__END_DECL is not correctly parsed.
To Reproduce
Using latest commit 190e286a66d2d4ab8e9888e6c3bde8ee7d927233.
Just prepare this simple header and put it in one of test header, for example,
diff --git a/test/src/FreeFunctionClass.hh b/test/src/FreeFunctionClass.hh
index a0f8477..b0585b3 100644
--- a/test/src/FreeFunctionClass.hh
+++ b/test/src/FreeFunctionClass.hh
@@ -5,6 +5,13 @@
 #ifndef FSEAM_FREEFUNCTIONCLASS_HH
 #define FSEAM_FREEFUNCTIONCLASS_HH

+#include <sys/cdefs.h>
+
+__BEGIN_DECLS
+
+int test(void);
+
+__END_DECLS

 namespace source {

Then just build the project, the following error occurs:
cd /tmp/build/test && /usr/bin/c++  -DFSEAM_USE_CATCH2 -I/tmp/FSeam/test/src -I/tmp/build/test -I/tmp/FSeam/test/../FSeam -I/tmp/FSeam/FSeam  -std=gnu++17 -o CMakeFiles/testFSeamFreeFunction.dir/FreeFunctionClass.fseam.cc.o
-c /tmp/build/test/FreeFunctionClass.fseam.cc
In file included from /tmp/build/test/FreeFunctionClass.fseam.cc:9:
/tmp/build/test/FSeamMockData.hpp:61:5: error: expected member name or ';' after declaration specifiers
    __BEGIN_DECLS int test_ReturnValue;
    ^~~~~~~~~~~~~
/usr/include/x86_64-linux-gnu/sys/cdefs.h:109:31: note: expanded from macro '__BEGIN_DECLS'
# define __BEGIN_DECLS  extern "C" {
                        ~~~~~~ ^

Expected behavior
Maybe leave __BEGIN_DECLS/__END_DECLS along?

Unfortunately the parser used by FSeam is naive and just get information from a single header file, which means it cannot determine what is or not is a MACRO.
I see some way of fixing this :

Defining a list of MACRO used in the CMakeList.txt that will then be ignored in the parsing.
Use a more fined grained parsing (which would be the perfect answer but would require to re-write everything).

As a short term solution the list of MACRO to be defined is certainly a good idea as it may help for quick-fixes for other parsing issue.
But in the long term, the best would be to re-write the code generator in C++ and using libclang to parse headers and generate the appropriate source code with all the context (AST) required to not mistage a random character in the header file or an actual macro.


Share feature

Description
Fixes #1284
Added share icon in About page through menu inflater. On click of this icon, the google play store link of the app can be sent via any application through which text messages can be sent.
Tests performed
Tested on Motorola Moto G(5) (Android 7.0, API 24)

Codecov Report

Merging #1338 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #1338      +/-   ##
=========================================
- Coverage    3.73%   3.72%   -0.01%
=========================================
  Files         127     127
  Lines        6054    6065      +11
  Branches      588     588
=========================================
  Hits          226     226
- Misses       5813    5824      +11
  Partials       15      15

Impacted Files
Coverage Δ

...c/main/java/fr/free/nrw/commons/AboutActivity.java
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Sorry for the delayed review. The screenshot and code looks good to me, happy to merge pending a successful manual test.

Okay...no problem :-)
On Wed, Mar 28, 2018 at 4:27 PM, Josephine Lim<EMAIL_ADDRESS>wrote:

Sorry for the delayed review. The screenshot and code looks good to me,
happy to merge pending a successful manual test.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/commons-app/apps-android-commons/pull/1338#issuecomment-376846155,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfShUSWoN34CLeg3-vW6n_ZPRvRj_nS6ks5ti2yLgaJpZM4SwvU-
.

Thanks, @Mansi449 this works perfectly. :)


RNAudioRecorderPlayer.modulemap not found

Please fill the template to help you out. Also, please try the Example project compare before submiting the issue when you have certain issue with your project setup.

Version of react-native-audio-recorder-player
"react-native-audio-recorder-player": "^3.3.0"
Version of React Native
"react-native": "0.65.1"
Platforms you faced the error (IOS or Android or both?)
iOS
Expected behavior
Build succeeded
Actual behavior
Build failed
When I am trying to build the iOS project, the following error appears, and the build fails. Following is the error.
module map file '/Users/musabgulfam/Library/Developer/Xcode/DerivedData/myfriendsroom-dsfcxfbdkcvzjffdfrrspbydyvtt/Build/Products/Debug-iphonesimulator/RNAudioRecorderPlayer/RNAudioRecorderPlayer.modulemap' not found
I have tried to re-install pods, re-install node modules, cleaning build folder, also tried  arch -x86_64 pod cache clean --all, but to no avail.

Could you kindly try 3.5.0 and come back? Please use the recent version of the library.

Upgraded to version 3.5.0 but error still persists...

Upgraded to version 3.5.0 but error still persists...

Could you please make some reproducible example so that I can look into? Did you also try the example project in the repo?

same here

same here for M1

@AlexSmirnov9107 @albert2102  A reproducible example will be appreciated...
In my case it's a private project, owner of the project isn't allowing it to be public at the moment.

facing the same issue with XCode

i have same problem :/

@WilliamPeralta, a reproducible example will be appreciated...

I was getting the same error and finally realized there are a lot of wrong solutions on SO etc., so I share what solved it for me:
First, don't run Xcode with Rosetta, as mentioned here this is not recommended: https://developer.apple.com/documentation/technotes/tn3117-resolving-build-errors-for-apple-silicon
I excluded arm64 in my pod-file which was supposed to solve the problem together with running on rosetta. So I removed the EXCLUDED_ARCHS setting. I also removed the snippet in my Podfile which was doing exactly this. So if you have a snippet in your Podfile saying something withe EXCLUDED_ARCHS and arm64, remove that part and run pod-install again.
Lastly I got problems that Swift was not included, there are a lot or recommendations to fix this with a dummy file, and add the bridging header, finally the correct build setting was for me with the SDK-ROOT from this post:
https://stackoverflow.com/a/65602500/1505074
Hope it helps other people as well.

I had same issue while building the my app for release.
Opening .xcworkspace instead of .xcodeproject in Xcode and building the project fixed my issue.


Switch all of Angular Dart's tests to use code generation instead of reflection

Nobody runs their app with dart:mirrors in production, so why run the tests that way?

Just an update, I'll be tackling next week. Early work is there with a new NgTestBed pattern.

This is now planned for the 4.0 release.

This is currently in progress.

Fixed in various commits.


Requirements aren't being updated.

Details

Project URL: https://readthedocs.org/projects/tagnumelite-pyffi
Build URL: https://readthedocs.org/projects/tagnumelite-pyffi/builds/5956862/
Read the Docs username: TagnumElite

Expected Result
The Sphinx version would update.
Actual Result
When updating sphinx from the requirements-dev.txt file it says the requirements are already satisfied when trying to build the doc it says Sphinx is outdated.

The project has been removed.

I gave up and switched to github pages.


I messed up Xcode runner file for Flutter application

Is there away I can fix this without recreating a new project. It was asking for camera deployment iOS 10.0 at least. I had 8.0 but I pressed fix on the yellow yield then it changed the code which gave me all of this errors. I have tried to clean pod and upgrade flutter and did not work. I just could not redo what I have done mistake.
I have tried copy and pasta my back up Xcode and did not work.
Xcode's output: ↳
/Users/zuhairhallak/Desktop/flutter/.pub-cache/hosted/pub.dartlang.org/camer
a-0.4.3+2/ios/Classes/CameraPlugin.m:651:45: warning: unused variable
'discoverySession' [-Wunused-variable]
AVCaptureDeviceDiscoverySession * discoverySession =
[AVCaptureDeviceDiscoverySession
^
/Users/zuhairhallak/Desktop/flutter/.pub-cache/hosted/pub.dartlang.org/camer
a-0.4.3+2/ios/Classes/CameraPlugin.m:662:40: warning: unused variable
'devices' [-Wunused-variable]
NSArray<AVCaptureDevice *> * devices = discoverySession.devices;
^
/Users/zuhairhallak/Desktop/flutter/.pub-cache/hosted/pub.dartlang.org/camer
a-0.4.3+2/ios/Classes/CameraPlugin.m:667:51: error: use of undeclared
identifier 'devices'
[[NSMutableArray alloc] initWithCapacity: devices.count];
^
/Users/zuhairhallak/Desktop/flutter/.pub-cache/hosted/pub.dartlang.org/camer
a-0.4.3+2/ios/Classes/CameraPlugin.m:668:7: warning: collection expression
type 'AVCaptureDevice *' may not respond to
'countByEnumeratingWithState:objects:count:'
for (AVCaptureDevice *device in device) {
^                               ~~~~~~
/Users/zuhairhallak/Desktop/flutter/.pub-cache/hosted/pub.dartlang.org/camer
a-0.4.3+2/ios/Classes/CameraPlugin.m:693:19: warning: unused variable 'cam'
[-Wunused-variable]
FLTCam *cam = [[FLTCam alloc] initWithCameraName:cameraName
/Users/zuhairhallak/Desktop/flutter/.pub-cache/hosted/pub.dartlang.org/camer
a-0.4.3+2/ios/Classes/CameraPlugin.m:706:54: error: use of undeclared
identifier 'cam'
int64_t textureId = [_registry registerTexture:cam];
^
/Users/zuhairhallak/Desktop/flutter/.pub-cache/hosted/pub.dartlang.org/camer
a-0.4.3+2/ios/Classes/CameraPlugin.m:707:17: error: use of undeclared
identifier 'cam'
_camera = cam;
^
/Users/zuhairhallak/Desktop/flutter/.pub-cache/hosted/pub.dartlang.org/camer
a-0.4.3+2/ios/Classes/CameraPlugin.m:708:7: error: use of undeclared
identifier 'cam'
cam.onFrameAvailable = ^{
^
/Users/zuhairhallak/Desktop/flutter/.pub-cache/hosted/pub.dartlang.org/camer
a-0.4.3+2/ios/Classes/CameraPlugin.m:716:38: error: use of undeclared
identifier 'cam'
[eventChannel setStreamHandler:cam];
^
/Users/zuhairhallak/Desktop/flutter/.pub-cache/hosted/pub.dartlang.org/camer
a-0.4.3+2/ios/Classes/CameraPlugin.m:717:7: error: use of undeclared
identifier 'cam'
cam.eventChannel = eventChannel;
^
/Users/zuhairhallak/Desktop/flutter/.pub-cache/hosted/pub.dartlang.org/camer
a-0.4.3+2/ios/Classes/CameraPlugin.m:725:8: error: use of undeclared
identifier 'cam'
[cam start];
^
4 warnings and 7 errors generated.

Pods/Development Pods/ camera/  open311_s /ios/ .symlinks/camera/ iOS/ classes/ Camera.plugin.m

I missed cameraPlugin.m. Delete the code and go to flutter GitHub then plugin and find the class to copy the code and pasta it inside messed up code in the Xcode.


JAX support in Accelerate

Hi all,
I want to first thank you all for providing this great library.
I have a quick question. Is there any plan to add JAX support to Accelerate? It would be an awesome way to reduce boilerplate codes when working with JAX.

No, as Accelerate is strictly a PyTorch library


agent.load_model(path=path) NotFoundError

Hi,
I tried to use agent.save_model() and agent.load_model() without any examples but got errors:
NotFoundError (see above for traceback): Unsuccessful TensorSliceReader constructor: Failed to find any matching files for /home/llu/Data/Saved_TRPPO/
[[Node: save/RestoreV2_1 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_arg_save/Const_0_0, save/RestoreV2_1/tensor_names, save/RestoreV2_1/shape_and_slices)]]
[[Node: save/RestoreV2_31/_23 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/gpu:0", send_device="/job:localhost/replica:0/task:0/cpu:0", send_device_incarnation=1, tensor_name="edge_122_save/RestoreV2_31", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/gpu:0"]]
can anyone help me? Thanks

You need to supply the path to the checkpoint file .ckpt, not the directory

Can you give me some example? I have found one example from https://gist.github.com/Tjorriemorrie/40980a4f6c7a96f790a2184cc1714580
but agent.load_model() did not work.

I am using agent.store_model('/username/myfolder/mymodel.ckpt'), and agent.load_model('/username/myfolder/mymodel.ckpt') with full paths.
That  aside, this is a tensorflow mechanics issue, not a tensorforce bug, I would suggest consulting the tensorflow documentation to understand checkpointing; I will close this in the meantime.

I do not think that is tensorflow issue! I have tried what you suggested with a name ".ckpt". But there are four different files (NOT ONE with XXX.ckpt but FOUR) produced by tensorforce which is NOT the same way what tensorflow does.

It's definitely the case that TensorFlow (depending on the arguments to tf.train.Saver) creates more than one file. Can you post the lines where you save/load the model? One thing that I found on StackOverflow is the issue here. If you save it in the current directory, that could be the problem.

I had those:
path = "/home/llu/Data/Saved_NAF/naf_model.ckpt"
agent.save_model(path=path)
agent.load_model(path=path)
then errors.

What if you remove the suffix ".ckpt"? Otherwise we might have to investigate whether we're doing the restoring wrong, since we're not quite following what e.g. they do here, using this method.

Without .ckpt:
INFO:tensorflow:Restoring parameters from /home/llu/Data/Saved_NAF/naf_model
[2017-09-29 21:34:31,259] Restoring parameters from /home/llu/Data/Saved_NAF/naf_model
2017-09-29 21:34:31.300080: W tensorflow/core/framework/op_kernel.cc:1192] Not found: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for /home/llu/Data/Saved_NAF/naf_model
INFO:tensorflow:Restoring parameters from /home/llu/Data/Saved_NAF/naf_model
[2017-09-29 21:34:31,259] Restoring parameters from /home/llu/Data/Saved_NAF/naf_model
2017-09-29 21:34:31.300080: W tensorflow/core/framework/op_kernel.cc:1192] Not found: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for /home/llu/Data/Saved_NAF/naf_model

Yeah, okay, then we have to look into this. What you could do, if you want, is to change the load_model function to what they do in the links I posted and see whether it works.

Please post the content of the folder, all files. If there is a file of the form .ckpt-XXXX with some number, that is the step you last trained and you have to provide that as the load path

Thanks, guys. Now it works.


[SPARK-13089][ML] [Doc] spark.ml Naive Bayes user guide and examples

jira: https://issues.apache.org/jira/browse/SPARK-13089
Add section in ml-classification.md for NaiveBayes DataFrame-based API, plus example code (using include_example to clip code from examples/ folder files).

Test build #50531 has started for PR 11015 at commit be5586b.

Test build #50531 has finished for PR 11015 at commit be5586b.

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

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

Merged build finished. Test PASSed.

test this please

Test build #55389 has started for PR 11015 at commit be5586b.

Test build #55389 has finished for PR 11015 at commit be5586b.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #55471 has started for PR 11015 at commit 769595f.

Test build #55471 has finished for PR 11015 at commit 769595f.

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

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

Merged build finished. Test PASSed.

Thanks for the review. Updated according to the comments.

Test build #55596 has started for PR 11015 at commit 03d22b3.

Test build #55596 has finished for PR 11015 at commit 03d22b3.

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

Merged build finished. Test PASSed.

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

Just that 1 comment

Test build #55694 has started for PR 11015 at commit e27fe71.

Test build #55694 has finished for PR 11015 at commit e27fe71.

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

Merged build finished. Test PASSed.

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

LGTM
Merging with master
Thanks!

Thanks @jkbradley


feat: filing codes for CPRD

This PR also adds a devcontainer configuration and audits npm packages.

Resolves #181

Deployed as https://github.com/bcgov/orgbook-bc-client/releases/tag/v1.0.12


Exclude pull_request from fetch-tags step, fixes #7108

This PR excludes the drone fetch-tags step from the pull_request event as suggested in the comments of #7108. I don't believe this will have any unintended side effects in the CI system.
Signed-off-by: Nick Meyer<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@160e7ed). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #7120   +/-   ##
=========================================
  Coverage          ?   41.57%
=========================================
  Files             ?      446
  Lines             ?    60848
  Branches          ?        0
=========================================
  Hits              ?    25295
  Misses            ?    32270
  Partials          ?     3283

Continue to review full report at Codecov.

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


Test with official 3.13 and 3.14 alphas

This change updates the unit testing to use the official Python 3.13 released yesterday (Oct 7). It also starts testing against the alpha versions of Python 3.14 to catch potential problems early before it is officially released.

Seems GitHub hasn't released builds of 3.14 for windows and macos yet, just linux.
https://github.com/actions/python-versions/releases/tag/3.14.0-alpha.0-10616720958


Feature/add chunk to 8x (WIP)

WORK IN PROGRESS, DO NOT MERGE
I would be interested in any immediate feedback on my proposed changes. However, I have not tested this code beyond running the unit tests. Please refrain from merging until I can confirm that this actually works. 😝
Introduction
This pull request backports the method AbstractCsv::chunk (see https://github.com/thephpleague/csv/blob/master/src/AbstractCsv.php#L277) to allow output as a streaming response in a memory-efficient way for very large CSVs.
Proposal
Describe the new/upated/fixed feature
See above.
Backward Incompatible Changes
This change is strictly additive–no incompatibility should arise.
Targeted release version
I'm not sure how you manage releases. Please provide guidance.
PR Impact
Should add the method AbstractCsv::chunk to allow memory-efficient output of large CSVs.

@AC-TimRourke thanks for the PR but I have one major objection for this as stated on the documentation website:

Once a new major version is released, the previous stable release remains supported for six more months through patches and security fixes.

Since AbstractCsv::chunk is not a patch nor a security fix I can not merge your PR in the 8.x branch.
Having said that, your PR also as a bug. You are assuming that AbstractCsv::getIterator will always return an object which exposes the fread method, which is not true since 8.2 and the introduction of stream support. So What I would suggest is either fork 8.x (since you are guaranted that no new features will land on it) or use the proposed workaround.

👍  thanks for the heads up, I'll go ahead and close this. I'm hoping to support PHP7+ in the project I intend to use this in, so I'd prefer the 9.x branch anyway. Appreciate the feedback, and the warning about the bug!


Update the release annotation docs

I updated the release annotation docs to better describe the scenario when the release annotation will automatically be created. The current wording made it seem like the Application Insights resource needed to be in the same subscription as the Azure DevOps instance.

@ChristopherHaws : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ChristopherHaws : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks for taking the time to contribute to the docs =D
#sign-off


[Python API] Update device check

Update device check from .is to .equal.
Landing original bugfix from https://github.com/openxla/iree/commit/11e912b05c3f5889fff38e5569f919ef28f83f43

Original PR from @Eliasj42


chore: Introducing release profile for maps

The artifacts are available in Maven Central. Example: https://central.sonatype.com/artifact/com.google.maps/google-maps-mapsplatformdatasets/0.1.0

🥚 You hatched a release! The release build finished successfully! 💜

What going on in this PR? It's not a release PR.
Also, I would suggest changing the PR title because the profile is not just for maps, but for all non-cloud clients.

This PR was raised to trigger the release for non-cloud clients. The release went through, but we had a required test failing (which was unrelated to the releasing modules). I've ignored the test now and will merge it in as soon as the build is done.


Support to twig namespaces / bundle-name

In order to get a better compatibility between twig being used in front-end and in the back-end (with PHP) we've made a fork in order to add "namespace" compatibility.
Namespace
...
var template = Twig.twig({
    data: breadcrumbTpl,
    namespaces: { 'my-namespace': 'path/to/views/' }
}).render();
...

Done this, when referencing another template, instead of using the relative path, it is possible to use the namespaces that were previously defined. Ex:
{# random-template.twig #}
{% extends "kameleon-skin::default.twig" %}
...

The "my-namespace::" will now point to "path/to/views/".

@rafaellyra I don't think the coverage is tracked in twig.js.
I did a quick look for any tests in the PHP version covering the code that you implemented here and nothing jumped out at me. It's been a while since I reviewed this PR and may have missed it.
Either way, looks like it's time for a rebase! :-)

Any chance this any closer to getting merged in? Evaluating twigjs and I really need this feature.

@micahgodbolt we are using our fork since the PR is open and it's working very well :+1:

Rebase done, a new PR was created here -> https://github.com/justjohn/twig.js/pull/251


Add ImageTexture unit tests

Added ImageTexture unit tests.
I wasn't entirely sure on how to go about testing the draw methods and some feedback around that area would be appreciated. Thanks.
Link back to #43440

You can use the viewport texture for this and access the images pixels.
Please have a look at https://docs.godotengine.org/en/latest/classes/class_viewport.html#class-viewport-method-get-texture

You can use the viewport texture for this and access the images pixels.

Will this work in headless mode without a GPU?

Will this work in headless mode without a GPU? CI doesn't have access to any GPU.

I didn't think about this - I'm not yet familiar with the rendering server.
After looking in detail at the PR, I'm no longer sure, if my previously mentioned method can work, since no nodes are added to the scene tree.

Can confirm it if the style errors are fixed and the CI can run

Thanks!


Production build issue with migration file timestamp

Bug severity
3
Describe the bug
After compiling a production build (Ionic/Capacitor), then when the app attempts to createConnection to the database, the following error is returned. A development build works OK.
"migration name is wrong. Migration class name should have a JavaScript timestamp appended"
To Reproduce
Steps to reproduce the behaviour:

Build with --prod flag
Attempt to open the database

Observed behaviour
On createConnection:
g:  migration name is wrong. Migration class name should have a JavaScript timestamp appended.
at new g (http://localhost/main.9f7448e4642ebf0c79a3.js:1:3116881)
at http://localhost/main.9f7448e4642ebf0c79a3.js:1:3149533
at Array.map ()
at qe.getMigrations (http://localhost/main.9f7448e4642ebf0c79a3.js:1:3149430)
at qe. (http://localhost/main.9f7448e4642ebf0c79a3.js:1:3143521)
at http://localhost/main.9f7448e4642ebf0c79a3.js:1:4118445
at Object.next (http://localhost/main.9f7448e4642ebf0c79a3.js:1:4118552)
at z (http://localhost/main.9f7448e4642ebf0c79a3.js:1:4117269)
at P.invoke (http://localhost/polyfills.37dca26f56c14ea67b5d.js:1:36019)
at Object.onInvoke (http://localhost/main.9f7448e4642ebf0c79a3.js:1:155879)
Expected behaviour
The db connection should be established
Versions:

Veramo: 3.1.0
Browser Chrome/Brave
Node Version 14.16.1

Thank you for posting about this.
This is not a bug in veramo, but rather a requirement of typeorm which is used by @veramo/data-store to abstract the database connection.
As a workaround, you can try to configure your build system to preserve the migration file names.
This issue from their repo might help https://github.com/typeorm/typeorm/issues/2164
Please check back with results as many more could benefit from your findings

I agree that it's because uglifyjs is mangling the typeorm table names. This comment issuecomment-495185024 from the link above show how to setup a config file to turn off mangling.
I couldn't get it to work, so I turned off script optimization in the angular.json file for production builds, and that works.


Golang Twitter bot tutorial does not include instructions to import packages from go-twitter module.

I would have opened a PR for this, but I did not want to mess around with the way your excellent content is organized. In the Golang Twitter bot tutorial, it mentions that we are using packages from go-twitter, but does not tell the us to import those files. The "oauth-1" and "twitter" variables throw errors without this import.
This might be a small thing, but for someone who is still getting used to Golang, it would be helpful to state that and include those lines of code in the tutorial.
The go-twitter documentation does include an example of this import (https://godoc.org/github.com/dghubble/go-twitter/twitter).
import (
	"github.com/dghubble/go-twitter/twitter"
	"github.com/dghubble/oauth1"
)

config := oauth1.NewConfig("consumerKey", "consumerSecret")
token := oauth1.NewToken("accessToken", "accessSecret")
// http.Client will automatically authorize Requests
httpClient := config.Client(oauth1.NoContext, token)

// twitter client
client := twitter.NewClient(httpClient)

Absolutely, I'm sorry I missed this and caused you some issues! I've added in the imports to the article!

Thank you! Great tutorial!


Logging setting

What kind of request is this (question/bug/enhancement/feature request):
question / feature request
Steps to reproduce (least amount of steps as possible):
I should to send logging on Elasticsearch cluster. We use static alias index with ILM / Rolleover underground.
There are no way to set static index when configure Elasticsearch logging (from ui). So I edit setting as form, to set my static index. It's work fine, but with this way I can't set the buffer (flush internal). Default to 60s, it break the wall Elasticsearch cluster because of the bulk request is too big.
There are a way to edit setting as form and control the buffer ?
Result:
No way to controll payload size
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.4.8
Installation option (single install/HA): single

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Hosted
Machine type (cloud/VM/metal) and specifications (CPU/memory): VM
Kubernetes version (use kubectl version): 1.18.6

(paste the output here)

Docker version (use docker version): 19.3.8

(paste the output here)

Hi @disaster37, i'm also facing this issue. Could you show me how to add ILM alias to output. I tried ilm_rollover_alias but no luck. Thanks!


Please Add Keep Ratio function to Image Resizer

Description of the new feature / enhancement
Please Add Keep Ratio function to Image Resizer
Scenario when this would be used?
I want resize but keep it's ratio. Ex: I can input new width's value and height auto. Thanks
Supporting information
No response

Thank you.


Remote API version mismatch shouldn't be an HTTP 404

(Low priority issue)
https://github.com/docker/docker/blob/2a8a2d24285b40cf450f10dcb174dbef50e276ad/api/server/server.go#L1268
Currently we return HTTP 404 if client's remote API version is > server's remote API version. I think we can find a better status code to explain this (even BadRequest, PreconditionFailed, NotAcceptable are better than NotFound)

I agree.
It would be a one-line code change though (if we agree on something, I can open a PR for that :blush: )

I think it should return a 409 Conflict (http.StatusConflict) to be exact.
Also I think this should require a version bump (maybe) because client relying on 404 error won't be able to check this anymore (but again, I don't really know exactly)

Closing this for the sake of #15188.


Add classes for GET /distribution/{name}/json

This pull request add three new messages classes:

DIstribution: https://docs.docker.com/engine/api/v1.30/#tag/Distributionv

Descriptor: containing details on the distro

Platforms: a list of platforms and their details.

@davidxia I am currently working on the method frrom DockerClient

Codecov Report

Merging #969 into master will increase coverage by 0.14%.
The diff coverage is 94.11%.

@@             Coverage Diff              @@
##             master     #969      +/-   ##
============================================
+ Coverage     67.29%   67.43%   +0.14%
- Complexity      764      772       +8
============================================
  Files           173      176       +3
  Lines          3204     3218      +14
  Branches        366      366
============================================
+ Hits           2156     2170      +14
  Misses          892      892
  Partials        156      156


Make CLI use IDs consistently

          Let's always use `magistrala-cli users <user_id> update <type_of_update>` so we don't confuse the end-users. I.e. `mg-cli <entity_type> <entity_id_where_we_need_id> <operation>`.

Originally posted by @dborovcanin in https://github.com/absmach/magistrala/pull/2436#discussion_r1818860262

@felixgateru What's the status of this one?

@felixgateru What's the status of this one?

Currently working on updates to the sdk which will unblock this


HP LaserJet Professional P1102 no longer working

Hi,
I'm running an up-to-date Arch Linux with its latest official kernel, 4.18.5-arch1-1-ARCH.
Current CUPS version is 2.2.8-3.
My HP LaserJet Professional P1102 printer used to work with no problems at all with foo2zjs drivers (foo2zjs-nightly from the AUR) until about 1 month ago. Now when I try and print anything I get a new job stating "sending data to the printer", but nothing gets printed, although the activity led on the printer starts to blink as it does when it prints. Job status states "sending data to the printer" until I cancel the job; even after this, printer's activity led continues blinking until I turn the printer off.
The printer does work without problems with hplip and hplip-plugin 3.18.6 (which were not installed during the tests I report here).
I can't tell for sure which package upgrade caused it to stop working, anyway here are the relevant lines from /var/log/pacman.log (everything cups related and kernel related since about 1 month ago --- there's nothing foo2zjs related since many months, so foo2zjs should not be the problem here)...
[2018-07-23 07:30] [ALPM] upgraded cups-filters (1.20.3-2 -> 1.20.4-1)
[2018-08-12 08:39] [ALPM] upgraded cups-filters (1.20.4-1 -> 1.20.4-2)
[2018-08-24 16:12] [ALPM] upgraded cups-filters (1.20.4-2 -> 1.21.0-1)
[2018-08-27 00:30] [ALPM] upgraded cups-filters (1.21.0-1 -> 1.21.1-1)

[2018-07-23 07:30] [ALPM] upgraded linux (4.17.5-1 -> 4.17.8-1)
[2018-08-12 08:40] [ALPM] upgraded linux (4.17.8-1 -> 4.17.14.arch1-1)
[2018-08-18 19:09] [ALPM] upgraded linux (4.17.14.arch1-1 -> 4.18.1.arch1-1)
[2018-08-24 16:12] [ALPM] upgraded linux (4.18.1.arch1-1 -> 4.18.4.arch1-1)
[2018-08-27 00:30] [ALPM] upgraded linux (4.18.4.arch1-1 -> 4.18.5.arch1-1)

Here is what I get from system log when starting cups, turning the printer on, tryng and print the test page...
ago 27 05:39:52 gargantua systemd[1]: Starting CUPS Scheduler...
ago 27 05:39:52 gargantua colord[453]: failed to get session [pid 7588]: Nessun dato disponibile
ago 27 05:39:52 gargantua dbus-daemon[419]: [system] Activating via systemd: service name='org.freedesktop.Avahi' unit='dbus-org.freedesktop.Avahi.service' requested by ':1.365' (uid=0 pid=7588 comm="/usr/bin/cupsd -l ")
ago 27 05:39:52 gargantua dbus-daemon[419]: [system] Activation via systemd failed for unit 'dbus-org.freedesktop.Avahi.service': Unit dbus-org.freedesktop.Avahi.service not found.
ago 27 05:39:52 gargantua systemd[1]: Started CUPS Scheduler.
ago 27 05:40:27 gargantua kernel: usb 1-8: new high-speed USB device number 5 using xhci_hcd
ago 27 05:40:27 gargantua kernel: usb 1-8: New USB device found, idVendor=03f0, idProduct=002a, bcdDevice= 1.00
ago 27 05:40:27 gargantua kernel: usb 1-8: New USB device strings: Mfr=1, Product=2, SerialNumber=3
ago 27 05:40:27 gargantua kernel: usb 1-8: Product: HP LaserJet Professional P1102
ago 27 05:40:27 gargantua kernel: usb 1-8: Manufacturer: Hewlett-Packard
ago 27 05:40:27 gargantua kernel: usb 1-8: SerialNumber: 000000000Q87FWBPSI1c
ago 27 05:40:27 gargantua kernel: usb-storage 1-8:1.0: USB Mass Storage device detected
ago 27 05:40:27 gargantua kernel: scsi host6: usb-storage 1-8:1.0
ago 27 05:40:27 gargantua upowerd[730]: unhandled action 'bind' on /sys/devices/pci0000:00/0000:00:14.0/usb1/1-8/1-8:1.0
ago 27 05:40:27 gargantua upowerd[730]: unhandled action 'bind' on /sys/devices/pci0000:00/0000:00:14.0/usb1/1-8
ago 27 05:40:28 gargantua kernel: usb 1-8: USB disconnect, device number 5
ago 27 05:40:29 gargantua upowerd[730]: unhandled action 'unbind' on /sys/devices/pci0000:00/0000:00:14.0/usb1/1-8/1-8:1.0
ago 27 05:40:29 gargantua upowerd[730]: unhandled action 'unbind' on /sys/devices/pci0000:00/0000:00:14.0/usb1/1-8
ago 27 05:40:29 gargantua kernel: usb 1-8: new high-speed USB device number 6 using xhci_hcd
ago 27 05:40:29 gargantua kernel: usb 1-8: New USB device found, idVendor=03f0, idProduct=002a, bcdDevice= 1.00
ago 27 05:40:29 gargantua kernel: usb 1-8: New USB device strings: Mfr=1, Product=2, SerialNumber=3
ago 27 05:40:29 gargantua kernel: usb 1-8: Product: HP LaserJet Professional P1102
ago 27 05:40:29 gargantua kernel: usb 1-8: Manufacturer: Hewlett-Packard
ago 27 05:40:29 gargantua kernel: usb 1-8: SerialNumber: 000000000Q87FWBPPR1a
ago 27 05:40:29 gargantua kernel: usblp 1-8:1.0: usblp1: USB Bidirectional printer dev 6 if 0 alt 0 proto 2 vid 0x03F0 pid 0x002A
ago 27 05:40:29 gargantua upowerd[730]: unhandled action 'bind' on /sys/devices/pci0000:00/0000:00:14.0/usb1/1-8/1-8:1.0
ago 27 05:40:29 gargantua upowerd[730]: unhandled action 'bind' on /sys/devices/pci0000:00/0000:00:14.0/usb1/1-8
ago 27 05:40:56 gargantua kernel: usblp1: removed
ago 27 05:40:56 gargantua upowerd[730]: unhandled action 'unbind' on /sys/devices/pci0000:00/0000:00:14.0/usb1/1-8/1-8:1.0
ago 27 05:40:56 gargantua upowerd[730]: unhandled action 'bind' on /sys/devices/pci0000:00/0000:00:14.0/usb1/1-8/1-8:1.0
ago 27 05:40:56 gargantua foo2zjs-wrapper[7630]: foo2zjs-wrapper -z2 -P -L0 -r1200x600 -p9 -T3 -m1 -s7 -n1
ago 27 05:40:56 gargantua foo2zjs-wrapper[7642]: gs -sPAPERSIZE=a4 -g9920x7016 -r1200x600 -sDEVICE=pbmraw -dCOLORSCREEN -dMaxBitmap=500000000
ago 27 05:40:56 gargantua foo2zjs-wrapper[7643]: foo2zjs -r1200x600 -g9920x7016 -p9 -m1 -n1 -d1 -s7 -z2  -u 192x96 -l 192x96 -L 0  -T3    -P
ago 27 05:42:18 gargantua upowerd[730]: unhandled action 'unbind' on /sys/devices/pci0000:00/0000:00:14.0/usb1/1-8/1-8:1.0

Is it normal that as soon as a job is started I get that "usblp1: removed" line?
And here is what I get from /var/log/cups/error_log (debug level): https://pastebin.com/ew32Swby
Kind regards

On Sun 26 Aug 2018 at 22:41:11 -0700, pezcurrel wrote:

[2018-07-23 07:30] [ALPM] upgraded cups-filters (1.20.3-2 -> 1.20.4-1)
[2018-08-12 08:39] [ALPM] upgraded cups-filters (1.20.4-1 -> 1.20.4-2)
[2018-08-24 16:12] [ALPM] upgraded cups-filters (1.20.4-2 -> 1.21.0-1)
[2018-08-27 00:30] [ALPM] upgraded cups-filters (1.21.0-1 -> 1.21.1-1)

Possiblely Debian Bug#907026:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=907026
--
Brian.

Yes, all of those updates are not the core part of CUPS; cups-filters can be found here.


Fix text formatting in result display

The application is still showing output from v1.2, which may
confuse users.
To ensure the application is up to date with the changes to the
user guide, let's change the following to close AY2122S2-CS2103T-T17-3#122:

Update command usage texts
Remove deprecated parse exception from AddressBookParser and
LogicManager
Update UniqueTagList
Update StatusBarFooter
Update MainApp logger messages

Codecov Report

Merging #124 (c148bd6) into master (d529c10) will decrease coverage by 0.01%.
The diff coverage is 35.71%.

@@             Coverage Diff              @@
##             master     #124      +/-   ##
============================================
- Coverage     71.59%   71.57%   -0.02%
+ Complexity      653      652       -1
============================================
  Files           110      110
  Lines          2112     2104       -8
  Branches        219      218       -1
============================================
- Hits           1512     1506       -6
+ Misses          542      540       -2
  Partials         58       58

Impacted Files
Coverage Δ

src/main/java/seedu/address/MainApp.java
0.00% <0.00%> (ø)

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)

.../java/seedu/address/logic/commands/AddCommand.java
100.00% <ø> (ø)

.../seedu/address/logic/commands/BirthdayCommand.java
100.00% <ø> (ø)

...s/logic/commands/ContactedOutsideRangeCommand.java
100.00% <ø> (ø)

...ss/logic/commands/ContactedWithinRangeCommand.java
100.00% <ø> (ø)

...va/seedu/address/logic/commands/DeleteCommand.java
100.00% <ø> (ø)

...ess/logic/commands/DeleteContactedInfoCommand.java
95.23% <ø> (ø)

...seedu/address/logic/commands/DeleteTagCommand.java
94.87% <ø> (ø)

...java/seedu/address/logic/commands/EditCommand.java
96.00% <ø> (ø)

... and 10 more

Continue to review full report at Codecov.

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


Remove use of deprecated null_data_source

Moved logic from null_data_source into definition of locals.public_dns.
null_data_source is now deprecated (using tf 0.14.5) and throws warnings.  See PR below.
https://registry.terraform.io/providers/hashicorp/null/latest/docs/data-sources/data_source
https://github.com/cloudposse/terraform-aws-ec2-instance/issues/93

/rebuild-readme

/test all

@bwmetcalf our CI got stuck which happens sometimes.   Thanks.

/test all

@Gowiem I'm not sure of the solution to the Bridgecrew analysis failure. It's unrelated to this change and seems to be invalid regardless.

@woz5999 agreed. Let's get @maximmi or @Nuru thoughts on the matter. They're more involved in the BridgeCrew work than I am.
Gents, do we need to hold off merging this PR? It seems the BridgeCrew CI is failing on this PR due to something that is already merged into master. How are we handling that elsewhere?

hi team, I hit a similar issue on https://github.com/cloudposse/terraform-aws-alb/pull/86, drew the short straw as a PR that went in after enhancements to code analysis.  it's definitely new changes to the analytics of bridgecrew as I was able to pull an old container and never saw the results produced in the new containers.  Seems the consensus is skips if they make sense to skip ...

/test all

@Gowiem I think this is ready to go

Thanks for the contribution @bwmetcalf (and help @jwstric2 + @woz5999)! Released as 0.31.0.


Use carto to parse cartocss

Now we are using a custom style parser, but the approach is so similar to the Carto library approach that it could be better adapting that library for this project.

#5 fix this issue


[Bug] Datatables Wordwrap on Bootstrap 5

Version
1.03
Reproduction link
https://demos.creative-tim.com/soft-ui-dashboard-pro/pages/applications/datatables.html
Operating System
Mac OS
Device
Macbook
Browser & Version
Safari 14.1.2
Steps to reproduce
The "word wrap" function for a column is not working.  I used your example here: https://demos.creative-tim.com/soft-ui-dashboard-pro/pages/applications/datatables.html and added a long string of text to the first row of the final column.  I've used the "wrap" commands, I've tried to style the table column and it doesn't wrap the string.
What is expected?
The long string should wrap inside the card without adding a horizontal scroll to the card.
What is actually happening?
The string in the column is displaying as a single line and stretching the table horizontally as a single line.  It stretches the card and will horizontally scroll, but I would like to keep the entire table inside the fixed card.

Solution
I do not have this issue using a basic downloaded file of the Bootstrap 5 data tables directly from their website, but when I try to use it on your pages, it does not wrap.  Is there some kind of CSS conflicting with it?
Thank you for your assistance!
Additional comments

Hi @sstvdj,
Thank you for working with our products.
Can you please give us the code that you used so we can test it and find a solution?
Best regards,
Rares

datatables2.html.zip
It happens with your example datatables.html page in pages/applications/datatables.html.  I zipped up the file and attached it with a string in the last column to demonstrate the issue.  I've also attached a screenshot of what happens.

Hello, do you have any updates on this issue?
Thank you!

Hi @sstvdj,
Sorry for my late response.
To remove the horizontal scroll please try removing  white-space: nowrap; option from  .table td, .table th in the css file:
.table td, .table th {
    white-space: nowrap;
}

Please let me know if it works.
Best regards,
Rares

Hi, that was close.  I changed it to white-space: normal; and it is now wrapping correctly for my needs.  Appreciate the help!


Composer

composer.lock added to the repository to force consistent package versions at installation

@hellowluan Well I do accept to gitignore composer.phar file but excluding composer.lock is debatable. Can you put some light onto why we should remove composer.lock from .gitignore? 🙂

@hellowluan Well I do accept to gitignore composer.phar file but excluding composer.lock is debatable. Can you put some light onto why we should remove composer.lock from .gitignore? 🙂

Basically having composer.lock ensures that each developer is using the same versions.
Should composer.lock be committed to version control?

Basically having composer.lock ensures that each developer is using the same versions.
Should composer.lock be committed to version control?

Cool. I Think this article by Davey Shafik explains it much better. 👍
https://blog.engineyard.com/composer-its-all-about-the-lock-file

Cool. I Think this article by Davey Shafik explains it much better. 👍
https://blog.engineyard.com/composer-its-all-about-the-lock-file


throw a error if input have not default color value

colors.js   in line: 221
colors[typ] = convert[from + '2' + typ](colors[from]);

Please give me more information. This is not replicable for me.
If I have a <input class="color" /> instead of <input class="color" value="#B6BD79" /> it falls back to #FFFFFF but it doesn't throw an error.


#7324 Ensure Open Types tests are run (gradle)

Description
Ensures open types FVT tests run as part of gradle build

Related Issue(s)
See #7374

Testing

Open types fvt test source files created ok
Test launches ok
Cleanup code removes files afterwards

Note, this current fails as the actual tests have issues:

The changes to the scripts appear good & ready for merge when we have fixed the tests.

Release Notes & Documentation

Additional notes

Once #7325 is merged, I will merge or rebase, as this PR currently also contains that fix - needed to validate
FYI @davidradl

I notice codeQL is failing with lots of 'notes' (minor level issues)  due to the generated code.
These can be seen at https://github.com/odpi/egeria/security/code-scanning?query=pr%3A7327+tool%3ACodeQL+is%3Aopen+sort%3Acreated-desc
It might be useful to fix the generator to build cleaner code (I suspect it's a small number of issues in the template)
Directories can't be excluded from the codeQL scan itself, the only control is by not-building certain modules.
We might be able to add properties, but this will change how users need to exclude tests (-x tests) -- the issue here is that the FVT generated code (which is a compile task, not test) will still run. A fix here will need some further refactoring ...
So probably easiest to take a look at the warnings.

^ fyi @davidradl

@davidradl Do you want to do anything with the code template?

I'm trying to get these build issues integrated before some refactoring needed to fix connectors.
I think I can remove the open types test from the build - for the purposes of codeQL only.
This should fix the large number of observations.
If the build is good I will merge

codeQL build still failing (the build itself is clean)

rules evaluation just seems to stop.
https://github.com/github/codeql/issues/7294 seems to confirm that @suppresswarnings is not respected by codeQL. However that issue does also point to a way of filtering the sarif file before analysis.
This may be what is needed to fix codeQL. However at this point I think it's clear this is a codeql specific issue. It's fragile.
I therefore plan on merging this anyway & raising a new issue to track codeQL issues


More vertical space above/below checkbox list

Add vertical space (+ 5-10 pixels) above and below checkbox lists. Otherwise it is harder to distinguish between checkboxes of different adjoining lists:

@rolandschaub All form fields does have the same padding/spacing on bottom/top. I don't think we can change it only for checkboxes, otherwhise we heave to increase the margin top/bottom globally. What do you think @TheMaaarc?

Each form type has a custom class, so i can change the margin only on checkbox lists.

ok, but i don't think its a good practice to change the margin only for a custom field.

I don't think it is bad if it improves usability.

@rolandschaub we are not going to change the margin for a specific field.

Okay.


Smali filenames generated that easily exceed MAX_PATH

Information

Apktool Version (apktool -version) - 2.2.4
Operating System (Mac, Linux, Windows) - Windows 10 Enterprise (Build 15063)
APK From? (Playstore, ROM, Other) - Amlogic SDK 7.1

Stacktrace/Logcat
See dir output of files created

Steps to Reproduce

apktool d services.jar

Frameworks
If this APK is from an OEM ROM (Samsung, HTC, LG). Please attach framework files
(.apks that live in /system/framework or /system/priv-app) - Done
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything? Yes
If you are trying to install a modified apk, did you resign it? N/A
Are you using the latest apktool version? Yes

Looking through the smali files created, some of these are extremely long and include code from the beginning of the contents.
For example,

ConnectivityService$-void__init__android_content_Context_context_android_os_INetworkManagementService_netManager_android_net#_statsService_android_net_INetworkPolicyManager_policyManager_android_net_metrics_IpConnectivityLog_logger_LambdaImpl0.smali
AccountManagerService$-boolean_removeAccountInternal_com_android_server_accounts_AccountManagerService$UserAccounts_accounts_android_accounts_Account_account_int_callingUid_LambdaImpl0.smali
SyncManager$-void_scheduleSync_android_accounts_Account_requestedAccount_int_userId_int_reason_java_lang_String_requestedAuthority_android_os_Bundle_extras_int_targetSyncState_long_minDelayMillis_LambdaImpl0.smali
DreamController$DreamRecord$-void__init__com_android_server_dreams_DreamController_this$0_android_os_Binder_token_android_con#nt_ComponentName_name_boolean_isTest_boolean_canDoze_int_userId_android_os_PowerManager$WakeLock_wakeLock_LambdaImpl0.smali
ShortcutService$LocalService$-java_util_List_getShortcuts_int_launcherUserId_java_lang_String_callingPackage_long_changedSinc#kageName_java_util_List_shortcutIds_android_content_ComponentName_componentName_int_queryFlags_int_userId_LambdaImpl0.smali

As you can see, these suckers are huge and quickly eat up the 260 characters (MAX_PATH) available in Windows to address a file.  You can't open them in Notepad or Visual Studio.  Basically, all you can do is move them to the root or delete them.
files.txt
services.zip

Hmm. Thought smali had something internally to handle this. Let me look into this some more.

Yeah, smali still has support for this https://github.com/JesusFreke/smali/blob/master/util/src/main/java/org/jf/util/ClassFileNameHandler.java
Per the comment.
/**
 * This class handles the complexities of translating a class name into a file name. i.e. dealing with case insensitive
 * file systems, windows reserved filenames, class names with extremely long package/class elements, etc.
 *
 * The types of transformations this class does include:
 * - append a '#123' style numeric suffix if 2 physical representations collide
 * - replace some number of characters in the middle with a '#' character name if an individual path element is too long
 * - append a '#' if an individual path element would otherwise be considered a reserved filename
 */

I don't have access to a Windows machine right now, so obviously can't test at the moment, but code says this shouldn't be a problem.

C:\Users\Connor\Desktop\bin>java -jar apktool.jar d services.jar
I: Using Apktool 2.2.4 on services.jar
I: Baksmaling classes.dex...
I: Copying assets and libs...
I: Copying unknown files...
I: Copying original files...
C:\Users\Connor\Desktop\bin>java -jar apktool.jar b services.jar.out
I: Using Apktool 2.2.4
I: Checking whether sources has changed...
I: Smaling smali folder into classes.dex...
W: Could not find resources
I: Building apk file...
I: Copying unknown files/dir...
C:\Users\Connor\Desktop\bin>

I also edited these long files in Notepad without a problem. My directory tree wasn't anything special. It went Users\Connor\Desktop\bin. Additionally the files you provided are at 249 characters so smali's workaround did not occur. I don't see a problem here as the limit code is introduced at 250 characters. These files stop at 249.

So, you're saying it's by design to include all this junk in the smali filename?  And the "fix" is to just move them closer to the root?
Note: The problem is not as much with the ClassFileNameHandler, but with the synthetic class name generation.  If you look at the contents of these files (which may require moving them file closer to the root), they all start with ".class final synthetic".

The smali filename is by generation the class name of the file, except when the filename conflicts with the rules in place by ClassFileNameHandler.
.class final synthetic Lcom/android/server/ConnectivityService$-void__init__android_content_Context_context_android_os_INetworkManagementService_netManager_android_net_INetworkStatsService_statsService_android_net_INetworkPolicyManager_policyManager_android_net_metrics_IpConnectivityLog_logger_LambdaImpl0;
.super Ljava/lang/Object;
.source "ConnectivityService.java"

That is the header of a file. The class is that massive name therefore the filename is. If the naming is invalid that would be something I would bring up with the smali project - https://github.com/JesusFreke/smali/issues as Apktool leverages smali as a dependency to handle this.

Also, while Java seems to handle the long paths and the apktool build does succeed, I can't open these files in the file explorer by double-clicking them, even when just on my desktop (C:\Users\Chad\Desktop).  For that, I've got to move them closer to the root.
More importantly, I can't edit them in Visual Studio 2017 or search their contents using Find in Files.

Okay, I opened a related issue there:  https://github.com/JesusFreke/smali/issues/541

workarounds. for Windows users. if you are interested...
short the apk as much as possible and place the apk as low as possible (D:\APKs\).
switch to oracle java 17, grab the portable version from archive, modify yourself (patch) the exe and dll files everywhere so their manifest would include long path awareness.
https://github.com/eladkarako/manifest/blob/master/example_manifests/generic.manifest#L13
there are some batch files for easy drag and drop here https://github.com/eladkarako/manifest
Win10 needs a reg-patch to opt-in this feature, Win11 has it turned on already.
I'm not explaining 1-to-1 what to do. but as long you are working on a separate folder and afterwards changing the JAVA_HOME, (etc..) environment variables, you'll be fine, maximum length will be about 32k characters. and on some cases pretty much unlimited (and MAX_PATH will be ignored by all the file API).
https://docs.microsoft.com/en-us/windows/win32/fileio/maximum-file-path-limitation?tabs=cmd#enable-long-paths-in-windows-10-version-1607-and-later
https://knowledge.autodesk.com/support/autocad/learn-explore/caas/sfdcarticles/sfdcarticles/The-Windows-10-default-path-length-limitation-MAX-PATH-is-256-characters.html


Web Push doesn't open app, if the app is in background

Web Push click does open the app if the app is fully closed, but if it is in background click does nothing.
Service worker code:
self.addEventListener('notificationclick', function(event){
        event.notification.close();
        event.waitUntil(clients.matchAll({
           type: "window"
        }).then(function(clientList) {

            var url=event.notification.data.url || '/';

            for (var i = 0; i < clientList.length; i++) {
                if ('focus' in client){
                    return client.focus();
                }
            }
            if (clients.openWindow) {
                return clients.openWindow(url);
            }
        }));
    });

Looks like client.focus and clients.openWindow doesn't work, if the app is in background

I have the same problem as vlasovilya and my service worker code looks the same as his. Everything is working fine when the app runs as PWA but not as TWA.

Can you provide more information on the browser version? Also, can you take a look on #15 and point if that is a similar issue?

I have tested this on Android Studio Emulator (Chrome 74 and 75 on android 7.0 and 8.0), on Pixel XL (Android 8.0, Chrome 74 and 75) and on Huawei (Android 8.0, Chrome 74). On the Pixel devices the clients.openWindow(url) is ignored when the app is in background (not waking up the app) and on Huawai the app just closes/crashes when in background/foreground (it only opens the app when it is completely closed). When i run the page as PWA the clients.openWindow(url) brings the app to the foreground as expected.

re: the issue with Pixel and Chrome 74 / 75, could you please test on Chrome Canary? An issue with bringing was fixed and is available there.

Thank you for the information andreban. I have now tested my TWA with the canary Chrome version and it is working as intended.

Hello @vlasovilya and @LukasSarnowskiC9,
I've faced the same issue but it still doesn't work for me.
self.addEventListener('notificationclick', function(event) {
        function sendMsg(client, url, notification) {
            client.postMessage(self.JSON.stringify({ url, notification }));
        }

        function openUrl(openingUrl) {
            return self.clients.openWindow(openingUrl);
        }
        event.notification.close();

        const data = event.notification.data;
        const url = data.url;
        const notification = data.notification;

        event.waitUntil(
            self.clients
                .matchAll({ type: 'window', includeUncontrolled: true })
                .then(function(clientList) {
                    const nonNested = clientList.filter(client => client.frameType !== 'nested');

                    // if there are no non-framed clients, just open the new window with the specified url
                    if (nonNested.length === 0) {
                        return openUrl(url);
                    } else {
                        for (let i = 0; i < clientList.length; i++) {
                            const firstClient = nonNested[i];

                            if ('focus' in firstClient) {
                                return firstClient
                                    .focus()
                                    .then(sendMsg.bind(null, firstClient, url, notification))
                                    .catch(openUrl.bind(null, url));
                            }
                        }
                    }
                })
        );
    });

Could you please share your piece of code which is working on Canary?

@LukasSarnowskiC9 Can you please tell me how did you test your TWA in chrome canary? Because TWA always opens in default chrome browser. How did you specify to open in canary instead of chrome? Or did you mean PWA, not TWA?


[HUDI-3781] fix spark delete sql can not delete record

What is the purpose of the pull request
fix can not delete record by use  spark sql. operation.key must be "delete"
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@hudi-bot run azure

fix conflicts

other command has the same question, I open  a issue to discuss #5248

hi @KnightChess Retrigger the CI of the github action.

@KnightChess   @XuQianJin-Stars  how about forbidden set hoodie.datasource.write.operation  when we create hudi table.
maybe we can do some check when we create table， or ignore this paramter and print some log for it.

@XuQianJin-Stars how to retrigger it?

@KnightChess @XuQianJin-Stars how about forbidden set hoodie.datasource.write.operation when we create hudi table. maybe we can do some check when we create table， or ignore this paramter and print some log for it.

@xiarixiaoyao in #5248 , I think check is better

@XuQianJin-Stars how to retrigger it?

reopen this pr or commit pr comments.

@KnightChess  thanks for your contribution.


Unix Makefile: Make sure to use $(PERL) when running ./Configure

Checklist

[x] CLA is signed

Description of change

For consistency, it's better to use the perl that was specified to
Configure last time it was called.
Use case:
perl v5.8.8 was first along $PATH, perl v5.22.2 was available and
specified as: PERL=/opt/local/bin/perl ./config.  When make wanted to
reconfigure and called ./Configure reconf, configuration broke down,
complaining about a perl that's too old.

I've labeled this PR now. Master and 1.1.0

Merged in both.  Thank you


Add support for Annotations

Please add support for Annotations, perhaps initially with the supported Protege types for annotation properties. (This is a cut-down version of the property implementation)

Hi Chris,  this sounds like an interesting idea!
This could easily be accommodated from UML as TSV to OWL by adding additional columns to the TSV.
Have you given any thought to how to support annotations when translating from XMI?

Hi Henriette, initially, I was looking at an EMF Ecore model and profile attributes such as EClass and EReference and thinking about how I could make the origin of the things in OWL visible.


Add playsinline to youtube and possibly vimeo

It might be impossible with vimeo.  A workaround could be using Vimeo url in video player.
Possible with Youtube:  https://developers.google.com/youtube/player_parameters
Expected behaviour
Video Plays inline
Actual behaviour
Video opens in video player popover
Environment

Browser:  Chrome, Safari
Version:
Operating System: iOS
Version: 10.0.2

Players affected:

[ ] YouTube
[ ] Vimeo

Related: https://github.com/sampotts/plyr/issues/485

Closing this one as I think it's fixed but feel free to re-open if not solved in v3 👍


CI/CD: fix offline_sev_kbc CI run failed: 'rustfmt' is not installed

I submitted PR #34 almost at the same time :-)


tests: ensure the virt-handler killer pod has gone after deletion

Make sure the killer-pod has been deleted in order not to affect other tests.
Signed-off-by: Igor Bezukh<EMAIL_ADDRESS>What this PR does / why we need it:
In time frame between a deletion TS was added to
the killer pod and until it's actually has been deleted
we may have an inconsistent bahavior where the killer pod
keeps killing the virt-handler in the upcoming tests.
Release note:
None

/cc @stu-gott

cherry-pick release-0.34

/cherry-pick release-0.34

/approve

/retest


CRA5 source map warning

Hi team.
I'm getting this error with create-react-app 5.0 for all hooks.
WARNING in ./node_modules/usehooks-ts/esm/useWindowSize/useWindowSize.js
Module Warning (from ./node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from '$REPO_ROOT/node_modules/usehooks-ts/src/useWindowSize/useWindowSize.ts' file: Error: ENOENT: no such file or directory, open '$REPO_ROOT/node_modules/usehooks-ts/src/useWindowSize/useWindowSize.ts'

And it has been complained a lot here:
https://github.com/facebook/create-react-app/discussions/11767
Should we add some source map support?

Hi @PabloLION, interesting, thanks for the report, we will check it asap.

This is fixed, available on<EMAIL_ADDRESS>🎉
Thanks, @PabloLION for the bug report! 👍

This happens again :(

Maybe we should @juliencrn

Source-map have been removed in<EMAIL_ADDRESS>(https://github.com/juliencrn/usehooks-ts/releases/tag/usehooks-ts%402.7.1). Thanks for reminding :)


Default value for Cache::increment() varies accross drivers

The following code have different result depending on the cache driver.
Cache::increment('x');
echo Cache::get('x');

Output:

Redis: 1
Dababase: null
Memcached: null
APCu: 1
File: 1

I guess the best is for all to have the same results.

fell free to open a PR :)
Closing since it's not a bug.

It violates Liskov substitution principle.

@themsaid I just noticed this while looking at something else in the cache. I was going to work on a PR, but then realized I didn't know which way the framework would want to go with it.
It would be great to normalize the behavior so that all cache drivers act the same. So, what would the expected behavior be? Should the value be added to the cache, or should the increment request be ignored and return null?
The majority of the drivers (including the array driver) add the value to the cache, so that would be my assumption, but I think it would be good to nail down what the framework owners actually want. :)


Attempted import error: 'Annotation' is not exported from '@codemirror/state'.

Bug report
Packages affected

[ ] sandpack-react

Description of the problem
I have an existing react project where it uses react version "react": "^17.0.2". I am trying to use the sandpack-react in my react project but getting the following error while running the application.
<EMAIL_ADDRESS>Attempted import error: 'Annotation' is not exported from '@codemirror/state'.

sandpack-react package version I am using is "@codesandbox/sandpack-react": "^2.6.7"
Note: The same package is working with the new project.
What were you doing when the problem occurred?
What steps can we take to reproduce the problem?

Do an npm install @codesandbox/sandpack-react on the existing project which uses the older version of react.
Then just import the import {  SandpackProvider,  SandpackLayout,  SandpackCodeEditor,  SandpackFileExplorer,  SandpackPreview, } from "@codesandbox/sandpack-react"; this line.
Then run your app and it's start showing the above error.

Link to sandbox: link (optional)
Your Environment

Software
Name/Version

Sandpack-client version

Sandpack-react version
^2.6.7

Browser
Chromium

Operating System
Ubuntu 22

Does your issue got resolved? I am getting same error for react v17.0

I'm closing this issue as it's been a while without any activity. Please feel free to report it again if the problem persists. Thanks so much!


Grad2-2434 Archived students user name corrected

Fixed code for list of students

You have failed unit tests. Please fix