Support for TensorRT-LLM

Outlines currently support the vLLM inference engine, it would be great if it could also support the tensorRT-LLM inference engine.

Yes waiting for this as welll

TensorRT-LLM supports logits processors so it should be possible to integrate.
How are you hoping to use it?

I'm looking at something similar to outlines.models.tensorrt right now as my usecase is mostly offline batched inference. Could you give me a starting point to how I can build this out, I'm eager to contribute and add such a feature.

@SupreethRao99 glad to hear you're interested in contributing.
I think a good starting point is looking into how TensorRT handles performs generation and handles LogitsProcessors
https://github.com/NVIDIA/TensorRT-LLM/blob/0ab9d17a59c284d2de36889832fe9fc7c8697604/tensorrt_llm/runtime/generation.py#L368-L386
https://github.com/NVIDIA/TensorRT-LLM/blob/0ab9d17a59c284d2de36889832fe9fc7c8697604/tensorrt_llm/runtime/model_runner_cpp.py#L266-L267
Then I'd review how llamacpp is being implemented, it shares similarities with how TensorRT would work https://github.com/outlines-dev/outlines/pull/556/ Specifically llamacpp.py https://github.com/dtiarks/outlines/blob/726ec242fb1695c5a67d489689be13ac84ef472c/outlines/models/llamacpp.py
Please let me know if you have any questions!

Thank you for the resources , I'll definitely get back to you with questions after going through these links.
Thanks!

Related to #655

This can likely be implemented with the Executor API:
https://github.com/NVIDIA/TensorRT-LLM/blob/31ac30e928a2db795799fdcab6be446bfa3a3998/examples/cpp/executor/README.md#L4


dashboard UI indicate issue

Describe the bug
The dashboard UI did not indicate any issues even when the cluster became unstable: when a command to create a materialize view failed, but would not run again because it believed the command was still running.
Error message/log
No response
To Reproduce
No response
Expected behavior
No response
How did you deploy RisingWave?
No response
The version of RisingWave
No response
Additional context
No response

This is supposed to be provided in Cloud UI (console) instead of Kernel Dashboard. The later one doesn’t depend on Prometheus.
BTW, will do https://github.com/risingwavelabs/risingwave/issues/12366 on kernel side.


Add reducers, actions and client functions for user access tokens

Summary
Add reducers, actions and client functions for user access tokens.
Ticket Link
https://mattermost.atlassian.net/browse/PLT-6987
Checklist

[x] Added or updated unit tests (required for all new features)
[x] All new/modified APIs include changes to the drivers

Waiting on server PR to merge, I'll request reviews when it goes in

@enahum changes made


Candidate fills in and submits consent form

Is your feature request related to a problem? Please describe.
As a candidate I need to be able to provide information and consent for character checks.
Describe the solution you'd like
The candidate to be taken through the pages on the prototype for professional body info, HMRC info and declaration of consent. https://jac-apply-prototype.herokuapp.com/current/submit-consent-form/
Consent statement

Personal info

Professional bodies

HMRC

Summary page

Declaration

Submission

Reviewed :)

@HalcyonJAC  - We will need to provide updated wording for these pages - we are in the process of getting this signed off


zfile fix and test

Spurious problems on Windows seem to happen with gzopen(filename) as we
used in ZfileInput::valid_file, but not with the gzdopen(fileno(fd)) as
we used in ZfileInput::open(). So change valid_file to the method that
seems to work. This has been linked to crashes during app shutdown on
Windows (even when no zfiles are encountered, since it can happen with
invalid files when all input readers are tried).
Also add a zfile testsuite entry.
Fixes #1817.

@riviera-kid

I'm still getting the assert. I think that the problem is not in valid_file, but in the ZfileInput::open. BTW the ZfileInput::valid_file is not called when opening the corrupted image - ZfileInput::open is called and return false - error ("Not a valid Zfile")
I think that the code with native zlib file system functions is working (valid_file before the fix)...

Did I do it backwards?
Oh, I see, you're saying that adding an explicit fclose() seems to do the trick?
Let me take another stab at it.

There is a problem with mixing stdio with zlib. I think there is no reason to use stdio fopen/fclose in a zfile reader - using only gz*func for IO could solved the issue.

The reason we go through fopen is because of utf filenames. But I see now that there's a gzopen_w that takes wide char filenames.
Update has been pushed. Can you give a try again to the latest version in my PR?

The latest version is working well! Thanks

Great to hear! I will merge it later today.

Merged, and also backporting to 1.8.


Twist-shift BC updater

This is an implementation of the twist-shift (TS) boundary condition (BC) for 5D gyrokinetics. The original algorithms is described in M. Francisquez, et al. CPC 298 (2024) 109109, but we (@akashukla, @jRoeltgen and I) have implemented a simplified version of it, see issue #384  (in particular the comment on July 31 2024), and the comments at the top of zero/bc_twistshift.c. I reproduce a couple of these comments here:

Unlike the procedures described in M. Francisquez, et al. CPC 298 (2024) 109109, all subcell integrals are now done with variable y limits. This is possible once we realize that figure 4 is not drawn accurately; the blue lines should be separated by Delta y at all points.
This updater only works on 5D distributions. Likely only minor changes are needed to make it work in other dimensions.
99% of the code is written to support a BC, a shift and shear in any direction. Maybe the only thing that needs to change is the permutted range and its use.

There is a unit test, which runs on CPUs and GPUs, and with which I've been able to reproduce figures 6, 9 and 11 from the paper. I've also confirmed the conversation property shown in figure 16a.
The way this works is:

One must first apply regular periodicity (outside of this updater).
Pass the distribution to this updater, and it will twist-shift the distribution that was previously in the ghost cell (in-place).

See issue #384 for possible improvements, which we should only pursue after some simulations have been done.

I would like to see a 5D 'clopen' simulation tested before merging, and I am happy to help with this.

My logic was to PR the updater alone here. The clopen-specific stuff will come in a separate PR (since that will mostly involve changes to the app).
If you guys want clopen to happen here that's fine by me, but it'll make this PR bigger.

That makes sense to wait for clopen. What about Fig. 17 in your 2024 paper? Are you able to reproduce that?

We are not able to reproduce figure 17 because that requires a delta-f
solver we had in g2 but not in g0. It would take a bit of work (probably a
new app) to do that.
On Thu, 1 Aug 2024 at 18:50, tnbernard @.***> wrote:

That makes sense to wait for clopen. What about Fig. 17 in your 2024
paper? Are you able to reproduce that?
—
Reply to this email directly, view it on GitHub
https://github.com/ammarhakim/gkylzero/pull/438#issuecomment-2263518781,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQ6DKAXI3AHMBJ2H6TS5WLZPJROXAVCNFSM6AAAAABLYEI6XSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRTGUYTQNZYGE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Remove hard destination count requirement for NavigationRail

I'm not sure if it's the right place to send this feedback, please let me know if I come to the wrong place.
Currently the NavigationRail material widget requires user provide 2 or more destinations to work. I suggest remove this limitation so NavigationRail can be more generic use cases.
Use case
Image when developing an app similar to Discord, when user doesn't join any Discord server, the navigation rail on the left side doesn't contain any entry except the "Add server" entry. As suggested by the Material guideline, the Add Server button should be put in the container (or the FAB in Material 2). If user joined a server, there will be a server entry / navigation destination available, and more destinations will be there when user adds them.
Other use cases are similar to this one, when we build an app that is designed to let user add entries by themself and the app doesn't provide or only provide one default entry, the NavigationRail should still fit this requirement and developer should be able to use it.
Proposal
Remove the destinations.length assert or change it to  >= 0 in the NavigationRail widget:
https://github.com/flutter/flutter/blob/64b08f3c5658b66d7a19b71bea0aa18bc30a3d01/packages/flutter/lib/src/material/navigation_rail.dart#L108
Additional Information
Navigation Rail in Jetpack Compose doesn't have such limitation.

@BLumia
This looks like similar case with https://github.com/flutter/flutter/issues/58367. You may follow-up in that issue and raise or link the PR you created to the original one. Closing from here.


fix: fastify ERR_HTTP_HEADERS_SENT is reply is not awaited

Hey there,
I was trying to use trpc-openapi with fastify-session, and I was wondering why I have ERR_HTTP_HEADERS_SENT when sending request to trpc-openapi.
I found that @mgcrea/fastify-session tried to save multiple times the session.
fastify reply.send call must be awaited to ensure the request is done.

@jlalmes Is there anything I can do help you ?


feat(tari-dan-cli): initial version

Description
This is the initial version of Tari DAN CLI that includes the following:

Automatic CLI configuration creation/restoration at every command run
Configurable templates repository
Automatically keep up-to-date locally the templates repository (if changed, just checks out and uses that, so multiple template repositories can be in place that can be switched by changing config)
Generating new project (basically empty Cargo workspace project with some extra detail)
Generating new WASM project (if generating into a Cargo project, it updates it's Cargo.toml automatically to include new project as workspace member)

Motivation and Context
The developers creating new templates needs a simple but powerful tool (a CLI in our case) to easily generate new projects, wasm projects, test them and deploy them with simple commands.
How Has This Been Tested?
Going through all CLI commands, create project, create multiple wasm projects under that etc..
NOTE: until https://github.com/tari-project/wasm-template/pull/9 gets merged, you should (after running any command there will be the config for CLI, on MAC: /Users/<username>/Library/Application Support/tari_dan_cli/tari.config.toml) update config to use a specific repo and branch:
[project_template_repository]
url = "https://github.com/ksrichard/wasm-template"
branch = "feature/tari-dan-cli-updates"
folder = "project_templates"

[wasm_template_repository]
url = "https://github.com/ksrichard/wasm-template"
branch = "feature/tari-dan-cli-updates"
folder = "wasm_templates"

What process can a PR reviewer use to test or verify this change?
Breaking Changes

[x] None
[ ] Requires data directory to be deleted
[ ] Other - Please specify

LGTM - looks like something unrelated to this PR has gone wrong with CI and hickory client used in L1. Will see if I can fix it first thing tomorrow, if all is good will review and merge.

I think we should call this tari_cli (binary will be tari) DAN is not a term we want to use in future
okay, makes sense, let me update that

Closing, because the development has been moved to https://github.com/ksrichard/tari-cli


Bulloak fails when word ‘when’ is used deep in the ‘It’ leaf

Foo
└── It reverts when X.

Running bulloak scaffold Foo.tree results in an error: bulloak error: invalid identifier: ..
I understand it’s because bulloak sees the word ‘when’ in the leaf name and thinks it’s a condition instead of an action.
While I understand I could write:
Foo
└── When X
    └── It reverts.

I don’t like that because then the action statement becomes a comment which is not checked with bulloak check if removed.
Besides, there are legitimate usecases for the word ‘when’ used to describe the action.
Expected behavior: The word ‘when’ has a special meaning only at the beginning of a leaf name.

I understand it’s because bulloak sees the word ‘when’ in the leaf name and thinks it’s a condition instead of an action.

Ah, not exactly. It's because top-level actions are scaffolded as tests, and thus must be valid identifiers.

Hmmm… sort of.
But then why
Foo
└── It reverts when X.

fails while
Foo
└── It reverts X.

works?

Ooh, that's because of a parsing error. Great catch! it enters identifier mode in the tokenizer after the when token. I'll make some time to fix it.

Okay, finally had time to fix this! You can check it [here].(https://github.com/alexfertel/bulloak/commit/488fb6351b49da2c71ce09cd1a06b572988d03c9)
Will publish a new version in a few.


Add Expo and React Native examples to CI

This PR:

[X] Adds Expo and React Native examples to CI workflow.

Can we add any caching to this to speed things up? This slows the test suite by nearly 5x and this library isn't React-specific. This seems like overkill to me.

@timdorr Sure I can figure out some sort of caching to speed this up.

Updated the RN examples upstream to be much much faster, but also added a cache. Thanks for getting this together!

@timdorr

Updated the RN examples upstream to be much much faster, but also added a cache. Thanks for getting this together!

That's awesome, thank you Tim! This makes me wonder, would it be okay then to re-open https://github.com/reduxjs/redux-thunk/pull/365?

Unfortunately, I can't reopen this PR. But feel free to submit a new one and we'll get it merged in.

@timdorr Thank you!!!

i reopened it 👍🏻

@EskiMojo14 Thank you Ben!


Display certificate state (EXPOSUREAPP-8243)

Adds the "Expiring soon/Expired/Invalid" states to the certificates in the certificates tab, person details screen and certificate details screen. Can be seen under the QR code of each screen and also on the individual certificate cards on person details screen.
Scan recovery/test/vaccination certificates. Travel to the future and check displayed status.

The following are still missing for Invalid certificate:
1- Overlay

2- Icon

The following are still missing for Invalid certificate:
1- Overlay

2- Icon

They're being added in #3775


Unexpected behavior in ChatSession.ChatAsync methods

I've been trying to create a chatbot with the ChatSession class and IHistoryTransform implementation to use the correct template for the LLM I am testing LLamaSharp with. I ran into unexpected behavior and could not get good responses from the model.
The only way I got a decent response was when I sent the whole history as a prompt with the correct formatting. Saving and loading the session would lead to ever growing context and significant slowdown.
I noticed that the use of the IHistoryTransform interface in the ChatSession class is leading to these unexpected results. When the ChatSession class was refactored to use the IHistoryTransform interface, the behavior changed dramatically by parsing the text in the prompt argument and adding the prompt (which may be the whole history) into the history of the session.
It's a little bit hard to untangle. I forked the repo and changed the behavior to be a bit more straightforward. With the overload method that accepts a string prompt managing the internal history for the user and the overload that accepts a ChatHistory argument allowing the user to manage the history themselves.
See here: https://github.com/SciSharp/LLamaSharp/compare/master...philippjbauer:LLamaSharp:master
I see many more improvements here, especially in how the IHistoryTransform interface could be changed to accommodate better templating support. Making use of public properties for role tokens and end tokens so that these tokens can be replaced internally before adding the generated message to the history instead of relying on the inferenceParam.AntiPrompt property.
If this is missing the mark completely let me know, but I couldn't figure out how this should be used after reading the documentation and looking at the code.
Edit: A little bit lengthy but here is my IHistoryTransform implementation:
public class HistoryTransform : IHistoryTransform
{
	private string _nl = Environment.NewLine;
	private string _roleTemplate = "<|{0}|>";
	private string _endToken = "</s>";
	private string _regexPattern = @"(?:<\|([a-z]+)\|>\n)([^<]*)(?:</s>)";

	public string RoleTemplate => _roleTemplate;
	public string EndToken => _endToken;

	public HistoryTransform(
		string? roleTemplate = null,
		string? endToken = null,
		string? regexPattern = null)
	{
		_roleTemplate = roleTemplate ?? _roleTemplate;
		_endToken = endToken ?? _endToken;
		_regexPattern = regexPattern ?? _regexPattern;
	}

	public string HistoryToText(ChatHistory history)
	{
		string text = "";

		foreach (Message message in history.Messages)
		{
			text += FormatMessage(message);
		}

		AuthorRole nextRole = history.Messages.Last().AuthorRole == AuthorRole.User
			? AuthorRole.Assistant
			: AuthorRole.User;

		text += $"{FormatRole(nextRole)}{_nl}";

		return text;
	}

	public ChatHistory TextToHistory(AuthorRole role, string text)
	{
		ChatHistory history = new();

		Match match = Regex.Match(text, _regexPattern);
		string message = match.Groups[2].Value.Trim();

		history.AddMessage(role, message);

		return history;
	}

	// Not part of interface, use in program to load saved history
	public ChatHistory FullTextToHistory(string text)
	{
		ChatHistory history = new();

		MatchCollection matches = Regex.Matches(text, _regexPattern);

		foreach (Match match in matches.Cast<Match>())
		{
			AuthorRole role = match.Groups[1].Value switch
			{
				"system" => AuthorRole.System,
				"user" => AuthorRole.User,
				"assistant" => AuthorRole.Assistant,
				_ => AuthorRole.System
			};

			string message = match.Groups[2].Value.Trim();

			history.AddMessage(role, message);
		}

		return history;
	}

	public string FormatRole(AuthorRole role)
	{
		return string.Format(_roleTemplate, role.ToString().ToLower());
	}

	public string FormatMessage(Message message)
	{
		return $"{FormatRole(message.AuthorRole)}{_nl}{message.Content.Replace(_endToken, "")}{_endToken}{_nl}";
	}
}

The FullTextToHistory method is a crutch to load the history I save alongside the session data. I think this should be handled by the ChatSession class and saved as a JSON formatted file or similar for easy portability when necessary.

This kind of confusion with ChatSession/History/various executors is actually exactly what got me started contributing to LLamaSharp! If you're interested in making any PRs to improve to current behaviour (even if it requires breaking changes) I'd be very interested to happy to review them.
In the longer term I think we're going to need to do a complete redesign of the higher level features at some point. To me it's very confusing to have 3 types of executors (two of which seem very similar) and the ChatSession built on top of that (which seems fairly similar again).
To kick off that discussion: what do you think would be the "ideal" high API for LLamaSharp (completely ignoring all backwards compatibility problems for now)?

Good, I'm not alone in my confusion :)
I do have some ideas regarding the ChatSession design. It's good to know that you are not averse to breaking changes, that will make it a lot easier to bring this part of LLamaSharp forward.
Long-term it would be great to have the following:

Context Overflow Policy (see LM Studio)

Stop at limit
Keep system prompt and first user message, truncate middle
Maintain rolling window and truncate past (I'd say keep system prompt as well though)

Robust templating configuration

Reading template from GGUF (I think they may be saved in there nowadays?)

Saving Session with history restore

History stored as portable JSON structure
Saving inference parameters as portable structure

ChatAsync overloads that allow for the system to handle history and Context Overflow Policy and for the user to handle history (exposing the lower level method so to say)

Just of the top of my head. There's probably more that can be added but I'd have to fiddle around and spend some time thinking about what makes sense to expose to the user.
Short-term, I think the change I linked in my fork would greatly improve the utility of the ChatSession (make it usable at all really). Maybe this could be merged soon?

Short-term, I think the change I linked in my fork would greatly improve the utility of the ChatSession (make it usable at all really). Maybe this could be merged soon?

If you want to open up a PR with this change I'll be happy to review it :)

Long-term it would be great to have the following...

Context Overflow Policy (see LM Studio)

This one is tricky, because there's are several different "layers" you can do it at. You might want to rebuild a new chunk of text (e.g. keep prompt, summarise the rest). Or you might want to go a level "lower" and just rearrange tokens (e.g. keep prompt tokens and most recent output tokens). Or you might want to do some magic with shifting around the KV cache.

Robust templating configuration

Looks like there's some discussion of embedding jinja templates into gguf (here).

Just of the top of my head. There's probably more that can be added but I'd have to fiddle around and spend some time thinking about what makes sense to expose to the user.

if you come up with anything interesting I opened up a discussion the other day, around thoughts for an entire new executor in LLamaSharp. I'd love to hear your thoughts.


v4.0.0 still gives compiling error: module did not self-register

I updated gcc and g++ versions to 4.8 but It still gives error pointing bindings.js
I followed https://github.com/fivdi/onoff/wiki/Node.js-v4-and-native-addons
Any advice?
Thanks in advance!

What is the exact command being used to install onoff and what is the complete output of that command?
Also, what is the output of the following commands?
node -v
uname -a
gcc --version

npm install onoff
V4.0.0
Linux raspberrypi 4.1.18-v7+ #846 SMP Thu Feb 25 14:22:53 GMT 2016 armv7l GNU/Linux
gcc (Raspbian 4.8.4-1) 4.8.4
onoff was just working fine with 0.10.30. I updated node to 4.0.0 and If I run the file where onoff included is:
home/pi/Desktop/ledWithNode/anotherTest/raspberry-nodejs-simple-led/node_modules/onoff/node_modules/epoll/node_modules/bindings/bindings.js:83
throw e
^
Error: Module did not self-register.
at Error (native)
at Module.load (module.js:355:32)
at Function.Module._load (module.js:310:12)
at Module.require (module.js:365:17)
at require (module.js:384:17)
at bindings (/home/pi/Desktop/ledWithNode/anotherTest/raspberry-nodejs-simple-led/node_modules/onoff/node_modules/epoll/node_modules/bindings/bindings.js:76:44)
at Object. (/home/pi/Desktop/ledWithNode/anotherTest/raspberry-nodejs-simple-led/node_modules/onoff/node_modules/epoll/epoll.js:1:99)
at Module._compile (module.js:434:26)
at Object.Module._extensions..js (module.js:452:10)
at Module.load (module.js:355:32)

Did you uninstall and reinstall onoff after updating Node.js from v0.10.30 to v4.0.0? It's essential to do this as onoff depends on a compiled native module called epoll. A native module compiled for Node.js v0.10.30 will not function with Node.js v4.0.0. It needs to be compiled for the version of Node.js being used.

That makes sense and it works! Thank you!

Your welcome. Have fun with onoff.

Additional information that may help others reading this issue: The "Module did not self-register" error was a runtime error, not a compile time error.


Problem when passing in large string as input

I am trying to automate a task where I have many options. I am passing these options in as a string:
./tagui flow.tagui  '{"topSelector":["Burrito"],"fillings":["Chicken","Barbacoa"],"rices":["White Rice","Brown Rice"]}'

however when I do
echo p1
order = JSON.parse(p1)

I get
{"topSelector":["Burrito"],"fillings":["Chicken","Barbacoa"],"rices":["White
SyntaxError: JSON Parse error: Unterminated string

shorter strings  appear to work fine with JSON.parse()

Closing, as I realized that this is cutting off at the space, I will just escape the spaces or replace them'

Hi @HenriPro cool use scenario :) Adding on, another way that can be used to do this is save the large string or block of text into a file, then use the load step to read from the file into a variable.

Thanks awesome work!


Remove keyboard attachable plugin bc it is causing exceptions

@zlshames now that we've narrowed down the keyboard issue I think this is okay to merge now as well.


Setup logic to publish the Godot Android library to MavenCentral

This PR follows the instructions from https://getstream.io/blog/publishing-libraries-to-mavencentral-2021/ to setup the logic to publish the Godot Engine Android Library to MavenCentral.
MavenCentral is a repository provided by the Maven community which contains a large number of commonly used libraries.
It's an important tool for java and Android developers as it simplifies the resolution and fetching process for a project's dependencies by build tools like Gradle.
The updated logic will enable to push (automatically) the generated Godot Android Library to MavenCentral so that users can access it by adding a single line to their gradle build file:
dependencies {
    ...
    implementation 'io.github.m4gr3d:godot:<versions>'
    ...
}

Publishing To MavenCentral

This requires a sonatype account and a GPG key to sign the generated library.
Provide values for the variables defined in publish-root.gradle. This can either be done via environment variables, or through the local.properties file (that file is not and should not be added to git).
Generate the Godot Android Library via the usual build process.
Run the following commands to push the generated library to MavenCentral:

cd platform/android/java
./gradlew :lib:publishReleasePublicationToSonatypeRepository

Follow these steps to review the uploaded binary and release it publicly on MavenCentral.

Notes & TODOs

[ ] I'm using my own sonatype credentials at the moment to publish the library. Eventually, we would want to switch to Godot's own set of credentials.
[ ] Automatic publishing is blocked by above todo. Once we have a set of credentials for Godot, we can automate the publishing of the Godot Android Library by leveraging github workflows as described here.
[ ] Update internal logic (e.g: Android template logic) to leverage the published library via gradle
[ ] Update Android related documentation accordingly.

Part of https://github.com/godotengine/godot-docs/issues/4018
Implements https://github.com/godotengine/godot-proposals/issues/4230

Cherry-picked for 3.5.

Cherry-picked for 3.4.4.

@m4gr3d Hey, thanks for providing this! Is the "export" mono-compatible, meaning we can use it for games using mono scripts?

io.github.m4gr3d:godot:

The dependency under io.github.m4gr3d:godot:<versions> is not mono-compatible. Though this is only because the Godot Android library I generate and upload doesn't have mono-enabled.
The sonatype upload script is agnostic, so if a Godot Android library with mono enabled is generated instead, it can be uploaded similarly.
Once we have an official Godot sonatype account (io.github.m4gr3d is my personal account), it makes sense to include the mono library as part of the offered dependencies.
cc @akien-mga


prettier: printWidth=100

説明 / Description
Change prettier printWidth setting: 80(default) => 100
チェックリスト / Checklist

MUST

[x] npm test passed
[x] npm run lint was applied without warnings
[x] changes of /docs/webapp not included (except release branch)
[x] console.log not included (except script file)

RECOMMENDED (it depends on what you change)

[ ] unit test added/updated
[ ] i18n

Codecov Report
Patch coverage: 54.58% and project coverage change: +0.01% :tada:

Comparison is base (40b6ae5) 69.94% compared to head (e063488) 69.96%.
Report is 2 commits behind head on main.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #595      +/-   ##
==========================================
+ Coverage   69.94%   69.96%   +0.01%
==========================================
  Files          83       83
  Lines       17393    16357    -1036
  Branches     2320     2277      -43
==========================================
- Hits        12166    11444     -722
+ Misses       2931     2667     -264
+ Partials     2296     2246      -50

Files Changed
Coverage Δ

src/background/encrypt.ts
26.31% <0.00%> (+3.58%)
:arrow_up:

src/background/usi/process.ts
6.66% <0.00%> (+0.31%)
:arrow_up:

src/common/helpers/encode.ts
78.04% <0.00%> (-1.50%)
:arrow_down:

src/common/helpers/metadata.ts
64.70% <0.00%> (-6.73%)
:arrow_down:

src/common/helpers/string.ts
44.00% <0.00%> (-3.17%)
:arrow_down:

src/common/settings/analysis.ts
92.30% <0.00%> (-0.20%)
:arrow_down:

src/common/settings/csa.ts
78.27% <0.00%> (-0.17%)
:arrow_down:

src/common/settings/game.ts
64.35% <0.00%> (+5.79%)
:arrow_up:

src/common/settings/mate.ts
43.75% <0.00%> (+4.86%)
:arrow_up:

src/common/settings/research.ts
89.74% <0.00%> (-0.96%)
:arrow_down:

... and 41 more

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Switch to thin, versioned ES clients

What changed?
Switching to ES clients

Why?
It is easier to add different version of ES client as business logic now lives in a single place. To add new version, you need to implement code related to client-server communication only.

How did you test it?
Unit tests

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 0187c3f8-7c03-49d7-bc28-530d0a8df9c4

79 of 135 (58.52%)  changed or added relevant lines in 3 files are covered.
101 unchanged lines in 12 files lost coverage.
Overall coverage increased (+0.09%) to 57.269%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

common/elasticsearch/interfaces.go
10
13
76.92%

common/elasticsearch/client.go
44
97
45.36%

Files with Coverage Reduction
New Missed Lines
%

common/task/fifoTaskScheduler.go
2
87.63%

common/util.go
2
52.61%

service/history/task/transfer_standby_task_executor.go
2
87.24%

common/persistence/nosql/nosqlplugin/cassandra/workflow.go
3
59.55%

common/persistence/serialization/parser.go
4
65.41%

common/persistence/serialization/thrift_decoder.go
4
61.22%

service/history/task/fetcher.go
4
91.24%

common/persistence/nosql/nosqlplugin/cassandra/workflowUtils.go
8
77.1%

common/persistence/sql/workflowStateMaps.go
11
85.56%

common/persistence/nosql/nosqlplugin/cassandra/workflowParsingUtils.go
14
83.23%

Totals

Change from base Build 0187c381-2820-43d9-9265-cc64b125d6e7:
0.09%

Covered Lines:
85809

Relevant Lines:
149834

💛  - Coveralls


training script

Can you provide easy-follow instructions on training of the model?

Hi @hq0709,
I have provided some basic training scripts. You can find them at https://github.com/RunpeiDong/DreamLLM/tree/master/scripts/train/dreamllm

Hi @hq0709,
Are the training scripts helpful?


Merge LdpHandler... integration tests with Server... integration tests

And at the same time we could also get rid of the node-mocks-http library which has been causing some problems.

Just want to comment that they test a different range; the handler only tests the LDP handlers, whereas Server tests a much large stack.
As an alternative to merging (if they are indeed testing different things!), we could just slap a very simple ExpressHttpServerFactory around the LdpHandler, without middleware etc. Then we achieve the same goal of dropping that lib.

The tests in both files are very similar currently: ServerWithAuth.test.ts has 3 tests which all involve some tests about writing/reading resources, so all very related to the LdpHandler.
On the other hand I don't see why we wouldn't do our LdpHandler integration tests with a full server setup as close to the default config as possible, instead of pulling the LdpHandler out of that config.

Also discovered we have LdpHandlerWithAuth, LdpHandlerWithoutAuth, and LdpHandlerOperations tests, so there definitely is some duplication there.


keep-presence: init at 1.0.5

This is my first contribution to nixpkgs so please let me know if I've missed anything!
Description of changes
This adds a new package keep-presence from https://github.com/carrot69/keep-presence.
keep-presence is a simple python script to move your mouse and keep your computer active. Can be used to prevent sleep or to keep yourself 'active' on the current chat app of choice.
I'll be opening a PR to the repo above to see if the maintainer is interested in integrating the changes. If so, I will update this package at that time to use the primary repo instead of my fork. The original author has merged the PR so I've updated this package.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@mwdomino are you still interested in getting this merged?

Tentatively closing this since the requested changes haven't been made and Janik's query has gone unanswered for about 2 months.
@mwdomino Please don't let this discourage you if you want to get this merged. I'm more than happy to reopen it if you are :)


feat: add old 3d model

What does this PR do?
Replaces current 3d model with the old working version.

temporarily closes #231


[Forge 1.20.1] ResPackOpts functionality doesn't work using Connector.

Mod Name
ResPackOpts
Mod Homepage
https://modrinth.com/mod/respackopts
Describe the bug
Mod config doesn't load properly, functionality doesn't work.
Steps to reproduce

Load mod with Connector and required dependency LibJF (personally using the resource pack "RedstoneTweaks").
Open up the mod's config while the resource pack is enabled.
Try to change settings for textures and accept.
Resource pack should update, but textures won't change. Also, the actual settings themselves should appear debug/dev with no proper description.
...

Logs
https://gist.github.com/PlayfulPiano/b099ea7e07cc45e1bf5eb373a4665b1e
Additional context
debug - Copy.zip
debug file was too large to copy
Video comparison of what it should look like: https://www.youtube.com/watch?v=iJObRPuR6Bs

oh my bad, didn't realize it was already reported


Screen capture low performance

when using
wf-recorder -f  x.mp4
the fps drops very low.
Found in https://github.com/hyprwm/Hyprland/issues/2501#issuecomment-1589548427
https://github.com/hyprwm/Hyprland/issues/2501#issuecomment-1589915169
Maybe Screencopy dmabuf could be worth a look
https://github.com/swaywm/wlroots/pull/2133/files

Hyprland already supports dmabuf since a while back. It might be a dt issue

patch.txt
try this patch

https://github.com/hyprwm/Hyprland/assets/5576250/688e0142-531f-4d2e-9718-e56164df2fdc
Unfortunately, I still have stuttering

I have the same issue with wf-recorder--I wrote an alternative that works much better for me. Only works on Intel and AMD hardware: https://github.com/russelltg/wl-screenrec

closing, not a hl issue


Upgrade the project meta data to support new catalog commands

Add metadata to the repository that matches the example laid out here.

[x] Add a config.json file to a catalog directory in the repo root, matching this format.
[x] Move the /screenshots directory into catalog:

[x] Update the screenshot file names (to match this format)
[x] Update the screenshot links in the README.md.

[x] Add the new header image to the top of README.md.
[x] Add a documentation.md file to a catalog directory in the repo root, matching this format.

[x] Update the relative paths in documentation.md to be absolute paths

[x] (Optional) If there's additional documentation referenced in the repo, add a additionalInfo.md file to a catalog directory in the repo root, matching this format.
[x] Ensure the README.md commands section matches Justin's snippet.
[x] Run npm start and let the CLI move the nr1.json

[x] Edit that file to make sure the name and description make sense.

Unsure about my change log, so I'm waiting to push this one further:
https://github.com/newrelic/nr1-top/pull/26/files#diff-a1196f05202e8e494afd808c884a81f4R15-R22

:tada: This issue has been resolved in version 0.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[SVCS-479] Raise exception for copy/move replace folder that orphans itself.

#Purpose:
SVCS-479
Replace folder from inside folder deletes both.
#Changes:
Updated waterbutler/core/provider.py
-- Add def replace_will_orphan
-- Update def move calls replace_will_orphan
-- Update def copy calls replace_will_orphan
Update waterbutler/core/exceptions.py
-- Add class OrphanSelfError
Update tests/providers/osfstorage/test_provider.py
-- Add test test_intra_copy_folder_orphan
#Side effects
Folder move/copies that use conflict=replace may now be more costly

See previous PR for discussion https://github.com/CenterForOpenScience/waterbutler/pull/274

Coverage decreased (-0.003%) to 91.981% when pulling e012a516771d097259946c596317ffc9b77eeff1 on birdbrained:ft/move-orphan-delete into 5c637a2affa8dfb43159ff80cf430c9ac5d369f8 on CenterForOpenScience:develop.

@cslzchen Can you note here what specifically blocks this PR that is resolved by #341?


[BUG] Serialization emits invalid file with missing sections

I have a large application where I dump about 30 MB worth of smaller objects into a file. All objects are using serde auto-derive [derive(Serialize, Deserialize)].
I am using serde_yaml = "0.9" and the code which writes my file looks like this:
let mut file = std::fs::OpenOptions::new()
  .write(true)
  .create(true)
  .open(filename)
  .context(filename.to_owned())?;
let mut out = BufWriter::new(file);
serde_yaml::to_writer(&mut out, &my_stuff).context("serialization to YAML")?;
out.flush()?;

(Note: I tried and it also happens without the BufWriter.)
my_stuff above is a large map with about 20000 objects.
Unfortunately this emits an invalid YAML file where there are obviously some pieces missing. Here is an excerpt of the file which was written:
    SelectionMarker:
      color:
        r: 1.0
        g: 1.0
        b: 1.0
      is_visible: true
      is_selected: false
:
      food: 30.0
      food_current: 30.0
      water: 20.0

There is a single line with a stray : and there is some data missing at that location.
So far I can say: it does seem to happen exactly once in the file and it happens towards the end of the file (this was line 1727330 of 1743267).

"Stray :" is not necessarily invalid YAML. You are just showing an excerpt of the file by the following is valid YAML syntax:
?
    SelectionMarker:
      color:
        r: 1.0
        g: 1.0
        b: 1.0
      is_visible: true
      is_selected: false
:
      food: 30.0
      food_current: 30.0
      water: 20.0

Since what you showed is valid YAML, this issue is not going to be actionable unless you can provide a minimal repro of incorrect behavior.


Help: Frequent GPU OOM errors during model loading on custom MLServer Runtime

Discussed in https://github.com/kserve/kserve/discussions/2973

Originally posted by ericlu88 June  7, 2023
Hi folks,
We are running a custom built ServingRuntime which extends from Seldon's Python MLServer. We are running LLMs like StableDiffusions and Alpaca on GPUs (A100, T4, etc). The normal inferences works fine, but what we encountered is frequent GPU OOM errors when new models are being loaded. It seems ModelMesh doesn't have visibility into the GPU side metric, but instead replies on CPU metrics (cpu load, mem) which isn't relevant in our case. We observed ModelMesh orchestrates models to already FULL pod while other empty pods are waiting in idle.
Our questions:

Is this expected behavior? Based on this ModelMesh issue, it does seems ModelMesh should be able to handle it.
If this isn't expected, how can I configure my custom runtime to honor the GPU metric?
If this is indeed expected (i.e. not supported currently), can you share any pointers on how to extend this functionality? We are open to contribute as it seems a very commonly needed feature.

Environment (please complete the following information):

OS: Ubuntu 18.04
Version: v0.10.0

Additional context
Our custom MLServer ServingRuntime spec:
apiVersion: serving.kserve.io/v1alpha1
kind: ServingRuntime
metadata:
  name: mega-mlserver-runtime
  namespace: modelmesh-serving
spec:
  supportedModelFormats:
    - name: mega-mlserver
      version: "1"
      autoSelect: true
  multiModel: true
  grpcDataEndpoint: port:8001
  grpcEndpoint: port:8085
  replicas: 1
  containers:
    - name: mlserver
      image: <IMAGE>
      env:
        - name: MLSERVER_MODELS_DIR
          value: "/models/_mlserver_models/"
        - name: MLSERVER_GRPC_PORT
          value: "8001"
        # default value for HTTP port is 8080 which conflicts with MMesh's
        # Litelinks port
        - name: MLSERVER_HTTP_PORT
          value: "8002"
        - name: MLSERVER_LOAD_MODELS_AT_STARTUP
          value: "false"
        # Set a dummy model name via environment so that MLServer doesn't
        # error on a RepositoryIndex call when no models exist
        - name: MLSERVER_MODEL_NAME
          value: dummy-model
        # Set server addr to localhost to ensure MLServer only listen inside the pod
        - name: MLSERVER_HOST
          value: "<IP_ADDRESS>"
        # Increase gRPC max message size to support larger payloads
        # Unlimited because it will be restricted at the model mesh layer
        - name: MLSERVER_GRPC_MAX_MESSAGE_LENGTH
          value: "-1"
        - name: MLSERVER_DEBUG
          value: "true"
        - name: MLSERVER_MODEL_PARALLEL_WORKERS
          value: "4"
      resources:
        requests:
          cpu: 10
          memory: 40Gi
          nvidia.com/gpu: 1
        limits:
          cpu: "12"
          memory: 42Gi
          nvidia.com/gpu: 1
  builtInAdapter:
    serverType: mlserver
    runtimeManagementPort: 8001
    memBufferBytes: 134217728
    modelLoadingTimeoutMillis: 900000
```</div>

Hello guys,
I experienced similar issues as above. Basically I observed that ModelMesh tried to load models onto a machine that caused GPU OOMs, and then resulted failed copies.
Logs showed something like this:
torch.cuda.OutOfMemoryError: CUDA out of memory
Is there a way to control whether or not to load model onto a specific instance? For example:

ModelMesh can observe what is the remaining GPU memory for a certain instance, or in another way, we explicitly specify total available GPU memory for instances.
For models, we can configure memory request, ModelMesh can decide whether to load such a model onto an instance, by checking the instance's remaining memory vs model's required memory.

Thanks

Closing this issue out thanks to @GolanLevy's suggestion on this thread: https://github.com/kserve/modelmesh/issues/82#issuecomment-1582028690. It is fairly static but does solve our most immediate need.

/close


Please make it possible to output the Details.

Request
I am using nowplaying2 from this repository: https://github.com/shirafukayayoi/Node_fb2kRichPresence with the Discord Rich Presence plugin. However, I am unable to output the album art URL (e.g., an Imgur URL) in the Details field, which prevents me from implementing the album art in Rich Presence. If it becomes possible to output the song details, it would solve this issue. Is it possible to implement this feature?

I believe this may be a limitation of foobar2000. It will require supporting WXXX id3v2.4 frames, and I have been unable to find any documentation regarding this. We would like to see this as well. It is also used in live and file HLS streams.

I see. I'll also look into various methods, but I'd appreciate it if you could let me know if you find anything

Thanks for contacting me.
I will certainly let you know if I find something.
I also plan to post on foobar about this.
Maybe I missed something…
Please keep in touch.
/greg.
From: shirafukayayoi @.>
Sent: Wednesday, 12 June, 2024 14:07
To: foxx1337/foo_nowplaying2 @.>
Cc: gogonowski @.>; Comment @.>
Subject: Re: [foxx1337/foo_nowplaying2] Please make it possible to output the Details. (Issue #8)
I see. I'll also look into various methods, but I'd appreciate it if you could let me know if you find anything
—
Reply to this email directly, view it on GitHub https://github.com/foxx1337/foo_nowplaying2/issues/8#issuecomment-2163901889 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AC4LC56V3VCO7OR54EKCZ5DZHCZ73AVCNFSM6AAAAABJG6DA4KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRTHEYDCOBYHE .
You are receiving this because you commented.  https://github.com/notifications/beacon/AC4LC52TW6KAM255TVI4QMTZHCZ73A5CNFSM6AAAAABJG6DA4KWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUA7KC4C.gif Message ID: @.*** @.***> >


Re-enable swift-numerics

This project should be fixed by https://github.com/apple/swift/pull/60008 and https://github.com/apple/swift-numerics/pull/233.
Reverts apple/swift-source-compat-suite#678

@swift-ci test

/Users/ec2-user/jenkins/workspace/swift-PR-source-compat-suite-test-macOS/swift-source-compat-suite/project_cache/swift-numerics/Sources/_TestSupport/Error.swift:21:13: error: cannot use conformance of 'Float' to 'Real' here; 'RealModule' was not imported by this file
public func componentwiseError(_ tst: Complex<Float>, _ ref: Complex<Double>) -> Double {

Hi @xymus 👋 , I think you'll need to bump up the revision of the swift-numerics project in the projects.json in order to contain your fix from https://github.com/apple/swift-numerics/pull/233. That should get the build passing and allow us to remove the xfail

Thanks for the reminder!
@swift-ci test


Add ABI for Cobo Safe Module

Transaction from Cobo Safe Module cannot be decoded. Please decode the methods from Cobo Safe Module listed below, thank you.
abi.json.zip
.

Hi, if you verify the module on Sourcify/etherscan it will appear. Also address of the contract should be provided


Unit tests to interface_filter

Signed-off-by: Alona Paz<EMAIL_ADDRESS>

we have each lane twice
pull_request and push ....
need to fix maybe the actions

PR desc:
'status-manager: Add interface filter unit tests"

https://stackoverflow.com/questions/70767090/github-actions-on-push-and-on-pull-request-difference#:~:text=In general%2C push will trigger,from a fork for example.
There were double maybe because you used some other way to create a PR ?


Feature/mango reimburse

👍

IX was run in production, PR no longer needed, code has now been overwritten by subsequent updates and is now no longer on main-net chain, great work king, closing now


Add the ability to make link cross multiple points across its route from source node to target node

closes #373 (Even though additional functionality to the behavior mentioned in the issue may be added in the future).
This PR adds the ability to describe the Link route from its source node to target node more precise way.
The behavior can be seen in this screenshot:

@danielcaldas I think this PR is ready to be merged.


#discuss

So i apply some middlewares on the main app object.  If i create a router in another file, import it inside the main app and use app.use(router), where router is  the name of the secondary router object, every middleware applied on the main app object will be used for that router too, right?

After having fun with a simple middleware, the conclusion: Yes! It works!


Feature Request - Enhanced Scroll Bar Diff Tracking Optional Settings

I have looked to see if there is a feature like this but I could not find anything.
So currently the scroll bar is able to track diff code until committed. So all your changes for one commit will show up on the scroll bar markers, but the moment you commit they are gone. What if there were settings that allowed for the enhanced scroll bar to always compare to master/(any other branch) or from branch point? It could track all committed changes up to that specified as a different color, and then all uncommitted changes as another color.

/duplicate #36893


Pr 2580 consular fees kazakhstan kyrgyzstan

This is a version of PR #2580 that's been rebased on master. I've created it to demonstrate how I think that PR should look given that master has changed since it was first created.
Differences between this PR and PR #2580:

This PR only has two commits versus the three commits in PR #2580. The regeneration of marriage-abroad-files.yml was (confusingly) split between the first and third commit in PR #2580. It's regenerated in the first commit alone in this branch which meant that the third commit was empty and has disappeared.

This PR only contains changes to the artefacts for residents of the ceremony_country, versus changes to the artefacts for residents of the UK and ceremony_country in PR #2580. This is because PR #2578 has been merged to master since #2580 was opened. PR #2578 ensures that we're only displaying the link to consular fees if there are chargeable services available for the combination of options leading to the outcome. For Kazakhstan and Kyrgyzstan the chargeable services are only available for ceremonies where the user is resident in the ceremony_country (see marriage_abroad_services.yml).

I'm going to close this. It doesn't need to be open to serve as a demonstration.

Hi @chrisroos, I am curious to know if you found any particular reason for not generating the artefacts as a part of the first commit.

@chrisroos thanks for reviewing this PR #2580  and creating this  PR to further clarify things.. I have cheery picked and included it here

Hi @chrisroos, I am curious to know if you found any particular reason for not generating the artefacts as a part of the first commit.

@pmanrubia: This is how the commits were structured in PR #2580. I simply rebased on master and edited the commits so that they continued to work after some of the more recent changes in master.


Add ability to find targets for diseases

Use the disesome GMT file as an alternative input to the app where the user selects a disease (instead of loading the example data or their own data) and then gets studies that show reverse or similar pattern as the input disease... If you need the GDS/GSE/GSM IDs for the diseasome... we can provide these.

I've been thinking about it for a while and I am still not sure how to handle it.
Most of the required logic is already in place. We still have to decide if it is better to cache characteristic direction or recompute it on demand but it is not that important.
What is bugging me here is how to incorporate it in the current workflow:

Switch between upload and disesome data inside upload dataset panel and go through the chdir, paea as previously
Break existing workflow into two parts - ChDir analysis and PAEA. Inside PAEA give user option to choose chdir output (if available) or disease signature.
some other solution?

I like the #1 option the best... I think it would be also easiest to implement...

I deployed POC solution yesterday.
It is based on nosology_accession_list data processed using following pipeline:
https://github.com/zero323/shrouded-chamber
It discards all series with more than one platform but we can change it if you want.
Since output is far to big to be pushed to shinyapps.io I keep it on my own server and fetch when required.

Awesome! Looks great!

I will close this one, but if you have any addtional requests feel free to reopen.


fix media preview_url nullable type

1. Description

Some media type may have nullable preview_url property, which didn't reflect in documentation.
You change check it with provided media id
https://mstdn.social/api/v1/media/110023777570725915

1.1. Checklist

[x] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[x] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[x] I have updated/added tests for ALL new/updated/fixed functionality.
[ ] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[ ] I have updated/added relevant examples in examples.

1.2. Breaking Change

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

1.3. Related Issues

Hi @Kraigo ,
Thanks for your contribution!:)


cert-manager 1.0.4 fails to install on rke1 hardened cluster configured cis-1.5 compliant

What kind of request is this:
bug
Steps to reproduce:

Create an RKE cluster with nodes/kubernetes cis-1.5 compliant (See doc)
Install cert-manager (step 4 in the docs)

Result:
Error creating pods
  Type     Reason        Age                   From                   Message
  ----     ------        ----                  ----                   -------
  Warning  FailedCreate  50s (x17 over 6m18s)  replicaset-controller  Error creating: pods "cert-manager-6d87886d5c-" is forbidden: PodSecurityPolicy: unable to admit pod: []

Other details that may be helpful:
Related: https://github.com/rancher/rancher/issues/29696
Environment information

Rancher version: NA
Installation option: 3 node RKE1

On 2.5.8 using cluster.yml for RKE local cluster from here - https://rancher.com/docs/rancher/v2.x/en/security/rancher-2.5/1.6-hardening-2.5/, no error is seen. HA setup of Rancher is deployed successfully

Closing based on https://github.com/rancher/rancher/issues/32823#issuecomment-852729565


[CI] remove unused inductor workflows

These tests have completely offloaded torch inductor tests to Meta  a few months ago. They are currently disabled on GitHub.
The core Triton is a small number of people, and we receive many PRs (thank
you!).  To help us review your code more quickly, if you are a new
contributor (less than 3 PRs merged) we ask that you complete the following
tasks and include the filled-out checklist in your PR description.

[ ] I am not making a trivial change, such as fixing a typo in a comment.
[x] I have written a PR description following these
rules.
[x] I have run pre-commit run --from-ref origin/main --to-ref HEAD.
Select one of the following.

[ ] I have added tests.

/test for lit tests
/unittest for C++ tests
/python/test for end-to-end tests

[x] This PR does not need a test because FILL THIS IN.

Select one of the following.

[x] I have not added any lit tests.
[ ] The lit tests I have added follow these best practices,
including the "tests should be minimal" section. (Usually running Python code
and using the instructions it generates is not minimal.)

You should only remove inductor related tests

https://github.com/triton-lang/triton/tree/main/.github/workflows/torch-inductor/scripts
https://github.com/triton-lang/triton/blob/main/.github/workflows/torch-inductor-tests.yml

Sorry I got confused by this one https://github.com/triton-lang/triton/actions/workflows/integration-tests.in.yml, but now I realise it's the template so it makes sense to have disabled.

@ThomasRaoux is there anything missing to get this through? Thanks!


Jj checkbox

Update the docs for usage of CDN rather than hard code links.

How did you verify this?

Looks reasonable

How did you verify this?

Ran locally to determine if docs look correct.


Fix compiler warning in utils/conversions.h

/data/caffe2/caffe2/utils/conversions.h(137): warning: variable "r" was declared but never referenced
Comes from https://github.com/caffe2/caffe2/commit/0fccf7a4f7bb930c884aa70894f087d85e870b0a
/cc @asaadaldien

Looks like this was fixed in https://github.com/caffe2/caffe2/pull/400.


Fix typo in Compute ImageDiskReference::id description

Note it exists in the historical versions of the spec too. But I'm just fixing the latest one as that's the one used in the public docs...
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Can one of the admins verify this patch?

Automation for azure-sdk-for-js

Automation for azure-sdk-for-node

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

@wilsonge thanks for the contribution.
@hyonholee while running CI for this PR, we came to know that the default package causes codegen to fail. See more info here  https://github.com/Azure/azure-rest-api-specs/issues/3968, please address the issue ASAP.

Please fix the following files too:
preview\2016-04-30-preview\disk.json
stable\2017-03-30\disk.json
stable\2018-04-01\disk.json

Fixed in the other files mentioned


Duplicate characters / words

Issue Type: Bug
This happended when I wrote a new code and clicked mouse to another section in my file. It duplicated word and reduce my time a lot, hope you would fix it as soon as possible. I've tried to google it but didn't get any answer appropriately.
VS Code version: Code 1.44.2 (ff915844119ce9485abfe8aa9076ec76b5300ddd, 2020-04-16T16:36:23.138Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7 CPU       M 620  @ 2.67GHz (4 x 2660)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: unavailable_offprotected_video_decode: unavailable_offrasterization: unavailable_offskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.86GB (3.24GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

prettier-vscode
esb
4.4.0

LiveServer
rit
5.6.1

Did you maybe set a second text cursor? https://code.visualstudio.com/docs/editor/codebasics#_multiple-selections-multicursor

No i did not set anything but however, iy has been removed somehow. Sorry
for reply late
Vào 16:18, T.2, 20 Th4, 2020 Christof Marti<EMAIL_ADDRESS>đã
viết:

Did you maybe set a second text cursor?
https://code.visualstudio.com/docs/editor/codebasics#_multiple-selections-multicursor
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/95520#issuecomment-616420945,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN5QJXSOSRV4Z5BGJNL52HTRNQHNLANCNFSM4MKVXVJA
.


After docker restart, restarted containers show plugin volume as in-use but volume is detached

Found this when trying the restart tests where,

A container is started in a docker host with the same volume.
Docker is restarted and so is the volume plugin
Container are also restarted and showing as using the volume (via docker inspect
Volume itself is detached.

docker run --restart always --volume-driver vsphere -v restart_test_volume_1086450@sharedVmfs-0:/vol ubuntu tail -f /dev/null
docker ps
CONTAINER ID        IMAGE               COMMAND               CREATED             STATUS          PORTS   NAMES
ceb40a651252        ubuntu              "tail -f /dev/null"   16 seconds ago      Up 5 seconds      sleepy_sammet
docker volume inspect restart_test_volume_1086450@sharedVmfs-0
[
{
"Driver": "vsphere:latest",
"Labels": null,
"Mountpoint": "/mnt/vmdk/restart_test_volume_1086450@sharedVmfs-0",
"Name": "restart_test_volume_1086450@sharedVmfs-0",
"Options": {},
"Scope": "global",
"Status": {
"access": "read-write",
"attach-as": "independent_persistent",
"attached to VM": "worker1-VM1.0",
"attachedVMDevice": {
"ControllerPciSlotNumber": "160",
"Unit": "2"
},
"capacity": {
"allocated": "15MB",
"size": "100MB"
},
"clone-from": "None",
"created": "Wed Jun 28 09:41:19 2017",
"created by VM": "worker1-VM1.0",
"datastore": "sharedVmfs-0",
"diskformat": "thin",
"fstype": "ext4",
"status": "attached" <---------------   volume in use
}
}
]
systemctl restart docker
root@photon-machine [ ~ ]# docker ps
CONTAINER ID        IMAGE               COMMAND               CREATED              STATUS              PORTS               NAMES
ceb40a651252        ubuntu              "tail -f /dev/null"   About a minute ago   Up 8 seconds
docker volume inspect restart_test_volume_1086450@sharedVmfs-0
[
{
"Driver": "vsphere:latest",
"Labels": null,
"Mountpoint": "/mnt/vmdk/restart_test_volume_1086450@sharedVmfs-0",
"Name": "restart_test_volume_1086450@sharedVmfs-0",
"Options": {},
"Scope": "global",
"Status": {
"access": "read-write",
"attach-as": "independent_persistent",
"capacity": {
"allocated": "15MB",
"size": "100MB"
},
"clone-from": "None",
"created": "Wed Jun 28 09:41:19 2017",
"created by VM": "worker1-VM1.0",
"datastore": "sharedVmfs-0",
"diskformat": "thin",
"fstype": "ext4",
"status": "detached" <--------------- volume is detached
}
}
]
docker inspect ceb40a651252
[
{
"Id": "ceb40a651252524459d86f4eb9f80961cde4a93868af297cb201010781c44a5f",
"Created": "2017-06-28T09:44:02.998003405Z",
"Path": "tail",
"Args": [
"-f",
"/dev/null"
],
    "Mounts": [
        {
            "Type": "volume",
            "Name": "restart_test_volume_1086450@sharedVmfs-0",
            "Source": "/vol/docker/plugins/64eb6a41c3edcaa29d9eda4f7bcb11e9e76ae43aaa5a92900cad9ad0e6db984f/rootfs/mnt/vmdk/restart_test_volume_1086450@sharedVmfs-0", <------------ volume is in use AFADIC
            "Destination": "/vol",
            "Driver": "vsphere:latest",
            "Mode": "",
            "RW": true,
            "Propagation": ""
        }
    ],

@govint  By any chance do you have logs? I tried manually to mimic the exact steps you have pointed out in description.
But at the end I see the volume inspect saying it is attached.
root@sc-rdops-vm02-dhcp-52-237:~# docker volume create -d vsphere --name restart_vol
restart_vol
root@sc-rdops-vm02-dhcp-52-237:~# docker volume ls
DRIVER              VOLUME NAME
vsphere:latest      restart_vol@sharedVmfs-0

root@sc-rdops-vm02-dhcp-52-237:~# docker run --restart always --volume-driver vsphere -v restart_vol@sharedVmfs-0:/vol busybox tail -f /dev/null

From another terminal
root@sc-rdops-vm02-dhcp-52-237:~# docker ps
CONTAINER ID        IMAGE               COMMAND               CREATED             STATUS              PORTS               NAMES
661dee276b8f        busybox             "tail -f /dev/null"   55 seconds ago      Up 54 seconds                           admiring_cori

root@sc-rdops-vm02-dhcp-52-237:~# docker volume ls
DRIVER              VOLUME NAME
vsphere:latest      restart_vol@sharedVmfs-0
root@sc-rdops-vm02-dhcp-52-237:~# docker volume inspect restart_vol@sharedVmfs-0
[
    {
        "Driver": "vsphere:latest",
        "Labels": null,
        "Mountpoint": "/mnt/vmdk/restart_vol@sharedVmfs-0",
        "Name": "restart_vol@sharedVmfs-0",
        "Options": {},
        "Scope": "global",
        "Status": {
            "access": "read-write",
            "attach-as": "independent_persistent",
            "attached to VM": "ubuntu-VM0.0",
            "attachedVMDevice": {
                "ControllerPciSlotNumber": "160",
                "Unit": "0"
            },
            "capacity": {
                "allocated": "15MB",
                "size": "100MB"
            },
            "clone-from": "None",
            "created": "Wed Jun 28 16:38:21 2017",
            "created by VM": "ubuntu-VM0.0",
            "datastore": "sharedVmfs-0",
            "diskformat": "thin",
            "fstype": "ext4",
            "status": "attached"
        }
    }
]

root@sc-rdops-vm02-dhcp-52-237:~# service docker restart
docker stop/waiting
docker start/running, process 10471

root@sc-rdops-vm02-dhcp-52-237:~# docker ps
CONTAINER ID        IMAGE               COMMAND               CREATED             STATUS              PORTS               NAMES
661dee276b8f        busybox             "tail -f /dev/null"   2 minutes ago       Up 14 seconds                           admiring_cori

root@sc-rdops-vm02-dhcp-52-237:~# docker volume inspect restart_vol@sharedVmfs-0
[
    {
        "Driver": "vsphere:latest",
        "Labels": null,
        "Mountpoint": "/mnt/vmdk/restart_vol@sharedVmfs-0",
        "Name": "restart_vol@sharedVmfs-0",
        "Options": {},
        "Scope": "global",
        "Status": {
            "access": "read-write",
            "attach-as": "independent_persistent",
            "attached to VM": "ubuntu-VM0.0",
            "attachedVMDevice": {
                "ControllerPciSlotNumber": "160",
                "Unit": "0"
            },
            "capacity": {
                "allocated": "15MB",
                "size": "100MB"
            },
            "clone-from": "None",
            "created": "Wed Jun 28 16:38:21 2017",
            "created by VM": "ubuntu-VM0.0",
            "datastore": "sharedVmfs-0",
            "diskformat": "thin",
            "fstype": "ext4",
            "status": "attached"
        }
    }
]

To add to this, for every docker volume inspect we do a volume get, which fetches volume meta from vmdkops service on ESX. Hence will also need to take a look at vmdkops log from ESX side.

I did this on the same docker host. I can try a repro and post logs.

@govint For test TestRecoverMountsAfterRestart in PR #1500, I have a comment on the failed assert.
I wasn't able to reproduce.
But I enabled just test TestRecoverMountsAfterRestart and corrected the assert as per the comment and looks like this test passed.

Repro again,
root@photon-machine [ ~ ]# docker inspect rvol0
[
{
"Driver": "vsphere:latest",
"Labels": {},
"Mountpoint": "/mnt/vmdk/rvol0",
"Name": "rvol0",
"Options": {},
"Scope": "global",
"Status": {
"access": "read-write",
"attach-as": "independent_persistent",
"attached to VM": "master-VM0.0",
"attachedVMDevice": {
"ControllerPciSlotNumber": "160",
"Unit": "0"
},
"capacity": {
"allocated": "15MB",
"size": "100MB"
},
"clone-from": "None",
"created": "Fri Jun 30 04:40:50 2017",
"created by VM": "master-VM0.0",
"datastore": "sharedVmfs-0",
"diskformat": "thin",
"fstype": "ext4",
"status": "attached"
}
}
]
root@photon-machine [ ~ ]# docker ps
CONTAINER ID        IMAGE               COMMAND               CREATED             STATUS              PORTS               NAMES
7b269b63a86e        ubuntu              "tail -f /dev/null"   13 seconds ago      Up 11 seconds                           kind_roentgen
root@photon-machine [ ~ ]# systemctl restart docker
root@photon-machine [ ~ ]# docker ps
CONTAINER ID        IMAGE               COMMAND               CREATED             STATUS              PORTS               NAMES
7b269b63a86e        ubuntu              "tail -f /dev/null"   53 seconds ago      Up 13 seconds                           kind_roentgen
root@photon-machine [ ~ ]# docker inspect rvol0
[
{
"Driver": "vsphere:latest",
"Labels": {},
"Mountpoint": "/mnt/vmdk/rvol0",
"Name": "rvol0",
"Options": {},
"Scope": "global",
"Status": {
"access": "read-write",
"attach-as": "independent_persistent",
"capacity": {
"allocated": "15MB",
"size": "100MB"
},
"clone-from": "None",
"created": "Fri Jun 30 04:40:50 2017",
"created by VM": "master-VM0.0",
"datastore": "sharedVmfs-0",
"diskformat": "thin",
"fstype": "ext4",
"status": "detached"
}
}
]
    "Mounts": [
        {
            "Type": "volume",
            "Name": "rvol0",
            "Source": "/vol/docker/plugins/c028788ff0a9e8e6b9ef55c5533ed14a1d98b22be80ee6ab2199cfd82bd3cbe8/rootfs/mnt/vmdk/rvol0@sharedVmfs-0",
            "Destination": "/vol",
            "Driver": "vsphere:latest",
            "Mode": "",
            "RW": true,
            "Propagation": ""
        }
    ],

2017-06-30 05:12:49.571178871 +0000 UTC [INFO] Mounting volume name=rvol0
2017-06-30 05:12:50.211706152 +0000 UTC [INFO] Scan complete event="/dev/disk/by-path/pci-0000:03:00.0-scsi-0:0:0:0": 0x100 == IN_CREATE device="/dev/disk/by-path/pci-0000:03:00.0-scsi-0:0:0:0"
2017-06-30 05:13:17.650387974 +0000 UTC [INFO] Unmounting Volume name=rvol0
Restarted by docker restart
2017-06-30 05:13:18.41902903 +0000 UTC [WARNING] Received signal signal=terminated
2017-06-30 05:13:21.681999577 +0000 UTC [INFO] No config file found. Using defaults.
2017-06-30 05:13:21.685313868 +0000 UTC [WARNING] Failed to load config file /etc/docker-volume-vsphere.conf: open /etc/docker-volume-vsphere.conf: no such file or directory
2017-06-30 05:13:21.685331881 +0000 UTC [INFO] Starting plugin driver=vsphere log_level=info config="/etc/docker-volume-vsphere.conf"
2017-06-30 05:13:21.685350761 +0000 UTC [INFO] Plugin options - port=1019
2017-06-30 05:13:21.686168009 +0000 UTC [INFO] Getting volume data from unix:///var/run/docker.sock
2017-06-30 05:13:23.686530963 +0000 UTC [INFO] Can't connect to unix:///var/run/docker.sock due to (An error occurred trying to connect: context deadline exceeded), skipping discovery
2017-06-30 05:13:23.686578624 +0000 UTC [INFO] Refcounting failed: (An error occurred trying to connect: context deadline exceeded).
2017-06-30 05:13:23.68659359 +0000 UTC [INFO] Docker VMDK plugin started version="vSphere Volume Driver v0.4" port=1019 mock_esx=false
2017-06-30 05:13:23.688472749 +0000 UTC [INFO] Going into ServeUnix - Listening on Unix socket address="/run/docker/plugins/vsphere.sock"
2017-06-30 05:13:23.690969817 +0000 UTC [INFO] Scheduling again after 2 seconds
2017-06-30 05:13:25.691247484 +0000 UTC [INFO] Getting volume data from unix:///var/run/docker.sock
Container is restarted by docker and re-mounting the volume
2017-06-30 05:13:26.990279566 +0000 UTC [INFO] Mounting volume name=rvol0
2017-06-30 05:13:27.670195681 +0000 UTC [INFO] Scan complete device="/dev/disk/by-path/pci-0000:03:00.0-scsi-0:0:0:0" event="/dev/disk/by-path/pci-0000:03:00.0-scsi-0:0:0:0": 0x100 == IN_CREATE
2017-06-30 05:13:27.691465519 +0000 UTC [INFO] Can't connect to unix:///var/run/docker.sock due to (An error occurred trying to connect: context deadline exceeded), skipping discovery
2017-06-30 05:13:27.691522168 +0000 UTC [INFO] Refcounting failed: (An error occurred trying to connect: context deadline exceeded). Attempts left: 20
2017-06-30 05:13:27.691530759 +0000 UTC [INFO] Scheduling again after 4 seconds
2017-06-30 05:13:31.692163945 +0000 UTC [INFO] Getting volume data from unix:///var/run/docker.sock
Refcounter found the one container
2017-06-30 05:13:31.718334778 +0000 UTC [INFO] Found 1 running or paused containers
Refcounter doesn't get a refcount for the volume - its not a VMDK volume? Unmounts the volume
2017-06-30 05:13:31.724049229 +0000 UTC [INFO] Initiating recovery unmount. name="rvol0@sharedVmfs-0" refcnt=0 mounted=true dev="/dev/disk/by-path/pci-0000:03:00.0-scsi-0:0:0:0"
2017-06-30 05:13:32.474790482 +0000 UTC [INFO] Discovered 1 volumes in use.
Ref count is zero
2017-06-30 05:13:32.474835895 +0000 UTC [INFO] Volume name=rvol0@sharedVmfs-0 count=0 mounted=true device='/dev/disk/by-path/pci-0000:03:00.0-scsi-0:0:0:0'
2017-06-30 05:13:32.474843722 +0000 UTC [INFO] Refcounting successfully completed

root@photon-machine [ ~ ]# docker plugin ls
ID                  NAME                DESCRIPTION                           ENABLED
c028788ff0a9        vsphere:latest      VMWare vSphere Docker Volume plugin   true

OK, found the issue, docker by default uses the run time root (where all meta-data and images are placed) at /var/lib/docker.
On my VM i have no space on the root FS and hence made a new disk to place the docker meta-data. Docker allows specifying the runtime as -g, --graph string                          Root of the Docker runtime (default "/var/lib/docker")
In my case dockerd is run with -g /vol/docker as below,
dockerd -H unix:///var/run/docker.sock -g /vol/docker --containerd /run/containerd.sock
The bug is in refcnt.go:isVMDKMount() where its expecting that any path with /var/lib/docker should he a VMDK mount. Which is wrong, the code should just check if the driver is vsphere and accept that as a VMDK volume.
    "Mounts": [
        {
            "Type": "volume",
            "Name": "rvol0",
            "Source": "/vol/docker/plugins/c028788ff0a9e8e6b9ef55c5533ed14a1d98b22be80ee6ab2199cfd82bd3cbe8/rootfs/

^^^^^^^^^^^^^^
mnt/vmdk/rvol0@sharedVmfs-0",
"Destination": "/vol",
"Driver": "vsphere:latest", <--------- should check this instead.
"Mode": "",
"RW": true,
"Propagation": ""
}
],

Making it a P1 as users may not frequently be using a different location for the docker runtime, but its something to be fixed.

Actually, this is a good use case for vSphere where the images can be located on a single disk thats attached to multiple docker hosts. It may be used as a multi-writer VMDK and allow docker hosts to share the images that docker is pulling on those VMs. Allowing much less storage to be used across a set of docker hosts as well.

Closing this as presently Docker doesn't allow a plugin to uniquely identify a volume as belonging to it. Using the driver name to identify a volume is going to pick the alias used to install the plugin which can be anything that the user set. And surely not uniquely identifying the volume as a vsphere managed volume.


Added a section to the getting started for phase 3

I added a section that explains the webpage the server will use for testing. It's at the end of the getting started page.

It looks like we lost the instruction to set up the server main function to call server.run. That probably needs to be added independent of this suggestion. After that the web page should just work. There are a couple references to the web page in the instruction, but it might be nice to have a final one here.
There is some discussion about if the web page is really necessary. Perhaps it should just be replaced with an emphasis on using curl instead.


flinkcdc3 official website operation error

My question is below ，please help

When I followed the official website's operating documentation step by step, an error message was reported and I don't know how to solve it

then i run this script called bash  'bash flink-cdc.sh mysql-to-doris.yaml'
it goes wrong
java.lang.NoClassDefFoundError: Could not initialize class io.debezium.connector.mysql.MySqlConnectorConfig
The following is the version I use

flink1.18
2.flinkcdc3.1
by the way : i follow this doc 'https://nightlies.apache.org/flink/flink-cdc-docs-release-3.1/zh/docs/get-started/quickstart/mysql-to-doris/'

please help!

this server is done! flinkcdc3.0 can work

Has the problem been resolved

mysql-connector-java should be placed to $FLINK_HOME/lib or be passed with CDC --jar argument.
It is preferred to report issues / request new features on GitHub Discussions, Flink JIRA, Apache mailing list, etc. Questions on GitHub issues may not be actively checked, and will be closed without further explanation.

thank
------------------ 原始邮件 ------------------
发件人:                                                                                "apache/flink-cdc"                                                                                @.>;
发送时间: 2024年7月11日(星期四) 下午5:03
@.>;
<EMAIL_ADDRESS>主题: Re: [apache/flink-cdc] flinkcdc3 official website operation error (Issue #3457)
mysql-connector-java should be placed to $FLINK_HOME/lib or be passed with CDC --jar argument.
It is preferred to report issues / request new features on GitHub Discussions, Flink JIRA, Apache mailing list, etc. Questions on GitHub issues may not be actively checked, and will be closed without further explanation.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


[manager] Remove activation input exec order from backwarding

This patch removes activation input exec order from backwarding as input
of the activation layer is not used in the backwarding.
This leads to change in the unittests as we cannot check all the
outputs, especially close to the end of the model. So, with optimization
enabled, only the output layer's forwarding is checked.
Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1767. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.


Create TextInputField based on TextInput using uesNodeKeypress

Currently TextInput is built on useInput which really doesn't work w/ useNodeKeypress.

useInput is hooked into data event via StdinContext - https://github.com/nazmulidris/ink/blob/ece996911c187b1ce3c601315c84cfb3e6219715/src/hooks/use-input.ts#L115
TextInput gets its key events from useInput - https://github.com/vadimdemedes/ink-text-input/blob/27c40731bcf1b490dd90f67a49cde992715e0da1/source/index.tsx#L52
useNodeKeypress is hooked into keypress events from readline

Here are the changes that need to be made.

Copy TextInput into TextInputField (in r3bl-ts-utils)
1.1 Replace useInput w/ useNodeKeypress
1.2 Remove all the needless code
Change useNodeKeypress
2.1 It should only have 1 active keypress listener at any given time
2.2 Remove all existing listeners when a new one is attached, restore them on detach
2.3 Might need to create an ActionRegistry (singleton, static, just like TimerRegistry) to handle these things.

Test it in the existing experiments that use TextInput.
Next - Apply similar thinking to migrating confirm-input to useNodeKeypress

Upon further analysis, this won't be necessary. Even w/ only 1 keypress listener being active, it won't help if there are multiple components that are visible at the same time that have all used this hook. They are all hooked up to get events.
The only way to manage this situation is to have a focus system which ink does provide.
In fact, it would be best to have confirm-input leverage useInput rather than useNodeKepress for this reason.


API-1835: realign manifestclient on fs.FS

/lgtm
/hold
so that @bertinatto can take a look as well.


checkout with fetchDepth: 1 does fetch tags

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Feature
Enter Task Name: checkout
Environment

Azure Pipelines

Issue Description
When using
  - checkout: self
    fetchDepth: 1

it does a git fetch --tags ... behind the scenes, which fetches more history than needed.
We expected it to speed up our build pipelines with big repos but there was not much change.
Could fetching tags be an option to disable?

Hi @yene since this feature would require some change on Azure DevOps side also (new parameters for checkout task) - could you please create ticket on https://developercommunity.visualstudio.com/search?space=21 to get right eyes on it?

@yene let me close this one here since it requires some changes on DevOps side also.

https://developercommunity.visualstudio.com is where feature requests go to die

Haha, it died there :) https://developercommunity.visualstudio.com/content/problem/1095986/azure-pipelines-checkout-fetches-all-tags.html

So sad that so many good suggestions just go to die in the devloper community whenever they have to be suggested as ideas for azure pipeline 😞.
This should have been prioritised option in the spirit of the ongoing effort for a greener environment where we spend less un-waisted energy (irony kidding not kidding 😂 ),  since a git checkout with no tags means less bandwidth used when fetching a repo and less output to stdout means less energy used by every pipeline run 🤷🏽
i.e.: we have a repo with over 2000k tags whenever we do a git checkout it fetches all the tags and prints them to stdout…

We do checkout manually, now it takes 6 seconds instead of 6 minutes.
steps:
- task: Bash@3
  displayName: 'Fast Checkout'
  inputs:
    targetType: 'inline'
    script: |
      set -ex
      git init $(System.DefaultWorkingDirectory)
      git remote add origin $(Build.Repository.Uri) || true
      git config gc.auto 0
      git config advice.detachedHead false
      git config http.version HTTP/1.1
      git -c http.extraheader="AUTHORIZATION: bearer $(System.AccessToken)" fetch --force --prune --no-recurse-submodules --depth=1 origin +$(Build.SourceVersion):refs/remotes/origin/$(Build.SourceVersion)
      git checkout --progress --force $(Build.SourceVersion)

We do our own checkout step, now it takes 6 seconds instead of 6 minutes.
steps:
- checkout: none
- task: Bash@3
  displayName: 'Fast Checkout'
  inputs:
    targetType: 'inline'
    script: |
      set -ex
      git init $(System.DefaultWorkingDirectory)
      git remote add origin $(Build.Repository.Uri) || true
      git config gc.auto 0
      git config advice.detachedHead false
      git config http.version HTTP/1.1
      git -c http.extraheader="AUTHORIZATION: bearer $(System.AccessToken)" fetch --force --prune --no-recurse-submodules --depth=1 origin +$(Build.SourceVersion):refs/remotes/origin/$(Build.SourceVersion)
      git checkout --progress --force $(Build.SourceVersion)

Thanks for sharing 👍🏽

Azure pipeline has now an option fetchTags: false
          - checkout: self
            fetchTags: false
            fetchDepth: 1

How to run this for UI Pipeline? Thank You

@IbrahimUmar Wasim :)  you can set this from pipeline setting from UI as well.


The number of apps taking up storage space on mobile phones has skyrocketed

Dear developer: Hello! While using the plug-in, I encountered a problem and searched for a long time without finding a solution. Questions as follows: in the use of CameraPicker. PickFromCamera create pictures after the APP takes mobile storage space will continue to increase. If continuous photos are taken, the number of APP storage space occupied by the phone will rise sharply. Plug-in versions: 3.6.3, 4.1.1. Is there any good solution? thank you.
尊敬的开发者： 您好！在使用插件期间，遇到一个问题，查找很久未找到解决方案。问题如下：在使用CameraPicker.pickFromCamera创建拍照后，APP占用手机储存空间会持续增加。如连续拍照，APP占用手机储存空间数值会急剧上升。插件版本：3.6.3，4.1.1。请问有什么好的解决方式吗？谢谢。

https://github.com/fluttercandies/flutter_wechat_camera_picker/blob/414b7dc1e447c72d371d30a79e5c2404e0f4ea20/lib/src/constants/config.dart#L89-L93


DigitalPebble partner information

**NOTE: This is an issue template for new partner pages we are building. Existing partners can use them to submit information needed as we build out enhanced pages. New partners, please continue to use the process outlined here for the time being: https://opensearch.org/new-partner.html
Organization name
DigitalPebble Ltd
Brief description of your business and what it does (50 words or less)
DigitalPebble's unique expertise covers all aspects of a document’s life cycle, from web-wide crawling and collection, content analysis, filtering and categorization to indexing.
DigitalPebble can help your organisation by advising on best practice and identifying suitable resources, designing scalable solutions as well as implementing them.
Name of your relevant product(s) or service for the OpenSearch audience
We are the authors of StormCrawler, a popular open-source web crawler which can  use OpenSearch as a back-end. We also provide consulting services on search related technologies, including OpenSearch.
What OpenSearch technologies do you specialize in
Is there an area of the project where your business specializes?
We specialise in search, analytics and observability.
Type of business
(e.g., MSP, CSP, ISV, product/technology, services organization, other). If other, please define.
This will inform the community how they can expect to use your product(s) and services.
Consultancy
Home/main office location or address
Bristol, UK
How can people contact you (email and phone)
email<EMAIL_ADDRESS>Regions you serve
(e.g.,North America, South America, Europe, Middle East, Africa, Asia Pacific, Australia)
What regions of the world do you serve?
Worldwide
Industry specialization
(e.g. business services, consumer services, education, energy and utilities, financial services, healthcare, media and entertainment, public sector, non-profit, retail, software and technology)? Add all that apply.
Our clients operate in various industries,
Logo
Please upload a company logo
Additional image
Attach any additional image you would like to provide that showcase your business (product, technology, offices)
Links
StormCrawler
URLFrontier
OpenSearch community meetup

@jnioche - Thank you for submitting the requested info.

@pajuric Would you like to assign this to me so I can add the data?


Product landing pages

Since we've talked about this and we've also got it listed on the public website I thought we'd might as well make it official and post a feature request here ;).
The idea here is to have an editable landing page for a product so serve as an entry-point into the rest of the documentation for the product contributed by the feature files.

Ideally, each product would start out with a default/empty page. The page should also allow the user to add new markdown which then replaces the empty page.
Aditionally, there should be a page where product pages can be maintained. Ideally this page would also be used to add or remove products.


[Snyk] Security upgrade openjdk from 8-slim to 18-slim

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

Dockerfile

We recommend upgrading to openjdk:18-slim, as this image has only 61 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

614
Integer Overflow or Wraparound SNYK-DEBIAN11-KRB5-3120880
No Known Exploit

714
Out-of-bounds Read SNYK-DEBIAN11-LIBTASN16-3061097
No Known Exploit

714
Out-of-bounds Read SNYK-DEBIAN11-PCRE2-2808697
No Known Exploit

714
Out-of-bounds Read SNYK-DEBIAN11-PCRE2-2808704
No Known Exploit

714
Out-of-bounds Write SNYK-DEBIAN11-ZLIB-2976151
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

trying ah ting


[REQUEST] Determine delay from input

The usual convention for demuxed audio files is usually something such as
whatever DELAY 9ms.anything
and the value can be both positive and negative.
It would be nice that deew could parse the delay and apply it automatically to have a neutral delay output file.

implemented in 2.8.0


Use RouterTestServer in every tests that performs Network requests to…

… make sure we don't do real requests by mistacke
Hopefully might help to troubleshoot the occasionally failing test #79

Current coverage is 99.57% (diff: 100%)

Merging #121 into master will not change coverage

@@             master       #121   diff @@
==========================================
  Files            10         10
  Lines           474        474
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            472        472
  Misses            2          2
  Partials          0          0

Powered by Codecov. Last update 384a633...82bdaab


Install MSYS to build native extension for http_parser.rb - jekyll-feed gem depends on it

Without msys, building native extensions for ruby gems fail. See https://github.com/rubygems/rubygems/issues/977
Remove comment from https://github.com/juliusgb/dev-laptop-setup/blob/main/packer/template.pkr.hcl#L159 to install it.
On fresh jekyll site, run bundle install, I get
In Gemfile:
  jekyll-feed was resolved to 0.16.0, which depends on
    jekyll was resolved to 4.0.1, which depends on
      em-websocket was resolved to 0.5.3, which depends on
        http_parser.rb
mingw32-make: *** [makefile:5: prep] Error 5

Manual isntall of the http_parser gem also fails.
Command: /path/to/jekyll-site> gem install http_parser.rb --version 0.6.0
Output:
Temporarily enhancing PATH for MSYS/MINGW...
Building native extensions. This could take a while...
ERROR:  Error installing http_parser.rb:
        ERROR: Failed to build gem native extension.

Output of ridk version doesn't have the title and version as in https://github.com/rubygems/rubygems/issues/977#issuecomment-414073762:
---
ruby:
  path: C:/opt/tools/Ruby/3.0.4/x64
  version: 3.0.4
  platform: x64-mingw32
  cc: gcc.exe (Rev10, Built by MSYS2 project) 11.2.0
ruby_installer:
  package_version: 3.0.4-1
  git_commit: 17502f9
msys2:
  path: C:\Program Files\Git
cc: gcc.exe (MinGW-W64 x86_64-posix-seh, built by Brecht Sanders) 11.2.0
sh: GNU bash, version 4.4.23(1)-release (x86_64-pc-msys)
os: Microsoft Windows [Version 10.0.19044.1826]

Update from https://rubyinstaller.org/downloads/

RubyInstaller uses the MSYS2 toolchain as its development kit. . . MSYS2 is required in order to build native C/C++ extensions for Ruby and is necessary for Ruby on Rails. . . .The MSYS2 development environment can be activated in a running cmd or powershell console by ridk enable. This adds commands like make, gcc, pacman or sh to the search path. See more in the Wiki.

So

[ ] Install MSYS2
[ ] Install Ruby DevKit


Celanthe/Rin Checking In

Hi all! I am a technical writer, livestream and podcast producer, and K8s Member/Storyteller on the Upstream Marketing Team.
In the vein of 'bringing a dish to a potluck,' I can offer my skills in livestream production, writing docs, and writing how-to's/setup instructions, etc.
I also have a bit of front-end development experience, and a very small amount of experience with APIs and would be grateful for the chance to pair with someone to tackle some of those issues.
I look forward to contributing.

Hi welcome!
Would you be interested in reviewing my prototype once I finish it?

Absolutely, @kris-nova :) that sounds great!


Make Android queryRenderedFeatures() consistent with iOS and GL JS

Is your feature request related to a problem? Please describe.
I am currently trying to call queryRenderedFeatures() from flutter-maplibre-gl and there's a big difference between how each platform treats the layerIds parameter.
The core of the problem is that on iOS and for GL JS passing an empty array to layerIds parameter still applies the filter and returns an empty result every time, but on Android an empty array gets ignored as if null was passed.
Describe the solution you'd like
I find the iOS and GL JS implementation sensible - null means I don't want to filter by layer ID, empty array means I want to filter by layer ID, but there's no match since the array is empty. I think Android implementation should use the same logic.
I think the following line  should not have the array length check and should accept and pass down empty arrays as well. With that the behavior will be dictated by mbgl implementation and not by the platform glue code.
Unfortunately the solution is not as simple, because it would break current Java API. Since varargs are (IMO inappropriately) used here, passing no argument to layerIds results in empty layerIds array to be passed down to native and only then it is filtered out. Removing varargs from there would be painful but correct solution in my opinion so it is possible to pass either nothing (null) or an empty array.
Describe alternatives you've considered
In my case I could also attempt to write some glue code directly into flutter-maplibre-gl to make it work consistently across all three platforms, but I believe it should be fixed here.

Would adding another API and deprecating the old one be a good option perhaps?

Yes, I think deprecating the varargs API and replacing it with one which accepts nullable array makes sense. If the deprecated API then converts empty varargs to null layerIds parameter it should be functionally equivalent and not break anything.

Something to watch out for is that Kotlin can't easily pass null to a nullable vararg. The iOS SDK accepts a Set, perhaps a nullable Set is a good approach for the Android side?