feat(deploy) introduce a preStop hook for graceful shutdown of Kong

k8s sends SIGTERM to the containers to terminate which results in nginx
dropping requests.
This ensures that nginx gets a SIGQUIT to gracefully shutdown before
getting a SIGTERM.

I note that for the docker-openresty packages, we addedSTOPSIGNAL SIGQUIT to the Dockerfiles to have a cleaner shutdown behavior intrinsic to the image itself.
https://github.com/openresty/docker-openresty/pull/80
Kubernetes honors this:  https://github.com/kubernetes/kubernetes/issues/30051

@neomantra Thank you for pointing that out.
I opened up https://github.com/Kong/docker-kong/pull/259 for discussion.


Arun service nav disapear

#56 created new services

The merge creates services that allow the nav and footer to disapear on some pages


Intellisense doesn't use jsdoc for class instance properties

VSCode Version: Version 1.19.3 (1.19.3)
OS Version: OSX (Darwin Kernel Version 17.4.0: Sun Dec 17 09:19:54 PST 2017; root:xnu-4570.41.2~1/RELEASE_X86_64 x86_64)

Looks like intellisence is using jsdoc param types only when method is declared as a prototype method.
prototype method (works as expected)

class property (fallback to any, why?)

Well, that's cool but to me it's more like a workaround for an issue. Isn't onStreamGot a function at the same time? The only difference is that it will be an own property of newly created object but not  a property of this object's prototype. So why should I use @type instead of @param here?

I have tried with JSDoc 3.5.5

/** Test class */
export class A {
    /**
     * Test method
     * @param {string} input
     */
    method(input) { }
}

Works

/** Test class */
export class A {
    /**
     * Test method
     * @param {string} input
     */
    method = (input) => { }
}

Doesn't work

/**
 * Test class
 * @constructor
 */
export function A() {

}

/**
 * Test method
 * @param {string} input
 */
A.prototype.method = (input) => { }

Works

/**
 * Test method
 * @param {string} input
 */
export const method = (input) => { }

Doesn't work

So I think it's intended, or this usage has not been specially treated by JSDoc.

Tracked upstream by https://github.com/Microsoft/TypeScript/issues/19231


Provide a DB version change API

Re: #116 -- we don't currently handle upgrade events, even in drivers that have very good support for upgrades (i.e. IndexedDB). We don't really employ a schema, but maybe it would make sense to provide an API to allow people to do data migrations across versions of their DB?
Or is this out-of-scope for localForage, and something an app/framework should handle?

Stumbling on this as I was wondering if localForage provided a way to help handle upgrades -- I personally would be very interested in this especially if we were able to provide a way for localStorage and WebSQL a way to migrate. I see packages like angular-localforage-migrations that wouldn't be as necessary if this were the case.
My original instinct that localForage versioning would somehow hook into IndexedDB but #658 showed that wasn't the case (thankfully there has been healthy discussion about this :D). Would like to see more thoughts around this though as I feel it would be a very useful feature.

Unfortunately right now upgrades aren’t really handled well by localForage :-(
It’d be an awesome feature to add but we haven’t been able to get to it.

It should be easy to make an plain js migration plugin based on what the angular one already does. I imagine having a main LF core (this repo) and several satellite plugins under the other. It would be easy to implement independently, evaluate & reiterate. It would also ease evaluating the implementation & tracking user acceptance (eg npm DLs).
On the other hand I'm thinking about a __localforage__system__ instance/db/store for some time now, holding other things like the used instances.

@thgreasi yeah I actually didn't mean to suggest that the migration plugins aren't good -- if that's the preferred route for people to incorporate migrations in their apps, I just feel that should be specified somewhere (essentially #116) 😛


latency info possibility

Hello,
I want to map the VGG16 on the Simba arch, will that be possible to get the layerwise latency info from the generated output?
Having seen from the example with Alexnet, does the level-based info in file shows the layerwise info?
Thanks alot

You may calculate layer-based statistics by summing the relevant statistics (energy, cycles, accesses) across the stats of all layers. This is not done for you (yet); it must be done outside of Accelergy/Timeloop.


Clock keeps rebooting

Hi!
Today i tried adjusting the utc offset to compensate for daylightsaving. After this the clock keeps rebooting. It gives the "connecting to network" signalling, then the clock shows but in the background i still see it circling. Then it goes off and on again, trying to connect again. If i'm lucky i can connect to the webpage but it fails to load often. I tried reinstalling the software, tried a different wifi accespoint, different NTP server and i have no idea why it is broken now. next step would be a new ESP. Do you have any idea what it could be? It worked flawelessly for over 2 years
With regards,
Daniël

I did have issues with my device where the webpage does not load completely or loads very slowly after long uptimes (usually after several weeks/months). Usually a reboot works for me, but I havent had much issues with the Wi-Fi being unable to connect. See if you can get an example ESP8266 WiFi sketch uploaded and connected to your network access point. I'm not 100% what could be the issue, its possible that the EEPROM emulation may have finally run its course. Let me know if you can get an example sketch working and connected and we can see what to try from there

It probably was the EEPROM indeed. A new ESP8266 did fix the problem! Now i cant access my webpage anymore, prlly a config error on my side. but since i use the http requests anyway that isnt a big deal for me. Now i can finally continue to enjoy your work every day :)


Beta 2 - Azure AI Projects SDK

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


hls choice in format list

I get this list from yt-dlp -F command
hls-aac-Audiodescrizione mp4 audio only │                m3u8  │ audio only       mp4a       [Audiodescrizione] Audiodescrizione
hls-aac-Italiano         mp4 audio only │                m3u8  │ audio only       mp4a       [Italiano] Italiano
hls-1800                 mp4 1024x576   │ ~1.34GiB 1758k m3u8  │ avc1       1758k video only
hls-2400                 mp4 1280x720   │ ~1.79GiB 2344k m3u8  │ avc1       2344k video only
hls-3600                 mp4 1440x810   │ ~2.69GiB 3516k m3u8  │ avc1       3516k video only
hls-5000                 mp4 1920x1080  │ ~3.98GiB 5210k m3u8  │ avc1       5210k video only
to get simple one of this list...
I tried
yt-dlp -f hls-5000
error
tried also a simple yt-dlp -f best
same
where am I wrong?

Several ways.

#30839 "if you were actually using yt-dlp ..."
You didn't fill out the template.
You didn't supply a text log of your failing command.

Nonetheless a general point that applies for yt-dl and yt-dlp is that a manifest (m3u8, say) may not be downloadable outside the context of the site from which it was captured. It may be time-limited, or require additional HTTP headers, such as Referer, Origin, Authorization or cookies.
Try passing the URL of the page with the media instead of just the manifest URL.

very simple...
yt-dlp -F https://www.raiplay.it/video/2019/10/battute-e2734ab5-6d35-4fa6-b9ea-a3cdd9a03726.html
[RaiPlay] Extracting URL: https://www.raiplay.it/video/2019/10/battute-e2734ab5-6d35-4fa6-b9ea-a3cdd9a03726.html
[RaiPlay] e2734ab5-6d35-4fa6-b9ea-a3cdd9a03726: Downloading video JSON
[RaiPlay] e2734ab5-6d35-4fa6-b9ea-a3cdd9a03726: Downloading XML metadata
[RaiPlay] e2734ab5-6d35-4fa6-b9ea-a3cdd9a03726: Downloading m3u8 information
[RaiPlay] e2734ab5-6d35-4fa6-b9ea-a3cdd9a03726: Checking MP4 availability
[info] Available formats for e2734ab5-6d35-4fa6-b9ea-a3cdd9a03726:
ID         EXT RESOLUTION FPS │   FILESIZE   TBR PROTO │ VCODEC      ACODEC
──────────────────────────────────────────────────────────────────────────────
hls-854    mp4 512x288     25 │ ~118.93MiB  855k m3u8  │ avc1.4d401f mp4a.40.2
https-800  mp4 512x288     25 │ ≈118.93MiB  855k https │ avc1.4d401f mp4a.40.2
hls-1896   mp4 928x522     25 │ ~263.91MiB 1896k m3u8  │ avc1.4d401f mp4a.40.2
https-1800 mp4 928x522     25 │ ≈263.91MiB 1896k https │ avc1.4d401f mp4a.40.2
normally yt-dlp download the last one in the list, but If I want to specify one in particular, for example hls-854
with the command
yt-dlp -F hls-854 https://www.raiplay.it/video/2019/10/battute-e2734ab5-6d35-4fa6-b9ea-a3cdd9a03726.html
[generic] Extracting URL: hls-854
ERROR: [generic] 'hls-854' is not a valid URL. Set --default-search "ytsearch" (or run  yt-dlp "ytsearch:hls-854" ) to search YouTube
so I can't figure out how to give the selection of that resolution

No, it isn't.
Ask your question in the right place (point 1 above), with the required information, and someone may be able to help. You could start by reviewing https://github.com/yt-dlp/yt-dlp/issues/9154.


Android: Expose HyphenationFrequency on Text

Is this a bug report?
yes
Have you read the Contributing Guidelines?
Yes
Environment
OS: Windows 10
Node: 9.2.1
Yarn: 1.3.2
npm: 5.5.1
Watchman: Not Found
Xcode: N/A
Android Studio: Version  <IP_ADDRESS> AI-162.4069837
Packages: (wanted => installed)
react: ^16.2.0 => 16.2.0
react-native: ^0.50.4 => 0.50.4
Target Platform: Android (6)
Steps to Reproduce
Using a Text view in Android and iOS to show text with hyphenation characters: in iOS the text is displayed using the whole line width with hyphen characters but in Android the text lines break usually without showing hyphen characters when needed, making the text look ugly in Android and not using the full width of the line.
The problem in Android is that TextView attribute hyphenationFrequency is set to HYPHENATION_FREQUENCY_NORMAL by default and the desired behavior consistent with iOS is to use HYPHENATION_FREQUENCY_FULL in the React Native ReactTextView
Example:
Current Adnroid outcome:
"very long sentences
usually break line incorrect-
ly"
Expected Adnroid outcome:
"in very long senten-
ces break line correc-
tly"
Reproduction steps

Create a text component
Use this long hyphenated text in a ReactTextView
"Freelancejournalist Freelancejournalist Freelancejournalist Freelancejournalist Freelancejournalist Freelancejournalist Freelancejournalist Freelancejournalist Freelancejournalist"
Make sure the screen width is small enough so the text has to break a couple of times (phone portrait mode)

Expected Behavior
At the end of the lines there are hyphen characters breaking long words before text continues to the next line.
The text has a similar line breaking with hyphens in Android and iOS.
Actual Behavior
At the end of the lines there are no hyphen characters breaking long words before text continues to the next line and text lines don't use the full width of the text view, leaving big white spaces at the end of a line.
The text has different line breaking structure and most lines don't show hyphens in Android compared to iOS.
Reproducible Demo
https://snack.expo.io/Syh0901MM

Still reproducible.

This is not an issue itself, but a feature request that to summarize, there is currently no way to set hyphenation frequency on Android.
Having no way to set this reduces the control over how the platform breaks text as there are time that's not enough to set textBreakingStrategy to simple to avoid cutting words in the middle, where setting hyphenation frequency to none leaves no other alternative to the OS and cutting is entirely prevented.
Currently it's set to normal as it can be seen here and here where IMHO a better default would have been none or exposing this to the user as this issue is asking for.


