$Host.PrivateData colors are completely broken in PowerShell 7.2

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
$PSStyle has completely broken the PrivateData colors. Setting them no longer changes anything.

Expected behavior
The colors on `$Host.PrivateData` should affect the output

Actual behavior
Changes to `$Host.PrivateData` don't do anything anymore

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.2.0
PSEdition                      Core
GitCommitId                    7.2.0
OS                             Microsoft Windows 10.0.22000
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
This is what I expected:

/cc @SteveL-MSFT

$Host.PrivateData works if there is no virtual terminal or it is disabled.

Setting them no longer changes anything.

If I change $PSStyle.Progress.View to Classic then the progress colors are from $Host.PrivateData.Progress*

Hm.  Error output obeys $Host.PrivateData.ErrorForegroundColor but not $Host.PrivateData.ErrorBackgroundColor for me, and only when $ErrorView = ConciseView.  For any other value of $ErrorView, no coloration is applied.
Meanwhile, the Get-Error cmdlet applies $PSStyle.Formatting.Error coloration to the value of the returned object's Exception.Message property, as well as to the position-marking tildes of InvocationInfo.PositionMessage

So WITH NO NOTICE a bunch of code was broken that customized error and verbose output colors, and there is now no way to customize them in a compatible fashion, except to feature-sniff for types/versions and do it both ways.
I raised this MONTHS AGO as a REGRESSION BUG and I feel like it was completely brushed off and basically ignored.
What is going on with prioritization of development over there?!?

@dwtaber Error for classic view was a bug that was fixed via https://github.com/PowerShell/PowerShell/pull/17705
@Jaykul the WGs are working through issues, but haven't gotten to this one.  I would propose to have a new $PSStyle.Formatting.UsePrivateDataColors set to $true by default which would have the old stream output use the old settings.  We could automatically flip this to $false if any of the new formatting colors are set.

