__webpack_public_path__ is undefined

After working through the docs here, I had trouble getting code splitting working using the React.lazy(() => import('../component')) due to a 404 on the split bundles being generated.  The generated path to the split bundle looked like this http://localhost/existing-slug/undefinedwpack/7-c401d14a.js
After doing some digging, it turned out __webpack_public_path__ was never getting set based on the inserted wpack script (if (__WPACK__) ... file).  After setting the variable by hand, at the entry point, it worked fine from there on.  Example below:
__webpack_public_path__ = window.__wpackIoappNamePublicPath;

import ...

Not sure what would be causing this?

Tagging in on this issue, I seem to have something very similar to this.
I'm developping a theme, and my structure is something like this :
myTheme

src

img

bg.png

img2.png

...

and when I import these in my JS file like so, by using a webpack alias 'Images' (and even without an alias) :
import bgTile from 'Images/bg.png';
this works in dev-server mode (wpackio-scripts start), however it outputs something really weird in build :
undefinedapp/assets/bg-1b7cccae.png
The worst thing about this is that it does this only in my JS.. Using the alias in my SCSS outputs the right URL both in dev and build.
I've tried a number of things, ranging from upgrading my wpackio version all the way to trying out custom images loaders, and nothing gave a working result, it's always the very same weird "undefined".
I develop with Windows and Windows Subsystem for Linux. I can't really try anything else right now, but if OP has the same issue and the same env, it might come from that, I do remember having troubles with gatsby.js with WSL. Any idea?

Hi,
The undefined part is the appName of wpackio.project.js file. Please make sure you have it.
Also for more help, kindly provide a reproducible repo.

@swashata I can try to set up but a repo that reproduces the problem as the issue is on a closed soruce repo currently.  I have confirmed that we have an appName in the project config atm

Hi,
One more thing about this is, wpackio/enqueue from PHP sets the value. Are you sure it's being set? It uses the following hooks
		// wp_head is done before printing any script or style tag
		\add_action( 'wp_head', [ $this, 'printPublicPath' ], -1000 );
		// in case of admin, styles and scripts are printed before admin_head
		// so we use admin_print_scripts
		\add_action( 'admin_print_scripts', [ $this, 'printPublicPath' ], -1000 );

So maybe, it never gets executed.

I have the same problem but it happens only in production. I don't get what's missing....


Refactor add_bounds_axes -> show_bounds

Refactor add_bounds_axes -> show_bounds. This naming convention is more consistent with the rest of the plotting code (e.g. show_grid and show_axes)
This maintains backward compatibility and throws a logging warning for add_bounds_axes

Codecov Report

Merging #141 into master will decrease coverage by 0.02%.
The diff coverage is 78.57%.

@@            Coverage Diff             @@
##           master     #141      +/-   ##
==========================================
- Coverage   89.21%   89.19%   -0.03%
==========================================
  Files          19       19
  Lines        5036     5043       +7
==========================================
+ Hits         4493     4498       +5
- Misses        543      545       +2

Impacted Files
Coverage Δ

vtki/ipy_tools.py
76.47% <0%> (ø)
:arrow_up:

vtki/qt_plotting.py
97.86% <100%> (ø)
:arrow_up:

vtki/plotting.py
81.04% <100%> (+0.05%)
:arrow_up:

vtki/renderer.py
91.6% <60%> (-0.43%)
:arrow_down:

Continue to review full report at Codecov.

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


Fix segfault caused by pop from numba.typed.List

Fixes #5264

May also fix #4960

@stuartarchibald I have addressed all comments except one and added the reproducers. Let me know how to proceed.

@stuartarchibald I have addressed all comments except one and added the reproducers. Let me know how to proceed.

Thanks, have responded to the remaining item.


Do not call kernel driver functionality when running on win32

I've been hit by #147, and I think that a nicer approach would be to just skip calls to __isKernelDriverActive, __detachKernelDriver and __attachKernelDriver when running on a win32 platform.
This PR does that, and makes a brief note in the readme.
(The use case is a JS implementation of USB CDC ACM which, in fact, has to detach the cdc-acm kernel driver)

@ivankennethwang: that's not the same error. LIBUSB_ERROR_NOT_SUPPORTED means an incompatible driver is currently bound to the device. Replace the driver with WinUSB using http://zadig.akeo.ie/.

@kevinmehall  I did try this one out however it failed to upgrade my drivers. Also, I notice that the problem only happens with node-usb being rebuilt on an electron project. If I create a regular project with node-usb, my application can actually connect to the USB device properly. That's why I'm a bit confused whether it is really the drivers or not that is causing my issue.

Any code that needs this is going to be platform-specific anyway, so making it a no-op I think just makes it more confusing.


Remove multiple or all forwarded ports

If a process creates an http server at a random port and I need to forward the port each time, there will be a lot of forwarded ports along the time. I wonder if it makes sense to add a button or command to clear all forwarded ports, or remove multiple selected ports in the ports view?

If you turn on port auto forwarding with the setting remote.autoForwardPorts, and set the port source (remote.autoForwardPortsSource) to process, you should get some better behavior here. The ports will be automatically forwarded and unforwarded when the process that listens on the port start/stops.

I set remote.autoForwardPortsSource to process but it still does not remove the ports when the process stops.
The ports are forwarded by vscode.env.asExternalUri (https://github.com/microsoft/vscode/issues/102449#issuecomment-744405611). When the process is ended, the ports are still there.

Ah. That makes sense then, thanks for the explanation. The manual forwarding through asExternalUri probably happens first (the auto forwarding is on a slight delay), so then they count as manually forwarded and don't get removed.

@joaomoreno, how would you feel about allowing a column label to be more than just a string? It seems like an "x" in the column header would be a nice UX for remove all.

Adding a button next to "Add Port", or adding a menu item to the context menu, or as you suggest, adding an "x" to the column header, all seem to make good sense to me.

Closing a port seems to take a few seconds (~5s) for me. Unsure if that's the case for everyone but if so it might be worth adding some kind of a progress bar otherwise it looks like the whole app is frozen.

Closing a port seems to take a few seconds (~5s) for me

Also takes me ~2s most of the time. I wonder if dropping all forwarded ports would take tens of seconds?

In the meantime - is there a non-UI way of closing all ports?

Does it make sense to have an option not to recover forwarded ports when a remote session is recovered. In my use case, I don't really need those forwarded ports in history.

You can disable port restore with the setting remote.restoreForwardedPorts

Wish I had checked here a few minutes ago as I just manually deleted all the 280 ports I've acquired over time 😄 It seems that you can continue clicking the X for deleting even while the UI is frozen - and the ports get deleted after a bit.
The deletion massively sped up as the port count went down, seems like there may be some excessive work being done when the port count is 100+

@dtasev there was a perf issue with the view, but it should be fixed in insiders. What version of VS Code are you on?

You can disable port restore with the setting remote.restoreForwardedPorts

Thanks! That's exactly what I'm looking for.

@alexr00 I wouldn't have that yet, but great to know it's already been addressed, I stay on the main releases :) From the About:
Version: 1.55.2 (system setup)
Commit: 3c4e3df9e89829dce27b7b5c24508306b151f30d
Date: 2021-04-13T09:35:57.887Z
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19043

@joaomoreno, how would you feel about allowing a column label to be more than just a string? It seems like an "x" in the column header would be a nice UX for remove all.

I don't mind augmenting the column rendering feature, but I'm not so sure that would be a good place for it. Why not enabling multiple selection and pressing Delete?

It's less discoverable, but simple, and Delete is already supported for individual . I'll look into multiselect.

To verify, use the Ports view to:

Forward some ports.
Multi select some ports.
Right click and "Stop Forwarding Port"
Multi select some more ports.
Use the "delete" key to stop forwarding the ports.

I could confirm that the Ports view in the latest insider build work quite well. Thanks!

@renkun-ken thanks for verifying!


Add template email to signature on gmail

[ ] Create html of just the bottom part of the template
[x] Add signature to clothingloop gmail account

Done!
Looks really great btw!

I'll set it as the default signature and close this issue


Customer feedback -

Type of issue
Other (describe below)
Feedback

How do I fix this problem
Page URL
https://learn.microsoft.com/en-gb/windows-hardware/drivers/debugger/bug-check-0x124---whea-uncorrectable-error
Content source URL
https://github.com/MicrosoftDocs/windows-driver-docs/blob/staging/windows-driver-docs-pr/debugger/bug-check-0x124---whea-uncorrectable-error.md
Author
@DOMARS

Hi @chigozyie, while this documentation is for commercial driver developers, there are a couple potential causes for that error in the Cause section of the doc page.
For consumer driver issues, your best bet is to look at the Microsoft Help topics that are on support.microsoft.com.  You might start with https://support.microsoft.com/en-US/windows/resolving-blue-screen-errors-in-windows-60b01860-58f2-be66-7516-5c45a66ae3c6.
Best of luck with your issue.


Documentation improvements

typos and presentation in fht Readme.md
improvements in the doc comments for User and UserAgentApplication

@jmprieur,
Thanks for your contribution.  To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
The agreement will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@jmprieur, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

Most feedback are merged. Will send another PR for missing bits


TableSet.from_json

I'm working on implementing this but I'm running into an issue. I'm mimicking the TableSet.from_csv function and tests but my implementation hits an error with column_types.
The current Table.from_json implementation does not take column_types so I just called the function with the path as an argument but I get ValueError: Not all tables have the same column types! on initializing the TableSet:
The TableSet.from_csv test passes column_types as an argument even though it is not required. When I tried the test without passing column_types, I got the same error.
When I print the column_types for each table I get this:
(<agate.data_types.text.Text object at 0x10bc21e90>, <agate.data_types.number.Number object at 0x10bc21c10>)
(<agate.data_types.text.Text object at 0x10bc26150>, <agate.data_types.number.Number object at 0x10bc21c90>)
(<agate.data_types.text.Text object at 0x10bc263d0>, <agate.data_types.number.Number object at 0x10bc21d10>)

The check for column_type equality returns false since the addresses are different. Does an existing implementation need to be adjusted or am I missing something? Thanks!


[Bug]: MySQL IS NULL Query breaks front-end in local and behaves unexpected in release

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I run a MySQL query with IS NULL syntax turning off the prepared statement then the following unexpected behaviors are noticed in the local environment (release branch) and in the release.app.appsmith.com as well.
Shows Oops both in local and release.app.appsmith.com

Broken Front-end only in local (release branch)

Expected Behaviour found in app.appsmith.com

Steps To Reproduce

Create a MySQL datasource
Write a query containing IS NULL syntax where the query is not supposed to return any row because of no matching rows found in the table
You should see a page saying Oops! Something went wrong and later you will see some front-end source code that basically breaks the entire front-end (Happens to exist in the local environment on the release branch but Oops page appears in release.app.appsmith.com too)'

