Add completions for closing tag name

TypeScript Version:  3.0.0-dev.201xxxxx

Search Terms:
Code
var x = <Comp> <|  // Completion here

Expected behavior:
The list should contain /Comp>.
Actual behavior:
No completion list
Playground Link:
Related Issues: https://github.com/Microsoft/TypeScript/issues/20811

Don't think this is necessary now that we have #24543; we also already have a completion after / is typed, so this would only save a single character, that you would probably have to type anyway to select the completion out of the list.


Incorrect message in alerts KubeJobFailed and KubeJobCompletion #2094

Fixed message for alert KubeJobCompletion and KubeJobFailed.
The message was:
Job /kube-state-metrics is taking more than 30 minutes to complete.
The new message is:
Job default/my-job is taking more than 30 minutes to complete.

These alerts are not defined in this repository, they are just a dependency that happens to be used here, therefore this needs to be fixed here: https://github.com/kubernetes-monitoring/kubernetes-mixin

This has been fixed upstream in the monitoring-mixins now.
https://github.com/kubernetes-monitoring/kubernetes-mixin/pull/115
We should create another PR to update the dependency.

Just found out that https://github.com/coreos/prometheus-operator/pull/2096 is taking care of updating the jsonnet dependencies already.


Feature request - Add all v1/ routes

@npuichigo I am trying to use Triton Inference Server with TensorRT-LLM backend with openweb-ui as frontend, but not all routes are provided, e.g. /v1/models etc.
Is there any plan to support all openapi v1 routes?
It will be really great if full openai api support is available, since kserve is still under works.

@visitsb It's fine to add /v1/models. But the list of full openai api is long, like /v1/audio, /v1/embedding. What's the minimal subset is needed?

@npuichigo Thanks for the quick reply!
Are you able to add below? Looking at open-webui's implementation, at minimum-
/v1//models
/v1//chat/completions
/v1/embeddings
/v1/audio/speech
/v1//audio/transcriptions

Wish there was an easier way to provide full compatibility, but sometime in future.

The exposed API depends on the actual model hosted in triton backend. Since there's no embedding model available in trtllm, /v1/embeddings is not possible. For embedding model, maybe you can refer to https://github.com/huggingface/text-embeddings-inference.
The same reason applies to /v1/audio/* since no ASR and TTS models are available now in trtllm.


Host Program does not support User Interaction

Have a simple test file like:
Describe 'the host program' {
    It 'supports user interaction' {
        $Host.UI.RawUI.WindowTitle = "magic"
        $title = $Host.UI.RawUI.WindowTitle
        $title | Should -Not -BeNullOrEmpty
    }
}

It will successfully pass when executed with "Debug Test". However, when stared via "Run Test", it will fail on its first line (e.g. $Host.UI.RawUI.WindowTitle = "magic") with a message as:

Exception setting "WindowTitle": "A command that prompts the user failed because the host program or the command type does not support user interaction. Try a host program that supports user interaction, such as the PowerShell Console, and remove prompt-related commands from command types that do not support user interaction."

It would be also useful to have some defaults within properties of $Host.UI.RawUI.

The test runs operate within their own dedicated process, so UI pester tests are unsupported there, however if you run the tests via debug then they run in the PS Extension which should be supported.
I do have a potential refactor coming to move everything into the PS Extension as a separate runspace which may make this functional


Query History by State Change

Sometimes you may want to query Simulator history in order to find out the last time a particular event occurred. In the case of logs you may want to know last time a Simulator was booted in order to narrow down information and discard pre-last-boot information. By adding queries by last-change-to-state it's possible to find this information.
In addition, the recursive description has been improved so that the ordering is correct and less-noisy descriptions are used.

lgtm


How to get the "stable" version with release branches?

https://docs.readthedocs.org/en/latest/versions.html says "We also create a stable version, if your project has any releases."
Is it possible to have the "stable" version if I'm using release branches, or does "stable" only work for release tags?
I currently have a "master" and "1.0.1" branch, but I don't see the option to build a "stable" version in my admin panel (I would like to have "/en/stable/" point to the "1.0.1" branch). How is this supposed to work, exactly? My docs are at http://docs.citationstyles.org/en/latest/.

It's also unclear to me what a "release" means in version control terms here. Tag? Branch? GitHub release notes without the "pre-release" bit set?

I noticed that the RTD documentation was recently updated to clarify that the "stable" version is only available when there are "tagged releases". See https://github.com/rtfd/readthedocs.org/commit/c5f696edba17376c54747fed786f7346fafb54e6
That said, I pushed some tags to my repository, and I still don't see any option to build "stable". (see also https://github.com/rtfd/readthedocs.org/pull/1046#issuecomment-73639262 )

I've had tags on my project for a while and I still don't see a "stable" option for my project either.  I think the original docs were set up before any tags were added, is it possible that it doesn't re-check after you've added tags?

I'm also unclear on how the stable branch is determined.
My project has the following branched

1.2-line : Latest stable release line
dev : Development branch
pypi: Latest release on PyPI

I also use git tags for releases.
I released a 2.0.0a1 version which is now on both the dev and pypi branch. The stable version on RTD shows up as the 2.0.0a1. This is incorrect; I would expect it to show the latest release on the stable 1.2-line branch.
Does RTD do any parsing of version numbers according to, e.g. semver.org or PEP440 to determine which is the stable branch? If not, it would be a nice feature to add. It would also be great if we could specify which git branch should be used for the stable version docs.

readthedocs basically goes through all the versions you see in the admin panel and takes the names there and parses them with distlib. So all the strings like 'v1.0', 2.0.0a1 etc should work. Branch names like master will fail during parsing and then be left out when trying to figure out the available version numbers.
The distlib version in use is 0.1.2 which is quite old, so if some versions are parsed unexpectedly than that's probably the reason. distlib 0.1.2 is following PEP-386 for parsing, where as the "newer" Python community probably expects PEP-440.
@sloria I think your problem is directly related to that. Maybe we should upgrade the distlib library.

@rmzelle We have merged a change which might have fixed the problem with you not beeing able to build a staging version. Is this possible for you now?

I got it to work a while back (https://readthedocs.org/projects/citation-style-language/). Not sure how, but I did switch back and forth between GitHub repos, which might have done the trick.

Great :)


Handle Ledger THORChain & Cosmos app open request

Detecting if the THORChain app is open on Ledger is currently broken, which I suspect is caused by await this.transport.call(null, "getAppAndVersion") returning THORChain, whereas the slip key is written as Thorchain in HDWallet's slip44Table (I suspect incorrectly, as SLIP-0044 lists it as THORChain too).
This might require a slight tweak in HDWallet.

Cormos is in the same boat, debug this at the same time 🙏

When attempting to manage accounts for thorchain or cosmos, the app crashes due to missing support for validateCurrentApp on these chains. When calling validateCurrentApp, the promise is rejected with Unable to find associated app name for coin: ${coin} because support for these chains is actually missing from hdwallet.
This issue never presented in web previously because we never attempt to validate the correct app being open on ledger, and instead swallow errors (hence the requirement for the new throwOnReject flag added in this PR.
The workaround here is to bypass this check and allow the user to attempt to proceed, then display an error toast if account fetching fails.
We'll have to follow up with a fix in hdwallet and remove this temporary patch.

Once https://github.com/shapeshift/hdwallet/pull/670 is merged, will have to follow up in web and remove the patches.


Google Map Link page, Undefined data now showing properly

Related to ticket #295.
Showing correctly with JSON server data.

@Rando-cal could you fix these merge conflicts and merge when you get the chance?


build: update linkinator config

@bcoe should we delete the windows tests we have running on kokorkokoroko?


[Android][WebView] Getting No ViewManager defined for class RCTWebView

I'm in the process of trying to port a react native iOS app over to Android but am running into an issue with the built in WebView Component. Even when trying to use the example code listed both in the docs and in this repo I keep getting the "No ViewManager defined for class RCTWebView" error in the Android Emulator when attempting to load a view with the WebView component. The emulator has Google API 23, I have v0.11.0 of react native, updated my react native cli, and am running node v4.0.0. I have tested the existing code in iOS against this new version of react and it still works fine. Anyone have any idea what could be causing this error?
This is the view containing the a component LoginWebView:
/*
 *  React
 */

var React = require('react-native');
var {
  StyleSheet,
  Text,
  View,
  Navigator
} = React;

/*
 *  Flux
 */

var Fluxxor = require('fluxxor');
var FluxMixin = Fluxxor.FluxMixin(React);

/*
 *  Components
 */

var LoginWebView = require('../../components/login-webview');

/*
 *  LoginWebViewScene
 */

var LoginWebViewScene = React.createClass({

  mixins: [FluxMixin],

  render: function() {
    return(
      <View style={styles.container}>
        <LoginWebView flux={this.getFlux()}></LoginWebView>
      </View>
    );
  }

});

/*
 *  Styling
 */

var styles = StyleSheet.create({
  container: {
    flex: 1,
    alignItems: 'stretch',
  }
});

module.exports = LoginWebViewScene;

And this is the LoginWebView itself, which should just display the content at a given URL:
/*
 *  Properties
 */

var React = require('react-native');
var constants = require('../constants.js');
var {
  View,
  WebView,
  StyleSheet
} = React;

/*
 *  Flux
 */

var Fluxxor = require('fluxxor');
var FluxMixin = Fluxxor.FluxMixin(React);

/*
 *  LoginWebView
 */

var LoginWebView = React.createClass({

  mixins: [FluxMixin],

  onNavigationStateChange: function(obj) {
    // Don't forward to Flux until the webview has stopped loading
    if (!obj.loading) {
        this.getFlux().actions.newRouteForWebview(obj.url);
    }
  },

  render: function() {
    return(
      <View style={styles.webview}>
        <WebView
          url={constants.url}
          onNavigationStateChange={this.onNavigationStateChange}
          javaScriptEnabledAndroid={true}>
        </WebView>
      </View>
    );
  }

});

/*
 *  Styling
 */

var styles = StyleSheet.create({
  webview: {
    flex: 1,
    alignItems: 'stretch',
    backgroundColor: '#f00'
  },
});

module.exports = LoginWebView;

@ecompton3 WebView component is not a part of initial android release. As we have implementation of that component internally (we use it in AdsManager) we should be able to export it to github very soon

@ecompton3 - list of upcoming components for Android: https://facebook.github.io/react-native/docs/known-issues.html#missing-modules-and-native-views

@brentvatne maybe point out in the opacity portion that pointerEvents:none could be used to stop touches?

@browniefed - done! https://github.com/facebook/react-native/commit/4fd0a2de0d39107103470e40ad38cdb501acd926

@brentvatne I had seen that initially but then was confused when I saw the WebView.android.js file that exists in the current release of react native. Perhaps it should be removed until the native module for it exists?

