TISPARK-450: Fix tiflash task distributes as tikv

What problem does this PR solve?
https://internal.pingcap.net/jira/browse/TISPARK-450
What is changed and how it works?

Add new structure TiStoreType identifying which store (TiKV/TiFlash) to read;
Let region task generation (by RegionSplitter) get store based on TiStoreType;
Point 2 also handles retrying on TiFlash region movement, as region task regeneration is still based on TiStoreType;
Hard code TiFlash engine label as every other project (TiKV/TiFlash/TiDB) does;
Remove some useless methods.

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has exported function/method change
Has exported variable/fields change
Has interface methods change
Has persistent data change

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation
Need to update the tidb-ansible repository
Need to be included in the release note

/run-integration-tests

/run-all-tests tikv=v3.0.5 tidb=v3.0.5 pd=v3.0.5

/run-all-tests tikv=v3.0.5 tidb=v3.0.5 pd=v3.0.5

/run-all-tests tikv=v3.0.5 tidb=v3.0.5 pd=v3.0.5

/run-all-tests tikv=v3.0.5 tidb=v3.0.5 pd=v3.0.5

/run-all-tests tikv=v3.0.5 tidb=v3.0.5 pd=v3.0.5


MetaWorm purpose

I can't figure out what MetaWorm.py does. It looks incomplete: is it outdated legacy code that can be deleted, or something in progress? Doesn't seem used anywhere.

Incomplete in progress (though obviously not edited for a couple years).
It's supposed to be a second implementation of the API in addition to the SQL implementation that implements the API on top of itself, so that you can distribute an API query across multiple servers.
This has two conceivable purposes:

For very large bookworms (> 10m individual texts), makes it possible to distribute the files across a number of different servers. (The summary statistics work better than some of the other methods; I don't know how to best return search results, for example.)
When you want to compare two bookworms to each other, you could make that possible through an intermediary site. Compare the lines on the Hathi bookworm to those from ChronAm over the same period, for example. This would be useful, but is hampered by the need for identical key names in each.

Ideally, I guess, it should be cordoned off into its own branch until it works.

Closing this out, because the heir to this method is now in use in one of my production installations with a different name.


Key Agents support [DRAFT]

Key agents are users/services that protect and use a key to sign transactions at hire of the subscriber. Key agents have a general profile and 1:n extended public keys associated with their profile.

What data on the key agent profile is public? IMO, it's like a public social media profile.

A key agent may share their key with users (or vice versa), and either party can create vaults with both participants. In many cases, the key agent will be the vault creator.
The key agent profile can be like a normal nostr profile with additional metadata, either key/value pairs or attributes like jurisdiction, social media links, etc.
Each key will have metadata, either key/value pairs, or attributes like temperature, response time, device type, cost per signature, monthly cost in basis points, etc.

What data on the key agent profile is public? IMO, it's like a public social media profile.

Key agents should be able to communicate with their clients directly in the app using Nostr DM protocols.
Key agents should be able to post public events to the Nostr timeline and have them appear with their profile in the app.
When new activity occurs (key shares, proposals, etc), users get a DM style message and push notification.

@mgravitt how do you envision the monthly cost being charged?

@sebastianmontero the monthly charge could be paid directly from the vault. The price could be in basis points (like 1% per year), a monthly flat rate, or a per signature fee. I think the basis points pricing is most relevant based on the research. It would calculate [days since last payment] / 365 * 0.01 to get the payment required to "catch up" on a subscription at any point.

@mgravitt but would you charge the accumulated fees when the vault owner does a transaction?

It'll likely be a standalone transaction, once per month

Transaction fees could be an issue with a monthly transaction don't you think?

Yes, if it is auto-calculated since the last key agent fee, it doesn't have to be monthly, but it would just calculate the amount since the last key agent fee payment


tW Navigation

Since tW can have multiple checks per verse, as a user, I want to be able to navigate (one by one, or in a list) from one check to the next within a particular resource card. As I navigate, the associated word in the scripture pane should be highlighted (currently selected words turn blue). An associated TW card should show the content of the currently selected TW article. (Links inside the article do not need to open for the MVP. See #12).

[ ] Show TWL in a one by one view
[ ] Show TWL in a table (GL Quote | OL Quote)
[ ] The currently selected TWL record highlights a word in the scripture pane(s)
[ ] The TW card updates to show the TW article for the current TWL record

mockup of TSV table view:

The latest TWL is here: https://git.door43.org/unfoldingWord/en_twl


[hi] Add content/hi/docs/tutorials/kubernetes-basics/update/update-interactive.html

This PR is a part of ongoing hi localization effort here #29353
/language hi

/assign verma-kunal

/assign

any other changes to be made here?

/lgtm
/approve
Thank you @verma-kunal


Translation Samples needs to translate adaptive cards

CSS Sees issues come in related to translation, where cards (adaptive, etc) are not being translated.
Consider updating this sample to include an Adaptive Card translation.

Just some quick input. There's two routes we can go with this (that I can think of):

If users want translated Adaptive Cards, they'd need to use the AdaptiveCards NuGet package to build them with translated strings. Something like:

AdaptiveCard card = new AdaptiveCard();

card.Body.Add(new AdaptiveTextBlock()
{
    Text = translatedString,
    Size = AdaptiveTextSize.ExtraLarge
});

If users want to use a JSON card, we'd need a recursive nested JSON search similar to this to manually replace the strings

I have encountered this in IcM form twice and I wrote a C# function to translate Adaptive Cards so I'll paste it here:

The nested functions may appear a little confusing but I thought it would be better to have one self-contained function rather than have multiple functions.

public async Task<object> TranslateAdaptiveCardAsync(object card, string targetLocale, CancellationToken cancellationToken = default)
{
    var propertiesToTranslate = new[] { "text", "altText", "fallbackText", "title", "placeholder", "data" };

    var cardJObject = JObject.FromObject(card);
    var list = new List<(JContainer Container, object Key, string Text)>();

    void recurseThroughJObject(JObject jObject)
    {
        var type = jObject["type"];
        var parent = jObject.Parent;
        var grandParent = parent?.Parent;
        // value should be translated in facts and Input.Text, and ignored in Input.Date and Input.Time and Input.Toggle and Input.ChoiceSet and Input.Choice
        var valueIsTranslatable = type?.Type == JTokenType.String && (string)type == "Input.Text"
            || type == null && parent?.Type == JTokenType.Array && grandParent?.Type == JTokenType.Property && ((JProperty)grandParent)?.Name == "facts";

        foreach (var key in ((IDictionary<string, JToken>)jObject).Keys)
        {
            switchOnJToken(jObject, key, propertiesToTranslate.Contains(key) || (key == "value" && valueIsTranslatable));
        }
    }

    void switchOnJToken(JContainer jContainer, object key, bool shouldTranslate)
    {
        var jToken = jContainer[key];

        switch (jToken.Type)
        {
            case JTokenType.Object:

                recurseThroughJObject((JObject)jToken);
                break;

            case JTokenType.Array:

                var jArray = (JArray)jToken;
                var shouldTranslateChild = key as string == "inlines";

                for (int i = 0; i < jArray.Count; i++)
                {
                    switchOnJToken(jArray, i, shouldTranslateChild);
                }

                break;

            case JTokenType.String:

                if (shouldTranslate)
                {
                    // Store the text to translate as well as the JToken information to apply the translated text to
                    list.Add((jContainer, key, (string)jToken));
                }

                break;
        }
    }

    recurseThroughJObject(cardJObject);

    // From Cognitive Services translation documentation:
    // https://docs.microsoft.com/en-us/azure/cognitive-services/translator/quickstart-csharp-translate
    var requestBody = JsonConvert.SerializeObject(list.Select(item => new { item.Text }));

    using (var request = new HttpRequestMessage())
    {
        var uri = $"https://api.cognitive.microsofttranslator.com/translate?api-version=3.0&to={targetLocale}";
        request.Method = HttpMethod.Post;
        request.RequestUri = new Uri(uri);
        request.Content = new StringContent(requestBody, Encoding.UTF8, "application/json");
        request.Headers.Add("Ocp-Apim-Subscription-Key", _key);

        var response = await _client.SendAsync(request, cancellationToken);

        response.EnsureSuccessStatusCode();

        var responseBody = await response.Content.ReadAsStringAsync();
        var result = JsonConvert.DeserializeObject<TranslatorResponse[]>(responseBody);

        if (result == null)
        {
            return null;
        }

        for (int i = 0; i < result.Length && i < list.Count; i++)
        {
            var item = list[i];
            var translatedText = result[i]?.Translations?.FirstOrDefault()?.Text;

            if (!string.IsNullOrWhiteSpace(translatedText))
            {
                // Modify each stored JToken with the translated text
                item.Container[item.Key] = translatedText;
            }
        }

        // Return the modified JObject representing the Adaptive Card
        return cardJObject;
    }
}

I would like to extend a word of caution. If you know everything your bot can say ahead of time, which is the case for almost all bots, you should translate that text ahead of time and store the results in resource files. This will reduce the amount of calls you have to make to the translator service and also give you better translations because you will have more control over them. This is especially true if you have any text that is hard for the translator service to interpret, like complicated markdown.
The translator service is best used for incoming messages. If you must use it for outgoing messages, this Adaptive Card translator function should help you. But know that Adaptive Cards are complex by nature, and you may want to tweak this code to suit your own needs if there are some components of Adaptive Cards that you don't want to be translated, or if there are edge cases I haven't considered where my code can't handle certain cards.

I have also included Adaptive Card translations in my Bot Builder Community proposal: https://github.com/BotBuilderCommunity/botbuilder-community-dotnet/issues/137

Can we use the same function to translate hero cards also ?
Thanks..

@madbeher - No, the function is specific to Adaptive Cards. I'd need to make different functions to translate Bot Framework cards. Please comment on the Bot Builder Community issue with any requests.


Updated exit code for LVS regression

Fixes #64

Why are the LVS tests passing, but the DRC tests fail?

@atorkmabrains

@mkkassem @QuantamHD @mithro We have moved all the LVS/DRC to the pv repo. No longer required.


ghz: use image from official repo

This replaces the community-supported image I introduced earlier while refactoring CI.
Since then I developed the "official" ghz image and that PR got merged recently, so am replacing grpc_bench's ghz usage with it.

Thanks!


docs: update cn docs slugs

Changes:
Issue https://github.com/apache/apisix-website/issues/713 leads to a discussion and solution in PR https://github.com/apache/apisix-website/pull/903 .
https://github.com/apache/apisix-website/pull/903 solved the issue in English pages but introduced another two issues.

The slug was used as slug: /blog/{date}/{blog name} in #903. The blog links were changed a weird way, for exapmple: https://apisix.apache.org/blog/blog/2021/01/11/interview-apache-apisix-contributor-wang-pengcheng-senior-security-advisor-of-pwc-south-china-data-security-and-privacy-protection-team. Notice there are two blogs in beteen, i.e. blog/blog/{date}/{blog name}. Despite the fact that it somehow works, this is not what we want to have.

The zh-cn files were left behind. We all somehow misunderstood the process of language switch button. I thought adding to en-us files were good enough. But as a matter of fact, we do need to add slug to both en-us files and zh-cn files.

This PR does the follow things to fix the issue.

Modify slug: /blog/{date}/{blog name}  to slug: {date}/{blog name}
Add slug: {date}/{blog name} to crosspoding zh-cn files as well.
Remove a duplicated file.

cc @mangoGoForward please have a look when available, thx~

cc @mangoGoForward please have a look when available, thx~

Nice catch!


Can't add valid Email to my profile

Describe the issue
When I attempt to<EMAIL_ADDRESS>I don't receive a confirmation challenge.
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

Forgot to apply changes.


Do you know an app, extension or similar to create and update all json files in a simpler way than manually?

@basketball-ico - I`m using https://www.codeandweb.com/babeledit and it works pretty good.

It looks good, but the license is paid :(, you don't know any alternative?

No, sorry.

@basketball-ico
This plugin uses the Google Translate API to generate Json files in other languages:
https://marketplace.visualstudio.com/items?itemName=esskar.vscode-flutter-i18n-json
I switched to flutter_translate though.


breaking change in ES5 target mode in typescript >= 2.3.0

TypeScript Version:  2.4.0, 2.3.x
Target: ES5
Code
namespace App.modules {

	@Config(modules.main)

	class CompilerConfig {
		constructor(private $compileProvider) {
			$compileProvider.preAssignBindingsEnabled(true);
			$compileProvider.commentDirectivesEnabled(false);
			$compileProvider.cssClassDirectivesEnabled(false);
		}
	}

}

function Config(module: ng.IModule): Function {
	return (target: Function) => {
		return module.config(target);
	};
}

Expected behavior:
on versions 2.2.2 and older this code compiles to:
var App;
(function (App) {
    var modules;
    (function (modules) {
        var CompilerConfig = (function () {
            function CompilerConfig($compileProvider) {
                this.$compileProvider = $compileProvider;
                $compileProvider.preAssignBindingsEnabled(true);
                $compileProvider.commentDirectivesEnabled(false);
                $compileProvider.cssClassDirectivesEnabled(false);
            }
            return CompilerConfig;
        }());
        CompilerConfig = __decorate([
            Config(modules.main),
            __metadata("design:paramtypes", [Object])
        ], CompilerConfig);
    })(modules = App.modules || (App.modules = {}));
})(App || (App = {}));

Actual behavior:
on versions >= 2.3 as follows
var App;
(function (App) {
    var modules;
    (function (modules) {
        var CompilerConfig = (function () {
            function CompilerConfig($compileProvider) {
                this.$compileProvider = $compileProvider;
                $compileProvider.preAssignBindingsEnabled(true);
                $compileProvider.commentDirectivesEnabled(false);
                $compileProvider.cssClassDirectivesEnabled(false);
            }
            CompilerConfig = __decorate([
                Config(modules.main),
                __metadata("design:paramtypes", [Object])
            ], CompilerConfig);
            return CompilerConfig;
        }());
    })(modules = App.modules || (App.modules = {}));
})(App || (App = {}));

Note the CompilerConfig = __decorate was outside the closure, and in newer versions it's inside, which causes an error like this:
TypeError: Cannot set property '$compileProvider' of undefined
for the following statement this.$compileProvider = $compileProvider;

You said you're seeing this in 2.3? We only made this change in 2.4 as far as I recall.

exactly, I've seen this first in 2.3, in multiple versions.

We only moved the __decorate call inside of the IIFE in 2.4. I'm also not seeing how an error would be raised here, unless the Config decorator or some other code is calling CompilerConfig as a function rather than constructing it as a class. Can you review the call stack for the exception and determine how CompilerConfig is being called. That might provide more insight into the cause of this issue.

@rbuckton you are right. it was indeed moved to IIFE in 2.4, I was just getting the same error starting from 2.3.
Did a diff on outFile from 2.3 and 2.2, and the only difference is the following fact:
if there is a "use strict"; statement found at the top of at least one .ts file, typescript >= 2.3 moves the "use strict"; to the top of outFile, thus forcing strict mode for the whole code, no matter if "strict": false is explicitely set in compilerOptions.

no matter if "strict": false is explicitely set in compilerOptions.

set --alwaysStrict to false instead.

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


implementation of logistic regression

Implementation Algorithm for Logistic regression under python directory

🚀 Follow me, 💬 Join the WhatsApp Group, and ⭐ Star the Repo to support our journey together!


What if we changed the presentation around a bit so that when clicking on a fighter (or the current little atom icon

)
This would do some kind of animation which would flip the hero/villain/location card and show more information about it. In the future, we could even add some more AI-generated info that would show on the "back" of the card.
For today, though, we would show the powers/location details.

@all-contributors
please add @edeandrea for bug, code, data, doc, design, ideas, infra, maintenance, mentoring, projectManagement, promotion, question, review, security, tool, test.
please add @agoncal for bug, code, data, doc, design, ideas, infra, maintenance, promotion, question, review, tool, test.


Mediapipe Pose livestream segmentation causes Python to quit

I am trying to run Mediapipe Pose in livestream mode
options = PoseLandmarkerOptions(
    base_options=BaseOptions(model_asset_path='pose_landmarker_full.task'),
    running_mode=VisionRunningMode.LIVE_STREAM,
    output_segmentation_masks=True,
    result_callback=print_result)

with PoseLandmarker.create_from_options(options) as pose:
    cap = cv2.VideoCapture(0)
    i = 0
    while (1):
        succes, image = cap.read()
        image = np.array(image)
        imgRGB = cv2.cvtColor(image, cv2.COLOR_BGR2RGB)
        mp_image = mp.Image(image_format=mp.ImageFormat.SRGB, data=imgRGB)
        pose.detect_async(mp_image,i)
        i += 1

however, if I try to output the segmentation mask, the Python interpreter quits without any error messages
I have narrowed it down to this line
def print_result(result: PoseLandmarkerResult, output_image: mp.Image, timestamp_ms: int):

    alpha= result.segmentation_masks[0].numpy_view()

if that code is alpha= result.segmentation_masks , then Mediapipe will run and alpha is of type mediapipe.python._framework_bindings.image.Image object but if I wish to return the pixel data with .numpy_view() , it will cause Python to quit without any messages

Hey @ccarmatic. I'm waiting for a response internally, but I think this has to do with a lack of flow control in MediaPipe for the segmentation task portion. I was able to replicate it with one of my own projects. It's being looked into. Thanks!


Added a new contribution file and associated images in documentation,…

… added the name in the left menu
⚠ PLEASE DO NOT DELETE THE TEXT BELOW ⚠
Pull request's description
Add here a description of the modifications you made.
Checklist before merging (please do not edit)
You must review you work before to submit the review to others.
Self-review of your content
Remember the content must be readable and understandable by someone else than yourself.

[ ] From a technical point of view.
[ ] From a grammatical point of view (spelling mistakes, typos, clarity, etc.), and following the guidelines at the bottom.

External reviews of your content

[ ] You requested a technical review.
[ ] You requested a grammatical review.
[ ] You validated with @K0rdan or @alexgeron-Luos that it is safe to merge.

Some guidelines to keep in mind

Colons (:), semi-colons (;), exclamation (!), and interrogation points (?) are not preceded by a space (like full stops and commas). E.g.: Colons!

File names and/or address are put in italic. E.g.: The file main.c.

Functions, variables, or more generally short codes are put between grave accents. E.g.: To obtain code(), type `code()`.

Long codes are put into blocks of code with three grave accent on each side, and the language's name:
```c
// Some C language
```

"Luos engine" has a upper case on the L for "Luos" and lower case on the e for "engine".

We call it "Luos engine" as a proper name, and not "the Luos engine".

Following that fashion, anything that's owned by "Luos engine" implies that we must use 's as the standard English rule to show the possessive case, e.g. "Luos engine's API".

The names pipe, gate, inspector, sniffer, app or application, driver, etc. have a lower case and can be refereed with the determiner the. E.g.: The inspector.

I updated the contribution files (page, sidebar and images) from the turborepo (https://github.com/Luos-io/luos/pull/39).
The page contribute-to-luos.mdx must be available in current and next version of the documentation.


Add support for Beancount markup language

This PR adds highlight support for Beancount, a plaintext accounting tool.
It's a first naive implementation with very good result. Thanks to Beancount simple and unambiguous syntax.

It seems I need to workout the relevance stuff.
I need some help to run the tests on my machine. I went through the building and testing instructions on the  documentation but my end result is bellow.
Any tip?
highlight.js(master)$ npm test

><EMAIL_ADDRESS>test /Users/henrique/workspace/highlight.js
> mocha test/

module.js:472
    throw err;
    ^

Error: Cannot find module '../../build'
    at Function.Module._resolveFilename (module.js:470:15)
    at Function.Module._load (module.js:418:25)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/henrique/workspace/highlight.js/test/api/ident.js:3:12)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Suite.<anonymous> (/Users/henrique/workspace/highlight.js/test/api/index.js:4:3)
    at context.describe.context.context (/Users/henrique/workspace/highlight.js/node_modules/mocha/lib/interfaces/bdd.js:47:10)
    at Object.<anonymous> (/Users/henrique/workspace/highlight.js/test/api/index.js:3:1)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/henrique/workspace/highlight.js/test/index.js:5:1)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at /Users/henrique/workspace/highlight.js/node_modules/mocha/lib/mocha.js:220:27
    at Array.forEach (native)
    at Mocha.loadFiles (/Users/henrique/workspace/highlight.js/node_modules/mocha/lib/mocha.js:217:14)
    at Mocha.run (/Users/henrique/workspace/highlight.js/node_modules/mocha/lib/mocha.js:469:10)
    at Object.<anonymous> (/Users/henrique/workspace/highlight.js/node_modules/mocha/bin/_mocha:404:18)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.runMain (module.js:605:10)
    at run (bootstrap_node.js:420:7)
    at startup (bootstrap_node.js:139:9)
    at bootstrap_node.js:535:3
npm ERR! Test failed.  See above for more details.

@henriquebastos You need to first build the library before you can run tests:
node ./tools/build.js -t node

@henriquebastos
First of all, truly sorry for the long delay in anyone responding to this PR.
Thank you so much for desiring to contribute and writing a language grammar.  Unfortunately, we currently are no longer accepting new languages into the core repository.  The existing maintainers simply don't have time to maintain new languages (fixing bugs, dealing with issues, etc).  We'd still love to host your language as a 3rd party module though - if you're willing to help maintain it.
It's easy to turn your language into a module that anyone who likes can easily integrate with Highlight.js using a single line of code:
<script src="/path/to/highlight.min.js"></script>
<!-- they'd simply add your language modules -->
<script src="/path/to/highlightjs-beancount.js"></script>
<script>hljs.initHighlightingOnLoad();</script>

It's also possible for users to rebuild the highlight.js library to include specific 3rd party modules, all in a single file (vs loading a bunch of 3rd party languages one at a time).
If you'd like to become the maintainer for this language then we're happy to create a new repository in the highlightjs organization and host the repository there.  That assumes you'd be open to reviewing issues and fixing bugs as they arise.  You're also welcome to host the repo yourself and we'll link to it from our README.
There isn't much in the way of how to docs on this yet, but there are sample repositories and it's honestly pretty easy.  We're still fleshing out an official "spec" for 3rd party modules, but even after that's done it should require minimal changes from the examples we have already.  IE, if you wanted to get started now there isn't any big reason to wait.
My own robots.txt grammar repository is a great, simple example:
https://github.com/highlightjs/highlightjs-robots-txt
If you have any questions feel free to ask.

Closing for inactivity; (also finally cleaning all these old PRs out for the new year).
As mentioned above if you're interested in becoming the maintainer of a 3rd party module, please respond [whenever you find time] and we'd be happy to help you get started.


Shouldn't check(myObj, MyCollection.simpleSchema()) run clean?

I have a method that looks roughly as follows.
Meteor.methods({
  'Players.addToGame': function (playerDoc, gameId) {
    check(playerDoc, Players.simpleSchema());
    // do some other stuff.
  }
});

This does not work as intended. I checked, and it looks like the autoValues never fire on the check. This, however, does work:
check(Players.simpleSchema().clean(playerDoc), Players.simpleSchema());

But that seems a bit redundant. Can the check clean it automatically?

I wouldn't want to assume that everyone wants it cleaned. But I will be making some different checking functions in an upcoming release that can have options to do the cleaning.

Makes sense, I have tons of autovalues on all of my schemas.

Moved to https://github.com/aldeed/node-simple-schema/issues/10


Removed unnecessary chevron and fixed margins for overflow ellipsis.

Removed unnecessary chevron at overflow and fixed margins.

Approved


Append speed

Pull request
Please confirm that this pull request has done the following:

[x] Tests added
[x] Documentation added (where applicable)
[x] Example added (either to an existing notebook or as a new notebook, where applicable)
[x] Description in CHANGELOG.rst added

Adding to CHANGELOG.rst
Please add a single line in the changelog notes similar to one of the following:
- (`#XX <https://github.com/openscm/scmdata/pull/XX>`_) Added feature which does something
- (`#XX <https://github.com/openscm/scmdata/pull/XX>`_) Fixed bug identified in (`#YY <https://github.com/openscm/scmdata/issues/YY>`_)

50x at the top end...
I would have prefered maintaining the indexes where ever possible, but speed is king


fix convert all fee values to fixed

Signed-off-by: Denys Bitaccess<EMAIL_ADDRESS>

Rarely and never are two different things. When do we use them?

@dzdidi Correction, I thought they were only used when manually specified, but in fact they're used to bump fees


Move PTC runtime into individual modules

@johnjbarton PTAL
This just moves the code around. This is in preparation for #1979

Sorry for the delay, I was away.
LGTM. Consider spelling out PTC in the commit message.


Improve constraint of psr/log

As suggested in https://github.com/psliwa/image-optimizer/pull/64#issuecomment-440953170

@psliwa Some people really need this (me included), can you merge it and tag a new release please ?

@psliwa ?


Fix coverity issue in coap_client sample

Ignore socket close() return value in this sample.
Fixes #18961

We might want to backport this (and other stuff) to 2.0.1, if we are to have that.

We might want to backport this (and other stuff) to 2.0.1, if we are to have that.

Yep, that's why I did not merge it yet. We need to have the backport bot running for 2.0.1

As agreed in release meeting, only really major issues would go to 2.0.1 and this is not that stuff.


New package: Lenovo.RealtekAudioDriver version 6.0.9159.1

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/113301&drop=dogfoodAlpha

Automatic Validation ended with:

/azp run

Automatic Validation ended with:

Automatic Validation ended with:


D-01

Pull Request Type
What kind of change does this PR introduce?
[ ] Solved an issue/task
added a file in d programming lang

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Fix generalizeLocationName

I'm testing some stuff in the "North Central US (Stage)" location and our current logic doesn't quite work because the location sometimes shows up with or without "(Stage)" and we need to know they're the same.

This problem is more complicated than I thought. Will file a new PR separately


Simplify throttling logic

This introduces a couple of changes in the parameter interpretation:

copy_from_minimum_input_speed is now the total number of bytes in the interval (so we could multiply the currently configured values by the interval to preserve min. intended throughput.
the minimum speed is now inclusive (only smaller values will cause an exception).

@Algunenano will this satisfy you? 😛
@dgaubert do you think this looks safe enough?

@Algunenano will this satisfy you? stuck_out_tongue

LGTM but it's confusing, it's saying this.minimunBytesPerSecondThreshold but the threshold isn't per second, but per this.sampleLength.

Resolve conflict before proceeding, please move on. We are refactoring/optimizing a 10 lines algorithm but we still don't know if we solve the issue or not.

Can you add a line in NEWS about this change, please?


[FEATURE] Improve the Dashbaord Layout

Detailed description
We have to improve the dashboard layout

Context

We have to make the dashboard user-friendly

Possible implementation
No response
Additional information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

can i work on this issue

can i work on this issue ? Please assign it to me

That's Great
We have assigned this to you..
Thank you.

@Arun-cn  hey are you working on this issue?
if yes, checkout dev branch and checkout settings page, u can make this one similar to that


Refactor/rename CSI and USB Sources - derive from new common Camera Source

The CSI and USB Camera sources share common properties including the important device-location which must be set when using multiple Camera sources.
Need to create a new CameraSourceBintr class as a supper class for both Camera Sources.  The constructors will be renamed from

dsl_source_csi_new
dsl_source_usb_new

to

dsl_source_camera_csi_new
dsl_source_camera_usb_new

The above assumptions were incorrect.  Not going to implement.


Impoved readme: The readme is very difficult to undersatnd beause it referes to the open-clip project

It would be good to have a more concise description of using the codebase.

Sure, we will proof the readme in this two weeks, to make the introduction of the readme more clear, by adding some introductions of what each main code file are used for.

Hi, we now make the readme file more clear by establishing experiment scripts, examples, paper, appendix and infer examples.
In May the project is still under-research so we did not write the readme. Now you can see an updated  project with milestone .


Implement the ComplEx knowledge graph algorithm

See: #756

Codecov Report

Merging #848 into develop will decrease coverage by 0.22%.
The diff coverage is 95.51%.

@@             Coverage Diff             @@
##           develop     #848      +/-   ##
===========================================
- Coverage    82.74%   82.52%   -0.23%
===========================================
  Files           46       48       +2
  Lines         5391     5299      -92
===========================================
- Hits          4461     4373      -88
+ Misses         930      926       -4

Impacted Files
Coverage Δ

stellargraph/core/graph.py
96.96% <100%> (-0.1%)
:arrow_down:

stellargraph/core/element_data.py
92.81% <100%> (+0.44%)
:arrow_up:

stellargraph/layer/knowledge_graph.py
90% <90%> (ø)

stellargraph/mapper/knowledge_graph.py
98.76% <98.76%> (ø)

stellargraph/data/edge_splitter.py
89.61% <0%> (-0.28%)
:arrow_down:

stellargraph/data/explorer.py
97.27% <0%> (+1.08%)
: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 9401f6e...ce5034f. Read the comment docs.


Sum of Squares #246 java

Close #246

Hii @KikiManjaro
Thanks for contributing to this open source project 😃
This PR will be reviewed and merged shortly.

Preview : https://codinasion--pr19-dev-57ydyon8.web.app/programme/sum-of-square/


[BUG]: I can select the groq llama-3.2-90b-vision-preview model, but uploading images fails to recognize

How are you running AnythingLLM?
AnythingLLM desktop app
What happened?
groq released the image recognition language for llama 3.2, and you can also choose related 90B and 11B models, but when I upload an image and ask the language model to recognize it, it always fails, I don't know if it's stuck on the base64 issue
Are there known steps to reproduce?
選取groq的llama-3.2-90b-vision-preview  或是 llama-3.2-11b-vision-preview 模型 , 然後上傳一張圖片

Not a bug, needs to be done as a feature. We dont currently pass in images for the Groq provider
https://github.com/Mintplex-Labs/anything-llm/blob/7342839e779e6dee0c2e1a670857dfe3e9ce6edf/server/utils/AiProviders/groq/index.js#L62-L67

okay , thanks for help .


[CI] Fix e2e detox build step

Summary
Fixes the e2e detox build step by manually overriding PROJECT_ROOT for the project's JS bundle build step.
After seeing quite a few comments suggesting running some variant of the react-native bundle manually on your own so as to build the jsbundle required as part of the build step for RNTester project...
The main issue I found was that the working directory in which react-native-xcode.sh executed the CLI bundle step was not correct. The PROJECT_ROOT was not resolving to the root of the react-native project directory, but instead to something to the effect of /Users/gibson.cheng/IdeaProjects/react-native/../.. - of which of course the build step would not be able to find the react-native project to run the build against.
I'm not sure if new generated react-native projects require this manual override, so I only applied it to the RNTester project. Reviewers are welcome to correct my understanding and solutioning to this matter :)
@hramos, if this works, perhaps there would not be a need to push through with #24936. Also, this  contributes to #23561.
Changelog
[Internal] [Fixed] - Fix build-ios-e2e build step
Test Plan

build-ios-e2e should work now

Thanks for the PR! I'll update mine and avoid disabling the Detox tests.

Thanks for the merge, @hramos! :)


ssh-keygen: add commands to print the fingerprint

A small edit of the cheat sheet to include the commands to get the fingerprint of a public key (also in Github format).

Very useful. Thanks!


Migrate TrackingProtectionAppsAdapter

Task/Issue URL: https://app.asana.com/0/0/1203984881672161/f
Description
Removed usage of kotlinx.android.synthetic in TrackingProtectionAppsAdapter.
Steps to test this PR

[x] Install from this branch.
[x] Enable AppTP from Settings.
[x] Wait until you see some trackers on App Tracking Protection screen.
[x] On App Tracking Protection  scroll down to Having issues with an app? and tap on it.
[x] On Recent App Activity everything should work as expected: you can see the list, enable / disable apps.

NO UI changes

NIT comment about accessing context from the itemView but feel free to skip it as it is not related to migrating away from kotlin synthetics +1

Thanks @nalcalag ! I committed your suggestions :tada:


Remove unnecessary flag when running JS server

Summary
This removes the --projectRoot flag in launchPackager.bat as it was removed,  fixing this bundler not opening issue on cli repo when yarn react-native run-android is run.
This is Windows related only so should be only tested on Windows.
Changelog
[Internal] [Fixed] - Fixed Metro Bundler not opening when running yarn react-native run-android
Test Plan

Create a new repo using react-native init
Edit the launchPackager.bat file
Run yarn react-native run-android

@theweavrs please check this: https://github.com/react-native-community/cli/issues/484#issuecomment-508792593 looks like we need another solution

You should see https://github.com/react-native-community/cli/issues/484#issuecomment-508792593 to clarification.


polish: Show boundary_downstream/upstream alerts V2 bus

Notion task
For V2 bus, we always want to show boundary_downstream alerts and want to show boundary_upstream alerts for all effects except :suspension.

[ ] Tests added?

QA-ing the performance, first. I think there's a bug here? With this suspension, shouldn't there be a flex zone alert?

QA-ing the performance, first. I think there's a bug here? With this suspension, shouldn't there be a flex zone alert?

No, because the home stop is outside the list of affected stops. Bus alerts at minimum have to affect the home stop for it to show on screens. This alert also wouldn't show if it were the first stop in the stop range because it would then be boundary_upstream, which is not a location we care about for suspensions.
The reasoning behind these decisions involves assumptions based on this guide. Mindy also added some graphics to the comments of this task that are super helpful. If things are still fuzzy, let me know and I'd be happy to walk through it.

The reasoning behind these decisions involves assumptions based on this guide. Mindy also added some graphics to the comments of this task that are super helpful. If things are still fuzzy, let me know and I'd be happy to walk through it.

Ohhh hmm. The guide helped fill in some blanks for me, because the notion diagrams really confused me. So, when a PIO makes a detour alert, only the skipped stops are included in the selection; when they make a suspension, the "new endpoints" for the route are selected also, in addition to the skipped stops.
I think that's how subway works, also, but I guess I didn't expect that for bus.

The reasoning behind these decisions involves assumptions based on this guide. Mindy also added some graphics to the comments of this task that are super helpful. If things are still fuzzy, let me know and I'd be happy to walk through it.

Ohhh hmm. The guide helped fill in some blanks for me, because the notion diagrams really confused me. So, when a PIO makes a detour alert, only the skipped stops are included in the selection; when they make a suspension, the "new endpoints" for the route are selected also, in addition to the skipped stops.
I think that's how subway works, also, but I guess I didn't expect that for bus.

That's correct. So like with subway, the stops on either side of the range of the alert aren't actually affected by the suspension directly. With that being the case, the last stop of the range of the alert (boundary_upstream) doesn't care at all about the suspension. None of the affected stops are reachable from it so it's irrelevant info if you are continuing downstream.

This alert also wouldn't show if it were the first stop in the stop range because it would then be boundary_upstream, which is not a location we care about for suspensions.

I said this earlier and that is actually incorrect. The first stop in the range is actually considered boundary_downstream, so it would show an alert if included. Broadway @ Cary would be the boundary_upstream stop of the alert, and that stop would not show an alert.
For detour, we talked about how it isn't ideal that the "endpoints" are not included in the alert because that means there is no info about the detour at the stop before the detour starts. That's out of scope here though and will be discussed in the future.


Footer is Missing

It's better to have quick links at footer, or at least some licensing and contributors info.

As we have good top navigation, we would like to leave much usable space for end-user. Though it's nice to have.
@HarshulChavda - Will you please take care of this with minimal use of height?

Addressed with PR - https://github.com/prafullmaru/ePanchayat/pull/16


Make g++ optimized aggregators optional

Signed-off-by: Dimitris Karnikis<EMAIL_ADDRESS>

Due to the recent changes, the aggregators test fail.
I have already included a patch that should fix the issue for the following PRs.
Merging to main


Bump vagrant box versions for tests

This change is

I'm going to merge this to clear up tests. It's yolo fine ;)


Opening images on click - support request

If I'm not wrong minimaJake should open images in full-size in a new tab on click. I saw this feature on https://blog.jakelee.co.uk for instance.
However on my blog this is working only for the header image. What should I check to make it work?

This isn't specific to the theme, and generally depends on how images are defined in your posts. For example, I typically use:
[![](/assets/images/my-image-thumbnail.png)](/assets/images/my-image-fullsize.png)

What this does is embed an image AND make it a link to a fullsize image (another example). You can use the same image for both.


Add TransformerLens logo to docs and GitHub

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #237
Type of change
Please delete options that are not relevant.

[x] Non-breaking cosmetic change

Screenshots
Please attach before and after screenshots of the change if applicable.

Context
Screenshot

README.md

Sphinx Docs

Checklist:

[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes
[x] I have not rewritten tests relating to key interfaces which would affect backward compatibility

Cheers mate!


Improve composer.json snippet in README

Right now the composer.json example on the README has two ellipses indicating that this is just an example.
The ellipses are not legal JSON characters so they appear as a red error in the README.
by swapping to jsonc and adding // these now appear with "comment" syntax highlighting and do not distract.

updated to

Thanks!


option to open new or existing editor

I basically need a TextEditor object for a given url.
If that file is already open I want that editor to be reused.
This works well if the editor is in the active view column but not otherwise, always creating a new editor.
There are the ViewColumn.Active and Beside options for showTextDocument but they don't cover this case.
The other options in TextDocumentShowOptions don't seem to be useful either.
(Preview editors are disabled.)

//cc @jrieken

//cc @jrieken

I have a same request, but I accidentally found a way:
async function showExistingTextDocument(document: vscode.TextDocument) {
  const editor = vscode.window.visibleTextEditors.find(editor => editor.document.uri.toString() === document.uri.toString());
  await vscode.window.showTextDocument(document, editor?.viewColumn);
}

I have a same request, but I accidentally found a way:
async function showExistingTextDocument(document: vscode.TextDocument) {
  const editor = vscode.window.visibleTextEditors.find(editor => editor.document.uri.toString() === document.uri.toString());
  await vscode.window.showTextDocument(document, editor?.viewColumn);
}

@johnsoncodehk  Would that not just cycle through the visible ones (say you have 3 columns of opened documents -each with a bunch of opened tabs, only one in each column is the 'visible' one) in each column only - therefore not poping up if already opened but in a tab not in focus in a particular column?

@johnsoncodehk easy to test with console.log('visibleTextEditors.length' , vscode.window.visibleTextEditors.length); while having 2 columns of 3 documents each seeing a total of 6 tabs, does it print 6 or 2 ?

@Trass3r I also have this requirement - did you ever found a solution for this?


403 redirections (formerly "Redirect to change password page when session is expired")

This PR adds the ability to redirect users (belonging to an organization or not) to the password change page when their session has expired.
The PR implements a general approach to the problem using a network request interceptor, that allows the developer to add any behaviours on top of the request-defined ones. It defines success and error interceptors for all the filtered requests.
Related to #13794
Closes #14435

retest this please

I'm starting to work on this PR to add what it's required to solve this issue.
I'll let you know when it's ready for CR/Acceptance.

@jesusbotella This should be ready for review. I'd like to have this merged this week if possible 🙏
I'm adding it to the backend kanban as well, to get a quick CR for the backend part.
Regarding acceptance I guess it can be done by someone at backend (since it involves enabling MFA, password expiration, etc.)


no access to https://aka.ms/binb

I don't have access to the link provided about Batch Inference. no access to https://aka.ms/binb

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

ID: 61a394fa-db82-07f5-caae-8f837fcd950f
Version Independent ID: 50d10215-439a-b050-642b-d729bf53647f
Content: What's new in the release? - Azure Machine Learning
Content Source: articles/machine-learning/service/azure-machine-learning-release-notes.md
Service: machine-learning
Sub-service: core
GitHub Login: @j-martens
Microsoft Alias: jmartens

@swanderz - thanks for the report. I'll look into this now.
https://aka.ms/binb

@swanderz - while I track it down, it may have been designed to point to:
https://docs.microsoft.com/en-us/azure/machine-learning/service/how-to-run-batch-predictions
https://github.com/Azure/MachineLearningNotebooks/tree/master/contrib/batch_inferencing

Hope those links worked, @swanderz
#please-close

@j-martens. the links you gave worked, but shouldn't this issue be closed once the link in the documentation is fixed?

@swanderz it was. :-)

ok great. when I look at the what's new page I don't see any link to documentation for batch inference at all anymore. Perhaps it was fixed in a non-public facing release and the change will show up later?


Add getting started documents for IDP

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?
Add more docs prior to Beta launch.
Preview URL - https://6480711c45bbf00c48e37f16--harness-developer.netlify.app/

@OrkoHunter I'm going to do some additional review to help Deepak. I'll go ahead and commit small changes to grammar, punctuation, and style. I'll comment where I have questions.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Support Markdown's image size specification

I modified the 'Renderer.prototype.image' function so that it supports Markdown's image size specification.
For example, a Markdown code
![](your_image.jpg =100x200)

can be interpreted as
<img src="your_image.jpg" width="100" height="200" />

Thanks!

:+1: would be nicer with a test though

+1
May I know when this pull-request will be merge?

@tatsy by the way, where can I find Markdown's image size specification? Does spec exist?

Hello @doortts.
As you know, official Markdown does not support image size specification. So this is only an option.
If you'd like to specify image size in Markdown, I recommend you to use markdown-it.
I already provides the plugin for this to specify image sizes (markdown-it-imsize).

Which implementations is this currently a part of?

@chjj
Apparently Mou does, but most that actually support this consider themselves derivatives AFAICT [1], like Kramdown and MultiMarkdown.
[1] This was only from a simple Google search, BTW. I'm not that aware of the likely hundreds of implementations out there.

👍 for this support in the future. Was wondering why this wasn't working. Thanks @tatsy, though it would be better with this merged

Closing due to merge conflicts, missing tests, and no reference to which supported specification we are targeting. See #956
Sorry @doortts


Update travis with supported versions of python

What? and Why?
Since upgrading everything to 3.6, we no longer need to test 3.4 and 3.5.
Also added 3.7 for when we eventually upgrade to it so we can have confidence that it works
Checklist

[x] CHANGELOG.md updated? (if required)

@insacuri Are you aware of this build failure?


feat(MetadataContentCard): Add descriptionDetails TextBox below description in MetadataContentCard

Description
Adds description details TextBox below the description text in MetadataContentCard which is inherited by CardContent components.
Before:

After:

References
LUI-1041
Testing
Go to MetadataContentCard story and note new descriptionDetails Metadata Control.
Checklist

[✅] all commented code has been removed
[✅] any new console issues have been resolved
[✅] code linter and formatter has been run
[✅] test coverage meets repo requirements
[✅] PR name matches the expected semantic-commit syntax

 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.Aaron Begley seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/452/rdkcentral/Lightning-UI-Components

Commit: c2e261b65b6f3c46ec050f95f1284284661146fc

Report detail: gist

Test Execution Passed.

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/452/rdkcentral/Lightning-UI-Components

Commit: 9396172db590fd03ae1f24394476340a839b7f68

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/452/rdkcentral/Lightning-UI-Components

Commit: 9479c8ac35c877c737a199085ad7e91ddea30091

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/452/rdkcentral/Lightning-UI-Components

Commit: 9cbd36f07193330976e2edaf07220cf40f16da2f

Report detail: gist

Test Execution Passed.

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/452/rdkcentral/Lightning-UI-Components

Commit: 6fd33832e4b7956bb3f4c2252fa9f3d939350346

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/452/rdkcentral/Lightning-UI-Components

Commit: f2567168ef445065d64891548af3c1aa7f01313c

Report detail: gist

Test Execution Passed.

Test Execution Passed.

Test Execution Passed.

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/452/rdkcentral/Lightning-UI-Components

Commit: f2567168ef445065d64891548af3c1aa7f01313c

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: spurious match

Commit: f2567168ef445065d64891548af3c1aa7f01313c

Test Execution Failed.

Test Execution Failed.

Test Execution Failed.

Test Execution Failed.

CardContent.test.js is going to need its announce test updated now that the MetadataCardContent has DescriptionDetails. After that's updated, can you run yarn test:updateSnapshot to make sure tests pass and the snapshots are all up to date?

Test Execution Failed.

Test Execution Failed.

Test Execution Failed.

Test Execution Failed.

Test Execution Failed.

Test Execution Failed.

:tada: This PR is included in version @lightningjs/ui-components-v2.20.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Cache callbacks for long running processes

See https://github.com/erlang/otp/pull/5831 for details as to why it's faster to do that. Might not be worth doing in many parts of Cowboy, but some like Websocket or the loop handlers may benefit from this.

Could I give it a try?

I'd like to give it a try. Sounds like a nice challenge

The hard part is figuring out how to measure the improvements (or lack thereof). It might not be worth outside of receiving Erlang messages (the network is likely slower).

Maybe we could measure the execution time with those Erlang profiling tools?

Profiling tools are useful but the deciding factors for performance improvements in Cowboy are latency and throughput so for Websocket for example this means being able to make the clients either receive data faster than before, or receive more data than before, for specific workloads. Currently the only such measurements we have are the run times of the Autobahn test suite.


Sign Up Page

Module: React
Page: Sign Up Page

Hello sir I want to work upon this, I have a good experience of React

Hello @Ayushjain373, this issue is already assigned to @mafzaldev. Kindly have a look at React-module, you will find plenty of things to work. Open a new issue of the page on which you want to work other than sign up page.


Update scala3-library, ... to 3.4.2

About this PR
📦 Updates

org.scala-lang:scala3-library
org.scala-lang:scala3-library_sjs1

from 3.4.1 to 3.4.2
📜 GitHub Release Notes - Version Diff - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-lang" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, version-scheme:semver-spec, commit-count:1

Superseded by #178.


Module EventServiceProvider

Create an EventServiceProvider class on module creation and an artisan command php artisan module:EventServiceProvider Blog to create a new EventServiceProvider for an existing module.

Auto registered.
Auto-discovery enabled by default.
Enables module's Observers and Events to be registered locally instead of using the App\Providers\EventServiceProvider

can you fix the failing tests.
Also, EventServiceProvider should not be created by default instead use the new command to generate them as needed. No benefit in creating a provider when in most cases it won't be used.
Better to generate the provider when its needed.

No events or observers will work without this file.

yes so generate an events module when you need to use events or observers otherwise its simply not needed.

I need to create an Event or Observer within any Module and checked if it will work.
This package is a solution, and of what essence is it if it requires a complete work around to make included features (Events, Observers, etc.) function.
Please, note that this also require modifying the package core code if a user are to fix it themselves. Is it recommended to edit the package core code to enable a feature?

The reason you've created the module:EventServiceProvider command is to generate the provider, surely you can use that when needed.
Also the command would be better to be renamed to 'module:event-providerto match themodule:route-provider` style.
Every module does not need an events service provider. I've worked on systems with around 20 modules only a couple had event service providers since they used observers the rest did not and still work absolutely fine.

okay a misunderstanding, lets move on.
My understanding is that the PR is a command to generate module EventServiceProviders.
You can use that command to generate EventServiceProviders for modules that need it. Help me to understand why you cannot use that?

What about we add an option to the config to allow dev choose when to enable or disable the auto creation feature? I believe that will solve the problem, right?

yes, that would be ideal.


adapter not initialized inside a nf-if

Hi, the adapter is not initialized when the ui-scroll element is inside another element with ng-if. here is an example if you eliminate the ng-if, the adapter works just fine.

@jamartinezc This is how Angular $parse works. Due to ui-scroll documentation "Angular $parse service is being used in assignable expressions implementation" and "the scroller will traverse its parents (from the ui-scroll element's scope up to the $rootScope) to locate the target scope". Ng-if creates a separate scope before your controller scope. And it stops $parse traversing. To avoid such situations Controller As syntax was introduced in Angular 1.2.
Here is exactly your case with intermediate ng-if: https://rawgit.com/angular-ui/ui-scroll/master/demo/controllerAs/controllerAs.html


API velocity graph point popup cannot be hidden

Steps to reproduce:

In an API velocity graph, click any point
Scroll to another graph and click that point
There are now two popups showing. Click outside to close the second of them.

The first popup now sits around, impossible to dismiss

Ah, I see. I think click-to-cancel is only bound to the graph where the popup originated.


[DO NOT MERGE] Test pull request

Docs Build status updates of commit f65bd30:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/api/NuGet-Protocols.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Closing for now!


Cannot join with long hostname

When running a microk8s join command on a node with a long hostname I run into the following error
Contacting cluster at <IP_ADDRESS>
Traceback (most recent call last):
  File "/snap/microk8s/5891/scripts/wrappers/join.py", line 1033, in <module>
    join(prog_name="microk8s join")
  File "/snap/microk8s/5891/usr/lib/python3/dist-packages/click/core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "/snap/microk8s/5891/usr/lib/python3/dist-packages/click/core.py", line 717, in main
    rv = self.invoke(ctx)
  File "/snap/microk8s/5891/usr/lib/python3/dist-packages/click/core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/snap/microk8s/5891/usr/lib/python3/dist-packages/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/snap/microk8s/5891/scripts/wrappers/join.py", line 1024, in join
    join_dqlite(connection_parts, verify, worker)
  File "/snap/microk8s/5891/scripts/wrappers/join.py", line 783, in join_dqlite
    join_dqlite_worker_node(info, master_ip, master_port, token)
  File "/snap/microk8s/5891/scripts/wrappers/join.py", line 851, in join_dqlite_worker_node
    update_cert_auth_kubelet(token, info["ca"], master_ip, master_port)
  File "/snap/microk8s/5891/scripts/wrappers/join.py", line 460, in update_cert_auth_kubelet
    cert = get_client_cert(
  File "/snap/microk8s/5891/scripts/wrappers/join.py", line 281, in get_client_cert
    subprocess.check_call(cmd_cert.split(), stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
  File "/snap/microk8s/5891/usr/lib/python3.8/subprocess.py", line 364, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/snap/microk8s/5891/usr/bin/openssl', 'req', '-new', '-sha256', '-key', '/var/snap/microk8s/current/certs/kubelet.key', '-out', '/var/snap/microk8s/current/certs/kubelet.csr', '-subj', '/CN=system:node:node2.cskama-171507.advosuwmadison-pg0.cloudlab.umass.edu/O=system:nodes']' returned non-zero exit status 1.

Running the openssl command manually reveals the problem
$ sudo /snap/microk8s/5891/usr/bin/openssl req -new -sha256 -key /var/snap/microk8s/current/certs/kubelet.key -out /var/snap/microk8s/current/certs/kubelet.csr -subj /CN=system:node:node2.cskama-171507.advosuwmadison-pg0.cloudlab.umass.edu/O=system:nodes
problems making Certificate Request
140169048745280:error:0D07A097:asn1 encoding routines:ASN1_mbstring_ncopy:string too long:../crypto/asn1/a_mbstr.c:107:maxsize=64

microk8s should not put the entire hostname into the CN field. They cannot be longer than 64 characters per the ASN1 standard. See for example this answer on StackOverflow.

Would it be an option to shorten the hostname of the node (e.g. by removing the domain name)?

Sorry, I never replied to this. This happened on Cloudlab, which is fully automated and sets up the hostnames for me.
I
I ended up changing the hostnames at runtime using hostnamectl, but that seems more like a workaround than a solution.


Initial take on CAMEL-6380

Created a ModelJAXBContextFactory and replaced all usages of JAXBContext.newInstance() with a call to the factory.
This would be the first step in a multistep change, wanted to get feedback on what I have done so far..

Thanks Aaron I have just merged this. Do you mind closing this PR?


Option to Change Names for a Resultant HTML File

Version Observed In
1.0.0
Description
There should be an option to (like -n or --name) to change the name of the resultant file. The current behaviour is that the resultant file name has to be the same as the child Chasse file.
Errors
None.
Expected Behaviour

The given options should work perfectly, with no errors.
Information about the name change should also be reflected in the success message log.

Further Comments
None.

To be tackled in v1.0.1.

Version Fixed In
1.1.0
Solution Implemented

Add a new argument.
Pass parameters to the required modules.
Final parameter passed on to the writer that opens the file to write on.

Drawbacks to Solution
Code is highly coupled. Refactor to ease coupling needs to be taken care of as soon as possible.
Further Comments
None.


chore(refactor): d-operators deployment yamls

Signed-off-by: Harsh Shekhar<EMAIL_ADDRESS>This PR intends to do the following:

Add steps to deploy d-operators in README.
Refactor d-operators deployment yamls.

:tada: This PR is included in version 1.10.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Rules for hostname

My hostname is something like LAPTOP-STUVWXYZ. I'm pretty sure I need to change it because when I run sudo apt-get update or sudo apt-get upgrade, I get this message:
sudo: unable to resolve host LAPTOP-STUVWXYZ

Also, I installed the news (NNTP) reader nn and when I run it, I get this message:
hostname=LAPTOP-STUVWXYZ, You need a fully qualified domain name

I understand from issue #125 that I need to change the hostname from Windows, but before I do I'm wondering what, if any, rules there are for a hostname that will work for Windows, Bash, nn, Crashplan, and whatever tools I use now (and in the future :smile_cat:). While searching the net for answers to this question, I read that a nix hostname can't contain an underscore. I don't know if that's true, but Im guessing there are some restrictions like that.
I'd appreciate any hostname tips. Thanks!

Could you try something for me?  Could you remove your /etc/hosts file and restart bash.exe and see if the error goes away?  This will regenerate the /etc/hosts file with an entry that contains your machine name.
sudo rm /etc/hosts

Wow, that worked to solve the sudo apt-get update and sudo apt-get upgrade "unable to resove host" messages! 👏 🎆 Thank you!!
(I'll try to solve the nn problem some other way. ~25 years ago I was an nn expert, but in those days I only ran it while ssh'd or telnetted into a remote nix server and never had to think about or configure the hostname. Tips are welcome about this nn issue.)

Just ran into this issue myself, but solved it a different way:

Make sure you know what your hostname is by running cat /etc/hostname.
Open the hosts file in VIM sudo vim /etc/hosts.
Navigate to an empty row with your arrow keys and hit the i key to enter insert mode.
Type the localhost address followed by a space and your hostname exactly as it appears in the hostname file.
Hit the Esc key to switch back to normal mode.
Type :wq followed by enter, to write your changes and quit VIM.
Close bash and reopen it just for good measure.

GitHub Selfies

Per https://github.com/Microsoft/BashOnWindows/issues/125 you need change change the hostname in Windows. I tried changing it in Ubuntu and it does not persist.

WSL's /init has populated /etc/hosts for a while now. Calling this fixed on that basis. Changing the hosts in WSL won't change it in Windows. That's by-design for the time being; you are spiritually in a container and might have multiple WSL instances each with their own hostname.


Downloader progress bar issue

Actual Behaviour
There is an absurd behaviour in the progress bar on a certain followed pattern. The download progress bar refuses to close too.
Expected Behaviour
The progress bar should work fine.
Steps to reproduce it
1.Open the app.
2.Turn the internet connection off.
3.Turn it on again.
4. The download progress bar starts.
5. After it completes, turn the internet off again.
6. Turn it back on.
Screenshots of the issue

Would you like to work on the issue?
Yes.
Target Milestone to solve the issue
milestone 9. @iamareebjamal please add milestone.

@rohanagarwal94
are you working on this?

Yeah I am. Its just that its difficult to debug this. The download events are getting subscribed again and again. Will take help if I am unsuccessful on this.

@iamareebjamal @arp95 @shreyasnbhat @Shailesh351  need some help with this. As far as I debugged the issue, the problem is that the onReceive method is being called 4 times simultaneously. Hence 4 API requests by 2 different intent actions of the receiver are being made thereby causing the bug. Now I found a workaround to it - https://stackoverflow.com/a/8413512 but it seems not a good solution. Please suggest whether to use the work around or maybe any other solution which might work.

Either block API request if already in transit. Easiest and correct way to handle it
Or, cancel previous ones when a new one is made

Actually, the current interaction where network is checked via BroadcastReceiver is very convoluted, and besides that, app even sends ping to google to verify if connection is working. This creates complex problems and user needs to wait for whole round trip time before being able to do any action. I think we should only check by context is device is actually connected to internet at the time of action as retrofit already supports timeout.
@the-dagger What do you think?

I think we should only check by context is device is actually connected to the internet at the time of action as retrofit already supports timeout.

@iamareebjamal Sounds good to me.

@iamareebjamal the method is called 4 times when there is an active internet connection which is why the event bus results in 4 consecutive new download events.

@rohanagarwal94 Now as @the-dagger has given a go to it, you can use https://github.com/fossasia/open-event-android/blob/development/android/app/src/main/java/org/fossasia/openevent/utils/NetworkUtils.java#L34 instead

:+1:


No connection to GitHub

Logging into GitHub through PlantIT always shows the screen attached. Any help with that?

It was a GitHub OAuth app integration config issue. Sorry about that! Working for you now?


Warning: Object # has no method 'indexOf' Use --force to continue.

  	copy: {
  		files: [
  			{
  			expand: true,
  			src: ['index.html'],
  			dest: 'deploy/',
  			filter: 'isFile'
  			}
  	    ]
  	}

When i run grunt copy i get:

Warning: Object # has no method 'indexOf' Use --force to continue.

Any idea why i get this error?

+1. Missed to add 'target' key

newbie to grunt, same problem, solved by this thread Thx Guys


store quote type on the StringLiteral AST instance

Changes the Grammer to pass DOUBLE_QUOTED_STRING or
SINGLE_QUOTED_STRING instead of STRING, so that information
about the quote type can be passed to the AST nodes.
This can be useful in reprinting an AST without changing
the style of the original template author.
This is part of https://github.com/wycats/handlebars.js/issues/935

Weird that NVM is failing...

Closing as @mmun has convinced me this is a rabbit hole if we can't preserve whitespace.

@kpdecker I'm reopening as I got some guidance from Yehuda and Rob from the Ember Core Team from their recent work on HTMLBars. I think the printer is more feasible than I thought since the location info is already stored for all(?) node types.

Do you think we can introduce this in the 3.x series? The AST remains semver compatible but the tokens do not. I don't know how much people use tokens vs the AST.

I don't understand the point of this. If you're going to write a re-printer you're going to need to use location info + the original string input in order to preserve whitespace. In particular, you can already see the quotes by inspecting the location info of a string literal + the original string. What does this buy you?
I guess I am OK with storing the raw value on the AST since esprima does it for JS (probably for a good reason). e.g. 'foo'; 0; null; undefined; 5.6e7; parses as
{
    "type": "Program",
    "body": [
        {
            "type": "ExpressionStatement",
            "expression": {
                "type": "Literal",
                "value": "foo",
                "raw": "'foo'"
            }
        },
        {
            "type": "ExpressionStatement",
            "expression": {
                "type": "Literal",
                "value": 0,
                "raw": "0"
            }
        },
        {
            "type": "ExpressionStatement",
            "expression": {
                "type": "Literal",
                "value": null,
                "raw": "null"
            }
        },
        {
            "type": "ExpressionStatement",
            "expression": {
                "type": "Identifier",
                "name": "undefined"
            }
        },
        {
            "type": "ExpressionStatement",
            "expression": {
                "type": "Literal",
                "value": 56000000,
                "raw": "5.6e7"
            }
        }
    ]
}

@fivetanley I'm not a huge fan of adding parameters after the locInfo parameter but that is mostly an OCD concern relative to all of the other constructors that have it as the last parameter.
That being said, I think @mmun brings up an interesting point. Rather than complicating the parser (each  new production can add a fair amount of overhead), what if we added a raw value and move the escape handling into the StringLiteral constructor? I.e. something like
StringLiteral: function(string, locInfo) {
  if (/^"(\\"|[^"])*"$/.test(string)) {
    this.raw = string;
    string = strip(1,2).replace(/\\"/g,'"');
  } else if (/^'(\\'|[^'])*'$/.test(string)) {
    this.raw = string;
    string = strip(1,2).replace(/\\'/g,"'");
  }
...

And then in the tokenizer we would simplify to:
<mu>'"'("\\"["]|[^"])*'"'        return 'STRING';
<mu>"'"("\\"[']|[^'])*"'"        return 'STRING';

(Potentially either of these could be simplified further but I'm not sure how much O want to further tempt the regex gods)
This would break for cases where people are passing in a string that happens to parse to a string literal syntax so we might want to do this as a 4.x but I have no problem rolling a 4.x as long as we continue to document what the changes are and how people have to deal with them. Given the amount of hell I catch on every release I have no problem doing majors more.

sounds good


Readme update

Coverage remained the same at 100.0% when pulling db4ba5a20f113751879d250f4e1c4ed0f7b2cf41 on readme-update into d28462978433e6066a203d814be7533e5a89d9da on master.


feat(bridge-ui): code review and small fix

dialog close not working in StatusInfoDialog and code refactor

@defigeneration Either create an issue describing the bug and what you did to fix it or do it in the PR.


Sessions view: add explain plan

A good feature of the Sessions view could be to show the explain plan for the query currently executed.

Yep, that's a good feature. See it in 5.2.1


update_storage requires check for None

(in zerver/views/storage.py)
This is probably in your area, @roberthoenig?
Essentially, storage may be None, but this is not checked for in the function.

@neiljp  can I work on this? And please explain your issue

@neiljp which function?

When testing this file with mypy with the strict-optional setting (being worked on in one of my PRs #6017), storage is specified as being Optional[Dict[str, str] in update_storage but is used in the function before checking whether it is None.
The typing may be incorrect, or the function needs adjusting, by my reckoning; this may also require changes elsewhere.
I tagged @roberthoenig as most lines in this file are attributed to him according to git blame, however please feel free to explore this issue if you wish. I'm happy to chat about this on chat.zulip.org, if that would help.

@shreyanshdwivedi @dgarvit I'm going to assign this to @roberthoenig, as we need clarification here.

@zulipbot claim


[Kernel] w4a16 support for compressed-tensors

Summary

Adds an additional CompressedTensorsScheme to supoprt w4a16 models, CompressedTensorsW4A16
Add support for w4a16 models quantized through sparseml and saved through compressed-tensors
Leverages the kernels (gptq_marlin_gemm) used for gptq_marlin

From Neural Magic, Co-authored by @alexm-neuralmagic

In a follow up PR, we should support bfloat16 activations as well (which we support in marlin)


Updated: far 3.0.5484.1188

Newer version of far (3.0.5503.1263) is now available in master. e6b9eae4db64098a404e77c61e3124d8eedcb89e

Newer version of far (3.0.5503.1265) is now available in master. b0ccff130b92b8ea01515bc2dd390438e0566b29

Newer version of far (3.0.5503.1275) is now available in master. 804d05aeff09b84edbaf784e9c5944cd09e7e95b

Newer version of far (3.0.5503.1277) is now available in master. a5c3ca25caf2f96d94e788dec91db3cffde3e89e

Newer version of far (3.0.5504.1279) is now available in master. 0273dc59ec06f78c5af077dac0e455aecac5cb55


Fix #126: Implement location renaming

I realize now you probably don't have access to the editing interface on that GitHub Labels link. You can test them on one of your own repos, or see for reference these two screenshots:

Sounds good - I'll get on it!

I made the changes - let me know what you think. One thing in particular that I wasn't able to do is automatically focus on the input field when a user clicks "Edit". The changing DOM makes this tricky.

Oh, and to answer you question about focusing the input: we certainly have to use jQuery for this.
We could do it in the click handler, but run the risk of the UI not having updated by the time the code is run.
Alternatively, we can use the state to toggle a hidden-xs-up class on the divs so the UI elements always exist, and this way our jQuery will succeed for sure.

Thanks for the great comments! Two main things:

The text-right class didn't seem to have the desired effect, so I added an entry to the scss file where this class applies text-align: right. You might want to play around with it to see what's going on.
I didn't do the error display thing because it seems like an error would only surface if there's a network issue or something. It also seems like it would be a non-trivial addition so I figured it would be best to leave it to another issue/pull request.

Ah, I know what the problem is. The class was introduced as part of Bootstrap Alpha v6 and we're on v5. I've blocked a couple of other PRs as a result too. But let's merge this one and we can remove that class when we move on to v6.

Looks like the eslint checks are failing. Could you install an ESLint plugin for your text editor (or run it from the command line) and fix the errors please?

When I use eslint in other projects I can run it just by doing eslint . from the project root. Is this correct? When I try it for signmeup (after installing all the dependencies) I get this error regarding an improper configuration:
/usr/local/lib/node_modules/eslint-config-airbnb/node_modules/eslint-config-airbnb-base/rules/es6.js:
    Configuration for rule "arrow-parens" is invalid:
    Value "as-needed,[object Object]" has more items than allowed.

Referenced from: /usr/local/lib/node_modules/eslint-config-airbnb/node_modules/eslint-config-airbnb-base/index.js
Referenced from: airbnb
Referenced from: /Users/Greg/Documents/Programming/signmeup/.eslintrc.json
Error: /usr/local/lib/node_modules/eslint-config-airbnb/node_modules/eslint-config-airbnb-base/rules/es6.js:
    Configuration for rule "arrow-parens" is invalid:
    Value "as-needed,[object Object]" has more items than allowed.

Referenced from: /usr/local/lib/node_modules/eslint-config-airbnb/node_modules/eslint-config-airbnb-base/index.js
Referenced from: airbnb
Referenced from: /Users/Greg/Documents/Programming/signmeup/.eslintrc.json
    at validateRuleOptions (/usr/local/lib/node_modules/eslint/lib/config/config-validator.js:116:15)
    at /usr/local/lib/node_modules/eslint/lib/config/config-validator.js:163:13
    at Array.forEach (native)
    at Object.validate (/usr/local/lib/node_modules/eslint/lib/config/config-validator.js:162:35)
    at load (/usr/local/lib/node_modules/eslint/lib/config/config-file.js:523:19)
    at /usr/local/lib/node_modules/eslint/lib/config/config-file.js:392:36
    at Array.reduceRight (native)
    at applyExtends (/usr/local/lib/node_modules/eslint/lib/config/config-file.js:363:28)
    at load (/usr/local/lib/node_modules/eslint/lib/config/config-file.js:530:22)
    at /usr/local/lib/node_modules/eslint/lib/config/config-file.js:392:36`

Actually nevermind, reinstalling my node modules fixed this. However, I am now getting a lot of errors (669), most of which are in files that I didn't edit. Most of them seem to be issues with import statements. Should I just focus on the errors in the files I changed?

Yes, that should be fine! You can also see the specific issues here (ignore the Meteor resolver issues): https://lima.codeclimate.com/github/signmeup/signmeup/pull/160.

I fixed some of the linter errors, the ones remaining are related to imports and the use of confirm/console.error. Should any of these also be fixed? If so, I'm not quite sure how.

Sorry for the delayed response, Greg, midterms week has been busy. I wouldn't worry about the console / confirm issues. This is such a fantastic contribution, and I want to thank you for your patience with the various reviews. LET'S MERGE!


feat(Menu): remove window & add currentUrl to props

change

move currentUrl to Menu props
remove window reference for ssr compatibility

breaking change?

Yes, I think it would be. I was hoping adding the ? would not make ssr puke, but it did anyways.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Issue with wrong eager fetch reported

I have following Repository:
public interface ObjectRepository extends JpaRepository<ObjectEntity, Long>,
		JpaSpecificationExecutor<ObjectEntity> {

	@Query("select distinct so from ObjectEntity so join fetch so.objectAssignmentCounts where so.customerId= :customerId and so.id IN (:ids)")
	List<ObjectEntity> getObjectsById(@Param("customerId") Long customerId, @Param("ids") List<Long> ids, Sort sort);
}

In my test code I call my API several times which will call the ObjectRepository.getObjectsById with different paramerts and sorting.
I see that sometimes the testcase fails if running multiple tests in sequence.
It seems that the proxy method for the getObjectById is then the same but only the sorting is different.
Eg.: proxyMethodName= com.sun.proxy.$Proxy195getObjectsById
initial query:
select distinct ... order by obje0_.created_at asc
new query:
select distinct ... order by obje0_.name asc
This is then causing to increment the select queries count and report an error:
com/yannbriancon/interceptor/HibernateQueryInterceptor.java:212
selectQueriesInfoPerProxyMethod.put(proxyMethodName, selectQueriesInfo.incrementSelectQueriesCount());

2021-05-12 21:28:11.635 ERROR 4168c0df-cbe7-47d7-aa89-18ec834b214f  GlobalExceptionHandler:332 - N+1 queries detected with eager fetching on the entity com.test.jpa.model.ObjectEntity
    at com.test.service.QueryService.getObjects(QueryService.java:148)
    Hint: Missing Lazy fetching configuration on a field of type com.test.jpa.model.ObjectEntity of one of the entities fetched in the query

I can't reproduce it if I debug a single test or try to stop the debugger on this testcase.
I need to set a breakpoint on com/yannbriancon/interceptor/HibernateQueryInterceptor.java:212 while running tests in sequence.
I think it's a mix of a threading/testcase-seperation issue together with the sorting of spring data so that same proxy method is used.

I was able now to reproduce the issue on a single test case.
SpringBoot tests are using on my side by default 10 different http-nio-exec threads.
After 10 request the existing exec threads are reused with existing SelectQueriesInfos.
When now the same query is called on this thread with a different sorting then previously on an older api request then the N+1 query exception is thrown wrongly.


Treat boolean as string in Expr\Comparison

Hi,
I was getting an error when I try to use comparison with boolean false.
Here my issue: when you are calling $queryBuilder->set('o.enabled', false) then the result is o.enabled =  and then you have an SQL exception.
This PR allow to use boolean for comparison.
For example, you can now use ->set('o.enabled', false) instead of ->set('o.enabled', 'false')
The previous exemple will give o.enabled = false.
What do you think of it ?

I just closed an issue about this as "invalid": please refer to #5546 for further explanation.

Thanks for the link.


Compile loop because output directory is being included

Hello,
I'm having problems using the watch option of typescript compiler. The output directory is being watched/compiled which in turn triggers another compilation, that triggers another compilation, you get the idea right? Each time a compilation is triggered a new sub-directory is created that contains all the compiled code.
TypeScript Version:  2.1.5
Code
Actually I think it is better if I specify the configuration that I'm using
{
    "compilerOptions": {
        "compileOnSave":true,
        "watch": true,
        "module": "commonjs",
        "target": "es5",
        "allowJs": true,
        "noImplicitAny": true,
        "sourceMap": true,
        "strictNullChecks": true,
        "outDir": "./dist",
        "rootDir": "./src"
    },
    "exclude": [
        "node_modules",
    ]
}

Expected behavior:
I want the compiler to generate compiled output to the dist directory and ignore its content.
Actual behavior:
As I explained, it is compiling the compiled code on an infinite loop.
I think that I can just put the src directory on the exclude array, but it is not documented on any place that the exclude array can be used for this purpose.

should not the exclude be:
    "exclude": [
        "./dist",
        "node_modules",
    ]

Dear @mhegazy,
I also came to that conclusion, but it doesn't looks like a right solution because:

There is an option called outup dir. Shouldn't tsc ignore it by default ?
There is an option called rootDir. Shouldn't tsc process only the contents within that directory ?
The documentation about the exclude property is unclear , and seems that it only should be used for excluding node_modules and that kind of directories.

Checking the documentation makes me think stronger that this is a bug or an error on the documentation :

Files in the directory specified using the "outDir" compiler option are always excluded unless explicitly included via the "files" property (even when the “exclude” property is specified).

As you can see ,my assumptions were correct according to the documentation,so something is wrong here.
In my opinion this is a bug or an unexpected behavior.
Here is the documentation link
https://www.typescriptlang.org/docs/handbook/tsconfig-json.html

you have allowJs turned on. i.e. you are telling the compiler to pick up all .js files in your included files.  and your include/exclude pattern does not exclude the outDir. the compiler does not know what files it will generate, when it starts to look for files to process, since it needs the input to know what are the expected output. By default the exclude pattern includes outDir. but you overwrote that with one that only excludes node_modules. so options:

remove the exclude pattern, or just add outDir your self
add an include pattern to only include ./src/*
do not enable allowJs if you do not need it

you have allowJs turned on. i.e. you are telling the compiler to pick up all .js files in your included files.

I know that is what is causing it (can't call it "the problem"). But there is nothing on the documentation that specifies that you can not turn allowJs on and use the outDir,

By default the exclude pattern includes outDir. but you overwrote that with one that only excludes node_modules

I expected those two to be merged, not exclusive. Is there any place on the documentation that specifies that the outDir is ignored as exclude if you use the exclude property ? If it does, I missed it.

remove the exclude pattern, or just add outDir your self

If I just remove the exclude patter, will it pick up the node_modules folder? I guess the answer is yes.

add an include pattern to only include ./src/*

That seems to be the best option, but the documentation is a bit confusing about how and when you should use the include pattern.

do not enable allowJs if you do not need it

I have enabled JS because I want to use it. Maybe in a near future I could use just TS.
I'll try your suggestions and report back.


In persistStateMiddleware, return result action instead of input action

In order to make dispatch return the expected action, the persistStateMiddleware needs to return the result of next(action) rather than action itself.

Merged, thanks!


Out-of-bounds exception in io::gzstream

The underflow method in meta/io/gzstream.cpp addresses memory outside of the array bounds.  It generates an exception in Windows (MSVC).
The buffer size is 512 bytes.
 Thanks for catching this. The second line should read something like
setg(&buffer_[0], &buffer_[0], &buffer_[0] + static_cast<std::size_t>(bytes));

Would you like to make a pull request with the fix?

I'm sorry, my schedule, right now, prevents me from being a contributor.   I just wanted to save you the time researching such a minor bug.   Hopefully, when things settle down here and I can delve more deeply into using Meta, I can contribute.  Thanks!

Fixed in v2.4.2.


VLAZ: Watch (?-i:9wzHsbGf)(?#cutt.ly)

VLAZ requests the watch of the watch_keyword (?-i:9wzHsbGf)(?#cutt.ly). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?-i:9wzHsbGf)(?#cutt.ly) has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Fix dimension ids duplication

Details of main problem defined in issue #2206 and pr #2207, which was not fixed due to mods using negative dimensionIDs.
Since BitSet cannot work with negative dimensionIDs, it is neccessary to use IntSet.
All behavioral changes are taken from Forge 1.12 DimensionManager.class
I don't know why forge marks "DimensionArray" compound as legacy data and uses "UsedIDs" compound.

Thanks a bunch @GlassSpirit for this PR. Appreciate it.

@GlassSpirit Thank you for your contribution
This is missing a SpongeForge PR.
We're shadowing the BitSet here.
You need to shadow the IntSet and update the method using that field.

Does that mean that I need to make another PR to SpongeForge?
Or I must close this and open a PR directly in SpongeForge with all changes?
I think I don't know how to make a pr to SpongeForge without my changes in SpongeCommon...

Does that mean that I need to make another PR to SpongeForge?

yes

I think I don't know how to make a pr to SpongeForge without my changes in SpongeCommon...

SpongeCommon is a submodule of SpongeForge.

Clone SpongeForge
Checkout fix/dimension-ids-duplication (your branch) on SpongeCommon
Create a new branch on SpongeForge and fix the issue with old shadowed field


API window.visibleTextEditors unintuitive behavior

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.56.2
OS Version: Win 10 (19042.985)

Steps to Reproduce:

Run window.visibleTextEditors with one document open in 2 editor groups and no other documents open
It returns a list of two editors. Fine
Run window.visibleTextEditors with one document open in 2 editor groups with another document open
It returns a list of two editors while it should be three

I would expect one document open in multiple editor groups to each have its own entry in the list of visible editors. They're condensed to one editor if any other document is open.

Never mind, I understand what the list is now. I thought it's a list of open Editors


fix readme typo

Nitpicking: npm should be lowercase.
https://docs.npmjs.com/misc/faq#is-it-npm-or-npm-or-npm
Interesting project, by the way. I'm interested to see where it goes. Thanks for making it.

Hey Dustin, thanks a lot! I'm glad you noticed the error.
Thanks for the feedback too. If you want to add any feature if you find some way that you could use the tool, by all means do so.


Dump guest physical and linear address from VMCS when dumping state

These may or may not be valid for a particular fault but other
information which is already logged (such as the vmexit qualification)
should allow a developer to work out if the values are valid on a case
by case basis.
Signed-off-by: Ian Campbell<EMAIL_ADDRESS>

LGTM


Update README.md

plugin format is deprecated, grpc-go is its own protoc plugin now.

more context: https://github.com/golang/protobuf/issues/1070
I can also update the examples if you want.


[Bug] XSOverlay notifications not being pushed

Describe the bug
Notifications aren't being shown on XSOverlay unless manually pushed via devtools
AppApi.XSNotification("VRCX", "test", 5, "");
To Reproduce
Steps to reproduce the behavior:

Reset VRCX by removing all files in appdata
Set Notifications > Notification Filter to Everyone/On

Expected behavior
XSOverlay notifications to function correctly
What version you are running
VRCX 2024.05.09
XSOverlay Build 667

Is your in-game status set to busy?


Made role a more generic

Added a possibility to config extra exec commands.  Made repo management
optional. Made collectd_process_slurmstepd optional.
Defaults variables were added so that default behaviour didn't change.

This role is a bit messy.
I'll close this PR and make the epel, slurm and processes things optional.
To configure an exec plugin using the macro one would define a variable like this:

collectd_plugins_exec_db:
  - options:
    - LoadPlugin: exec
  - sections:
    - name: Plugin
      param: exec
      content:
       - options:
         - Exec:
           - nobody
           - /usr/local/bin/collectd_postgresql_db_sizes.sh
         - Exec:
           - nobody
           - /usr/local/bin/check_dcache_pools.py

and then add it to the default or custom plugins:

collectd_plugins_default: "{{
  collectd_plugins_cpu +
  collectd_plugins_load +
  collectd_plugins_procs +
  collectd_plugins_exec_db +
  collectd_plugins_memory
}}"

Then this variable will pick it up:

collectd_config:
  content: "{{
    collectd_global_options_custom +
    collectd_plugins_default +
    collectd_plugins_custom
  }}"

and then this part of the template will write it:

###
{% from 'encoder/macros/apache_encode_macro.j2' import apache_encode with context -%}
{# This encoder was taken from
# https://github.com/picotrading/ansible-collectd/blob/master/templates/collectd.conf.j2 #}

{{ apache_encode(
     collectd_config,
     convert_bools=true,
     convert_nums=true,
     quote_all_strings=true) }}
###