Demonstration
Loom Video
Public Sample App
No response
Version
Cloud / Self Hosted - v1.7.1

Happens on Postgres DS too.

https://user-images.githubusercontent.com/91450662/173360718-9b7e090a-e834-4c14-9c77-dc17da52e864.mov

Originated from PR 14274. Adding extra check.


Enable CIFS / NFS Mounts

The iobroker module backitup https://github.com/simatec/ioBroker.backitup/blob/master/docs/de/backitup.md needs some extra packages
       nfs-common \
        cifs-utils \

to enable backups on dynamical mounted volumes.
Please add this packages .. I've tested the localy and it works with --privileged flag.
Thx

You are able to install the packages witch the "PACKAGES" environment variable. They should not be added to the image itself. And by the way, don't use the privileged flag, it's a big security risk.

As backitup is not part of a default iobroker installation I see no reason to add the packages to the Docker image.
For additional packages needed from adapters please use ENV "PACKAGES" as described in readme.md.
BTW, what do you mean with "I've tested the localy and it works with --privileged flag"?
Regards,
André

Uh oh, sorry, yes "RTFM" thanks for that hint !
"locally" means I have build a new docker-image on my local pc with this packages and used this to test it with a mount from my QNAP
Several tries with the capability  SYS_ADMIN gave me a "read-only" mount of the NFS-Mount, only --privileged works correct.
I would gladly prevent --privileged if someone could tell me the correct capabilities  to do a NFS-Mount out of the container.
The NFS volume is set up with "any IP allowed / NO_ROOT_SQUASH"

Ah, now I see the problem with the mount inside the container. I think it would be better to mount the nfs share outside the container to prevent the privileged option. Because I don't use a NFS-Mount I'm not sure if that works but I found two Howto manuals. I will setup a NFS-Share on my NAS and test it this week:
Mounting a NFS Directory into Host Volume
Docker (Swarm) and NFS volumes

Yes, I know how to mount a volume inside the container.
The iobroke-Module "backitup" ist the problem, a backup-prozess inside the container does the following steps

mount NFS Volume
create a tar on this Volume
unmount the Volume


HDDS-11873. Skip old-only xcompat read tests

What changes were proposed in this pull request?
Cluster/client/data combinations where all versions are "old" can be skipped, since no new code is exercised:
xcompat-cluster-1.1.0-client-1.1.0-read-1.1.0 :: Read Compatibility
xcompat-cluster-1.2.1-client-1.2.1-read-1.2.1 :: Read Compatibility
...

https://issues.apache.org/jira/browse/HDDS-11873
How was this patch tested?
CI:
https://github.com/adoroszlai/ozone/actions/runs/12199003296/job/34034385847

Thanks @dombizita for the review.


Cannot install on RHEL 7

I'm running RHEL 7 and I have the ROCm platform installed.
I'm following the RPM Install instructions but I get the following errors from failed dependencies:
libclang.so.8_HCC2_Version_0.5-3()(64bit) is needed by hcc2-0.5-3.x86_64         libclang.so.8_HCC2_Version_0.5-3(LLVM_8)(64bit) is needed by hcc2-0.5-3.x86_64         libstdc++.so.6(GLIBCXX_3.4.20)(64bit) is needed by hcc2-0.5-3.x86_64         libstdc++.so.6(GLIBCXX_3.4.21)(64bit) is needed by hcc2-0.5-3.x86_64
If I follow the No root Install instructions I get same errors. Any suggestion on how to complete the installation for HCC2?

I also had an issue with the RPM in CentOS. I had to unpack the rpm manually, and that let me used hcc2 (after setting up the LD_LIBRARY_PATH and the PATH env variables).
I used the following:
rpm2cpio hcc2-0.5-3.x86_64.rpm | cpio -i --make-directories
sudo cp -r opt/rocm/ /opt/rocm
export LD_LIBRARY_PATH=/opt/rocm/hcc2/lib/:$LD_LIBRARY_PATH
export PATH=/opt/rocm/hcc2/bin:$PATH
export HCC2=/opt/rocm/hcc2

rpm2cpio hcc2-0.5-3.x86_64.rpm | cpio -i --make-directories
sudo cp -r opt/rocm/ /opt/rocm
export LD_LIBRARY_PATH=/opt/rocm/hcc2/lib/:$LD_LIBRARY_PATH
export PATH=/opt/rocm/hcc2/bin:$PATH
export HCC2=/opt/rocm/hcc2

Thank, the above is useful, but I still need newer version of libstdc++ (GLIBCXX_3.4.20 and GLIBCXX_3.4.21 are required). This should also be loaded in the LD_LIBRARY_PATH. Where did you get libstdc++ from?

I actually had GCC 8.1 and Clang 7.0.0 in my system before. I am not sure if they are both needed. I also have that in the LD_LIBRARY_PATH but I was not sure if that is required, now we know that GCC is.
If you are familiarized with lua modules. Here is my lua conf file. I changed /opt/rocm to  /software/apps/rocm/
whatis("Description: hcc2")
load("cuda/9.2")

prepend_path('PATH','/software/apps/gcc/8.1/bin')
prepend_path('LD_LIBRARY_PATH','/software/apps/rocm/hcc2/lib')
prepend_path('PATH','/software/apps/rocm/hcc2/bin')
prepend_path('C_INCLUDE_PATH','/software/apps/clang/7.0.0/include:/software/apps/clang/7.0.0/include/clang/7.0.0/include')
prepend_path('CPLUS_INCLUDE_PATH','/software/apps/clang/7.0.0/include:/software/apps/clang/7.0.0/lib/clang/7.0.0/include')
prepend_path('LD_LIBRARY_PATH','/software/apps/gcc/8.1/lib')
prepend_path('LD_LIBRARY_PATH','/software/apps/gcc/8.1/lib64')

pushenv("HCC2","/software/apps/rocm/hcc2")

family('compilers')

If you are familiarized with lua modules.

well, I haven't really looked at lua before. I guess I could do this in batch (?). Basically I have to modify, PATH, LD_LIBRARY_PATH, etc...?

It works! Instead of lua I used bash and I can compile the examples. thank you!


[Angular] Bug(application): java.lang.NullPointerException not caught by uncaughtErrorEvent

From @CoreyCole on August 24, 2018 23:33

Did you verify this is a real problem by searching the NativeScript Forum and the other open issues in this repo?
Yes
Tell us about the problem
I've been trying to setup crash reporting in my application where if something goes wrong for the user, I send a request to my server with information about the error/crash. I was having trouble and I found #2309 to follow as an example. But, in the current version of nativescript it doesn't seem like uncaughtErrorEvent is called anymore in the scenario @NickIliev presented in that issue.
Which platform(s) does your issue occur on?
Android
Please provide the following version numbers that your issue occurs with:
Running in playground
Please tell us how to recreate the issue in as much detail as possible.
https://play.nativescript.org/?template=play-ng&id=0QxXQK
Is there code involved? If so, please share the minimal amount of code needed to recreate the problem.
I have a button
<Button text="Log In" (tap)="login()"></Button>

Where the login function is defined in the component:
declare var android: any;
// ...
export class AuthPageComponent implements OnInit {
// ...
login() {
    console.log('attempting to throw error');
    var btn = new android.widget.Button(null);
    throw new Error('test');
}

And in my main app.component file I am registering the uncaughtErrorEvent:
import { on as applicationOn, uncaughtErrorEvent, UnhandledErrorEventData } from 'application';
// ...
export class AppComponent {
    constructor(
        private http: HttpClient,
        private api: ApiService
    ) {
        applicationOn(uncaughtErrorEvent, (errData: UnhandledErrorEventData) => {
            console.log('caught error', errData);
        });
    }
}

The app doesn't crash. Has something changed since #2309 was discussed? Does uncaughtErrorEvent only get called when the application crashes?
In my console, I never see caught error printed. I see
ERROR Error: java.lang.NullPointerException: Attempt to invoke virtual method 'android.content.res.Resources android.content.Context.getResources()' on a null object reference
    android.view.ViewConfiguration.get(ViewConfiguration.java:432)
    android.view.View.<init>(View.java:4573)
    android.view.View.<init>(View.java:4706)
    android.widget.TextView.<init>(TextView.java:826)
    android.widget.Button.<init>(Button.java:166)
    android.widget.Button.<init>(Button.java:141)
    android.widget.Button.<init>(Button.java:117)
    android.widget.Button.<init>(Button.java:96)
    com.tns.Runtime.callJSMethodNative(Native Method)
    com.tns.Runtime.dispatchCallJSMethodNative(Runtime.java:1116)
    com.tns.Runtime.callJSMethodImpl(Runtime.java:996)
    com.tns.Runtime.callJSMethod(Runtime.java:983)
    com.tns.Runtime.callJSMethod(Runtime.java:967)
    com.tns.Runtime.callJSMethod(Runtime.java:959)
    com.tns.gen.java.lang.Object_button_19_32_ClickListenerImpl.onClick(Object_button_19_32_ClickListenerImpl.java:17)
    android.view.View.performClick(View.java:6294)
    android.view.View$PerformClick.run(View.java:24770)
    android.os.Handler.handleCallback(Handler.java:790)
    android.os.Handler.dispatchMessage(Handler.java:99)
    android.os.Looper.loop(Looper.java:164)
    android.app.ActivityThread.main(ActivityThread.java:6494)
    java.lang.reflect.Method.invoke(Native Method)
    com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438)
    com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)

Copied from original issue: NativeScript/NativeScript#6217

@CoreyCole I see that you are using aAngular based application and the thing is that the error is caught by the Angular frame so it is not going in the application uncaughtErrorEvent handler. The very same would still work as expected in NativeScript Core (using plain JS or TypeScript)
Marking this one as a bug as the behavior is different in nativeScript Core and in NativeScript Angular applications.

From @CoreyCole on August 28, 2018 22:42
Thank you @NickIliev, getting one place in our application where we can learn of any user experience errors and send to our backend for logging is very valuable to us.

On second though the errors in an Angular application could and should be handled by the Angular - here is a good blog post on how to handle errors in an Angular based application using ErrorHandler

How can we log the stacktrace in an angular application, like you've shown with Nativescript Core in this issue. https://github.com/NativeScript/NativeScript/issues/2309#issuecomment-226134131


removed @Override from prefersHomeIndicatorAutoHidden

I got a compilation error when making a local maven package. The error was
[ERROR] bootstrap class path not set in conjunction with -source 1.6
[ERROR] libgdx/backends/gdx-backend-moe/src/com/badlogic/gdx/backends/iosmoe/IOSUIViewController.java:[111,1] error: method does not override or implement a method from a supertype