WIP: Telegram Bot: Call the tweepy auth function to the telegram class

Please check the code and suggest me how to do it. The code is incorrect

It was tested but was not working. Error was api was undefined.

In a overall, look good. I will test it out later today and will merge accordingly.


Getting Error: spawn ENOTDIR with activeWindow()

same problem here, did you find a solution?

@Ladvace @sidh01 Anyone find the solution?

nope, I get this error on macos, not sure if it's supposed to ask for permissions or the user need to set it manually (didn't try this yet)

@Ladvace The app is working fine using electron start but not with the build, Not sure if the terminal has the permissions.
BTW, what you are building with this?

Electron-builder

same for me, you mean what I'm using to build it?

No, I mean what purpose you're using this package?

how is this related to he problem?

Filing a bug report for a native dependency without even specifying the OS is just asking to never get it addressed.

@Nantris It's in MacOS(Silicon chip), and happening when creating build, Dev mode with electron is working fine.

@qualwebs did you see this? https://github.com/sindresorhus/get-windows/issues/99#issuecomment-870874546
If this fixes the issue, there probably needs to be a note in the README for macOS too.

@qualwebs did you see this? #99 (comment)
If this fixes the issue, there probably needs to be a note in the README for macOS too.

I just tried this and it doesn't seem to fix it to me

I'm planning to try this myself on macOS tomorrow, though not Silicon. Any other findings at all that might be helpful towards debugging?

@Nantris
My macOS version is 14.6.1.
It works perfectly fine with dev mode in electron but not after build.
Also, Can I fetch browser URLs in linux and windows? if not, why and if that can be achieved?

It works for me on macOS 14.5 - but specifically I tested with the optional permissions disabled.
I suspect the app needs to be signed and notarized to work. I can't answer about URLs because we don't need them, sorry.

@Nantris - Related to this issue: #189