Fair point @ecompton3! cc @kmagiera @mkonicek

WebView.android.js is required to build JS bundles that use WebView on iOS. Otherwise packager will pick ios.js file which is also not what we want.
@mkonicek We should prehaps replace WebView.android with UnimplementedView

@kmagiera could you tell me when the implementation will be published? :eyes:

+1 on the ETA for implementation. My team has an app that requires a WebView for OAuth 2.0 and would love to get an Android app going soon

SO there is actually no WebView support currently.. feel like April 1st

+1 for ETA as well.

On this list the highly requested ones are maps, intent and webview. I'll be away for two weeks preparing for a conference, and there are many other high-pri things to work on. Will work on open sourcing more view once I'm back though.

In the meantime you can look for 3rd party modules at https://react.parts/native

To give more context, the React Native for Android project was started much later than iOS. The plan is to eventually catch up on feature parity but it is not realistic to expect everything that works on iOS will work out of the box on Android, as explained in Known Issues.

I guess we will have to be patient! Would also be helpful to see the "iOS namespace" on WebView in the docs, to make it apparent it's not available on Android yet: WebViewIOS

+1, I'm waiting this impatiently, anyway kudos this is really great project! anyway for the time being if anyone need map solution mapbox already has support for map on android,
https://github.com/mapbox/react-native-mapbox-gl

Any ETA for this?

+1 on ETA :)

Maybe we should wait at https://github.com/facebook/react-native/issues/3796

@mkonicek do you think WebView.android.js will be available by Jan 2016, just to set a time frame?

Currently we're using react-native-webview-android and it runs good. Anyway, we plan to do the upgrade without issues with official react-native webview for android release.


Reproducible tarball build

Within the context of packaging, ideally anything that is fetched should have its integrity checked. In most cases, this involves comparing the checksum of the file to what's expected. In the case of tarball, there's reasonable expectation that it should be reproducible, as it involves fetching source-code exactly to the commit as defined in eng/Versions.Details.xml. Unfortunately, this is not the case, as the produced tarball contains many volatile files:

./src/*/.git directory in each product repo
./src/*/.git/modules in each product repo
./git-info vis-a-vis OfficialBuildId for certain products

Within the context of producing the Alpine aport for dotnet6, I wanted the ability to integrity check the tarball and cache it. This was done using the following patch:
From 518e05e8b4cd823c2c7365f46178322331206416 Mon Sep 17 00:00:00 2001
From: Antoine Martin<EMAIL_ADDRESS>Date: Wed, 16 Feb 2022 15:59:42 +0000
Subject: [PATCH 1/1] reproducible tarball

---
 .../src/Tarball_WriteSourceRepoProperties.cs  |  6 ++---
 .../tools/SourceBuildArcadeTarball.targets    |  8 +++----
 .../SourceBuildArcadeTarball_strip-git.sh     | 24 +++++++++++++++++++
 3 files changed, 31 insertions(+), 7 deletions(-)
 create mode 100755 src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball_strip-git.sh