The problem is sue to https://github.com/libgdx/libgdx/pull/5413, that method is apparently not bound on Moe

It seems that your MOE library is outdated.
When I build a new project with the gdx-setup.jar the method is actually there:
@Generated 	@Selector("prefersHomeIndicatorAutoHidden") 	public native boolean prefersHomeIndicatorAutoHidden();
The last commit of this class was on Sep 27, 2017:
https://github.com/multi-os-engine/moe-core/blob/moe-master/moe.apple/moe.platform.ios/src/main/java/apple/uikit/UIViewController.java

I ran ./gradlew fetchNatives and then did mvn install and that's where I saw this compilation error. Is there another step necessary in order to build the local maven package?

abandoning in favor of https://github.com/libgdx/libgdx/pull/5416


Restored Windows after relaunch are not fitting display

Issue Type: Bug
Multi-display setup, all bottom-aligned.
display 1, left, built-in laptop
display 2, middle, 27"
display 3, right, 27"
Since forever I have the issue that windows on display 2 and 3 are getting restored in display 1 when I restart VS Code Insiders. This does not happen on Stable... When restoring, the window size was adapted to fit the window on display 1.
Now, windows are still restored on display 1 but only their height gets adapter to fit display 1. They retain the width they had on display 2 / 3. This is particularly unpleasant when switching from multi-display setup to laptop only.
VS Code version: Code - Insiders 1.37.0-insider (9296aaa2201b638e2b1dc71a9b9fe38c5ab2e904, 2019-07-09T05:24:16.868Z)
OS version: Darwin x64 18.6.0
This also happens when all extensions are disabled.

Since forever I have the issue that windows on display 2 and 3 are getting restored in display 1 when I restart VS Code Insiders. This does not happen on Stable...