@qualwebs @sidh01 @Ladvace
I had the same error:
Error getting active window: Error: spawn ENOTDIR
    at ChildProcess.spawn (node:internal/child_process:421:11)
    at spawn (node:child_process:776:9)
    at execFile (node:child_process:351:17)
    at node:child_process:243:21
    at activeWindow (file:///Applications/deepfocus.app/Contents/Resources/app.asar/node_modules/get-windows/lib/macos.js:38:25)
    at activeWindow (file:///Applications/deepfocus.app/Contents/Resources/app.asar/node_modules/get-windows/index.js:18:10)
    at async Timeout._onTimeout (file:///Applications/deepfocus.app/Contents/Resources/app.asar/out/main/index.js:220:26) {
  errno: -20,
  code: 'ENOTDIR',
  syscall: 'spawn'
}

I use an M1 Macbook Pro (16GB) & Sonoma 14.5
What worked for me was modifying the node-modules directly and changing this file:
node_modules/get-windows/lib/macos.js
Here's the code:
import path from 'node:path';
import {promisify} from 'node:util';
import childProcess from 'node:child_process';
import {fileURLToPath} from 'node:url';
import fs from 'node:fs';
const __dirname = path.dirname(fileURLToPath(import.meta.url));
const execFile = promisify(childProcess.execFile);
const isPackaged = process.mainModule?.filename.indexOf('app.asar') !== -1;
const binary = isPackaged
? path.join(process.resourcesPath, 'app.asar.unpacked', 'node_modules', 'get-windows', 'main')
: path.join(__dirname, '../main');
console.log('Binary path:', binary);
if (!fs.existsSync(binary)) {
console.error('Binary file does not exist:', binary);
throw new Error('Binary file not found');
}
const parseMac = stdout => {
try {
return JSON.parse(stdout);
} catch (error) {
console.error(error);
throw new Error('Error parsing window data');
}
};
const getArguments = options => {
if (!options) {
return [];
}
const arguments_ = [];
if (options.accessibilityPermission === false) {
    arguments_.push('--no-accessibility-permission');
}

if (options.screenRecordingPermission === false) {
    arguments_.push('--no-screen-recording-permission');
}

return arguments_;

};
export async function activeWindow(options) {
const {stdout} = await execFile(binary, getArguments(options));
return parseMac(stdout);
}
export function activeWindowSync(options) {
const stdout = childProcess.execFileSync(binary, getArguments(options), {encoding: 'utf8'});
return parseMac(stdout);
}
export async function openWindows(options) {
const {stdout} = await execFile(binary, [...getArguments(options), '--open-windows-list']);
return parseMac(stdout);
}
export function openWindowsSync(options) {
const stdout = childProcess.execFileSync(binary, [...getArguments(options), '--open-windows-list'], {encoding: 'utf8'});
return parseMac(stdout);
}
Note: some of the logic is meant for use with building an Electron app, but shouldn't be an issue.
Afterwards, I can get the activeWindow info with no issues.
Time spent on Getting Error: spawn ENOTDIR with activeWindow() · Issue #185 · sindresorhus/get-windows: 2m 0s
I'm considering raising a PR if this is a more common issue.

@timeowilliams - thanks for the tip, could you raise the PR though, I'm curious to see what the fix is?  Also are you using electron-forge?

@trinhxyz :
https://github.com/Tech-Nest-Ventures/get-windows/blob/main/lib/macos.js
Something similar to this. I use electron-builder right now in combination with https://electron-vite.org/

I'm using electron-builder with https://electron-vite.org/ on Macos M1

I just tried to replace node_modules/get-windows/lib/macos.js as @timeowilliams did but I just get the following error:
App threw an error during load
Error: Binary file not found

because path.join(process.resourcesPath, 'app.asar.unpacked', 'node_modules', 'get-windows', 'main') doesn't exist

actually building it, I don't get the error, I just get this:
  stdout: 'get-windows requires the accessibility permission in “System Settings › Privacy & Security › Accessibility”.\n',

I can confirm that giving the right permission and the changes from  @timeowilliams, it works!

@trinhxyz ,
Ended up straying away quite a bit from this repo, but if you're interested in just using the dependency without having to mess with the code/just install, feel free to use this dependency:
https://github.com/Tech-Nest-Ventures/get-windows
^ Specifically optimized for MacOS at this time.

@Ladvace ,
100%! Was using the wrong method.
Please change this in  node_modules/get-windows/lib/macos.js
const isPackaged = process.main?.filename.indexOf("app.asar") !== undefined;

to this:
const isPackaged = process.resourcesPath !== process.cwd();

seems working great now!

https://github.com/sindresorhus/get-windows/pull/192#issuecomment-2374780649


Fix -Wformat and -Wtype-limits warnings

Split from https://github.com/zlib-ng/zlib-ng/pull/706

On MSVC this now produces:
D:\a\zlib-ng\zlib-ng\deflate_p.h(36,9): warning C4068: unknown pragma 'GCC' [D:\a\zlib-ng\zlib-ng\zlib.vcxproj]

There are 18 separate warnings due to this.

Well, that is hardly better than the warning it was supposed to silence unfortunately. ☹
We should use a different approach then.

Yeah, let's revert this.


pgstac client can return null links

The logic in stac_fastapi.pgstac.models.links.BaseLinks.get_links uses some implicit naming to gather link via subclass naming conventions. However, if link_* methods return (which the PaginationLinks.link_next or link_prev can despite their type declarations), this can end up with None in this link and therefore null in the eventual response.
Fix is to filter for Nones in the linked method.
I think there's not enough logic in these models to warrant a implicit method-naming trick like this, and took me a good bit of time to track down the cause of this, so I'd also recommend a refactor that uses a more straightforward approach.

This issue is being tracked by stac-utils/stac-fastapi-pgstac#8


Color Themes

I am visually impaired and I can't read blue, red und green colors from vim for example in wetty. Is there a way to specify a color scheme like in most terminal emulators?
I found an issue about foreground and background color. I need to change the default terminals colors. Most terminal emulators have this feature.

currently, there's no support for themes but I really like the idea.

Setting terminal emulation on profile script could solve that issue, I think. It's stored in TERM variable. If the programs read from that value and set the colors according to that, there'd be no problem.

I'm new to wetty. Wetty is based on xterm.js right?
xterm.js has some possibilities for theming:
https://github.com/xtermjs/xterm.js/issues/1719
I'm not very experienced in JS-development. I guess I'll try it.


Want to Create a Contributor's Guidelines.

Is your feature request related to a problem? Please describe.
A Guidelines for the new geeks who are new to Open source.
There are certain steps and instructions which should be followed by every new commerce.

If you think this should be applied, then plz assign me this task.

@Rishikeshrajrxl  it is already present there in contributing.md

Hey , is it still open  , can I work on this ?

Sorry it is over for now
On Sun, 29 Nov, 2020, 11:15 am KHUSHI JAIN<EMAIL_ADDRESS>wrote:

Hey , is it still open , can I work on this ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/harsh-9in/Blogger/issues/147#issuecomment-735347590,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALRCQGW2EPEGDADDV4D3Q3TSSHNXBANCNFSM4RJIYQCA
.

Hi. Can I work on this?
What I plan:

Create a file with images and pictures showcasing how to contribute to this project.
Include beginner's guide to contributing.

I also suggest giving text about contributing for NJACK'20 along with the points.

can you please assign this issue to me? @harsh-9in

Hey @harsh-9in, is this issue still open?

@harsh-9in, I would like to know why this issue is still open when CONTRIBUTING.md file does exists.

@harsh-9in, I would like to know why this issue is still open when CONTRIBUTING.md file does exists.

@atharva-upadhye-01 Yeah  pr is merged I am closing this issue.

@atharva-upadhye-01 Yeah  pr is merged I am closing this issue.


[BUG] Amethyst crash with invalid geohash

Describe the bug
Amethyst crash with invalid geohash.
To Reproduce
Steps to reproduce the behavior:

Post note with invalid geohash (algia post -geohash xxx "Oh No!")
Show the note on Amethyst

Expected behavior
Don't crash
Device (please complete the following information):

Android Version [e.g. 33]: 12
App Version [e.g. v0.20.3]: v0.79-2

It might have to check others.
https://image.nostr.build/f4fd50b2782a157cef2ceebe1fcc4b380d38ba50e5c25b5842e2884a3068d7e2.jpg


Unit testing for api.js

This pull request includes a

[ ] Bug fix
[x] New feature
[x] Testing
[ ] Translation

The following changes were made:

Added api.test.js into integration subfolder

Comments

This is the last file for issue #5. Can close issue #5 after merge

#5

@mmmaaatttttt thanks for reviewing! made the suggested changes!


Update handling of JavaInfo.compilation_info.javac_opts

To optimize memory retained by Bazel, the type of JavaInfo.compilation_info.javac_opts is being changed from a list of tokenized strings to a depset of de-tokenized strings.
In order to use the utility method from @rules_java to convert this back into a list, this change also needed the following additional changes:

Update rules_java version to 7.2.0 (the first version that has the utility)
Update bazel version(s) (the new rules_java requires a bazel version >= 6.4.0). (in any case, now that bazel 6.4.0 has been released, 6.3.0 is unsupported)
Registering the new java bootstrap toolchains (see https://github.com/bazelbuild/bazel/pull/18262)
rules_java dropped definitions for (non-LTS) jdk 20, replacing it with jdk 21. So used the definitions for jdk21 from rules_java and updated the local jdk21 definitions back to jdk20.

Happy to split these changes off into separate PRs if it would make review easier.

go/gcbrun

Presubmit is failing with:
Step #0 - "bazel-minversion": Repository rule http_archive defined at:
Step #0 - "bazel-minversion":   /builder/home/.cache/bazel/_bazel_root/eab0d61a99b6696edb3d2aff87b585e8/external/bazel_tools/tools/build_defs/repo/http.bzl:372:31: in <toplevel>
Step #0 - "bazel-minversion": ERROR: /builder/home/.cache/bazel/_bazel_root/eab0d61a99b6696edb3d2aff87b585e8/external/rules_java/toolchains/BUILD:223:14: While resolving toolchains for target @rules_java//toolchains:platformclasspath: No matching toolchains found for types @bazel_tools//tools/jdk:bootstrap_runtime_toolchain_type.
Step #0 - "bazel-minversion": To debug, rerun with --toolchain_resolution_debug='@bazel_tools//tools/jdk:bootstrap_runtime_toolchain_type'
Step #0 - "bazel-minversion": If platforms or toolchains are a new concept for you, we'd encourage reading https://bazel.build/concepts/platforms-intro.
Step #0 - "bazel-minversion": ERROR: Analysis of target '//buildenv/java:toolchain_jdk_17' failed; build aborted:
Step #0 - "bazel-minversion": INFO: Elapsed time: 120.430s
Step #0 - "bazel-minversion": INFO: 0 processes.
Step #0 - "bazel-minversion": FAILED: Build did NOT complete successfully (476 packages loaded, 9629 targets configured)
Step #0 - "bazel-minversion": ERROR: Couldn't start the build. Unable to run tests

TLDR: It's a bit of a guess without the complete log, but if my diagnosis is correct, it should work now.
It appears this repo too is using github.com/bazelbuild/bazel-toolchains/pkg/rbeconfigsgen for RBE. I had been led to believe that was only used by Bazel CI. I've added the same workaround we're currently using there. I'm not thrilled about having to do this. If y'all are averse to this, I'll see if I can cherry-pick the needed change onto rules_java 6.5.1. That would have the added benefit of not requiring all the additional, unrelated changes ( (although anyone using bazel 6 should really be on 6.4.0).

Friendly ping :)

Step #0 - "bazel-minversion": ERROR: /builder/home/.cache/bazel/_bazel_root/eab0d61a99b6696edb3d2aff87b585e8/external/rules_java/toolchains/BUILD:223:14: While resolving toolchains for target @rules_java//toolchains:platformclasspath: No matching toolchains found for types @bazel_tools//tools/jdk:bootstrap_runtime_toolchain_type.
Step #0 - "bazel-minversion": To debug, rerun with --toolchain_resolution_debug='@bazel_tools//tools/jdk:bootstrap_runtime_toolchain_type'
Step #0 - "bazel-minversion": If platforms or toolchains are a new concept for you, we'd encourage reading https://bazel.build/concepts/platforms-intro.
Step #0 - "bazel-minversion": ERROR: Analysis of target '//buildenv/java:toolchain_jdk_11' failed; build aborted:
Step #0 - "bazel-minversion": INFO: Elapsed time: 128.364s
Step #0 - "bazel-minversion": INFO: 0 processes.
Step #0 - "bazel-minversion": FAILED: Build did NOT complete successfully (508 packages loaded, 4397 targets configured)
Step #0 - "bazel-minversion": ERROR: Couldn't start the build. Unable to run tests
Finished Step #0 - "bazel-minversion"
ERROR
ERROR: build step 0 "gcr.io/kythe-repo/bazelisk-builder-client" failed: step exited with non-zero status: 1

Alright, this is practically impossible to debug without being able to repro. I've gone ahead and made a release of rules_java based on the currently used version (6.5.1) with the necessary changes.

go/gcbrun

I'm hoping the timeout is just here on github and not in the GCP build (which would have taken longer than usual because it's the first build with a new bazel version)?

go/gcbrun

/gcbrun

Hi, does this need anything else, or can it be merged?

Just needed to wait for the CI to finish.


Missing support for 'tilemap' chart type.

Bug or Feature Request?
Feature request.
Description
Hello, I discovered react-jsx-highcharts and it seems to be conseptually more efficient solution comparing to flat highcharts adapters for reactjs.
I didn't really come to work with it too much, then I realized that data series of type tilemap is not supported.
I'm aware of the fact that additional modules should be added to highcharts when using a tilemap type: heatmap and tilemap.
How to reproduce

Import highcharts and required modules:

import Highcharts from 'highcharts';
import highChartsTilemap from 'highcharts/modules/tilemap';
import highChartsHeatmap from 'highcharts/modules/heatmap'
import highchartsMore from 'highcharts/highcharts-more';

Add the module to the highcharts provider:

highchartsMore(Highcharts);
highChartsHeatmap(Highcharts);
highChartsTilemap(Highcharts);

Try defining a chart of type 'tilemap'

<HighchartsChart colorAxis={colorAxis}
				plotOptions={this.state.plotOptions}
				callback={this.getChart}>
<Chart type="tilemap" />
<XAxis />
<YAxis />
<Series type="tilemap" name="s1" data={data}/>

</HighchartsChart>

Expected to see tilemap, instead empty chart shown without any errors or warnings in browser console.
 component doesn't exist.
 component seems to ignore the type "tilemap".

Live demo demonstrating bug

https://stackblitz.com/edit/base-chart-example-62g6mx?file=MyChart.js
Versions

React JSX Highcharts version: 3.1.0
Highcharts version: 6.0.4
React version: 16.2.0

Hi @alexei-tilikin there is an outstanding issue to add Highmaps support (#56)
Judging by the imports required by vanilla Highcharts, looks like this would be a prerequisite.

Hi @whawker ,
Thanks for your response.
As far as I'm familiar with highcharts, the type 'tilemap' is not a highmap, but a highchart.
It doesn't require highmaps module, and rather depends on heatmap module (which is also part of highcharts).
The imported modules that I used in the reproduce example don't contain highmaps, and they are working for other highcharts react wrappers (such as  or highcharts-react-official).
Here is the highcharts demo page for Tilemap, it is categorized under highcharts and not under highmaps.
Do you support 'heatmap' type? I didn't find that one as well.
Supporting it would probably be a prerequisite...

@alexei-tilikin ah good spot.
Looks like the problem is that Series needs to child of an Axis - usually the YAxis.
Also, in your StackBlitz example above, you were importing the modules but not applying them, I think this might have just been a copy paste error though, as you mentioned that in your original post.
Here is that America tile chart, working in React JSX Highcharts: https://stackblitz.com/edit/base-chart-example-crydtv?file=MyChart.js

@whawker it works!
Thank you for the help!
Funny enough I tried adding the series to XAxis, but probably missed something else then...


Please consider allowing replace-logic in ITI-105

See discussion on Zulip: https://chat.fhir.org/#narrow/stream/179223-ihe/topic/MHD.20Simplified.20Publish
"We (ISiK-Specifiers and  Implementers from Germany) would love to see the replaces feature maintained in ITI-105. In our use case, amendments or replacements of documents are common occurances but in our opinion don't warrant the complexitiy that comes with ITI-65. We would argue that adding Provenance to the documents doesn't necessarily require a submission set, since many details about the originating client/user/agent/etc. could be derived from HTTP metadata and OAUTH Tokens if needed."

discussed in ITI-Tech meeting.
will allow transforms including replace.
will add an open-issue asking for specific comment about this. If no comment is received after public-comment then this open-issue will be closed.


De-multiplex STUN replies based on transaction ID

Previously, accept_message would validate that incoming STUN requests match the credentials we have locally. This allows a server to de-multiplex, which agent should handle an particular STUN request.
In a peer-to-peer setting, both parties will make STUN requests and thus, we also need to de-multiplexing incoming STUN replies. Luckily this is quite easy because the candidate pairs that we have formed locally keep track of the transaction IDs that we've used so far. If an inbound message is a STUN reply but we don't recognise the transaction ID, we now return false from accepts_message.

Nice find!
This is such a "duh", I don't even understand why this wasn't done already.

Thanks! I was a bit worried that I am just missing something. Great to hear that it was simply overlooked. Sometimes things do come easy :)

Thanks!


Error while trying to get the Windows 11 ISO Download Link.

I get the following error when trying to run the powershell script, I believe it's related to #4 but I'm not sure.
[Info] Trying to obtain Windows 11 iso download link...
Unhandled exception. System.TimeoutException: Timeout 30000ms exceeded.
=========================== logs ===========================
waiting for Locator("#product-edition")
  locator resolved to <select id="product-edition" href="#product-info-content">.</select>
  selecting specified option(s)
    did not find some options - waiting...
============================================================
   at Microsoft.Playwright.Transport.Connection.InnerSendMessageToServerAsync[T](String guid, String method, Dictionary`2 dictionary) in /_/src/Playwright/Transport/Connection.cs:line 143
   at Microsoft.Playwright.Transport.Connection.WrapApiCallAsync[T](Func`1 action, Boolean isInternal) in /_/src/Playwright/Transport/Connection.cs:line 465
   at Microsoft.Playwright.Transport.Channels.FrameChannel.SelectOptionAsync(String selector, IEnumerable`1 values, Nullable`1 noWaitAfter, Nullable`1 strict, Nullable`1 force, Nullable`1 timeout) in /_/src/Playwright/Transport/Channels/FrameChannel.cs:line 469
   at Microsoft.Playwright.Core.Frame.SelectOptionAsync(String selector, IEnumerable`1 values, FrameSelectOptionOptions options) in /_/src/Playwright/Core/Frame.cs:line 154
   at Program.<Main>$(String[] args)
   at Program.<Main>(String[] args)
Unable to build the tiny11 iso (an error occured while trying to download the original iso using WindowsIsoDownloader).
Removing work folders...

Thanks for opening an issue. The fix has been made and it should work now (no change on this repo, changes are on https://github.com/ianis58/WindowsIsoDownloader so you should not need to redownload the tool).


TSO to 3.188

Critical Changes
Changes
Updated the TSO id in trial.json to the template for NPSP 3.188.
Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

@codexmarcel The note in the Changes section shouldn't go in the release notes, right? Just wanted to check before I move it out. Thanks!


Fix label for xr.Dataset

Closes #261
The last commit in this PR makes the warning only show if the user explicitly sets attr_labels to True. Not sure whether or not that change is acceptable. But I kept the commit separate so that it is easily revertable if people don't like that idea.

Hmm, don't think this is sufficient, will need to work out a better fix for Dataset.

Hmm, don't think this is sufficient, will need to work out a better fix for Dataset.

What were you seeing in particular?
Also do you agree with failing silently unless the user explicitly asks for this functionality?

Nevermind missed your other commit, looks good!


Fix compilation of vcpkg on OpenBSD

Describe the bug
The SIOCGIFHWADDR name, referenced in the fallback branch within vcpkg::get_user_mac_hash, is not defined on OpenBSD.
Further, the environ name referenced in vcpkg::cmd_execute_background, among others, is not defined on OpenBSD, either.
Environment

OS: OpenBSD vultr.guest 7.4 GENERIC.MP#2 amd64
Compiler: clang version 16.0.6

To Reproduce
Steps to reproduce the behavior:

git clone git@github.com:microsoft/vcpkg-tool.git
mkdir vcpkg-tool/build
cmake -S vcpkg-tool -B vcpkg-tool/build
cmake --build vcpkg-tool/build

Expected behavior
cmake compiles a usable vcpkg binary
Failure logs
stdout.txt
Additional context
For most parts of it, OpenBSD is compatible with FreeBSD, which already has platform branches in vcpkg-tool to resolve these issues.

We did a few tool releases since this :)


FASTLANE

New Issue Checklist

[ ] Updated fastlane to the latest version
[ ] I have read the Contribution Guidelines

Issue Description
Complete output when running fastlane, including the stack trace and command used

You can use: --capture_output as the last commandline argument to get that collected for you

[INSERT OUTPUT HERE]
Environment
Please run fastlane env and copy the output below. This will help us help you :+1:
If you used --capture_output option please remove this block - as it is already included there.
[INSERT OUTPUT HERE]

@luohs  can  you provide a bit more information about your issue? thx


io.h header file

Hi,
the io.h file  of SDK, that is included in read_write.hpp file, is covered with header file (io.h) in order \windows-builds\packages\geos\include\geos
Compiling invironment: Windows 7, x64, Visual Studio 2015, project platform x64.

Sorry, I don't understand what you are asking here? Can you try to elaborate?

I get the error message by compiling:
Fehler	C3861	"_setmode": Bezeichner wurde nicht gefunden.		d:\builds\windows-builds\packages\libosmium\include\osmium\io\detail\read_write.hpp	 87
although the <io.h> header file is included.

@isowanderer could you verify if there several io.hs on your include path?
See the accepted solution here: https://social.msdn.microsoft.com/Forums/vstudio/en-US/6dda2264-ae01-4010-8cc0-b50bc937417b/setmode-visual-studio-2008-c?forum=vcgeneral

Closing this now, please re-open if this didn't fix it for you.


Introduce create method code action

Description
Currently we do not provide a code action for creating a method.  Diagnostic - undefined method 'print' in object 'Person'(BCE2013)
Describe your problem(s)
Scenario 1:
class Person {
    private int age = 0;

    public function getAge() returns int {
        return self.age;
    }

    public function setAge(int age) {
        self.age = age;
        self.print(age);
    }
}

Scenario 2:
public class Counter {
    private int n;

    public function init(int n) {
        self.n = n;
    }
}

public function main () {
   Counter obj = new (0);
   obj.getSome();
}

Scenario 3:
Creating the method In another file.
Describe your solution(s)
Create a method inside the class (or inside the object-constructor-expr for that usecase)
Related area
-> Editor
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

Encountered the below issues,

https://github.com/ballerina-platform/ballerina-lang/issues/38408
https://github.com/ballerina-platform/ballerina-lang/issues/38427

Initial impl in https://github.com/ballerina-platform/ballerina-lang/compare/master...mohanvive:ballerina-lang:create-method-ca


Check #691bug

From vNext_yarong_780
Added test file that fires setup and teardown failures

This change is

PR is added to the queue for testing as 2. task. (f0af700)

Hi @artemjackson, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

Test (Fail) 80241505 build is completed (f0af700)
See https://github.com/ThaliTester/TestResults/tree/80241505f0af700_Check__691bug_artemjackson/ for the logs

Should be submitted to vNext not master. Closed.


List Formatting

Currently, all the VMs and containers are put in one big list. I currently have ~30 total so the UI window extends past my screen. So it would be beneficial to add some formatting. Something along the lines of making the list scroll able or add multiple columns. Also differentiating between VMs and containers  would be helpful.

completion,
As this tool is intended for end-user VDI access differentiating between containers and vm's isn't really in the scope, however I did implement scrolling when going past 5 VMs in this commit


No Detections using Yolov4-sam-mish cfg

I'm trying to convert a yolov4-sam-mish weight file to a pb model for tensorflow. When I use detect.py I get no detections whatsoever. I've used yolov4-tiny trained on the same dataset and get detections the same detections as I would in darknet. Is the yolov4-sam-mish configuration not supported?

@AdamCuellar Hi, how you converted the SAM in AlexeyAB darknet weight to tflite? Can you give me some advice? tks

@KuoEuran I was unable to get the yolov4-sam-mish working in tflite. I ended up using the compiled darknet library and darknet.py from the AlexeyAB repo. I'm sure its possible to get the SAM module in tflite but I didn't have the time to implement it myself.

@AdamCuellar Thank you for your replying.


[FEATURE][DocBot] Docbot needs to call cluster bootstrap

Is your feature request related to a problem?
The DocBot class needs to call cluster bootstrap and store it as a variable so that the model parameters can be accessed by other methods of the docbot class.

Catch All Triage - 1 2 3 4 5


Upgrading C++ DataFrame to version 3.0.0

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[x] The "supports" clause reflects platforms that may be fixed by this new version.
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

How do I find out why the checks are failing?


To Telemetry, or not to Telemetry, that is the question

Describe the issue
As an ISV publishing our app to AppSource and also to OnPrem customers, we decided to add the instrumentation key to the app.json file so that we get extension level telemetry data in our telemetry resource on Azure.  This is great as long as we have data from production environments where people actually do things and we know that the telemetry data is actually meaningful.
However, we and our partners use our app also in Docker containers for development, testing, AL validation, build pipelines, etc. And all of those containers send telemetry data to that one Azure telemetry resource. Having telemetry data from Docker containers doesn't really help us because the data in there has not much of value. There is no aad tanant id for instance.
I understand that there are scenarios where you actually want to have telemetry enabled in Docker containers for testing purposes... but I don't think it should be enabled by default, tbqh.
My questions for @freddydk :

What is the easiest way to disable telemetry when creating new containers today?
Could you add -DisableTelemetry to New-NavContainer.ps1?

As this isn't something I can fix in ContainerHelper, I will close this issue.


Add Destination relay in error msg

https://github.com/threefoldtech/tfgrid-sdk-ts/issues/1210


Fix output signals of station control combinator.

Due to some typos, station control combinators are not providing the correct output signals. This PR fixes these.
Also added some comments and improved the Lua type annotations around the related code.

PR wasn't based on main branch, going to open a new one to fix it.


Review CAS/ENV digital strategies for relevant alignment

The ENV & CAS Digital Strategies offer a ton of alignment with this project. This task captures the identification of relevant digital goals and opportunities outlined in the strategies.

[x]  Review CAS digital strategy
[x] Review ENV digital strategy
[x]  Create documentation of relevant points
[x]  Upload documentation to Public Reporting Teams channel

Awesome - thank you @jaimiebutton ! I've left a couple of comments in the document for your consideration, namely:

This obviously aligns with the digital strategy, but it is so much more than simply digital...but also services. How can we ensure we keep this in mind?
Perhaps a separate task, but it would be good to also align to the roadmap to 2030.
How do we bring this all together in a nic visual way: e.g., digital strategy, Roadmap, etc? Thinking ahead to future presentations.
Can we use any of these goals/alignment opportunities to our KPIs?


OGCEndpoints output formats support enhancement

Overview
This PR adds formats support to all OGC endpoints and fixes endpooints mime types to match the requested output param.
Checklist

[x] Description of PR is in an appropriate section of the CHANGELOG and grouped with similar changes if possible

Closes #255
Closes #195

Merging to take care of https://github.com/geotrellis/geotrellis-server/issues/350


[ITensors] Remove extraneous backup files

Description
These files don't seem to be actually used anywhere and are dragging coverage stats down.
How Has This Been Tested?
Tests passed locally.
Checklist:

[x] My code follows the style guidelines of this project. Please run using JuliaFormatter; format(".") in the base directory of the repository (~/.julia/dev/ITensors) to format your code according to our style guidelines.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have added tests that verify the behavior of the changes I made.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] Any dependent changes have been merged and published in downstream modules.

