Ionide hangs when writing a comment having a certain format.

Describe the bug
When writing a comment a text having a format //.*``.*\(.*\).*``.+, Ionide slows down and hangs in the end.
To Reproduce
Steps to reproduce the behaviour:

Open any F# file.
Start commenting.
Input ``()`` and keep pushing a. It will look like  //``()``aaaaaaaaaaa....
VSCode hangs.

Expected behaviour
Nothing bad happens.
Environment (please complete the following information):

OS: Debian GNU/Linux buster/sid
Ionide version: 3.35.0
VSCode version: 1.33.1
dotnet SDK version: 2.2.203
mono / .Net Framework version: Mono <IP_ADDRESS>

Additional context
This bug does not happen while using fsharp-language-server, so I think it is somewhat related to Ionide.

Just to confirm that this also happens if you don't comment i.e. you can omit step 2 above.

At first, I would have said that the issue is coming from the grammar.
But after checking grammar of both fsharp-language-server and Ionide they are almost the same.
fsharp-language-server is using ionide-fsgrammar too but trailling a bit behind
The only difference between both grammar is this one:

Diff online
As you can see it's just adding support for static let mutable.
And if you say that  fsharp-language-server doesn't have the problem. Then the grammar is probably not the source of the issue.

@MangelMaxime I tested it right now and I'm pretty sure fsharp-language-server does not have the problem. Hope this helps :pray:

I can't repro it with Ionide 4 so I think it has been fixed.

hmm, I just reproduced it with Ionide 4.0.4, .NET Core 3.0.100, VSCode 1.36, Debian buster/sid.
I also confirmed that fsharp-language-server does not have the problem as in before.
Here is an interesting screenshot: when the problem is happening, the code process is eating up the CPUs while the dotnet process (FSAC) is not.

When I input aaaaaa..., the cpu usage of dotnet process spikes for a very short period of time and returns to normal, while code process uses considerable CPU % for much longer time.
Hope this helps!

@cannorin You may have found a new issue there.
I was running the experimental branch (apparently 0.3.5) and I was able to reproduce this still. I then uninstalled and installed the main branch (4.0.4) and the issue was gone. When I uninstalled and re-installed experimental 0.3.5, the issue is still gone... maybe try having a play with re-installing or swapping the extension out?
Thanks Krys or whoever fixed this!

@drk-mtr I can reproduce that. Reinstalling and swapping the extension temporarily solves the problem, and the fresh install don't have the problem, but it's back after restarting VSCode in both cases.

I still can repro, on latest VSCode Insider + latest Ionide

I did another test in order to isolate the problem, if I deactivate the F# grammar using:
{
    "name": "fsharp",
    "scopeName": "source.fsharp",
    "fileTypes": [
        "fs"
    ],
    "foldingStartMarker": "",
    "foldingStopMarker": "",
    "patterns": [

    ],
    "repository": {
    }
}

The problem still exist, which confirm that the grammar is not the problem.

I'm having this issue as well. Any non-alphanumeric character while writing between   hangs VSCode.
I'm guessing, since this has been open for over a year, that no fix is incoming?

@clindholm I guess we sinply don't know why this happens?