I am not sure I understand that. Are you referring to our current stable or the previous one? Because both stable and insiders are now on Electron 4 and there have been issues around restoring window bounds (see https://github.com/microsoft/vscode/issues/74872).

Now, windows are still restored on display 1 but only their height gets adapter to fit display 1. They retain the width they had on display 2 / 3. This is particularly unpleasant when switching from multi-display setup to laptop only.

This is probably a result of the OS trying to fit the window on display 1 when it has the bounds of the other displays that are probably larger and cannot fit. It would still be interesting why we fail to restore the window on the other monitors.
@kieferrm I am assigning you, since you could help out given you have this setup (and I don't). Throw some code around here: restoreWindowState

I cannot reproduce. I just added a third monitor to my setup and opened Code (insiders) on each monitor with the full bounds of the display and then restarted. In all cases, the window size was restored to the size of the display.
In order to debug this further I would need the exact display resolution of each monitor and would probably also need to know what the primary monitor is (e.g. on which one do you see the dock).


Appointment table is not visible

The query returns 400 Bad Request.

Added missing harmony flag for node.


[BUG] Azure AppConfiguration: Configuration is not updated dynamically with Spring Boot 3

Describe the bug
Following this tutorial https://learn.microsoft.com/en-us/azure/azure-app-configuration/enable-dynamic-configuration-java-spring-app, our Spring Boot App (version 2.7.6) was refreshing the configuration context for Azure AppConfiguration regularly. After the upgrade to Spring Boot 3.0.5, the values of the feature toggles in the cloud are only fetched during start up of the server, but there is no regular polling for updates anymore.
Exception or Stack Trace
There is no logging of the app configuration at all after the initial "Located property source: [BootstrapPropertySource ..."
To Reproduce
Following this tutorial https://learn.microsoft.com/en-us/azure/azure-app-configuration/enable-dynamic-configuration-java-spring-app with a Spring Boot version 3.0.5.
Expected behavior
The application should refresh all feature toggles by polling regularly for changes from the cloud.
Setup (please complete the following information):

Library/Libraries:

org.springframework.boot:spring-boot-starter-parent:3.0.5
com.azure.spring:azure-spring-cloud-appconfiguration-config-web:2.11.0
com.azure.spring:azure-spring-cloud-feature-management:2.10.0

Java version: 17

Hi, @MaxKiesel
Thanks for reaching out.
Now app configuration related artifacts not supported to be used in Spring Boot 3.
Here is related issue: https://github.com/Azure/azure-sdk-for-java/issues/33758

Hi, @mrm9084 .
Please confirm that this problem has been fixed before release 5.1.0.

@MaxKiesel, 5.1.0 has been released. It does have a few breaking changes if you are moving from a 2.x version, not from the 4.x version. If you are updating from 2.x you can reference https://learn.microsoft.com/en-us/azure/azure-app-configuration/howto-convert-to-the-new-spring-boot

Hello everybody,
I upgraded to version 5.1.0 and it is working again.
Thank you very much!


[Java] Update Maven project details

Describe the enhancement requested
Follow up of the conversation at #43191
Information generated and published to Apache Maven repository/Maven Central is either missing or incorrect:

Modules under maven/ and bom/ directories do not have project url, mailing list, scm and issue management links
Children modules have incorrect project and scm urls as Maven appends by default the name of the name of the artifact to the url from the parent module
${project.scm.tag} hasn't been updated and is still apache-arrow-2.0.0

Component(s)
Java

Issue resolved by pull request 43231
https://github.com/apache/arrow/pull/43231


[TN] - SVG contains solid white background

Describe the bug
It appears that there is a solid white background on the tail-tn.svg file.

Please remove the background for non-white backgrounds.

@leeejune please review this SVG.

tail-TN.svg.zip


Profile_card - Ashwin Telmore

@ashwintelmore Thank you for your valuable contribution. As I found, they(Hacktoberfest) are verifying the code in the PR so I recommend that you add your code to a "code folder" to avoid any consequences (your code should be in a folder named after you). Sincerely apologize for the inconvenience.

Hey @DiyaVj  thanks for your response i have added the code part in repo.
actually i am beginner, so do i need to create another pull request


add application deadline

Can you make the "Applications close March 15" text bigger? It's almost impossible to see on the photo background.

Updated.


Replace arrays of inputs and outputs with named tuples

(LOW PRIORITY)
Currently the input values are organised in a large array that is position-sensitive and therefore vulnerable to human errors. Similarly, outputs are selected by explicitly calling and returning specific parameters from each function, which is a bit unwieldy. It would be better to use a named tuple to organise both the inputs to snicar and the outputs, so we just pass "inputs" and just return "outputs" and inside each are all the necessary params. This would greatly improve the flow of data through the various model components and also enable us to return all possible output variables from the model neatly (currently only a subset are used).
For examples, see the FlowModel repository or the new LUT generator scripts in this repository (development branch)

Resolved via commit "namedtuple" ab60ce5


Add arm64 third-party CI

System information

TensorFlow version (you are using): 2.0+ and master branch
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state.
Currently, Tensorflow only has the official build CI on X86 and third-party build CIs on x86 and ppc64. There is no CI for arm64. Adding a arm64 CI can help community to discover arm64 problems easily.
OpenLab supports public CI system for opensource projects[1]. Now it supports arm64 arch and the tensorflow nightly build jobs for 2.0+ and master version have been added there as well[2]. It runs tensorflow build everyday at UTC-18.
Just like what tensorflow do currently, we can just easily add a new badge in the README.md file to link the Openlab arm64 third-party CI.
As you can see in the page[2], tensorflow 2.0, 2.1 and 2.2(master) build well on aarch64. But in some aws libs strongly based on x86 ARCH, so in master branch, I skip that part for build. You can see the build brief in [3], download the build whl packages there, and see the details logs in [4].
So adding the arm64 build CI is useful for the community.
1: https://openlabtesting.org
2: http://status.openlabtesting.org/builds?project=tensorflow%2Ftensorflow
3: http://status.openlabtesting.org/build/c816e5c9d6cc4519b933414fc6044d28
4: https://logs.openlabtesting.org/logs/periodic-18/github.com/tensorflow/tensorflow/master/tensorflow-arm64-build-daily-v2.1.0/c816e5c/
Additional context
Now the test is CPU only and is basing on Ubuntu 18.04 and python3.6. More can be added in the future.
And I'm from OpenLab commuinty. I'll keep looking after the tensorflow arm64 CI and try my best to fix the arm64 failure then.
Here is an example[1] we done in pytorch community. See 'Linux (aarch64) CPU' badge in the README.md.
Also for another community[2], we done in greenplum-db community. See 'Zuul Regression Test On Arm' badge in the README.md
1: https://github.com/pytorch/pytorch/blob/master/README.md
2: https://github.com/greenplum-db/gpdb
Will this change the current api? How?
No
Who will benefit with this feature?
The arm64 users and developers

Hi team, any idea about this? @terryheo Hi, we are welcome any advice/feedback from you guys. Thanks very much

Hi @bzhaoopenstack, would it be possible to open a separate issue for `AWS lib build for AArch64'. It is easier to discuss in an issue than via emails or other mechanisms.

@ymodak , could you pass this to TF infra team?

@all @AshokBhat
Hi,
sorry for late.
Test env:
Ubuntu 1804, 8vCpus/16G memory on aarch64 platform
Tests use the tensorflow master branch.
I tested with build option like below:
bazel build --config=opt //tensorflow/tools/pip_package:build_pip_package --local_ram_resources=13240 --local_cpu_resources=6 --verbose_failures
And I got the first error is
ERROR: /home/zuul/tensorflow/tensorflow/python/BUILD:3018:1: Linking of rule '//tensorflow/python:gen_list_ops_py_wrappers_cc' failed (Exit 1): gcc failed: error executing command
  (cd /home/zuul/.cache/bazel/_bazel_zuul/ce9de8ddb6d970afbff1ea6d450e3cd7/execroot/org_tensorflow && \
  exec env - \
    PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin \
    PWD=/proc/self/cwd \
  /usr/bin/gcc @bazel-out/aarch64-opt-exec-50AE0418/bin/tensorflow/python/gen_list_ops_py_wrappers_cc-2.params)
Execution platform: @local_execution_config_platform//:platform
bazel-out/aarch64-opt-exec-50AE0418/bin/_solib_aarch64/_U_S_Stensorflow_Spython_Cgen_Ulist_Uops_Upy_Uwrappers_Ucc___Utensorflow/libtensorflow_framework.so.2: error: undefined reference to 'aws_checksums_crc32c_hw'
bazel-out/aarch64-opt-exec-50AE0418/bin/_solib_aarch64/_U_S_Stensorflow_Spython_Cgen_Ulist_Uops_Upy_Uwrappers_Ucc___Utensorflow/libtensorflow_framework.so.2: error: undefined reference to 'aws_checksums_do_cpu_id'
collect2: error: ld returned 1 exit status
Target //tensorflow/tools/pip_package:build_pip_package failed to build
INFO: Elapsed time: 642.843s, Critical Path: 77.35s
INFO: 2161 processes: 2161 local.
FAILED: Build did NOT complete successfully

This error from https://github.com/awslabs/aws-checksums .
Will update in the followting tests

@av8ramit Hi, any idea about this? Thanks

Sorry I'm not sure, I have not seen this error before. @mihaimaruseac any chance you have?

It looks like the arm source files are missing in the glob here. Adding  source/arm/*.c to the following file should do the trick, but I haven't tested it.
Also there is a PR to add HW support for Arm to aws-checksums: https://github.com/awslabs/aws-checksums/pull/29

I haven't seen it before but it looks to be caused by having all the filesystems included in the framework. tensorflow/community#101 aims to remove filesystems, reducing pip size and the need to compile platforms that are not needed

@AGSaidi @mihaimaruseac Cool. Thanks for your kind advice.
That part already pass. But I get the following error.

FAILED: Build did NOT complete successfully
'''
Also @av8ramit , hi, could you guys consider add a 3rd party ARM CI for tensorflow community like this issue description mentioned above? Thank you

For above error, I use the build cmd is:
'''
bazel build --config=opt //tensorflow/tools/pip_package:build_pip_package --local_ram_resources=14240 --local_cpu_resources=7 --verbose_failures
'''
Which is the same with other builds(2.0, 2.1 tensorflow whl build jobs)

Looks the same issue like https://github.com/tensorflow/tensorflow/issues/40688 , will test on it and let you know the result later. Thanks

After merge https://github.com/tensorflow/tensorflow/pull/40700 in my local repo, and downgrade numpy via pip3 install numpy==1.18.0.  Now I can success build the whl package on master branch using ```
bazel clean --expunge ; bazel build --config=opt //tensorflow/tools/pip_package:build_pip_package --local_ram_resources=10240 --local_cpu_resources=7 --verbose_failures

That's good news. Seem the issue is solved, right? Can we close if that is the case?

That's good news. Seem the issue is solved, right? Can we close if that is the case?
The original request was to add Arm64 third party CI, which is still unresolved.

@mihaimaruseac @av8ramit Yeah, the fix is only about the master branch build, the issue is about to add a arm64 3rd party CI.. I think you guys already consider it. Please give some feedback. Any advice is welcome..Thanks

Are you a member of SIG Build? This might be a better conversation to have there. I vaguely remember some discussions about ARM CI, although I myself do not have plans to implement it.

I agree with Amit here. The ARM CI should be via SIG Build, as a community maintained CI

@mihaimaruseac @av8ramit Hi, Thanks. I already raised a ARM CI discussion in SIG Build for a long time. As I think this one looks like a aarch64(ARM) basic CI to add into tensorflow community as a 3rd-party CI. Also the discussion in SIG Build is a totally new build CI system, they can be done in parallel. And for pytorch https://github.com/pytorch/pytorch/blob/master/README.md , my colleage pushed the same into the community as a 3rd-party ARM CI, and it successed. So, do you guys raise this in your team? Because I think the basic ARM CI is necessary and valuable thing for the AI area.
And all things about ARM CI will be maintaned by us, including the code change ,package building, arm issues fix and human resources. The only thing is add a link or badge into tensorflow README.MD to show to the customers and developers, and the way to show is based on community's feedback. How about to consider it? Thanks
We're happy to discuss more.

@bzhaoopenstack As far as I understand, community-supported build status is dealt with SIG BUILD team. The whole process is documented in https://github.com/tensorflow/community/blob/master/sigs/build/community-builds.md page. As the first step of the process, it says that a tracking bug needs to be raised in GitHub. My guess is that this should be raised in https://github.com/tensorflow/build repo rather than TensorFlow/TensorFlow repo.

@AshokBhat Thanks for your suggest. Let's raise this there.

sure


OHBM abstract | Deadline Dec 19

As suggested by @ChristophePhillips in https://github.com/bids-standard/bep001/issues/75#issuecomment-549262868 we should submit an abstract to OHBM 2020.
@Gilles86 has said that he can make a shortened version of the doc - stay tuned here for a version of the ISMRM abstract ✨

Isn't the deadline for OHBM abstracts on December 19? I just read the title of the issue and had a mini-panic :)

If you're ready for the 1st, you will for the 19th... ;-)
I corrected the date though.

I made a frist draft. Looking for feedback!
https://docs.google.com/document/d/1Zwg_dpwkk9YJzHU3wAV8jPvCZ9MUMCtgepee4ATx4hE/edit?usp=sharing


Backup Queue maximum

Is there a maximum value for the backup queue as described below?
"BackupConcurrencyResourceType: This wait indicates that a database is being backed up. The maximum value for this resource type is 1. If multiple backups have been requested at the same time, the others queue."
Thanks!

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

ID: 0c0278ab-46f2-4b79-1f34-7ff150a7d5ef
Version Independent ID: 744751b0-1d7a-cb59-556b-a49bf881458a
Content: Analyze your workload - Azure SQL Data Warehouse
Content Source: articles/sql-data-warehouse/analyze-your-workload.md
Service: sql-data-warehouse
GitHub Login: @kevinvngo
Microsoft Alias: kevin

Thanks for the question! We are investigating and will update you shortly.

@kevinvngo mentioned he would document the recommended spacing of 10 minutes between snapshots to avoid internal timeouts. It sounds like this is more likely to be the issue if trying to run many backups in quick succession (rather than the cap on the number of jobs in the queue).

@dcbrown16 I have assigned this to the content owner (@kevinvngo) to make an update. Thank you!

Update is made here: https://github.com/MicrosoftDocs/azure-docs-pr/pull/70332

#please-close


Gh 776 map store

Codecov Report

Merging #777 into develop will increase coverage by 0.63%.
The diff coverage is 85.32%.

@@              Coverage Diff              @@
##             develop     #777      +/-   ##
=============================================
+ Coverage      59.82%   60.45%   +0.63%
- Complexity      2805     2910     +105
=============================================
  Files            433      440       +7
  Lines          12874    13201     +327
  Branches        1438     1483      +45
=============================================
+ Hits            7702     7981     +279
- Misses          4572     4605      +33
- Partials         600      615      +15

Impacted Files
Coverage Δ
Complexity Δ

...andler/GetAdjacentEntitySeedsOperationHandler.java
100% <100%> (ø)
3 <3> (?)

...mapstore/operation/handler/AddElementsHandler.java
100% <100%> (ø)
3 <3> (?)

...store/operation/handler/GetAllElementsHandler.java
100% <100%> (ø)
3 <3> (?)

...operation/handler/GetElementsOperationHandler.java
100% <100%> (ø)
3 <3> (?)

...n/java/uk/gov/gchq/gaffer/mapstore/utils/Pair.java
27.27% <27.27%> (ø)
3 <3> (?)

...k/gov/gchq/gaffer/mapstore/MapStoreProperties.java
62.5% <62.5%> (ø)
6 <6> (?)

...ain/java/uk/gov/gchq/gaffer/mapstore/MapStore.java
90.54% <90.54%> (ø)
84 <84> (?)

... and 4 more

Continue to review full report at Codecov.

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

Closing whilst working on feedback.


Create config.json

Description:
This mainly fixes the log spam that says it's keeping the best of 3, but never recycles. So, I added in some code, and bam, fixed it! It recycles everything but two of each by default.

Fixes:

Recycling

New Config:
http://pastebin.com/UStttHL2

You should improve the existing config examples if you think you have something to add. not create a new config file.

@leadboots5 Sorry, I'm new to PRs. I knew after I submitted it I did it wrong so I just uploaded the pastbin. I was hoping that the devs could just integrate the "release" portion since that's what I was trying to fix in the first place.


npm package

I see that a huge refactoring was merged but the package version has not changed on npm. @mariolamacchia ?

Just released v0.9.0


[concurrency] update test missed in #34640, resolving rdar://71212904

Resolves  rdar://71212904.

@swift-ci please smoke test and merge

The Linux smoke test failed with:
FAIL: Swift-Unit :: runtime/./SwiftRuntimeTests/ConcurrentReadableArrayTest.MultiThreaded2 (608 of 13768)
******************** TEST 'Swift-Unit :: runtime/./SwiftRuntimeTests/ConcurrentReadableArrayTest.MultiThreaded2' FAILED ********************
Note: Google Test filter = ConcurrentReadableArrayTest.MultiThreaded2
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from ConcurrentReadableArrayTest
[ RUN      ] ConcurrentReadableArrayTest.MultiThreaded2
/home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test/branch-main/swift/unittests/runtime/Concurrent.cpp:134: Failure
Expected: (element.threadNumber) < (writerCount), actual: 528 vs 16
[  FAILED  ] ConcurrentReadableArrayTest.MultiThreaded2 (2084 ms)
[----------] 1 test from ConcurrentReadableArrayTest (2084 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (2084 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] ConcurrentReadableArrayTest.MultiThreaded2

 1 FAILED TEST

This is either a spurious test failure or a pre-existing condition of main.

@swift-ci please smoke test and merge


Error message for priviledge escalation

Provided a meaningful error message to the users when they try to invite a admin, but they themselves are a manager.

Looks good, works well. Merge at will!


[Feature request] Add multilanguage support

Please add the framework for translating the module to other languages (as other modules do, with an en.json file in a lang folder with keys that we can translate).
Thanks and great work!

just added it 🤣

Thank you, added PR for es.json


ci: if the "pnpm install" command fails, retry this command 10 times

In Node.js 14.0.0, the pnpm install command may fail.
Therefore, if it fails, retry 10 times every 1 second.

This is not a Node.js 14.0.0 issue. On my computer, the installation succeeds.


Hacktoberfest - 2022

Please add my file in your repository with label hacktoberfest .

You can follow me on GitHub


i18n(zh-cn): translate /guides/cms/ghost.mdx

What kind of changes does this PR include?

Translated content

Description

translate /guides/cms/ghost.mdx

cc @liruifengv

Thanks for the review @liruifengv


Restructure runtime to support tree shaking

The primary change to support tree shaking is separating submodules into new files. JS tree shaking operates on functions exported by each file, so including submodules in the same file means they're always bundled. This does not require changing atdgen at all, which is nice!
I tried to keep my changes to separate commits so they're easier to review than the overall diff.
I also made some minor improvements:

Upgraded from bs-platform to rescript and removed the peer dependency - it should still work with bs-platform so not having a peer dependency is best imo.
ReScript no longer allows int64 to be redefined so that had to be renamed (only affects the test)
ReScript 10 is removing Printf. The decoder wasn't making advanced use of it, though, so replacing it was easy and saves a lot of bundle size
The behaviour of Decode.fieldOptional was incorrect; I thought this was an atdgen bug but it turns out null fields should be treated as missing by default. So I've fixed that here.

I've pulled out the bugfix commit and will make a new PR with it after this has been merged.

I chose not to revert the rescript change, as I've tried to explain it should make no difference to anyone depending on this library, but I can still do that if you really want to.

The only thing remaining would be to upgrade minor version in package.json

I thought that would happen during the publish process, but sure I can do it now!

I can't comment on the .ocamlformat file because it is empty...
Anyway, I don't think that an empty .ocamlformat is a good practice. It should at least contain a version=XXX line so that there is no reformatting with a wrong version by accident.

I can't comment on the .ocamlformat file because it is empty...
Anyway, I don't think that an empty .ocamlformat is a good practice. It should at least contain a version=XXX line so that there is no reformatting with a wrong version by accident.

That’s fair. I’m not familiar with ocamlformat, I just know that it doesn’t work without a file there.

Thanks!


Fails on Windows

I initially used version 0.1.0 and everything was fine, but after upgrading to higher versions, on Windows the module fails with a message like
test/tap/020-checker.js ............................. 34/35
  outdated version untuned
  not ok ENOENT: no such file or directory, lstat 'C:\Users\ilg\AppData\Roaming\npm\node_modules'
    stack: >
      Error:
          Object.<anonymous> (node_modules/is-installed-globally/index.js:8:29)
          Object.Module._compile (node_modules/source-map-support/source-map-support.js:541:25)
    errno: -4058
    syscall: lstat
    code: ENOENT
    path: C:\Users\ilg\AppData\Roaming\npm\node_modules
    tapCaught: returnedPromiseRejection
    test: outdated version untuned

total ............................................... 40/42

The problem was introduced since 0.2.0.
It would probably be useful to run CI not only on Linux, but on Windows too (I use AppVeyor, but I think Travis also added support for Windows).

// @vladimyr

I'm afraid I can't help you with this one. I don't use Windows and don't have any Win machine to test on. Best thing I could do is to boot a vm but that's kinda unreliable. :man_shrugging:

boot a vm but that's kinda unreliable.

If you install the official Microsoft Windows 10 ISO in a VirtualBox machine, it is quite reliable, I have two Windows VMs (32/64-bit), plus several 32/64-bit Linux VMs (I'm on Mac) and this is how I test my modules on all platforms; Windows 10 will complain about the missing activation, but for occasional tests it is ok.
Once you fix the problem, you can add a set of CI Windows tests, for example in the project using your module I have the following AppVeyor configuration file:
https://github.com/xpack/update-checker-js/blob/master/.appveyor.yml

BTW, you problem seems related to the weird Windows paths.
And most probably is inherited from the dependencies (global-dirs?)


feature: ConnectionType supports custom fields

These fields can be added to either the Connection or the Edge types

Awesome work @NeedleInAJayStack!
So if I'm understanding correctly, the way to have custom connections or edges is to extend Connection and/or Edge and specify that the Node is a certain type?  Very elegant!
Will you update the Usage Guide with an example?

I use the custom connection function to retrieve paginated objects from the database. How can I add custom field totalCount to Connection structure in this implementation? I get the total count from the database and I need to pass it to the Connection structure during initialization. At the moment I am using my own Connection, PageInfo and Edge structures. The last change with adding public access to PageInfo and Edge doesn't work for me either because I can't initialize them, I still need to add public inits to them.

I use the custom connection function to retrieve paginated objects from the database. How can I add custom field totalCount to Connection structure in this implementation? I get the total count from the database and I need to pass it to the Connection structure during initialization. At the moment I am using my own Connection, PageInfo and Edge structures. The last change with adding public access to PageInfo and Edge doesn't work for me either because I can't initialize them, I still need to add public inits to them.

Regarding that, I don't think making the PageInfo and Edge struct have a public constructor wouldn't immediately solve the issue because you still can't really add custom properties to it or really extends it, unless I am missing something here.
Not 100% what is the best approach here. I am not sure what's the role of the built in Connection type is. Whether it is suppose to be a convenient built in solution that doesn't support all scenarios or the all in one solution where it should be used in any scenarios big or small.

I use the custom connection function to retrieve paginated objects from the database. How can I add custom field totalCount to Connection structure in this implementation? I get the total count from the database and I need to pass it to the Connection structure during initialization. At the moment I am using my own Connection, PageInfo and Edge structures. The last change with adding public access to PageInfo and Edge doesn't work for me either because I can't initialize them, I still need to add public inits to them.

Regarding that, I don't think making the PageInfo and Edge struct have a public constructor wouldn't immediately solve the issue because you still can't really add custom properties to it or really extends it, unless I am missing something here.
Not 100% what is the best approach here. I am not sure what's the role of the built in Connection type is. Whether it is suppose to be a convenient built in solution that doesn't support all scenarios or the all in one solution where it should be used in any scenarios big or small.

I have a few initial ideas on this:

Add a key value dictionary property to Connection similar to Vapor's Storage.  This would allow you to effectively pass data to the Connection resolvers without having to create a custom Connection constructor.  A new public constructor could be created on Connection that takes an optional dictionary argument.

Add something analogous to Graphiti's Context to pass to the connection type.  I'm not entirely sure how this would work.

Add an optional totalCount field to Connection since that is probably one of the most common use cases and then add a constructor that takes an optional async throws -> Int closure that is used to return the count.

Do nothing

Thoughts?

@NeedleInAJayStack This seems like a solution. But how do I pass arguments with filters that are in the resolver? I need to calculate the total number taking into account the filters that are used for pagination.

@NeedleInAJayStack This seems like a solution. But how do I pass arguments with filters that are in the resolver? I need to calculate the total number taking into account the filters that are used for pagination.

I agree.
For example, if you had a connection for friends of a user (not necessarily the current user), how would you accomplish getting the total count?  You might not want to query all the friends (it could be in the thousands), so edges.count might not work.
And I'm not sure how you'd pass the originating user's id to get the friends of the user to the connection.  This is the reasoning behind having a way to pass more data to the connection.  This is a simplified example, but you could have much more complex cases where you pass many arguments, and it would be nice to not have to reimplement all the connection cursor and slicing logic.

Ok cool, still catching up. To make sure I understand - the issue is that within the Connection extensions that define our custom field resolvers, we've lost the information as to how we got there (for example the pagination args like first, before, etc as well as the upstream object like user:1). Makes sense.
Adding those in is definitely a tougher problem and maybe we hold off on these changes until we figure it out. I think cshadek gave some interesting approaches. If one of you would like to take a crack at it, I'd be happy to review!

Ok cool, still catching up. To make sure I understand - the issue is that within the Connection extensions that define our custom field resolvers, we've lost the information as to how we got there (for example the pagination args like first, before, etc as well as the upstream object like user:1). Makes sense.
Adding those in is definitely a tougher problem and maybe we hold off on these changes until we figure it out. I think cshadek gave some interesting approaches. If one of you would like to take a crack at it, I'd be happy to review!

I can take a crack at it.  Do you guys have a preference on which of the 3 options (not including 4) is the best way to go?  My instinct says to go with (1) unless there's another better alternative.

If we did use (1) what would be the best way to implement the dictionary?  [String: Task<Any, Error>]?  That would allow you to pass an async task into the connection for specific string keys.
So for (1) you could have something like:
let connectionParams = [String: Task<Any, Error>]()
connectionParams["totalCount"] = Task {
    try await UserModel.query(...).filter(...).count()
}

let users = ....().connection(from: arguments, params: connectionParams)

extension Connection where Node == User {

    func totalCount(context: Context, arguments: NoArguments) async throws -> Int {
        return self.params["totalCount"] as? Int
   }
}

(I'm not sure about the naming, connectionParams vs connectionContext, etc)
I also don't like using the strings instead of something more strongly typed.  Is there a better way such as keyPaths, enums, etc?

Option 3 could also be a good option for now.  It takes care of a big use case by adding an optional totalCount: Task<Int, Error>? property to Connection.
let totalCount = Task {
    try await UserModel.query(...).filter(...).count()
}

let users = ....().connection(from: arguments, totalCount: totalCount)

A 5th option would be to somehow let Connection be aware of the upstream object and arguments.  This may actually be the best option, but it would require changing the Connection type to have another associated type Parent.  So then the Connection resolvers would be aware of Parent and the arguments and could resolve accordingly.  Then you could change the constructor for Connection to include a parent: Parent argument and you could use then still use the extensions similar to before:
extension Connection where Node == User, Parent == SomeUpstreamObject {

    func totalCount(context: Context, arguments: NoArguments) async throws -> Int {

        // Use self.parent and maybe self.arguments to fine tune this count query
   }
}

A potential problem with this approach is that one parent object could have multiple fields that resolve to the same connection type, so you'd need a way to differentiate between the different fields.
Thoughts?  Also how does a future desire to have connections of Unions and Interfaces affect this decision?

The 5th option looks better and I think something like this would work for this

I don't have a preference here. I think I would normally just use a custom connection and edge types if I need more that what it asked

@d-exclaimation What about option 3 then?  Or what if we made more of the inner connection logic accessible (slicing, etc).  That way it would be easier to create custom connections.  Maybe the standard should be to create custom connection and edge types, but at the same time, couldn't we make that process more efficient?
@ZirgVoice It seems to me like option 5 could be very complex and requires a lot of changes.

what if we made more of the inner connection logic accessible (slicing, etc). That way it would be easier to create custom connections?

I like this idea, it make sense to allow user to have control over that. I haven't look into it so I don't know how difficult that may be.

What about option 3 then?

I don't mind having that but I assumed the original design based on this, not sure if I want to go beyond that

I think the issue stems from the fact that we can't really get to the Edge and Connection initializers and we can't add properties with storage in extensions.  We still want to use the internal slicing and count logic though.
What if we could do something like the following:
let users = [User]()

// Custom Connection Case
users.connection(from: arguments, connection: FriendConnection(..... custom args), edge:
{ node -> FriendEdge  in
   return FriendEdge(..... custom args)
} )

// Default Connection Case
users.connection(from: arguments)

Then we can use the edge closure to create the edges when handling pagination.  Ideally, Connectable would have protocol extensions/ default implementation for all the pagination logic.
FriendConnection is an example, but it could be any custom Connection type that implements a new protocol Connectable.
FriendEdge is an example, but it could be any custom Edge type that implements Edgeable.
I think this implementation would solve the issues that @ZirgVoice, @d-exclaimation and @NeedleInAJayStack mentioned.
A user could then create the custom types as follows:
struct FriendEdge: Edgeable where Node == User {
    var upstreamUserId: UUID?
    var node: Node
    var cursor: String

    init(upstreamUserId: UUID?) {
        self.upstreamUserId = upstreamUserId
    }

    // Custom fields on the friend edge that use upstreamUserId, etc

    func friendsSince(context: Context, arguments: NoArguments) async throws -> Date? {
        // Use the custom args provided in `init` to return this
    }
}

struct FriendConnection: Connectable where Edge == FriendEdge {

     //  Something similar to FriendEdge with custom fields and a custom init

    func totalCount(context: Context, arguments: NoArguments) async throws -> Int {
         // Use the custom args provided in `init` to return this
    }
}

I'm not sure how this would work with ConnectionType and the schema creation logic, but this seems like it would be intuitive from the developer's perspective.  It likely requires a bunch of breaking changes though.
This is just an initial idea and I haven't actually tested any code.
Thoughts?

@NeedleInAJayStack any thoughts?

@cshadek, Sorry about the slow response, it's been a busy weekend 😀.
WRT the exposing the slicing logic, one concern I have is that the slicing logic is based on having an array in memory prior to pagination, whereas that may not be true in some cases (loading from a database, for example). So more general closure-based tools may be better.
The solution you outline above could work well too, but it is certainly a more complex API for the basic case.
Honestly, the more I think about it, Connection stuff may live best in a separate "relay" extension package. In that case I think we would have more freedom to be creative and iterative with the implementation. If there's big difficulties in doing this as a separate package, we could prioritize extensibility in Graphiti.

Honestly, the more I think about it, Connection stuff may live best in a separate "relay" extension package. In that case I think we would have more freedom to be creative and iterative with the implementation. If there's big difficulties in doing this as a separate package, we could prioritize extensibility in Graphiti.

@NeedleInAJayStack sorry it's been a busy few weeks for me haha.  I agree with this approach.  It probably makes sense to keep Graphiti as close to the GraphQL spec as possible.

WRT the exposing the slicing logic, one concern I have is that the slicing logic is based on having an array in memory prior to pagination, whereas that may not be true in some cases (loading from a database, for example). So more general closure-based tools may be better.

I agree with this approach as well.
Should we just merge this as is and revisit in the future?


feat(model): optimize the getting model to framework

This PR is related to issue https://github.com/ckb-js/kuai/issues/160.
This PR is to optimize the getting model to framework. To achieve this, I changed the way to bind Actor. Modify the ActorProvider class decorator to save the router path and add the Param parameter decorator to inject the parameter to Stores. And I added two default decorators to be used to create cell pattern for Stores, the DataCellPattern is to compare the whole data in cells and the DataPrefixCellPattern is to compare the prefix string of data in cells.

Please add a brief description of what this PR is for, with that we can have a quick peek with git log
Besides, a reference to the feature issue could be appended in the top message for tracking

CI failed


umbraco lifehack and umbraco cloud blog posts

The first article was published 7 days ago.
The second article "Unlocking Efficiency and Flexibility with Umbraco Cloud"  will be published by Eleftheria Tsatsi at Umbraco HQ in November 2023 on Umbraco's website and social channels.

Hi @NurhakKaya ,
Thanks a lot for those two new contributions 👍 Looks like you make the best of the Hacktoberfest vibe! 😁


HDFS-16910. Fix incorrectly initializing RandomAccessFile caused flush performance decreased for JN

Description of PR
At present, after our cluster backport patch HDFS-15882,
when set shouldSyncWritesAndSkipFsync to false, there will be flush performance degradation caused by JN.
Root Cause：
when setting shouldSyncWritesAndSkipFsync to false, the mode of init RandomAccessFile will be rws.
even if fc.force(false) is executed when flushAndSync is executed (hopefully, only requires updates to the file's content to be written to storage and the metadata is not update),
but since the mode of RandomAccessFile is rws, It will requires updates to both the file's content and its metadata to be written,
there will be flush performance degradation caused by JN.
For code changes:
Need to update RandomAccessFile's mode from rws to rwd:
rwd: Open for reading and writing, as with "rw", and also require that every update to the file's content be written synchronously to the underlying storage device.

if (shouldSyncWritesAndSkipFsync) {
rp = new RandomAccessFile(name, "rwd");
} else {
rp = new RandomAccessFile(name, "rw");
}

In this way, when flushAndSync is executed,
if shouldSyncWritesAndSkipFsync is false and the mode of RandomAccessFile is 'rw', it will call fc.force(false) to execute,
otherwise should use rwd to perform the operation.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  4s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
xmllint
0m  1s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
43m 10s

trunk passed

+1 :green_heart:
compile
1m 27s

trunk passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
compile
1m 21s

trunk passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
checkstyle
1m  7s

trunk passed

+1 :green_heart:
mvnsite
1m 31s

trunk passed

+1 :green_heart:
javadoc
1m  6s

trunk passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javadoc
1m 31s

trunk passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
spotbugs
3m 23s

trunk passed

+1 :green_heart:
shadedclient
26m 25s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 28s

the patch passed

+1 :green_heart:
compile
1m 20s

the patch passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javac
1m 20s

the patch passed

+1 :green_heart:
compile
1m 11s

the patch passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
javac
1m 11s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 52s

the patch passed

+1 :green_heart:
mvnsite
1m 22s

the patch passed

+1 :green_heart:
javadoc
0m 54s

the patch passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javadoc
1m 26s

the patch passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
spotbugs
3m 24s

the patch passed

+1 :green_heart:
shadedclient
25m 34s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
301m 24s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 50s

The patch does not generate ASF License warnings.

419m 40s

Reason
Tests

Failed junit tests
hadoop.hdfs.TestLeaseRecovery2

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5359/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5359

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint

uname
Linux 25d578b27c91 4.15.0-200-generic #211-Ubuntu SMP Thu Nov 24 18:16:04 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 883e4f9f75601ee59d65d6c0512a69f404d0568f

Default Java
Private Build-1.8.0_352-8u352-ga-1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_352-8u352-ga-1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5359/1/testReport/

Max. process+thread count
3215 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5359/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@haiyang1987 Can you share the improvement of this PR with some screenshots?

@haiyang1987 Can you share the improvement of this PR with some screenshots?

sure, already share.

The failed UT hadoop.hdfs.TestLeaseRecovery2 is traced by HDFS-16853.
Merged. Thanks @haiyang1987 for your report and thanks @Hexiaoqiao for your review.

Thanks @ZanderXu @Hexiaoqiao help me review and merge.


K1 Exemption Question

Hey @fabio-looker, thanks for the previous fixes. I've finally gotten back to implementing v2, and I'm running into one thing I think is an issue but need to verify with you. I have several views that are extended and as a result have no PKs. This was erroring with rule K1. We put a rule_exemptions block for K1 in the view files. In the past I think this removed the error.
Now, with v2.1.1, it seems like this is recognizing the exemption, but still flagging it as an error. Is this by design? Or is there an issue here? I can't seem to get the error to go away. What's also interesting is that for some other view files that are extension:required with no PKs, lams doesn't flag these with K1 at all (the ones that do get the error all have extensions, a couple are further extended, some are final views).
Example --output legacy-cli output:
{ rule: 'K1',       location: 'view: base_charge',       path: '/projects/.../files/.../base_charge.view.lkml#view:base_charge',       exempt: '2021-07-30 - The primary key is defined when the base view is implemented.',       level: 'error',       description: 'No Primary Key Dimensions found in base_charge' }
Exemption block:
view ... {
...
# LAMS
# rule_exemptions: {
#  K1: "2021-07-30 - The primary key is defined when the base view is implemented."
# }
...
}

Hey @alec-flatiron !
Generally speaking, the intention behind the v2 release (and one of the breaking changes) is that exemptions should be entirely ignored and not reported at all anymore.
Of course, the implementation may vary from my intention, so I'll take a look at this during the week to see what I find

Gotcha, yeah it seems like in K1 it's flagging everything as an error unless there's a globalExemption, I don't see any handling of local view exemptions (but it catches them and logs them in the message)

@fabio-looker Have a chance to look at this yet? Hoping to make the update to use a v2 version

Hi @alec-flatiron - yes, sorry about the delay. I do plan to fix this next, just got started and it seems there's a lot of outdated tests to sift through and re-align. Will be working on it today actually, but I may need to continue on it into next week

Awesome no worries, thanks for the update!

Looks good @fabio-looker thank you!

Sorry for the terribly long turnaround time. This should be fixed now, with a bunch of tests now set up around both local and project-level exemptions for a bunch of the more complex built-in rules


Favorites Feature

Hello 👋,
I open this issue because I want to implement the Favorite Feature in Article Screen, I will open PR asap.
Hope you guys have a good day and stay safe and healthy 🙏
Thanks

Hi there! Thanks very much for your interest.
This sample is a work in progress and we need to make sure it actually makes sense to implement that feature (does it add anything new to the app that we want to showcase?). I don't think it would be interesting right now because we already have the similar bookmark feature.

Hi @JoseAlcerreca , Okay no problem 😁, I thought someone need to impl it because there was a toast that show to replace it in the future when we clicked on that icon.


Hi guys, I wanna to do this Codelab in Kotlin

I see this codelab like others ones was be written in Java, but I don't know if you like the idea of me to export all the code to Kotlin and the Codelab to Kotlin
Happy Hacktoberfest 😎
Cheers from Firebase Perú community.

@carlosgub thanks for your enthusiasm and your offer to convert the codelab!
@kroikie have you decided on our plan for Kotlin + Codelabs?

I will be waiting for your response @samtstern @kroikie. I can translate the codelabs to Spanish 💃

I will be waiting for your response @samtstern  @kroikie. As well, I can translate the codelab to Spanish

Hello 💃


nx and svelte 4 upgrade

Issue 2: This is related to issue #1379. With the upgrade a index.d.ts file is being generated, but instead the types it just references the svelte files. This might be reason to include 1379 fixes in this PR.
Issue 3 is fixed :+1:

:tada: This PR is included in version 1.17.0-alpha.11 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.6.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[SPARK-40800][SQL][FOLLOW-UP] Add a config to control whether to always inline one-row relation subquery

What changes were proposed in this pull request?

This PR is a follow-up for SPARK-40800. It introduces a new Spark config to control the behavior of whether to always inline one-row relation subquery: spark.sql.optimizer.optimizeOneRowRelationSubquery.alwaysInline (default: true).
Why are the changes needed?

To give users more flexibility to control the correlated subquery performance.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Edited existing unit tests.

cc @cloud-fan

thanks, merging to master!


Make banners responsive for better small-screen display

The text on the current banners gets hard to read on small screens, but SVG supports CSS @media queries that can allow different styling on small screens.
This PR shows a slightly reduced content at a larger font size when the SVG is smaller than 600px wide.

Thank you!


Can download sites to custom folders and the path is set

What do you exactly mean? (This miss understandings could be solved through the usage of the Issue Template)

Instead of saving all sites in "data" folder, when visiting each site user can save in a custom folder, and it will work.


Improve the documentation for ListView and ListViewDataSource

Motivation (required)
What existing problem does the pull request solve?
My motivation was to improve the documentation for ListView and ListViewDataSource because I always have trouble remembering how they work and the documentation was lacking an example for cloneWithRowsAndSections
My PR does the following:

add a note about how headers and footers are rendered on a horizontal ListView
fix a typo in an example
clarify that the sectionIdentities should match the keys or array indexes of the data source
add an example of cloneWithRowsAndSections
add notes on the return values of getRowCount and getRowAndSectionCount

Test Plan (required)
n/a; documentation only

@javache any advice on how I can move this one along?

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with Canny! Check out its new home: https://react-native.canny.io/feature-requests/p/improve-the-documentation-for-listview-and-listviewdatasource

@hramos Do you need me to make any updates so this can be merged?

LGTM.


WebP v1.2.0 is out

https://github.com/webmproject/libwebp/releases/tag/v1.2.0

API changes:

libwebp: encode.h: add a qmin / qmax range for quality factor (cwebp adds -qrange)

lossless encoder improvements
SIMD support for Wasm builds
add xcframeworkbuild.sh, supports Mac Catalyst builds
import fuzzers from oss-fuzz & chromium (#409)
webpmux: add an '-set loop ' option (#494)
toolchain updates and bug fixes (#449, #463, #470, #475, #477, #478, #479, #488, #491)

Version 1.3.5 is now built with it.


inject different allowlist content based on different cluster type

https://github.com/stolostron/backlog/issues/23650
Signed-off-by: marcolan018<EMAIL_ADDRESS>

/approve
/lgtm
/hold

/retest

/retest

/retest

/retest

/retest

/retest

/retest


Example 2nd Chance Performance

BCT PGN 61450 from Engine #1 (0) with SPNs 132
BCT PGN 61454 from Engine #1 (0) with SPNs 3216, 3217
BCT PGN 61455 from Engine #1 (0) with SPNs 3226
BCT PGN 61466 from Engine #1 (0) with SPNs 3464
…
PGN 61454 with Supported SPNs 3216, 3217
14:53:59.4671 18F00E00 [8] 00 FB FF FF FF FF FF FF
Found: AFT 1 Intake Gas 1 from Engine #1 (0):
SPN  3216, Engine Exhaust 1 NOx 1: 0xFB00
SPN  3217, Engine Exhaust 1 Percent Oxygen 1: Not Available
SPN  3218, Engine Exhaust 1 Gas Sensor 1 Power In Range: 11
SPN  3219, AFT 1 Intake Gas Sensor 1 at Temperature: 11
SPN  3220, Engine Exhaust 1 NOx 1 Reading Stable: Not available
SPN  3221, AFT 1 Intake Wide-Range Percent Oxygen 1 Reading Stable: 11
SPN  3222, AFT 1 Intake Gas Sensor 1 Heater Preliminary FMI: 31.000
SPN  3223, AFT 1 Intake Gas Sensor 1 Heater Control: 11
SPN  3224, Engine Exhaust 1 NOx Sensor 1 Preliminary FMI: 31.000
SPN  5714, Engine Exhaust 1 NOx Sensor 1 Self-diagnosis Status: 111
SPN  3225, AFT 1 Intake Oxygen  Sensor 1 Preliminary FMI: 31.000
FAIL: <IP_ADDRESS>.a - SPN 3217 was received as NOT AVAILABLE from Engine #1 (0)
…
14:54:19.7483 Destination Specific for PGN 61454 for SPNs 3217 Request to Engine #1 (0)
14:54:19.7510 18EA00F9 [3] 0E F0 00 (TX)
14:54:19.7550 18E8FF00 [8] 01 FF FF FF F9 0E F0 00
Acknowledgment from Engine #1 (0): Response: NACK, Group Function: 255, Address Acknowledged: 249, PGN Requested: 61454
FAIL: <IP_ADDRESS>.b - No DS response for PGN 61454 from Engine #1 (0)
WARN: <IP_ADDRESS>.b - Global request was required for PGN 61454 for broadcast SPNs 3216, 3217
14:54:19.7580 Global PGN 61454 for SPNs 3216, 3217 Request
14:54:19.7621 18EAFFF9 [3] 0E F0 00 (TX)
14:54:19.7668 18F00E00 [8] 00 FB FF FF FF FF FF FF
AFT 1 Intake Gas 1 from Engine #1 (0):
SPN  3216, Engine Exhaust 1 NOx 1: 0xFB00
SPN  3217, Engine Exhaust 1 Percent Oxygen 1: Not Available
SPN  3218, Engine Exhaust 1 Gas Sensor 1 Power In Range: 11
SPN  3219, AFT 1 Intake Gas Sensor 1 at Temperature: 11
SPN  3220, Engine Exhaust 1 NOx 1 Reading Stable: Not available
SPN  3221, AFT 1 Intake Wide-Range Percent Oxygen 1 Reading Stable: 11
SPN  3222, AFT 1 Intake Gas Sensor 1 Heater Preliminary FMI: 31.000
SPN  3223, AFT 1 Intake Gas Sensor 1 Heater Control: 11
SPN  3224, Engine Exhaust 1 NOx Sensor 1 Preliminary FMI: 31.000
SPN  5714, Engine Exhaust 1 NOx Sensor 1 Self-diagnosis Status: 111
SPN  3225, AFT 1 Intake Oxygen  Sensor 1 Preliminary FMI: 31.000

Not sure DS for NA data in SPN 3217 is exactly correct … for further review

Confirmed in 3.1.4. Whereas 3217 was received as NA in the broadcast It should not be requested in for on request data. This creates multiple late responses from the broadcast data. DS messages to the ECU(s) that indicated support in DM24 for upon request SPs and broadcast SPs that were not observed in step 1 from both lists. [SPs that were observed as ‘not available’ in broadcast data from a particular ECU need not be queried — the data evaluation from the broadcast data suffices]. Where a PG from a particular OBD ECU contains more than one SP (to be evaluated), that PG need only be queried one time].
b.If no response/no valid data for any SP requested in <IP_ADDRESS>.a, send global message to request that SP(s).
// Splitting the setup (Actions1) from the monitoring (Actions2) kept the test steps from <IP_ADDRESS> to <IP_ADDRESS>.
// Checking for omitted broadcast SPs in a. is the “second chance concept”.  Such SPs failed <IP_ADDRESS>.a.
// This (a.) provides evidence of support for undesirable circumstances (e.g. two OBD ECUs providing the same PG).
// Both supported and unsupported SPs in Table A-1 are queried (in a.) to check for omitted DM24 support bit settings.
// These changes increase the value of tests on development engines, by being more thorough with DM24 support bits.

SecondChanceClarification3.docx

Table A-1 additions not yet displayed.
Messages from Table A-1 not expected from DM24:           // This is the Second (consolidated) list
Req PGN 40448 from Engine #1 (0) less SPNs 4106             // Show unsupported SPs only here
Req PGN 40960 from Engine #1 (0) less SPNs 4127, 4130           // Show a PG in both lists
BCT PGN 61444 from Engine #1 (0) less SPNs 190, 512, 513, 2432   // But query it once,
BCT PGN 61450 from Engine #1 (0) less SPNs 132                   // if it is not broadcast.
BCT PGN 61454 from Engine #1 (0) less SPNs 3216, 3217
BCT PGN 64908 from Engine #1 (0) less SPNs 3610
BCT PGN 64916 from Engine #1 (0) less SPNs 27, 5444
Req PGN 64920 from Engine #1 (0) less SPNs 5454
BCT PGN 64923 from Engine #1 (0) less SPNs 3516
Req PGN 65214 from Engine #1 (0) less SPNs 166
…
Step <IP_ADDRESS>.a - Reading bus for 20 seconds

White space requests not yet implemented
FAIL: <IP_ADDRESS>.a SPN 12345, not observed in 20 seconds. // Let’s put the missing broadcast SPs here
Step <IP_ADDRESS>.c Observed Broadcast Periods                    // Add heading for observed periods
PGN 61443 from Engine #1 (0)
13:41:28.0015 0CF00300 [8] D1 00 00 FF FF FF 65 FF
13:41:28.0512 0CF00300 [8] D1 00 00 FF FF FF 65 FF
13:41:28.1015 0CF00300 [8] D1 00 00 FF FF FF 65 FF
…
PGN 61454 from Engine #1 (0)
13:41:28.0050 18F00E00 [8] 00 FB FE FA FF FF FF FF
13:41:28.0532 18F00E00 [8] 00 FB FE FA FF FF FF FF
13:41:28.1050 18F00E00 [8] 00 FB FE FA FF FF FF FF …
Step <IP_ADDRESS> Destination Specific Requests                  // Add heading for DS Queries
13:41:48.2142 DS Request for PGN 40448 to Engine #1 (0) for SPNs 4106
13:41:48.2168 18EA00F9 [3] 00 9E 00 (TX)
13:41:48.2224 189EF900 [8] [00 00 00 00 00](tel:00 00 00 00 00) FF FF FF
DM29 from Engine #1 (0):
Emission-Related Pending DTC Count                               0
All Pending DTC Count                                            0
Emission-Related MIL-On DTC Count                                0
Emission-Related Previously MIL-On DTC Count                     0
Emission-Related Permanent DTC Count                             0


$queryRaw's unknown return type is too broad

Bug description
I might be missing something, but it seems$queryRaw will always return an array of objects with string keys. As of version 3.1.0, Prisma has the return type from this function as Promise<unknown> but it could be narrowed to Promise<Record<string, unknown>[]>.
This would be extremely helpful, since trying to operate on the array will result in a type error unless the developer manually narrows it from unknown.
For example:
const rows = await prisma.$queryRaw(sqltag`SELECT current_user;`)

for(const row of rows){ // <- TypeScript error: "Object is of type 'unknown'.ts(2571)"
  // do something
}

How to reproduce
With a Postgres database, run
const rows = await prisma.$queryRaw(sqltag`SELECT current_user;`) // rows will have type `unknown`

Expected behavior
With a Postgres database, running:
const rows = await prisma.$queryRaw(sqltag`SELECT current_user;`)

should result in rows having type Record<string, unknown>[]
Prisma information
Using prisma v3.10.0
Environment & setup

OS: macOS
Database: PostgreSQL
Node.js version: v16.14.2

Prisma Version
v3.10.0

I kinda have the same problem here.
export const fetchAppsOfUsers = (source: any, { id }: FetchAppsOfUsersProps) =>
  new Promise<UserAppsQuery>(async (resolve, reject) => {
    try {
      const listOfAppsForUser: UserAppsQuery = await prisma.$queryRaw<Array<FetchAppsOfUsersDB>>`
        SELECT *
        FROM graphql_test."Users" u
        INNER JOIN graphql_test."UserApps" uapp ON u.id = uapp."userId"
        INNER JOIN graphql_test."GooglePlayStore" gps ON uapp."appId" = gps."id"
        WHERE u.id = ${id}
      `.then((resp: any) => {                    <==================== HERE
        const firstItem: FetchAppsOfUsersDB = resp[0] as FetchAppsOfUsersDB

        return {
          id: id,
          user: {
            id: firstItem.userId,
            name: firstItem.name,
          },
          application: resp.map((item: FetchAppsOfUsersDB) => ({
            id: item.id,
            app: item.app,
            ...otherProps,
          })),
        }
      })

      resolve(listOfAppsForUser)
    } catch (error) {
      console.error('Fetch Apps error:', error)
      reject(error)
    }
  })

Ideally, this should be something like
.then((resp: Array<FetchAppsOfUsersDB>) => {

since that is the expected response type from the prisma.$queryRaw above. However, this does not happen, i.e. the type is not inferred into the .then block, so I'm forced to typecast it as any to get it working.
Am I doing something wrong here?

Hey @alexarena,
You could declare the return type of your response as
const rows = await prisma.$queryRaw<RETURN_TYPE_HERE>(sqltag`SELECT current_user;`)

and you should be able to see the type definition here
for(const row of rows){ // <- TypeScript error: "Object is of type 'unknown'.ts(2571)" ||||| Rows should be defined correctly here - inferred from `RETURN_TYPE_HERE`
  // do something
}

Just changing the signature to at least indicate that it always returns an array would be a huge step up.
- $queryRaw<T = unknown>(query: TemplateStringsArray | Prisma.Sql, ...values: any[]): Prisma.PrismaPromise<T>
+ $queryRaw<T = unknown>(query: TemplateStringsArray | Prisma.Sql, ...values: any[]): Prisma.PrismaPromise<T[]>

If this was fixed, then testing .length on the returned value would work without having to specify any type parameters, and you also wouldn't need to add an extra Array<...> when specifying the type.
This would be a breaking change at this point, but I think that it would be great if this could be scheduled for inclusion for Prisma 6.0.0. @Jolg42 are you tracking changes for next major version somewhere? 🙏

@LinusU Can you open a feature request issue with the full context and explanation for that? Then we can properly track this indeed.

Here it is: #23987 🎉


Missing Google Analytics logo in meltano.com home page

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1035
Originally created by @iroussos on 2019-09-24 11:45:36

In meltano.com home page, under the Plugins to Extract & Load Data section we do not have a logo for Google Analytics.
I think that we should add it as a lot of people are interested in extracting data from GA, we have thoroughly tested tap-google-analytics and it works great.

View 4 previous comments from the original issue on GitLab


Socket hang up when deploying Azure Functions

Deployment using Azure Functions Extension (Deploy to Function App...) fails with VSCode version 1.66. When changed to previous version 1.65.2 of VSCode, everything works fine.
Azure Functions output logs:
9:45:47 AM xx-functions: Starting deployment...
9:45:51 AM xx-functions: Creating zip package...
9:46:06 AM xx-functions: Zip package size: 20.2 MB
10:01:42 AM: Error: request to https://xx-functions.scm.azurewebsites.net/api/zipdeploy?isAsync=true&author=VS Code failed, reason: socket hang up

Same here. Restarted deployment slot. Created new deployment slot. Restarted computer. Nothing helped except downgrading VS Code to 1.65.2 that I downloaded from here.

Please file this against the Azure Functions Extension

Socket hang up while deploying to azure functions
VS Code Version: 1.66.0
OS: Darwin x64 21.3.0 (Mac OS Monterey 12.2.1)
11:46:39 AM Starting deployment...
11:46:46 AM Creating zip package...
11:46:49 AM Zip package size: 1.27 MB
12:02:58 PM: Error: request to https://.scm.azurewebsites.net/api/zipdeploy?isAsync=true&author=VS%20Code failed, reason: socket hang up

Same here. Restarted deployment slot. Created new deployment slot. Restarted computer. Nothing helped except downgrading VS Code to 1.65.2 that I downloaded from here.

Thank you!


Add command line and multiple-choice tools.

No changes to existing code. Just added some files used for testing answers to multiple choice questions and for running command line tests. This is my first pull request, so hopefully I did this correctly :)

Good idea.

@tormoder should he revise his pull request, or send a new one. How does it work?

I think he could just push to his master and the commit will be added. Since the PR is from ericnorway:master.

I pushed doc.go to my master branch. It looks like it is visible from the pull request.


Add Ruby Endpoints sample

The repo I pushed is not in sync with GCP/ruby-docs-samples.  Additionally, Googlebot is upset.  Sending a follow-up pull.


Advice needed - Ability to iterate over several tools/functions, best practices?

Hi Maxime,
This is an awesome initiative, I'd love to get started and hopefully contribute wherever possible! I went over your docs and examples and coming from Langchain and Llamaindex I think I have a reasonable understanding on how to use the framework, but I would like your advice on how to handle something like this.
Scenario - Using the Chat Agent to pull data and iterate over various functions.
Setup:
"Prompt: Your task is to help the user with day to day maintenance tasks for their web agency. You can use several functions to retrieve information. Here are the tools you can use to get info
tool/function list + description
When you have enough information to answer the question provide a response with the following tone.
custom prompt instructions
Example

User asks: Which of my client sites need to be updated? And what needs to be done?
Agent Tool: Get Sites
use function (GetSitesOfClientWithUpdates($client)
Result: $sitesarray
Agent Tool: Get Tasks
user function (GetSiteTasks($sites array)
Result: json response of all tasks for all sites
Agent Tool: Analyse JSON and answer
Final Answer: Your site hallo.com has dropped in performance, we should probably look at improving it! goodbye.com has an SSL certificate that expires next week.

Do you think that LLPhant could achieve something like this already? I'm using Laravel and I could "loop" over responses and iterate, but if I understand correctly you might have already planned for this exact usage and you could provide me with an example of guidance?
Have a good day, and thanks for starting this project

Hey @BoweFrankema,
Yes I think you can build something like this. It would look like AutoPHP but more specialized. It could even work with AutoPHP with the right functions.
Few advices:

Max 3 functions to the agent otherwise performance drops (2 is fine)
"json response of all tasks for all sites" should not be super long
you need to explain how you will prioritize the tasks to be done
Start with one client site and make your agent return "action needed: XXX" or "no action needed". After that you can just loop over you client sites.

I hope it helps. Please feel free to ask other questions

Thank you Maxime! That's really helpful!  I already have a base prototype working so it's now just a matter of learning the framework and seeing how far I'll get. I'll absolutely report back once I'm a bit further ahead!


multilang: bad language translation check

may fix #29454 (bad language check)

[X] use LDNOOBW to check for bad language
[X] handle incorrectly labeled bad words (override list or local copy)
[X] check numerusforms

comma@comma-33e0a0c6:/data/openpilot$ ./selfdrive/ui/tests/test_translations.py
checking main_en
checking main_de
checking main_fr
Fchecking main_pt-BR
checking main_tr
checking main_ar
checking main_th
checking main_zh-CHT
checking main_zh-CHS
checking main_ko
checking main_ja
....s.
======================================================================
FAIL: test_bad_language (__main__.TestTranslations.test_bad_language) (name='Français', file='main_fr')
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/data/openpilot/./selfdrive/ui/tests/test_translations.py", line 166, in test_bad_language
    assert word not in banned_words, f"Bad language found: {word} - {translation_text}"
           ^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: Bad language found: pédale - Désengager avec la pédale d'accélérateur

----------------------------------------------------------------------
Ran 7 tests in 0.982s

FAILED (failures=1, skipped=1)

I split the bounty into two parts to make this easier to merge - https://github.com/commaai/openpilot/issues/29454#issuecomment-1867151696.

$100 sub-bounty claimed! For the accuracy part, if there really isn't a good local solution, we can consider using the OpenAI APIs if the cost is reasonable (<$50/month for all our CI runs).

@royjr I thought about this some more and I don't think we need to hit an API at all. We should be able to use a good open source embeddings model for this; we've just gotta make sure it'll be fast enough on the GHA runners.


borealis logs

i'm trying to get borealis work on my machine, ye, unsupported 4 gb ram( wanna take some logs, to learn how it works, maybe it help me with setup on unsupported hardware

You can search for the keyword crosvm/vm_concierge in /var/log/messages

FYI, log provided by the user: https://imgur.com/a/F03q02C

Looks like SMT not disabled and other vulnerable check not passed. Will try to skip this check next release.


[backport-v1.13] ipcache don't short-circuit InjectLabels if source differs

Backport of #24875  to v1.13.
Setting as draft, as that PR has not merged.

Closing for bureaucratic reasons - backport coming soon!


Fix broken headings in Markdown files

GitHub changed the way Markdown headings are parsed, so this change fixes it.
See bryant1410/readmesfix for more information.
Tackles bryant1410/readmesfix#1

:+1: thank you @bryant1410 !


tmux plugin: can't start new session

When I run tmux while using the tmux zsh plugin, I am unable to start a new session, just reattach to an old one. When I remove the plugin from my .zshrc, everything is fine. Currently using tmux 1.9a, and OS X Yosemite, though this was a problem in Mavericks too.

What's your tmux.conf?


Add Rollbar support to tile server

Create a Raster Foundry project within Rollbar and integrate it with the Scala tile server.

Duplicate of https://github.com/azavea/raster-foundry/issues/1010


System.Security.Claims.ClaimTypes.Email value does not match the summary URI for the claim

The value of the Email claim type does not match the URI for the claim, is this intentional?
General
Current value:
"http://schemas.xmlsoap.org/ws/2005/05/identity/claims/emailaddress"
Should this be:
"http://schemas.xmlsoap.org/ws/2005/05/identity/claims/email"
The documentation below leads me to believe it should be the ladder:
https://docs.microsoft.com/en-us/dotnet/api/system.security.claims.claimtypes?view=netcore-2.1
//
// Summary:
//     The URI for a claim that specifies the email address of an entity, http://schemas.xmlsoap.org/ws/2005/05/identity/claims/email.
public const string Email = "http://schemas.xmlsoap.org/ws/2005/05/identity/claims/emailaddress";

Issue moved to dotnet/corefx #32236 via ZenHub