Maybe the more helpful thing would be to continue using PSStyle, but update PSStyle when the PrivateData values are set.
That is, if something does: $Host.PrivateData.ErrorForeground = "Red" then update $PSStyle.Formatting.Error += "e[31;1m"` (optionally, you could remove existing foreground colors, but it doesn't really matter too much)

@Jaykul that's is also an option.  The only problem is if then the user updates $PSStyle, it would not be possible to reflect that back into PrivateData unless they happen to use one of the 8 colors only.

Actually, there's a problem with your proposal.  $Host.PrivateData is managed by the host (ConsoleHost, EditorServices, etc...), so every host would have to adopt that pattern.  PrivateData is just a PSObject so a host can have whatever they want in there.  The colors are just a de facto convention.

I think going one way would probably be good enough. I mean, I have code for converting from 16bit colors to 16 colors.

Actually, there's a problem with your proposal. $Host.PrivateData is managed by the host (ConsoleHost, EditorServices, etc...), so every host would have to adopt that pattern. PrivateData is just a PSObject so a host can have whatever they want in there. The colors are just a de facto convention.

So what you're saying is that the HOST is still trying to set the colors, but the engine is injecting escape sequences into the text that override the host's settings?

Write-Error, Write-Verbose, etc... are host callbacks.  It's up to the host to decide how to display them.  Here's the relevant section for ConsoleHost: https://github.com/PowerShell/PowerShell/blob/edac193cf05a540b1f5f59f23fd1eb29c2776ce1/src/Microsoft.PowerShell.ConsoleHost/host/msh/ConsoleHostUserInterface.cs#L1395-L1402.  Currently, if VT is supported, it goes the VT route and by-passes the older code path that relies on [Console] APIs that accept [ConsoleColor] and overrides the color codes in the string (the string value still contains them).
So in the end, it's up to the host to decide what to do.  This was an unfortunate design choice early on.

So, bottom line: you added a non-host way of injecting virtual terminal escape sequences, and then changed the default host so that if it thinks it's running in a terminal that supports escape sequences ... it ignores it's own settings.
Is that right? Is backward compatibility just out the window forever?

I've proposed a solution due to the limitations of the existing design.  You had a different proposal which seemed good at first until I looked into it and it's not feasible.  If you have an alternate proposal, let's discuss it.

I'm frustrated, but I don't know how I didn't understand that you'd deliberately deprecated PrivateData. Honestly, I don't think there's any point in putting a switch in. It would basically be the same as setting $ENV:TERM = "dumb" right?
Writing a warning that they're deprecated when someone sets those properties (in a host that SupportsVirtualTerminal?) would be nice, so people can be reminded those things are now useless ... but unless we can do that, I think we might as well just close this "as designed". 😣

I've just started reading about this and I can't believe in 7 years nothing has been done.
The time spent arguing about this since @Jaykul first asked about it on Sep 29, 2016, could have been used to implement his rather straight forward request.
There does need to be any breaking changes, just override the Write-Host to allow for HEX values for ForegroundColor.
The terminal window is not a graphic design tool, the RGB space is perfectly sufficient and slight display variations are the least of a coder's problems.
We just want a way of distinguishing one thing from another in the terminal - this doesn't have to made into rocket science.
Interlectual wars aside, this is would benefit everyone - Support for RGB console colors

The Interactive WG has reviewed this long standing issue.  We don't think that PrivateData can be linked well to PSStyle to remove a poor affect from PrivateData. We think the current behavior of PSStyle was the intended design change, however we want to review this committee before closing this issue.

Thank you to all that commented on this issue.  The PowerShell committee has reviewed and concurs with the Interactive WG that this feature is by designed.  We will investigate in the future to emit a depreciation warning if any private data is changed.


🐛 add execution permission to ./scripts/ci-make.sh

What this PR does / why we need it:
Currently CI fails with
+ ./scripts/ci-make.sh /usr/local/bin/runner.sh: line 114: ./scripts/ci-make.sh: Permission denied
+ EXIT_VALUE=126

This PR adds mission permission to the script.

/retest

/cherry-pick release-0.3

We need this for release-0.3 as well https://github.com/kubernetes-sigs/cluster-api-operator/pull/147


Music stops when wake word is triggered

Loving the skill so far! But on annoying thing I've noticed is that the music stops when you say the wake work 'Hey Mycroft'.
Appreciate your work man 👍

Glad you like it!
The idea is that the music pauses for 5 seconds and then resumes when the wake word is heard so Mycroft can hear the request properly.
Does the music stop and then not resume?
What platform are you on?

It does continue the music again after 5 seconds but my problem is MyCroft often thinks I’m saying the wake up word when I’m not so it interrupts Spotify :/
I’m currently running it on a Raspberry Pi 3 but just about to try MyCroft + Plasmoid

I understand. In that case an option in the settings might be the way to go. Making it possible to disable the auto-pause

Looks like the audio ducking feature has been made optional. Can this issue be closed now?

Indeed. Closing.


java-sample build failed due to network io timeout, can we do offline build?

I am following the java-sample steps.
~ kubectl apply -f java-function-sample.yml

~ cat java-function-sample.yml
apiVersion: core.openfunction.io/v1beta1
kind: Function
metadata:
  name: java-sample
spec:
  version: "v1.0.0"
  image: "harbor.local/sample-java-func:latest"
  imageCredentials:
    name: harbor-secret
  port: 80 # default to 8080
  build:
    builder: "openfunction/gcp-builder:v1"
    env:
      GOOGLE_FUNCTION_TARGET: "com.openfunction.HelloWorld"
      GOOGLE_FUNCTION_SIGNATURE_TYPE: "http"
    srcRepo:
      url: "https://github.com/OpenFunction/samples.git"
      sourceSubPath: "functions/knative/hello-world-java"
      revision: "main"
  serving:
    runtime: knative # default to knative
    template:
      containers:
        - name: function
          imagePullPolicy: Always

The builder failed when step-create, the error log is following:
 ===> DETECTING
 4 of 5 buildpacks participating
 google.java.runtime             0.9.1
 google.java.maven               0.9.0
 google.java.functions-framework 1.0.1
 google.utils.label              0.0.1
 ===> ANALYZING
 Previous image with name "harbor.local/sample-java-func:latest" not found
 ===> RESTORING
 ===> BUILDING
 === Java - Runtime<EMAIL_ADDRESS>===
 Using latest Java 11 runtime version. You can specify a different version with GOOGLE_RUNTIME_VERSION: https://github.com/GoogleCloudPlatform/buildpacks#configuration
 Failure: (ID: 2ff5e9d8) getting status code for https://api.adoptopenjdk.net/v3/assets/feature_releases/11/ga?architecture=x64&heap_size=normal&image_type=jdk&jvm_impl=hotspot&os=linux&page=0&page_size=1&project=jdk&sort_order=DESC&vendor=adoptopenjdk: Head "https://api.adoptopenjdk.net/v3/assets/feature_releases/11/ga?architecture=x64&heap_size=normal&image_type=jdk&jvm_impl=hotspot&os=linux&page=0&page_size=1&project=jdk&sort_order=DESC&vendor=adoptopenjdk": dial tcp <IP_ADDRESS>:443: i/o timeout
 --------------------------------------------------------------------------------
 Sorry your project couldn't be built.
 Our documentation explains ways to configure Buildpacks to better recognise your project:
  -> https://github.com/GoogleCloudPlatform/buildpacks/blob/main/README.md
 If you think you've found an issue, please report it:
  -> https://github.com/GoogleCloudPlatform/buildpacks/issues/new
 --------------------------------------------------------------------------------
 ERROR: failed to build: exit status 1

Since many production environments cannot access internet, can we build Function from source code without internet?

It looks like your cluster can't access this address: dial tcp <IP_ADDRESS>:443: i/o timeout
The openfunction/gcp-builder:v1 is a builder provided by Google, and it may require your network to be unrestricted. So we are working on builders that can adapt to multiple network environments, including the Java builder (in progress)
The workaround is to configure a proxy for your cluster so that the builder can access api.adoptopenjdk.net

Alternatively if your local network is good,  you can build it in your local and push the image to your harbor.

Install Docker and the pack tool.

Build a container from your function using the Functions buildpacks:

pack build \
	--builder openfunction/gcp-builder:v1 \
	--env GOOGLE_FUNCTION_TARGET=com.openfunction.HelloWorld \
	--env GOOGLE_FUNCTION_SIGNATURE_TYPE=http \
	my-java-sample

Deploy the serving-only function like this example.

Alternatively if your local network is good, you can build it in your local and push the image to your harbor.

Install Docker and the pack tool.
Build a container from your function using the Functions buildpacks:

pack build \
	--builder openfunction/gcp-builder:v1 \
	--env GOOGLE_FUNCTION_TARGET=com.openfunction.HelloWorld \
	--env GOOGLE_FUNCTION_SIGNATURE_TYPE=http \
	my-java-sample

Deploy the serving-only function like this example.

Thanks @lizzzcai , I'm going to add the pack build method to the official doc


config:revert empty stat update fix

Signed-off-by: Rama<EMAIL_ADDRESS>Description: Reverting the empty stat update fix so that we fix this bug https://github.com/envoyproxy/envoy/issues/4485
Risk Level: Low
Testing: Automated tests
Docs Changes: N/A
Release Notes: N/A
Fixes https://github.com/envoyproxy/envoy/issues/4485

@htuch reverted that fix

@ramaraochavali thanks for this. @rfaulk can you confirm this fixes the issue (if you can't patch it onto the internal tree, just make the changes by hand, it's small)?
I'd prefer not to make too many changes to how these stats are computed, as consumers will see anomalies. While @rfaulk confirms, maybe we can try and figure out the right long term fix. I would argue as a strawman that the previous "empty stat bump" could be considered correct once we have incremental xDS, since the right way to provide partial updates will be via that mechanism. Does that sound too crazy?

@htuch I agree that "empty stat bump" could be correct with incremental xDS. I am fine to wait on this till increments xDS comes and we can fix it if it is still confusing then.

@ramaraochavali OK. Let's wait for @rfaulk to confirm that this does fix the bug and we can ship the revert. Thanks.

Ok. No problem. I was able to reproduce the problem locally and this change fixed it. So we should be good.

OK, SGTM, will merge, thanks.


Confused on Documentation

In the ReadMe for exposify it says:
Or use it via browserify-shim which allows you to provide exposify config inside your package.json among other features.

I'm really confused where in my package.json I would set exposify.config using browserify-shim.   Can you provide an example or show a simple setup in the main page Readme?
Is it in the browserify transform option:
  "browserify": {
    "transform": [
      "browserify-shim",
      "exposify" <-- HERE
    ]
  },

Or perhaps in the browserify-shim config itself:
  "browserify-shim": {
    "jquery": "global:jQuery",
    "angular": {
      "exports": "angular",
      "depends": [
        "jquery:jQuery"
      ]
    },
    "hammer": "HammerJS",
    "kendo": {
      "depends": [
        "jquery:jQuery"
      ]
    }
  },

Thanks in advance!

browserify-shim uses exposify under the hood for cases where you set global:{name} in your config. It also does the dramatically more complicated work of shimming badly behaved packages. You don't insert exposify into your config or use it at all if you're using b-shim.


Timeline: Item preview cannot be loaded.

The issue appears because BxDolPage is used in dynamically generated popup and 'does not know' about it. Therefore it doesn't include the JS/CSS files in appropriate way.

Fixed in #1249


Feature request: Linechart

I think it would be nice if we had an option for the chart widget to draw lines between each marker. I saw you were able to have a linechart in examples/demo.rs but only when feeding in continuous data. Maybe add an option to the chart widget or create a new linechart struct to enable drawing a line between consecutive points. I could probably hack a pr together but I wasn't sure what the best way to implement it would be.

I would like to first make sure that I understand your proposition. Is it a matter of adding new points in between using the same marker or to use dot symbol for the points and braille symbols for the lines ? In the first case, it may be hard to achieve with the current version of the Chart widget (maybe a wrapping iterator over the data provided by the user). In the second case, a new widget is definitively a better idea.

I was thinking of the first option I think. Basically getting the functionality as seen in the gif in the readme without having to provide continuous data. So adding lines between markers using the same marker style.


ContextPoint incorrect with scaled custom device when listening to screen

When listening to Framer.Device.screen.onMouseMove(), and when there is a custom device, and when the window is not 100%, the event.contextPoint coordinates are not with respect to the scaled screen space as they are when listening to a layer. I'm not sure if this is expected behavior (contextPoint isn't technically in the built-in documentation) but it is kind-of undesirable... Example: https://framer.cloud/RYPBi
The coordinates listed in this image are for the top right corner.

This becomes an extra problem in cases like this:
The linked project only works as expected when the window is at 100%. When full-screened, scaled, or in a browser the coordinates are incorrect. Assuming these things shouldn't affect convertToPoint ?
Works fine:

No good:

https://framer.cloud/sPihv


feature: Add support for python markdown and image attachments

I updated the code to work with Python Markdown extension:
https://github.com/ipython-contrib/jupyter_contrib_nbextensions/tree/master/src/jupyter_contrib_nbextensions/nbextensions/python-markdown
It will likely work with other Jupyter extensions as well since it uses jupyter nbconvert shell command to get content now.
I also adjusted the code so that the system copies all images to output directory. It does not currently work with attachments since I do not use them myself. To include image, use the following Markdown syntax:
![image](image.url.jpg)

Issue reference:
https://github.com/danielfrg/pelican-ipynb/issues/86

Thanks for the PR! I would like to support those extensions. I have not updated this in a while so i have a few questions.
Does it still work with the CSS tricks I am using? Is the HTMLExporter is now basically ignored? I would love to find a way to support the extensions without the explicit subcall. I will try to test this soon.


[Question] safety of encrypting the master key

With the latest version, the release notes state:
Removed comparison of the QuickUnlock key and the input of the user. Instead this version encrypts the master key of the database with the QuickUnlock key and decrypts the master key with the user input. If wrong user input is provided the decryption just generates garbage and the database stays locked.
When the master key is encrypted, how safe is this process? Is this stored in memory and destroyed when KeePass closes? Can you elaborate how it works to assure everyone that may have similar questions?
 It's not fun retyping a long key every 10min!
Cheeers

When the master key is encrypted, how safe is this process?

The master password gets encrypted with ChaCha20 (>=2.35) or Salsa20 (<= 2.34) and is stored with the ProtectedBinary class from KeePass. If you trust the encrypted KeePass database file, you can trust the encrypted master password.

Is this stored in memory and destroyed when KeePass closes?

The ProtectedBinary data gets destroyed when KeePass closes. That's the reason QuickUnlock doesn't work for the first time because there is no stored data.
If this doesn't answer your questions or you have more questions, please ask.

I now understand how your plugin works and my mind is now at ease.
Thanks again for the great on this plugin. I can't live without it now!


Authorization.HasPermission return always false

hi
Authorization.HasPermission return always false.
I want to check if a particular user is a member of a role.
For example: I created a role named Test.
I'm trying to get if this code:
  if (Authorization.HasPermission ("Test"))
{
run if true .....
}

But he returns me false all the time.
Where am I wrong?
Thanks!

Roles and permissions are different things. HasPermission checks permissions, not roles.

Thanks.
How can I tell if a certain role belongs to a particular user? (Or current user?)


What bit of code is doing the popup over the column header?

When I roll the mouse over the column header a little popup appears under the mouse with the column header text in it.
What bit of code is doing that? I want to see how you are doing the popup.

That is just an HTML title. See here:
https://github.com/AllenFang/react-bootstrap-table/blob/2e6144c9994ce38d03fff58d50c04f04f343647d/src/TableHeaderColumn.js#L106-L112

Thanks @bontibon  :)

Well ya learn something new every day.  I thought I had to make my own popup but there it is built in.


JavaScript Intellisense

The Monaco component needs to be configured to offer intellisense for JS syntax.

BlazorMonaco is currently lacking support to add ExtraLibs (how it works in the js designer)
I see there is a open PR that enables support for custom languages, which seems like a neat way to extend the intellisense capabilities: https://github.com/serdarciplak/BlazorMonaco/pull/66

Can you give me an estimation when it can be fixed? I'm using a lot of JS scripting and this is a big pro of elsa since elsa v1 Thank you, regards Jochen

Thanks for reaching out. Given the competing priorities and commitments, I can't provide a specific timeline for this feat. However, it's on my radar, and I'll prioritize it accordingly among the other requests. Thanks for your understanding.

Is it possible to use the JS lib directly without the hampering/imitation of the blazor ecosystem?
Microsoft themselves make great use of it for their interactive docs 🤔

For sure, and it may have to come down to that. The Blazor Monaco project is a wrapper around Monaco, and we’d have to do something similar in terms of JS Interop.

I will make a PR this week! But what I did to get codehinting to work is to create a typedefinitionservice in studio.core.
Then in the studio.shared project in ExpressionInput.razor.cs I added a method that will use that service to retrieve the Javascript type edfinition from the backend.
Using JSRuntime.InvoikeVoidAsync you can then push the received body to: monaco.languages.typescript.javascriptDefaults.addExtraLib
To receive the javascript hinting for the correct workflow/activity you need the following parameters:
var activityTypeName = EditorContext.ActivityDescriptor.TypeName;
var propertyName = EditorContext.InputDescriptor.Name;
var definitionId = EditorContext.WorkflowDefinition.DefinitionId;
If you are in a hurry and cannot wait on the PR + release you can use the above to get it working.

That’s great! I think it’s ok to await your PR, given that it’s been a long time already without intellisense. It would be amazing to get it back in. Appreciate your efforts!

Many thanks for your PR @sNakiex , it works well and it was merged today 🙌🏻


linux_4_18: HID: core: fix grouping by application

Motivation for this change
This patch fixes #45165
commit f07b3c1da92d ("HID: generic: create one input report per
application type") was effectively the same as MULTI_INPUT:
hidinput->report was never set, so hidinput_match_application()
always returned null.
Fix that by testing against the real application.
Note that this breaks some old eGalax touchscreens that expect MULTI_INPUT
instead of HID_QUIRK_INPUT_PER_APP. Enable this quirk for backward
compatibility on all non-Win8 touchscreens.
link: https://bugzilla.kernel.org/show_bug.cgi?id=200847
link: https://bugzilla.kernel.org/show_bug.cgi?id=200849
link: https://bugs.archlinux.org/task/59699
link: https://github.com/NixOS/nixpkgs/issues/45165
Cc<EMAIL_ADDRESS># v4.18+
Signed-off-by: Benjamin Tissoires<EMAIL_ADDRESS>Things done

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

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

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

cc @NeQuissimus

@GrahamcOfBorg build linux_4_18

Timed out, unknown build status on x86_64-linux (full log)
Attempted: linux_4_18
Partial log (click to expand)
  CC [M]  drivers/usb/usbip/stub_dev.o
  CC [M]  drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.o
  CC [M]  drivers/usb/serial/sierra.o
  CC [M]  drivers/usb/usbip/stub_main.o
  CC [M]  drivers/staging/rtl8723bs/hal/odm.o
  CC [M]  drivers/usb/serial/usb-serial-simple.o
  CC [M]  drivers/usb/usbip/stub_rx.o
  CC [M]  drivers/usb/serial/spcp8x5.o
building of '/nix/store/j1wkjn2dhcyx62iqkgj2lg8wppivhb9k-linux-4.18.5.drv' timed out after 1800 seconds
error: build of '/nix/store/j1wkjn2dhcyx62iqkgj2lg8wppivhb9k-linux-4.18.5.drv' failed

Success on aarch64-linux (full log)
Attempted: linux_4_18
Partial log (click to expand)
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
cannot find section .dynamic
/nix/store/3rm6q01fxx377gbjbqshad3ivrjimzbv-linux-4.18.5

Timed out, unknown build status on x86_64-linux (full log)
Attempted: linux_4_18
Partial log (click to expand)
  CC      drivers/cpuidle/sysfs.o
  CC      drivers/cpuidle/poll_state.o
  AR      drivers/cpuidle/built-in.a
  CC [M]  drivers/crypto/cavium/nitrox/nitrox_main.o
  CC [M]  drivers/crypto/cavium/nitrox/nitrox_isr.o
  CC [M]  drivers/crypto/cavium/nitrox/nitrox_lib.o
  CC [M]  drivers/crypto/cavium/nitrox/nitrox_hal.o
  CC [M]  drivers/crypto/cavium/nitrox/nitrox_reqmgr.o
building of '/nix/store/07jpgxrj0dv5d5pvndnzbvpj8mrz7j1x-linux-4.18.5.drv' timed out after 3600 seconds
error: build of '/nix/store/07jpgxrj0dv5d5pvndnzbvpj8mrz7j1x-linux-4.18.5.drv' failed

@grahamc Is the 1800s timeout a hard limit? Needs a good amount of steam to build a kernel in 30 mins :D

@NeQuissimus it's controlled by the package's meta, See Chromium

Ah, good find, did not know about this!

@NeQuissimus I filed https://github.com/NixOS/nixpkgs/pull/46073 for increasing the timeout.

I saw :D

nice! Thanks.


Composite Index and composite Unique supports

Having a unique constraint that spans multiple columns is not that uncommon, so a unique index that works on multiple columns would be very welcome.

No doubt it's a useful feature.


WIFI-1032 Repair EC420 MAC retrieval from manufacturing data

Signed-off-by: Rick Sommerville<EMAIL_ADDRESS>

File is required because 11-ath10k-caldata uses it to correctly set the WiFI MAC addresses from the calibration data.


POC - Clearly identify Slots in DocSite

Identify slot properties explicitly and mark with blue 'S'

Add ref to Shorthand page.

Identified explicitly in code:

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS


Add Analytics to the Submission Form

We need the ability to track visits and bounce rates on our submission form. Requesting the team to add any anonymous analytics tracker such as Google Analytics or equivalent.

Having analytics is good. However I would suggest we do not use Google Analytics as it is not GDPR compliant.
We can go with using CloudFlare Pages just like the way we do for the Kindly project as it is compliant.
Would this require us to move the submission form from our current hosting (Vercel) to CloudFlare.
Any thoughts @nathanbaleeta cc @lacabra ?

Having analytics is good. However I would suggest we do not use Google Analytics as it is not GDPR compliant. We can go with using CloudFlare Pages just like the way we do for the Kindly project as it is compliant.
Would this require us to move the submission form from our current hosting (Vercel) to CloudFlare. Any thoughts @nathanbaleeta cc @lacabra ?

@nathanfletcher Google web analytics can still be used but in a more cautious manner as other websites do:

DPG submission form uses cookies if the visitor agrees. Google Analytics uses "cookies", which are text files placed on your computer, to help the website analyse how users use the site. The information generated by the cookie about visitor use of the website will be transmitted to and stored by Google on servers in the United States. This cookie is only set if the visitor has agreed to track by clicking on the cookie banner.


SNOW-1347394: Remove BaseQueryCompiler + error in QC default to pandas

Please answer these questions before submitting your pull requests. Thanks!

What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes SNOW-1347394

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code
[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency

Please describe how your code solves the related issue.

This PR removes our vendored copy of the BaseQueryCompiler class, inheriting the class from upstream Modin instead. Similarly, it removes all the operator registration classes defined in snowflake.snowpark.modin.core.dataframe.algebra.default2pandas, with one exception. Upstream Modin does not properly render the names of property objects (https://github.com/modin-project/modin/issues/7233), so we should override DataFrameDefault.register to fix this until this issue is fixed upstream.
This PR incidentally removes Series.dt.week + Series.dt.weekofyear, which were already removed in pandas 2.0.

Do we not want the message to include the specified parameters? Some query
compiler methods directly call DataFrameDefault.register when certain
parameter combinations are encountered, while the method is generally
supported.
On Tue, Apr 30, 2024 at 17:34 Naresh Kumar @.***> wrote:

@.**** approved this pull request.
In
src/snowflake/snowpark/modin/plugin/compiler/snowflake_query_compiler.py
https://github.com/snowflakedb/snowpark-python/pull/1454#discussion_r1585733449
:

   )

   return_query_compiler.snowpark_pandas_api_calls.append(

       {

           TelemetryField.NAME.value: f"{object_type}.{fn_name}",

           SnowparkPandasTelemetryField.IS_FALLBACK.value: True,

       }

   # Previously, Snowpark pandas would register a stored procedure that materializes the frame

   # and performs the native pandas operation. Because this fallback has extremely poor

   # performance, we now raise NotImplementedError instead.

   args_str = ", ".join(map(str, args))

   if args and kwargs:

       args_str += ", "

   if kwargs:

       args_str += ", ".join(f"{key}={value}" for key, value in kwargs.items())

   ErrorMessage.not_implemented(

       f"Snowpark pandas doesn't yet support {object_type}.{fn_name}({args_str}"

nit: ...doesn't yet support the method {object_type}.{fn_name}
—
Reply to this email directly, view it on GitHub
https://github.com/snowflakedb/snowpark-python/pull/1454#pullrequestreview-2032857055,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDT7LZVBBULMQ3PNKXN5IMTZAA2C7AVCNFSM6AAAAABG7D2UY6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMZSHA2TOMBVGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Make "Unnecessary using directive" an code action instead of warning

It generates too much warnings.

Remove Unnecessary Using isn't really a warning. There's a diagnostic from the compiler, and IIRC it's supposed to be "Info" with a hint to the editor to simply fade it out. Unfortunately, VS Code doesn't support fading today.
This should be user configurable. We're working on a better configuration story for diagnostics (likely using .editorconfig) and we'll need to ensure that gets threaded through OmniSharp.

+1

This happened in C# for VS Code 1.9.


Add linting script for prowjobs

Added Prowjob linting logic for presubmits and postsubmits.
Had to create a new PR for linting as I pushed to the closed PR by mistake.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I'm late to the party, but looks good! Thanks

I'm late to the party, but looks good! Thanks


Math can't use deepClone correctly

This maybe split the package and export the clone Mode

Math can use custom clone with this PR, higher performance :https://github.com/galacean/engine/pull/2100


Improve flow for users stuck on old sync data format that cannot be migrated

I got into this situation by trying to sync to a dev server after a long time since the {} -> { version: 1, content: ... } migration has been removed. This may affect a handful of preview users, but it could also happen in the future when/if we remove a version 1->2 migration path. Problems/suggestions below:
This happens when you try to turn on sync:

I think we should create a specific message for this case where the format is no longer/not compatible with the current version. Perhaps with a button to help action it (open synced data, clear data?).
Once I figured out how to get to the synced data view, clicking on the bad settings gave me this error:

What I needed to do was reset the data in the ... menu:

We should add a clear data button to the cannot parse sync data notification to help the user get out of this situation.

Following notification is shown when you have old data in cloud

Reset action will show following confirmation dialog before resetting

Show synced data action will show sync activity view and opening data will show following error with reset action

@gregvanl It would be great if you have any suggestions for wordings.
Context - this error is shown when server has older data and client is on newer version that requires to reset data on server.

@sandy081 The text looks fine except there is an extra "in".
"Preferences sync cannot be turned on because your data in the cloud is older than that of in the client."
I think it should be:
"Preferences sync cannot be turned on because your data in the cloud is older than that of the client."

Not sure if this can be verified as it needs to have some old data or changing code.
@Tyriar Is it possible for you to verify this either by tweaking data in the test server or its also ok to just do code review and looking at the images I attached when this happens?

I had some trouble verifying as well, reviewed the messages and commit, looks good 👌


[HxCheckbox] Tri-State feature request (indeterminate state)

Discussed in https://github.com/havit/Havit.Blazor/discussions/648

Originally posted by bluisanajmr November  6, 2023
It would be very useful if the checkbox or switch controls had a tri-state option.  I am currently needing this to be used as the header in a HxGrid where one of the columns is a boolean.  All of my grid headers allow searching but it isn't currently feasible to allow searching within a boolean column header with a switch or checkbox because that would force all records to be either True or False when initially displayed.  Ideally, the user would be able to see all records and then choose to see only true or false records by clicking the checkbox or switch.

https://getbootstrap.com/docs/5.0/forms/checks-radios/#indeterminate
Currently available only via JavaScript (the :indeterminate pseudo-class has to be applied).

Interesting,
I hadn't seen that version of a tri-state checkbox.  I used it from the demo and it acted a little strangely.  It started out in what I would call the null state and then when you clicked it, it became checked.  You could then uncheck it but you couldn't get back to the null state without refreshing the page.  Maybe that was just an issue with the demo but I would expect it to go to true when you check it,  to false if you check it again and then null if you clicked it a 3rd time.

It started out in what I would call the null state and then when you clicked it, it became checked. You could then uncheck it but you couldn't get back to the null state without refreshing the page.

I think this is how it is meant (the demo).
But we can provide different implementation.


Tasks and migrations env utilities

The rendering context now includes _copier_conf, which is a copy of the raw ConfigData object, but removing anything not JSON-serializable (a.k.a. data.now and data.make_secret).
This is useful for a number of purposes:

Imagine you have migration scripts but you want to store them on your template and NOT in your copies. Then you can use _copier_conf.src_path to find the local clone and call anything from there. This fixes #95 and fixes #157.
In the execution environment, it includes $STAGE (can be task, after or before), and in the case of migrations also $VERSION_{FROM,TO,CURRENT} to let migration scripts know where they are in the moment they're called, so we can reuse migration scripts and unleash their full power.

Now you can write your scripts in any language you want, and use any task execution system you want. Tests include a bash script and a python script, and docs include how to integrate with Invoke.
@Tecnativa TT20357

As a present, I added smarter caching.
1st run: 3m29s https://github.com/pykong/copier/actions/runs/55047625
2nd run: 1m12s https://github.com/pykong/copier/actions/runs/55049232


Definition mismatch between Cvariant in pdsc and component in cproject yml file

From https://github.com/Open-CMSIS-Pack/Open-CMSIS-Pack-Spec/blob/main/schema/PACK.xsd#L329
Seems that variant name could be a string "with no condition"
In https://github.com/Open-CMSIS-Pack/devtools/blob/main/tools/projmgr/schemas/common.schema.json#L340
component and variant name are not "full" string and doesn't accept "(" for example

@jeromecoutant this is something to discuss for all parts of the componentID.
Probably this should become a change request for the PACK.xsd schema to restrict the allowed characters and then align them with cproject/clayer schema file.
What exact requirements do you have/see?

Maybe space ?
ex:
/ARM.CMSIS.pdsc:    <component Cclass="Device" Cgroup="Startup" Cvariant="C Startup"

Analyzing current package descriptions listed in https://www.keil.com/pack/index.pidx we find in:
Cbundle, Cclass, Cgroup, Csub, Cvariant
A-Za-z0-9
space
_ - + . ' ( ) /
: found in Csub is the most troublesome
In conditions we additionally find * ?
No leading: space - + . ' / ( ) found

@jkrech can we come to a conclusion here?  Are there changes suggested? What exactly should change?

Both the Pack Schema File (https://github.com/Open-CMSIS-Pack/Open-CMSIS-Pack-Spec/tree/main/schema) as well as the csolution, cproject, clayer json schema files (https://github.com/Open-CMSIS-Pack/devtools/tree/main/tools/projmgr/schemas) shall be refined to restrict the allowed characters in componentIDs and to align on the exact same restrictions.
<xs:pattern value="[A-Za-z0-9_-+ . :' \(\)/ ]*"

@soumeh01, could you please check the restrictions now present in PACK.xsd and match them with the componentID schema for the yml files so we are consistent.

Will be part of cmsis-toolbox 1.5.0

fixed in CMSIS-Toolbox 1.5.0


feat(transformer): class static block transform

Add ES2022 class static block transform.

#6733  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite

Merge activity

Oct 21, 11:39 PM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


Modify the user-info and task-detail forms such that the user can know all the fields are mandatory easily.

As discussed offline, I think you can make this more consistent. At the moment you handle empty fields differently (display an error on the form) to spaces (redirecting to the 400 page). It would be better if these were handled the same way.
Also, think of ways you can make it clearer to the user that the fields are mandatory, these errors are only shown if the field above is populated when they could all be shown at the same time.

As Steven commented on the PR#21, I will make the forms across different pages more consistent.


Bundle gets es6 syntax

Hi, I've encountered a problem. My code should be in es5 standard, but the bundle gets code with es6. I started looking into why this is happening and found a line of code that adds the 'const' declaration to the bundle.
If I change it here from const to var, the problem goes away:
- path.node.body.unshift(t.variableDeclaration("const", declarators));
+ path.node.body.unshift(t.variableDeclaration("var", declarators));

(link to reference)
Example of code from bundle (before fix):
const _tmpl$ = /*#__PURE__*/template(`<div><div></div><div></div></div>`, 6);

var ConfigOptionsPreview = function ConfigOptionsPreview(props) {
  // some code (es5)
};

Example of code from bundle (after fix):
var _tmpl$ = /*#__PURE__*/template(`<div><div></div><div></div></div>`, 6);
// ^ the fix replace 'const' by 'var'

var ConfigOptionsPreview = function ConfigOptionsPreview(props) {
  // some code (es5)
};

I think this is the wrong solution to the problem, but I haven't found any other way to downgrade the standard from es6 to es5. I started researching how other babel plugins are written and noticed that they specify var instead of const/let (for example: 1, 2, etc...).
How to solve this problem? I can create repo for reproduce problem if needed.
babel config:
module.exports = {
	// ...
	presets: [
		"solid",
		[
			"@babel/preset-env",
			{
				exclude: [
					"transform-async-to-generator",
					"transform-regenerator",
				],
			},
		],
	],
  // ...
}

P.S: Sorry for my bad English, it's not my native language.

I think you need to set preset-env's targets option to the older targets you support.

It's already setupped.

Hi, Have you solved it yet?
I had the same problem, my targets were set low enough (chrome >= 40), but the jsx template product I built still contained const
// ...
function template(html, isCE, isSVG) {
  var node;
  var create = function create() {
    var t = document.createElement("template");
    t.innerHTML = html;
    return isSVG ? t.content.firstChild.firstChild : t.content.firstChild;
  };
  var fn = isCE ? function () {
    return untrack(function () {
      return document.importNode(node || (node = create()), true);
    });
  } : function () {
    return (node || (node = create())).cloneNode(true);
  };
  fn.cloneNode = fn;
  return fn;
}

const _tmpl$ = /*#__PURE__*/template(`<div b=2 a>`);
// ^ this


How to prevent CompletionItemProvider from being executed in strings and comments?

I'm working on an extension and have a CompletionItemProvider. It works good when writing code. But it's annoying when it pops up suggestions inside of strings and comments.
I don't want the suggestions to appear in that cases.
I had a look at other extensions on how they solve this problem. The TypeScript plugin has got these lines:
function TypeScriptCompletionItemProvider(client) {
    this.triggerCharacters = ['.'];
    this.excludeTokens = ['string', 'comment', 'numeric'];
    this.sortBy = [{ type: 'reference', partSeparator: '/' }];
    this.client = client;
    this.config = configuration_1.defaultConfiguration;
}

I tried to define the excludeTokens in the same way but it doesn't work. How can I prevent the CompletionItemProvider from being executed in strings and comments?

I think this is the same as https://github.com/Microsoft/vscode/issues/719?

@lukehoban Yes, you're right.

yes, closing as a duplicate of #719


const IList, ISet, IMap etc?

class Sample {
  final Ilist<String> list;
  Sample({ this.list = const <String>[].lock});
}

all default values of constructor should be const , how can i set default value for IList ?
above code fails with  Context: Extension operations can't be used in constant expressions.

You should use this pattern instead:
class Sample {
  final IList<String> list;

  Sample({ Iterable list = const [] })
         : list = IList(list);
}

This way you can accept any Iterable in the constructor. Note, if the constructor is passed an IList it will just reuse the same instance (the IList constructor will return the original list instance if it's an IList already).
Also note: I intend on allowing for a const IList in the future, but that's not as easy as it seems, because the IList has mutable information. For example, it caches the hashCode and it can be "flushed". However, it is possible to allow for a const constructor, but it needs some work, and I haven't had the time yet.

Version [5.0.0-dev.1 ] now has an experimental const IList.
Examples:
const IList<String> myList = IListConst.empty();
const IList<int> myList = IListConst([1, 2, 3]);

Version [5.0.0-dev.8] now has both a const IList and a const ISet.
It now also includes the necessary explanation in the Readme.

@marcglasberg Perhaps we could sponsor the development of IMapConst?

@venkatd Sure. I'll ping you in Whatsapp tomorrow and we can discuss it.

Done: version 7.0.0.


Make it possible to thread the classification of solutions

When classifying the solutions of 4 parametrons (81 stable branches), it takes one hour to complete on one thread. Instead, now the loop inside the classification function will be threaded to speed up the process.
Resolves #86

Works fine but I see the problem here is get_single_solution. Constructing a whole dictionary to feed to the classifying function is just really really slow.


"One or more fields contain incorrect values" error while creating new API through apimanagement python sdk

Describe the bug
We are trying to use apimanagement python sdk to automate api deployment into api management service over azure.
Using apimanagement v 0.2.0.
Using following configs and code to create a simple test api using sdk and its throwing "One or more fields contain incorrect values" error. It doesn't tell us which fields are incorrect, making it harder to debug.
apim_client = azure.mgmt.apimanagement.ApiManagementClient(credentials=credentials, subscription_id=subscription_id)

api_create_or_update_parameters = azure.mgmt.apimanagement.models.ApiCreateOrUpdateParameter(
    description="AI reserve deployed sample api",
    authentication_settings = None,
    subscription_key_parameter_names = None,
    api_type = None,
    api_revision= "1",
    api_version= None,
    is_current= True,
    is_online= None,
    api_revision_description= None,
    api_version_description= None,
    api_version_set_id= None,
    subscription_required= True,
    source_api_id= None,
    display_name= "Sample API",
    service_url= "https://conferenceapi.azurewebsites.net",
    path= 'sample',
    protocols= ['https'],
    api_version_set= None,
    value= "sample_test",
    format="openapi"
)

deployed_api_response = apim_client.api.create_or_update( resource_group_name, apim_service_name, "test-api", api_create_or_update_parameters)
print(deployed_api_response.result())

Exception or Stack Trace
Traceback (most recent call last):
  File "/Users/devpatel/Desktop/projects/Ai-Reserve-PlatformProvisioning/aml_service/configureAPIM.py", line 63, in <module>
    deployed_api_response = apim_client.api.create_or_update( resource_group_name, apim_service_name, "test-api", api_create_or_update_parameters)
  File "/Users/devpatel/Desktop/projects/.venv/lib/python3.7/site-packages/azure/mgmt/apimanagement/operations/_api_operations.py", line 378, in create_or_update
    **operation_config
  File "/Users/devpatel/Desktop/projects/.venv/lib/python3.7/site-packages/azure/mgmt/apimanagement/operations/_api_operations.py", line 314, in _create_or_update_initial
    raise models.ErrorResponseException(self._deserialize, response)
azure.mgmt.apimanagement.models._models_py3.ErrorResponseException: One or more fields contain incorrect values:

Note: I am able to trigger api.delete() or api.get() operations successfully. Facing the error while trying to create a new one.
Environment Setup

Python Version: [e.g. Python 3.7]
SDK Version: [azure-mgmt-apimanagement v0.2.0]

Hi service team, would you have a look at this issue? I don't reproduce this in my side, and the message should comes from the service http response. Thank you.

@changlong-liu Were you able to create an api using the api parameters listed above? Are those parameters looks good to you?

Hi, @devang93 , I get ErrorResponseException with a different error message using the api parameters listed above.
The error message is returned by the service side.

@changlong-liu  got it. I am trying to debug it by changing some parameters to None but still unable to figure out for which parameters its giving me this error. Waiting on the service team's response. Thanks

@changlong-liu Any updates here? I am not sure how to reach out to the service team and we are blocked here.

@devang93 are you using actual OpenApiSpec in the Value field (instead of sample_test)?  you can first try to do this using REST API https://docs.microsoft.com/en-us/rest/api/apimanagement/2019-12-01/apis/createorupdate
value= "sample_test",
format="openapi"

@KedarJoshi I didn't quite understand it. What is the expected value for "value" field?
I just commented the "value" field out and I was able to create the api in apim. However, it doesn't import all the operations automatically.

@KedarJoshi after going through the rest api documentation I understand that "value" field requires api spec documentation.
I am passing the following values for a sample api but it still fails with the same error. The sample api in this example uses swagger specs.
 value= "https://conferenceapi.azurewebsites.net/",
 format="swagger-link-json"

I also tried to download the swaggerspec file and then pass that entire file string into value but it again failed with same error.
with open ("api_spec.yaml", "r") as specfile:
    api_spec=specfile.readlines()

...
 value= "https://conferenceapi.azurewebsites.net/",
 format="swagger-json"

Please try this one:
value= "https://conferenceapi.azurewebsites.net/",  format="openapi-link"

@RupengLiu yes, it worked. I was also able to figure out another swagger-link-json as well. I figured out that the ConferenceApi I was trying out with was not swagger formatted.
My issue has been resolved but however, it would be nice if the error/exception given described which field is missing or have incorrect value. Following error doesn't explain much or help in debugging:
azure.mgmt.apimanagement.models._models_py3.ErrorResponseException: One or more fields contain incorrect values:

Hi @devang93 , the detail reasons are in the response body of the apimanagement service response. It's too complex to show in one exception message.
Would you try this code:
try:
    deployed_api_response = apim_client.api.create_or_update( resource_group_name, apim_service_name, "test-api", api_create_or_update_parameters)
except Exception as e:
    print(e.response.content)   # all response body returned by service side.

You may find details information for the error.

Hi, this issue seems to be resolved, so I'll close it for now. If you still have more questions, please reopen. Thanks!

Hi, this issue seems to be resolved, so I'll close it for now. If you still have more questions, please reopen. Thanks!


Allow multiple updates

First of all thank you for this great library !
I would like to be able to build the following kind of query.
I think it isn't possible at the moment or maybe I just could not find how.
        update trad.bcas as t
        set quantity = v.quantity::integer
        from (values
            ($1, $2),
            ($3, $4),
            ($5, $6)
        ) as v(code, quantity)
        where t.store_id = $7
        and t.schedule_id = $8
        and t.product_code = v.code
        returning *

I did it using the following, code
    const bcas = [{
        quantity: 1,
        code: '2310'
    }, {
        quantity: 2,
        code: '2730'
    }, {
        quantity: 3,
        code: '3511'
    }];

    let qb = sq.l`
        update trad.bcas
        set quantity = v.quantity::integer
        from (values
    `;

    const last = bcas.length - 1;
    bcas.map((b, index) => {
        qb = qb.l`(${ b.code}, ${ b.quantity })`;
        if (index !== last) qb = qb.l`,`;
    });

    qb = qb.l`) as v(code, quantity)
        where store_id = ${ storeId }
        and schedule_id = ${ scheduleId }
        and product_code = v.code
    `;

is there a better way atm ?
Thanks :)

You're right. Right now there is no better way to write it because there's no mechanism for adding an additional from clause to an update query.
According to Postgres's grammar, there can only be a single update table. I think I'll change how update queries are compiled so that  the first table passed to .from is used as the update table, and subsequent tables are used in the from clause.
I've also been thinking about adding an overload to .from that accepts an object, and this seems like the perfect use case. Object keys will be table names (to the right of as), and object values will the source tables (either names or an array of values).
You'd then be able to write your query as:
const bcas = [{
  quantity: 1,
  code: '2310'
}, {
  quantity: 2,
  code: '2730'
}, {
  quantity: 3,
  code: '3511'
}];

sq.from({
    t: 'trad.bcas',
    v: bcas
  })
  .set`quantity = v.quantity::integer`
  .where({
    't.storeId': storeId,
    't.scheduleId': scheduleId,
     equalProductCode: sq.l`t.product_code = v.code`
  })

That would be great ! Feel free to use my case for your needs.
I'd be glad to try out some wip if you want me to.
Thank you for your quick answer :)

This is now possible using sqorn-pg v0.0.27. Your query can be written as:
sq.from({ t: 'trad.bcas' }) // first .from call used for UPDATE CLAUSE
  .from({ v: bcas }) // subsequent .from calls used for FROM CLAUSE
  .set`quantity = v.quantity::integer`
  .where({
    't.storeId': storeId,
    't.scheduleId': scheduleId,
    join: sq.l`t.product_code = v.code`
  })
  .return`*`

or as
const values = sq .extend(...bcas.map(b => sq.l`(${b.quantity}, ${b.code})`)).join(', ')

sq.l`update trad.bcas as t`.l`set quantity = v.quantity::integer`
  .l`from (values ${values}) as v(quantity, code)`
  .l`where (t.store_id = ${storeId} and t.schedule_id = ${scheduleId} and t.product_code = v.code)`
  .l`returning *`

Thanks for the feedback. Let me know if you have any other suggestions.


Bonnie 1376 code equivalence

Integrates cql-execution-engine updates from this PR: https://github.com/cqframework/cql-execution/pull/6. This makes the engine appropriately compare codes for equivalency and check code systems even if a code system is null.
Additionally, makes patient characteristics return codes that include code systems to work appropriately with the cql-execution-engine changes.
=====
Pull requests into CQL QDM Patient API require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.
Submitter:

[x] This pull request describes why these changes were made.
[x] This PR is into the correct branch.
[x] JIRA ticket for this PR: https://jira.mitre.org/browse/BONNIE-1384
[x] JIRA ticket links to this PR
[x] Code diff has been done and been reviewed (it does not contain: additional white space, not applicable code changes, debug statements, etc.)
[x] Tests are included and test edge cases
[x] Tests have been run locally and pass (remember to update Gemfile when applicable)
[x] Code coverage has not gone down and all code touched or added is covered.
bonnie-v2.1

=============================== Coverage summary ===============================
Statements   : 63.3% ( 1359/2147 )
Branches     : 33.66% ( 239/710 )
Functions    : 50.94% ( 245/481 )
Lines        : 60.03% ( 1143/1904 )
================================================================================

bonnie
=============================== Coverage summary ===============================
Statements   : 63.3% ( 1359/2147 )
Branches     : 33.66% ( 239/710 )
Functions    : 50.94% ( 245/481 )
Lines        : 60.03% ( 1143/1904 )
================================================================================

[ ] Updated after https://github.com/cqframework/cql-execution/pull/6 accepted to confirm cql4browsers is appropriately updated
[ ] Automated regression test(s) pass

If JIRA tests were used to supplement or replace automated tests:

[x] JIRA test links: N/A
[x] Justification for using JIRA tests: N/A
[x] JIRA tests have been added to sprint N/A

Reviewer 1: @mayerm94
Name:

[x] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[x] The tests appropriately test the new code, including edge cases
[x] You have tried to break the code

If JIRA tests were used to supplement or replace automated tests:

[x] JIRA tests have been run and pass
[x] You agree with the justification for use of JIRA tests or have provided input on why you disagree

Reviewer 2:
Name: @losborne

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[x] The tests appropriately test the new code, including edge cases
[x] You have tried to break the code

If JIRA tests were used to supplement or replace automated tests:

[x] JIRA tests have been run and pass
[x] You agree with the justification for use of JIRA tests or have provided input on why you disagree

Regression test is clean using updated cql4browsers.js

Updated w @losborne's test change recommendation.
After this has approvals, will tell @cmoesel to merge the cql-execution.
Will then update this with the latest cql4browsers from that.
Will then request a final regression check.
After final regression check passes, I will merge this in.


[BUG REPORT] Error: net::ERR_CONNECTION_CLOSED

An error occurred
Error: net::ERR_CONNECTION_CLOSED
    at SimpleURLLoaderWrapper.<anonymous> (electron/js2c/browser_init.js:105:7068)

Debug info

Version
3.6.0

Commit Hash
c5946bcd63b611ab574ba1cf7d12513f949ca357

Commit Date
2021-10-15T08:18:01Z

Electron
13.4.0

Chrome
91.0.4472.164

Node.js
14.16.0

V8
9.1.269.39-electron.0

OS version
Windows 7 Professional Service Pack 1

OS release
Windows_NT x64 6.1.7601

CPUs
Intel(R) Core(TM) i5-2520M CPU @ 2.50GHz x 4

RAM
7.88GB (4.47GB free)

App RAM limit
2072Mb (10.39MB used)

Main log
[2023-11-09 14:03:43.786] [error] Error: net::ERR_NAME_NOT_RESOLVED
    at SimpleURLLoaderWrapper.<anonymous> (electron/js2c/
[2024-10-04 13:22:33.549] [error] Error: net::ERR_CONNECTION_CLOSED
    at SimpleURLLoaderWrapper.<anonymous> (electron/js2c/browser_init.js:105:7068)

Worker errors
WORKER:ERROR [2024-10-04T11:14:11.737Z] 	    Error: read ECONNRESET
  - METHOD_NAME: getAccountSummary [PROTECTED]
  - STATUS_CODE: 500
  - STATUS_MESSAGE: Internal Server Error
  - STACK_TRACE RequestError: Error: read ECONNRESET
    at new RequestError (C

Worker exceptions
Empty

The issue was opened from the electron app

Could you re-install and try the latest release?
https://github.com/bitfinexcom/bfx-report-electron/releases/tag/v4.28.0


luci-app-speedtest: Add packages

LuCI for Speedtest

I feel this is very helpful for those of us who don't want the hassle of going to the terminal just to check the internet speed of OpenWrt devices. Especially for those who are remote accessing from outside the network.
#6949

Please don't close you PR:s as soon as you make changes as that makes all the comments get lost, you can just continue pushing to the same branch and your current PR will get updated automatically.

You can also squash (= merge together) the existing commits to clean up things.
Please read advice in:
https://github.com/openwrt/luci/blob/master/CONTRIBUTING.md#advice-on-pull-requests

What the others are saying @animegasan is to go practice force pushing and adding commits to a branch. Do this before ooening a pr and observe what happens in your GitHub repo when making a force push despite any scary warnings that might appear in your client telling you not to do that.
Practice.

Please don't close you PR:s as soon as you make changes as that makes all the comments get lost, you can just continue pushing to the same branch and your current PR will get updated automatically. You should probably close this and re-use #6949 where the comments are.

I appreciate your feedback. I'd like to explain that I couldn't reopen the previous PR because I had already deleted the associated branch. As I'm relatively new to using GitHub, I'm still learning about best practices. However, I truly value your advice not to close PRs prematurely and will strive to continue learning and improving. Thank you for your understanding and support.

You can also squash (= merge together) the existing commits to clean up things. Please read advice in: https://github.com/openwrt/luci/blob/master/CONTRIBUTING.md#advice-on-pull-requests

I understand, and I admit that I've often struggled with this. I recognize that my previous PRs may have seemed spammy. This time around, I'm making a conscious effort not to repeat the same mistakes. Thank you for your patience and understanding

What the others are saying @animegasan is to go practice force pushing and adding commits to a branch. Do this before opening a pr and observe what happens in your GitHub repo when making a force push despite any scary warnings that might appear in your client telling you not to do that.
Practice.

Yes, I spent the whole night practicing on another repository, and I finally understand how to maintain my PRs according to the rules. Thank you.

Maybe this package can be forwarded by @systemcrash .
I have to learn again how to contribute. Thank You

Maybe this package can be forwarded by @systemcrash .
Look how I messed it up again

I have to learn again how to contribute. Thank You

No worries, we are all beginners at some point in time! As long as your PR isn't merged you're free to force push to make the code/commits look correct.
In your case I would reset the merge, run git fetch to get the latest from origin (probably your repo if you haven't done any custom setup with the remotes) and run git rebase origin/master (remember to sync the changes to your fork's master from this repo's master otherwise you won't get new commits to pull down). I'm not personally a fan of git merge as it introduces merge commits. Also, the reflog contains all your historical commands, so even if you delete a branch you can get the content back.

Yeah no worries. I had to learn the ropes also.
Also no merge commits. Read up on how to do a rebase of your branch on master. It will be problem free since this is a new app.

ping @animegasan

A speedtest app like OpenSpeedTest wouldn't hurt.


tour: "many implementations" link is broken

Context: https://tour.golang.org/methods/21
many implementations link is broken - open REDACTED: file does not exist

dup


Error in event handler: modeler-init

In modeler i'm getting Error in event handler for "modeler-init": "TypeError: Cannot read property 'implementation' of undefined" somewhere in the script task control neighborhood since no other controls are loading load after it

This issue is fixed in latest develop branch of spark-modeler. I noticed the error on a build of the latest develop on spark, but when I npm link'ed the latest spark-modeler develop branch, the error was gone.


[Change Proposal] Further OTP/LC Updates for Secure Manufacturing

This is a continuation of #7330 and proposes additional changes to OTP/LC to support vendor-specific OTP smoke checks for the secure manufacturing flow.
In particular:

OTP_CTRL:

allocate a new SW partition, to be used for vendor specific testing purposes only. the partition will be called VENDOR_TEST.
the partition will be 512bit in size, and allocated at offset 0x0 of the OTP memory map (i.e., before the *_SW_CFG partitions)
the corresponding offset and size parameters generated by the memory map script will be forwarded into the prim_otp wrapper
the prim_otp wrapper may disable ECC checks on that region, and implement vendor-specific OTP smoke checks

LC_CTRL:

two additional CSRs will be added to the life cycle controller TAP: a vendor test control and status register (both 32bit in size)
these two registers will be connected to the prim_otp wrapper, and the bits in those registers can be used for vendor-specific OTP smoke checks
these two registers will only be usable in TEST*, RAW and RMA life cycle states

The design and DV impact on the open-source side is not substantial, since we only add a new partition (this is partially scripted), and two CSRs with no functionality attached (the CSRs are routed to the prim_otp wrapper).
The changes have been discussed and agreed upon.

CC: @danaagur

Thanks @msfschaffner!
Sounds good.

Ok, these changes have now been implemented.


How to use german translation?

Hi.
I'm using the plugin to scan a QR-code in my app. As soon as the camera window opens, the button to cancel the operation is named to...a well..."cancel" ;-) Is it possible to change it to german? I took a look into the source files an found a file called "strings.xml" in the "res"-folder, but don't know how to active it...
Regards,
Marc

@kaju74 (the following is only valid for iOS!): you won't find something translatable for this button, for it is a standard iOS one (UIBarButtonSystemItemCancel), which is displayed in the language the App itself runs in … if your App would be "allowed" to run German (and – of course – it does run on a German iOS device :wink:), then this button would automatically read "Abbrechen".
In your Info.plist you have to include something like this:
    <key>CFBundleLocalizations</key>
    <array>
      <string>en</string>
      <string>de</string>
    </array>

(or if CFBundleLocalizations is already there, extend it accordingly).


Update VSS parsing error

Help JoyID identify the specific error by updating VSS error

https://github.com/utxostack/mutiny-node/pull/78 will solve the error, I will close this PR


[Maintenance] Find Alternatives for hosts file changes

Relevant template
No response
Problem description (optional)
If the gateway is runnning inside a container, it needs redirecting to the keycloak. At the moment it is only possible if you manipulate the hostsfile to redirect keycloak:8080 to localhost:8080. For changing the hostsfile itself you need admin rights on your pc.
Desired solution
find a solution to the redirect possible without changes inside the hostsfile
Considered alternatives (optional)
No response
Additional context (optional)
No response
No duplicate

[X] I confirm that this issue is not a duplicate

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Maybe we can use https://github.com/dvddarias/docker-hoster running as privileged container.
This looks pretty promising.

@devtobi
After trying out different fixes and giving it some thought, I reckon the best move is to tweak the host file and make sure we have some solid documentation to go with it. I chatted with @darenegade about it, and he's on board too. Sure, it's not the prettiest fix, but it's probably the best move we can make right now.

@devtobi After trying out different fixes and giving it some thought, I reckon the best move is to tweak the host file and make sure we have some solid documentation to go with it. I chatted with @darenegade about it, and he's on board too. Sure, it's not the prettiest fix, but it's probably the best move we can make right now.

Fine for me as a temporary solution. However in the future we should investigate if we can tweak the response the API Gateway sends to the browser and then redirect the browser to the exposed localhost URL. Maybe we should create an issue for that topic over at https://github.com/it-at-m/refarch?

Since we only need this for local development, there's no need to focus on optimization. Modifying the host file takes just a minute. Altering the application solely for improved local functionality isn't the right approach. Additionally, the cost-benefit ratio doesn't justify the effort. For now it would go with "wont fix"

Yeah thats fine for me. However we need to make sure company internal machines are set up correctly without admin privileges. We should still document the required change in hosts file for external users.


Unified account

Related issue(s)

Unified account

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

Unified account

Describe the solution you'd like

Unified account

Describe alternatives you've considered

No

Additional context

No

🚀 Deployed on https://pr-3754--sw-web-runner.netlify.app


Update deprecated Net::HTTPServerException with Net::HTTPClientException

Replaced Net::HTTPServerException with Net::HTTPClientException
As Net::HTTPServerException is deprecated so used, alised exception class Net::HTTPClientException
Issues Resolved
Fixes  https://github.com/chef-cookbooks/audit/issues/394
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD
Signed-off-by: Vivek Singh<EMAIL_ADDRESS>

when we planing to release these changes?


[da-vinci]Fixed a bug in DaVinci bootstrapping mode

Summary, imperative, start upper case, don't end with a period
Fixed a bug in DaVinci bootstrapping mode

How was this PR tested?
CI

Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Oops, my miss in code review.

:D Luckily, I discovered this issue during the EI test and I should add a unit test for this function in the original PR.


lightdm-mini-greeter: 0.3.3 -> 0.3.4

Motivation for this change
New upstream release.
Things done

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

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

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

@GrahamcOfBorg build lightdm-mini-greeter

@GrahamcOfBorg build lightdm-mini-greeter


listBreakpoint querystring encoding was incorrect

Added a bug (internal) to track lack of test coverage for success_on_timeout.


Talk Proposal: How to run (streaming) databases in Kubernetes

Talk Information

Draft Title: How to run (streaming) databases in Kubernetes
Length:

[ ] Lighting talk/Demo (10 min)
[x] Full Presentation (20 min)

Links: (slides, article, notes) https://www.timeplus.com/post/streaming-database

Which dates, particularly Tue/Wed/Thu during the 3rd/4th week of a month, are you likely available? (e.g. "Most Thursdays with 4 weeks notice", "June or later", etc.).    Oct 18, Friday
Short Summary of your talk:
You probably know what is a database, even what is a OLAP database. Snowflake and duckdb are OLAP, while Postgresql/MySQL/Oracle are OLTP. There are some realtime OLAP databases such as Apache Pinot, ClickHouse. They can support high QPS(Query-per-second) and scan lots of data in few seconds. However they are not stream processing. Tools like Apache Flink are stream processor, but require a OLAP as the downstream, or RocksDB as the state store. These days, there is a new category called Streaming Databases, to combine OLAP with stream processing. Timeplus is one of the vendor, with open source core engine on GitHub https://github.com/timeplus-io/proton.

In this talk, I am going to share how we can easily deploy Timeplus in Kubernetes with Helm chart, while in our fully managed cloud, why we use a slightly different approach to use Kubernetes Operator to manage multi-tenants. Key technologies we are using: AWS EKS, Fargate serverless compute, Karpenter.sh, customer terraform and GitOps.
Speaker Bio

Name: Jove Zhong
Mini-bio: (This is the introduction our emcee will give about you, one paragraph is usually best) Co-founder at Timeplus. Ex-Splunk, ex-EMC, ex-IBM. He is a board sports fanatic, incl. agile board.
Picture for slides: (We use this pictures to create promotional content and to present your talk in our socials.)
Would you like help with your presentation? (Feedback on notes, content?) Sure
Social media link(s): (twitter, website, linkedin, etc.) https://www.linkedin.com/in/jovezhong/
Do you agree to the CNCF Code of Conduct?

[x] I agree

Thanks for submitting! We will be reviewing this application soon

Hi @jovezhong,
We discussed your content with our team and have some questions about how it aligns with CNCF's goal of making cloud technology ubiquitous. We aim to ensure all our topics are connected to our mission and vision for the cloud.
Do you have any other topics that might align better with cloud initiatives?
Additionally, we do have spots open for sponsored talks if you're interested in that option.
Thanks!

Thanks for the review and feedback. Will that help if I also talk about how we run our fully-managed cloud in the session? Our software can be deployed locally with few files, but also can be deployed on Kubernetes with helm chart or docker compose. In our fully-managed cloud, we use EKS and built our own Kubernetes operator to manage multi-tenant. AWS Fargate and Karpenter are used, with GitOps deployment model.
Me or our cloud engineering lead can present this. For the audience, they will learn something know about database trends, as well as real-life lesson&learns regarding building&operating a SaaS with modern tools.

Hey @jovezhong, what would you think about scoping this a bit differently,
The talk could be something like Streaming databases from a Kubernetes cluster or something like that,
And we would love if in the content you could go through what a stateful is, if it makes sense to have db's in Kubernetes, your open source operator and all of that!
Essentially, if we make Kubernetes the main actor of this presentation this will be fully aligned!
Thanks a lot for considering giving a talk in CNCF Vancouver!

Thanks for the feedback. I have updated the original ticket with new title and summary. Look forwards to the upcoming session.

Thanks for doing the correction @jovezhong!
Your talk is now approved; see you in the event! Please aim to come a bit earlier (5-10 minutes) before the scheduled time to make sure we test our A/V setup!
Thanks a lot!

Thank you, @ShankyJS  I will come early for sure. See you next Friday.

Thank you @jovezhong please let us know if you would like us to review your slides, feel free to share with<EMAIL_ADDRESS>Have a great night!

I just started working on the slides deck. I will also invite my colleague @ye11ow as co-presenter to share more our k8s details. Aim to share you the draft slides later this week or next Monday.

Greetings, before the long weekend, we just finished the initial draft of the talk and share you the link via email. Feel free to comment. Enjoy the long weekend and we can make updates next Tuesday.

Thanks for the talk @jovezhong we enjoyed your presence! If in the future you would like to give another talk more focused in your product/sponsored the CNCF would love to have you!


Fix when only single value is passed in m2m field

When user enters single value in m2m field, then the split would not work as excel returns a float value and this error is generated AttributeError: 'float' object has no attribute 'split'.
So to fix it, we can make it a str or make a list directly in except block.

Hi @am1tyadava, this has been fixed in a different way, see https://github.com/django-import-export/django-import-export/issues/177 for details on two different commits that fixed the behaviour and added tests. Closing. Thanks for contributing!


Run tests doesn't provide any output

We've been using mstest which works fine sa per [https://github.com/OmniSharp/omnisharp-vscode/wiki/How-to-run-and-debug-unit-tests]. However the output from the test doesn't appear to go to the terminal window.
i.e. when I run a test all I see is
Build succeeded.
    0 Warning(s)
    0 Error(s)

Time Elapsed 00:00:13.84

Total tests: 1. Passed: 0. Failed: 1. Skipped: 0

If I run from from console i.e. >dotnet test then messages from the failed tests appear in the terminal window (and log messages as we have a console logger configured for our tests, i.e.
Starting test execution, please wait...
Failed   TestReports
Error Message:
 Assert.IsNull failed. items is not null

...

Total tests: 3. Passed: 2. Failed: 1. Skipped: 0.
Test Run Failed.
Test execution time: 3.1644 Seconds

Is there a way of capturing this output when running using the code lense links?
Regards
Dave

It looks like O# returns the output of individual unit tests but the C# extension doesn't print the output.
@david-waterworth Thanks for filing this, we will take a look.

After further investigation, I'm not sure O# is actually sending along all the output.

@rchande I have added some more logging around the tests in the PR: https://github.com/OmniSharp/omnisharp-vscode/pull/2337. We do need some more information from omnisharp and I have filed an issue here : https://github.com/OmniSharp/omnisharp-roslyn/issues/1198

@akshita31 Do we have a way to get console output produced by unit tests?

@rchande I created a PR to imnisharp-roslyn to do so: https://github.com/OmniSharp/omnisharp-roslyn/pull/1203


AFK broadcast color formatting crashes player

Spark link
https://spark.lucko.me/pB7iKOMDEi
Expected behavior

Using the afk-tab-list-prefix and related settings players can be displayed with color formatting in the tablist
Players marked as AFK lose that color formatting when they come back
Players don't crash

Observed/Actual behavior

Players are marked as AFK, with the correct color formatting
Players coming back from AFK keep the color formatting in the tablist
Players that go AFK for a second time experience a crash

Steps/models to reproduce

Create a new purpur server
Set a player-idle-timeout in server.properties (I set it to 1 for testing)
In purpur.yml:

Disable kick-if-idle
Enable update-tab-list
Set afk-tab-list-prefix to <gray><italic> (haven't tried with other color and formatting changes yet)

Join the world, and wait for the idle timer to run out
Player should correctly be marked as afk, with color formatting in the tablist
Move around, afk-broadcast-back message will be sent, but player will incorrectly keep color formatting in the tablist
Wait for the idle timer to run out again
Experience client crash

Purpur version
[19:37:28 INFO]: Checking version, please wait...
[19:37:28 INFO]: Current: git-Purpur-1964 (MC: 1.19.4)*

You are running the latest version

Agreements

[X] I am running the latest version of Purpur available from https://purpurmc.org/downloads.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[ ] I ticked all the boxes without actually reading them
[X] My version of Minecraft is supported by Purpur.

Other
Server logs: https://mclo.gs/6KNG5Rz
Error snippet from that log:
[19:31:02] [Server thread/INFO]: 404_not_found_ is now AFK
[19:31:02] [Server thread/WARN]: Failed to handle packet for /<IP_ADDRESS>:59065
net.kyori.adventure.text.minimessage.internal.parser.ParsingExceptionImpl: Legacy formatting codes have been detected in a MiniMessage string - this is unsupported behaviour. Please refer to the Adventure documentation (https://docs.advntr.dev) for more information.
	<gray><italic> 7 o404_not_found_
	              ^^
	at net.kyori.adventure.text.minimessage.internal.parser.TokenParser.parseString(TokenParser.java:170) ~[adventure-text-minimessage-4.13.1.jar:?]
	at net.kyori.adventure.text.minimessage.internal.parser.TokenParser.resolvePreProcessTags(TokenParser.java:112) ~[adventure-text-minimessage-4.13.1.jar:?]
	at net.kyori.adventure.text.minimessage.MiniMessageParser.parseToTree(MiniMessageParser.java:195) ~[adventure-text-minimessage-4.13.1.jar:?]
	at net.kyori.adventure.text.minimessage.MiniMessageParser.parseFormat(MiniMessageParser.java:209) ~[adventure-text-minimessage-4.13.1.jar:?]
	at net.kyori.adventure.text.minimessage.MiniMessageImpl.deserialize(MiniMessageImpl.java:79) ~[adventure-text-minimessage-4.13.1.jar:?]
	at net.kyori.adventure.text.minimessage.MiniMessageImpl.deserialize(MiniMessageImpl.java:45) ~[adventure-text-minimessage-4.13.1.jar:?]
	at org.bukkit.craftbukkit.v1_19_R3.entity.CraftPlayer.setPlayerListName(CraftPlayer.java:540) ~[purpur-1.19.4.jar:git-Purpur-1964]
	at net.minecraft.server.level.ServerPlayer.setAfk(ServerPlayer.java:2141) ~[?:?]
	at net.minecraft.server.network.ServerGamePacketListenerImpl.tick(ServerGamePacketListenerImpl.java:466) ~[?:?]
	at net.minecraft.network.Connection.tick(Connection.java:601) ~[?:?]
	at net.minecraft.server.network.ServerConnectionListener.tick(ServerConnectionListener.java:234) ~[?:?]
	at net.minecraft.server.MinecraftServer.tickChildren(MinecraftServer.java:1605) ~[purpur-1.19.4.jar:git-Purpur-1964]
	at net.minecraft.server.dedicated.DedicatedServer.tickChildren(DedicatedServer.java:487) ~[purpur-1.19.4.jar:git-Purpur-1964]
	at net.minecraft.server.MinecraftServer.tickServer(MinecraftServer.java:1424) ~[purpur-1.19.4.jar:git-Purpur-1964]
	at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1195) ~[purpur-1.19.4.jar:git-Purpur-1964]
	at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:325) ~[purpur-1.19.4.jar:git-Purpur-1964]
	at java.lang.Thread.run(Thread.java:833) ~[?:?]
[19:31:02] [Server thread/INFO]: 404_not_found_ lost connection: Internal server error
[19:31:02] [Server thread/INFO]: 404_not_found_ left the game

If you have legacy formatting codes, remove them (for example §7 or &7)

I don't, see spark link for configs

I don't, this is a freshly created server to test this issue
Steps/models to reproduce has a list of the only changes made to the defaults
See spark link for configs

I can confirm this issue:
Player going AFK, tab screen changes to the formatted text

Player leaving AFK, tab screen does not revert to normal text

Player goes AFK again and crashes

Server Logs
WARNING: Using incubator modules: jdk.incubator.vector
Starting org.bukkit.craftbukkit.Main
System Info: Java 17 (OpenJDK 64-Bit Server VM 17.0.7+7-LTS) Host: Linux 5.19.0-41-generic (amd64)
Loading libraries, please wait...
[17:03:04 INFO]: Environment: authHost='https://authserver.mojang.com', accountsHost='https://api.mojang.com', sessionHost='https://sessionserver.mojang.com', servicesHost='https://api.minecraftservices.com', name='PROD'
[17:03:05 INFO]: Loaded 7 recipes
[17:03:05 INFO]: Starting minecraft server version 1.19.4
[17:03:05 INFO]: Loading properties
[17:03:05 INFO]: This server is running Purpur version git-Purpur-1970 (MC: 1.19.4) (Implementing API version 1.19.4-R0.1-SNAPSHOT) (Git: 3325421 on HEAD)
[17:03:05 INFO]: Server Ping Player Sample Count: 12
[17:03:05 INFO]: Using 4 threads for Netty based IO
[17:03:06 WARN]: [!] The timings profiler has been enabled but has been scheduled for removal from Paper in the future.
    We recommend installing the spark profiler as a replacement: https://spark.lucko.me/
    For more information please visit: https://github.com/PaperMC/Paper/issues/8948
[17:03:06 INFO]: [ChunkTaskScheduler] Chunk system is using 1 I/O threads, 3 worker threads, and gen parallelism of 3 threads
[17:03:06 INFO]: [Pufferfish] Max SIMD vector size on this system is 256 bits (int)
[17:03:06 INFO]: [Pufferfish] Max SIMD vector size on this system is 256 bits (float)
[17:03:06 INFO]: [Pufferfish] SIMD operations detected as functional. Will replace some operations with faster versions.
[17:03:06 INFO]: Default game type: SURVIVAL
[17:03:06 INFO]: Generating keypair
[17:03:06 INFO]: Starting Minecraft server on *:25565
[17:03:06 INFO]: Using epoll channel type
[17:03:06 INFO]: Paper: Using libdeflate (Linux x86_64) compression from Velocity.
[17:03:06 INFO]: Paper: Using OpenSSL 3.0.x (Linux x86_64) cipher from Velocity.
[17:03:07 INFO]: [spark] Loading server plugin spark v1.10.38
[17:03:07 INFO]: Server permissions file permissions.yml is empty, ignoring it
[17:03:07 INFO]: Preparing level "world"
[17:03:07 INFO]: Preparing start region for dimension minecraft:overworld
[17:03:07 INFO]: Time elapsed: 68 ms
[17:03:07 INFO]: Preparing start region for dimension minecraft:the_nether
[17:03:07 INFO]: Time elapsed: 26 ms
[17:03:07 INFO]: Preparing start region for dimension minecraft:the_end
[17:03:07 INFO]: Time elapsed: 23 ms
[17:03:07 INFO]: [spark] Enabling spark v1.10.38
[17:03:08 INFO]: [spark] Using Paper ServerTickStartEvent for tick monitoring
[17:03:08 INFO]: [spark] Starting background profiler...
[17:03:08 INFO]: Running delayed init tasks
[17:03:08 INFO]: Done (2.585s)! For help, type "help"
[17:03:14 INFO]: UUID of player DuskOfDarkness is 344f7a7a-beeb-4926-93b5-db81fcfeee1b
[17:03:14 INFO]: DuskOfDarkness joined the game
[17:03:14 INFO]: DuskOfDarkness[/[0:0:0:0:0:0:0:1%0]:51924] logged in with entity id 128 at ([world]12.5, 65.0, -1.5)
[17:05:21 INFO]: DuskOfDarkness is now AFK
[17:05:39 INFO]: DuskOfDarkness is no longer AFK
[17:06:06 INFO]: <DuskOfDarkness> e
[17:08:27 INFO]: DuskOfDarkness is now AFK
[17:08:27 WARN]: Failed to handle packet for /[0:0:0:0:0:0:0:1%0]:51924
net.kyori.adventure.text.minimessage.internal.parser.ParsingExceptionImpl: Legacy formatting codes have been detected in a MiniMessage string - this is unsupported behaviour. Please refer to the Adventure documentation (https://docs.advntr.dev) for more information.
	<gray><italic>[AFK] §7§o[AFK] DuskOfDarkness
	                    ^^
	at net.kyori.adventure.text.minimessage.internal.parser.TokenParser.parseString(TokenParser.java:170) ~[adventure-text-minimessage-4.13.1.jar:?]
	at net.kyori.adventure.text.minimessage.internal.parser.TokenParser.resolvePreProcessTags(TokenParser.java:112) ~[adventure-text-minimessage-4.13.1.jar:?]
	at net.kyori.adventure.text.minimessage.MiniMessageParser.parseToTree(MiniMessageParser.java:195) ~[adventure-text-minimessage-4.13.1.jar:?]
	at net.kyori.adventure.text.minimessage.MiniMessageParser.parseFormat(MiniMessageParser.java:209) ~[adventure-text-minimessage-4.13.1.jar:?]
	at net.kyori.adventure.text.minimessage.MiniMessageImpl.deserialize(MiniMessageImpl.java:79) ~[adventure-text-minimessage-4.13.1.jar:?]
	at net.kyori.adventure.text.minimessage.MiniMessageImpl.deserialize(MiniMessageImpl.java:45) ~[adventure-text-minimessage-4.13.1.jar:?]
	at org.bukkit.craftbukkit.v1_19_R3.entity.CraftPlayer.setPlayerListName(CraftPlayer.java:544) ~[purpur-1.19.4.jar:git-Purpur-1970]
	at net.minecraft.server.level.ServerPlayer.setAfk(ServerPlayer.java:2143) ~[?:?]
	at net.minecraft.server.network.ServerGamePacketListenerImpl.tick(ServerGamePacketListenerImpl.java:467) ~[?:?]
	at net.minecraft.network.Connection.tick(Connection.java:604) ~[?:?]
	at net.minecraft.server.network.ServerConnectionListener.tick(ServerConnectionListener.java:234) ~[?:?]
	at net.minecraft.server.MinecraftServer.tickChildren(MinecraftServer.java:1605) ~[purpur-1.19.4.jar:git-Purpur-1970]
	at net.minecraft.server.dedicated.DedicatedServer.tickChildren(DedicatedServer.java:487) ~[purpur-1.19.4.jar:git-Purpur-1970]
	at net.minecraft.server.MinecraftServer.tickServer(MinecraftServer.java:1424) ~[purpur-1.19.4.jar:git-Purpur-1970]
	at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1195) ~[purpur-1.19.4.jar:git-Purpur-1970]
	at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:325) ~[purpur-1.19.4.jar:git-Purpur-1970]
	at java.lang.Thread.run(Thread.java:833) ~[?:?]
[17:08:27 INFO]: DuskOfDarkness lost connection: Internal server error
[17:08:27 INFO]: DuskOfDarkness left the game
> stop
[17:08:52 INFO]: Stopping the server
[17:08:52 INFO]: Stopping server
[17:08:52 INFO]: [spark] Disabling spark v1.10.38
[17:08:52 INFO]: Saving players
[17:08:52 INFO]: Saving worlds
[17:08:52 INFO]: Saving chunks for level 'ServerLevel[world]'/minecraft:overworld
[17:08:52 INFO]: [ChunkHolderManager] Waiting 60s for chunk system to halt for world 'world'
[17:08:52 INFO]: [ChunkHolderManager] Halted chunk system for world 'world'
[17:08:52 INFO]: [ChunkHolderManager] Saving all chunkholders for world 'world'
[17:08:52 INFO]: [ChunkHolderManager] Saved 1603 block chunks, 241 entity chunks, 3 poi chunks in world 'world' in 0.50s
[17:08:52 INFO]: ThreadedAnvilChunkStorage (world): All chunks are saved
[17:08:52 INFO]: Saving chunks for level 'ServerLevel[world_nether]'/minecraft:the_nether
[17:08:52 INFO]: [ChunkHolderManager] Waiting 60s for chunk system to halt for world 'world_nether'
[17:08:52 INFO]: [ChunkHolderManager] Halted chunk system for world 'world_nether'
[17:08:52 INFO]: [ChunkHolderManager] Saving all chunkholders for world 'world_nether'
[17:08:52 INFO]: [ChunkHolderManager] Saved 6 block chunks, 20 entity chunks, 0 poi chunks in world 'world_nether' in 0.01s
[17:08:52 INFO]: ThreadedAnvilChunkStorage (DIM-1): All chunks are saved
[17:08:52 INFO]: Saving chunks for level 'ServerLevel[world_the_end]'/minecraft:the_end
[17:08:52 INFO]: [ChunkHolderManager] Waiting 60s for chunk system to halt for world 'world_the_end'
[17:08:52 INFO]: [ChunkHolderManager] Halted chunk system for world 'world_the_end'
[17:08:52 INFO]: [ChunkHolderManager] Saving all chunkholders for world 'world_the_end'
[17:08:52 INFO]: [ChunkHolderManager] Saved 10 block chunks, 10 entity chunks, 0 poi chunks in world 'world_the_end' in 0.01s
[17:08:52 INFO]: ThreadedAnvilChunkStorage (DIM1): All chunks are saved
[17:08:52 INFO]: ThreadedAnvilChunkStorage: All dimensions are saved
[17:08:52 INFO]: Flushing Chunk IO
> 2023-05-13 17:08:52,855 Log4j2-AsyncAppenderEventDispatcher-1-Async WARN Advanced terminal features are not available in this environment
[17:08:52 INFO]: Closing Thread Pool
[17:08:52 INFO]: Closing Server

From what I understand, the problem is that MiniMessage parses text to legacy color codes (§), but when a player goes AFK and then comes back, the server does not remove the color codes. This means that when the player goes AFK again, MiniMessage tries to parse the legacy color codes (§).


Defold game engine scripting.

Hi, I just have tried to use TypeScript and the transpiller to script Defold engine (lua scripting originaly) and it works well. Cool! Will investigate further and add comments to this thread.

Nice! If you build up some nice declarations let us know and we can add a link to them in the readme. If you need help with those don't hesitate to ask!

First problem I have faced its no option to specify output file name.
For example, in Defold, I need lua script files with extensions ".script" or ".gui_script" depends on what part of engine we script.
$> tstl input1.ts -o output1.script
$> tstl input2.ts -o output2.gui_script


Can lorri shell pass attributes to the nix-shell?

This is an example shell.nix:
{ pkgs ? import <nixpkgs> {} }:

let
  env = python: pkgs.poetry2nix.mkPoetryEnv {
    python = python;
    pyproject = ./pyproject.toml;
    poetrylock = ./poetry.lock;
  };
in
{
  py38 = env pkgs.python38;
}

I can spawn a Python 3.8 shell with nix-shell -A py38, can the same be achieved with lorri shell? There doesn't seem to be as many options.
$ lorri shell --help
lorri-shell 1.1.1
Graham Christensen<EMAIL_ADDRESS>Open a new project shell

USAGE:
    lorri shell [FLAGS] [OPTIONS]

FLAGS:
        --cached     If true, load environment from cache
    -h, --help       Prints help information
    -V, --version    Prints version information

OPTIONS:
        --shell-file <nix_file>    The .nix file in the current directory to use [default: shell.nix]

We could add an --attr attribute to mirror the one in nix-shell.
On Thu, Aug 6, 2020 at 12:16 AM Marcos Benevides<EMAIL_ADDRESS>wrote:

This is an example shell.nix:
{ pkgs ? import  {} }:
let
makePoetryEnv = python: pkgs.poetry2nix.mkPoetryEnv {
python = python;
pyproject = ./pyproject.toml;
poetrylock = ./poetry.lock;
editablePackageSources = {
app = ./app;
};
};
makeDevEnv = python: pkgs.mkShell {
buildInputs = [
(makePoetryEnv python)
];
PATH = "$PWD/.venv/bin:$PATH";
};
in
{
py38 = makeDevEnv pkgs.python38;
}
I can spawn a Python 3.8 shell with nix-shell -A py38, can the same be
achieved with lorri shell? There doesn't seem to be as many options.
$ lorri shell --help
lorri-shell 1.1.1
Graham Christensen<EMAIL_ADDRESS>Open a new project shell
USAGE:
lorri shell [FLAGS] [OPTIONS]
FLAGS:
--cached     If true, load environment from cache
-h, --help       Prints help information
-V, --version    Prints version information
OPTIONS:
--shell-file <nix_file>    The .nix file in the current directory to use [default: shell.nix]
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/target/lorri/issues/461, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYB5ZSGOW3IWTIRCIWRHLLR7HK3VANCNFSM4PV6ULYQ
.


Unknown column

I was in need to manually override the SQLite version, see https://github.com/simolus3/drift/issues/1857
Since this change, I get this warning:
[WARNING] drift_dev:drift_dev on lib/src/database/database.dart:
line 708, column 20: Unknown column. These columns are available: key, value, type, atom, type, id, parent, fullkey
    ╷
708 │              WHERE position_id = json_extract(j.value, '$.positionid')
    │                    ^^^^^^^^^^^
    ╵

This is the table definition:
CREATE TABLE IF NOT EXISTS contract_has_add_fees
(
   position_id INTEGER NOT NULL,
   position_db INTEGER NOT NULL,
   addfee      TEXT    NOT NULL,
   amount      REAL    NOT NULL,
   changed     DATETIME,
   removed     BOOLEAN NOT NULL DEFAULT FALSE,
   PRIMARY KEY (position_id, position_db, addfee),
   FOREIGN KEY (position_id, position_db) REFERENCES contracts ON DELETE CASCADE
) WITHOUT ROWID;

And this is my query where the warning happens:
deleteAllContractHasAddfees:
DELETE
FROM contract_has_add_fees
WHERE EXISTS(SELECT *
             FROM json_each(:json) AS j
             WHERE position_id = json_extract(j.value, '$.positionid')
               AND position_db = json_extract(j.value, '$.positiondb')
               AND addfee = json_extract(j.value, '$.addfee')
         );

But it is just a warning, everything works.

Thanks for the report, I can reproduce this locally. It seems like the error doesn't happen if you add a table alias:
DELETE
FROM contract_has_add_fees d
WHERE EXISTS(SELECT *
             FROM json_each(:json) AS j
             WHERE d.position_id = json_extract(j.value, '$.positionid')
               AND d.position_db = json_extract(j.value, '$.positiondb')
               AND d.addfee = json_extract(j.value, '$.addfee')

I'll take a look at why these columns aren't available by default.

Quick update: This is turning out to be a lot harder than I expected it to be :D
The problem is that, at the moment, sqlparser assumes that only columns to which a * would expand to can be referenced without an explicit table name. But in the inner select, the * expands to the columns of json_each and you can still refer to outer columns. Fixing this caused all kinds of other problems because the way column lookups are currently implemented in sqlparser. I've started refactoring this, but the implementation is pretty complicated and I need to make sure it doesn't break anything else.

Please no pressure, the workaround is fine for me.

Should be good now! If you want to, you could try adding this to your pubspec and running the build again:
dependency_overrides:
  drift:
    git:
      url: https://github.com/simolus3/drift.git
      ref: develop
      path: drift
  drift_dev:
    git:
      url: https://github.com/simolus3/drift.git
      ref: develop
      path: drift_dev
  sqlparser:
    git:
      url: https://github.com/simolus3/drift.git
      ref: develop
      path: sqlparser

There are lots of internal tests around reference handling but as this issue and others have shown, it's a tricky thing to get exactly right. If anything new breaks (new warnings, changes in generated code) for you with these git overrides, please let me know.


Repo tests are failing on Windows

Some tests fail on py.test tests due to:
TypeError: argument of type 'WindowsPath' is not iterable

To reproduce:

Clone repo
Run pip install -r requirements.txt
Run py.test tests

This is a Python pathlib bug which is only fixed on Python 3.8: https://github.com/python/cpython/pull/7071
However a possible workaround is to change implementation to not iterate through the path.
  A quoted string can be embedded in an
           argument.

        3) A double quotation mark preceded by a backslash is
           interpreted as a literal double quotation mark.

        4) Backslashes are interpreted literally, unless they
           immediately precede a double quotation mark.

        5) If backslashes immediately precede a double quotation mark,
           every pair of backslashes is interpreted as a literal
           backslash.  If the number of backslashes is odd, the last
           backslash escapes the next double quotation mark as
           described in rule 3.
        """

        # See
        # http://msdn.microsoft.com/en-us/library/17w5ykft.aspx
        # or search http://msdn.microsoft.com for
        # "Parsing C++ Command-Line Arguments"
        result = []
        needquote = False
        for arg in seq:
            bs_buf = []

            # Add a space to separate this argument from the others
            if result:
                result.append(' ')

>           needquote = (" " in arg) or ("\t" in arg) or not arg
E           TypeError: argument of type 'WindowsPath' is not iterable

I think the right fix here is to force a string before calling check_output. Can you see if that works? Do the other tests pass on Windows?
Definitely would accept a PR with that tweak:
args = ['python', str(setup_), '--author']

My thoughts exactly. Checked and it's working. I'll submit a PR in the next few days


Import function not working

Hello Team,
I like your resume builder website very much and I  want to highlight the issue that I came across. Few days back I created sample resume and downloaded on my drive. Today, I wanted to edit the resume but import feature is not working. I selected both file types i.e pdf and docx and your website is not uploading my files due to which I can't edit my resume.
It would be great if you fix this issue as I don't want to create new resume again.

Closing this in favor of Reactive Resume v3 release.
If you continue to face this issue, please raise a new one and I'll have a look at it as soon as I can.
In the meantime, if you are able to, please consider donating to keep supporting the project: https://www.buymeacoffee.com/AmruthPillai :)
Thank you!