Codecov Report

Merging #1091 (7a3ffc1) into main (27b9676) will decrease coverage by 19.18%.
The diff coverage is n/a.

:exclamation: Current head 7a3ffc1 differs from pull request most recent head e1d3c36. Consider uploading reports for the commit e1d3c36 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #1091       +/-   ##
===========================================
- Coverage   72.86%   53.69%   -19.18%
===========================================
  Files          94       85        -9
  Lines        9572     8223     -1349
===========================================
- Hits         6975     4415     -2560
- Misses       2597     3808     +1211

Impacted Files
Coverage Δ

src/ITensors.jl
100.00% <ø> (ø)

src/adapt.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/mps/adapt.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/mps/projmpo.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/mps/observer.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/autompo/matelem.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/autompo/qnmatelem.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/autompo/opsum_to_mpo.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/autompo/opsum_to_mpo_qn.jl
0.00% <0.00%> (-97.62%)
:arrow_down:

src/physics/autompo/opsum_to_mpo_generic.jl
0.00% <0.00%> (-95.75%)
:arrow_down:

... and 29 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Adding Sphinx Output to CI Builds

DRAFT for now. Still need to work through a bunch more, including:

Support the management package source generation that is encapsulated by doc/sphinx/generate_docs.py
Automation of the index
Write JS for the versioning drop down
Update layout to include space for versioning drop_down
Add link back to documentation hub
Other feedback
Work sphinx extension for transformation that Xiang is working on.

Leverage by using tox environment sphinx!
@Azure/azure-sdk-eng
@xiangyan99

Can one of the admins verify this patch?

Closing to re-open with different branch.


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
12 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
8 seconds
deprecated-base-image-check

✅ Succeeded
11 seconds
inspect-image

✅ Succeeded
17 seconds
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
11 seconds
sbom-json-check

✅ Succeeded
15 seconds
label-check

✅ Succeeded
14 seconds
optional-label-check

✅ Succeeded
5 seconds
show-summary


Portainer logs filled with registry auth warnings

Bug description
Logs are filled with "WRN images/digest.go:88 > cannot find registry auth for image "
Expected behavior
no warnings!
Portainer Logs
Provide the logs of your Portainer container or Service.
2023/02/01 10:59PM WRN images/digest.go:88 > cannot find registry auth for image | image=docker.io/portainer/portainer-ee:latest
Steps to reproduce the issue:

Start portainer
View logs3
See error

Technical details:

Portainer version: 2.16.2
Docker version (managed by Portainer): latest
Kubernetes version (managed by Portainer):  unknown?
Platform (windows/linux): Debian GNU/Linux 11 (bullseye)
Command used to start Portainer (docker run -p 9443:9443 portainer/portainer): sudo docker run -d
-p 8000:8000
-p 9000:9000
-p 9443:9443
--name=portainer
--restart=always
--pull=always
-v /var/run/docker.sock:/var/run/docker.sock
-v portainer_data:/data
portainer/portainer-ee:latest
Browser: any
Use Case (delete as appropriate): home
Have you reviewed our technical documentation and knowledge base? Yes

Additional context
I don't know if this is an issue but I would love my log files to not just constantly fill with auth issues.