@cannorin I did some testing to see if I could narrow it down a bit more. There is no problem opening files that already have extra characters in backtick functions (I don't know if they have some special name). I disabled autosuggestions which didn't resolve the issue, but it seems to me that it made it less severe. Where before, I had to force quit vscode, it now sometimes become responsive after a few seconds. It freezes when I click anywhere inside the backtick fucntion name. And then if it unfreezes, the whole name is highlighted. So maybe it has something to do with the code that decides the highlight range of a function name?

A few updates on this:

I still get the issue with Ionide v5.2.0
It occurs on macOS and Linux
Happens most often when writing test names. Seems to hang when in the cursor is in the middle of the backticks name:

open Xunit
open FsUnit.Xunit

[<Fact>]
let ``hangs when changing the test name`` () =
  true |> should equal true

After restoring VS Code from the crash, it will lock-up again. Editing the file in Vim and then restoring VS Code will make the issue go away.

A few updates on this:

I still get the issue with Ionide v5.2.0
It occurs on macOS and Linux
Happens most often when writing test names. Seems to hang when in the cursor is in the middle of the backticks name:

open Xunit
open FsUnit.Xunit

[<Fact>]
let ``hangs when changing the test name`` () =
  true |> should equal true

After restoring VS Code from the crash, it will lock-up again. Editing the file in Vim and then restoring VS Code will make the issue go away.

Since this was happening to me quite often, and it seems to be a hard bug to trace, I left the F# output open while editing.
Whilst working on some tests (tests are named with backticks like ``this is my test``) it happened again!
Here are the last lines of the output before it froze:
[14:39:20.170 VRB] [Checker] Current Queue Length is 0
[14:39:20.170 INF] [Checker] TryGetRecentCheckResultsForFile - "/home/path/to/file/Test.fs"
[Error - 14:39:20] Request textDocument/hover failed.
  Message: Cannot find ident for tooltip
  Code: -32603

Happy to provide mode details.

Same thing is happening to me. Somehow, it will also hang other vs code instances. A way to easily trigger this is to add a ; at the end of a test like such:
let ``this is a test;`` () =

The following consistently reproduces the problem for me:
Steps

Create a new xUnit project. dotnet new xunit --lang F#.
Start VS Code, code .
Wait for Ionide to load the project.
Change the name of the unit test to something like the following: When naming the unit test do not include a comma in the name
Insert a comma into the name.

Expected Behavior
VS Code editor does not hang.
Actual Behavior
VS Code editor hangs and a dialog appears stating, "The window is no longer responding."

Workaround
Do not include commas in the unit test names.
Environment

Windows 10
VS Code 1.56.2
Ionide-fsharp v5.5.5

Still happens

In my steps to reproduce above, I had this workaround:

Do not include commas in the unit test names.

Today, hyphens are causing this same hang behavior for me. For example,
let ``Get does not throw for non-existent id ()`` =
Maybe I should have said:

Do not include ~commas~ non-alphanumeric characters in the test names.

Apologies for the off topic comment but I really appreciate how people are posting genuinely useful debugging info above, and no irate "still not fixed!?" or any comments like that, the sign of a good community, full respect!

In my steps to reproduce above, I had this workaround:

Do not include commas in the unit test names.

Today, hyphens are causing this same hang behavior for me. For example, letGet does not throw for non-existent id ()=
Maybe I should have said:

Do not include ~commas~ non-alphanumeric characters in the test names.

You saved my day! Apparently, a hyphen (-) inside ``…``` made VS Code crash every time I opened the file. Now that I have removed the hyphen, everything is back to normal.

After returning to my project after a few months, I noticed a new problem with VS Code hanging when I open an F# file with tests. My tests use double-backtick quoting for names. At first I thought it was this same issue, but now I'm not sure. I definitely had problems that match the descriptions here in the past until I took out special characters from my test names (everything except letters and spaces) and then it was fine. The new problem seems a bit different, however it's related closely enough that I want to add it here first.
The new problem seems more like it has to do with the number of characters after the backtick qouting, even though it's not part of the backtick quoting. The problem does not occur when Ionide is disabled and occurs when it is the only extension enabled. After some trial and error, I've found what seems to be the minimal repro.
Repro:

Create new F# library with dotnet new classlib -lang f#
Open folder in VS Code
Open Library.fs and erase everything but the namespace
On a new line, type a double-backtick (erase the closing double-backtick if it adds it automatically) then type letters until you observe VS Code become unresponsive and hang. For me it starts to lag at text column 29 and hangs completely at column 35.

Observations

The issue still occurs if the line is commented out beforehand with // or (* *)
If you have the closing double-backticks, you can type all the text you want between them with no problem. However if you add text after it, you get the same problem
The slowdown/hang occurs not just when the cursor is on the line with the backtick quoting, it happens when it is merely on the screen at all

@mattstermiller Well done! I followed your instructions and see the exact same behavior... the lag becomes noticeable around column 29.

There is some interesting data if you take a dump of processes when this hang occurs - last time I took a look (~2 month ago), The FSAC process (which handle typechecking and some semantic highlighting) was at <2% CPU, and the VSCode renderer process was going haywire.
This signals to me that the textmate regexes might bear investigations again. I know in the past we've tried to eliminate it from consideration, but that's just what I saw.

Hello @baronfel,
I re-tested different configurations:
Test scenario

Open a file test.fsx which contains the following code
let ``Get does not throw for non-existent id ()`` =
    ""

If using Ionide wait for it to kicks up (see the // string inline hint) or have the hover feature working

Place the cursor in the middle of the function name

Add some random text to it

Ionide from marketplace 7.0.0 (standard usage)
Results:

When placing the cursor in the function name there is a freeze.
Writing letter per letter is freezing for 5-10sec and then ok
Writing normally make the application freeze and ask we want to make it crash/reopen.

Note: We can't really test Ionide without a grammar because if we disable it by editing the configuration of from the marketplace extension there will still be the default one from VSCode loaded.
Grammar without Ionide
Results:
Everything is working fine
F# Language Server updated from marketplace 0.1.82
Results:
Everything is working fine
dotnet fsautocomplete + Sublime Text LSP
Results:
Everything is working fine

@baronfel
Compare to previous assumptions it seems like the problematic combination is Ionide + VSCode.
As FSAC seems to be working correctly when used from Sublime text.


window.cookie.initialise is not a function

Laravel Framework: 5.8.24
PHP Version: 7.2.19
Describe the bug
Cookieconsent not working installed via npm.
I installed cookieconsent with npm and included in my project ( main app.js ) :
window.cookieconsent = require('cookieconsent');
And at bottom wrote this code:
window.addEventListener('load', function(){
        window.cookieconsent.initialise({
          "palette": {
            "popup": {
              "background": "#3c3c3c"
            },
            "button": {
              "background": "#21859e"
            }
          },
          "position": "bottom-left",
          "content": {
            "message": "Informuojame, kad šioje svetainėje naudojami slapukai (angl. Cookies). Sutikdami, paspauskite ,,Sutinku” arba naršykite toliau.",
            "dismiss": "Sutinku",
            "link": "Plačiau",
            "href": "/privatumo-politika"
          }
        })
    });

In console I always get window.cookieconsent.initialise is not a function.
But if I load your JS via CDN, it works perfectly. Any suggestions?

Requiring a javascript on the client side is not the same as requiring in a node project.
The correct way to implement cookie consent when you build via NPM is to add a <script> tag with an src to the build/cookieconsent.min.js file. From the way you describe your use case, I don't see a real benefit to loading the NPM copy versus loading the CDN copy. With the CDN version, you will get non-breaking fixes automatically whereas with NPM you have to keep your code up to date.
Generally, people use the NPM version when they want to compile the script into a single script file along with all of their other client-side code.
If however you absolutely must use the require format then I would recommend looking at one of the numerous libraries that add require capabilities to client-side scripts such as Browserify although be forewarned that this now adds additional weight/load-time to your page.


Is there a demo?

dear
I want to use SM2 and RSA,but I dont konw how to use bouncycastle for my program.So,could you please provide SM2 and RSA demo code for encrypting and decrypting and adding signatures and verifying signatures？Or what way to get these demo code?

You can find plenty of examples in the test code for the various APIs. There is also further documentation at: https://www.bouncycastle.org/documentation.html


Service fabric api version 2020-03-01 adding missing properties

we recently added this version in this pr #8746 but missed the following properties:
serviceDnsName to service,
instanceCloseDelayDuration to stateless service,
applicationTypeVersionsCleanupPolicy to cluster
This api version has not been released yet so it will not afect any customer.
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

@a-santamaria , can you fix the model validation check failures about readonly properties not allowed in example response?

/azp run automation - sdk

@a-santamaria , can you fix the model validation check failures about readonly properties not allowed in example response?

@raych1 fixed the model validation thanks

@akning-ms , please force merge this as this version is not released yet.


about annotation_dir_path

in that shell
"annotation_dir_path includes brush_hair_test_split1.txt, ..."
i can't run it correctly.please explain the 'brush_hair_test_split1.txt',before we run the shell,it has exist?

question update
In annotation_dir_path ,I have benn add brush_hair_test_split1.txt, ...
But when i run shell code,nothing happened,no json  format file.please tell me what can I do.

你好，想问一下你是怎么解决的，annotation文件夹是可以在网上下载的吗

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

hi, similarly confused about this issue, where was this file generated?

你好，想问一下你是怎么解决的，annotation文件夹是可以在网上下载的吗

hi, have you solved it? I also want to know how to generate it.


No icon shown in extension viewlet / editor when falling back to older version

Found and discussed with @sandy081 when publishing a newer version of Azure Account that was not compatible with the current VS Code version.

@sandy081 Need verification steps.

I would leave @chrmarti to verify this as he created.


Special case error for .glksave files

It's not likely that a save file will get loaded into Quixe. But in the Lectrote app environment, the user could double-click or drag a save file in.
Right now the error message is "Blorb file contains no Glulx game!" We could be more specific.

Done.


[DGS] Fix License formatting

Type of request

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Refactoring
[x] Documentation or documentation changes

Related Issue(s)

Concept

Description of the change

Description of current state
Description of target state
Estimated investment / approximate investment

Initiator
name / company / position / created at
Technical information
Platform / target area

known side-effects
other
DB Changes (if so: incl. scripts with definitions and testdata)
How has this Tested?

Screenshots, Uploads, other documents (if appropriate):

Checklist:

[x] My code follows the code style of this project.

[x] I have read the CONTRIBUTING docs.
[x] I have added/updated necessary documentation (if appropriate).

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


[BUG] : Documentation

Description
Documentation is not containing redux as technology.

#49 Please review it!

closed by #49

closed by #49


griddle-react + react 0.14.0-rc1

Griddle can be used with react 0.14rc1?
npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>
Regards.

I'm also seeing:
Warning: validateDOMNesting(...): <span> cannot appear as a child of <table>. See Router > RoutingContext > SiteIndex > Table > Griddle > GridTable > table > span.

I just installed Griddle version 0.2.14 from npm and it seems to work with react 0.14rc-1. But i just did a base table for now

React 0.14 is out!
https://facebook.github.io/react/blog/2015/10/07/react-v0.14.html

0.14 is working for me locally -- hope to have a version of Griddle pushed to the master branch / npm using that before too long. Are you encountering issues with it still?

I see Warning: validateDOMNesting(...):  cannot appear as a child of . See Router > RoutingContext > Connect(VerificationGrid) > VerificationGrid > Griddle > GridTable > table > span.
Can you push to npm as soon as possible? Thanks.

Yeah, looks like you'll need to refactor the DOM node stuff a little bit too.
Warning: ReactDOMComponent: Do not access .getDOMNode() of a DOM node; instead, use the node directly. This DOM node was rendered by `GridTable`.

Changing Rows per page in settings also causes TypeError: Cannot assign to read only property 'resultsPerPage' of #<Object>

I've got something with rendering GridRow:
Invariant Violation: Objects are not valid as a React child (found: object with keys {$oid}). If you meant to render a collection of children, use an array instead or wrap the object using createFragment(object) from the React add-ons. Check the render method of GridRow.

I getting the same issue as @ThePainnn . Reason for me is my application use React 0.14.1 but Griddle strictly depends on 0.14.3. Any reason it should be necessary 0.14.3?

I am still having this issue.. getDOMNode is deprecated.. please use ReactDOM.findDOMNode().

We recently release 1.0 and this should no longer be an issue. Please feel free to open a new issue for any problems that you are seeing. Thanks!


Redesign Peer management/punishment

Codecov Report

Merging #46 (3c9e525) into master (8e39d11) will increase coverage by 0.16%.
The diff coverage is 75.83%.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
+ Coverage   78.07%   78.24%   +0.16%
==========================================
  Files          69       70       +1
  Lines        4392     4389       -3
==========================================
+ Hits         3429     3434       +5
+ Misses        963      955       -8

Impacted Files
Coverage Δ

src/config/node.rs
40.00% <ø> (ø)

src/node/api/transact_contract_payment.rs
0.00% <0.00%> (ø)

src/node/api/transact_zero.rs
0.00% <0.00%> (ø)

src/node/mod.rs
61.21% <46.66%> (-2.34%)
:arrow_down:

src/node/heartbeat/log_info.rs
63.33% <50.00%> (-2.19%)
:arrow_down:

src/node/api/get_peers.rs
75.00% <60.00%> (-25.00%)
:arrow_down:

src/node/heartbeat/sync_blocks.rs
63.35% <61.53%> (+0.67%)
:arrow_up:

src/node/heartbeat/sync_state.rs
86.84% <66.66%> (-2.35%)
:arrow_down:

src/node/test/simulation.rs
91.04% <66.66%> (ø)

src/node/heartbeat/discover_peers.rs
86.36% <85.00%> (-13.64%)
:arrow_down:

... and 12 more

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


[gen_l10n] Document messages with translations for supported locales

Consider this localization file:
{
    "@@locale": "en",
    "@@last_modified": "2020-10-03T09:49:55.929624",
    "helloThere": "Hello there {name}",
    "@helloThere": {
        "placeholders": {
            "name": {}
        },
        "description": "Lorem ipsum...",
        "type": "text"
    },
}

It will generate this property:
// Lorem ipsum...
String heloThere(Object arg0);

But it would be much nicer if it also added documentation comment with translations:
/// en: Hello there {name}
/// Lorem ipsum...
/// de: !!! translation missing !!!
/// pl: No elo {name}
String heloThere(Object arg0);

This way, IDE will guide you as your are writing widgets, and you can now easily find missing translations with search functions.
@shihaohong Does this sound good? If yes, I could try to implement this and submit PR
Related ticket: https://github.com/flutter/flutter/issues/41437

Also, current @comment logic does not handle new lines. On one hand, it's a bug, on the other hand, I can use it to insert the documentation I want here.

@shihaohong could you take a look?

I like the idea in concept, but applications that support a larger number of translations would have a massive number of comments. For example, the gallery supports dozens of locales and the comments for each message would be extremely long if we wanted to document every single translation for each message.
To clarify -- Is the issue here difficulty in figuring which translations are missing? There's an option called --untranslated-messages-file that helps list out the messages that are missing translations and for which locales.

Good point. For me the most common case is two supported languages (english + developer's native language), so putting all translations there made sense.
So instead we could put only template language + info for missing translations + maybe some docs about placeholders.
This issue is not really about untranslated messages, I just mentioned it as it seemed nice to have. Actual issue is about having clear information about given localization string without looking up the .arb file or generated code. When writing widgets it's not always clear which text will be used.
BTW, I'm aware of --untranslated-messages-file it's printed every time I'm running the app, but I don't really know which command to use. flutter pub get --untranslated-messages-file x.txt doesn't work.

Actually, that's a bug and has been printing incorrectly. I just submitted https://github.com/flutter/flutter/pull/68553 which should fix that issue.
I believe the issue you're facing here is that you're using --untranslated-messages-file correctly, but since you have all translations completed, the file is just not generating. I've made an adjustment in the same PR that causes the tool to generate an empty JSON instead of not doing anything in the event that all messages for every locale have existing translations.

Cool. Then this ticket is mostly about adding some comment docs, could be as simple as:
/// en: Hello there {name}
String helloThere(Object arg0);

I'll take a look into what we can do to make the docs helpful without cluttering it with too much noise. Maybe I'll start with the simple string getters first as a start and iterate to address the more complex ones, like methods with placeholders.

@szotp I created a PR that adds a new comment line with the message value in the template arb locale. Take a look and provide any feedback if you have any!


Add ScriptCallerAliases

Issue #2320
Hello, please review.
I'm not sure about naming and place where I put that Alias.
Also there is problem with calling: when it calls like Context.GetCallerInfo(); everything works as expected  but when it calls without explicit use Context (just GetCallerInfo();) it doesn't. I have no idea what is the issue. Could you please suggest?
var buildInfo = Context.GetCallerInfo();

Information(
    "build.cake: {0}",
    buildInfo
);

var myMethodInfo = MyMethod();
Information(
    "Got called from: {0}",
    myMethodInfo.MemberName
);

public ScriptCallerInfo MyMethod()
{
    return Context.GetCallerInfo();
}

Thank you in advance.

@Marusyk your changes have been merged, thanks for your contribution 👍


feat(graph): Added minimum height trees.

Coverage increased (+1.2%) to 100.0% when pulling 4428af4dbb40be45f8b6782d1c5194eff1e344b0 on minimum-height-trees into 538b5115d3a5e294f2f1a973f4faa6c9ab2e28ff on master.


Discussion, drop vs block on logger

From here: https://github.com/aspnet/Logging/pull/468#issuecomment-236306894
I would prefer to block instead of dropping message but let's discuss it

@victorhurdugaci Let's have a look when you're back.

If you head towards the "drop" policy, Serilog.Sinks.Async implements this in BackgroundWorkerSink which can be applied with:
    .WiteTo.Async(a => a.SomeOtherSink())

Functionally, this is pretty much identical to the background sink here. Just thought I'd leave a note in case you're able to make use of it, as any feedback would be valuable to us. Cheers!

Thanks @nblumhardt, I'll take a look at that. Maybe I can replace our background sink completely with that implementation.

fyi @pakrym

Closing this as we're preparing to move this repo over to Extensions.


Unable to upload .exe or .dll files

While following the tutorial, having issue when trying to upload the .exe file.
No error shown, but the uploaded .exe does not appear in the file list. (Refreshed many times but no help.) Same applies to .dll files as well. Tried to change the extension of .exe to upload, but no success. Looks like somehow the metadata is being detected. And no issues when uploading other files like .txt or .json etc.
(Btw, using a corporate Azure subscription for the demo.)
Could there be a setting I'm missing while following the tutorial?
Thanks.

The file has to appear on the file list on the right side. That's correct.
Maybe you did not specify "Windows" while creating the function (I did not try this. However, I tried a lot of alternatives and the file always appeared).


test_disabled_ring EEXIST

Hello.
Running the test msg-ring.t with runtests-loop.sh, eventually the cqe->res will be -17 (EEXIST) in test_disabled_ring.
This happens with the newest kernel and liburing version.
That's all.
Bye

What does "newest kernel" mean exactly? And can you post the exact output, there are two ways this test case is run. Can't seem to reproduce it here, so more details on system/etc might be useful.

Running test msg-ring.t                                             0 sec [0]
All tests passed
Finished loop 476
Running test msg-ring.t                                             wait cqe -17
cqe res 32
test_disabled_ring failed
ring flags defer failed
Test msg-ring.t failed with ret 1
Tests failed (1): <msg-ring.t>
Tests failed at loop 477

liburing is 1cbd4cffa881772d54d7d1a98755101253a66d0a
linux is 6.9 rc4
I was wrong though, wait cqe return -17.

I think I got the gist of it.
In test_remote, the aim is to setup a sqe in ring1 with io_uring_prep_msg_ring and pass the fd of the target ring2, thereby messaging ring2. So a pthread is created that runs wait_cqe_fn, waiting for messages on ring2. Under whichever circumstances the io_uring_wait_cqe in the pthread fails with EEXIST and leaves the message on ring2. Then we pass ring2 to test_disabled_ring, which incorrectly reads the message that was left on ring2.
What's more interesting now is why io_uring_wait_cqe fails with EEXIST. There are only few instances of EEXIST in your code and most of them do something along the line of
static int __io_run_local_work(struct io_ring_ctx *ctx, struct io_tw_state *ts,
			       int min_events)
{
	struct llist_node *node;
	unsigned int loops = 0;
	int ret = 0;

	if (WARN_ON_ONCE(ctx->submitter_task != current))
		return -EEXIST;

I think that's all I can do. Bye

Ok, here goes nothing. The issue is your basic race condition. If the pthread running wait_cqe_fn is slower than test_remote, it works out. If wait_cqe_fn is faster, which happens once every few thousand iterations, it needs to make the syscall io_uring_enter to actually wait for cqes that would have otherwise already been created. In the kernel, io_uring_enter calls io_cqring_wait, which in turn calls io_allowed_run_tw.
static inline bool io_allowed_run_tw(struct io_ring_ctx *ctx)
{
	return likely(!(ctx->flags & IORING_SETUP_DEFER_TASKRUN) ||
		      ctx->submitter_task == current);
}

I don't know how much clearer it could be. Calling io_cqring_wait to get cqes is explicitly forbidden when IORING_SETUP_DEFER_TASKRUN is set and the submitter_task is different from the task calling io_cqring_wait. It even says something in the manpage about how if IORING_SETUP_DEFER_TASKRUN is set, the task getting the cqes has to be the one issuing them.
The only reason this test worked at all is because the race condition with the pthread running wait_cqe_fn is usually slower and the thread didn't need to actually call io_uring_enter. The nature of race conditions is also what made it difficult for you to reproduce the bug.
If you put sleep(1) after pthread_create(&thread, NULL, wait_cqe_fn, target) in test_remote, the thread will be faster and you'll be able to see the bug for yourself.
Maybe you wanna fix this, I don't know...

Pushed a potential fix for this, does it work for you?

Yup, all good. Thanks for bearing with me.

Thanks for reporting! I'll close this one up then.


Benchmark to compare transport/http to bindings/http

This benchmark emulates a typical message forwarding situation and doesn't pass through the network (the http client is mocked for this purpose), http methods are called directly and mocked.
This could be used as a comparison tool to improve further both transport/http & bindings/http
I've also included an example gnuplot script to graph the results
Signed-off-by: Francesco Guardiani<EMAIL_ADDRESS>

cc @alanconway

LGTM
I look forward to using this :D


Question about rotations in Embodied Question Answering task dataset

I wanted to check if the rotations in the embodied question answering matterport task dataset are in the format: [w, x, y, z]? Looking at the values, that seems to make sense, but I saw in another discussion of rotations that habitat uses [x, y, z, w], so I'm unsure.

No, they are also in [x, y, z, w]

Thank you for the quick reply! I'm still a little confused about interpreting them--haven't worked with quaternions before. Would they correspond to a + bi + cj + dk as w = a, x = b, y = c, z = d? And i,j,k are unit vectors from a global origin defined by the mesh? or would it be cos(w/2) + sin(w/2)(xi+yj+zk)?

The former.  However, the i,j,k unit vectors are in habitat-sim's coordinate frame, which is +Y up and -Z forward while the MP3D meshes are +Z up (they don't have a defined notion of forward but +Y gets mapped to -Z)


wpebackend-rdk: bump the version, fix deps, remove unnecessary patches.

I tried your version and I can build the image just fine but when I try to run cog and press any button on the keyboard I get:
xkbcommon: ERROR: Failed to add default include path /usr/share/X11/xkb
Segmentation Fault

Is there just a configuration missing or is another dependency missing?

@miikeat is that directory present on your target device? Is libxkbcommon installed?

@philn the directory is not present, but when I create it I get the following error output:
xkbcommon: ERROR: Couldn't find file "rules/evdev" in include paths
xkbcommon: ERROR: 1 include paths searched:
xkbcommon: ERROR:       /usr/share/X11/xkb
xkbcommon: ERROR: 1 include paths could not be added:
xkbcommon: ERROR:       /home/root/.xkb
xkbcommon: ERROR: Couldn't look up rules 'evdev', model 'pc105', layout 'us', variant '', options ''

I'll try now the current master as this PR got already merged, but the last time I tried to build with master it failed as well but because it required a newer version of gstreamer which is not available in sumo...


Please check column exist on entity

Please check or exclude Query.orderBy, ExpressionList.eq etc API path/property on entity.
If check, please throw PropertyNotExistsException.

And please throw a CheckRowCountException.
now, i'm use this try cach

if (e instanceof OptimisticLockException) {
            if ("checkRowCount".equals(e.getStackTrace()[0].getMethodName())) {
                if (process != null)
                    return process.call();
            }
 }

Thanks!

So no that is actually a feature allowing any DB functions to used
naturally within the expression language. I won't change Ebean to throw a
PropertyNotExistsException and instead allow the query to execute and fail
with some DB error unless there is some very good reason.
I don't understand the CheckRowCountException, you'll need to explain that
better with a use case.
Also note that you should not put 2 separate issues into a single ticket.
There should only be 1 issue raised / resolved per ticket.
Cheers, Rob.
On 13 May 2015 at 02:07, icode<EMAIL_ADDRESS>wrote:

Please check or exclude Query.orderBy, ExpressionList.eq etc API
path/property on entity.
If check, please throw PropertyNotExistsException.
2.
And please throw a CheckRowCountException.
now, i'm use this try cach
if (e instanceof OptimisticLockException) {
if ("checkRowCount".equals(e.getStackTrace()[0].getMethodName())) {
if (process != null)
return process.call();
}
}
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/ebean-orm/avaje-ebeanorm/issues/291.

1rd issue, can provide a utils check this?
OK, please see #292 2rd issue .

I set orderBy from JAX-RS query params, so i need a info for front-end user DB error.

I set orderBy from JAX-RS query params, so i need a info for front-end user not a DB error.

Ok I understand now why you are wanting this feature. So perhaps have a 'strict mode' on the Query or a separate utility to validate properties/expressions.

thanks, but  'strict mode' need support fragment Expression e.g. query.where().eq('c1',1)...strictFragment().eq('notExitst',0). Then only check strict fragment expression.
thanks.

So I'm adding 2 mechanisms.

Query.validate()

    Query<Customer> query = Ebean.find(Customer.class)
        .where()
          .eq("junk","blah")
          .eq("name","jim")
        .orderBy("id desc,path.that.does.not.exist,contacts.group.name asc");

    Set<String> unknownProperties = query.validate();
    assertThat(unknownProperties).isNotEmpty();
    assertThat(unknownProperties).hasSize(2);
    assertThat(unknownProperties).contains("junk","path.that.does.not.exist");

SpiBeanType.validateExpression()

    EbeanServer defaultServer = Ebean.getDefaultServer();
    SpiServer pluginApi = defaultServer.getPluginApi();

    SpiBeanType<Customer> custType = pluginApi.getBeanType(Customer.class);

    // these properties / paths are valid (for Customer)
    assertTrue(custType.isValidExpression("name"));
    assertTrue(custType.isValidExpression("contacts.firstName"));
    assertTrue(custType.isValidExpression("contacts.group.name"));

    // these properties / paths are not valid (for Customer)
    assertFalse(custType.isValidExpression("junk"));
    assertFalse(beanType.isValidExpression("Name"));
    assertFalse(custType.isValidExpression("contacts.name"));

In the case of needing support fragment Expression e.g. ... you can remove the set of properties/paths from the set that an be ignored.  In the case above:
    Query<Customer> query = Ebean.find(Customer.class)
        .where()
           .eq("c1",1)
           .eq("junk","blah")
           ...

    Set<String> invalid = query.validate();

  // consider "c1" as valid
  invalid.remove("c1");

  // if !invalid.isEmpty() ...

Can i exclude (not ignore)  invalid props and continue query?

The query will execute if you run it.  Before you run the query you can
call validate() and get the set of properties Ebean does not know about ...
and at that point your application code can then choose to execute the
query or return some error to the user.
On 30 November 2015 at 15:28, icode<EMAIL_ADDRESS>wrote:

Can i exclude (not ignore) invalid props and continue query?
—
Reply to this email directly or view it on GitHub
https://github.com/ebean-orm/avaje-ebeanorm/issues/291#issuecomment-160500489
.

Maven is deploy？

Yes it has been released to maven central - the associated milestone is 6.12.1.  The latest release is 6.12.1.


Including data-gjs-type in exported html

I recently tried exporting the html and css of my sample site to load it back into grapesjs as the index.html. (I'm making a custom plugin and wanted to make a site to demo it.) After some troubleshooting, it seems like the components won't register without including the data-gjs-type.
Is it possible to configure the export so the generated html includes these data-gjs-type attributes?
Thanks in advance.

I'd suggest you export the project data JSON in order to be able to reimport the project easily.
root: {
 // ...
 'project.json': (editor) => JSON.stringify(editor.getProjectData()),
}

Thanks.  I created a basic storage server/db, and I'm using it to hold the json data.
I think there are genuine use cases for preserving the data-gjs-type attributes in the html though... Below, I linked an issue ticket where people are editing the html in a modal, and the attributes are being deleted.
https://github.com/artf/grapesjs/issues/324
If you're looking for a demo, you can use https://editor.gradyserver.com/ (I know the load time sucks, but it's just a static html file, so you can see all of the code.)


Document deadlink in Chapeter 7

http://infrasim.readthedocs.io/en/latest/userguide.html#vracksystem
the hyperlink of vRackSystem User Manual/Test InfraSIM will re-direct to "sorry" page
Index of User Guide:

Setup InfraSIM on ESXi <--
How to install ESXi on physical server
Deploy and control vNode
Deploy and control vPDU
Access vBMC data
vSwitch setup
vRackSystem User Manual <--
Test InfraSIM <--

@wuy1

Good catch; thanks Peter!

Thanks Peter. Patch submitted to solve this issue.
https://github.com/InfraSIM/docs/pull/4


Duplication of results in Uploader.result when the event finished with Task ID 3 or 4

I'm trying to execute some piece of code when the uploader finishes its uploading but it seems it is duplicates due to multi notifications from the plugin that the task has been finished for 4-20 times until it stops at some random point

It's because the plugin uses CachingStreamHandler. It cached all taskId and send it to flutter when a new event came. I don't really understand the purpose of this cache.
Maybe the author can shed the light on this?

@ramisadek - did you find a workaround for this? Do you just track on the flutter side that the completion for that task has already been called?
I'm seeing what I believe to be this behavior on android devices, but it happens randomly.


New side docs, supporting all views.

New side docs supporting all views:

On Hover, the icon changes to show that you can pull it out.

Popout:

LGTM. How does it look in iPhone 5 and 1024x768?

Mobile defaults to collapsed Docs


this program make my laptop sound wierd noise, oh my god!

i test your video_demo.py, when i run it, my laptop begin sound jijiji~~ wierd noise！i debug simply, found it execute backbone ViT-B-16.pt, the code is in clipvit16.py, the forward function! this is the root cause! what the hell are you doing with this model??
i warry about this can make my laptop go to dead!
my enviroment:
pytorch 1.13, torchvision:0.14 cuda:11.8
windows 11, intel core i7-12700H, nvidia rtx 3080ti
please help me!

Hello there, I am sorry to hear that. I didn't use video_demo.py before. It is from the baseline method STARK (https://github.com/researchmm/Stark). I am not sure what happened to your laptop. Maybe it is because the model ViT-B-16.pt is too big for your laptop.


Modification to the VS code startup batch file for #7619

This as another attempt (previously #10162) at handling issue #7619 . This attempt handles situations involving  the --wait and --verbose arguments. In this revision the batch script will only use the start command when the arguments are empty. I have tested this by building and installing code-oss on my system and trying various scenarios:

code-oss from a command prompt opens a new GUI window and does not block the command window it was launched from.
code-oss --wait and code-oss --verbose block the command prompt while still opening the GUI window.
Running code-oss from Windows 10 search (I had to enter the full path) launches the VS code GUI and only briefly flashes the console window.
Running code-oss --verbose from Windows 10 search (I had to enter the full path) launches the VS code GUI and leaves a console open in the background with information written to it.
I am not yet sure if code-oss file.txt works as my build is not complete but given that CALL would be used in that case I expect it to be fine.

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

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

A better solution than this PR to the problem would be to have the VS Code shortcut ordered before the batch on the PATH when performing a search on the start menu for "Code" but nothing I tried seemed to place a result above something found on the PATH. If maybe you all could have a little chat with somebody on the windows team they may know something that could be done about this as well.

Closing this. The real issue with #7619 is having the code.bat appear in the start menu "best match" feature... Will reach to the Windows team and see what we can do about it.


Add gitattributes for GitHub Linguist

GitHub Linguist would classify the test HTML files as code, concluding that the whole project is written mostly in HTML:

HTML 73.1%
Python 26.9%

This PR makes GitHub Linguist ignore the test HTML files, which makes the statistics way more helpful:

Python 90.2%
HTML 9.8%

Oh wow! I had no idea this was possible.
Thanks for finally getting this categorized correctly 😄


No builtin before builtin functions can break NVM when those functions are overwriten

I know the prompt said not to delete any parts.
Install script

What steps did you perform?
See below
Is there anything in any of your profile files that modifies the PATH?

No
The issue and its fix is just to add the builtin keyword before each call of a builtin function.
In my case, I declared a function called hash in my .zshrc which overwrites the builtin hash function before nvm.sh gets sourced.
nvm.sh calls this new, overwritten hash function which may cause unintended behavior. I replaced the two calls of hash I could find in my build of nvm.sh with builtin hash instead and this fixed my issue.
This fix shouldn't break anything because it will always reference the intended hash function.
I know I could fix this by putting the lines
export NVM_DIR="$HOME/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"  # This loads nvm
[ -s "$NVM_DIR/bash_completion" ] && \. "$NVM_DIR/bash_completion"  # This loads nvm bash_completion

before the part that sources all my custom .zsh files, but I found this issue and figured I would report it.
Side note:
I don't really use NVM or even Node, so I don't know what errors this specific case might cause.
What I did to cause this issue (overwriting the hash function) probably breaks some aspect of nvm so if someone does the same thing it might lead someone in the complete wrong direction of how to fix their issue.
The only reason I caught this was because of the way I implemented my hash function which caused there to be some output whenever I opened up a new terminal.

The proper change would be hash → command hash; good catch. I'll push up a fix once https://github.com/Vampire/setup-wsl/issues/50 is resolved.
Note that it's a bad idea to override builtins anyways.

This seems to break under zsh for me, where command hash returns zsh: command not found: hash.


Implement Hello World tests

The sets of tests that are base minimal to test autorest generates something. This is the list of operation ID from the Swaggers in the testserver:

string_getMbcs (read an UTF8 string in a response body). String to read.
string_putMbcs (put an UTF8 string in a request body). String to write.
basic_getValid (get a basic model). Body to get: {id: 2, name: 'abc', color: 'YELLOW'}
basic_putValid (put a basicmodel). Body to put: {id: 2, name: 'abc', color: 'Magenta'}

Swagger definition are here:
https://github.com/Azure/autorest.testserver/tree/master/swagger

All tests pass as of https://github.com/Azure/autorest.csharp/pull/260


Write DTOs and mappers

Include validation constraints (Min, Max, NotBlank etc)
Write mappers for the DTOs that map the objects to their corresponding entities.

Няма пък
On Thu, 11 Jan 2024 at 21:18, Yoan Petrov @.***> wrote:

Assigned #41 https://github.com/yoanpetrov02/todo-app/issues/41 to
@SilvioStoykov https://github.com/SilvioStoykov.
—
Reply to this email directly, view it on GitHub
https://github.com/yoanpetrov02/todo-app/issues/41#event-11460621849,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGC2LBEEHLTXZPG5AGVBUILYOA3IBAVCNFSM6AAAAABBXAEKSCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGQ3DANRSGE4DIOI
.
You are receiving this because you were assigned.Message ID:
@.***>


Release Google.LongRunning version 2.3.0

Changes in this release:

Commit 604c39b: Regenerated Google.LongRunning with extended_operations.proto (issue 7117)
Commit ac367e2: feat: Regenerate all APIs to support self-signed JWTs

(This commit supercedes commit 788ebbf which failed to release due to a tooling bug.)

Created release for Google.LongRunning-2.3.0


Systemic Fairness, and minor updates

Updates of entries for Chad Jenkins and James McLurkin, along with a comment regarding systemic fairness in robotics research below.
Currently, many Black roboticists are too young or too ambitious to remember the decades of institutional discrimination that has brought us to this point.  Given my colleagues around the country, it would be difficult to explain how this effort, with its inception from the San Francisco Bay Area, has the best interests of all Black roboticists in mind.  Given the view of future generations, it would be difficult to explain why collaboration on projects that propose to help Black roboticists makes sense with allies that have few (if any) Black people in their academic genealogy.  Given the protests this Summer, it would be difficult to explain how such an effort addressed the exclusion of Black roboticists from the primary sources of funding and investment, such for the National Robotics Initiative and DARPA XAI programs.  Given my conscience, it would be difficult to explain how the work supported and heralded by our field has impacted Black lives, as in the case of Robert Williams.
While I am highly supportive of the younger generation raising visibility for Black roboticists, I am also mindful that celebrity for some does not necessarily translate into equal opportunity for all.  The real story of Black roboticists is best told by those who have been discarded by the system: those weeded out of undergraduate degree programs, those facing hostile workplaces and development teams (only to leave the field completely), those isolated and terminated from programs of graduate study, those not invited to interview for faculty positions, those deemed unworthy by peers for sponsored projects or citation for their research, those overlooked for investment and venture capital, those displaced by  technology without reinstatement, and the marginalization and disregard imposed on past generations that built the foundation for progress today.  Much of the work and mentoring towards systemic fairness in robotics and computing goes unseen and unrewarded, putting sincere Black roboticists at a significant competitive disadvantage.  As the rest of the "basic sciences" are coming to realize regarding systemic change to funding practices, there cannot be equal opportunity for anyone without equal opportunity for everyone.
Given the collective history, I view the Black in Robotics and Black in AI organizations as valuable efforts that are orthogonal to the movement for civil rights.  We will know there is true change in our field when expressions of support are matched with policies, practices, and economics that realize equal opportunity.

Thank you @aaronjoh for merging in these updates and my commentary.

@odestcj Thank you for the updates and for your thoughtful and constructive insights!


sdk_container: publish the SDK on a Docker registry

Signed-off-by: Mathieu Tortuyaux<EMAIL_ADDRESS>
On the CI side, we need to generate a PAT for the Flatcar buildbot and pass it from Jenkins to the script.
Related to: https://github.com/flatcar-linux/Flatcar/issues/739

closed in favor of: https://github.com/flatcar/scripts/pull/475


Update Code - Proof_Of_ContributionSnapshot4.md - Mor.Codes

Website: https://mor.codes
Developing mor.codes, a code contribution dashboard system that simplifies the proof of contribution on GitHub with basic search and filter functionalities (Ship by Friday 4th).
We have normalized weight on all snapshot by 2000 times and calculated implied values for better visualization.
Features checklist

[x] Backend for fetching and parsing proof of contributions markdown table
[x] Frontend to display proof of contributions
[x] Basic search system
[x] Basic filter system

Due to uncertainty in weight distribution, we currently used x2000 for snapshot 3 which may not inline with the onchain values. We will adjust that on next snapshot if weight distribution is finalized.

Looks like a great start to a dashboard.  50,000 is too high though.  let’s start with 5,000 - 10,000.  Please propose new features in advance for more weights.


sources/env: change to pure generic functions

Each function should then take an object, it can be a map[string]string of key-value.
For environment variables that would allow to show if a variable is set but is empty. That would also later be useful for flags.

Done in a12214b8e817a3a03ebaf3a3203a643994a8af22


Clicking on a grouped cluster does not zoom

Clicking on a grouped cluster of resources on a map doesn't do anything, but the UX feels like it should.  The expectation from other implementations using marker clusters would be for it to zoom into the level where the cluster is declustered.

Meant to claim this! Will be sending a PR shortly. Not perfection, but it does function as one would expect.


Add Not Applicable as valid value for all "Days to/from" attributes

Closes https://github.com/ncihtan/data-models/issues/206
Closes https://github.com/ncihtan/data-models/issues/85
Applying rule regex match \d+$|Not\sApplicable$|unknown$ to all attributes listed in https://github.com/ncihtan/data-models/issues/26#issue-1194538554

Will merge now. But I wont get to redeploy DCA to expose in staging until tomorrow

This should now be in staging @clarisse-lau

Thank you @adamjtaylor !


parallelize typechecking so it runs faster, fix failing examples

resolves a TODO and enables us to bring back typechecking without sacrificing latency of CI/CD on example PRs

3x faster! 🎉

at ~50s it is now almost as fast as the pytest check, which is 30s

note that it will also start failing if the main typechecking component starts taking longer than 60s!


RuntimeError: element 0 of tensors does not require grad and does not have a grad_fn

I had this problem, and solved it making the gradient enable , adding
"loss.requires_grad = True" before  "loss.backward()"

You can also take a look at here (https://github.com/mazurowski-lab/finetune-SAM/blob/e41f73287329ea68df3bfb153c867bdec4eb23b2/SingleGPU_train_finetune_noprompt.py#L51C5-L72C1) to see if the parameters you want to optimize are enabled for gradient or not


Implemented yii\db\mssql\QueryBuilder::resetSequence()

Mentioned in #13577 and #13581 and compatible with SQL Server 2008+.

Q
A

Is bugfix?
no

New feature?
yes

Breaks BC?
no

Tests pass?
yes

Fixed issues
#13577

No unit test has been written, as none of the other resetSequence() implementations are covered by tests either.

@boboldehampsink still would be great to cover it with tests. Probably w/ general implementation.

Ok. Maybe its better to open a new PR for the general implementation of resetSequence() tests?

Yes.

Ok. For now I first implemented a simple test for MSSQL.

Merged. Thanks!


Suggestion: expose stream instead of listener

Did you ever consider exposing a Stream<double> instead of having the listener() and removeListener() methods which means you also manage the stream subscription yourself internally?
I guess maybe you want to keep it simpler for end users consuming the plugin? But IMO developers are/should be capable of managing a stream subscription.
An advantage of exposing a stream instead of a subscription means consumers can chain operations and manipulate the stream type like:
  final StreamSubscription<int> subscription = VolumeController()
      .watchVolume()
      .map((value) => (value * 100).toInt())
      .listen((volume) {
    // int result
  });

If you wanted to keep it this way, I would make listener() return a void, so that the caller does not create an unnecessary copy of the stream subscription (since you already manage it).

@kurenai7968 not sure if you've had a chance to see the above question/suggestion

I agree with your suggestion. I will release this in the next version.

It looks like the docs need to be updated as result of this change right? This is no longer the API:
    _subscription = VolumeController().watchVolume().listen((volume) {
      setState(() => _volumeListenerValue = volume);
    });

It looks like the docs need to be updated as result of this change right? This is no longer the API:
    _subscription = VolumeController().watchVolume().listen((volume) {
      setState(() => _volumeListenerValue = volume);
    });

I updated the docs as part of this PR: https://github.com/kurenai7968/volume_controller/pull/26/commits/d8ddaabc6ad1e336d3d72347cdf036d4d05382c6


Updates the trending api endpoint

Finishes #142248635

Look at all that red!

          1 Message

:book:
Nice, more deletions than insertions :red_circle:

  Generated by :no_entry_sign: danger


Is there a way to checkout without billing or shipping addresses?

I'm trying to set up product classes for digital-delivered goods, which don't need shipping or billing addresses. Further, I'd like to minimize the amount of PII exists on the site to simplify PCI-DSS auditing requirements. Yet I keep hitting:
django.core.exceptions.ImproperlyConfigured: No class implements abstract base model: `BaseShippingAddress`.

What things do I need to override to get rid of any need for addresses? Alternatively, is there a way to indicate that a product is not physical, so if the cart contains only digital items, no address is required?

... Best I can tell is that, contrary to the documentation, both billing and shipping addresses are mandatory. From what I've dug through, not only does the system require materialized subclasses of BaseShippingAddress and BaseBillingAddress, they're also hardwired in several places in the cart code.


[Suggestion] ForSale notify

Add a notify-feature for claims, that are flagged as "for sell".

This could be enabled in the config or put in as a flag, that can be set for the specific claim.
Example config lines:
enable-forsell-notify: true (default, can be changed to false or undefinied (if used as flag))
forsell-notify-message: "This claim is up for sell. Buy it for  using /gd buy claim" (default string, can be changed of course)
permission: griefdefender.user.claim.command.notify
allows the user to toggle this feature on or off so they dont get the notifier if they are an advanced user or already got a claim.
(could potentially be used for other notifiers as well in the future)

Duplicate https://github.com/bloodmc/GriefDefender/issues/19

thanks @Shybella, will close.


Reflow multiple paragraphs

It's not currently possible to reflow multiple paragraphs at once.

When multiple cursors are placed, only the paragraph containing the first cursor is reflowed.
When text is selected across multiple paragraphs, only the paragraph containing the beginning of the selection is reflowed.

I think in these cases the expected behaviour would be to reflow every paragraph that contains a cursor or is part of the selection.
Additionally, I think a separate command for reflowing the whole buffer would be useful.

This issue was moved to atom/atom#18034


⚠ Bofrost.de has degraded performance

In 1ed9f49, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 743 ms

Resolved: Bofrost.de performance has improved in a8a792d after 24 minutes.


Retrieving a contact's presence ?

I managed to retrieve a skpy.user.SkypeContact, however I have looked into this object's attributes in the docs and I can't find a way to get the contact's presence (online, ...).
Using the event system, I sometimes receive SkypeEndpointsEvents that indicate the presence of an active contact, but is there a way to get it for any contact even if they are inactive ?

There's no presence info available in the main APIs, you'll have to use the event stream.
An endpoint represents one connection for that contact (be that desktop, mobile, web etc.).  A contact with no active endpoints is not connected via any means, and is therefore offline.  As such, you'll probably need to work with the assumption that contacts are offline until told otherwise.
Additional pitfall: some clients (endpoints with push notifications, like mobile) use a form of extended away, and will always appear "connected" even if not...

Thanks, I didn't notice that the endpoints showed all the users' status at the beginning, I thought that it only recorded presence change after sk.loop().
If you don't mind (and that this is the right place to ask for this), I have also been struggling with detecting from who a message comes from: I can capture SkypeMessageEvents, but how do I get the characteristics using the msgId ? I couldn't find any method that took it as an argument and returned date, user, conversation, etc...  How can I use these ids ?

You're looking for the msg attribute, which is a SkypeMsg object containing the interesting details.

Thank you very much for the very rapid response, that's exactly what I'm looking for !


Make sure viminfo file is written when creating a new file

When creating a new file, v:oldfiles might not be updated. So make sure the filemarks will be written out correctly, by setting curbuf->b_marks_read is set even when writing the buffer.
Note sure this is the right approach.
On a related note, I noticed that rviminfo <file> will fail if the viminfofile option has not been set. I wonder if that is correct.

Hmm, the swap test still fails once in a while...


[docs] add more explanations in how to prepare for babs-init

In this PR, I've changed:

added more explanations in docs/rephrased previous docs:

docs for how to prepare a container datalad dataset for BABS
docs for how to prepare container configuration yaml file for BABS
docs for what if babs-init fails

@djarecka I'm merging this branch to main now. If you check the updated readthedocs in a minute, you'll see more explanations on:

how to prepare a container datalad dataset (which I've gone thru with you last week);
how to prepare container configuration YAML file

Let me know if any of these docs does is hard to understand/does not make sense!


Tests for DoubleMapGenotype

Coverage increased (+0.5%) to 15.408% when pulling a355c03d8ecfad66b250cfc0c1616df46f9977e4 on FedorSmirnov89:master into 506230aab623fafcf8215dd236eb79022fafa04f on felixreimann:master.


Display of array fields fails on single entry fields

Tell us where you are using Dejavu (Hosted web app, Chrome extension, Docker Image, Within appbase.io)
appbase.io
What is the version of Dejavu that you are using?
latest as of today
Describe the issue that you are seeing, or the feature request :-), include any screenshots as necessary.
subject: a column that has an array of strings, with column description in dejavu
{
 "type": "text",
 "fields": {
  "keyword": {
   "type": "keyword",
   "ignore_above": 256
  }
 }
}

The column has "string" type in elasticsearch. The column was created in elasticsearch, not via dejavu.
When the field's content is
[
 "line 1",
 "line 2"
]

, the display in dejavu data view, in Viewing mode, will be [...], with a popup showing the values. That is acceptable.
However, when the field is like
[
 "line 1"
]

, one sees the text "Enter or select values", and no possibility to see the value.
By the way, once you edit the field to hold more values, via the 'editing' button above, the screen is not updated.
If your issue deals with accessing Elasticsearch cluster, share the necessary steps to replicate the behavior (the configuration.yml settings, where your ES cluster is hosted, any relevant logs).
If your issue deals with a UI issue, share with us a screenshot of the failing network request or the browser console log showing the error.

Hi @hb020 thanks for letting us know. Can you pull in the latest changes on dev branch and give it a try? I think it should solve the issue you were seeing.

pulled it, but can't get it to compile.
>npm install
npm WARN engine<EMAIL_ADDRESS>wanted: {"node":">= 4"} (current: {"node":"0.10.48","npm":"2.15.1"})
npm WARN engine<EMAIL_ADDRESS>wanted: {"node":">=4.5.0"} (current: {"node":"0.10.48","npm":"2.15.1"})
npm WARN engine<EMAIL_ADDRESS>wanted: {"node":">=4"} (current: {"node":"0.10.48","npm":"2.15.1"})
npm WARN engine<EMAIL_ADDRESS>wanted: {"node":">= 0.12"} (current: {"node":"0.10.48","npm":"2.15.1"})
npm WARN engine<EMAIL_ADDRESS>wanted: {"node":">=4"} (current: {"node":"0.10.48","npm":"2.15.1"})
npm WARN engine<EMAIL_ADDRESS>wanted: {"node":">=4.0.0"} (current: {"node":"0.10.48","npm":"2.15.1"})
npm WARN engine<EMAIL_ADDRESS>wanted: {"node":">=4.0.0"} (current: {"node":"0.10.48","npm":"2.15.1"})
npm WARN engine<EMAIL_ADDRESS>wanted: {"node":">=4.0.0"} (current: {"node":"0.10.48","npm":"2.15.1"})
npm WARN engine<EMAIL_ADDRESS>wanted: {"node":">=4.0.0"} (current: {"node":"0.10.48","npm":"2.15.1"})
npm WARN engine<EMAIL_ADDRESS>wanted: {"node":">=4.0.0"} (current: {"node":"0.10.48","npm":"2.15.1"})
npm WARN engine<EMAIL_ADDRESS>wanted: {"iojs":">= 1.0.0","node":">= 0.12.0"} (current: {"node":"0.10.48","npm":"2.15.1"})
\
><EMAIL_ADDRESS>install /home/hboot/dejavu/node_modules/gulp-sass/node_modules/node-sass
> node scripts/install.js

/home/hboot/dejavu/node_modules/gulp-sass/node_modules/node-sass/node_modules/request/node_modules/hawk/node_modules/boom/lib/index.js:5
const Hoek = require('hoek');
^^^^^
SyntaxError: Use of const in strict mode.
    at Module._compile (module.js:439:25)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/home/hboot/dejavu/node_modules/gulp-sass/node_modules/node-sass/node_modules/request/node_modules/hawk/lib/index.js:5:33)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
npm ERR! Linux 4.4.0-97-generic
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install"
npm ERR! node v0.10.48
npm ERR! npm  v2.15.1
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node scripts/install.js`
npm ERR! Exit status 8
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node scripts/install.js'.
npm ERR! This is most likely a problem with the node-sass package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node scripts/install.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs node-sass
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!
npm ERR!     npm owner ls node-sass
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/hboot/dejavu/npm-debug.log

on Ubuntu 16.04 LTS
I'm no node.js expert, so I'm lost at how to solve this.

Hey @hb020 can you try after upgrading your node https://nodejs.org/en/
If you're unable to we should be able to do a release soon so you can try it on the hosted webapp without doing a build. Let me know

Thanks, it works. It would however be much nicer if we didn't need the click to see the contents. If it is possible to show truncated contents instead of the [...] on which we must click to see the contents, it would be much nicer.

Awesome @hb020, right now we've fixed sized columns in DejaVu and an array can hold a large number of items due to which we've to go with this UX to keep things clean. Closing the issue for now :slightly_smiling_face:


add Trapezoidal rule

!! Not yet ready to merge !!
To do:

[x] implement rule itself
[x] add tests
[x] add documentation
[x] put the type piracy into SciMLBase

I have added a trapezoidal rule for integrating sampled data. Let me know what you think. If/once you agree with the code, I will write docs (and add Simpson's as well in the same way). Right now, I only have a docstring for Trapezoidal.
Additionally, I have some duplicate code for in- and out-of-place operations, that I don't really know how to combine. See lines 224-241 of Integrals.jl, do you have an idea how to streamline such code?
The trapezoidal rule supports integration of pre-sampled data, stored in an array, as well as integration of
functions. It does not support batching or integration over multidimensional spaces.
To use the trapezoidal rule to integrate a function on a regular grid with n points:
using Integrals
f = (x, p) -> x^9
n = 1000
method = Trapezoidal(n)
problem = IntegralProblem(f, 0.0, 1.0)
solve(problem, method) # u: 0.10000075150155016

To use the trapezoidal rule to integrate a function on an predefined irregular grid, see the following example.
Note that the lower and upper bound of integration must coincide with the first and last element of the grid.
using Integrals
f = (x, p) -> x^9
x = sort(rand(1000))
x = [0.0; x; 1.0]
method = Trapezoidal(x)
problem = IntegralProblem(f, 0.0, 1.0)
solve(problem, method) # u: 0.10000490350275731

To use the Trapezoidal rule to integrate a set of sampled data, see the following example.
By default, the integration occurs over the first dimension of the input array.
using Integrals
x = sort(rand(1000))
x = [0.0; x; 1.0]
y1 = x' .^ 4
y2 = x' .^ 9
y = [y1; y2]
method = Trapezoidal(x; dim=2)
problem = IntegralProblem(y, 0.0, 1.0)
solve(problem, method)
#u: 2-element Vector{Float64}:
# 0.2000015680246171
# 0.10000365747228422

In order to make integration over arrays work, I needed to add a method (because isinplace(f::AbstractArray) is not defined:
import SciMLBase.IntegralProblem
function IntegralProblem(y::AbstractArray, lb, ub, args...; kwargs...)
    IntegralProblem{false}(y, lb, ub, args...; kwargs...)
end

I think this belongs in SciMLBase. I will make a PR there too if you agree this is the right approach.

Just small organizational things, but otherwise looks great!

In order to make integration over arrays work, I needed to add a method (because isinplace(f::AbstractArray) is not defined:

Yes, let's get that merged and bumped first, then bump the lower bound and build off of that.

Though I think more data is actually required, since it may need to know the x at which the value f(x) is associated. So it needs two arrays. Might as well just make that a DataIntegralProblem or something at that point to keep it clean.

Is it also possible to implement a method to return a vector of integrated values - i.e. from beginning to that point - like in https://github.com/dextorious/NumericalIntegration.jl/blob/master/src/NumericalIntegration.jl#L44?

Is it also possible to implement a method to return a vector of integrated values - i.e. from beginning to that point - like in https://github.com/dextorious/NumericalIntegration.jl/blob/master/src/NumericalIntegration.jl#L44?

Yeah, that is possible of course... We can define a CumulativeTrapezoidal() quadrature to do this. Let me implement the standard methods first though.

Though I think more data is actually required, since it may need to know the x at which the value f(x) is associated. So it needs two arrays. Might as well just make that a DataIntegralProblem or something at that point to keep it clean.

Ah so you prefer the x-data to be inside the problem, not the method. I guess that makes sense.
So the API should be:
x = 0.0:0.1:1.0
y = x.^2
method = Trapezoidal()
problem = DataIntegralProblem(x, y; dim=1) # dim optional
solve(problem, method)

f = x -> x^2
method = Trapezoidal()
problem = IntegralProblem(f, 0.0, 1.0)
solve(problem, method; x = 0.0:0.1:1.0)

Do you agree?

Ah so you prefer the x-data to be inside the problem, not the method. I guess that makes sense.

Yes, it's part of the problem definition, so it should be defined in solve either. So not solve(problem, method; x = 0.0:0.1:1.0), that x should move into the problem definition somehow?

Ah so you prefer the x-data to be inside the problem, not the method. I guess that makes sense.

Yes, it's part of the problem definition, so it should be defined in solve either. So not solve(problem, method; x = 0.0:0.1:1.0), that x should move into the problem definition somehow?

Okay, so then I need to make another PR to SciMLBase to put an x=nothing field into IntegralProblem, right? That also makes the IntegralDataProblem redundant. Yes?

Ah so you prefer the x-data to be inside the problem, not the method. I guess that makes sense.

Yes, it's part of the problem definition, so it should be defined in solve either. So not solve(problem, method; x = 0.0:0.1:1.0), that x should move into the problem definition somehow?

Alright, so that means we need to put an x=nothing field into the IntegralProblem type. This also makes the DataIntegralProblem type redundant.

Regarding point 2, I agree that integrating a function with the trapezoidal (or any other) quadrature rule  is a different problem to that of integrating data. However i do think we should at least try to get the api for the former to be consistent with the current one of this package, so something of the form
solve(IntegralProblem(f, lb, ub), Trapezoidal(n))

The Trapezoidal struct could be responsible for the optional caching of the points and weights. Do you agree?

I implemented something similar to this for one of my packages here so I could open a pr for this.

Yes that would be nice to have.

Regarding point 2, I agree that integrating a function with the trapezoidal (or any other) quadrature rule is a different problem to that of integrating data. However i do think we should at least try to get the api for the former to be consistent with the current one of this package, so something of the form

Yes

BTW, the name Trapezoidal is already used in OrdinaryDiffEq so we should probably prevent the name clash.

👍 I like that suggestion.

I'll follow up with a QuadratureFunction pr today

Awesome! It's great to see this finally getting some love again.

So given the QuadratureFunction I proposed above, I would implement Trapezoidal as
and you could implement GaussLegendre(n) = QuadratureFunction(gausslegendre, n). Then all of these different quadrature rules could be implemented in the Integrals.jl api because they are all basically computing the same thing: sum(w .* f.(x)).
The Trapezoidal struct or function doesn't need to store the weights because the SciML init interface creates a cache that we can use. However, this only applies to point 2 and if you want to apply the Trapezoidal rule to nonuniform data, then the grid should be passed to a DataIntegralProblem.

This looks great, and i agree this would be a nice way to implement many rules. In the simple cases though, it may be possible to avoid allocating. eg, x does not need to be collected, and w could be a vector-like struct that has a getindex/dot method.

Still needs more documentation, but this should work. Once the QuadratureFunction is implemented, this code could perhaps be refactored to use that, since also for non-uniform, sampled integration it is sometimes useful to extract weights.

Sorry for the delay, here is a rewrite of the original code. If approved I can straightforwardly implement also Simpson's rule.
Let me know how it can be improved.

Just a few comments, other than that I think it's pretty ready to go.

put the type piracy into SciMLBase

Where is this? I didn't see a piracy.

Just a few comments, other than that I think it's pretty ready to go.

put the type piracy into SciMLBase

Where is this? I didn't see a piracy.

It's already in SciMLBase

oh it wasn't checked 😅

oh it wasn't checked 😅

Yes, my bad :)

@IlianPihlajamaa What does this PR do about multidimensional grids? We can certainly define a multidimensional trapezoidal rule on regular grids as a product of 1d rules, but what would the API be? Would the user pass in a x array constructed like an Iterators.product? Or would x be a vector of SVectors?
If we don't want to support this then should we check x isa AbstractVector{<:Number}?

The constructor in SciMLBase already guarantees that x isa AbstractVector. Because multidimensional integration with a product rule is so easy to do by just repeating single-dimensional integration, I havent implemented it explicitly. If we want to add it anyway for the convenience or added performance, we should indeed think of an API (pethaps in a separate issue, because it is not trivial if we also want the option to support non-product grids or other domain shapes).


Rename html_theme_options keys "pygment_light_style" and "pygment_dark_style"

The keys html_theme_options["pygment_light_style"] and html_theme_options["pygment_dark_style"] are (well-entrenched) misnomers - since the project name is "Pygments" with an "s", they should be called
html_theme_options["pygments_light_style"] and html_theme_options["pygments_dark_style"].
(This is obviously a very minor issue, I admit :))

I don't disagree. However, Sphinx itself has a config option pygments_dark_style already: https://www.sphinx-doc.org/en/master/development/theming.html#creating-themes
...which I think (?) was the reason why @12rambau chose to (mis)name the theme-specific options here.  If that's right, we could I suppose change them to something else like highlight_style_light and highlight_style_dark, but IMO that just creates unnecessary work for downstream theme users (who, if they've customized the highlighting theme, will all need to change their conf.pys just to keep things working).  To me that's not worth it...  but I'll wait for @12rambau to weigh in before closing.

I know myself and the simplest explanation is the right one: it's a typo....
@drammock I checked, and Furo uses a dedicated pygments_style_light and pygments_style_dark parameter to control the rendeing if highlighted code. So I guess we could change it back with a deprecation process.

I still think that changing the name

creates unnecessary work for downstream theme users (who, if they've customized the highlighting theme, will all need to change their conf.pys just to keep things working). To me that's not worth it

Even with a super long deprecation cycle (like 1.0) ?
I don't really like to have typos included in our parameters.

Feels too small to open an issue or PR, so I'll mention it here: the file src/pydata_sphinx_theme/pygment.py is also missing an "s".

Even with a super long deprecation cycle (like 1.0) ? I don't really like to have typos included in our parameters.

Fair enough. Ok with me to change, with deprecation warning.

Feels too small to open an issue or PR, so I'll mention it here: the file src/pydata_sphinx_theme/pygment.py is also missing an "s".

@12rambau let's fix this at the same time


error: unknown switch `j'

% znap source zsh-users/zsh-completions
error: unknown switch `j'
usage: git clone [<options>] [--] <repo> [<dir>]

    -v, --verbose         be more verbose
    -q, --quiet           be more quiet
    --progress            force progress reporting
    -n, --no-checkout     don't create a checkout
    --bare                create a bare repository
    --mirror              create a mirror repository (implies bare)
    -l, --local           to clone from a local repository
    --no-hardlinks        don't use local hardlinks, always copy
    -s, --shared          setup as shared repository
    --recursive           initialize submodules in the clone
    --recurse-submodules  initialize submodules in the clone
    --template <template-directory>
                          directory from which templates will be used
    --reference <repo>    reference repository
    --dissociate          use --reference only while cloning
    -o, --origin <name>   use <name> instead of 'origin' to track upstream
    -b, --branch <branch>
                          checkout <branch> instead of the remote's HEAD
    -u, --upload-pack <path>
                          path to git-upload-pack on the remote
    --depth <depth>       create a shallow clone of that depth
    --single-branch       clone only one branch, HEAD or --branch
    --separate-git-dir <gitdir>
                          separate git dir from working tree
    -c, --config <key=value>
                          set config inside the new repository

(anon):3: no directory expansion: ~[zsh-users/zsh-completions]

maybe git is old? However I cannot update git easily on public server.
zsh 5.8 (x86_64-ubuntu-linux-gnu)
git version 2.7.4

git version 2.7.4

That release is from 2016. You might want to upgrade.
But it's fixed now, anyway: znap should now work with older git versions that don't support the -j flag.

Again, feel free to leave a tip. 🙂


PCL support?

Does this work with PCLs, specifically PCL profile 78?  That's the default profile used by Xamarin.Forms apps.
I can add it via Nuget with no errors, but I don't think it worked.  I can't add a using statement for it, and can't find the FeedReader() constructor.

FYI when using Profile111, when installing into a PCL library this package is not automatically added as a reference.
PM> Install-Package SimpleFeedReader
Attempting to gather dependencies information for package 'SimpleFeedReader.1.0.4' with respect to project 'ReactiveReader.Core', targeting '.NETPortable,Version=v4.5,Profile=Profile111'
Attempting to resolve dependencies for package 'SimpleFeedReader.1.0.4' with DependencyBehavior 'Lowest'
Resolving actions to install package 'SimpleFeedReader.1.0.4'
Resolved actions to install package 'SimpleFeedReader.1.0.4'
Adding package 'SimpleFeedReader.1.0.4' to folder 'D:\Dropbox\oss\rxui-samples\ReactiveReader\packages'
Added package 'SimpleFeedReader.1.0.4' to folder 'D:\Dropbox\oss\rxui-samples\ReactiveReader\packages'
Added package 'SimpleFeedReader.1.0.4' to 'packages.config'
Successfully installed 'SimpleFeedReader 1.0.4' to ReactiveReader.Core

FYI when using Profile111, when installing into a PCL library this package is not automatically added as a reference. Additionally Documentation.chm is incorrectly added to the project.
PM> Install-Package SimpleFeedReader
Attempting to gather dependencies information for package 'SimpleFeedReader.1.0.4' with respect to project 'ReactiveReader.Core', targeting '.NETPortable,Version=v4.5,Profile=Profile111'
Attempting to resolve dependencies for package 'SimpleFeedReader.1.0.4' with DependencyBehavior 'Lowest'
Resolving actions to install package 'SimpleFeedReader.1.0.4'
Resolved actions to install package 'SimpleFeedReader.1.0.4'
Adding package 'SimpleFeedReader.1.0.4' to folder 'D:\Dropbox\oss\rxui-samples\ReactiveReader\packages'
Added package 'SimpleFeedReader.1.0.4' to folder 'D:\Dropbox\oss\rxui-samples\ReactiveReader\packages'
Added package 'SimpleFeedReader.1.0.4' to 'packages.config'
Successfully installed 'SimpleFeedReader 1.0.4' to ReactiveReader.Core

I have never used Xamarin nor any idea how these "PCL profiles" work (or should work). I understand from @ghuntley's post that the reference is missing? Does it work when it's added manually?
Why is `documentation.chm' added "incorrectly"? For Windows/VisualStudio that's intended behaviour (though I do consider no longer adding it to the solution, leaving it in the '/packages' directory for reference).

Hey Rob,
re: xamarin
Free for open-source developers, like you if you fill in this form and link back to this issue/project.
When filling it out and ask for iOS, Android and Mac licenses.
https://resources.xamarin.com/open-source-contributor.html
re: documentation.chm
I'd recommend removing it from the solution and putting it (and the source code used to build) in /packages
re: pcl
Have a read of this which explains PCL's:
https://oren.codes/2015/06/16/demystifying-pcls-net-core-dnx-and-uwp-redux/
The numbers represent target, right now you target net40-client.
https://github.com/RobThree/SimpleFeedReader/blob/master/SimpleFeedReader.nuspec#L24-L25
Here's an example of my library which targets Profile111, Profile259 and Profile78 (which is what you want evolve towards)
https://github.com/ghuntley/conditions/blob/master/nuget/Conditions.nuspec#L22-L23
re: migrating to pcl

Create new PCL project, targeting Profile111 (look at the .csproj)
Drag and drop your existing code into the new project.
Update tests reference, ensure tests are green.
Delete old project.
Update your nuspec (see mine)

:+1:

Not sure but does #11 help?


Not detecting all keys from Squeekboard (Phosh) OSK

Aknowledgements

[X] I have checked that there is no other issue describing the same or
similar problem that I currently have, regardless if it has been
closed or open.

[ ] This bug affects Discord website.

[ ] This issue is confirmed to be reproduceable when WebCord is packaged
on at least all three latest supported Electron major releases.

[ ] This issue is reproduceable in Chrome, Chromium or any
Chromium-based browser, e.g Brave or Edge (please write in
Additional Context which browser you have used if it is neither
Chrome nor unmodified Chromium).

[ ] There are no fixes done to master which resolves this issue.

[ ] My issue describes one of the unstable and/or not fully implemented
features.

[ ] I have found a workaround to mitigate or temporarily fix this issue
in affected releases (please write it in Additional context section
below).

Operating System / Platform
🐧 Linux
Operating system architecture
aarch64 (64-bit ARM)
Electron version
25.2.0
Application version
4.3.0
Bug description
In WebCord, several keys on Phosh's Squeekboard OSK do not work (as in it is as if they weren't pressed) on soome profiles.
Notably, on "English (US)" (and many other tested standard english layouts), the q and k keys do not work. However, they do work when switching to the "Terminal" profile. But on that profile, the o, z, and x keys do not work. Some other keys for special characters do not work in some profiles aswell. For example the @ key does not work in English (US).
This issue does not exist on any other app that I have tested.
Additional context
Notice: This issue was automatically generated by WebCord.
I am running this on postmarketOS, on a OnePlus 6, with Phosh v0.27.0.

I highly doubt this has anything to be done on my side, I guess the blame is (most likely) on virtual keyboard application (I suppose it might not emit right events to the application/OS) you're using or Electron (I doubt it is adapted for Linux mobile devices or touch devices at all, so they might not be interested at fixing this bug).

thanks for the input, ill look into it

@TheKrafter see https://gitlab.gnome.org/World/Phosh/squeekboard/-/issues/380 (and related issues) as well as https://gitlab.gnome.org/World/Phosh/squeekboard/-/merge_requests/598 as an attempt to fix electron related apps.
Also phosh-osk-stub (https://gitlab.gnome.org/guidog/phosh-osk-stub) should work. We recently improved support for non US layouts (https://gitlab.gnome.org/guidog/phosh-osk-stub/-/merge_requests/101).


How iTransformer could training blazingly fast?

Hi, thanks for sharing the iTransformer code earlier. I’ve made several adjustments to make it more flexible for my research. I’m curious, how is the iTransformer able to train so quickly with univariate time-series data? On average, it only takes around 20 epochs and less than 10 seconds of training time. Could you explain how that's possible?
I used iTransformer external libraries from @lucidrains
Please take a look at my epoch training size, I've set the maximum epochs of 500.

For univariate time series, the token number is one.


Latest script not jumping to date

Describe your problem

Steps you have taken so far

Screenshots

Your system (please complete the following information):

Operating System:
Version:
Alarm goes off but it doesn't jump to the date

Version: 0.10.2
Windows 10

Closing this in favour of #86.


POC: Remote Kyma Runtime configuration from Kyma Control Plane

Description
This POC is about to find a way how from central place (Kyma Control Plane) user could pass configuration to Kyma Runtime dapr sidecars. Then Application in the Kyma Runtime calls localhost(dapr sidecar) and then can communicate with external service.
Reasons
We go into direction where Kyma Runtime would have much less components in the future and we'd like to move some of them to the central place (Kyma Control Plane).
Architecture

AC

[ ] prepare a proposal section in the new control-plane repository where POC result must be placed:

[ ] concept document
[ ] POC document

[ ] define how Kyma Control Plane could work with Kyma Runtimes as central place for configurations.

[ ] present how to use KCP and set configuration for given Kyma Runtime
[ ] present how Kyma Runtime receives configuration from KCP

consider Agent that syncs with Runtime Director, then configures dapr sidecars

[ ] present how Kyma Runtime Application calls External Services via dapr sidecar

[ ] POC Story:

KCP configures two Kyma Runtimes
KCP configures Kyma1 namespace with external service1
KCP configures Kyma2 namespace with external service2
Kyma1 and Kyma2 has Application deployment (App calls in the loop the localhost endpoint(dapr sidecar))
KCP updates configuration for Kyma1 namespace to use external service2

Links

https://dapr.io/
related to: https://github.com/kyma-project/kyma/issues/8781

Future investigations

new central Runtime Director, which is a database for all Runtimes managed by Control Plane
investigate how KEB configures Kyma Runtime via KCP. Also, remember about the "Bring your own cluster" case.

The concept document should depict the idea behind the story. The KCP should hold and expose the configuration for the runtimes (in more detail, the configuration should hold for example integration details with logging service).
For the POC scenario lets consider integration between Redis services and SKRs. Assume that we have two Redis services on the external cluster/managed service and we want to provide a configuration on the SKRs to allow them to smoothly communicate with those Redis instances.

To do:

Prototype of Runtime Director @pkosiec
Runtime Agent:

Fetch configuration for Runtime with a given ID (ID could be passed via env variable)
Apply configuration (update proper dapr-related K8s resources and restart pod with injected sidecar to reload config)


Write basic data viz plot for each L1 data file

Hello, I know this is already merged, but I wanted to share my code that inspired the error plots. Once main difference is that I colored by flag, which I find to be useful. I will share the code below if helpful,
# Script to test out L1_beta data from Portage
# COMPASS-FME | November 2023
# Stephanie Pennington |<EMAIL_ADDRESS>
# library load
pacman::p_load(cowsay, dplyr, googledrive, tidyr, purrr, readr, ggplot2)
say("Welcome to my script!", by = "random")

# data load
l1_beta_folder <- "https://drive.google.com/drive/u/1/folders/1J-tC1Wt3OqO-KXHV_PaiYj8URZCB57xO"

drive_ls(l1_beta_folder, recursive = TRUE) %>%
  filter(grepl(c(".csv"), name)) -> l1_beta_files

map(l1_beta_files$id, drive_download, overwrite = TRUE)

filePaths <- l1_beta_files$name

lapply(filePaths, read_csv) %>% bind_rows -> l1_beta

file.remove(c(filePaths))

# function to plot all research_names

beta_plots <- function(x){

  l1_beta %>%
    filter(research_name == x) %>%
    ggplot(aes(x = TIMESTAMP, y = value, group = design_link, color = as.factor(OOB))) +
    geom_point() + facet_wrap(~design_link, scales = "free_y") + theme_minimal() +
    scale_color_manual(values=c("0" = "#999999", "1" = "red")) +
    labs(color = "OOB", title = x) -> p

  ggsave(filename = paste0(x, "_beta_plot.jpg"),
         plot = p,
         path = "Desktop")
}

research_names <- unique(l1_beta$research_name)
lapply(research_names, beta_plots)


Bind to touch events

Allow the usage on some portable devices, tested only with iPad

that's fixed in master now
https://github.com/wout/svg.draggable.js/commit/00e68cd3ebeeea6a10f767ec190cfffc82ba270e
web,android,iphone,ipad tested


Unkown exception after upgrading to .Net 5

After upgrading to .NET 5 and consequently updating Microsoft.Data.SqlClient, we started to see strange errors related to the package. The call stack is very  poor and I am not sure how to reproduce it.
Exception message:System.AggregateException
A Task's exception(s) were not observed either by Waiting on the Task or accessing its Exception property. As a result, the unobserved exception was rethrown by the finalizer thread. (Object reference not set to an instance of an object.)

Stack trace: System.NullReferenceException: Object reference not set to an instance of an object.
  ?, in void SNIPacket.ReadFromStreamAsync(Stream stream, SNIAsyncCallback callback)
  ?, in uint SNITCPHandle.ReceiveAsync(ref SNIPacket packet)
  ?, in uint SNIMarsConnection.ReceiveAsync(ref SNIPacket packet)
  ?, in void SNIMarsConnection.HandleReceiveComplete(SNIPacket packet, uint sniErrorCode)
  ?, in void Task.ExecuteWithThreadLocal(ref Task currentTaskSlot, Thread threadPoolThread)

Further technical details
Microsoft.Data.SqlClient version: 2.0.20168.4
.NET target: .NET 5.0.5
SQL Server version: SQL Server 2019
Operating system: Docker, Linux 5.4.17-2011.5.3.el8uek.x86_64 #2 SMP Wed Jul 29 22:09:11 PDT 2020

Have you tried version 3?

Hi @kewinbrand. If the application setup allows, can you try your application with MARS set to false and also as @ErikEJ mentioned try to update to the latest version please?
If the issue persist can you provide a sample repro app as well?

Hi,

I closed the issue unintentionally.
Will update the packages and report in a few days. Just to show how sporadic the error is

Hi @kewinbrand
The driver version you're using is a non-LTS and is Out of Support. Also this issue had been fixed in v2.1 (ref #692).
Please upgrade to either v2.1 or v3.0.

I don't reference the package manually. I see that Microsoft.EntityFrameworkCore.SqlServer uses<EMAIL_ADDRESS>Should I raise a issue to them?

Just add an explicit packagereference to the newer version
https://erikej.github.io/efcore/sqlclient/2020/03/22/update_mds.html

Sadly EF Core 5 still depends on 2.0.1.

@kewinbrand do you have any further questions?
I'm hoping referencing to a newer version resolved your issue?

@cheenamalhotra the problem has gone away after referencing the package
i still don't believe ef core was released with such problem...

@kewinbrand far from all customers use the managed SNI layer, it is only applicable to non- Windows platforms


Any support for the Oracle database? What do I need to integrate with Oracle DB?

Hi @nghiabuilvb with #60 PR Oracle connector is included in distribution now
but you need to add oracle jdbc driver manually because of its license. please see discussion https://issues.redhat.com/browse/DBZ-4038
adding standard oracle configuration to application.properties should be all to get it working with oracle
https://debezium.io/documentation/reference/1.7/connectors/oracle.html

Oh, thanks for your information
With my best regards


An infinite refetch occurs when an error response is received when using useQueries.

Describe the bug
An infinite refetch occurs when an error response is received when using useQueries.
However, when remove the suspense query option, an infinite refresh does not occur.
And it is a situation where the Errorboundary cannot catch the error.
It can be reproduced by correcting the request url to cause an error in the link below.
Your minimal, reproducible example
https://codesandbox.io/s/angry-diffie-rekf90?file=/src/query.ts:278-577
Steps to reproduce

Go to example.
Modify the request url.
So you can see the infinite refetching.

Expected behavior
When using useQueries, I want the retry and suspense options to work properly and I hope the ErrorBoundary can catch errors well.
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
mac os
chrome
108.0.5359.98
Tanstack Query version
v4.19.1
TypeScript version
v4.9.4
Additional context
No response

interesting, thanks for reporting. I wonder why the same doesn't happen for useQuery. I think what's happening is:

query mounts, fetches, suspends
query errors, but we don't throw to the error boundary, so query re-mounts
erroneous queries are always considered stale, and since a component mounts, we fetch and suspend again.

There must be something in useQuery to prevent this that we haven't added to useQueries. I think it has something to do with retryOnMount, but we should set that to false in both cases:
https://github.com/TanStack/query/blob/34ca5581217dae4133046f6cfbc746a22be14dd9/packages/react-query/src/errorBoundaryUtils.ts#L27-L34

@TkDodo Thank you for your quick reply.
I tried many things by referring to your answer, but when I modified the retryOnMount to false, it worked correctly.
The problem seems to be that if the retry option is set to false, retry will not occur when the observer is mounted.
Anyway, your answer was very helpful. Thank you 🙇♂ 😃

I already created a wrapper around the useQueries function for better handling types and queries.
By adding retryOnMount: errorResetBoundary.isReset() the infinite loading problem the problem looks fixed and the reset function on the ErrorBoundary does indeed retry the query.
Important: I use suspense: true by default.
/** @version 1.0.0 */
export function useQueries<Qs extends any[]>({ queries }: { queries: readonly [...{ [Key in keyof Qs]: Qs[Key] & GetOptions<Qs[Key]> }] }): QueryResults<Qs>
{
	const loads = React.useMemo(() => queries.map(options => Query.prepare(options.queryFn)), [])
	const errorResetBoundary = useQueryErrorResetBoundary() // <- get the error boundary QueryErrorResetBoundaryValue

	return useQueriesBase({
		// @ts-ignore
		queries: queries.map(
			(options, index) =>
			{
				const queryKey = options.queryKey
				const enabled = areValidArgs(queryKey)
				const load = loads[index]

				return {
					...Query.Options.get(options.queryFn),
					retryOnMount: errorResetBoundary.isReset(), // <- this line here
					...options,
					queryFn: enabled ? load : undefined,
					enabled,
					queryKey: enabled ? queryKey : undefined,
				}
			}
		),
	}) as QueryResults<Qs>
}

if someone wants to tackle this one, please do. The failing test case provided by @chekrd should be helpful:

#4882

@TkDodo I gave this a shot in the dark. There were some clear differences with the order of operations between useQuery and useQueries.
More info in the PR. Let me know your thoughts!


Retry on etcd too many requests error

Solves MGMT-17651
tl;dr
A fix for a rare error:
Error: finalizing

Caused by:
    0: commiting etcd cache to actual etcd
    1: grpc request error: status: Unknown, message: "etcdserver: too many requests", details: [], metadata: MetadataMap { headers: {"content-type": "application/grpc"} }

Background
When committing our in-memory etcd representation to actual etcd, we send all delete requests concurrently (we have many).
Issue
Sometimes this leads to us receiving an error from etcd which says "etcdserver: too many requests". Recert treated this error as a hard error and as a result it exits.
Solution
Compare the error string to this exact phrasing (as there doesn't seem to be a more robust error code we can check, the code just says Unknown), and if we encounter it, just repeat the request again. Eventually hopefully all requests should go through.

/retest

/retest

/test baremetalds-sno-recert-cluster-rename

/lgtm

/retest

/retest

/retest

/retest

Checking clean CI in #134

/retest

/retest

/retest

/test baremetalds-sno-recert-cluster-rename

/lgtm

/hold not sure if works

/retest

/test e2e-aws-ovn-single-node-recert-serial

/test e2e-aws-ovn-single-node-recert-serial

/unhold

/test e2e-aws-ovn-single-node-recert-serial

/override ci/prow/e2e-aws-ovn-single-node-recert-parallel


Create Pull Request template

Fixed in https://github.com/kowainik/relude/commit/f5556a277ae33f2671bd9d880962ab8ce7a7fe08


Doc: add caveats section in the documentation

Let's gather the tricky parts of QDax usage in a section of our documentation. Elements to add:

auto reset potential issues when using Brax
the in-place replacement of state descriptors in Brax and the necessity to care of this in the play step function

Closed by #99