diff --git a/src/SourceBuild/Arcade/src/Tarball_WriteSourceRepoProperties.cs b/src/SourceBuild/Arcade/src/Tarball_WriteSourceRepoProperties.cs
index b6b2793f3..55d680971 100644
--- a/src/SourceBuild/Arcade/src/Tarball_WriteSourceRepoProperties.cs
+++ b/src/SourceBuild/Arcade/src/Tarball_WriteSourceRepoProperties.cs
@@ -105,12 +105,12 @@ private static DerivedVersion GetVersionInfo(string version, string commitCount)
                 if (releaseParts.Length == 2)
                 {
                     // NuGet does this - arbitrary build IDs
-                    return new DerivedVersion { OfficialBuildId = DateTime.Now.ToString("yyyyMMdd.1"), PreReleaseVersionLabel = releaseParts[0] };
+                    return new DerivedVersion { OfficialBuildId = DateTime.Now.ToString("19900101.1"), PreReleaseVersionLabel = releaseParts[0] };
                 }
                 else if (releaseParts.Length == 3)
                 {
                     // VSTest uses full dates for the first part of their preview build numbers
-                    if (DateTime.TryParseExact(releaseParts[1], "yyyyMMdd", new CultureInfo("en-US"), DateTimeStyles.AssumeLocal, out DateTime fullDate))
+                    if (DateTime.TryParseExact(releaseParts[1], "19900101", new CultureInfo("en-US"), DateTimeStyles.AssumeLocal, out DateTime fullDate))
                     {
                         return new DerivedVersion { OfficialBuildId = $"{releaseParts[1]}.{releaseParts[2]}", PreReleaseVersionLabel = releaseParts[0] };
                     }
@@ -139,7 +139,7 @@ private static DerivedVersion GetVersionInfo(string version, string commitCount)
             {
                 // finalized version number (x.y.z) - probably not our code
                 // VSTest, Application Insights, Newtonsoft.Json do this
-                return new DerivedVersion { OfficialBuildId = DateTime.Now.ToString("yyyyMMdd.1"), PreReleaseVersionLabel = string.Empty };
+                return new DerivedVersion { OfficialBuildId = DateTime.Now.ToString("19900101.1"), PreReleaseVersionLabel = string.Empty };
             }

             throw new FormatException($"Can't derive a build ID from version {version} (commit count {commitCount}, release {string.Join(";", nugetVersion.Release.Split('-', '.'))})");
diff --git a/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball.targets b/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball.targets
index aaf297ac0..d50d4a63c 100644
--- a/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball.targets
+++ b/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball.targets
@@ -15,6 +15,7 @@
   <PropertyGroup>
     <ArtifactsDir>$(RepoRoot)artifacts/</ArtifactsDir>
     <TarballDir>$(ArtifactsDir)tarball/</TarballDir>
+    <TarballNameDir>$([MSBuild]::MakeRelative($([System.IO.Path]::GetDirectoryName($(TarballDir))), $(TarballDir)))</TarballNameDir>
     <TarballRootDir>$([MSBuild]::EnsureTrailingSlash('$(TarballDir)'))</TarballRootDir>
     <TarballSourceDir>$(TarballRootDir)src/</TarballSourceDir>
     <TarballGitInfoDir>$(TarballRootDir)git-info/</TarballGitInfoDir>
@@ -42,7 +43,7 @@
     <Error Text="TarballFilePath not specified." Condition=" '$(TarballFilePath)' == '' " />

     <MakeDir Directories="$(TarballFileDir)" Condition=" '$(TarballFileDir)' != '' " />
-    <Exec Command="tar --numeric-owner -zcf $(TarballFilePath) -C $(TarballDir) ." />
+    <Exec Command="tar --format=gnu --sort=name --owner=0 --group=0 --numeric-owner --mtime='1990-01-01Z' -zcf $(TarballFilePath) -C $(TarballDir)/.. $(TarballNameDir)" />

     <Message Text="Tarball '$(TarballFilePath)' was successfully created from '$(TarballDir)'" Importance="High" />
   </Target>
@@ -175,10 +176,10 @@
       Command="git submodule update --init --recursive --depth 1"
       WorkingDirectory="$(TarballRepoSourceDir)" />

-    <!-- Remove the git objects folder to free up tarball space -->
+    <!-- Strips git directories / submodules to bare minimum for reproducibility -->
     <Exec
-      Command="rm -rf objects"
-      WorkingDirectory="$(TarballRepoSourceDir).git"
+      Command="$(RepoRoot)/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball_strip-git.sh $(SourceDir)"
+      WorkingDirectory="$(TarballSourceDir)"
       Condition="$(PreserveTarballGitFolders) != 'true'" />

     <Message Text="--> Done Cloning Repo $(SourceBuildRepoName)" Importance="High" />
diff --git a/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball_strip-git.sh b/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball_strip-git.sh
new file mode 100755
index 000000000..51a4ed842
--- /dev/null
+++ b/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball_strip-git.sh
@@ -0,0 +1,24 @@
+#!/bin/bash
+repodir="$1"
+for i in $(find $repodir -name '.git' | sort -r); do
+	if [ -f "$i" ]; then
+		gitdir="${i/\/.git}/$(sed 's/gitdir: //' $i)"
+		commit=$(cat "$gitdir"/HEAD)
+	else
+		gitdir=$i
+		commit=${i%/.*}
+		commit=${commit##*.}
+	fi
+
+	url="$(grep -m 1 url "$gitdir"/config)"
+	url=${url#*= }
+
+	rm -rf "$i"
+	mkdir -p "$i"
+
+	echo $commit > "$i"/HEAD
+	{
+		echo "[remote \"origin\"]"
+		echo -n "url = \"$url\""
+	} > "$i"/config
+done
--
2.34.1

By default, SourceBuildArcadeTarball.targets only deleted .git/objects to save on space. This patch pushes the deletion a step further to the bare minimum. I defined this minimum as what is required by build and by git apply. This turns out to be the existence of .git/HEAD with commit and .git/config with remote url defined. This operation also needs to apply to git submodules. So during tarball build, instead of executing rm .git/objects, a script is executed that looks for all .git files / directories, parses the required information (commit + remote url), then deletes those files / directories and replaces them with a bare minimum git folder.
Another issue was that certain repos have volatile OfficialBuildId as they are generated based on tarball generation date. Thus, the code that generates those volatile build id now sets those to a generic date. In this case: 19900101.1. This is a problem, as we then need to change it back before build. Alpine aport compensates by setting these back to build date executing sed -i "s|<OfficialBuildId>19900101.1</OfficialBuildId>|<OfficialBuildId>$(date +"%Y%m%d.1")</OfficialBuildId>|" "$i" within a for loop. A better solution should be implemented.
Not all is done, though, as reproducibility also needs to happen at the compression level, hence the use of <Exec Command="tar --format=gnu --sort=name --owner=0 --group=0 --numeric-owner --mtime='1990-01-01Z' -zcf $(TarballFilePath) -C $(TarballDir)/.. $(TarballNameDir)" />. Inclusion of TarballNameDir is in there, as I wanted the tarball to extract, by default, in a subdir rather than on ./
I'm opening a discussion on this, as I'd love a better solution to the OfficialBuildId issue, and see if there's a broader appreciation for this.
Made as part of Alpine Linux dotnet6 packaging project, see dotnet/source-build#2782

[Triage] Putting this on the backlog.  Full reproducibility will take a sizeable effort.
When implementing, investigate the use of the SOURCE_DATE_EPOCH env rather than utilizing a hard coded timestamp.

@ayakael - I apologize, my previous comment was not formatted correctly.  What I saying is that we should consider an implementation that supports/utilizes https://reproducible-builds.org/docs/source-date-epoch/.  This will require some changes to the source-build infra.

Oh, I see! I agree, that would make the most sense!

Made as part of Alpine Linux dotnet6 packaging project, see #2782

Also, indeed, this makes only the first tarball build reproducible. I didn't target tarball build after full build of dotnet with prebuilts included.

I didn't target tarball build after full build of dotnet with prebuilts included.

Should any prebuilts be included in the second tarball? The plan upstream is to release "official" source tarballs and they dont contain any prebuilts: https://github.com/dotnet/source-build/issues/2301

Of note: the above patch broke submodules, breaking source-link. The following patch fixes this, based on what is expected by source-link per https://github.com/dotnet/sourcelink/blob/main/docs/README.md:
From 3669424e83ce393e2e0ee3f23b5a255c3a11a192 Mon Sep 17 00:00:00 2001
From: Antoine Martin<EMAIL_ADDRESS>Date: Wed, 16 Feb 2022 15:59:42 +0000
Subject: [PATCH 1/1] reproducible tarball

By default, tarball is not reprodicible due to a select few of volatile
files. This volatility is associated with .git directories and
OfficialBuildId of a few packages that source it from tarball build
date. This patch allows stripping of git directories to bare minimum and
sources OfficialBuildId with generic date to be modified later for
build.

---
 .../src/Tarball_WriteSourceRepoProperties.cs  |  6 +--
 .../tools/SourceBuildArcadeTarball.targets    |  9 +++--
 .../SourceBuildArcadeTarball_strip-git.sh     | 38 +++++++++++++++++++
 3 files changed, 46 insertions(+), 7 deletions(-)
 create mode 100755 src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball_strip-git.sh

diff --git a/src/installer/src/SourceBuild/Arcade/src/Tarball_WriteSourceRepoProperties.cs b/src/installer/src/SourceBuild/Arcade/src/Tarball_WriteSourceRepoProperties.cs
index b6b2793f3..55d680971 100644
--- a/src/installer/src/SourceBuild/Arcade/src/Tarball_WriteSourceRepoProperties.cs
+++ b/src/installer/src/SourceBuild/Arcade/src/Tarball_WriteSourceRepoProperties.cs
@@ -106,7 +106,7 @@ private static DerivedVersion GetVersionInfo(string repoName, string version, st
                 if (repoName.Contains("nuget"))
                 {
                     // NuGet does this - arbitrary build IDs
-                    return new DerivedVersion { OfficialBuildId = DateTime.Now.ToString("yyyyMMdd.1"), PreReleaseVersionLabel = releaseParts[0] };
+                    return new DerivedVersion { OfficialBuildId = DateTime.Now.ToString("19900101.1"), PreReleaseVersionLabel = releaseParts[0] };
                 }
                 else if (releaseParts.Length == 3)
                 {
@@ -140,7 +140,7 @@ private static DerivedVersion GetVersionInfo(string repoName, string version, st
             {
                 // finalized version number (x.y.z) - probably not our code
                 // Application Insights, Newtonsoft.Json do this
-                return new DerivedVersion { OfficialBuildId = DateTime.Now.ToString("yyyyMMdd.1"), PreReleaseVersionLabel = string.Empty };
+                return new DerivedVersion { OfficialBuildId = DateTime.Now.ToString("19900101.1"), PreReleaseVersionLabel = string.Empty };
             }

             throw new FormatException($"Can't derive a build ID from version {version} (commit count {commitCount}, release {string.Join(";", nugetVersion.Release.Split('-', '.'))})");
diff --git a/src/installer/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball.targets b/src/installer/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball.targets
index a516a86cf..eb5858ba3 100644
--- a/src/installer/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball.targets
+++ b/src/installer/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball.targets
@@ -15,6 +15,7 @@
   <PropertyGroup>
     <ArtifactsDir>$(RepoRoot)artifacts/</ArtifactsDir>
     <TarballDir>$(ArtifactsDir)tarball/</TarballDir>
+    <TarballNameDir>$([MSBuild]::MakeRelative($([System.IO.Path]::GetDirectoryName($(TarballDir))), $(TarballDir)))</TarballNameDir>
     <TarballRootDir>$([MSBuild]::EnsureTrailingSlash('$(TarballDir)'))</TarballRootDir>
     <TarballSourceDir>$(TarballRootDir)src/</TarballSourceDir>
     <TarballGitInfoDir>$(TarballRootDir)git-info/</TarballGitInfoDir>
@@ -42,7 +43,7 @@
     <Error Text="TarballFilePath not specified." Condition=" '$(TarballFilePath)' == '' " />

     <MakeDir Directories="$(TarballFileDir)" Condition=" '$(TarballFileDir)' != '' " />
-    <Exec Command="tar --numeric-owner -zcf $(TarballFilePath) -C $(TarballDir) ." />
+    <Exec Command="tar --format=gnu --sort=name --owner=0 --group=0 --numeric-owner --mtime='1990-01-01Z' -zcf $(TarballFilePath) -C $(TarballDir)/.. $(TarballNameDir)" />

     <Message Text="Tarball '$(TarballFilePath)' was successfully created from '$(TarballDir)'" Importance="High" />
   </Target>
@@ -189,10 +190,10 @@
       Command="git config --file $(TarballRepoSourceDir)/.git/config --add remote.origin.url $(OriginalRepoUri)"
       WorkingDirectory="$(RepoRoot)"/>

-    <!-- Remove the git objects folder to free up tarball space -->
+    <!-- Strips git directories / submodules to bare minimum for reproducibility -->
     <Exec
-      Command="rm -rf objects"
-      WorkingDirectory="$(TarballRepoSourceDir).git"
+      Command="$(RepoRoot)/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball_strip-git.sh $(SourceDir)"
+      WorkingDirectory="$(TarballSourceDir)"
       Condition="$(PreserveTarballGitFolders) != 'true'" />

     <Message Text="--> Done Cloning Repo $(SourceBuildRepoName)" Importance="High" />
diff --git a/src/installer/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball_strip-git.sh b/src/installer/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball_strip-git.sh
new file mode 100755
index 000000000..63858732f
--- /dev/null
+++ b/src/installer/src/SourceBuild/Arcade/tools/SourceBuildArcadeTarball_strip-git.sh
@@ -0,0 +1,38 @@
+#!/bin/bash
+repodir="$1"
+
+gitdir="$repodir"/.git
+stripdir="$repodir"/.strip
+mkdir "$stripdir"
+
+commit=$(git -C "$repodir" rev-parse HEAD)
+url="$(grep -m 1 url "$gitdir"/config)"
+url=${url#*= }
+
+echo $commit > "$stripdir"/HEAD
+{
+echo "[remote \"origin\"]"
+echo "	url = \"$url\""
+} > "$stripdir"/config
+
+if [ -f "$repodir"/.gitmodules ]; then
+	cat "$repodir"/.gitmodules >> "$stripdir"/config
+
+	for subdir in $(find "$gitdir"/modules  -type f -name 'config'); do
+		subdir=${subdir/\/config}
+		substripdir="${subdir/.git/.strip}"
+		commit=$(git -C "$subdir" rev-parse HEAD)
+		url="$(grep -m 1 url "$subdir"/config)"
+		url=${url#*= }
+
+		mkdir -p "$substripdir"
+		echo $commit > "$substripdir"/HEAD
+		{
+		echo "[remote \"origin\"]"
+		echo "	url = \"$url\""
+		} > "$substripdir"/config
+	done
+fi
+
+rm -fR "$gitdir"
+mv "$stripdir" "$gitdir"

This problem goes way in .NET 8 with the VMR. A git archive from the VMR always makes a reproducible tarball.

Closing now VMR is implemented with .NET 8


HTTPS connection - Minor bug

Minor bug in django-material/material/static/material/css/robotofix.css when django-material is used with an https connection. Maybe, you should change:
"src: url(http://themes.googleusercontent.com/static/fonts/roboto/v10..."
by
"src: url(//themes.googleusercontent.com/static/fonts/roboto/v10..."
Great job.

Generally, i'm thinking about to drop that hack, since materialze.css now have woff2 roboto font.
If you on windows, could you please check http://materializecss.com/ website?
Do you have a font rendering issues like that:
http://stackoverflow.com/questions/31011278/roboto-from-materializecss-weird-font-rendering-in-chrome-firefox-ok-with-ie


How to configure html2pdf in Angular 16?

How to configure html2pdf in Angular 16?
Step 1: Install html2pdf.js
npm install html2pdf.js

Step 2: Configure angular.json
"scripts": [
              "./node_modules/html2pdf.js/dist/html2pdf.bundle.min.js"
            ]

Step 3: Declare html2pdf as any directly in your component.
declare let html2pdf: any;

Although it is a quick solution that will avoid compilation errors, it is not efficient, as it loses type checking at development time.
Is there a way to do it by import html2pdf from 'html2pdf.js'?

@cortaza123. You can do something like this:
import * as html2pdf from 'html2pdf.js';


Update x/tools

Fixes this problem:
% ~/go/bin/errchkjson ./...
errchkjson: internal error: package "fmt" without types was imported from "(some package)"

https://github.com/golang/go/issues/37617

Can you please run ´go mod tidy´.

done

@breml would you mind tagging a new version so that golangci-lint can update this linter in their next release please?

done


[Enum Request] Table "Electronic Document Format" - Field Usage

Hello there,
Could you please change the Usage field to Enum.
We would like to have possibility to prepare other document types as electronic invoice e.g.: Finance Charge Memo
Thanks.
BR,
Bojan

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


overloaded reinterpreter API

The content shows the low-level API in overloaded_intrinsic_funcs.md rather than the overloaded API.

Sorry, the title is confusing. Do you mean low-level API should not show in overloaded_intrinsic_funcs.md?

Now, it shows the low-level functions
vint16m1_t vreinterpret_v_f16m1_i16m1(vfloat16m1_t src);
rather than
vint16m1_t vreinterpret_i16(vfloat16m1_t src);

BTW,
Overloaded APIs for vreinterpret are not generated in the following commit.
[main] 8f683366afcfd7c03faea615d1529a0014d7dbde - commit 4248 of 389948                                   1%
commit 8f683366afcfd7c03faea615d1529a0014d7dbde
Refs: llvmorg-13-init-7485-g8f683366afcf
Author:     Zakk Chen<EMAIL_ADDRESS>AuthorDate: Mon Apr 12 19:01:44 2021 -0700
Commit:     Zakk Chen<EMAIL_ADDRESS>CommitDate: Fri Apr 16 09:41:19 2021 -0700
[RISCV][Clang] Add RVV miscellaneous intrinsic functions.

1. vreinterpret
2. vundefined
3. LMUL truncation and extension.

Reviewed By: craig.topper

Differential Revision: https://reviews.llvm.org/D100391

Yes, currently overloaded document also shows the low-level APIs.
I think it's more convenient for users, they can easily understand  which intrinsic functions have no overloading supported.
so maybe they can understand that all miscellaneous intrinsic functions have no overloading supported now.
maybe we need to add more description for that in somewhere.

I may misunderstand your comment.
Do you mean this document won't show those unimplemented instructions?
If yes, why this document shows all unimplemented pseudo vector instructions? such as 'vfabs' ?

Do you have any plan to implement pseudo vector instructions at the intrinsic level?
vwcvtu.x.x.v, vnot.v, vneg.v, vncvt.x.x.w, vmsgeu.vx, vmsgeu.vv, vmset.m, vmnot.m, vmmv.m, vmclr.m, vfneg.v, vfabs.v
Those instructions are documented, but not implemented on LLVM yet.

I believe LLVM trunk had supported all pseudo vector instructions.
please see https://godbolt.org/z/onfzPv3cE.

about zfh, we need have more discussion about which type we should use for fp16:
https://github.com/riscv/rvv-intrinsic-doc/issues/18#issuecomment-818472454

about the overloading vreinterpret, I will update the document and support overloading api later.
Thanks for point out.

I believe LLVM trunk had supported all pseudo vector instructions.
please see https://godbolt.org/z/onfzPv3cE.

about zfh, we need have more discussion about which type we should use for fp16:
https://github.com/riscv/rvv-intrinsic-doc/issues/18#issuecomment-818472454

about the overloading vreinterpret, I will update the document and support overloading api later.
Thanks for point out.

Thanks for all feedback.


JuliaGeo in a Docker Container

Before I start building my own, is there a Docker Container containing all the JuliaGeo libraries?

Not to my knowledge, no. There are general julia ones over at https://hub.docker.com/_/julia. Just out of interest, what is your intended use case? I know some people like to combine docker with https://github.com/JuliaLang/PackageCompiler.jl/ to get containerized applications that start quickly.

Thanks for the advice, I'll definitely check it out!
In regard to your question about my use case: I come from R/python and have made it a habit to work in docker containers. E.g. for geospatialstuff in R, I build on top of rocker/geospatial, which is a one stop shop for all things geospatial in R. I was looking for something similar to get started in Julia.


Extract compression logic into Compressor

Wraps the compression settings in an object that knows how to compress entire message sets.

@bquorning does this make you all warm and fuzzy on the inside?

Nice cleanup! :heart:
:+1: apart from a single comment.


feat: add all new Leaderboard apis

Addresses #468, adds all the new Leaderboard API implementations all at once.
Currently targeted against the feature branch.

@virratanasangpunth @tylerburdsall
Heads up about the GetByScore and GetRank tests that are failing (above). Going to open a PR to merge Leaderboard stuff into main after merging this one. Should I wait to merge that upcoming PR until all integration tests pass?

Should be a different leaderboard name for each describe suite of tests. I used const leaderboardName = "leaderboard-${v4()}"; at the beginning of each describe block.


package.json version doesn't match github one

@jrit maybe a 6.0.2 release with the matching version in package.json?  Unless you wanna delete and re-add the 6.0.1 with the update.  Maybe that.

resolved

note it was 1.0.1 -> 1.0.2, not version 6

Er... yes.  That.


Improve WAST error messaging, NaN comparisons

This should be helpful as we investigate https://github.com/bytecodealliance/wasmtime/issues/2432. Please note that the hex-ifying of the assertion expressions and Wasm values is pretty confusing: scalars are shown as big endian whereas vectors are shown as little endian (I tried to describe why in the comments). I would appreciate a careful look to ensure I did this correctly and any suggestions to reduce the verbosity would be great!

No idea what the trouble is with CI. It's failing during submodule checkout here?

I was seeing that earlier too; after a few restarts the jobs went through. I had assumed it was some sort of spurious-failed-HTTP-request error. I'll hit the 're-run jobs' button for these...


Upgrade to Laravel 8

I want to upgrade Laravel 7 to 8; But I got the error about slug package version.
Here is the upgrading documentations :
https://laravel.com/docs/master/upgrade
Please upgrade the package to Laravel 8.
Thanks.

yes please

+1

Yes please

still not working, please update, we appreciate it

There are two PRs for this already. We'll just need to watch the status of these:
#524
#525

Version 8.0 is available now.

Thanks a ton for your work @cviebrock!

Thanks a lot @cviebrock !

Thanks a lot @cviebrock


add random commands

This PR related to #242
TODO

[x] random
[x] bool
chars
decimal
dice
integer

@arthur-targaryen
https://github.com/nushell/engine-q/issues/242#issuecomment-978330232
Sorry to see your comment late! As you said, I open a PR for the progress so far.
The other commands will make PR individually as they progress!
Thanks :)

Looks good!


RRD Defect matrix automation

/kind feature
/area testing
Develop scripts to pull issues (bugs) from GitHub into Google sheets to generate defect metrics for:

[ ] Defect backlog [WIP]
[ ] Defect regression
[ ] Defects resolution
[ ] Defects reopen

Related issue: https://github.com/openshift/odo/issues/4649

Scope of the issue:
Write google sheet scripting for api uri satisfy

[ ] Defect backlog [WIP] (Difference between number of issue opened and closed in a sprint)

[ ] Defect regression (Issue count satisfying is:issue label:area/regression label:kind/bug  created:s_date..e_date )

[ ] Defects resolution (Issue count satisfying)
-  is:issue label:kind/bug label:triage/unresolved
- is:issue label:kind/bug is:closed
- is:issue label:kind/bug label:triage/duplicate
- is:issue label:kind/bug label:triage/not-reproducible

[ ] Defects reopen

is:issue label:kind/bug label:triage/unresolved

Maybe consider issues that got closed with lifecycle/rotten.

Acceptance criteria has been met.
/close


[BUG] VIACOM says the carrier "is not a supercarrier" when taking off from a field and landing on the supercarrier

When calling "marking moms" on a carrier frequency after taking off from a field, the call fails and an error message stating that the contact "is not a supercarrier" is logged. Carrier communications must be handled using the keyboard comm menus in this scenario.

Do you have escort ships with Helo deck in your carrier group that have the same frequency? If so select tunes radio maybe contacting that ship. As mission design habit I change them to another frequency to avoid these sort of issues.

I indeed have an helipad boat in the carrier group. I'll test changing the frequency, thanks!

It indeed solves the issue to change the frequency of the other reachable boats in the carrier group.


Connection through Pgpool not working

What language are you using?.
Python3.9
What version are you using?
0.3.2
What database are you using?
PostgreSQL
What dataframe are you using?
Pandas
Can you describe your bug?
Connection through pgpool to postgresql instance results with "RuntimeError: connection closed". Direct connection works as intended.
What are the steps to reproduce the behavior?
Example from "Getting started" section of the connectorx package and docker-compose with pgpool
version: '3.8'
services:
  pgprimary:
    image: bitnami/postgresql:13.1.0
    ports:
      - 5432
    volumes:
      - pgprimary_data:/bitnami/postgresql
    environment:
      - POSTGRESQL_REPLICATION_MODE=master
      - POSTGRESQL_REPLICATION_USER=postgres
      - POSTGRESQL_REPLICATION_PASSWORD=postgres
      - POSTGRESQL_PASSWORD=postgres
      - POSTGRESQL_DATABASE=test_db
  pgreplica:
    image: bitnami/postgresql:13.1.0
    ports:
      - 5432
    depends_on:
      - pgprimary
    environment:
      - POSTGRESQL_REPLICATION_MODE=slave
      - POSTGRESQL_REPLICATION_USER=postgres
      - POSTGRESQL_REPLICATION_PASSWORD=postgres
      - POSTGRESQL_MASTER_HOST=pgprimary
      - POSTGRESQL_PASSWORD=postgres
  pgpool:
    image: bitnami/pgpool:4.2.1
    ports:
      - 5432:5432
    depends_on:
      - pgprimary
    environment:
      - PGPOOL_BACKEND_NODES=0:pgprimary:5432:4:primary:ALWAYS_PRIMARY,1:pgreplica:5432:6:replica
      - PGPOOL_POSTGRES_USERNAME=postgres
      - PGPOOL_POSTGRES_PASSWORD=postgres
      - PGPOOL_ADMIN_USERNAME=admin
      - PGPOOL_ADMIN_PASSWORD=postgres
      - PGPOOL_ENABLE_LOAD_BALANCING=yes
      - PGPOOL_ENABLE_STATEMENT_LOAD_BALANCING=yes
      - PGPOOL_NUM_INIT_CHILDREN=10
      - PGPOOL_MAX_POOL=1
      - PGPOOL_SR_CHECK_USER=postgres
      - PGPOOL_SR_CHECK_PASSWORD=postgres
volumes:
  pgprimary_data:

Database setup if the error only happens on specific data or data type

Example query / code
import connectorx as cx

cx.read_sql("postgresql://postgres:postgres@<IP_ADDRESS>:5432/postgres?sslmode=disable", "select * from information_schema.tables")

What is the error?
Traceback (most recent call last):
File "/home/mikhail/sandbox/py/test.py", line 3, in
cx.read_sql("postgresql://postgres:postgres@<IP_ADDRESS>:5432/postgres?sslmode=disable", "select * from information_schema.tables")
File "/home/mikhail/sandbox/py/.venv/lib/python3.9/site-packages/connectorx/init.py", line 264, in read_sql
result = _read_sql(
RuntimeError: connection closed

Seems like your database name is test_db instead of postgres. Can you try changing your connection string to postgresql://postgres:postgres@<IP_ADDRESS>:5432/test_db?sslmode=disable?

That's not the case, just a typo in the local stand yaml....
We have a Pgpool based cluster with data on a production server and we can access it with connectorx via normal route to the one of the replicas or master, but going through Pgpool is messed up. Feels like the driver laying behind connectorx is not waiting for query response and due to Pgpool rerouting mechanism connection closes before query result is acknowledged by the client...


[FLINK-3216] [FLINK-3217] [cep] Initial version of CEP library

This PR is the first version of Flink's CEP library.
The key components are the NFA which uses the SharedBuffer to efficiently maintain the state multiple non-deterministic runs. The NFA implementation is strongly based on this paper.
In order to define the NFA, the library supports the pattern API. The pattern API let's you easily construct complex patterns with type and filter conditions. The specified pattern is then compiled into a NFA which is responsible for detecting the patterns. See the online documentation for a full specification of the supported operations (docs/libs/cep/index.md).
In order to run the NFA, the library adds two custom stream operators: CEPPatternOperator and KeyedCEPPatternOperator. The former is used for non-keyed input streams and the latter for keyed data streams. The selection of the right operator is transparently done vie the CEP.from(input, pattern) method.
CEP.from(input, pattern) returns a PatternStream which contains the matched event sequences. The event sequences can be processed by specifying a PatternSelectFunction or a PatternFlatSelectFunction. Both methods receive the detected pattern as a Map<String, T> where T is the type of the input data stream. Each event is matched against a state of the pattern and the name of the state is the key of the map.
As it can be seen, the current CEP library is mostly a new stream operator which can be easily integrated into a normal DataStream program.
An example of the API can be seen next:
StreamExecutionEnvironment env = ...
env.setStreamTimeCharacteristic(TimeCharacteristic.EventTime);

DataStream<Event> input = ...

DataStream<Event> partitionedInput = input.keyBy(new KeySelector<Event, Integer>() {
	@Override
	public Integer getKey(Event value) throws Exception {
		return value.getId();
	}
});

Pattern<Event, ?> pattern = Pattern.<Event>begin("start")
	.next("middle").where(new FilterFunction<Event>() {
		@Override
		public boolean filter(Event value) throws Exception {
			return value.getName().equals("name");
		}
	}).followedBy("end").where(new FilterFunction<Event>() {
		@Override
		public boolean filter(Event value) throws Exception {
			return value.getName().equals("critical");
		}
	}).within(Time.seconds(10));

PatternStream<Event> patternStream = CEP.from(partitionedInput, pattern);

DataStream<Alert> alerts = patternStream.select(new PatternSelectFunction<Event, Alert>() {
	@Override
	public Alert select(Map<String, Event> pattern) throws Exception {
		return new Alert(pattern.get("start"), pattern.get("end"))
	}
});

Super nice!
Has documentation, tests, JavaDocs. Very good all in all!
The builds fail on Scala 2.11 build profiles in different maven projects. Errors look like there may be mixups with Scala versions. However, I cannot see how this pull request could have caused that...
If the Scala errors can be resolved with a rebase to the latest master, +1 from my side

The Scala problem seems to be related to missing suffixes in the pom file. I fixed it. Once travis gives green light, I'll merge the PR.

Failing test case is unrelated to changes.


caller id

How can I get caller id before accepting the call?
Sorry, I didn't find solution in docs, guides and during quick overview the source

I'm trying to get the displayName that I passed throught config object, like this:
var config = {             displayName: 'myDisplayName',     }
Then, try to get it with:
session.remoteIdentity.displayName
but it show me the extension, not config's displayName property.
What I am  doing wrong?

Hi Luis,
session.remoteIdentity.displayName should match exactly the display portion of From address in the received INVITE request, or for outbound calls, the To address of the sent INVITE.  If there is no display portion, I believe it will be set to the the user portion of those addresses.
You haven't provided a lot of details about your scenario, but please use the traceSip option to verify that the above behavior is accurate.  It's possible that you are placing an outbound call and there is no caller ID, or that a server in the middle of the call (like Asterisk or FreeSWITCH) is replacing it with the extension.  Alternatively, if you are looking for the local endpoint's caller ID, you might want to check session.localIdentity.displayName.
Until this is shown to be a bug in SIP.js, we'd prefer to keep the discussion on the mailing list, reserving GitHub for project issues rather than support.  Please create a post there for further help.
Thanks,
-Will

Ah thanks. I will not happen again.
Btw, I didn't know freeswitch could change the name. I will check the traceSip and the freeswitch.

Yup.  FreeSwitch is a back to back user agent.  When you put it between two WebRTC endpoints, it looks like they are talking to each other, but really FreeSwitch is answering one call and creating another.  The call you receive at Point B is completely different than the one sent from Point A.

I resolved this with session.remoteIdentity.uri.user


adding check test for isalpha

Helping out community user @Howe2015 regarding a plattform issue
https://github.com/KjellKod/g3log/issues/407

[x] TDD

New/modified code must be backed down with unit test - preferably TDD style development)

[N/A] Documentation

All new/modified functionality should be backed up with API documentation (API.markdown or README.markdown)
Cross-Platform Testing

[x] Travis-CI (Linux, OSX) + AppVeyor-CI (Windows)\
[X] Optional: Local/VM testing: OSX

ref: https://github.com/KjellKod/g3log/issues/407

Closing this.
std::isalpha isn't recognized on Windows in spite of the right header being included
Instead it defaults to <locale> isalpha which has a different API
error C2672: 'std::isalpha': no matching overloaded function found
 'bool std::isalpha(_Elem,const std::locale &)': expects 2 arguments - 1 provided


Queries Never Finish

Issue Type: Bug
Executing a simple query never finishes. Even after returning a result grid, the timer keeps ticking, and the "Run" button never re-activates.
I noticed this after running a query, doing something else, then coming back 2 hours later and seeing the timer ticking away.
Cancel button does not stop query. Have to exit ADS to stop.

Azure Data Studio version: azuredatastudio-insiders 1.42.0-insider (1e8519d54a6507504bfdbec4faa3082648609455, 2023-02-27T06:50:07.267Z)
OS version: Windows_NT x64 10.0.22621
Restricted Mode: No
Preview Features: Enabled

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8500 CPU @ 3.00GHz (6 x 3000)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
31.78GB (14.02GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

dacpac
Mic
1.13.0

import
Mic
1.5.4

schema-compare
Mic
1.18.0

sql-database-projects
Mic
0.22.0

sql-search
Red
0.4.0

ADS-add-columns
sof
0.0.12

vscodeintellicode
Vis
1.2.1

poor-sql-formatter
WSR
0.1.0

Thanks for reporting. this looks to be a dupe of #7440

I'm not so sure it's the same. It just started down by this with the latest insiders. And the query completes (complete results are shown) but the timer keeps running and the Run button never reactivates.

I can reproduce the issue now. Thanks, Will investigate.

@corivera , this issue is introduced by https://github.com/microsoft/azuredatastudio/pull/21995, the onQueryComplete notification is now being processed by the new controller you added

please treat this as high priority as the query editor feature is broken now.
Thanks
Alan

Sigh, yeah this looks like it's related to this long-standing issue with provider callbacks.
https://github.com/microsoft/sqlops-dataprotocolclient/issues/62
Fixing that would be great since this is something that very much limits the usefulness of the providers in general (and making it easy for extensions to completely break core behavior is pretty bad in general)

@Charles-Gagnon For query providers, wouldn't it be a problem if any extension can register a listener and get all the query outputs from the MSSQL provider? Or do we assume that an extension is trusted if the user installs it?

That's definitely a discussion we'll have to have - but that's something an extension can do currently anyways (it just breaks everything else) so something to keep in mind. But yeah, overall the general idea VS Code seems to go with is that if you install an extension you trust it so they have a lot of capabilities - for example reading any file in the workspace.

Yeah, it would make more sense if it behaves like other events where you can register multiple callbacks. Extensions could then route the outputs correctly using the editor URI.

dont want to pollute this but in the interim is there a version prior to this controller addition i can use in the interim? SSMS makes me wanna find a ledge it's been hard to live without ADS lol

@chris-larsen the fix is already in the latest insider build.


Inject CMP version from project version

Inject CMP version from project version
Set cmpVersion automatically using the first digit from the package.json version on build time:

remove cmpVersion from internals configuration file
receive cmpVersion from bootstrap
update readme
fix tests
include a webpack config file and use it to extract the major digit from package.json version
Simplify NPM tasks

Codecov Report

Merging #25 into feature/develop-v1.0.0 will increase coverage by <.01%.
The diff coverage is 100%.

@@                    Coverage Diff                     @@
##           feature/develop-v1.0.0      #25      +/-   ##
==========================================================
+ Coverage                   98.43%   98.43%   +<.01%
==========================================================
  Files                          29       29
  Lines                        1149     1152       +3
  Branches                      110      110
==========================================================
+ Hits                         1131     1134       +3
  Misses                         18       18

Impacted Files
Coverage Δ

src/cmp/infrastructure/configuration/internals.js
100% <100%> (ø)
:arrow_up:

src/cmp/infrastructure/cmpInitializer.js
100% <100%> (ø)
:arrow_up:

.../cmp/infrastructure/configuration/Configuration.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@victuxbb you are right, fixed.


[Snackbar] Convert to function component

[x] I have followed (at least) the PR section of the contributing guide.

Related to #15231

Don't forget we need to apply this fix: https://github.com/mui-org/material-ui/issues/15231#issuecomment-484006495

When I try converting the getDerivedStateFromProps method like
-static getDerivedStateFromProps(nextProps, prevState) {
-  if (typeof prevState.exited === 'undefined') {
-    return {
-      exited: !nextProps.open,
-    };
-  }
-
-  if (nextProps.open) {
-    return {
-      exited: false,
-    };
-  }
-
-  return null;
-}

+if (typeof exited === 'undefined') {
+  setExited(!open);
+} else if (open) {
+  setExited(false);
+}

I get an error like;

Too many re-renders. React limits the number of renders to prevent an infinite loop.

I looked into this a bit more 😅 and
if (typeof exited === 'undefined') {
  setExited(!open);
}

is called 7 times during the initial rendering time, & the although setExited(!open) is being called but the value of of exited  for those initial 7 renders is always undefined  💭 I am not sure why .. 😅

Don't forget we need to apply this fix: #15231 (comment)

🌮 💯

Don't forget we need to apply this fix: #15231 (comment)

🌮 💯

That fix means you no longer need getDerivedStateFromProps

That fix means you no longer need getDerivedStateFromProps

Yes I am looking into it 😄 😄 I didn't see your comment before I posted that long useless comment of mine 🏷 🤣

@joshwooding Can you kindly review, if you think this implementation is okay, should I move towards the tests?

@joshwooding Can you kindly review, if you think this implementation is okay, should I move towards the tests?

Looks good so far.

@adeelibr Well done :)


README jupyter notebook link broken

Describe the bug
The top-level README's link to example jupyter notebooks is broken.
To reproduce
Steps to reproduce the behavior:

Go to https://github.com/qctrl/python-open-controls#usage
Click the Jupyter notebooks link.

Hi Drew,
Thanks for the report! We recently made some changes to how all the Q-CTRL documentation is organised, and this slipped through the cracks. I'll work on migrating the appropriate content into this repo right away, but in the meantime you can see some example usage of the package (being used together with other Q-CTRL tools) at docs.q-ctrl.com.
In particular, take a look at the filter functions feature guide, which sets up several different driven controls from this package. You may also find the cold ions technology guide interesting—even though it only uses this package to generate a primitive control, the applications to ions may be relevant.
Harry


fields to return for verbose=True when finding studies or trees

Do we want the same fields as the current oti, or any added / deleted? Example of current oti returns:
{   "matched_studies" : [ {     "ot:studyPublicationReference" : "Tippery N., & Les D. 2011. Phylogenetic relationships and morphological evolution in Nymphoides (Menyanthaceae). Systematic Botany, 36: 1101-1113.",     "ot:curatorName" : "bryan drew",     "ot:studyId" : "pg_719",     "ot:studyYear" : 2011,     "ot:focalClade" : 694622,     "is_deprecated" : false,     "ot:focalCladeOTTTaxonName" : "Menyanthaceae",     "ot:dataDeposit" : "http://purl.org/phylo/treebase/phylows/study/TB2:S11079",     "ot:studyPublication" : "http://dx.doi.org/10.1600/036364411X605092"   } ] }
to parse this out, the fields are studyPublicationReference, curatorName, studyId, studyYear, focalClade, is_deprecated, focalCladeOTTTaxonName, dataDeposit, and studyPublication.

@jimallman Thoughts on this? Are there fields that we don't need for the current study list view, or fields that we should have but don't?

We show tags (ot:tag) in the expanded list view for a study, so we'll want to add that.
Most important is that we index and return multiple values for ot:curatorName and ot:tag, as described in the long-standing issue https://github.com/OpenTreeOfLife/oti/issues/35 . This will solve lots of problems and clean up UI glitches.

How do you get the list of ot:tag now? It isn't in the list of returned fields from find_studies?
The current ottreeindex code already handles multiple values, so that won't be an issue once we move over.

How do you get the list of ot:tag now? It isn't in the list of returned fields from find_studies?

The existing oti captures just one value, and returns it as a simple string, for example:
 {
  "ot:studyPublicationReference" : "Salmaki, Y., Zarre, S., Ryding, O., Lindqvist, C., Bräuchler, C., Heubl, G., ... & Bendiksby, M. (2013). Molecular phylogeny of tribe Stachydeae (Lamiaceae subfamily Lamioideae). Molecular phylogenetics and evolution.",
  "ot:curatorName" : "bryan drew",
  "ot:studyId" : "pg_2679",
  "ot:studyYear" : 2013,
  "ot:focalClade" : 534796,
  "is_deprecated" : false,
  "ot:focalCladeOTTTaxonName" : "Stachydeae",
  "ot:tag" : "ingroup added;"
}

Ah, I see. I just hadn't managed to find an example that has a tag, so I wasn't seeing that in the list of returned values.

Full list for ot:tag and ot:curator now included for find_studies where verbose=True. See commit


Implement necessary tooling for running the DKG

The DKG ceremony would still be coordinated by a dedicated Solidity smart contract. Provide a simple CLI interface to interact with the smart contract.

It may not be needed if running the DKG through Etherlink's kernel https://docs.etherlink.com/network/building-kernel/.


[grpc-c++] pick_first policy can end up in state where it is not watching channel connectivity

https://github.com/grpc/grpc/pull/26441 modified the pick_first policy to promote pending subchannel list to the active subchannel list if all subchannels in the pending list were attempted and unavailable. However when doing so selected_ is not cleared and thus points to now deleted memory (it was owned by the previous active subchannel list).
I am sending a PR to address this but am opening this issue to track it and add more details

Leaving the selected_ pointing to now deleted memory is generally not an issue except for one special case where the allocator reuses that for a new allocation of a PickFirstSubchannelData.  Allocators such as tcmalloc have freelists and caches so this is possible.
Then if PickFirst handles a resolution of address where the first address is such reallocated memory, and because selected_ is non-null and the pending list subchannels are all IDLE since they are new, it connects to the first subchannel of the pending list in the background.
Then when  PickFirst notices the state change of the subchannel from IDLE->CONNECTING. However the pointer equality check incorrectly interprets this notification as part of the current subchannel list not the pending list because of pointer equality with selected_. In that case, since it is not in the READY state it changes the pending list to the active list. However as it does so it cancels the connectivity watcher for selected_ (which was actually in the pending list). In this case we end up with selected_ now being valid (in that it points to allocated memory which is part of the active list), however we have lost the connectivity watcher for it.
Thus if the subchannel becomes connected, goes into an error etc it is unnoticed by the pick first policy and the channel remains in a CONNECTING state.  If using a resolver that requires requests to initiate resolution (ie c ares) then a resolution never occurs and the channel can remain in this state.


EF Core 1.1.2 Foreign Key Annotation for String Type doesn't generate Cascade delete

Describe what is not working as expected.
The following code doesn't produce cascade delete syntax during migration.
**

Missing for String type. Works for Int
.OnDelete(DeleteBehavior.Cascade);

**
Steps to reproduce
Create following model classes and perform migrations

public class CodeTerminationFlag
    {
        [Key]
        [Required]
        [StringLength(2)]
        public string TerminationFlagId { get; set; }

        [Required]
        [StringLength(25)]
        public string Name { get; set; }

        [StringLength(200)]
        public string Notes { get; set; }
    }
public class Activity
    {
        [Key]
        [Required]
        [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
        public int ActivityId { get; set;

        public CodeTerminationFlag CodeTerminationFlag { get; set; }

        [ForeignKey("CodeTerminationFlag")]
        public string TerminationFlagId { get; set; }

        [StringLength(250)]
        public string Notes { get; set; }

    }

**Output**

  b.HasOne("API.Data.Models.CodeTerminationFlag", "CodeTerminationFlag")
                        .WithMany()
                        .HasForeignKey("TerminationFlagId");

Further technical details
EF Core version: 1.1.2
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system:
MAC (VS Code)

Since string is a nullable type, it means the TerminationFlagId is nullable, which makes the relationship optional by default. Cascade delete is only on by convention for required relationships. To get cascade delete in this case, either make the relationship/FK required, or explicitly configure the cascade delete. See the docs here for more details: https://docs.microsoft.com/en-us/ef/core/modeling/relationships


Implement Modified Knapsack With repititon allowed

I want to implement this. Can I work on it?

@jenistenxavier


Add missing Docker Compose dependency to readme file

Docker Compose does not come installed with Docker. This change might save a few minutes of confusion and Google to some users willing to try the demo.

Thanks @blgo!


Consider if DataFrame Binary Ops and Computations need to return a copy

Returning a copy for every call is inefficient. However it may feel "natural" in a notebook. I'm opening this primarily to have a place to discuss this. Some points:

Pandas returns a copy, not that we necessarily have to keep the same semantics
If we operate in place, do we need to support an "undo" and/or "redo" of some sort? I'm not sure how "natural" this would feel in a real notebook. My initial thought is that if we store a command history, an undo is just an inverse op. For ex: Subtract is the inverse op for Add. I need to spend some more time to check if every op is invertible though.
Operating in place would obviously scale better.

Another option is to just have 2 clearly defined APIs, 1 for in memory and 1 that returns a copy.

What problems would be faced if a DataFrame stored operations rather than the changed states ?
Having both APIs where one returns a copy and the other not could help users with memory usage i think.


Toggle row events

Events fired when row toggle and toggleAll are called.

Allows you to have custom behavior when a row is clicked. Particularly useful if you have rows grouped by sub-heading. See #189.

I realize that it is your prerogative to accept or decline pull requests, but it would be nice to know why you made your decision.

v0.9.x has callbacks for when rows are selected or deselected. Also, using $scope.emit makes a reusable component less reusable. If you have multiple tables displayed at once, you may not no where the event is coming from.


Add Pycsou Framework trove-classifier

Request to add a new Trove classifier.
The name of the classifier(s) you would like to add:

Framework :: Pycsou

Why do you want to add this classifier?
The latest version of Pycsou, v2, introduces an exciting new feature for users: the ability to develop plugin extensions through setuptools entrypoints. To make it even more convenient for users, we're currently working on a catalogue website that will showcase these plugins. The website will utilize the trove classifier to automatically discover and display the available plugins, making it easy for users to find and implement the ones they need.

Hi! Thanks for the request. The PyPI moderators are in the process of developing a few guidelines for the creation of new Framework classifiers. Because removing classifiers is very difficult, and the longer the classifier list gets the harder it is to browse, I think we're expecting the bar to add new ones to be relatively high. Currently, we're looking at these guidelines:

New Framework classifiers should be created if they are "notable;" and
They will be of immediate use to existing projects (in other words, no creating classifiers simply in anticipation of future uses)

We understand that "notability" is subjective and we'd appreciate help judging that.
As to the other guideline ('immediate use to existing projects'), it's been suggested that

Requestors can demonstrate this by providing links to 10 or more existing PyPI projects that have expressed a desire to use the classifier; or
Another way to demonstrate this would be to have 10 or more other project maintainers submit comments on the requesting issue about their desire to use the classifier.

Thanks!

Hi @di !
Thank you for your feedback. Could you please provide further clarification on the process for interested maintainers to endorse this trove-classifier? For instance, would it be sufficient for ten developers to leave a comment on this pull-request to indicate their support?
Many thanks!

I endorse the proposal for a Pycsou trove-classifier on behalf of UncertaintyQuant, an uncertainty quantification software package built on Pycsou.

Hi there,
As a developer of Pycsou-GSP, a extension to Pycsou for graph signal processing, I fully support the introduction of a Pycsou trove-classifier on PyPI. This would not only benefit my software, but also
help other Pycsou-based projects gain more visibility and recognition.

Hello,
I am the main developer of HVOX, a novel radio-astronomy gridder based on the Heisenberg Voxelization method.
HVOX is strongly tied to the Pycsou framework, but our goal is to release it as an extension package since it is domain-specific software.
A dedicated framework classifier is beneficial to the community as it allows faster dissemination of novel ideas, while simultaneously easing use for beginners.
I also plan to submit another extension package to add ray-tracing operators to Pycsou.
I therefore endorse this proposal.

Good afternoon,
As a developer of pyfw-lasso, a python software that integrates a state-of-the-art optimization method for the LASSO problem, which fully relies on Pycsou, I strongly support the proposal for a Pycsou trove-classifier on PyPI. This would make it easier for potential users to find, deploy and use our software.

Good morning,
Having a Pycsou trove-classifier added to PyPI would significantly enhance the visibility of my in progess software package, EnvTrack, built for spatio-temporal (environmental) monitoring.

Hello,
Listing my TokamakRec package (plasma imaging for Tokamak data) in the Pycsou plugin catalogue is crucial for reaching a wider audience, and a Pycsou trove-classifier is a necessary prerequisite for this. I urge for its implementation on PyPl.

Thanks folks! This will be available once this dependency is updated for PyPI.


[Multiple Engine] FlutterEngineGroup.Options:  support more parameters (like constructor)

Use case
Currently, FlutterEngine.spawn() can only pass a few arguments
  @NonNull
  /*package*/ FlutterEngine spawn(
      @NonNull Context context,
      @NonNull DartEntrypoint dartEntrypoint,
      @Nullable String initialRoute,
      @Nullable List<String> dartEntrypointArgs) {
    if (!isAttachedToJni()) {
      throw new IllegalStateException(
          "Spawn can only be called on a fully constructed FlutterEngine");
    }

    FlutterJNI newFlutterJNI =
        flutterJNI.spawn(
            dartEntrypoint.dartEntrypointFunctionName,
            dartEntrypoint.dartEntrypointLibrary,
            initialRoute,
            dartEntrypointArgs);
    return new FlutterEngine(
        context, // Context.
        null, // FlutterLoader. A null value passed here causes the constructor to get it from the
        // FlutterInjector.
        newFlutterJNI); // FlutterJNI.
  }

ref: https://github.com/flutter/engine/blob/2b5d5c3b2d7e8d8d8b57612c4dc85aea6743e088/shell/platform/android/io/flutter/embedding/engine/FlutterEngine.java#L392
Internally, it calls the constructor of FlutterEngine.
 public FlutterEngine(
      @NonNull Context context,
      @Nullable FlutterLoader flutterLoader,
      @NonNull FlutterJNI flutterJNI) {
    this(context, flutterLoader, flutterJNI, null, true);
  }
->
  public FlutterEngine(
      @NonNull Context context,
      @Nullable FlutterLoader flutterLoader,
      @NonNull FlutterJNI flutterJNI,
      @Nullable String[] dartVmArgs,
      boolean automaticallyRegisterPlugins) {
    this(
        context,
        flutterLoader,
        flutterJNI,
        new PlatformViewsController(),
        dartVmArgs,
        automaticallyRegisterPlugins);
  }
->
  public FlutterEngine(
      @NonNull Context context,
      @Nullable FlutterLoader flutterLoader,
      @NonNull FlutterJNI flutterJNI,
      @NonNull PlatformViewsController platformViewsController,
      @Nullable String[] dartVmArgs,
      boolean automaticallyRegisterPlugins)
->
  public FlutterEngine(
      @NonNull Context context,
      @Nullable FlutterLoader flutterLoader,
      @NonNull FlutterJNI flutterJNI,
      @NonNull PlatformViewsController platformViewsController,
      @Nullable String[] dartVmArgs,
      boolean automaticallyRegisterPlugins,
      boolean waitForRestorationData)

Now I want to use the FlutterEngine.spawn,  but at the same time I also hope to include more parameters (such as automaticallyRegisterPlugins). But I don't see a way to support that.
So can we extend more parameters to spawn, like the FlutterEngine constructor. (I can submit PR if you think it's ok)
Proposal
Flutterengine.spawn supports more parameters (like constructor)
      engine = new FlutterEngine(context);
      engine.spawn(context, dartEntrypoint, initialRoute, dartEntrypointArgs,/*automaticallyRegisterPlugins*/false);

also add in the FlutterEngineGroup.Options：
  public static class Options {
    @NonNull private Context context;
    @Nullable private DartEntrypoint dartEntrypoint;
    @Nullable private String initialRoute;
    @Nullable private List<String> dartEntrypointArgs;
    //***ADD****//
    @Nullable boolean automaticallyRegisterPlugins;
    @Nullable PlatformViewsController platformViewsController;
    @Nullable boolean waitForRestorationData
}

cc @xster @dnfield

Closing it via https://github.com/flutter/engine/pull/34663


Keep-alive invalidating connection during query execution

latest Deaver 5.2.5 on win 8.1 x64 is invalidating connection during query execution, which result in error message:
SQL Error [12060] [HY000]: [Vertica][JDBC](12060) ResultSet object has been closed.   ProxySqlException(connectionId:14cadabf-733c-44eb-ba07-97871ccfa3fa, exceptionId:9651abfc-91ca-4c29-a03f-f5e49db9bb19, message:[Vertica][JDBC](12060) ResultSet object has been closed., errorCode:12060, sqlState:HY000)   [Vertica][JDBC](12060) ResultSet object has been closed.

The keep alive is set to one minute, while the query execution takes a bit longer, so the keep alive seem to be trying to execute on the active connection, which probably fails every time. This was not happening in older versions.

Basically parallel statements shouldn't cause any problems for JDBC connections. However some drivers (like recent Vertica driver) do not support this.
I'll add extra check (connection ping query will be executed only if there no other running queries).
Thanks for the bug report.

Just curious to read a some follow-up

Now I have the opposite problem. If a query takes more than hour (this is normal for our analytics), the connection may be closed by some network nodes between the client and the server. If I send pings in parallel to the running query, everything is OK. Can it be automated with DBeaver?

@yuri-sq you can set Keep Alive setting to some value other than 0 in connection configuration

@E1izabeth yes, that's the problem: Keep-Alive doesn't work when a query is running. So I set Keep-Alive setting, but the keep-alive packet aren't sent.

I'm working with Sqream, not Vertica.


Check CARGO_TARGET_WASM32_WASI_RUNNER for runtime override

Hello!
This PR uses the CARGO_TARGET_WASM32_WASI_RUNNER environment variable to conditionally override wasmtime as the WASI runner.  Let me know if you'd like to see any changes!
Resolves #13

Thanks for the review!  That's good feedback -- I completely missed the tests section before.
I updated the logic to handle both file paths and commands found by which because that seemed strictly more useful than just file paths and isn't much extra work.  The errors on the override branch aren't as precise as they could be, they're lumped together right now and it doesn't check that the override file path, when used as a file path, contains a file which is executable or anything like that right now.
I did my best to try to keep the Rust and error message tone the same.  Let me know if there's anything else you'd like me to improve before this ships!

Thanks!


improve: workflows

Error: YAML Exception reading /home/runner/work/blog/blog/_posts/2024-09-13-DontNeedNoStinkingOligarch.md: (): did not find expected key while parsing a block mapping at line 2 column 1


Resolves #3673 by only considering unreserved resources

for reserve/create operations.
Resources that are already associated with a role (are already reserved) cannot be reserved again. Marathon will now only consider unreserved resources when trying to reserve them for resident tasks. As a result, it makes no sense to define acceptedResourceRoles for resident apps, which is now prevented using a validator.

Looks nearly perfect. But you also have to allow "acceptedResourceRoles": ["*"]

Thanks for that quick review, @kolloch
As noted above, tell me whether you'd be OK with a clear validation error, something like resident apps cannot define acceptedResourceRoles. Reservations can only be processed based on unreserved resources, and I feel better with the TaskOpFactory explicitly using ResourceSelector.NoRole instead of the apps acceptedResourceRoles or the default ones.

I think that your proposal is a bit inconsistent. Our usual logic is that all apps have an effective acceptResourcesRole. Either an explicit override or a global default. Your code disallows a correct override of this setting and allows a scenario where the global default is incorrect without override.
To play devils advocate, we already chose defaults differently depending on context elsewhere, so I guess it is OK to treat no overridden acceptedResourceRoles smartly and use an effective acceptedResourceRoles that is compatible with reservations. Therefore my suggestion was to also allow the override which would have the same effect for normal apps, at least.
Does that make sense? Or another question: why is it important to you to disallow setting acceptedResourceRoles to ["*"]?

Hey @kolloch thanks for clarifying this. I'm now allowing correct acceptedResourceRoles and logging a warning when a reservation is actually tried but the effective acceptedResourceRoles do not contain "*"
I've added #3704 to improve on this; changing the validation to allow config aware checks is a bit more workload than I'd like to have in here (given the time we have ..)

please rebase

rebased

Perhaps we can even improve this logic to

set acceptedResourceRoles automatically to * if not set
make sure * is configured as resource role

Looks good for now.
Thanks!
            .
            |\
            | \
            |  \
            |   \
            |    \
            |     \
            |      \
            |       \       ____O
            |        \     .' ./
            |   _.,-~"\  .',/~'
            <-~"   _.,-~" ~ |
^"~-,._.,-~"^"~-,._\       /,._.,-~"^"~-,._.,-~"^"~-,._
~"^"~-,._.,-~"^"~-,._.,-~"^"~-,._.,-~"^"~-,._.,-~"^"~-,._
^"~-,._.,-~"^"~-,._.,-~"^"~-,._.,-~"^"~-,._.,-~"^"~-,._
~-,._.,-~"^"~-,._.,-~"^"~-,._.,-~"^"~-,._.,-~"^"~-,._


Indexes - projection listed as part of FROM clause, not SELECT

In our best practices on indexes, we say that users should store all columns that are only in the FROM clause, but it should be listed as the SELECT clause:
SELECT a, b FROM things WHERE c=10

https://github.com/cockroachdb/docs/tree/wc-fixitday-change-projection-wording-for-storing-indexes


Pushlink

Added PushLink Serializable to allow pubsubhubbub discovery.

Fine by me. How would you want to do that, something like an extensions file or a separate repo?

Yeah, a separate repo will be fine for that.


feat(sol-honk): integrate solidity honk verifier with bb and acir tests

Overview
Updated the existing solidity testing flow to run for a number of honk tests - some with public inputs and some without.

Benchmark results
Metrics with a significant change:

protocol_circuit_simulation_time_in_ms (private-kernel-tail-to-public): 4,013 (-41%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256_ms
5,783 (+1%)
1,574 (+2%)
711 (+1%)
748
777 (+1%)

proof_construction_time_sha256_30_ms
11,959 (+2%)
3,211 (+2%)
1,418 (+1%)
1,448 (+1%)
1,470

proof_construction_time_sha256_100_ms
44,304 (+1%)
12,084 (+3%)
5,522 (+2%)
5,568 (+3%)
5,421 (+1%)

proof_construction_time_poseidon_hash_ms
79.0 (+1%)
34.0
34.0
58.0 (-2%)
88.0 (+1%)

proof_construction_time_poseidon_hash_30_ms
1,537 (+1%)
425 (+2%)
203 (+2%)
227 (+3%)
269 (+1%)

proof_construction_time_poseidon_hash_100_ms
5,680 (+1%)
1,522
668 (-2%)
715
747 (+1%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
4 txs
8 txs
16 txs

l1_rollup_calldata_size_in_bytes
708
708
708

l1_rollup_calldata_gas
6,576
6,586
6,600

l1_rollup_execution_gas
583,038
583,199
583,364

l2_block_processing_time_in_ms
756 (-1%)
1,418
2,699

l2_block_building_time_in_ms
13,053 (-1%)
25,400
50,872

l2_block_rollup_simulation_time_in_ms
13,053 (-1%)
25,400
50,872

l2_block_public_tx_process_time_in_ms
10,963 (-1%)
23,107
48,477

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 8 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
7,045
9,952 (+1%)

node_database_size_in_bytes
12,722,256
16,887,888

pxe_database_size_in_bytes
16,254
26,813

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Function
time_ms
bytecode_size_in_bytes

GasToken:_increase_public_balance
97.9
13,790

GasToken:set_portal
12.2 (+19%)
3,339

Token:constructor
122
23,692

FPC:constructor
91.9
13,592

GasToken:mint_public
82.1 (-1%)
10,158

Token:mint_public
349 (+5%)
19,034

Token:assert_minter_and_mint
51.5 (+2%)
12,925

AuthRegistry:set_authorized
47.4 (+5%)
7,812

FPC:prepare_fee
119 (+14%)
11,068

Token:transfer_public
50.9 (+7%)
31,218

FPC:pay_refund
124 (+2%)
21,266

Benchmarking:increment_balance
1,292 (-1%)
11,281

Token:_increase_public_balance
15.1 (-1%)
15,006

FPC:pay_refund_with_shielded_rebate
177 (+1%)
22,361

Public DB Access
Time to access various public DBs.

Function
time_ms

get-nullifier-index
0.159

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
256 leaves
512 leaves
1024 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.4 (+1%)
16.9 (+2%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.7
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.602 (+1%)
0.521 (+2%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
48.5
75.9
131
245
470

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
287
543
1,055

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.496
0.466
0.451
0.444
0.439

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
60.2 (+1%)
111
183 (+1%)
351
692

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
109
207
355
691
1,363

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.508 (+1%)
0.498
0.484
0.476
0.475

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
72.8 (+1%)
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
133
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
0.518 (+1%)
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
77,362
668,542

Transaction size based on fee payment method
| Metric |  |
| - |  |


[AutoPR web/resource-manager] [Hub Generated] Review request for Microsoft.Web to add version 2018-02-01

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5137

Build output:


Re-login triggered by text file return

As discussed in #311, fixes re-login logic to catch cases where the PACER request returns a text file. When a text file was returned the existing session login check couldn't parse that it was a text file and an unnecessary re-login would be triggered.

Only one comment on this, but can you take a look at making black pass? It's an autoformatter we use to avoid conversations about code style.

I think it should be good now re: formatting.

Love it! Merged!
Do you need a fresh release with this or are you working from master?

Awesome! I've been using the version on pip, but I can work off of master if you don't have a release planned.

I'll just do it. Should go live in about ten or twenty minutes.

Great, thank you for your help!


[WIP] Add a service worker fallback for web notifications

This is an attempt at fixing #213.
It's still a bit of a work-in-progress, I'm curious to know what others think needs to be done before this is merge-able.
This code checks to see if the old Notification API is supported, and if not attempts to create a service worker instead, which will be created as long as the Zulip tab in Chrome was not killed for memory. The service worker will then create a notification in the background. Even though it's not super reliable because the tab can be killed and notifications will not be sent, I believe it's better than the current state of the code which throws an exception and doesn't send any notifications ever.
What I think still needs to be done:

clicking_notification_causes_narrow
use Push API so that notifications can be sent even if the Zulip tab is not loaded
I haven't tested that the clients.openWindow works as expected
remove second new Notification() in process_notification
will it be an issue that https is required for it to work?
needs tests

Automated message from Dropbox CLA bot
@thallada, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

Automated message from Dropbox CLA bot
@thallada, thanks for signing the CLA!

@thallada awesome!  Let me know if there's any way I can help :)

@thallada Will you be having time to complete this or can I take it?

I think @thallada is busy, but @brockwhittaker I think did some investigation into this; Brock, can you post your work?


Create index tracker of last checked row.

Between runs of the code, there will be new entries added to the spreadsheet. It wouldn't makes sense to start over from the first row every time. So instead we can create an index tracker the keeps track of the the last row-number accounted for in a computation.

please do not introduce state to the app. Rather interpret the position of the first empty row in result column


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_47h9qwx383/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_47h9qwx383/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_47h9qwx383/clone/recipe/meta.yaml:37: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_47h9qwx383/clone/recipe/meta.yaml:37: missing_description: The recipe is missing a description
ERROR: /tmp/abs_47h9qwx383/clone/recipe/meta.yaml:37: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_47h9qwx383/clone/recipe/meta.yaml:37: missing_license_family: The recipe is missing the about/license_family key.
Errors were found


Update outdated npm dependencies

Description
This is our monthly package update.
Issue number and link
N/A
Testing plan

Run eslint
Ensure CI passes

Author checklist

[x] I have submitted a Contributor License Agreement
[x] I have added my name to CONTRIBUTORS.md
[ ] ~I have updated CHANGES.md with a short summary of my change~
[ ] ~I have added or updated unit tests to ensure consistent code coverage~
[ ] ~I have update the inline documentation, and included code examples where relevant~
[x] I have performed a self-review of my code

@jjhembd Could you please review?

Since this was a minimal update and CI is passing, I'm going to go ahead and merge so we can start the release.


Broken on iOS 18 Beta 1

Seeing the same issue, my guess is we need to add .v18 to the introspect list
.introspect(.scrollView, on: .iOS(.v15, .v16, .v17)) { scrollView in
                configure(scrollView: scrollView)
            }

Thank you so much guys, @ioslife and @parkwolfapp! Please check out version 1.1.4, have a wonderful day!


Errors in the code editor crash the editor

e.g. starting to type const at the top level of the app.js file of a new project, the editor crashed with:
ReferenceError: c is not defined
    at eval (eval at exports.SafeFunctionCurriedErrorHandler (code-exec-utils.ts:NaN), <anonymous>:3:1)

This is easy to reproduce on Staging, but I can't reproduce it when running the editor locally.

This bug is particularly insidious and appears to be masked when running locally by another bug :grimacing:
If we change the functions clearConsoleLogs and addToConsoleLogs to Utils.NO_OP here, then when running locally we can reproduce the above issue. Without that change, the editor remains responsive (and shows the error overlay), but will crash the chrome dev tools due to it trying to log the same error an infinite number of times.


add link to transit.land

Ooh transit.land is super useful. :bus:

Actually maybe it should go into the data section?
I meant APIs as in software you run to provide an API (like OneBusAway or TransitTime)

Agreed, data makes more sense. Wasn't sure where to place it since there are a few pieces involved.


Update README.md

Changed MOFHY+ link to "Official Website"

@mahtab2003, anything wrong with this PR?

@mahtab2003, anything wrong with this PR?

No everything is fine.


Moving objects penetrate each other.

If you add the Body at once, the Body with IsStatic false will penetrate the Body with IsStatic false below.

We try to delete all Body once and then add Body. I also put an id on each Body, one by one.
And if I move the movable objects a little and then add the Body at once, this problem does not occur.
This text is translated by DeepL, so there may be some strange expressions.


exceeding dex limit

I got exeeding dex limit:
Error:Error converting bytecode to dex:
Cause: com.android.dex.DexIndexOverflowException: Cannot merge new index 66596 into a non-jumbo instruction!

Can you please share more details?
In the meantime can you try to fix it using this thread?
[(http://stackoverflow.com/questions/26093664/android-studio-only-dexexception-cannot-merge-new-index-65536-into-a-non-jumbo)]
Please let me know if it still exists

I tried using multiDexEnabled:
https://developer.android.com/studio/build/multidex.html
It builds successfully.
The error arrised when I 'Build Apk' (from Android Studio). However if I just 'run app (Shift F10)' it will be compiled alright even without multidex enabled.


testing: skip bad mtls by removing go files

system_tests.sh is a bit over-aggressive (but rightly so) in deciding
which tests to run.
Skip them by deleting them at the beginning of the CI run.

CC= @shinfan


[r2.9-rocm-enhanced] Added a nightly flag to build_rocm_python3

cherry-pick from develop-upstream

retest Ubuntu-GPU-multi please


sql project template not working on arm

SqlPackage or DacFx Version: 162.0.52 (Microsoft.Build.Sql.Templates 0.1.10-preview)
.NET Framework (Windows-only) or .NET Core: 4.8.1 for ARM64 (KB5028948)
Environment (local platform and source/target platforms): VS2022 (ARM64) Version 17.7.1, Windows 11 22H2 (OS Build 22621.2134)

Steps to Reproduce:

File -> New -> Project -> SQL Server Database Project -> (defaults) -> Create

Nothing happens

I tried upgrading an existing project, it fails to load with the following error messages:

The 'IntellisensehostingEngine Package' package did not load correctly.

The specified module could not be found. (Exception from HRESULT: 0x8007007E)

ActivityLog.xml

Did this occur in prior versions?  If not - which version(s) did it work in?
I'll try the previous versions later and report back
edit - same error even for the current non-preview SSDT. searching the error turns up this thread which is resolved as "not supported" and there's a link to vote to add support so I suppose this just isn't expected to be working yet
(DacFx/SqlPackage/SSMS/Azure Data Studio)

Was this issue actually fixed, or just deemed "not supported"? I am getting the same exact error on my machine.

Apple MacBook Pro M1 Max
Windows 11 in Parallels VM
Microsoft Visual Studio Professional 2022 (ARM 64-bit) Version 17.8.5
.NET 8

@GummiGerken - this issue has 2 intertwined components -

support for arm64 Visual Studio is on its way for SSDT and the original SQL projects (https://developercommunity.visualstudio.com/t/Arm64-support-for-Data-storage-and-proce/10114057)
the SDK-style SQL projects (Microsoft.Build.Sql), which we focus on in this repository, has a nuget package for the templates (Microsoft.Build.Sql.Templates).  Support for SDK-style SQL projects is available in VS Code and Azure Data Studio, not yet in Visual Studio.  There was a bug the SDK-style projects for version 0.1.10-preview where builds would fail on arm64 machines, this has been fixed in subsequent versions.

If your interest is in getting IDE support for SDK-style SQL projects in Visual Studio, please feel free to let the team know - https://developercommunity.visualstudio.com/t/Use-new-project-file-format-for-sqlproj/480461

Regarding #2 I’m kind of surprised there should be a need to ask for this. It’s a gap in functionality that is long overdue.On Jan 23, 2024, at 10:43 AM, Drew Skwiers-Koballa @.***> wrote:
@GummiGerken - this issue has 2 intertwined components -
support for arm64 Visual Studio is on its way for SSDT and the original SQL projects (https://developercommunity.visualstudio.com/t/Arm64-support-for-Data-storage-and-proce/10114057)
the SDK-style SQL projects (Microsoft.Build.Sql), which we focus on in this repository, has a nuget package for the templates (Microsoft.Build.Sql.Templates).  Support for SDK-style SQL projects is available in VS Code and Azure Data Studio, not yet in Visual Studio.  There was a bug the SDK-style projects for version 0.1.10-preview where builds would fail on arm64 machines, this has been fixed in subsequent versions.
If your interest is in getting IDE support for SDK-style SQL projects in Visual Studio, please feel free to let the team know - https://developercommunity.visualstudio.com/t/Use-new-project-file-format-for-sqlproj/480461
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

This doesn't answer your question in its entirety, but support in ARM for sqlproj files was just released today in the Visual Studio 2022 Preview channel (https://visualstudio.microsoft.com/vs/preview/). I can now open my sqlproj files in Visual Studio 2022 Preview and build the projects.


chore(main): release taiko-client 0.39.2

:robot: I have created a release beep boop
0.39.2 (2024-10-24)
Bug Fixes

protocol: fix issue in mainnet deployment script (#18283) (5c371a1)
taiko-client: fix path parsing in /eth/v1/config/spec (#18295) (6633c80)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

taiko-client-v0.39.2
:sunflower: