Running bentoctl build fails with stable_diffusion_fp32:latest not found in Bento Store

I am following tutorial and all works well until I reach the point of building docker image and pushing to AWS ECR. Following command fails:
(venv) ➜  bentoctl git:(main) ✗ bentoctl build -b stable_diffusion_fp32:latest -f deployment_config.yaml

Error [BentoNotFound]:  Bento with tag <stable_diffusion_fp32:latest> not found in Bento Store.

Perhaps I should call bentoml build instead?

Same issue for me. The tutorial is broken.

I don't remember how but i figured it out.

Hmm. Can you share more details when you have time/remembered?
Also we have a few exciting things coming up and will make it more feature rich and super fast inference speed.
Definitely keep an eye out here

Apologies but I don't remember exactly. I think there were some differences between the readme in this repo and https://modelserving.com/blog/deploying-your-own-stable-diffusion-service-mz9wk. Following the readme worked, following the blog post didn't.
One thing I remember it wasn't clear in the blog post from which directories should each command be executed.


Workspace dropdown not aligned properly for Fleet/CD App in v2 dashboard

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Go to Cluster Explorer, and click select Global Applications->Continuous Delivery
The dropdown for selection Fleet Workspace is not aligned properly with the top blue strip

Result:

Environment information

Fleet chart version: 0.3.400-rc3
Rancher version: v2.5.6-rc4
Installation option (single install/HA): single
Reproducible in Microsoft Edge 90.0.782.0, Vivaldi 3.6.2165.36 (both WebKit based) and FF 78.7.0esr on openSUSE 15.2

The fix appears to have worked.
Before
Rancher version: v2.5.6-rc4
Installation option (single install/HA): single
Browser: Chrome 8.0.4324.182
OS: macOS Big Sur 11.2.1
Arch: amd64

After
Rancher version: latest
Installation option (single install/HA): single
Browser: Chrome 8.0.4324.182
OS: macOS Big Sur 11.2.1
Arch: amd64


Script to generate configuration files & snowplow infrastructure

Would be awesome to have a script that prompt the user for what type of Snowplow setup they would like to setup and generate the appropriate configuration files. Bonus points for the script using the AWS api to setup infrastructure.

good idea, however I don't think we'll ever get to it, closing


GCS ref doc update set content type

fixes #861
GCS ref doc update set content type

Codecov Report

Merging #1328 into master will increase coverage by 0.37%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master   #1328      +/-   ##
===========================================
+ Coverage     67.72%   68.1%   +0.37%
- Complexity     1397    1402       +5
===========================================
  Files           202     202
  Lines          5643    5643
  Branches        567     567
===========================================
+ Hits           3822    3843      +21
+ Misses         1587    1563      -24
- Partials        234     237       +3

Flag
Coverage Δ
Complexity Δ

#integration
?
?

#unittests
68.1% <ø> (+0.37%)
1402 <ø> (+5)
:arrow_up:

Impacted Files
Coverage Δ
Complexity Δ

...gure/trace/sleuth/StackdriverHttpServerParser.java
87.5% <0%> (+50%)
3% <0%> (+2%)
:arrow_up:

...gure/trace/sleuth/StackdriverHttpClientParser.java
85.71% <0%> (+80.95%)
4% <0%> (+3%)
:arrow_up:

Continue to review full report at Codecov.

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


Add PHP as known bucket

I hope it's useful enough to be merged

Great — thank you!

That's hot! 👍

Thanks!


fixes qor/qor-example#59

filepath.Base seems to be crossplatform alternative for path.Base

thank you for your pr.

Can it be fixed?

Sorry, I must clicked the wrong button when try to merge it ;)
Merged.

Nice thanks! :+1:


Importer sometimes imports the same scene more than once

http call should act the same if the the scene already exists (use upsert or ignore)
Currently causes a psqlexception that isn't caught

Duplicate #668 I think?

This is for the backend, not the DAG. IE http request should not cause a 500


[5.5] Driver: do not generate a dSYM bundle for static archives

When building a static library with debug information, do not create a
dSYM generation job as it cannot be executed on a non-image target.
This is important for the case where the single invocation is both the
compile and link job.
This was detected in conjunction with @gottesmm.

Explanation: When emitting a static library, the driver attempts to run dsymutil on it. dsymutil is not supposed to be run on archives so it fails. As a result, one cannot compile static libraries with debug info at all. This patch just fixes the behavior by checking when adding the dsymutil job if we are emitting a static library and in such a case, do not add the dsymutil job.
Scope: Small. Just ensures that if we are compiling a static library we do not add a dsymutil job in the driver.
Risk: Low.
Testing: Adding a compiler test that validates the behavior.
Issue: rdar://78271443
Reviewer: Xi Ge

@swift-ci test

@nkcsgexi @artemcm

@swift-ci nominate


Feature Request - Make searching for assets case insensitive

I don't think searching within asset should be case sensitive as asset names should always be unique regardless of case.
Searching myworkstation-15 should be able to return the asset MyWorkStation-15

Merge resolves this.


[Docs] VUI documentation

Write the documentation for VUI

[x] Properties
[ ] Events
[x] Elements

I think that it doesn't make sense anymore. plus, we need to find a better way of documentation..

+1


Set env variables in shell using eval $(up env --shell -s test)

Prerequisites

[x] I am running the latest version. (up upgrade)
[x] I searched to see if the issue already exists.
[x] I inspected the verbose debug output with the -v, --verbose flag.
[x] Are you an Up Pro subscriber?

Description
I love the env feature in pro and we’re using it extensively as a team. There are cases where we’d like to access env variables stored in up, but without running up itself. Would be cool if there was a way to “set” the terminal env using an up command and then run a completely unrelated script.
Discussed on slack, but adding it here so it's easy to track.

Done! I'll have a new Pro release out tonight. Curious to hear more about the use-case, I was thinking about this a few months ago but I didn't have a concrete use myself

0.7.8-pro is good to go, now you can do:
$ eval "$(up env export)"
$ echo "name=$DB_NAME user=$DB_USER"
name=users user=sloth

or with a non-"development" stage:
$ eval "$(up env export -s production)"
$ echo "name=$DB_NAME user=$DB_USER"
name=users user=sloth


Support filtering by prefix in GitHub latest Release and Tag Badges

:clipboard: Description

Please see context and discussions at https://github.com/badges/shields/discussions/6253

I'd love to see this as well, my reason is mentioned here #7082

I want this for having multiple go packages with their own go.mod files within the same repo. Go allows versioning the subpackages by tagging like subfolder/v0.0.1. I would like to show badges for each subpackage in a table on the readme. That is why I made that PR ^.
The only issue I have in the PR is that the code already limits to getting 100 tags when sorting so if one subpackage is never updated and others have more that 100 releases then the filter might not work.

Hi, is there any updates? Need this for Go modules in same repo.

Hello, this feature would be really helpfull.
On the README we have we are checking for releases for an helm chart for example for argoCD.
https://github.com/argoproj/argo-helm/releases
In this case we have several different helm releases for the different argocd components, like events, workflows, etc.
It would be usefull if we can have some kind of regex filter to show for example per date the release for argo-events-* or argo-cd-*.
This way we could monitor different components in the same repo.
Keep up the good work!
Thanks

Hello. We have just shipped a new feature which allows you to pass an arbitrary filter to the github tag and release badges. For example
The filter param can be used to apply a filter to the project's tag or release names before selecting the latest from the list.

Hi @chris48s, is there an option to transform the tag name, in some context, the prefix (modules/socials) is redundant.

No the tag badge will always just display the name of the tag.
For releases you can pick either the tag name or the release name.


hope release a new version

@hustf

yes but I think there are some issues left https://github.com/JuliaWeb/WebSockets.jl/issues/180

WebSockets.jl now has a new version, requiring Julia 1.8.2+ and HTTP 1.1.0 to 1.5. Hopefully, this solves the issue; simply do Pkg.update()!


Included scripts are added to  and bundle

Results in external.js being referenced in the document  and added to user~scene.bundle.js:
FamousFramework.scene('user:scene', {
})
.config({
    includes: [
        'external.js'
    ]
});

Another great catch. We should only be inlining the external.js and not including it in the head. (This is related to a bit of vestigial code that should be removed.)


Signing an execute transaction with multicall

I'm trying to implement a multicall on the frontend
const acc = new Account(
  new Provider(),
  account?.address,
  account?.signer
);
const nonce = await acc.getNonce();
const result = await acc.execute(
  [
    {
      contractAddress:
        "0x088888284bf0dce7811aea8950eab7c39c22f2594cef7124366aa93edbcc6b8e",
      entrypoint: "mint",
      calldata: [account?.address, 1, 2, 0, 1, 77, 0],
    },
  ],
  undefined,
  { maxFee: 500, nonce }
);

This doesn't invoke Argent X popup window for signing
But passes through and gives me a tx hash with rejected status:
Error message: signer signature invalid

How can I sign this transaction on the frontend?

I don't think we do any integration in starknet.js directly with any wallet interface, as so no wallet interface will pop up.
Starknet.js is a low-level JS SDK to communicate with the Starknet network.
ArgentX has some daap examples on the repo that you can check out.
Did you resolve your issue in meantime, or would you need additional guidelines?

Hey there, if you want to use Argent X to execute transaction, you could use following code, but make sure you have Argent X installed and be active:
import { stark, Contract} from "starknet";

 await starknet.enable();
const starknet = window.starknet;
const signer = starknet.account;
const callContract = new Contract(abi, contractAddress, signer);
const callTx = await signer.execute(
      {
        contractAddress: contractAddress,
        entrypoint: "mint",
        calldata: [accountAddress, 1, 2, 0, 1, 77, 0]
      }
    );

Then you will see argentX pop up.

Help Considered resolved with example.


Add event/callbacks for forced pausing

In systems such as iOS, Android, Wii U, etc. applications can be "paused" by the user (by exciting the app, pressing the home button, etc).Support to intercept these events is quite important, as often the GPU must be reset and primed when switching back.
Thus, I propose the addition of pause/unpause events/callbacks.

This needs a lot of investigation and a lot of changes are needed in the core of FRAG to properly support android.
I'm going to need either -
A) Someone who is interested in trying to build a mobile game with FRAG who can work closely with me on implementing what needs to be implemented
B) Some time to do the above myself
Either way this is high priority and should be tackled soon.


[Android] Release BLE transport during kill update manager

Fixes #32

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


Nuget v3 not writing content and .pp files to .cs files

Okay, so I added LibLog to a project, but Nuget seems to have not done anything with the .pp files.
Might need to create a script or something to run that transformation step now.
I cant seem to find any documentation of that change so maybe its my machine?  Im using VS2017 15.9

Btw, I just finished up a utility that can dip into the CSPROJ file using the Roslyn and MSBuild workspace providers, not sure what the solution to this particular problem would be since a lot of packages depend on content trasformation

I can't repro this. Below is a screen shot of a fresh project that where I installed LibLog. Compiles fine, cs files are in obj and namespaces are correct (though had to add the Microsoft.CSharp package reference). Nuget (any version) isn't involved here; it part of an msbuild sdk task now.

Note: LibLog only works with new SDK csproj formats.
Marking it as invalid (aka - no-repro). If you can upload an example project to github where it is not working I can take a look. Cheers.


Introducing Tree of Thought Tool

Summary:
This pull request introduces a new tool, TreeOfThought tool. This tool is designed to solve problems by going through the steps of the Tree of Thought process, a method that involves generating and evaluating thoughts, selecting the most promising ones, looking ahead to predict outcomes, backtracking if necessary, considering global factors, planning and searching, and reevaluating the solution.
Key Changes:
New Schema: Added TreeOfThoughtSchema which takes a problem description as input.
New Tool: Added TreeOfThought, which uses the TreeOfThoughtSchema and follows the Tree of Thought process to solve the given problem. This tool uses the BaseTool class as a base and overrides the _execute method to implement the Tree of Thought process.
Integration with Existing Code: The TreeOfThoughtTool is designed to work with our existing BaseLlm language model and AgentPromptBuilder for generating prompts.
This new tool will enhance our project's problem-solving capabilities by incorporating the Tree of Thought process. It will allow our AI to generate multiple solutions, evaluate and compare them, and select the most promising one, making AI more robust and efficient in problem-solving.
Please review the changes and let me know if there are any issues or improvements needed.

@tensorwar
Is your commit working? I've been working on local LLM integration and this was where I had my gazes pointed to next.

@tensorwar Is your commit working? I've been working on local LLM integration and this was where I had my gazes pointed to next.
Hey !! Been following your work on the LLM integration and you did an amazing job!!! 👏 🙌  i feel lile this is finally what we were hoping autogpt would be! And absolutly .. once we get that magic ToT in There it will be a garden of eden!!! Unfortunately i gotta agree. Would love for it to work as simple as this commit but it looks like it needs more integration... however it looks like neelayan7 posted a githib to a ToT that works. https://github.com/princeton-nlp/tree-of-thought-llm

Not sure how to merge it into the current SuperAGI project but shoot.. once we got this with LLMs we could just leave our computer on for days and let it build a garden of eden. Good luck and will help however my "ChatGPT coding expertise: can help!


Add missing package-info.java for java12 package

Motivation:
Fix build failure

[ant:checkstyle] [ERROR] /Users/kezhenxu94/workspace/armeria/core/src/main/java12/com/linecorp/armeria/internal/common/CurrentJavaVersionSpecific.java:1: Missing package-info.java file. [JavadocPackage]

Modifications:

Add package-info.java in java12/com/linecorp/armeria/internal/common

Result:

Build successfully without error.

Thanks for a quick fix, @kezhenxu94 :bow:


Server throwing unhandledRejection

It happens when doing any move. Using 0.21.5.
unhandledRejection TypeError: Cannot read property 'allowedMoves' of undefined
    at canMakeMove (C:\projects\playbg\node_modules\boardgame.io\dist\core.js:1481:22)
    at Object.canMakeMove (C:\projects\playbg\node_modules\boardgame.io\dist\core.js:971:14)
    at Socket.<anonymous> (C:\projects\playbg\node_modules\boardgame.io\dist\server.js:1241:26)
    at <anonymous>

Is there anyway you could provide the object your passing to Game? It would help add context to the error.

Please add repro steps, or provide your code.
The multiplayer examples utilizing the server seem to work. I've manually deleted node_modules and dist, merged latest, ran npm i and then npm run examples. Maybe I've missed something, the command is showing that the dev server is using 0.21.5.
The code in question fetches the phase from the phase map, and calls allowedMoves on it. allowedMoves is guaranteed to be a function as per that code. Looks like the problem is somewhere in either the phase config or the phase code.

Here is the code: https://pastebin.com/a8U4fTBs
I observe that the canMakeMove() is called twice. First time it contains default phase only (not sure what it is), and it throws an error. Second time it contains all my phases.

Weird. The default phase is only used if no phases is passed to the flow. Since the code passes one, you should not see it at all.
Thanks for the code, I'm trying to somehow repro your scenario with it, atm to no avail. Could you share a complete example that demonstrates the issue?

Closing this for now. @Toktik Feel free to ask at https://gitter.im/boardgame-io/General if you continue to have any problems.


improve validation in tx handler

limits were not enabled in the final tx format
also check both fee and gas price

bors merge


WIP: Allow for pretty-printers to retrieve values

Closes https://github.com/anixe/doku/issues/1.

Closing in favor of https://github.com/anixe/doku/pull/9.


Address change in node causes forking

Nodes get address from other nodes on announcing, they fetch the address right after that. If user changes their address later, they produce blocks with a different hash, because they address is different and they split off to a fork.

peers can now choose to be removed from the system by disconnecting for 1 hour


Save messages for each user

Hi Team,
I would like to check if there is some examples for saving messages to traditional database and use it for each request received from client.
Thanks!

There are no examples of such in the current repo as this can be achieved in many different ways depending on your own database model.
Normally assuming the most simplistic scenario in a MS SQL SERVER.
UserMessages:

UserId
Role
Content
TimeStamp

Retrieving the last 10 messages in the conversation
SELECT TOP 10 Role, Message FROM UserMessages WHERE UserId = {UserId} ORDER BY TimeStamp DESC

// Initialize the kernel...
var chatCompletion = kernel.GetRequiredService<IChatCompletionService>();

// Retrieve Messages
var records = RetrieveFromDatabase(userId);
chatHistory chatHistory = [];
foreach (var record in records);
    chatHistory.Add(new ChatMessage(new AuthorRole(record.Role), record.Content));

var userInput = Console.ReadLine();
var userMessage = new ChatMessage(AuthorRole.User, userInput);
var assistantMessage = chatCompletion.GetChatMessageContentsAsync(chatHistory);

// Persist back the new messages
PersistMessagesIntoDatabase([userMessage, assistantMessage]);


Random improvements

Dropping support for net35, net40. Now multi-targeting netstandard2.0, netcoreapp2.1, netcoreapp3.1. 🎉
Removed helpers such as ArrayHelper, MethodImplOptionPortable, ExcludeFromCodeCoverage that were needed for framework target compat.
Unsealed MarkdownParserContext (@patriksvensson I don't suppose you somehow rely on it being sealed? I figure having it unsealed adds more value)
Removed some allocations (b090f24, 84ec832, bdfaa27, 30d0924), 2% overall.
Improved codegen for some StringSlice helpers.
Used ThrowHelper everywhere, hopefully more things can inline now.
Further optimized CharacterMap in 256b4d3. @craigwi while not the reason I did it, this may also workaround the UWP issue.

Overall ~6% parsing throughput increase. Allocation-wise we are now very close to noise-free. The vast majority of allocations are now the AST itself and some strings.

You changed BitVector128, a 16 byte struct, to BoolVector128, a 512 byte struct. Are you sure you want to do that?

The allocation overhead of 128 bytes is negligible as we don't construct character maps often. Calls to CharacterMap methods don't copy the struct either so I don't see a downside.

Lots of interesting changes.  My recommendation is to split the PR into about a half of a dozen smaller PR, each of which focuses on a specific change to make it easier to review.

I tried to keep changes isolated between commits and I don't think they should be too contentious (except perhaps a99e650). Is there any particular change you would find easier to review if I isolate it into a separate PR?

The allocation overhead of 128 bytes is negligible as we don't construct character maps often. Calls to CharacterMap methods don't copy the struct either so I don't see a downside.

Well, the struct no longer fits in a single cache line.
Note that going from a 128 byte array to a 16 byte struct resulted in a 3% throughput improvement for the parsing.

Is there any particular change you would find easier to review if I isolate it into a separate PR?

This might be such a change yes. It would be interesting to see before and after benchmark results.

This might be such a change yes. It would be interesting to see before and after benchmark results.

You got it, I'll move it out. The biggest chunk of throughput improvement from this PR comes from character map changes.

Should we not ensure that the array cannot grow without bounds?

We should, looks like I over-simplified that part.

@MihaZupan Just an old habit of sealing everything until there's a reason for it to be unsealed.

@KrisVandermotten I've extracted it to https://github.com/MihaZupan/markdig/pull/1, you can see the difference in the generated asm.

@MihaZupan I'm sorry I did not make myself clear. I'm curious about the effect is of only the change from the 16 byte struct to 512 byte struct. I have no doubt that some of the other changes you did there are beneficial.
FYI, I may find some time later this week to take a look myself as well.

I'm curious about the effect is of only the change from the 16 byte struct to 512 byte struct

Ah I see. The struct is 128 bytes, not 512. I can run a benchmark on just this change but I would expect the bool lookup to still be (not much) faster.

FYI, I may find some time later this week to take a look myself as well.

Glad to hear it!

The struct is 128 bytes, not 512.

I stand corrected. Somehow I was convinced that sizeof(bool) == 4. It goes without saying that the difference is significant.

cc @Sergio0694

@MihaZupan NICE! 🍻🚀
A couple questions:

Why don't you also add the netstandard.2.1 target while you're at it? Have you checked if there's some overlap with the .NET Core 3.1 APIs you're using and .NET Standard 2.1, so that those new features could also be available in the portable version?
Regarding the ThrowHelpers, not sure if taking an external dependency is out of the questions, but if isn't, you might be interested in referencing Microsoft.Toolkit when thre 6.1 update is out, and checking out the new Guard APIs (PR here), using those could simplify a lot of these conditional checks and remove some more code from markdig. They're already pretty optimized with respect to the codegen paths. Let me know if that helps! 😄

Why don't you also add the netstandard.2.1 target while you're at it? Have you checked if there's some overlap with the .NET Core 3.1 APIs you're using and .NET Standard 2.1, so that those new features could also be available in the portable version?

We can target netstandard2.1 as well, I added core 3.1 for the string.GetPinnableReference API, I don't know of an alternative to get ref char from a string (without going through Span / pinning).

Regarding the ThrowHelpers, not sure if taking an external dependency is out of the questions, but if isn't, you might be interested in referencing Microsoft.Toolkit

Considering the simplicity of those methods, I wouldn't add a dependency. See https://github.com/lunet-io/markdig/issues/296#issuecomment-461005586.

If you don't mind doing dirty tricks, there's this 🙈
https://github.com/Sergio0694/WindowsCommunityToolkit/blob/feature/high-perf-apis/Microsoft.Toolkit.HighPerformance/Extensions/StringExtensions.cs#L26-L31
This also happens to be faster than GetPinnableReference, since it can skip the bounds check, though technically it is only guaranteed to work on CoreCLR-like runtimes. But, since you're using that inside the #if NETCOREAPP3_1 directive anyway, it should be fine. I understand if you didn't want to have stuff like this in the repo for maintanability though 😅

Considering the simplicity of those methods, I wouldn't add a dependency. See #296 (comment).

Yup, figured as much, makes perfect sense! 👍

string's GetPinnableReference doesn't do any validation afaik (sharplab).

If you don't mind doing dirty tricks, there's this 🙈

👀 Thanks for sharing this. While I don't mind unsafe, I would rather have a preprocessor directive here in this case while we support targets without the API 😄

👀 Well thank you for sharing that too!
I was sure string.GetPinnableReference() was doing a check as well, but apparently I was wrong and that only happens for Span<T>.GetPinnableReference() (see here). That's a bit weird, I wonder why do those APIs act so differently (assuming it's not just sharplab.io acting up). I'll need to go find the source in CoreCLR, hopefully there will be some comments there that will shed some light on this! 😄
Well what do you know! Here's the source from CoreCLR:
https://github.com/dotnet/runtime/blob/master/src/libraries/System.Private.CoreLib/src/System/String.cs#L490-L495
This is great, thanks again! TIL 🍻

Small note on that string.GetPinnableReference(), while it's great to have it built-in, looks like it unfortunately suffers from a very similar codegen issue to https://github.com/dotnet/runtime/issues/12880.
In particular, we get this in x64:
public static ref char GetReference(this string text)
{
    return ref Unsafe.AsRef(in text.GetPinnableReference());
}

C.GetReference(System.String)
    L0000: mov eax, [rcx]
    L0002: lea rax, [rcx+0xc]
    L0006: ret

That first mov eax, [rcx] looks unnecessary, since the value in eax is immediately overwritten and never actually used. Also I'm not even sure what that was supposed to be in the first place, since it's apparently reading a 32bit value (in eax), from the address of... The method table pointer? The first char is located at [rcx + 0xc] since that's 8 bytes for the method table pointer (x64) + 4 bytes for the int representing the length of the string. Might be a codegen leftover for the bounds check 🤔
Anyway, hopefully it gets fixed in a future RyuJit update, thought it might be interesting to share 😄

`L0000: mov eax, [rcx]`

I should be a check for NullReferenceException so that access to text is still safe.

@xoofx Oh I see, thanks! Is that because if the address was null, trying to dereference [rcx+0xc] would have resulted in an access violation instead? As in, since that address would've been invalid but not exactlu null, so I'm not sure if the runtime is able to handle that the same? 🤔

@xoofx Oh I see, thanks! Is that because if the address was null, trying to dereference [rcx+0xc] would have resulted in an access violation instead? As in, since that address would've been invalid but not exactlu null, so I'm not sure if the runtime is able to handle that the same?

    L0002: lea rax, [rcx+0xc]

This instruction doesn't dereference anything but just calculate an offset. That means that if the first instruction mov eax, [rcx] was not emitted, you would return rax = 12 without any error in this function. So the problem is that you really want to know that text.GetPinnableReference() failed on a NullReferenceException not much later when accessing the pointer returned by GetReference
An access to the memory behind 0xc or 0x0 should generate a NullReferenceException, usually because it is in the first range of a 4K page fault that are mapped to generate such exception.

Ah, gotcha, that makes perfect sense. Also, good to know that memory accesses in that initial range would still properly trigger a NullReferenceException instead of a nasty AV, that's nice 😄
I guess if one really wanted to also skip that null check there, the solution would just be to either use the trick with the mapping type (sample here), or to just wait for .NET 5 (maybe?) to add some GetRawDataReference API or equivalent for the string type. Something like GetRawData (https://github.com/dotnet/runtime/issues/28001), which for now is only for arrays as far as I know (GetArrayDataReference).
Of course these are arguably just very minor details, but mostly I was just interested in learning more about this in general. Thanks for taking the time to chime in, really appreciate it! 😊

I've been benchmarking the difference between the 128 bool array and the 16 byte array.
It seems that the bool array is slightly faster (at least on my computer, .NET Core 3.1.3) on this benchmark:
public class Program
{
    private readonly string text = File.ReadAllText("spec.md");

    [Benchmark]
    public void TestMarkdig()
    {
        Markdown.Parse(text);
    }

    private static void Main(string[] args)
    {
        BenchmarkRunner.Run<Program>();
    }
}

LGTM

Could you re-merge back master to your branch @MihaZupan?, the CI was broken and I had to fix it on master.  You might have also to rerun the specs after merging. Thanks!

Thanks for fixing up the CI! I rebased and also added a commit to cross-target standard2.1 as Sergio suggested

Amazing job @MihaZupan !


chip-tool should randomize the node ids it assigns

Problem
chip-tool uses the same node id every time, which can cause problems due to multiple entities being commissioned with the same node id
Proposed Solution
Randomize node ids in chip-tool during pairing.

I guess right now the node id is not stored anywhere, just hardcoded....
I guess we could store it in the persistent storage, and reset for the pairing commands?  @vivien-apple


Added Phoenix Robotix to 'companies using Jest' section

Summary

Test plan

Codecov Report

Merging #3714 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3714   +/-   ##
=======================================
  Coverage   62.64%   62.64%
=======================================
  Files         183      183
  Lines        6709     6709
  Branches        6        6
=======================================
  Hits         4203     4203
  Misses       2503     2503
  Partials        3        3

Continue to review full report at Codecov.

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

Thank you. Unfortunately we are only adding companies right now that are well recognized in the industry, otherwise this list would grow to thousands of companies.


Resources insist as spawning as spices.

Already generated planets get overrridden while the game will not spawn them again


KAFKA-5609

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@rhauch @ewencp - Please review.

@guozhangwang @mjsax @rhauch @ewencp - This pull request is reference to https://github.com/apache/kafka/pull/4619 which has fix with implemented review comments provided by @rhauch. Please review.

I personally would consider using the RollingFileAppender with a sensible retention policy instead. Regrettably the DailyRollingFileAppender does not afford you a maxBackupIndex property which means you could potentially saturate disks.
https://logging.apache.org/log4j/1.2/apidocs/org/apache/log4j/RollingFileAppender.html

@rnpridgeon Awesome, input appreciated! Glad I asked for it before committing :) More concretely, maybe can we suggest what the sensible retention policy would be so @asutosh936 could update appropriately?

we talked reasonable default retention here https://github.com/apache/kafka/pull/1652
1G over 10 files is also reasonable for connect in my opinion.

+1

@rnpridgeon @ewencp @cotedm @rhauch @guozhangwang
All, Thank You for Your review and providing candid feedback. I am very sorry, have some issues with Git repo and the branch having this fix got deleted. Hence have made the necessary requested changes and have to create new Pull Request.
https://github.com/apache/kafka/pull/4753
Sorry Once again for any confusion.


why make-mirror documentation shows read only

I was trying to use make-mirror for replicating vault data across region.
Documentation on this mentions

Data under the destination cluster’s mirroring prefix should be treated as read only.

I think this is not true. We can as well write to the mirrored etcd cluster and infact in the same prefix.
This needs some clarification. I can correct it with a PR but needed someone to ack it before I proceed.
So suppose cluster-1 has secrets and mirroring tool copies the secrets to cluster-2 but this doesn't stop cluster-2 clients to write to the prefix. If they did write, I don't think it would affect the mirroring process or corrupt the data.
Ofcourse the new data added to the same prefix is not synced back to cluster-1.

@raoofm Yeah, it would still accept writes, technically. So, we need clarification on docs. Could you help improve? Thanks!

probably mention that the copy is uni-directional. changing the value on the mirrored cluster wont reflect the value back to the origin cluster.


function rand(seed) can not return fixed result.

function rand(seed) cannot return fixed result.
StarRocks > select rand(1);
+---------------------+
| rand(1)             |
+---------------------+
| 0.18517100684872181 |
+---------------------+
1 row in set (0.00 sec)

StarRocks > select rand(1);
+--------------------+
| rand(1)            |
+--------------------+
| 0.2376706186515989 |
+--------------------+
1 row in set (0.00 sec)

StarRocks > select rand(1);
+--------------------+
| rand(1)            |
+--------------------+
| 0.9819687331930045 |
+--------------------+
1 row in set (0.00 sec)

StarRocks > select rand(5);
+--------------------+
| rand(5)            |
+--------------------+
| 0.6355943365382248 |
+--------------------+
1 row in set (0.00 sec)

StarRocks > select rand(5);
+---------------------+
| rand(5)             |
+---------------------+
| 0.18223149391184523 |
+---------------------+
1 row in set (0.00 sec)

StarRocks > select rand(1);
+--------------------+
| rand(1)            |
+--------------------+
| 0.2523335304643378 |
+--------------------+
1 row in set (0.00 sec)

@wangsimo0 Hello, is there anyone working on resolving this issue? If not, I would like to try to resolve it.

The function is to return a random floating point number between 0 (inclusive) and 1 (exclusive), regardless of its parameters. If it is modified, I don't know if the final RP will be accepted; if it is accepted, it can be assigned to me to fix.


500px not all picture was loaded

Hello
RIPME loaded not all Picture from 500px with old Version 600 pics with new only 200 pics.
Thanks
Tim

I tried https://500px.com/fedorshmidt and it got 500 the first time, then all 705 pics the 2nd try.
If you have the logs when it stopped (like the last 5 lines of the log) that might help debug the issue.

I have tried a second download an a another day and it was all pics loaded.
Thanks for your help

Reopening because this is probably still an issue


feature: Add validation for NaN values in datablocks

Fixes #433

There is one code smell remaining, but I prefer to have the error message inside the tests, rather than as a variable somewhere else.


[BUG]

When you enter the startup command, you can normally display that the startup is successful, but when you open the address in the browser, the console will report an error on the picture, what is going on?

Logo not loaded, does not affect usage


[DO NOT MERGE YET] Upgrade to .NET 6 and Visual Studio 2022

Due to the need of this for my own project, just to get the data from the game that is in an automatable way, I have took the liberty of starting the upgrade of the projects to .NET 6 and Visual Studio 2022.
Iv seen requests for Linux support which is what I also need as soon as possible but, looking at the repository, it seemed to be a bit of a mess because it is trying to use the deprecated .NET Core 2.0 as well as .NET Framework 4.6.2 AND had ICSharpCode and ZSTD projects which are available on NuGet or could have been added as sub-module's.
Right now the unp4k and unforge build fine and supposedly work but, unforge I'm unable to use since I don't have any .dcb files and unp4ck seems to have incorrect header signatures.

Now, why .NET 6 and Visual Studio 2022?
Well, .NET 6 is the first LTS of the new .NET architecture, is cross-platform by default and is many times faster.
Visual Studio 2022 is the first x64 version of the IDE and is gaining traction very quickly, because of this, I see it being the preferred IDE version for a while to come.
The GUI project I might... Look into converting to WPF but, if this isnt wanted, ill try to make do with WinForms. It will be the last thing to be converted due to the amount of effort required.
Lastly, I'm taking the chance of making the project a bit more 'open source friendly' by removing preset directory and file paths in the project.
In a future PR, I may also add Github Actions support for both Windows x64 and Ubuntu x64 as well as turn unp4k and unforge into libraries which have executable wrapper projects so they can be used by the GUI project directly. Oh and dont forget, multi-threading, everyone loves multi-threading.

FYI, from memory I needed to customize ICsharp to add zstd support.

There was an old "Linux build" branch (at the pub so not looking it up now)
Sounds like you might have found it though, but just in case.

Looks like a few things in here probably break unp4k.gui, which is still extremely popular despite being unpolished, unfinished, and buggy.
Will need to make sure there's compatibility for these.

RE: GitHub Actions, I thought I'd done a  workflow for this project now  🤔

FYI, from memory I needed to customize ICsharp to add zstd support.

Right. It seems odd though but anyway, ill leave it out until everything else is done and ill see what can be done around that instead of copying entire repositories.

Ok, I just properly looked at the GUI project and it is WPF. Strange, VS was reporting it as WinForms. Oh well, that's in fact better.

It doesn't seem to automate though, I know nothing of AppVeyor nor LGTM. In fact half of the Actions are this PR weirdly enough.

.github/workflows/release.yaml will ensure PRs pulling into master can compile, and will automatically create a GitHub release and publish the binaries to it once the PR is merged.
AppVeyor was old - from before GitHub Actions time, can probably be removed at this stage - probably doesn't even run any more.
LGTM is just a linter I added to help with code quality, which admittedly was never great on this project, it evolved from a CLI that I hacked together on a train one weekend while researching the formats, into a functional (but not particularly graceful) project.
I'd always intended to do a rewrite with my learnings, but keep this one around as the "research" copy, as things were set up a little more raw which made it easier to track and trace changes when CIG inevitably release updates/changes to the project. Just never got around to it.
I definitely appreciate the interest in modernizing the project though 👌🏻

It doesn't seem to automate though, I know nothing of AppVeyor nor LGTM. In fact half of the Actions are this PR weirdly enough.

.github/workflows/release.yaml will ensure PRs pulling into master can compile, and will automatically create a GitHub release and publish the binaries to it once the PR is merged.
AppVeyor was old - from before GitHub Actions time, can probably be removed at this stage - probably doesn't even run any more.
LGTM is just a linter I added to help with code quality, which admittedly was never great on this project, it evolved from a CLI that I hacked together on a train one weekend while researching the formats, into a functional (but not particularly graceful) project.
I'd always intended to do a rewrite with my learnings, but keep this one around as the "research" copy, as things were set up a little more raw which made it easier to track and trace changes when CIG inevitably release updates/changes to the project. Just never got around to it.
I definitely appreciate the interest and help in modernizing the project though 👌🏻

Good good. Always glad to help! Ill remove AppVeyor and ill leave LGTM. Visual Studio 2022's Code Analyser is quite similar it seems and highly configurable so,

I have added in my own async logger. This will the stdout/in thread from locking the apps main thread.
Instead the apps main thread will send the Logger packages of info to write and this will be queued in another thread which will make the calls to stdout/in instead. Resulting in the app being far... faster, although, its not a problem but the Logger and app logic will become out of sync, I have a few ideas on how to kinda fix this but wont be related to any Logger code change.
Oh and it looks much cleaner now.

I have tried getting the modern version of SharpZipLib working with Zstd but it was a complete nope sadly. So, I created a fork/sub-module and put the code you had in it while also doing a little bit of modernising. Im not a pro on compression so, hopefully this can be figured out in future but, for now, it works.

To improve maintainability. I'm going to look into adding a way that the logger puts all errors into its own log file.

All that is left now is multi-threading for unp4ck then I can move onto unforge.

It is multithreaded now. Just in case its mis-concepted by some, this does not mean 100% CPU utilisation. The method its using creates new threads on different logical processors if the current targeted thread has a long enough block (Its being used by other things and enough to create performance drops).

Since MAUI is expected for Q2 2022, there will probably be a release without a GUI. MAUI is still very finicky with Visual Studio 2022. I for one cannot get the templates to show in New Project even though I meet all requirements and look at https://github.com/dotnet/maui/issues there seems to be many critical problems atm. Probably wont be able to start until around Christmas.

I have made a large amount of optimisation and error correction. Now all that is left besides the error logging options and forcefully overwriting is adding Linux support. Good start

.

@peter-dolkens This pull request has been completed! Although I cannot smelt the Game.dcb, it works, 126GB later... (It needs A LOT of free space) And I haven't found any other issue.


return UAA settings in configuration API

Coverage increased (+0.01%) to 58.086% when pulling 3518b8ac617a6314887f14bdf76575df20fabe45 on reasonerjt:uaa-config-api into 944fd1d97a4a7cd13356fc680f22641757530a08 on vmware:master.

Coverage increased (+0.02%) to 58.094% when pulling cae581979bbdaa81eaea342359812edeebcd5dbc on reasonerjt:uaa-config-api into 944fd1d97a4a7cd13356fc680f22641757530a08 on vmware:master.

Coverage increased (+0.05%) to 58.125% when pulling cae581979bbdaa81eaea342359812edeebcd5dbc on reasonerjt:uaa-config-api into 944fd1d97a4a7cd13356fc680f22641757530a08 on vmware:master.


Add reference to the ServiceNow Webhook

Hi!
As discussed with @simonpasquier in this issue, this PR add a reference to the ServiceNow Webhook.
It supports alerts grouping in one incident, updating an existing incident based on the group key, dynamic fields configurations and GO templating on fields values (from alert's annotations and labels).
Thanks!

Thanks!


FR: grpc client

IntelliJ has a built-in REST client: https://www.jetbrains.com/help/idea/http-client-in-product-code-editor.html
It would be nice to have a similar functionality for the gRPC.

This plugin is not supported anymore. Please switch to https://github.com/jvolkman/intellij-protobuf-editor/


Added option to print link rather than open browser

Great package!
I added a quick option to print the search link rather than open the browser window directly. I like to have the option to jump to the search results, but would prefer not to do so automatically, or to be prompted at every error. When it is set up this way, I can leave tracestack on all the time without disrupting my normal workflow.
I also looked into printing shortened links (for aesthetics only) using the pyshorteners package, but the API calls were much too slow to be worth it.
The changes are pretty trivial, but I am of course happy to discuss.
Many thanks,
Keith

This is super stale. Closing.


Added documentation for AI Data Mapper

Purpose
Adding documentation for the AI Data Mapping feature.
Pull request for feature

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


[tracker]: SFX editor

Currently the tracker is very much tailored towards 3-tracks music, but SFX also need to be designed in a game.
We could introduce an additional view (or even an additional editor if needed) to edit sound effects.
UI idea:

Tracked here https://github.com/latebit/latebit-studio-vscode/tree/slider-editor
There are a couple of UX challenges:

how should the volume be controlled? Maybe just an input below the slider would do?
how do we handle the case with many tracks? each gets set of sliders? maybe we prevent polyphony in this case?


Events table shows invalid date for "Stop"

In the events table (possibly also series), the stop column shows the text "Invalid date". Seems to happen if the "temporal" field is missing from the dublincore catalog. Probably nicer if the column would show nothing like in the old UI.

Seems like this got fixed at some point


Pick primary and secondary app categories for iOS and Android

Android:

@marcelomorgado For the Android content rating questionnaire, I was torn between Entertainment and Utilitiy, Productivity, Communication, or Other. I went with the latter.

@marcelomorgado For the Android content rating questionnaire, I was torn between Entertainment and Utilitiy, Productivity, Communication, or Other. I went with the latter.

Yes, I agree.

On Android I went with Lifestyle for the category. I was also considering Tools and Shopping.

This issue can be closed?

I still need to do this for iOS before non-beta app review


when link name/text is not present this is how it looks

when link name/text is not present:

this is how it looks:

Could you provide more details on how to replicate this? As the Text property in LinkData is required.

I've fixed an issue with [Required] attribute not registered correctly by metadata provider. See if 1.3.1 of the nuget fixes this issue.

I'm closing this because of inactivity and that the problem should be fixed.


Adding fixture Starway Zuni

Hi,
Please add this new fixture: a 6 channels dimmer for theater.
Sorry, but I didn't find a manual in english, but just in french: http://www.aevas-sono.com/evas/starway-zuni-p-1501.html
Thanks

Hi @pade, thanks for your contributing to QLC+. However, we usually don't have fixture definitions for dimmers since those can use the "Generic" fixture. Please see the discussion in #821 for details.

Hi @kripton,
I understand, thanks for your answer.


Update trakstar-tutorial.md

Value for Reply URL (Assertion Consumer Service URL) was missing
Value for Sign on URL was incorrect
Formatting and wording was improved for clarity

@shawn-trakstar : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


[AutoRelease] t2-servicefabricmanagedclusters-2024-01-31-31185(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/4928
Live test fail, detailed info is in pipeline log(search keyword FAILED)!!!
https://dev.azure.com/azure-sdk/internal/_build?definitionId=2422

issue link:https://github.com/Azure/sdk-release-request/issues/4928


Typing "/whisper username" and nothing else is visible to other users

This issue no longer occurs - tested with 2 accounts


add x-twitter social icon

resolve issue #39
I have confirmed that the icon is displayed by adding the following code to config.toml in the forked version.
socials = [
  { name = "x-twitter", url = "https://twitter.com/boykush315", icon = "x-twitter" },
]

forked version page: https://boykush.github.io/

Thanks! :)


qtip breaks jquery-ui selectmenu

When placing jquery-ui selectmenu inside a qtip title, the select stops working properly.
This is because the selector delegate() uses to handle close button clicks is generic enough to be used from other jquery-ui plugins.
I have changed selector to a more qtip-specific one.

Oops I should really read the bug before commenting heh. Fixed in the latest commit hopefully!

Checked last commit, it's still broken. Please have a look at the diff in my commit, I'm only changing 'ui-state-default' to the class that you already use elsewhere in the code to identify the close button, which is specific to qtip only, and seems to work perfectly.
ui-state-default is used in several other jquery ui plugins, shouldn't it be left alone?
Will put together a jsfiddle if you're still not convinced

Ok this is definitely fixed now :) My commit didn't get pushed upstream for some reason.