@andsoitgoes
Thank you for the information.  I am going to investigate.  I will update you as I learn more.
Thanks!

Oh I did not see this!
Yes, I do!
As well, I tried logging into my Docker account to see if that would be of benefit and the moment I did, Portainer stopped reporting the images and they were all unable to be verified. I love this feature and I use it alongside watchtower to automatically update specific docker containers, but alas it's still throwing that error.
Thank you in advance for your help, I really appreciate it! Portainer has been a lifesafer, It has made my Docker maintenance immaculate in comparison to the less than joyful CLI docker commands.

We believe this issue has now been resolved. Are you able to confirm whether you're still experiencing this on the latest version of Portainer?


Make use of svg-escape function

We should introduce a function to escape SVGs. Some SVGs are not escaped, causing them not to render in IE:
https://github.com/twbs/bootstrap/blob/78c9c8bdd4cad5e6d1fc349d45b7503bad84a19b/scss/_variables.scss#L541
Some SVGs are escaped, but they are harder to read/maintain:
https://github.com/twbs/bootstrap/blob/78c9c8bdd4cad5e6d1fc349d45b7503bad84a19b/scss/_variables.scss#L547
I've used these functions in the past, they're freakin awesome to get this fixed: https://codepen.io/kevinweber/pen/dXWoRw?editors=1100

After we have this, we can tackle #29066 easier.


Add UserAgentTest to WebView2Tests

Adds a test that ensures the correct UserAgent is given by WebView2.

/azp run

/azp run


Fix job posting card layout

fix the margins and paddings for the job posting cards now that we're collecting dynamic data.

Need to limit the description to something like 150 characters.

Also need to show Posted Date


New emmet: css property width expands to widows??

VSCode Version: Code - Insiders 1.14.0-insider (73015a5e052e6de8ecd60eed2670bea81fad8cac, 2017-06-16T05:13:31.107Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author (truncated)
Version

path-intellisense
chr
1.4.2

gitlens
eam
4.1.2

LogFileHighlighter
emi
1.1.1

vscode-reveal
evi
0.0.9

php-debug
fel
1.10.0

php-intellisense
fel
1.4.1

auto-close-tag
for
0.4.2

gc-excelviewer
Gra
1.1.15

GitHubIssues
Hoo
0.1.2

composer
ika
0.5.0

smarty
imp
0.2.0

json-to-ts
Mar
1.4.3

vscode-apache
mrm
1.1.1

cpptools
ms-
0.11.4

php-docblocker
nei
1.2.0

vscode-versionlens
pfl
0.19.0

vscode-statusbar-json-path
ric
1.0.7

vscode-icons
rob
7.9.0

sharecode
Rol
0.4.1

code-spell-checker
str
1.2.0

vscode-todo-highlight
way
0.5.5

highlight-trailing-white-spaces
yba
0.0.2

(1 theme extensions excluded)

have fun.
// cc @ramya-rao-a

I am having a deja vu :)
w gives width
wi and anything further gives widows
This is coming from the way snippets are defined in https://github.com/emmetio/snippets/blob/master/css.json
@sergeche thoughts?
cc @sergeche

I'm starting to get similar results with other properties:

the default hint for font-family is font: fantasy; which is a very loose association and ruins the hinting system

wi and anything further gives widows
but then I would expect that the label updates too...

Not sure what expected behaviour should be if you type full property name and try to expand it as Emmet abbreviation. CSS itself has too much clashing properties/abbreviations. The goal of shortcuts is to provide shorthands for most commonly used properties. Users should learn these shortcuts by typing first letters and seeing autocomplete popup with available options. If the full property names will be considered in abbreviation resolving, they will collide with shorthands, especially ones with embedded shorthand values, like m-a for margin: auto.
So the ultimate goal is to provide matched snippets and expanded abbreviation with previews

Another example:

Indeed, this eager matching to the Emmet shorthands severely breaks (this) user expectations.
When I have almost complete property name, like "borde", "paddin", "margi", etc, I would assume that VS Code does the right thing and expands these widely-used properties. Instead, I get completely random and unrelated (at least that's how it looks) suggestions from Emmet.
My expectation would be that Emmet's suggestions should not always be the number 1 in the suggestion list, and sometimes they should be ranked lower as almost fully-matched widely used properties.
Suggesting "pgba" shorthand as a top choice for "paddin" is just wrong (at least, to me).

One solution I have in mind here is for the css completion provider (from the css extension that is built in) to call the emmet completion provider (See https://github.com/Microsoft/vscode/issues/29114#issuecomment-310232680) instead of the emmet extension.
As the css extension will also have in hand the other suggestions, it can add some logic to filter/sort appropriately.
I'll look into that next week.
Until then, I am pushing the below changes

The label in the completion item for emmet will now show the expanded abbreviation and not the abbreviation itself. With this, it will be more clear as to what exactly is the user choosing.
For sorting, the expanded abbreviation will be used and not the abbreviation itself.

The changes will be out in Monday's Insiders build.
Please give it a try then and let me know how the improvements work out for you.
Again, thanks for all the feedback!

temporary fixed with https://github.com/ramya-rao-a/vscode-emmet-helper/commit/2bbf01801f012b62761407ce38b83ec072ee79e1

I have logged #29113 to track the work of moving emmet completion provider to css language service.
Closing this issue for the June milestone as the improvements listed in https://github.com/Microsoft/vscode/issues/28933#issuecomment-310802231 should help most of the cases.

Note to verifier:

In css like files (css/scss/less), ensure that the suggestion label reflects the actual completion rather than just the abbreviation
Ensure the suggestion from emmet is sorted among the css suggestions

Hi @ramya-rao-a , many thanks for the improvements, they significantly improve the user experience w.r.t editing CSS/SCSS files!
The sorting of the suggestions is much, much better now.
There is one issue though (actually two), regarding snippets.
Say, I have  a snippet called bidi (for bi-directional SCSS mix-ins).
== Issue 1 ==
I type 'b' and see the following:

The help for my snippet bidi is wrong.
== Issue 2 ==
When I type the full name of the snippet, bidi in this case, it is still listed below the Emmet suggestion (which seems completely unrelated to the name of my snippet):

Should I file separate issues for that?

@vvs wow, thats seriously messed up, thanks for reporting!
This is a separate issue indeed. I have logged #29468

I'm still running into issues with emmet and specifically the width shorthand.
white-space and widows both get priority over width which arguably is one of the most commonly used properties. Is there any way we can reorder the priority and shift some of these lesser-used properties further down the list?
Gif below to show what I'm seeing...

@ekfuhrmann  You can set a custom emmet snippet for wid
{
    "css": {
        "snippets": {
            "wid": "width:${1};"
        }
    }
}

See  Using Custom Snippets in Emmet
Or you can take advantage of emmet and instead of typing wid you could say type w10 which would expand to width: 10px (you can use any number instead of 10)


forc-fmt associated_const_trait_default results in panic

forc fmt ./test/src/e2e_vm_tests/test_programs/should_pass/language/associated_const_trait_default/src/main.sw

thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: ParseFileError([Parse { error: ParseError { span: Span { src (ptr): 0x7fb023f0bde0, source_id: None, start: 41, end: 42, as_str(): ";" }, kind: ExpectedAnItem } }])', swayfmt/src/comments.rs:160:10
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

This file builds, so the formatter should definitely be able to format it.

Related https://github.com/FuelLabs/sway/issues/5006


HDDS-5349. Avoid usage of locks in listStatus.

What changes were proposed in this pull request?
Avoid usage of locks when iterating rocksdb iterator.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-5349
How was this patch tested?
Existing UT's.

/pending How about moving the BUCKET_LOCK in PREFIX layout


Remove a remaining old-style preprocessor from src/core/fonts.js (PR 7322 follow-up)

Note that this code was added after PR #7322 was opened, which thus explains why it was missed during rebasing.
Edit: Much easier reviewing with https://github.com/mozilla/pdf.js/pull/7726/files?w=1.

/botio unittest


chore(*): migrate samples/templates to Spin manifest v2

Updates apps under samples and templates to use the Spin manifest v2 syntax.
Note: I experienced some errors when running the Fermyon.PetStore sample, but this also occurred on main.  We may want to track this in another issue.  Hoping one of the reviewers can compare notes here...
Also currently blocked on https://github.com/fermyon/spin-dotnet-sdk/issues/53; the template currently relies on the dotted_pascal_case filter.
Assuming this is otherwise ready to go, it should wait until on/after the Spin 2.0 release.  Putting in Draft for now, but initial reviews appreciated.

I had forgotten about this one.  Rebased... should be ready for reviews


Remove hardcoded dependency

This is part of the fix of https://github.com/lampepfl/dotty/issues/6207


property where: add attributes to the syntax class

add attributes to the class so the macro can use the class instead of parsing on its own
(single point of control)

@sidprasad


使用内存缓存，分页查询中search key exists总是返回false

问题描述
使用内存缓存，分页查询缓存不生效，追进去发现SearchKeyExists总视返回false，确认了cacheKey生成出来是相同的，查询结果也相同
缓存配置
cache, _ := cache.NewGorm2Cache(&cacheConfig.CacheConfig{
	CacheLevel:           cacheConfig.CacheLevelAll,
	CacheStorage:         cacheConfig.CacheStorageMemory,
	CacheSize:            5000,
	InvalidateWhenUpdate: true,  // when you create/update/delete objects, invalidate cache
	CacheTTL:             60000, // 60000 ms
	CacheMaxItemCnt:      60000, // if length of objects retrieved one single time
	DebugMode:            true,
	// exceeds this number, then don't cache
	})

日志
2022-09-09 05:44:32.635 [INFO] [BeforeQuery] search key exists ? false
2022-09-09 05:44:32.635 [INFO] [BeforeQuery] parse primary keys = []
2022-09-09 05:44:32.907 [INFO] [AfterQuery] start to set primary cache for kvs: []
2022-09-09 05:44:32.907 [INFO] [AfterQuery] start to set search cache for sql: SELECT count(*) FROM "peiz"."p_order" WHERE "p_order"."mc_id" = $1 AND "status" NOT IN ($2,$3,$4) AND "p_order"."deleted_at" IS NULL
2022-09-09 05:44:32.907 [INFO] [AfterQuery] set cache: 37
2022-09-09 05:44:32.907 [INFO] [AfterQuery] sql SELECT count(*) FROM "peiz"."p_order" WHERE "p_order"."mc_id" = $1 AND "status" NOT IN ($2,$3,$4) AND "p_order"."deleted_at" IS NULL cached
22-09-09 05:44:32 [DEBU] SELECT count(*) FROM "peiz"."p_order" WHERE "p_order"."mc_id" =<PHONE_NUMBER>84954368 AND "status" NOT IN ('start','deleted','panic') AND "p_order"."deleted_at" IS NULL [272.238116ms]
2022-09-09 05:44:32.907 [INFO] [BeforeQuery] search key exists ? false
2022-09-09 05:44:32.907 [INFO] [BeforeQuery] parse primary keys = []
2022-09-09 05:44:33.318 [INFO] [AfterQuery] start to set search cache for sql: SELECT * FROM "peiz"."p_order" WHERE "p_order"."mc_id" = $1 AND "status" NOT IN ($2,$3,$4) AND "p_order"."deleted_at" IS NULL ORDER BY create_time desc
2022-09-09 05:44:33.319 [INFO] [AfterQuery] start to set primary cache for kvs: [{Key:0x40000b7258 Value:{响应内容}]
2022-09-09 05:44:33.319 [INFO] [AfterQuery] sql SELECT * FROM "peiz"."p_order" WHERE "p_order"."mc_id" = $1 AND "status" NOT IN ($2,$3,$4) AND "p_order"."deleted_at" IS NULL ORDER BY create_time desc  cached
22-09-09 05:44:33 [DEBU] SELECT * FROM "peiz"."p_order" WHERE "p_order"."mc_id" =<PHONE_NUMBER>84954368 AND "status" NOT IN ('start','deleted','panic') AND "p_order"."deleted_at" IS NULL ORDER BY create_time desc  [411.897497ms]
22-09-09 05:44:33 [INFO] <IP_ADDRESS> "POST /api/p/order/page" 200 (691ms)
2022-09-09 05:44:40.891 [INFO] [BeforeQuery] search key exists ? false
2022-09-09 05:44:40.891 [INFO] [BeforeQuery] parse primary keys = []
2022-09-09 05:44:40.95 [INFO] [AfterQuery] start to set primary cache for kvs: []
2022-09-09 05:44:40.95 [INFO] [AfterQuery] start to set search cache for sql: SELECT count(*) FROM "peiz"."p_order" WHERE "p_order"."mc_id" = $1 AND "status" NOT IN ($2,$3,$4) AND "p_order"."deleted_at" IS NULL
2022-09-09 05:44:40.95 [INFO] [AfterQuery] set cache: 37
2022-09-09 05:44:40.95 [INFO] [AfterQuery] sql SELECT count(*) FROM "peiz"."p_order" WHERE "p_order"."mc_id" = $1 AND "status" NOT IN ($2,$3,$4) AND "p_order"."deleted_at" IS NULL cached
22-09-09 05:44:40 [DEBU] SELECT count(*) FROM "peiz"."p_order" WHERE "p_order"."mc_id" =<PHONE_NUMBER>84954368 AND "status" NOT IN ('start','deleted','panic') AND "p_order"."deleted_at" IS NULL [59.785883ms]
2022-09-09 05:44:40.95 [INFO] [BeforeQuery] search key exists ? false
2022-09-09 05:44:40.95 [INFO] [BeforeQuery] parse primary keys = []
2022-09-09 05:44:41.091 [INFO] [AfterQuery] start to set search cache for sql: SELECT * FROM "peiz"."p_order" WHERE "p_order"."mc_id" = $1 AND "status" NOT IN ($2,$3,$4) AND "p_order"."deleted_at" IS NULL ORDER BY create_time desc
2022-09-09 05:44:41.092 [INFO] [AfterQuery] start to set primary cache for kvs: [{Key:0x400050b2a8 Value:{响应内容}}]
2022-09-09 05:44:41.092 [INFO] [AfterQuery] sql SELECT * FROM "peiz"."p_order" WHERE "p_order"."mc_id" = $1 AND "status" NOT IN ($2,$3,$4) AND "p_order"."deleted_at" IS NULL ORDER BY create_time desc  cached
22-09-09 05:44:41 [DEBU] SELECT * FROM "peiz"."p_order" WHERE "p_order"."mc_id" =<PHONE_NUMBER>84954368 AND "status" NOT IN ('start','deleted','panic') AND "p_order"."deleted_at" IS NULL ORDER BY create_time desc  [141.459062ms]
22-09-09 05:44:41 [INFO] <IP_ADDRESS> "POST /api/p/order/page" 200 (203ms)

基本信息
gorm.io/gorm v1.23.8
github.com/Pacific73/gorm-cache v1.0.1-0.20220620032759-a6797d3696a5
v1.0.0 没有实现Plugin接口，用hash直接加载的合并了Plugin接口的pr

查询不用很复杂，只要不是主键查询就不生效了

SELECT * FROM "peiz"."s_param" WHERE "s_param"."clazz" = 'p_lock_time'

已解决，请检查pr是否有问题，

pr已merge


Nested Section Lists cause continuous re-render; dropped data; no sticky headers.

Nested Section Lists cause continuous re-render with dropped data;
Sticky headers do not work on nested section list
React Native version:
System:
    OS: Windows 10 10.0.17134
    CPU: (4) ia32 Intel(R) Pentium(R) CPU G4560T @ 2.90GHz
    Memory: 1.11 GB / 7.43 GB
  Binaries:
    Node: 12.10.0 - C:\Program Files (x86)\nodejs\node.EXE
    Yarn: 1.10.1 - C:\Users\mjfiandaca\AppData\Roaming\npm\yarn.CMD
    npm: 6.13.4 - C:\Program Files (x86)\nodejs\npm.CMD
  SDKs:
    Android SDK:
      API Levels: 23, 25, 26, 27, 28
      Build Tools: 23.0.1, 25.0.1, 26.0.3, 27.0.3, 28.0.0, 28.0.3
      System Images: android-23 | Intel x86 Atom, android-23 | Google APIs Intel x86 Atom, android-23 | Google APIs Intel x86 Atom_64, android-26 | Google APIs Intel x86 Atom, android-26 | Google APIs Intel x86 Atom_64, android-27 | Google APIs Intel x86 Atom, android-27 | Google Play Intel x86 Atom, android-28 | Intel x86 Atom, android-28 | Intel x86 Atom_64, android-28 | Google APIs Intel x86 Atom, android-28 | Google APIs Intel x86 Atom_64, android-28 | Google Play Intel x86 Atom
  IDEs:
    Android Studio: Version  <IP_ADDRESS> AI-191.8<IP_ADDRESS>10548
  npmPackages:
    react: 16.9.0 => 16.9.0
    **react-native: 0.61.5 => 0.61.5**

Steps To Reproduce

Copy supplied code into App.js (this was reproduced with a brand-new react native playground; however, it has manifested in earlier releases for my actual App).
Look. See.

Describe what you expected to happen:
The screen should render all data in the nested Section List.
Scrolling the data should not cause screen jumping.
Scrolling the data should maintain sticky headers for the nested section list.
Snack, code example, screenshot, or link to a repository:
import React from 'react';
import {
  SafeAreaView,
  StyleSheet,
  View,
  Text,
  StatusBar,
  SectionList,
} from 'react-native';

import {Colors} from 'react-native/Libraries/NewAppScreen';

const accordionTypes = ['David Bowie', 'Beatles', 'The Clash', 'HadesTown'];
const songList = ['Song One', 'Song Two', 'Song Three', 'Song Four'];

const sections = [
  {
    title: {
      item: 1,
      section: accordionTypes[0],
    },
    data: [
      [
        {
          title: {
            item: 'a',
            section: songList[0],
          },
          data: [
            {
              urn: 1,
              text: `Time, it's waiting in the wings.`,
            },
            {urn: 2, text: 'It speaks of senseless things.'},
            {urn: 3, text: `Its trick is you and me.`},
            {
              urn: 4,
              text: `Time, it's waiting in the wings.`,
            },
            {urn: 5, text: 'It speaks of senseless things.'},
            {urn: 6, text: `Its trick is you and me.`},
          ],
        },
        {
          title: {
            item: 'a',
            section: songList[1],
          },
          data: [
            {urn: 7, text: `Don't talk of dust and roses`},
            {urn: 8, text: 'Or should we powder our noses'},
            {
              urn: 9,
              text: `Don't live for last year's capers`,
            },
            {urn: 10, text: 'Give me steel give me steel.'},
            {urn: 11, text: `Give me pulses unreal.`},
          ],
        },
      ],
    ],
  },
  {
    title: {
      item: 1,
      section: accordionTypes[1],
    },
    data: [
      [
        {
          title: {
            item: 'a',
            section: songList[0],
          },
          data: [
            {
              urn: 12,
              text: 'It happened once before',
            },
            {urn: 13, text: 'When I came to your door'},
            {urn: 14, text: 'No reply'},
          ],
        },
        {
          title: {
            item: 'a',
            section: songList[1],
          },
          data: [
            {urn: 15, text: `Elanor Rigby`},
            {
              urn: 16,
              text: 'Picks up the rice in a church where a wedding has been',
            },
            {urn: 17, text: 'Lives in a dream'},
            {urn: 18, text: `Waits at the window`},
            {
              urn: 19,
              text: 'Wearing a face that she keeps in a jar by the door',
            },
            {urn: 20, text: 'Who is it for?'},
            {urn: 21, text: `All the lonely people`},
            {
              urn: 22,
              text: 'Where do they all come from?',
            },
            {urn: 23, text: 'All the lonely people'},
            {urn: 24, text: `Where do they all belong?`},
          ],
        },
      ],
    ],
  },
];

renderSectionHeader = info => {
  const {item, section} = info.section.title;
  return renderHeader(item, section);
};

renderHeader = (item, section) => {
  return (
    <View style={styles.sectionContainer}>
      <Text style={styles.sectionTitle}> {section} </Text>
    </View>
  );
};

renderSectionItem = data => {
  return renderItem(data.item, data.item.item?.urn);
};

keyExtractor = item => {
  const unique = new Date().getTime() + Math.random();
  return unique.toString();
};

renderItem = (data, item) => {
  return (
    <SectionList
      sections={data}
      stickySectionHeadersEnabled={true}
      renderSectionHeader={renderSubSectionHeader}
      renderItem={renderSubSectionItem}
      keyExtractor={keyExtractor}
    />
  );
};

renderSubSectionHeader = info => {
  const {item, section} = info.section.title;
  return renderSubHeader(item, section);
};

renderSubHeader = (item, section) => {
  return (
    <View style={styles.sectionContainer}>
      <Text style={styles.sectionTitle}> {section} </Text>
    </View>
  );
};

renderSubSectionItem = data => {
  return renderSubItem(data.item);
};

renderSubItem = data => {
  return (
    <View>
      <Text> {data.text} </Text>
    </View>
  );
};

const App = () => {
  return (
    <>
      <StatusBar barStyle="dark-content" />
      <SafeAreaView>
        <SectionList
          sections={sections}
          stickySectionHeadersEnabled={true}
          renderSectionHeader={renderSectionHeader}
          renderItem={renderSectionItem}
          keyExtractor={keyExtractor}
        />
      </SafeAreaView>
    </>
  );
};

const styles = StyleSheet.create({
  scrollView: {
    backgroundColor: Colors.lighter,
  },
  engine: {
    position: 'absolute',
    right: 0,
  },
  body: {
    backgroundColor: Colors.white,
  },
  sectionContainer: {
    marginTop: 32,
    paddingHorizontal: 24,
    backgroundColor: '#226600',
  },
  sectionTitle: {
    fontSize: 24,
    fontWeight: '600',
    color: '#ffffff',
  },
  sectionDescription: {
    marginTop: 8,
    fontSize: 18,
    fontWeight: '400',
    color: Colors.dark,
  },
  highlight: {
    fontWeight: '700',
  },
  footer: {
    color: Colors.dark,
    fontSize: 12,
    fontWeight: '600',
    padding: 4,
    paddingRight: 12,
    textAlign: 'right',
  },
});

export default App;

nested-section-lists.zip
Not sure whether this is what you are looking for, since I've never used snack before.
Funny, the red box was because I used a new react native app, and just stuck with the standard imports...duh.

Sorry, here is a link to the actual snack:
https://snack.expo.io/@mjfiandaca/nested-section-lists

The label 'needs repro' was added to this issue, but then I provided a snack illustrating the issue. Has anyone been able to look at this?

This remains an issue. Has anyone been able to look into this? Thank you.

I had the same problem，Nested SectionList requirements

https://github.com/facebook/react-native/issues/20390
@mjfiandaca That was the problem two years ago

SectionList->ScrollView(stickyHeaderIndices = [0])->Text（is ok）,FlatList
Just have to make do

#20390
@mjfiandaca That was the problem two years ago

I saw that former issue was closed two years ago. As stale. It was never fixed. I am unclear what your three comments mean on this issue. I am already making do. I would like nested section lists to work.
Thanks.


Feature/vuex refactor

@kacperknapik  please resolve the conflicts

@filrak done

Who's the brave one to merge this to master? :) @filrak @kacperknapik

let me test it one more time @filrak @pkarw

missing mutation added, ready to merge

@kacperknapik have you tested the variants? Does everything work like it should?

@pkarw variants work correctly imho, no errors show up, I think it is ready to go. you can double check that if you want

OK, I'll merge it for you @kacperknapik


Constance settings integrated

Added constance functionality into settings.py

Thanks for the contribution here. I don't think we're going to use this as is for this application but we did integrate it into one of our others in https://github.com/tl-its-umich-edu/my-learning-analytics/pull/1176. I'm just going to close this as it has a conflict now and would need to be updated if we implemented this.


Design for login dialog

Currently the login dialog is implemented is the easiest possible way.

It has some room for improvement in terms of elegance. Maybe there could be a welcome message. Maybe the web elements could be re-arranged. Maybe the colors could be fine-tuned. Etc.
Acceptance criteria

The new design adheres to the "maximum simplicity" principle, in order to keep the attention of the hospital staff on their actual work instead on the software.

Hints

See https://github.com/ksch-workflows/ksch-workflows/blob/master/user-interface/src/main/resources/static/ksch/Login.html
If you check out the source code, the login form can be previewed without the need to start the application. Just open the file in a web browser.

Resolved with https://github.com/ksch-workflows/ksch-workflows/pull/34


Problem with new version of libs

There is a problem in log.py, in new version of libs, its not possible to call 'from tensorflow.summary import FileWriter' and to call function  self._write_logs(stats, step)

use this instead from tensorboardX import FileWriter

Fixed by 32297e073bf0cf62a4b247bc962e7a58fdc1b731. Thanks for the report.


Switch to custom tkinter

Hello:)
I have switched to custom tkinter which makes the GUI look more modern.
Preview:

@Bashamega
Thanks Dude 😻❤

Your welcome.


Sign the seat count during premium upgrade

@timabbott This is ready for a review.

Merged, thanks @hackerkid! A few changes:

Changed generate_key to zerver.lib.utils.generate_random_token. (The generate_key code in confirmation is for a special case, when we're e.g. optimizing for a short URL.)
Changed "annual" to Plan.CLOUD_ANNUAL
Left a TODO in test_initial_upgrade to test that we're doing the signing and recovery correctly.


fail2ban_jail resource not working (for SSH service)

Cookbook version
6.0.0
Chef-client version
14.12.9
Platform Details
CentOS 7
Scenario:
Create Fail2Ban Jail Configuration using fail2ban_jail resource (here for SSH service).
Steps to Reproduce:
fail2ban_jail 'ssh' do
  ports %w(ssh)
  filter 'sshd'
  logpath node['fail2ban']['auth_log']
  maxretry 6
end

Expected Result:
Created Fail2Ban Jail Configuration (here for SSH service).
Actual Result:
The fail2ban_jail resource does not provide a filter property. The following may work:
property :filter, String, required: true

Results in the following error:
NoMethodError
-------------
undefined method `filter' for Custom resource fail2ban_jail from cookbook fail2ban

Cookbook Trace:
---------------
 /tmp/kitchen/cache/cookbooks/fail2ban-wrapper/recipes/default.rb:11:in `block in from_file'
 /tmp/kitchen/cache/cookbooks/fail2ban-wrapper/recipes/default.rb:9:in `from_file'

Relevant File Content:
----------------------
/tmp/kitchen/cache/cookbooks/fail2ban-wrapper/recipes/default.rb:

 6:
 7:  include_recipe 'fail2ban'
 8:
 9:  fail2ban_jail 'ssh' do
10:    ports %w(ssh)
11>>   filter 'sshd'
12:    logpath node['fail2ban']['auth_log']
13:  end
14:

System Info:
------------
chef_version=14.12.9
platform=centos
platform_version=7.6.1810
ruby=ruby 2.5.5p157 (2019-03-15 revision 67260) [x86_64-linux]
program_name=/opt/chef/bin/chef-client
executable=/opt/chef/bin/chef-client

Additonal Improvements:
Additionally the wrong attribute is given to internal template resource of fail2ban_jail resource. The problematic code is the following line:
# ...
property :source, String, default: 'jail.erb'
# ...
template "/etc/fail2ban/jail.d/50-#{new_resource.jail}.conf" do
  # ...
  source new_resource.filter # Prolematic code
  # ...
end
# ...

Should be:
# ...
property :source, String, default: 'jail.erb'
# ...
template "/etc/fail2ban/jail.d/50-#{new_resource.jail}.conf" do
  # ...
  source new_resource.source # Fix
  # ...
end
# ...

Without this fix the fail2ban_jail resource will not work properly. See: https://github.com/chef-cookbooks/fail2ban/blob/v6.0.0/resources/jail.rb#L33

Problem still exists, any hints yet?

@djessich please feel free to create a PR which addresses this. We can take a look and merge it in!


[Automation] Generate Fluent Lite from workloads#package-2023-04

[Automation] Generate Fluent Lite from workloads#package-2023-04

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-resourcemanager-workloads


test if posgres would make redis obsolete

postgres has internal types called inet and cidr which could theoretically be used the same way as I use my goripr implementation but also extended to ipv6.
Would be interesting to how postgres would manage deduplication/aggregation of overlapping ip ranges

Might as well check out https://github.com/nutsdb/nutsdb/blob/master/docs/user_guides/data-structure.md#sorted-set

nutsdb only supports sorting of max 64 bits (float64)


How do I work with two endpoints?

Currently I'm using something like the following to work with two different endpoints that I switch between in my work (a local and a remote), with the consequence that I (need to manually) overwrite the schema file when I switch between them.
Is there a way to do this with a different schema file for each endpoint? What is the correct idiom for working with (and switching between) two endpoints?
{
  "name": "My Schema",
  "schemaPath": "_schema.graphql",
  "extensions": {
    "endpoints": {
      "Local GraphQL Endpoint": {
        "url": "http://localhost:5000",
        "headers": {
          "user-agent": "JS GraphQL"
        },
        "introspect": true
      },
      "Remote GraphQL Endpoint": {
        "url": "http://my.remote.io",
        "headers": {
          "user-agent": "JS GraphQL"
        },
        "introspect": true
      }
    }
  }
}

You can now specify separate projects inside your config file as show here in the docs.


Provide/support "module" exports

It's a minor inconvenience to have the types break when writing the config file, since you have to finagle the export to get it to work in a "module":true project.
Personally I like the tsup cli tool for single-command config-less TS builders when I need to support both Module and CommonJS exports.

Are you talking about volar.config.js? If so you may want to exclude it in tsconfig.

I'm thinking of refactoring all plugins from export = to export default, I'm not entirely sure what your problem is, but I'm guessing this might handle yours.


Telescope quick fix list cannot work correctly

Version confirmation

[X] Confirm

Following prerequisites

[X] Confirm

Not a user config issue

[X] Confirm

Neovim version
0.9.0
Operating system/version
macOS 13.5.1
Terminal name/version
Warp
$TERM environment variable
xterm-256color
Branch info
main (Default/Latest)
Fetch Preferences
SSH (use_ssh = true)
How to reproduce the issue

Open nvim with Alpha and no other tabs
Open Telescope
Search something and use C+q to add all items in the search result to the quick fix list
Double click any item in the quick fix list

Expected behavior
In step 2, there won't be any error message.
In step 3, the file should display in the main window.
Actual behavior
In step 2:

In step 3:

Additional information
No response

The error of indent-blankline is fixed in 0ce5f35ef07bdff93bd31e8a979dc9801d0e0f2d.
The second issue maybe have to do with alpha-nvim itself. I found a similar thread in its discussions: https://github.com/goolord/alpha-nvim/discussions/195

I fixed it by using nvim-bqf and it's a awesome plugin.

Thanks I'll try it!!

I've tried nvim-bqf and it's pretty good! Thanks for the recommendation! And I don't care about the original secondary issue since nvim-bqf works greatly. If nobody else cares, feel free to close this issue.


NOJIRA: Improve uniqueness of file naming for client coverage reports

It's a pretty rare case but we've experienced some inconsistencies in gpii-app renderer processes coverage which makes use of the gpii.testem.coverage.express (coverageServer.js).
These are some changes of a collision of random numbers which are currently used for uniqueness of filenames. We could use a timestamp instead which should better ensure uniqueness.

Apologies for the extreme delay in looking at this.   I think the changes are fine, but would like to see it run in CI.  Can you merge with upstream main to pick up the CI configuration?

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add model_sql helper to enable isolated unit testing

Improves Unit testing capabilities.
Enabler for: https://github.com/fishtown-analytics/dbt/issues/2354
Discussed here: https://discourse.getdbt.com/t/testing-with-fixed-data-set/564/8
Related: https://discourse.getdbt.com/t/state-of-testing-in-dbt/1778
Description
It didn't seem to be a good(relatively speaking) method for testing models in isolation. The best was introducing branching conditions in the model based on global settings which at most allowed for 1 test / model.
This PR introduces a new helper-method model_sql() (the name is debatable, in previous discussions it was called cte)  which returns the compiled SQL code of a model but under a different context (allows one to overrides var/ref/source ). Models can be tested without changes outside test code and multiple tests can be build for the same model.
I think think approach is a cleaner approach to (unit) testing than what has been proposed so far.

Pure dbt (Jinja/SQL)
No need for custom macros.
No additional plugins (if this pr gets merged)
“Full" control per test
No changes/branching/flags required in production models (!)
Supports all data sizes.

Example use case:
Model
models/addder.sql
    SELECT
      *,
      a+b + {{ var ("base", "0") }} AS sum
    FROM {{ ref('source_data') }}

Test
tests/adder_0001.sql
{% set model_to_test = model_sql(
    'model.adder',  {
      'var': {
        'base': '1'
      },
      'ref': {
        'source_data': 'mocked_source_data'
      }})
%}

WITH
mocked_source_data AS (
  SELECT
    *
  FROM
  -- bigquery construct; same effect as union all select ....
  UNNEST([
     struct(1 as a,  2 as b,  4 as expectedSum),
     struct(2 as a,  2 as b,  5 as expectedSum),
     struct(0 as a,  0 as b,  1 as expectedSum),
     struct(1 as a, -2 as b, 0 as expectedSum)
   ])
),

final AS (
  {{ model_to_test }}
)

-- Some SQL assertion based on the specific mocked data.
SELECT * FROM final WHERE sum != expectedSum

_I haven't been writing python for some 6 years and and this is the first time interacting the the DBT core, as such the PR is not  yet complete. I would like to get some feedback before investing more time into this. No point in "polishing a turd".
If the maintainer(s) think it holds value i don't mind ticking all the boxes below (or hand it over to someone else)_
Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

@drewbanin I did sign the CLA some time ago but the PR seems stuck in this state. Anything you can help with?

@cla-bot re-check

@cla-bot check

@Zatte Apologies, the delay here is completely on me. I'm playing catch-up this week; I'm going to do my best to take a detailed look and give you feedback soon.

@jtcohen6 : Just a monthly ping, hoping this is not forgotten until the end of times :)


Add support for the PodNodeSelector admission controller

What this PR does / why we need it:
Add support for the PodNodeSelector admission controller.
The admission controller is enabled by enabling the appropriate feature, such as:
...
features:
  podNodeSelector:
    enable: true
    config:
      configFilePath: "./podnodeselector.yaml"

The podnodeselector.yaml file is located on the local file system and contains the plugin configuration. More about the configuration format can be found in the Kubernetes docs.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #852
Special notes for your reviewer:

To make the kube-system components able to get scheduled on all nodes, an empty scheduler.alpha.kubernetes.io/node-selector annotation is added to the kube-system namespace. The kube-system namespace mustn't be restricted in the plugin configuration, as pods such as CNI and kube-proxy must be scheduled on all nodes.
Before the kube-system namespace gets annotated, kubeadm creates deployments/daemonsets for the needed components. The underlying pods have the incorrect node selectors and therefore are stuck in the Pending state. To resolve this problem, all pending pods in the kube-system namespace are removed, so they get recreated with the correct node selector.
The AdmissionConfiguration API has been backported, so we don't have to import the k8s.io/apiserver package. Importing the package would require us to bump all Kubernetes dependencies to v1.18, as well as, controller-runtime to v0.5.0 or newer.

Does this PR introduce a user-facing change?:
Add support for the PodNodeSelector admission controller
BREAKING: Make paths for PodNodeSelector and StaticAudit configs relative to the manifest

/assign @kron4eg

@kron4eg PTAL

Everything looks good, except one part that really bothers me, file references (namely ConfigFilePath fields), those are points to external resources that we have to dereference somehow. I know we already doing this and I'm concern about this too :)
Yet, the API design discussion probably doesn't belongs to this PR.
/lgtm
/approve


Use zuul clonned dataplane-operator code for edpm deploy

Currently in Zuul CI, The default install_yamls default vars for DATAPLANE_REPO is pointing to
"https://github.com/openstack-k8s-operators/dataplane-operator.git". During make edpm_deploy, the dataplane-operator code is clonned from github and the job is not testing the actual code.
By setting DATAPLANE_REPO value under cifmw_edpm_deploy_env var, make sure zuul clonned dataplane-operator code is used.
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[ ] Appropriate documentation exists and/or is up-to-date:

[ ] README in the role
[ ] Content of the docs/source is reflecting the changes

/approve


Login Page with react router, redux, redux saga

Login screen

Hello, @MatheusLima7! We have no words to thank you for your time and effort in working on this issue. Unfortunately, we internally discussed and decided to go with #32 at this time to close this specific issue. We encourage you to keep looking at our issues as we will update them for the hacktoberfest event.
Your PR was very good, by the way! 😊


Unique email

Hi there it looks like, I can sign up multiple times with the same email address, is there a way to prevent this?

You need to add a unique index to the email field. It's usually done automatically if you ran mix pow.install to generate the migration file.

Thanks for your reply.
I was integrating pow into an existing project and I added some more fields into the index. This seemed to produce my problem. After separating them the "issue" is gone. So all fine, thanks for your advise.


fix: update tag patterns to support new XML TS tags

It seems the upstream Treesitter tags for xml was updated and is no longer 100% identical to the html syntax tree.
XML Tree according to :InspectTree

HTML Tree according to :InspectTree

Notice the tags now differ between the two.
Closes #163

Yep, applying this patch fixes the issue.

Hey @windwp, pinging you to get your thoughts on this PR if you have time.
It's been working well for me this far.
Thanks 🙂

For anyone subscribed to this PR, please see my comment: https://github.com/windwp/nvim-ts-autotag/issues/163#issuecomment-2107013155
Gist is I'll be maintaining a fork @ https://github.com/PriceHiller/nvim-ts-autotag/ until such a time the author comes back.

Closed in favor of #173


Add BLAKE3 hash function

We should add wrappers for BLAKE3 hash function into hash module. These could be similar to the ones implemented in Winterfell. In terms of variants, would be great to have the following:

Blake3_256 - same as in Winterfell.
Blake3_192 - same as in Winterfell.
Blake3_160 - a version of BLAKE3 with output truncated to 20 bytes.

To clarify; since they already implement Hasher, should we just re export Blake3_192 from winter crypto?

We could re-export both Blake3_256 and Blake_192 from winter-crypto, but I'd prefer to just copy the code here (it's not a lot of code). This way, all 3 implementations are in the same place, and we avoid the risk of them diverging later on.

Closed by #14


website (bad link)

https://www.haskell.org/alex/
hyperlink on cabal-install should be https://wiki.haskell.org/Cabal-Install

https://github.com/vaerksted/vaerksted.github.io
A temporary location of a validated update.
No attempt was made to update the links or textual content.

Thanks @askeblad, for the alert!
Actually, I do not know where the source code of the page https://www.haskell.org/alex/ resides.

I have not found it in this repository.
No further hints at https://github.com/haskell-infra/www.haskell.org/#details-alex

Who knows about this? @simonmar ? @Ericson2314 ?

github.com/haskell/haskell.github.io
with alex and happy source code on GitHub their webpages could be relocated to a newly created repo as above

github.com/haskell/haskell.github.io

This webpage does not exist, @askeblad.  Which one did you mean?

If the proposed new repo is created please do not require two-factor authentication on invitations soliciting new members.

The original idea for the new repo is found in haskell-infra/www.haskell.org issue #181.
I don't herein link to that issue as is is not pertinent to this current ticket.

I'd like to understand first how the current setup works, so I reported upstream:

https://github.com/haskell-infra/www.haskell.org/issues/183

source/mechanics of the alex and happy subsites

alex and happy
right clicking on the above webpages and from the appearing context menus clicking to View Page Source in new browser tabs
mostly the same structure could be retained (were the source copied to proposed new repo haskell/haskell.github.io)
e.g., from the alex webpage
doc/html/index.html
and from the foregoing file
doc/html/fptools.css

@andreasabel
https://github.com/vaerksted/vaerksted.github.io
links updated
https://vaerksted.github.io/alex.html
according to the Maintainer discretion
-
Simon Marlow <a<EMAIL_ADDRESS>+
Simon Marlow <a<EMAIL_ADDRESS>
you can use if you now have write authority to haskell.org

closed due to age

Thanks for your efforts, @askeblad.  I got now access to the Alex homepage and will fix it.  Ctd at #128.


Move QueryParamService to Zeta (Breaking Change)

The QueryParamService shall be migrated from FMAN to Zeta.

Service will be kept deprecatedly inside FMAN for the next releases and will then be deleted.

Attention
Keep issue open until the service has been removed from FMAN.

Must be kept open to finally remove the deprecated service four major releases later.


[DRIVER/FORMAT] support GeoPDF

similarly to https://github.com/qgis/QGIS-Mac-Packager/issues/75

I tried the zipped file in your previous comment.

this should be supported by input 1.3.0

peter.petrik/test-geopdf


go 1.16

ci(deps): bump Go to 1.16
ci(deps): bump GolangCI-Lint

Codecov Report

Merging #31 (c1c0ba5) into master (d67dd5e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #31   +/-   ##
=======================================
  Coverage   57.66%   57.66%
=======================================
  Files           6        6
  Lines         770      770
=======================================
  Hits          444      444
  Misses        260      260
  Partials       66       66


Running VSCode on a remote machine (remote display)

VSCode Version:

test@test:~$ code --version
1.6.1
9e4e44c19e393803e2b05fe2323cf4ed7e36880e

OS Version:

test@test:~$ uname -a
Linux test 4.2.0-38-generic #45~14.04.1-Ubuntu SMP Thu Jun 9 09:27:51 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
Steps to Reproduce:

Install vscode on a headless Linux machine
Install XMing Server on a Windows machine
connect to the Linux machine and export the display properly (test with xterm or any other X11 app)
Run vscode --verbose

Output:
[6492:1021/154155:ERROR:browser_main_loop.cc(231)] Running without the SUID sandbox! See https://chromium.googlesource.com/chromium/src/+/master/docs/linux_suid_sandbox_development.md for more information on developing with the sandbox on.
[6492:1021/154158:WARNING:audio_manager.cc(314)] Multiple instances of AudioManager detected
[6492:1021/154158:WARNING:audio_manager.cc(275)] Multiple instances of AudioManager detected
[6634:1021/154158:ERROR:gl_surface_glx.cc(386)] GLX 1.3 or later is required.
[6634:1021/154158:ERROR:gl_surface_x11.cc(252)] GLSurfaceGLX::InitializeOneOff failed.
[6634:1021/154158:ERROR:gpu_child_thread.cc(369)] Exiting GPU process due to errors during initialization
{ Error: ENOENT: no such file or directory, open '/home/test/.config/Code/storage.json'
at Error (native)
at Object.fs.openSync (original-fs.js:640:18)
at Object.fs.readFileSync (original-fs.js:508:33)
at e.load (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:9:2059)
at e.getItem (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:9:1597)
at e.handleUpdated (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:9:11350)
at new e (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:9:11251)
at v (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:4:28052)
at t._createInstance (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:8:13168)
at t._createAndCacheServiceInstance (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:8:14143)
errno: -2,
code: 'ENOENT',
syscall: 'open',
path: '/home/test/.config/Code/storage.json' }
[main 3:41:58 PM] Starting VS Code in verbose mode
[main 3:41:58 PM] from: /usr/share/code/resources/app
[main 3:41:58 PM] args: { _: [],
paths: [],
performance: false,
verbose: true,
debugPluginHost: undefined,
debugBrkPluginHost: undefined,
logExtensionHostCommunication: false,
'new-window': false,
'reuse-window': false,
goto: false,
diff: false,
extensionHomePath: undefined,
extensionDevelopmentPath: undefined,
extensionTestsPath: undefined,
'disable-extensions': false,
'open-url': undefined,
locale: undefined,
wait: false }
[6492:1021/154158:ERROR:browser_main_parts.cc(126)] X IO error received (X server probably went away)

Duplicate https://github.com/Microsoft/vscode/issues/3451


More ASTScope cleanups

@swift-ci Please smoke test

I’ll think about that along with the other assertions I promised you I would add.

I’ll think about that along with the other assertions I promised you I would add.

Thanks. It you figure out a good way do it, it would be so cool to just replace the other assertions. No need to add most of them if we can do that.