confirmed fixed, thanks!


[Bug]: Instance domain and postId are errors after changing

When I was trying to insert a toot and changed your demo(@rosnovsky@lounge.town:109860863149734322)
such as @AsyncX@mastodon.social:112054527697723909, It's unable to work properly then.
PS: When I changed username in your demo, they also can run.(@XXXXXXXX@lounge.town:109860863149734322)
Here is the err code:
Cannot read properties of null (reading 'accountUrl')
at Heading (file:///Users/ax/ALL-IN/repositories/DG/node_modules/astro-mastodon/dist/components/Heading.js:6:51)
    at Ib (/Users/ax/ALL-IN/repositories/DG/node_modules/react-dom/cjs/react-dom-server-legacy.node.production.min.js:67:254)
    at Jb (/Users/ax/ALL-IN/repositories/DG/node_modules/react-dom/cjs/react-dom-server-legacy.node.production.min.js:76:98)
    at W (/Users/ax/ALL-IN/repositories/DG/node_modules/react-dom/cjs/react-dom-server-legacy.node.production.min.js:73:345)
    at Ib (/Users/ax/ALL-IN/repositories/DG/node_modules/react-dom/cjs/react-dom-server-legacy.node.production.min.js:68:145)

@A5yncX thanks for reporting this, I'm looking into it now

Fixed! Love the photo 😻
The issue was that some instances (mastodon.social in this case) return empty [display_name](https://docs.joinmastodon.org/entities/Account/#display_name) field. It's not strictly required, but I haven't seen it being empty previously.

Thank you! 😁


Feature: Prevent weak hashing algorithms

This PR adds a check against starting Turnstile with a weak hashing algorithm. Because there is no relationship between the authorization scheme name and the actual hashing algorithm used to generate the signature it is possible to be in a state where, due to a misconfiguration, a weak hashing algorithm like MD4 could be used to generate the signature while a request could still contain the Rapid7-HMAC-V1-SHA256 scheme in the Authorization header and pass the check against an invalid authorization header.
signature := Base64<HMAC-MD4<#(key-secret), #(challenge-body)>>

Currently SHA256 and SHA512 are supported but additional hashing algorithms can be added in the future.
Resolves #26.

Something else we could do is force the Authorization header scheme algorithm to match the specified local hashing algorithm. Right now we just test that it matches Rapid7-HMAC-V1-SHA256 explicitly. My thought is to force it to match Rapid7-HMAC-V1-<ALGORITHM> where ALGORITHM is what's specified in Config.get('local:algorithm').

@fmitchell-r7 per our conversation yesterday, I'm going to add tests to cover the /properties endpoints. I have a few PRs related to updating linting rules and dependencies that I want to land before I do that work, so I'm going to merge this, submit those PRs, then write the tests.


Direction of 'oneway'

Using this:
 "source-layer": "roads",
  "type": "symbol",
  "layout": {
    "text-field": "{oneway}"
  }

The 'oneway' gives either 'yes' or none and sometimes 'no'. To make the oneway worthwhile it would be nice to know in which direction.

It should be in the same direction as the LineString. Generally, values of "oneway" are "yes", "no", and "reverse", meaning in the direction of the LineString, two-way and in the opposite direction to the LineString, although it's possible that some intermediate step resolves the last option by reversing the LineString itself.

Indeed, our service does reverse the LineString for values of reverse and -1.
https://github.com/mapzen/TileStache/blob/e29248b47caed25a61441b23b6addd053e9cc58d/TileStache/Goodies/VecTiles/transform.py#L291-L308

Documentation provided, no code changes needed.

Works. Great!


Rename container registry and test-infra pkg

change from the old name llm-instance-gateway to the new name gateway-api-inference-extension

the test-infra directory has been renamed.
The gcr registry not yet, we need to update the cloudbuild file when renamed: https://github.com/kubernetes-sigs/gateway-api-inference-extension/blob/main/cloudbuild.yaml

There is a new process that test-infra uses to update the repo, and they had to manually create our registry before. I've hounded them quite a bit over the last couple weeks, so letting this rest till the new year, hopefully we can just use the automated process to create the new registry.

It seems we need to open an issue in kubernetes/k8s.io repo: https://github.com/kubernetes/k8s.io/issues/7636


1.6.22 - Promise, IE note, etc.

In that the 1.6.22 release uses a Promise object, suggest modifying the browser support section to indicate that a polyfill will be required for IE, unless I misunderstand things.

@bazineta The Web Font Loader will detect support for promises, and if it isn't present it'll fall back on another method of detecting font loads. So there is no need to include a Promise polyfill.


Add expectations

Requires Wollok 1.3 support - which is not yet released

See uqbar-project/wollok#419


Accessibility Bug Fixes for MHA web/windows

1691247- Changed color contrast for Analyze header button on web MHA
1691683- Added Media query for Aquatic mode radio button selection issue
1697464- Fixed Luminosity contrast ratio of 'Radio buttons' and 'Check box',
1696469 - added title to the buttons for narrator to narrate name information of the buttons.

@microsoft-github-policy-service agree company="Microsoft"


Hidden config file on Windows breaks the application.

This is an odd one. Marking the .sqlectron.json file as hidden in Windows prevents the GUI from reading/parsing/displaying it's saved connections, and from creating new connections. I'm not sure where the error is happening, and nothing is logged to the console in the dev tools. I also tried poking around with Node a little to see if the fs.stat signature changes or returns an error for hidden files but I didn't notice anything. I'm using Sqlectron 1.16.0 on Windows 10.

Yeah that is weird. I will try to check it out once I have some free time. Thanks for reporting the issue.

I thought I would have time to investigate it. But I don't :( If someone else has interest in doing that. Please do.


The same model works successfully for small tracks but takendown for the full track

Hi,
I submit the same model (https://www.dynabench.org/models/379) to both small tracks and the full track to check whether there are some wrong settings. And it works for two small track but failed for the full track. Could you check this for me?
By the way, I even decrease the beam size to 2 and increase the batch size to 64 for the full track.

This was caused by time limit or memory limit.


Troubles after Telegram update on November 21, 2016

Hi guys, we got troubles after last Telegram update with this:

Your bot will now get updates about posts in channels. Added new fields channel_post and edited_channel_post to Update.

Here is an exception we were getting after update:
[2016-11-23 23:59:59] prod.ERROR: Symfony\Component\Debug\Exception\FatalThrowableError: Call to a member function getFrom() on null in /app/Services/TelegramCommands/SystemCommands/GenericmessageCommand.php:49
We fixed it with the following code, could you please add fixes to repository?
public function messenger() { if (array_key_exists("edited_channel_post", \Illuminate\Support\Facades\Input::get())) return; if (array_key_exists("channel_post", \Illuminate\Support\Facades\Input::get())) return; $this->api->handle(); }

It's because of the latest API update, will be fixed
Look here https://github.com/akalongman/php-telegram-bot/pull/333 for commit with a fix

Thanks!


perlPackages.TestMockHTTPTiny: init at 0.002

Description of changes
https://metacpan.org/pod/Test::Mock::HTTP::Tiny
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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 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

[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build perlPackages.TestMockHTTPTiny


app-specific logger messages in search_queries_controller

some log messages say FREEREG instead of app-specific (missing_document for example)

on test2

deployed to production, ready for user test

@Vino-S to check.


Web app create results in error

usage error: must specify --runtime | --deployment-container-image-name

@y-luis : Thanks for your contribution to the Azure documentation! The author, @rloutlaw, has been notified to review your proposed change.

#sign-off , this correction is on point.
Thanks!


Project metodology select

This need be changed by something most easily to configure in model

RUP are not work right.


Add new fan speed mapping for V2 Roborock Vacuum

Description:
Update fan speeds to reflect values that Mi Home app is sending to the vacuum. Tested on Roborock S50, this version correctly reflects values set by the app, and the app correctly highlights speeds set by this component.
To keep backwards compatibility, added a new boolean setting to yaml config - new_fan_speeds which defaults to the old fan speed mapping. Maybe someone can suggest shorter and more clear name for new variables.
Related issue (if applicable): fixes #17695
Link to an earlier attempt to fix this: #17090
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#8233
Example entry for configuration.yaml (if applicable):
vacuum:
- platform: xiaomi_miio
  host: <...>
  token: <...>
  new_fan_speeds: true

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

@syssi @poma Checking in on this + Can you fix the merge conflict?

This PR has gone stale.  Please update the code and open a new PR if you wish to continue with this contribution.


expose SASL property to the logging subsystems

Affects Version(s): 2.1.6.RELEASE

Enhancement
It would be interesting to enforce the "EXTERNAL" SASL mechanism when a client is using a keystore with a client cert to authenticate against a RabbitMQ server.
See this S.O. discussion for more details: https://stackoverflow.com/questions/42851250/saslconfig-using-spring-rabbitmq-not-working
This behavior should exist by default on the logging subsystems (log4j & logback) or be customizable over a new attribute (~"SaslConfig"). See this class for log4j2: https://github.com/spring-projects/spring-amqp/blob/master/spring-rabbit/src/main/java/org/springframework/amqp/rabbit/log4j2/AmqpAppender.java

see also https://github.com/spring-projects/spring-boot/issues/6719

There are many properties on the underlying RabbitMQ connection factory and the factory there is not created by the Spring factory bean (it's created by the appender itself, which is outside of the Spring context).
Since there are so many properties, I propose adding
protected void configureRabbitConnectionFactory(ConnectionFactory cf) {
}

so users can subclass the Appenders and perform additional configuration as necessary.
I believe that would suit your needs?

I see there is on the RabbitConnectionFactoryBean:
/**
	 * @param saslConfig the sasl config.
	 * @see com.rabbitmq.client.ConnectionFactory#setSaslConfig(com.rabbitmq.client.SaslConfig)
	 */
	public void setSaslConfig(SaslConfig saslConfig) {
		this.connectionFactory.setSaslConfig(saslConfig);
	}

and this one available for access from the overridden configureRabbitConnectionFactory(RabbitConnectionFactoryBean).
So, I think we are covered already and no need in a new method. Only the problem that Log4J appenders are based on @PluginFactory static method to be created. So, provide some customization into an AmqpAppender we need to pay some effort into an extension. essentially fully custom AmqpAppender .
Any other thoughts how to overcome static restrictions?

I didn't notice that we already have that method.
Looks like having protected methods there is useless though due to the static creation.
Given that we already support other SSL props, we should just go ahead and add this one.
Given the static class, and it already has SSL support, we should just go ahead.

Yes, indeed, extending the AMQPAppender is one solution.
I would simply go for this kind of change: https://github.com/bcollard/spring-amqp/commit/141dcda6b74a678e2b80dc0caa0e24b9d822debc
But I don't know about side effects here.

So quick, so cool! thanks.


Issue with Player Join & Player Leave Regex - Arma Reforger

There is an issue with the Player Join and Player Leave Regex for Arma Reforger. It is not counting joins and leaves—a possible regex change due to the update.
I have tested the below settings for join and leave, and they seem to be working if we update this. If you have any ideas, please let me know.
^[\d:\s]*Player #\d+ (?.+) (\d+.\d+.\d+.\d+:\d+) connected$
^[\d:\s]*Player #\d+ (?.+) disconnected$
Console Logs:
Player #0 SXon (IP:PORT GOES HERE) connected
Player #0 SXon disconnected
Thank you :)

Thanks. Is that logging the same on both windows and Linux servers?
It would be helpful to have the full logs around when players join and leave rather than just single lines. Can you share?

I can't speak for Windows as I don't use it in my environment - this is for Linux running the instance in Docker.
Here are the full logs.
DEFAULT      : BattlEye Server: RCon admin #0 (IP:PORT) logged in SCRIPT       : ScenarioFramework: -> LayerTask: SlotTask SlotDestroy_19 - generating task Destroy vehicle. Description: ScenarioFramework: -> LayerTask: SlotTask SlotDestroy_19 - generating task Destroy vehicle. Description: 20 Command Truck RPL          : ServerImpl event: authenticating (identity=0x00000000, address=IP:PORT) RPL          : ServerImpl event: connected (identity=0x00000000) WORLD        : UpdateEntities Player #0 SamXon (IP:PORT) connected Player #0 SamXon - BE GUID: 534aefe2ee481dc835b2729baca66e07 Connected to BE Master WORLD        : UpdateEntities  RPL          : rpl::Pip::ProcessNetToGame   SCRIPT    (W): Ban Settings could not be loaded from BE Storage: Relying on default config WORLD        : UpdateEntities  WORLD        : Frame
WORLD        : UpdateEntities  WORLD        : Frame   ENTITY       : SpawnEntityPrefab @"{23ADBBC31B6A3DC6}Prefabs/Characters/Factions/OPFOR/USSR_Army/Character_USSR_AR.et"    WORLD        : InitEntities 'world'     ENTITY       : Init entity @"ENTITY:4611686018427390137" ('SCR_ChimeraCharacter','Assets/Characters/Basebody/Basebody_Male_01.xob') at <5778.045898 217.250504 7060.766113> @"{23ADBBC31B6A3DC6}Prefabs/Characters/Factions/OPFOR/USSR_Army/Character_USSR_AR.et"      INVENTORY (E): Couldn't insert item @"ENTITY:4611686018427390173" ('GenericEntity','Assets/Weapons/Magazines/RPK74/Magazine_45rnd_RPK74.xob') at <0.000000 0.000000 0.000000> @"{D78C667F59829717}Prefabs/Weapons/Magazines/Magazine_545x39_RPK_45rnd_4Ball_1Tracer.et" into storage @"ENTITY:4611686018427390152" ('GenericEntity','Assets/Items/Equipment/Accessories/Pouch_Soviet_45rnd_RPK74/Pouch_Soviet_45rnd_RPK74.xob') at <5777.917480 218.366180 7060.869141> @"{B6EEF03975F21E4E}Prefabs/Items/Equipment/Accessories/Pouch_Soviet_45rnd_RPK74/Pouch_Soviet_45rnd_RPK74.et" while spawning initial items inside the storage.      INVENTORY (E): Couldn't insert item @"ENTITY:4611686018427390174" ('GenericEntity','Assets/Weapons/Magazines/RPK74/Magazine_45rnd_RPK74.xob') at <0.000000 0.000000 0.000000> @"{D78C667F59829717}Prefabs/Weapons/Magazines/Magazine_545x39_RPK_45rnd_4Ball_1Tracer.et" into storage @"ENTITY:4611686018427390152" ('GenericEntity','Assets/Items/Equipment/Accessories/Pouch_Soviet_45rnd_RPK74/Pouch_Soviet_45rnd_RPK74.xob') at <5777.917480 218.366180 7060.869141> @"{B6EEF03975F21E4E}Prefabs/Items/Equipment/Accessories/Pouch_Soviet_45rnd_RPK74/Pouch_Soviet_45rnd_RPK74.et" while spawning initial items inside the storage.      INVENTORY (E): Couldn't insert item @"ENTITY:4611686018427390175" ('GenericEntity','Assets/Weapons/Magazines/RPK74/Magazine_45rnd_RPK74.xob') at <0.000000 0.000000 0.000000> @"{D78C667F59829717}Prefabs/Weapons/Magazines/Magazine_545x39_RPK_45rnd_4Ball_1Tracer.et" into storage @"ENTITY:4611686018427390152" ('GenericEntity','Assets/Items/Equipment/Accessories/Pouch_Soviet_45rnd_RPK74/Pouch_Soviet_45rnd_RPK74.xob') at <5777.917480 218.366180 7060.869141> @"{B6EEF03975F21E4E}Prefabs/Items/Equipment/Accessories/Pouch_Soviet_45rnd_RPK74/Pouch_Soviet_45rnd_RPK74.et" while spawning initial items inside the storage.      INVENTORY (E): Couldn't insert item @"ENTITY:4611686018427390176" ('GenericEntity','Assets/Weapons/Magazines/RPK74/Magazine_45rnd_RPK74.xob') at <0.000000 0.000000 0.000000> @"{D78C667F59829717}Prefabs/Weapons/Magazines/Magazine_545x39_RPK_45rnd_4Ball_1Tracer.et" into storage @"ENTITY:4611686018427390152" ('GenericEntity','Assets/Items/Equipment/Accessories/Pouch_Soviet_45rnd_RPK74/Pouch_Soviet_45rnd_RPK74.xob') at <5777.917480 218.366180 7060.869141> @"{B6EEF03975F21E4E}Prefabs/Items/Equipment/Accessories/Pouch_Soviet_45rnd_RPK74/Pouch_Soviet_45rnd_RPK74.et" while spawning initial items inside the storage. WORLD        : UpdateEntities  WORLD        : Frame   DEFAULT   (W): DamageManager.HandleDamage - Struck HitZone doesn't belong to this damage manager! Prefab: @"ENTITY:813922" ('GenericEntity','Assets/Structures/Signs/Warnings/Sign_RectangleUp_A.xob') at <5756.173340 219.993515 7077.151855> RPL       (E): RpcError: Calling a RPC from an unregistered item! itemType='script::Game::SCR_BudgetEditorComponent', rpc='OnEntityCoreBudgetUpdatedOwner' Player #0 SamXon disconnected

Can you put that in three backticks at start and end?

Can you put that in three backticks at start and end?

See if that is better. My apologies.

Thanks. That's from the AMP logs rather than just the console?

Thanks. That's from the AMP logs rather than just the console?

Yes sir. Those are the amplogs in the instance. Would you like me to copy and paste straight from the console, or does that give you everything you need?

The amp logs is what is important, thanks


New Hover Design for Navbar in Landing page

Mention Here if you want to be assigned with this issue!!.
Make sure to have different animation for hover on buttons.

Hello, I would like to work on this issue,
Can you assign this to me?

Hi, If anyone is not working on this issue, Can you please assign it to me?


Update notebook cell paste behavior

Fixes #564.  Cells are pasted below the active cell and the last pasted cell is activated, matching the classic notebook behavior.

👍


Use version property from package.json instead of tauri.conf.json

Is your feature request related to a problem? Please describe.
Is there a way to use the version property inside the package,json instead of tauri.conf.json? I'm using Changesets for versioning my packages in CI. And of cause changesets has no idea that there is an other version property to update during release inside tauri.conf.json. This should also apply to similar packages like Semantic Release.
Describe the solution you'd like
The version number should be used from the package.json file, or at least there should be an option to do so (to make this a non breaking change).
Describe alternatives you've considered
Manually updating this property isn't that great of an experience. Using a custom action to copy the version from package.json to tauri.conf.json and commiting that neither. That would lead to the version bump being made after the git tag.

I'll implement this. I think it is a good addition.


[kong] Release 1.12.0 to next

What this PR does / why we need it:
Releases 1.12 to next. Includes several additional changes:

Bump the default Enterprise tags to the latest 2.1 patch, <IP_ADDRESS>.
Increase the default worker count to 2.
Handle deprecation warnings via a template to simplify and improve formatting.

Which issue this PR fixes

fixes #227
fixes #144

Special notes for your reviewer:
Re conversations about worker count increase and memory usage from planning:

By default, the chart does not apply memory limits to Kong deployments. However, it did include commented example limits with an unreasonably low default limit (128MB). From personal experience, this limit is too low for any deployment (even single-worker deployments), and would cause issues if applied. I've increased the commented example to 256MB, which is much more reasonable, though that's essentially a minimum lower bound for Kong to function at all rather than what I'd expect for most actual deployments. Non-demo/test environments that actually handle traffic will likely start around 1GB, and will usually have even higher limits.
There was a brief interlude where additional workers allocated resulted in unexpected higher memory use due to removal of limits elsewhere. AFAIK we've since re-introduced those limits (https://github.com/Kong/kong/pull/5864), and increasing worker count no longer causes the consumption issues we observed in earlier 2.x releases, so I don't expect the chart change to cause issues.

We still want to explore resource usage guidelines further, but the minimums proposed here should be reasonable.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] PR is based off the current tip of the next branch and targets next, not main
[x] Title of the PR and commit headers start with chart name (e.g. [kong])

LGTM but:

this does not fix #227 yet - after this PR, the checkbox below remains unsatisfied:

[ ] Update the default value to 2 in Kong Ingress Controller's repository

for future PRs, please avoid mix features and release chores in one PR

Please merge without squashing, too.

Part of #232.


from_iris converts dask array into numpy array

Code Sample, a copy-pastable example if possible
import xarray as xr
import numpy as np
import dask.array as da
import iris

# xarray (dask) -> iris (dask) -> xarray (numpy)
darr = xr.DataArray(
    da.from_array(np.random.rand(4, 5), chunks=4), dims=['x', 'y'],
    coords=dict(x=[10, 20, 30, 40], y=range(5)))

print(type(darr.data)) # <class 'dask.array.core.Array'>

cube = darr.to_iris()
print(type(cube.core_data())) # <class 'dask.array.core.Array'>

darr2 = xr.DataArray.from_iris(cube)
print(type(darr2.data)) # <class 'numpy.ndarray'>

Problem description
After converting an iris Cube into an xarray DataArray the core array object is no longer lazy.
The dask array is not immediately realised - the .from_iris() call completes instantly and plotting slices works as expected. I only noticed the issue when I tried to convert a large dataset back into a Cube, at which point everything ground to a halt.
(The ability to convert between libraries is very cool - thanks for developing this)
Expected Output
Array type to stay the same, data not to be computed if not needed.
Output of xr.show_versions()

INSTALLED VERSIONS
------------------
commit: None
python: 3.6.3.final.0
python-bits: 64
OS: Darwin
OS-release: 16.7.0
machine: x86_64
processor: i386
byteorder: little
LC_ALL: None
LANG: en_GB.UTF-8
LOCALE: en_GB.UTF-8
xarray: 0.10.2
pandas: 0.21.0
numpy: 1.13.3
scipy: 1.0.0
netCDF4: 1.3.1
h5netcdf: 0.5.0
h5py: 2.7.1
Nio: None
zarr: None
bottleneck: 1.2.1
cyordereddict: None
dask: 0.17.1
distributed: 1.21.3
matplotlib: 2.1.0
cartopy: 0.15.1
seaborn: 0.8.1
setuptools: 36.5.0.post20170921
pip: 9.0.1
conda: 4.4.6
pytest: None
IPython: 6.2.1
sphinx: None

Thanks for the report and the clear example!
I'm marking this as a bug for now as this is definitely unwanted behavior. I'll attempt to reproduce it later when I have time...

I think this may be fixed by the recently merged https://github.com/pydata/xarray/pull/2047. Can you try out the development version of xarray and see if that fixes things?

I can confirm that this seems to be fixed by #2047. When I run without that commit, I see:
<class 'dask.array.core.Array'>
<class 'dask.array.core.Array'>
<class 'numpy.ndarray'>

But now I see:
<class 'dask.array.core.Array'>
<class 'dask.array.core.Array'>
<class 'dask.array.core.Array'>

However, we should add a regression test to make sure this doesn't break again. Any interest in putting that together? The existing Iris/xarray tests can be found here:
https://github.com/pydata/xarray/blob/a9d1f3a36229636f0d519eb36a8d4a7c91f6e1cd/xarray/tests/test_dataarray.py#L2900

Thanks, I'll endeavour to add a regression test.
The conversion works fine now with the threaded scheduler or with a local distributed scheduler, but something fails when using a remote distributed scheduler. Computing the data after converting (I think in either direction) causes all of the distributed workers to fall over.
I'd guess that this is an issue with the distributed scheduler (I'm using dask-kubernetes on pangeo), I'll try to dig deeper.

import xarray as xr
import numpy as np
import dask.array as da
import dask
from dask.distributed import Client
import iris

def test():
    # xarray (dask) -> iris (dask) -> xarray (numpy)
    darr = xr.DataArray(
        da.from_array(np.random.rand(4, 5), chunks=4), dims=['x', 'y'],
        coords=dict(x=[10, 20, 30, 40], y=range(5)))

    print(type(darr.data)) # <class 'dask.array.core.Array'>
    darr.data.compute() # works

    cube = darr.to_iris()
    print(type(cube.core_data())) # <class 'dask.array.core.Array'>
    cube.core_data().compute() # this fails (hangs) with distributed scheduler

    darr2 = xr.DataArray.from_iris(cube)
    print(type(darr2.data))
    darr2.data.compute() # this fails (hangs) with distributed scheduler (if you comment out the failing line above)

print('threaded')
with dask.set_options(get=dask.threaded.get):
    test()
print('success\n')

print('local distributed')
client = Client(processes=True)
test()
print('success\n')

print('remote distributed')
client = Client(cluster.scheduler_address) # remote cluster (dask-kubernetes)
test() # this will cause all workers to fail and then hang forever
print('success')

@AlexHilson - iris is not part of the standard worker image on pangeo. My first guess is that you either have a version mismatch or a missing library in the workers. If things are working on the threaded scheduler / local cluster, I would think the issue w/ iris and dask has been resolved.

Perfect, thanks, missing library 😄 I'm on the Met Office Pangeo so it was xarray missing from the workers.
It's a side effect I wasn't thinking about, for the converted object to still require the original library. In this example the dask array is just wrapping a numpy function but I guess in real life the dask graph would would contain a bunch of library specific functions.

Re-running my original task (converting a large cube into xarray and back) throws up a related issue. I haven't had a chance to investigate this properly but it looks like the masking operation applied by xarray breaks the ability to selectively compute indexes of the converted cube. If you try and compute a slice the entire array will be realised.
This is different from the original issue, but does have a similar effect of making large objects not realistically convertible. I'm not going to be able to properly investigate this for a while, so if you'd like me to re-raise later as a new issue that's fine with me.
Not sure how to replicate this with dummy data, here are the steps I'm taking.
Given an input cube of shape (31, 56, 12, 3, 600, 800)
Try three ways of passing data through xarray and creating a (600, 800) slice.
slice1 = xr.convert.from_iris(cube).to_iris()[0,0,0,0]
slice2 = xr.convert.from_iris(cube)[0,0,0,0].to_iris()
slice3 = xr.convert.from_iris(cube[0,0,0,0]).to_iris()

Slice 2 + 3 will compute properly, but if you try and compute the core data of slice 1 dask will attempt to realise all the entire original array. So indexing pre calling to_iris() works, leading me to suspect the masking stuff that to_iris() seems to be doing. As an aside I was a little surprised to pass a regular array in and get a masked array back, but I don't know anything else about this topic so have no objection or opinion on that behaviour.
The task graphs are:
￼slice 1

slice2

slice3

Apologies I haven't had much more of a chance to look at this, but I do have a reproducible example. This is a bit clunky but hopefully it shows that you can no longer partially compute the dask array once to_iris() has been used.
import xarray as xr
import numpy as np
import dask.array as da
import iris
from dask import delayed

@delayed
def flakey_array(i):
    if i == 1:
        raise ValueError('I computed slice {}!'.format(i))
    else:
        return np.random.rand(1)

# array[0] should always work, array[1] should always fail
dask_arrays = da.stack([da.from_delayed(flakey_array(i), dtype=np.float32, shape=[1]) for i in [0, 1]])

darr = xr.DataArray(
    dask_arrays)

cube = iris.cube.Cube(
    dask_arrays)

try:
    _ = darr[0].data.compute()
    print('success (DataArray)')
except ValuError as e:
    print('failure (DataArray):', e)

try:
    _ = cube[0].core_data().compute()
    print('success (Cube)')
except ValueError as e:
    print('failure (Cube):', e)

cube_from_darr = darr.to_iris()
try:
    _ = cube_from_darr[0].core_data().compute()
    print('success (DataArray -> Cube)')
except ValueError as e:
    print('failure (DataArray -> Cube):', e)

darr_from_cube = xr.DataArray.from_iris(cube)
try:
    _ = darr_from_cube[0].data.compute()
    print('success (Cube -> DataArray)')
except ValueError as e:
    print('failure (Cube -> DataArray):', e)

darr_from_cube_from_darr = xr.DataArray.from_iris(cube_from_darr)
try:
    _ = darr_from_cube_from_darr[0].data.compute()
    print('success (DataArray -> Cube -> DataArray)')
except ValueError as e:
    print('failure (DataArray -> Cube -> DataArray):', e)

results:
success (DataArray)
success (Cube)
failure (DataArray -> Cube): I computed slice 1!
success (Cube -> DataArray)
failure (DataArray -> Cube -> DataArray): I computed slice 1!

 Thanks for the clear report!
Changing dataarray to dataarray.data on these two lines in to_iris() appears to fix the issue (the example code runs without errors):
https://github.com/pydata/xarray/blob/b9f40cc1da9c45b3dd33a3434b69c3d8fce57138/xarray/convert.py#L127-L129
@AlexHilson any interest in putting this into a PR? :)

Yep I'll PR this along with a test for the earlier issue.
Interesting that the masked behaviour is different, might be indicative of a deeper issue?
# docstrings for both of these methods say they expect an array like.

# works with DataArray
print(da.from_array(darr, chunks=1)[0].compute())

# fails with DataArray
print(da.ma.masked_array(darr, chunks=1)[0].compute())

da.ma.masked_invalid(darr) results in nested dask arrays, with the outer dask array holding an xarray.DataArray holding a dask array.
One way to check this would be to verify that original dask array's tasks ends up directly in the graph of the final dask array, e.g., assert set(darr.data.dask) <= set(darr.to_iris().core_data().dask).

I see - so if I understand correctly we end up in dask.array.core.asanyarray, which leads to return from_array(a, chunks=a.shape, getitem=getter_inline, asarray=False) (where a is our DataArray) and then the DataArray acts as a bottleneck in the task graph.
My instinct is to pull out this block from to_iris into a new to_masked function (or a private function?) that will explicitly maintain chunksize - feels like that function could be tested more cleanly than one that passes data through Iris. Or if you'd prefer the task graph test approach I can add that instead.

    # Create the right type of masked array (should be easier after #1769)
    if isinstance(dataarray.data, dask_array_type):
        from dask.array import ma as dask_ma
        # masked_invalid can be applied directly to the dataarray, but
        # results in a bad chunksize. So instead we use the internal
        # dask array (#2046)
        masked_data = dask_ma.masked_invalid(dataarray.data)
    else:
        masked_data = np.ma.masked_invalid(dataarray)

@AlexHilson Sure, it would make sure to put a masked_invalid function in https://github.com/pydata/xarray/blob/master/xarray/core/duck_array_ops.py


Print Signal Transformation Details in the logs

The default log does not contain this info.
The user should be able to compute the forecast by hand based on the logs.
JSON outputs also should be complete.

Closing


Update get-started.md - Minor grammer.

This pull request includes several improvements to the well-architected/ai/get-started.md file, focusing on refining the language for clarity and conciseness. The most important changes include adjustments to descriptions of AI workloads, considerations for choosing AI models, and guidelines for adopting new AI technologies.
Language and clarity improvements:

Corrected grammar and punctuation in the list of AI workloads to improve readability.
Simplified the description of custom and prebuilt models to make the expertise requirement clearer.
Refined the explanation of evaluating new AI technologies to emphasize user experience over unnecessary complexity.
Clarified the ethical requirements section to focus on determining the ethical suitability of AI use cases.

@lukemurraynz,
Thanks for bringing this article to our attention. Unfortunately, copy edits need to be done internally to ensure that articles follow Microsoft style and formatting guidelines. We'll close this PR and submit the file for an internal copy edit.
#please-close


Warnings when running pip.import more than once

import { pip } from<EMAIL_ADDRESS>await pip.import('numpy');

Second and consecutive runs of this code give me the following warnings:
WARNING: Target directory /Users/admin/Library/Caches/deno/plug/pip/numpy already exists. Specify --upgrade to force replacement.
WARNING: Target directory /Users/admin/Library/Caches/deno/plug/pip/numpy-1.26.0.dist-info already exists. Specify --upgrade to force replacement.
WARNING: Target directory /Users/admin/Library/Caches/deno/plug/pip/bin already exists. Specify --upgrade to force replacement.

My version of deno is 1.37.1

This is an issue with how we plug into pips cache and how it will try to install already installed packages anyways. I am not knowledgeable enough with pip to really know if there is a better way or a solution to this. If there is a solution it is the same in the console but I am unaware of any flag which solves this.


Question about using btrfs-optin-persistence using systemd service

I've been testing out using Systemd as the initrd system on NixOS (which at best seems to be in an alpha state...) and I've run into some issues/questions with the btrfs impermanence script that I'm hoping to resolve.
in btrfs-optin-persistence.nix, wiping the root partition is now done through a systemd service rather using the postDeviceCommands value for the custom NixOS initrd. That all makes sense to me. What I am wondering is how the service dependencies make sense.
requires = [ "initrd-root-device.target" ];
before = [ "sysroot.mount" ];
wantedBy = [ "initrd-root-fs.target" ];

When I tried this order, I encountered an error where initrd-btrfs-root-wipe would be run before the actual decryption was done for the root partion and therefore the service would fail.
I was able to resolve this, but the next issue was that sysroot.mount would activate while initrd-btrfs-root-wipe was still running and therefore fail again. To resolve these issues I ended up with something like this:
after = [<EMAIL_ADDRESS>];
before = [ "sysroot.mount" ];
wantedBy = [ "sysroot.mount" ];

To be honest at best I only slightly understand the systemd initrd boot structure/order. I've been reading up on how systemd handles boot, but this solution is more just a result of extensive trial-and-error testing than a concrete understanding of the system.
TLDR: I was wondering if you could explain to me your logic for the structure of the initrd-btrfs-root-wipe service and how it works on your end. I'm just trying to figure out the ideal way this should be handled as all other references to the impermanence module are based on NixOS' existing custom initrd.
PS: Thank you for having this as well as the nix-starter-config repository. They are by far the most helpful references I've encountered through my NixOS journey and I cannot say for certain that I would have stayed on NixOS if I didn't find these repos. Great work on finding out all of this information and then releasing it to the public.

Sorry for the delay (life's been crazy)!

What I am wondering is how the service dependencies make sense.

I'm actually not using systemd in initrd for now, I've tried my best to make that unit work, but I always get the dependency cycles or race conditions :/
The main benefit it offers (for me, at least) is having plymouth start up before the LUKS password prompt, but it seems it's still missing a few features.
A fellow nixer (Tom) e-mailed me with a clever solution for this issue:
let
   rootfsDevice = "/dev/disk/by-label/${hostname}";
   wipeScript = ''
     counter=0

     # systemd initrd offers no way to hook in between
     # 'root device is ready' and 'mount the root device at /sysroot'.
     # Instead, lets poll for the root device.
     # https://github.com/systemd/systemd/issues/24904

     echo wipe: Waiting for ${rootfsDevice}
     while [ ! -e ${rootfsDevice} ]; do
       sleep 0.1
       counter=$((counter + 1))
       if [ $counter -ge 300 ]; then
           echo wipe: Timed out waiting for ${rootfsDevice}
           exit
       fi
     done
     echo wipe: Found ${rootfsDevice}. Wiping ephemeral.

     mkdir -p /btrfs
     mount -o subvol=/ ${rootfsDevice} /btrfs

     echo "Cleaning subvolume"
     btrfs subvolume list -o /btrfs/root | cut -f9 -d ' ' |
       while read subvolume; do
         btrfs subvolume delete "/btrfs/$subvolume"
       done && btrfs subvolume delete /btrfs/root

     echo "Restoring blank subvolume"
     btrfs subvolume snapshot /btrfs/root-blank /btrfs/root

     umount /btrfs
     rm -d /btrfs
   '';
in {
   options = {
     /* truncated */
   };
   config = {
     boot.initrd = {
       supportedFilesystems = [ "btrfs" ];
       systemd = {
         enable = true;
         emergencyAccess = true;
         initrdBin = with pkgs; [ coreutils btrfs-progs ];
         services.initrd-btrfs-root-wipe = {
           description = "Wipe ephemeral btrfs root";
           script = wipeScript;
           serviceConfig.Type = "oneshot";
           unitConfig.DefaultDependencies = "no";

           after = [ "cryptsetup.target" "initrd-root-device.target" ];
           before = [ "sysroot.mount" ];
           wantedBy = [ "sysroot.mount" ];
         };
       };
     };
   };

     /* truncated */
}

Thank you for having this as well as the nix-starter-config repository

Thanks a lot for your kind words! I'm very glad to be able to help out :)


Build error with dayjs and element-plus

Describe the bug
vite-ssg build error with dayjs and element-plus
vite.config.ts
// fork by https://github.com/antfu/vitesse
import { ElementPlusResolver } from 'unplugin-vue-components/resolvers'
// ...
   Components({
      // allow auto load markdown components under `./src/components/`
      extensions: ['vue', 'md'],
      // allow auto import and register components used in markdown
      include: [/\.vue$/, /\.vue\?vue/, /\.md$/],
      dts: 'src/components.d.ts',
      resolvers: [
        ElementPlusResolver({
          importStyle: 'sass',
          directives: true,
          version: '1.3.0-beta.10',
        }),
      ],
    }),

  ssgOptions: {
    script: 'async',
    formatting: 'minify',
    onFinished() { generateSitemap() },
  },
// ...

plugin/dayjs

import 'dayjs/locale/zh-cn'
import type{ ConfigType, ManipulateType } from 'dayjs'
import isBetween from 'dayjs/plugin/isBetween'

dayjs.locale('zh-cn')
dayjs.extend(isBetween)

export default dayjs

executive build command
> vite-ssg build
...
An internal error occurred.
[vite-ssg] Please report an issue, if none already exists: https://github.com/antfu/vite-ssg/issues
node:internal/process/promises:246
          triggerUncaughtException(err, true /* fromPromise */);
          ^

Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/.../node_modules/dayjs/locale/zh-cn' imported from /.../.vite-ssg-temp/main.mjs
Did you mean to import<EMAIL_ADDRESS>    at new NodeError (node:internal/errors:363:5)
    at finalizeResolution (node:internal/modules/esm/resolve:307:11)
    at moduleResolve (node:internal/modules/esm/resolve:742:10)
    at Loader.defaultResolve [as _resolve] (node:internal/modules/esm/resolve:853:11)
    at Loader.resolve (node:internal/modules/esm/loader:89:40)
    at Loader.getModuleJob (node:internal/modules/esm/loader:242:28)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:73:40)
    at link (node:internal/modules/esm/module_job:72:36) {
  code: 'ERR_MODULE_NOT_FOUND'
}
 ELIFECYCLE  Command failed with exit code 1.

reference https://github.com/antfu/vite-ssg/issues/182
I'm going to change it to theta

ssgOptions: {
    script: 'async',
    formatting: 'minify',
    format: 'cjs',
    onFinished() { generateSitemap() },
  },

executive build command
> vite-ssg build
...
An internal error occurred.
[vite-ssg] Please report an issue, if none already exists: https://github.com/antfu/vite-ssg/issues
node:internal/process/promises:246
          triggerUncaughtException(err, true /* fromPromise */);
          ^

Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: /.../node_modules/.pnpm/element-plus<EMAIL_ADDRESS>    at new NodeError (node:internal/errors:363:5)
    at Module.load (node:internal/modules/cjs/loader:987:11)
    at Function.Module._load (node:internal/modules/cjs/loader:829:14)
    at Module.require (node:internal/modules/cjs/loader:1013:19)
    at require (node:internal/modules/cjs/helpers:93:18)
    at Module.<anonymous> (/.../.vite-ssg-temp/main.cjs:13:12)
    at Module._compile (node:internal/modules/cjs/loader:1109:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1138:10)
    at Module.load (node:internal/modules/cjs/loader:989:32)
    at Function.Module._load (node:internal/modules/cjs/loader:829:14) {
  code: 'ERR_REQUIRE_ESM'
}
 ELIFECYCLE  Command failed with exit code 1.

System Info
System:
    OS: macOS
  Binaries:
    Node: 16.1.0 - ~/.nvm/versions/node/v16.1.0/bin/node
    Yarn: 1.22.10 - ~/.nvm/versions/node/v16.1.0/bin/yarn
    npm: 8.3.2 - ~/.nvm/versions/node/v16.1.0/bin/npm
  Browsers:
    Chrome: 103.0.5060.134
    Firefox: 89.0
    Safari: 15.5

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

/cc @sxzz if you are interested in taking a look 👀

Please provide a minimum reproduction
https://antfu.me/posts/why-reproductions-are-required-zh

We temporarily close this due to the lack of enough information.
Please provide a minimal reproduction to reopen the issue.
Thanks.

Why reproduction is required


Remove cyrillic symbols from a .doc file.

This is the file im using
sample.doc
Uploaded using ZenHub.io
And this is how im doing it.
On the client side i have a simple.
Im reading the file using the File Reader API, and then passing to the server a base64 string. like this.
var base64 = data.target.result.replace(/^data:application\/(vnd.openxmlformats-officedocument.wordprocessingml.document);base64,/, "");
Then on the server, im doing this.
buffer = new Buffer(base64, 'base64')
type = 'application/msword';
config = {
        preserveLineBreaks: true
    };

textract.fromBufferWithMime(type, buffer, config, function(error, text) {
      if (error) {
        console.log(error);
      } else {
        console.log(text)
      }
    });

This works great for .docx files, but with .doc files im getting something like this

UPDATE doing a fromUrl works great, so maybe the problem its the way im passing the buffer to the method fromBufferWithMime?

I've plugged that file in and don't seem to be having any issues.  All of the cyrillic you are seeing at the top of the screenshot, that is being output by textract and you are not expecting it?
Given the fromUrl was working, closing this as in the end that uses the same code path. If you've got more details, toss them in here and I can reopen.  Thanks!


In the nginx ingress alert Detailed link is 'var-job=' empty

After clicking on the link, I have to wait 10 seconds to get the data in Nginx Ingress dashboard

It is nginxIngressCertificateExpiry alert


Anyway to intercept keyboard shortcut events?

Hi,
I would like to add certain keyboard shortcuts for the user, like Ctrl+A will highlight all currently selected values, and Ctrl+Z will restore previously deleted values, etc.
Do you think is it possible to support this interaction?

This can be achieved by implementing the inputProps.onKeyDown method, but sadly this gets overridden by the default implementation ATM, will need to invoke custom onKeyDown implementations (from the default implementation) to make it work.