Couldn't install on aarch64

I'm trying to use a remote-workspace the first time; ssh access does work and there's a .vscodium-server after first connection as remote-workspace.
[Trace  - 10:32:20.302] Server install command stdout:
Server script already installed in /data/data/com.termux/files/home/.vscodium-server/bin/09385e58b89a2ff657ce1aafaaced37f28aaa7e8/bin/codium-server
Error server did not start sucessfully
31b9aca8cfc5806a52b74418: start
exitCode==1==
listeningOn====
connectionToken==92d012d1-2446-4102-a572-188d62f88974==
logFile==/data/data/com.termux/files/home/.vscodium-server/.09385e58b89a2ff657ce1aafaaced37f28aaa7e8.log==
osReleaseId==unknown==
arch==aarch64==
platform==Linux==
tmpDir==/tmp==
31b9aca8cfc5806a52b74418: end

[Error  - 10:32:20.302] Error resolving authority
Error: Couldn't install vscode server on remote server, install script returned non-zero exit status
	at t.installCodeServer (c:\Users\me\.vscode-oss\extensions\jeanp413.open-remote-ssh-0.0.29-universal\out\extension.js:1:411937)
	at process.processTicksAndRejections (node:internal/process/task_queues:96:5)
	at async c:\Users\me\.vscode-oss\extensions\jeanp413.open-remote-ssh-0.0.29-universal\out\extension.js:1:376368

log content:
/data/data/com.termux/files/home/.vscodium-server/bin/09385e58b89a2ff657ce1aafaaced37f28aaa7e8/bin/codium-server: 12: /data/data/com.termux/files/home/.vscodium-server/bin/09385e58b89a2ff657ce1aafaaced37f28aaa7e8/node: not found

Note: node can be executed on this machine (installed via pkg install nodejs)
Any idea how to process?
Additional Note: works fine when connecting to debian x86_64.

ore checks done:
$ cd /data/data/com.termux/files/home/.vscodium-server/bin/09385e58b89a2ff657ce1aafaaced37f28aaa7e
data/data/com.termux/files/home/.vscodium-server/bin/09385e58b89a2ff657ce1aafaaced37f28aaa7e
$ ldd ./node
        libdl.so.2 => not found
        libstdc++.so.6 => not found
        libm.so.6 => not found
        libgcc_s.so.1 => not found
        libpthread.so.0 => not found
        libc.so.6 => not found
        ld-linux-aarch64.so.1 => not found

ok, that cannot work then (not sure if this node binary should be all-static)
$ mv ./node ./node-dist
$ cat - > node <<_EOF
> #!/bin/sh
> node $*
> _EOF
$ chmod +x node
$ rm ../../.09385e58b89a2ff657ce1aafaaced37f28aaa7e8.*

with the result:
[Trace  - 13:41:56.310] Server install command stdout:
Server script already installed in /data/data/com.termux/files/home/.vscodium-server/bin/09385e58b89a2ff657ce1aafaaced37f28aaa7e8/bin/codium-server
Error server did not start sucessfully
92dcec7f0fedddb7584738cf: start
exitCode==1==
listeningOn====
connectionToken==459e5d1d-f15c-48f8-8079-bb2b684f0408==
logFile==/data/data/com.termux/files/home/.vscodium-server/.09385e58b89a2ff657ce1aafaaced37f28aaa7e8.log==
osReleaseId==unknown==
arch==aarch64==
platform==Linux==
tmpDir==/tmp==
92dcec7f0fedddb7584738cf: end

[Error  - 13:41:56.310] Error resolving authority
Error: Couldn't install vscode server on remote server, install script returned non-zero exit status
	at t.installCodeServer (c:\Users\me\.vscode-oss\extensions\jeanp413.open-remote-ssh-0.0.29-universal\out\extension.js:1:411937)
	at process.processTicksAndRejections (node:internal/process/task_queues:96:5)
	at async c:\Users\me\.vscode-oss\extensions\jeanp413.open-remote-ssh-0.0.29-universal\out\extension.js:1:376368

and an empty log file...

mmm strange, I assume you have all the required packages, does using the official remote extension works? or maybe could be a vscodium issue :thinking:

I assume you have all the required packages

I don't know, they aren't listed in https://github.com/jeanp413/open-remote-ssh/blob/master/README.md and I've thought that vscodium-reh would either link statically to libraries that are not common on the system or distribute those along.

does using the official remote extension works?

I don't know, I've never used that.

Check this table https://code.visualstudio.com/docs/remote/linux#_remote-host-container-wsl-linux-prerequisites
You'll need to install any of these if they are missing libc6 libstdc++6 python-minimal ca-certificates tar

Would it be useful to add a link to this table in the README of this extension?`Does it actually apply or isn't this mostly depending on the reh-binaries downloaded which are up to the provider - in our case vscodium?
@daiyam Any ideas?
Note that this actually led me to the underlying issue - vscodium-reh for aarch64 is linked with a GNU/Linux toolchain, so expects libc and friends; while many arm based devices use bionic.
@jeanp413 Would it be possible to automatic download a bionic based version of vscodium-reh-bionic (apart from the issue that it doesn't exist yet, we only have linux + darwin for arm, bionic and win32 are missing)?
Also: would it be possible to use the local-installed (already "native") node instead of the one that is part of the unpacked vscodium-reh, if yes - how?
@daiyam Do you see an option to get vscodium-reh-bionic-aarch64 and maybe even vscodium-win32-aarch64 to get alive?

Do you see an option to get vscodium-reh-bionic-aarch64 and maybe even vscodium-win32-aarch64 to get alive?

You can create PRs to do that. But since I can't test them, I won't try.

Hm, looks like we do have a GitHub action building windows arm64 already, just the "reh" is missing, but I have no clue where the build definition for this is. If that's easy maybe activate  and ask users of vscodium-win32-arm64 to test it?
For aarch64-bionic that's a bit more complicated. It seems that the workflow would best add the NDK then use this to cross-compile. This would be more complicated, but I may give that a try "some day".

So "ignore" or "create issue in VSCodium, start when users post there they'll test"?

I'm trying to do the same thing, were you able to establish the ssh connection with termux? if yes can you explain to me how
I am inexperienced, but with another simple ssh client I can connect properly

@GitMensch I'm trying to do the same thing, were you able to establish the ssh connection with termux? If yes, how did you solve the problem?


Update AboutUs.md

HELP

Codecov Report

Merging #9 (ba616a7) into master (f822642) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #9   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

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


Lowering the completion time for beating the game

I was just wondering if anyone is working on the gamer command to get the bots to finish faster. If not I was thinking of taking that on as a side project. Any thoughts? If this should be asked somewhere else please let me know. Thanks!

Currently me and @JamesGreen31 have been making changes to @gamer, if you want to join hop on to the discord, we sometimes discuss there!


Allow building lambda functions which do not support containers, even if -u is specified.

Issue #, if available:
#1825
Why is this change necessary?
If you have a project that mixes lambda functions with the need to build inside a container (say a python project with a native library) and say a Go function which is not supported by container build.  Then the build fails, and there is no effective workaround.
How does it address the issue?
I simply check if the runtime attempting to be built supports building inside a container, BEFORE i change the build to be a containered build. Currently the code checks after and raises an exception which terminates the build.
What side effects does this change have?
Builds which are currently broken will build properly.  No other side effects.
Checklist:

[ ] Write Design Document (Do I need to write a design document?)
[X ] Write unit tests
[? ] Write/update functional tests
[X ] Write/update integration tests
[X ] make pr passes
[X ] Write documentation

I didn't write any functional tests, because I didn't see any functional tests which exercise container builds.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Both integration tests windows-build-dotnet and windows-build-go are now failing because they now no longer fail to build code if building in a container is specified.  I can't find those integration tests in the repo, but i am happy to fix them.

@stevenj You can directly specify that function you want to build in sam build.
Usage: samdev build [OPTIONS] [FUNCTION_IDENTIFIER]

  Use this command to build your AWS Lambda Functions source code to
  generate artifacts that target AWS Lambda's execution environment.

samdev build abc should work.

That does not provide a work around to the issue.  As @tebruno99 said in the linked issue:

Since sam build always clears the .aws-sam/build folder there isn't a way for a windows or osx user to end up with all the functions built for a deploy.
Your suggestion works great for single local function invokes but it would be nice for these developers to be able to deploy more than 1 python lambda at a time (the others are missing from the build package since sam build -u deletes the previous builds and sam build -u fails on the Go builds.)

(Also the resulting template only has the correct codeuri for the function build that just ran)

@stevenj Could you please add unit tests to cover the cases you are adding and handle the merge conflict with HEAD of develop?

I am not using aws-sam anymore,  i will try and add the test cases when time permits.


node-sass doesn't return POSIX paths, correct?

This is a general question, which I could not find guidelines for in https://github.com/sass/node-sass/blob/master/.github/CONTRIBUTING.md
My understanding is that source maps only work with URLs (though I have an issue open to confirm this), not file system paths, though I found the following comment in a popular node package
https://github.com/webpack-contrib/sass-loader/blob/2529c0716b1bca321c22d16636b1385682b1c730/lib/loader.js#L90-L94
            // node-sass returns POSIX paths, that's why we need to transform them back to native paths.
            // This fixes an error on windows where the source-map module cannot resolve the source maps.
            // @see https://github.com/webpack-contrib/sass-loader/issues/366#issuecomment-279460722
            result.map.sourceRoot = path.normalize(result.map.sourceRoot);
            result.map.sources = result.map.sources.map(path.normalize);

Can you confirm whether or not this comment is accurate?

We don't use Mozilla sourcemaps. Our sourcemaps are generated in C++ by
LibSass.
On Wed., 23 May 2018, 5:27 pm Nick<EMAIL_ADDRESS>wrote:

This is a general question, which I could not find guidelines for in
https://github.com/sass/node-sass/blob/master/.github/CONTRIBUTING.md
My understanding is that source maps only work with URLs (though I have an
issue open to confirm this
https://github.com/mozilla/source-map/issues/340), not file system
paths, though I found the following comment in a popular node package
https://github.com/webpack-contrib/sass-loader/blob/2529c0716b1bca321c22d16636b1385682b1c730/lib/loader.js#L90-L94
        // node-sass returns POSIX paths, that's why we need to transform them back to native paths.
        // This fixes an error on windows where the source-map module cannot resolve the source maps.
        // @see https://github.com/webpack-contrib/sass-loader/issues/366#issuecomment-279460722
        result.map.sourceRoot = path.normalize(result.map.sourceRoot);
        result.map.sources = result.map.sources.map(path.normalize);

Can you confirm whether or not this comment is accurate?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sass/node-sass/issues/2393, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWHqEhyzpl09Buji_fTQF2k7mBStQks5t1X_HgaJpZM4UKrqj
.

As far I know LibSass returns source map paths as relative filesystem paths.
On Wed., 23 May 2018, 6:48 pm Michael Mifsud<EMAIL_ADDRESS>wrote:

We don't use Mozilla sourcemaps. Our sourcemaps are generated in C++ by
LibSass.
On Wed., 23 May 2018, 5:27 pm Nick<EMAIL_ADDRESS>wrote:

This is a general question, which I could not find guidelines for in
https://github.com/sass/node-sass/blob/master/.github/CONTRIBUTING.md
My understanding is that source maps only work with URLs (though I have an
issue open to confirm this
https://github.com/mozilla/source-map/issues/340), not file system
paths, though I found the following comment in a popular node package
https://github.com/webpack-contrib/sass-loader/blob/2529c0716b1bca321c22d16636b1385682b1c730/lib/loader.js#L90-L94
        // node-sass returns POSIX paths, that's why we need to transform them back to native paths.
        // This fixes an error on windows where the source-map module cannot resolve the source maps.
        // @see https://github.com/webpack-contrib/sass-loader/issues/366#issuecomment-279460722
        result.map.sourceRoot = path.normalize(result.map.sourceRoot);
        result.map.sources = result.map.sources.map(path.normalize);

Can you confirm whether or not this comment is accurate?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sass/node-sass/issues/2393, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWHqEhyzpl09Buji_fTQF2k7mBStQks5t1X_HgaJpZM4UKrqj
.

Got it!
I'll ask in the LibSass repo then. I was under the assumption that both the Mozilla node module and LibSass were adhering to the to the same document (Source Map Revision 3 Proposal), but from what you are saying, that may not be the case.
Thank you for your time and for pointing me in the right direction!


use AvalonEdit in winForm

hi
Is it possible to use this component in Windows Form too?
I wanted to use it instead of wpf in winform C#.
Thanks

ElementHost is a .Net control that allows you to put WPF UserControls inside a WinForm.
So I believe that should cover your needs?


Inference

Hi! Thank you for the great work.
May I know how you can run inference on images? I would like to compare between two images.
Also, is there a way to load the evaluation dataset without using a binary file?
Thanks!

Hi! Thank you for the great work. May I know how you can run inference on images? I would like to compare between two images. Also, is there a way to load the evaluation dataset without using a binary file? Thanks!

Hi,
Yes, you can use eval_folder.py to perform inference on a folder or images.
I created multiple inference files, I attached them below. Hopefully they will help you.
eval.zip

Thanks so much!


Add Dylan Sharhon to Contributors list

Hi  @dsharhon, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.

It felt easy. Too easy :)
On Fri, Jun 29, 2018 at 4:04 PM, Roshan Jossey<EMAIL_ADDRESS>wrote:

Hi @dsharhon https://github.com/dsharhon, I'm quite elated about your
pull request. I wanna evolve this project to addresses various problems
faced by first-time contributors. I'd love to learn about your journey in
open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a
pull request and learned that I merged it?
We’ve recently added social share to our web app. Could you please go to
https://roshanjossey.github.io/first-contributions/#social-share and
share your first contribution to open source? Also, check out projects with
easy issues while you’re there.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Roshanjossey/first-contributions/pull/5173#issuecomment-401494775,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH4OTFRTsuS5RVk0Jl0ic6_ND2b3_Rbaks5uBrKKgaJpZM4U9uf0
.


Fix nested route handling

The new v4 doesn't support nested routes yet

It would be good to add this information to the readme file. I took me quite a long time to setup nested routing before finding this issue.


Filter Array action in new designer doesn't properly display expression

Describe the Bug with repro steps

Create a new workflow with any trigger.
Add a Compose step with the input: {"content":[{"ID":12345}]}
Add a Filter array step with the following inputs:  From @outputs('Compose')?['content'] Filter Query @equals(item()?['ID'],12345)
Save workflow

What type of Logic App Is this happening in?
Consumption (Portal)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
{
  "definition": {
    "$schema": "https://schema.management.azure.com/providers/Microsoft.Logic/schemas/2016-06-01/workflowdefinition.json#",
    "contentVersion": "<IP_ADDRESS>",
    "triggers": {
      "When_a_HTTP_request_is_received": {
        "type": "Request",
        "kind": "Http"
      }
    },
    "actions": {
      "Compose": {
        "type": "Compose",
        "inputs": {
          "content": [
            {
              "ID": 12345
            }
          ]
        },
        "runAfter": {}
      },
      "Filter_array": {
        "type": "Query",
        "inputs": {
          "from": "@outputs('Compose')?['content']",
          "where": "@equals(item()?['ID'],12345)"
        },
        "runAfter": {
          "Compose": [
            "Succeeded"
          ]
        }
      }
    },
    "outputs": {},
    "parameters": {
      "$connections": {
        "type": "Object",
        "defaultValue": {}
      }
    }
  },
  "parameters": {
    "$connections": {
      "value": {}
    }
  }
}

Screenshots or Videos

Browser
All browsers
Additional context
-Only seems to occur from new designer. Also the expression displays properly from code view.
-Able to repro from both Consumption, Standard & Power Automate
AB#25471950

We will be hotfixing this - should be in all regions by EOD today


Add proper headers to endpoints with file downloads

Some API endpoints let users to download files, so proper headers should be set to enable download prompts in browsers. An example for how to do so can be found here: https://ktor.io/docs/server-responses.html#file
One example of such an endpoint is the report download endpoint:
https://github.com/eclipse-apoapsis/ort-server/blob/f0dada51693869a0bea6cf6fe3c3e1ffac7b55b0/core/src/main/kotlin/api/RunsRoute.kt#L71-L73
Some additional information can be found in this Stack Overflow answer.

This was fixed by https://github.com/eclipse-apoapsis/ort-server/pull/176, but again auto-close did not work.

This was fixed by #176, but again auto-close did not work.

I have created #868 to collect such cases because I still have no idea why that happens.


Some Migration Questions

Hi I'm just wondering how I handle some simple migration scenarios when the data structure changes.

I found this, which talks about changing a data type of a field. However the code seems to be different now and requires some sort of query. Is this still a valid solution for changing a field's data type?

How do you handle changing the name of a field? So for example, I have a collection of items and the items class has a property called Price. However I would like to rename that property to Cost for example.

How do you remove a field? Say if I remove the Price property entirely from the items class. I notice that the column seems to stay in the database. Can I remove the column somehow?

I'm also wondering what happens if you change the order of properties within a collection's class. Does that matter at all?

I think you should add a page to the wiki that covers all of the common migration scenarios as it would be a common problem for any real world application.

Hi @wilberolive,

Yes, this tips are still valid for v4... You can use LiteEngine class to access your data using BsonDocument (an schemaless data structure). Using UserVersion you can control your database version.

There is no rename key, so you need add a new field (using old value) and then remove old field.

using(var db = new LiteDatabase("..."))
{
if(db.Engine.UserVersion == 0)
{
foreach(var doc in db.Engine.Find("MyCol"))
{
doc["NewCol"] = Convert.ToInt32(doc["OldCol"].AsString);
doc.Remove("OldCol");
db.Engine.Update("MyCol", doc);
}
db.Engine.UserVersion = 1;
}
....
}

To remove a field, just use Remove method in BsonDocument and execute Update.

There is no problem with field order. All you class data will be serialized WITH fields, as JSON do. So, order doesn't matter.

You can also run shell commands to update your data without write any code. It's ideal when you have a single database to update. You can run
db.MyCol.update NewCol = OldCol
db.MyCol.update Fullname = UPPER(Last) + ', ' + UPPER(First)
Shell command also supported using Run method. Shell does not have all functions implementations and suggestion are welcome.
I will add more options in shell and then will write a wiki page

Thanks for all the info. Will try all this out. Unfortunately I cannot use the Shell and have to do it through code as the database will reside on individual client machines that I don't have direct access to once the app is deployed. So anytime the data changes, I need the app itself to manage those changes and keep its own database updated.

I don't suppose there is a faster way to bulk update a collection when trying to remove a field (key) from it for example. I just tried it then with a collection that has 15000 entries and it took about 40 seconds. I think the problem is that I'm calling Update on every entry one at a time in the for loop. Is it not possible to remove the field, then do a bulk update of the entire collection?

Hi! With the objective of organizing our issues, we are closing old unsolved issues. Please check the latest version of LiteDB and open a new issue if your problem/question/suggestion still applies. Thanks!

Hi @wilberolive,

Yes, this tips are still valid for v4... You can use LiteEngine class to access your data using BsonDocument (an schemaless data structure). Using UserVersion you can control your database version.
There is no rename key, so you need add a new field (using old value) and then remove old field.

using(var db = new LiteDatabase("...")) { if(db.Engine.UserVersion == 0) { foreach(var doc in db.Engine.Find("MyCol")) { doc["NewCol"] = Convert.ToInt32(doc["OldCol"].AsString); doc.Remove("OldCol"); db.Engine.Update("MyCol", doc); } db.Engine.UserVersion = 1; } .... }

To remove a field, just use Remove method in BsonDocument and execute Update.
There is no problem with field order. All you class data will be serialized WITH fields, as JSON do. So, order doesn't matter.

You can also run shell commands to update your data without write any code. It's ideal when you have a single database to update. You can run
db.MyCol.update NewCol = OldCol db.MyCol.update Fullname = UPPER(Last) + ', ' + UPPER(First)
Shell command also supported using Run method. Shell does not have all functions implementations and suggestion are welcome.
I will add more options in shell and then will write a wiki page

This answer does not fit into current version of database (nuget package v 5.0.17).
Kindly provide new code sample. Thank you.


⚠ Bofrost.de has degraded performance

In 5b65fb6, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1095 ms

Resolved: Bofrost.de performance has improved in 580c505 after 2 hours, 2 minutes.


Display modes for all monitors should be restored instead of only restoring specific one monitor

Please see
https://source.winehq.org/git/wine.git/commitdiff/df90c0ef7ea053606e684901073b26368f34ac26
https://source.winehq.org/git/wine.git/commitdiff/ef81152c8cd17f1bfb2950753428a95a81728c97
https://source.winehq.org/git/wine.git/commitdiff/b911e560458ac7a05c04758efb8f5286a47cae7d
DXVK's RestoreMonitorDisplayMode() only restores the display mode for a specific monitor, which might cause monitor layout changes after mode changes. I would have implemented this myself if I find the time. Reporting it here if anyone wants to fix this.

What exactly is the issue here?

When using ChangeDisplaySettingsExW to change the primary monitor resolution while a secondary monitor is present. It's possible that the monitor layout will change, e.g., from aligning horizontally to vertically. And then if we only restore the primary monitor resolution, the layout is not kept. Some games expect the monitor layout is kept and the tests prove it. For example, there is a chance that RiME will hang if you quickly get in and out of fullscreen mode multiple times.

Why does restoring all monitors solve that problem? Doing so should be a no-op for all monitors that the game isn't currently running on?

The layout change is caused by calling ChangeDisplaySettings for a specific monitor. It happens in cases when moving the secondary monitor in one direction is closer than moving it to the original direction. For example, a primary 3840x2160 monitor and secondary 800x600 monitor on the right with the top side aligned horizontally. Now call ChangeDisplaySettings to set the primary monitor to 800x600. If the secondary monitor moves to the left. Then the offset is 3840-800=3040. However, if it moves upwards, it's -600. abs(-600) < abs(3040) so the secondary monitor end up on the top side of the primary monitor. You can test it on Windows as well. Finally restoring the primary monitor will keep the secondary monitor on the top side. Thus a layout change. I don't remember the exact reason why the layout change will solve the hang. If DXVK's DXGI can pass wine's DXGI tests, then things should be fine.
Also, CDS_FULLSCREEN may have something to do with this but it's not implemented in Wine. It might affect how monitors get restored. I'll have to look into it.

Does 1743a27669a9b2bc6ea6e6529c5bb9ea45876dc1 fix the problem? I can't really test this properly since i don't have a multi-monitor setup on my desktop and can't really reproduce issues on my notebook.

Actually, yeah, when changing the order around I get failures without that commit but not with it, so it should work.


Fix Regex::searchAll behaviour wrt empty capturing groups

Previously, searchAll would stop search when it encountered an empty
matching group in any position. This means that, for example,
regular expression "(a)(b?)(c)" would match string "ac", but the
resulting group list would be ["ac", "a"].
After this change, the resulting list for the aforementioned regular
expression becomes ["ac", "a", "", "c"] like it should've been.
Additionally, this also changes behaviour for multiple matches. For
example, when "aaa00bbb" is matched by "[a-z]*", previously only "aaa"
would be returned. Now the matching list is ["aaa", "", "", "bbb", ""].
The old behaviour was confusing and almost certainly a bug. The new
behaviour is the same as in Python's re.findall.
For reference, though, Go does it somewhat differently: empty matches
at the end of non-empty matches are ignored, so in Go above example is
["aaa", "", "bbb"] instead.
This is the root cause of issue #2336 which has been already fixed by
replacing searchAll call there with a new function.
Cherry-picked from #2012

@martinhsv  you may want to have a look at this one as well.

You are quite right that the underlying problem with searchAll, that was one of the initial triggers for creating the new function now used by the rx operator, still exists in searchAll for that other functionality (other than the rx operator) that still use it.
However, I believe there is a correctness issue with the particular solution in this pull request.
Consider a scenario where the regex is "^(|abc|def)" and the input is "abc".
In this case there should be two successful full matches.  The first match should be an empty string.  The second match should be "abc".
With the code in this pull request, the first match of an empty string will occur successfully, but then the code will advance the offset, and on the second iteration of the loop, "abc" can no longer be produced as a match.
(Note:  There is a new searchGlobal function is another recent pull request (to support a new rxGlobal operator).  I had a notion to implement that and then, after some stabilization time, look to migrate the remaining users of searchAll to the new function.
You may wish to wait for that process to play out.)

Interesting, I never thought about that. It's also interesting to note that Python used to have "my" behaviour (re.findall(r"^(|abc|def)", "abc") == [""]) until Python 3.7, where it was changed to allow non-empty matches right after empty match (re.findall(r"^(|abc|def)", "abc") == ["", "abc"]). https://github.com/python/cpython/commit/70d56fb52582d9d3f7c00860d6e90570c6259371

Hi @WGH- I am closing this and #2394 in favor of #2396. Please let me know if I am missing something.


Fix --use-system-libraries not working with Rake install

Previously the libmagic tarball and patches were required to be present
in the current directory for rake compile -- --use-system-libraries to
work. To fix this, we now dynamically add these dependencies when
building the gem.

@kwilczynski I think the build failures here are unrelated, so I'm just going to merge. 😄

I guess I can't merge without approval. 😄

Hi @stanhu,

I guess I can't merge without approval. 😄

Sorry about that! Although, I am not sure why - the only thing I enabled (or re-enabled, rather) yesterday was requirements to sign commits, as per:

Does it still pose problems?
Krzysztof

Apologies for not sending a PR for this earlier! Thanks for taking care of it!

Hi @flavorjones,

Apologies for not sending a PR for this earlier! Thanks for taking care of it!

Thank YOU for doing all the work on enabling us to ship native Ruby Gems and to better the users experience!
Krzysztof

Yes, thank you @flavorjones for helping us with this project! I cribbed more Nokogiri work into this project. 😄


Defining package hosts

Right now it's just assuming that the package is on github, pypi for python and npmjs for node.
Would make more sense to just define package host URLs in metadata.json
Would remove need for the owner parameter.
The correct package host class would just be determined from the URL.

Would need to separate dev: (github) from publish (pypi, npmjs)
How would django even be defined?


Updated openapi:generate to know about the individual IconData image type

Updated openapi:generate to know about the individual IconData image type

updated lib/tasks/openapi_generate.rake
updated public/doc/openapi-3-v0.1.0.json

Note: the generated openapi json removed the Tenants definition from 0.1.0 as that is only exposed with the internal v0.0 api.

/cc @Ladas and @gtanzillo please review.  Thanks!!


Use configurable timeouts in api

Add timeout as a config parameter
Add a concurrency joinall helper to make that functionality easy accessable
Check timeout only in outer most call in api
Instead of returning half of the events, return a timeout code.

I want to remove the gevent dependency in the whole api, because I think it is best to limit the gevent dependency to only some places. So using gevent.with_timeout there is not an option for me. Also the timeout parameter ads an abstraction, the user does not need to know in what way we do those concurrent jobs(gevent, async, threads).
The idea was to have the same functionality as gevent.joinall so I added it. I thought the idea is exactly that, it does not matter which of the functions returned first, because they do similar jobs. I can remove it.

"I think it is best to limit the gevent dependency to only some places"
That needs some further clarification. What's the reason for that? Do you plan to ditch gevent?

We agreed that we do not have to do it that way, but it is okay.
Will remove the count argument.


Sunday at the end

How can I place Sunday at the end of the week? like a spanish calendar

In createCalendar.ts
change
const firstDay = new Date(year, month, 1).getDay();
to
const firstDay = new Date(year, month, 7).getDay();
In consts.ts
change 2 arrays to
export const dayNames = [
"Monday",
"Tuesday",
"Wednesday",
"Thursday",
"Friday",
"Saturday",
"Sunday",
];
export const shortDayNames = ["Mon", "Tue", "Wed", "Thu", "Fri", "Sat", "Sun"];
In useCalendar.ts in return object
change
name: dayNames[day],
to
name: fullDate.toLocaleDateString(undefined, { weekday: "long" }),
For mo info see this https://stackoverflow.com/a/51145904


Improve Find-Replace dialog to not violate user interface design principles when doing replace-in-selection

The Find-Replace dialog which is opened with Ctrl+F or Ctrl+H (Windows/Linux) requires the user to provide order-independent information in a specific order for replace-in-selection to work. The order is as follows:

Open the dialog (Ctrl+H).
Disable "Find in selection" in case it is enabled.
Enter find and replacement strings.
Select the text within which to find and replace.
Enable "Find in selection".
Begin replacing result by result, or replace all.

Step 2 should not be necessary. Steps 3 through 5 should be possible to be performed in any order without changing the end result. However, any deviation from the above order changes the result of performing step 6.

steps 2 4 and 5 seem to be the only order dependent steps, and I agree that is not a good experience.
I can select the text to be used in a selection before entering the search/replace strings
But if I have a search term entered, disabling "Find in Selection" will drop the new selection I was trying to use.
Also, configuring editor.find.autoFindInSelection might make this slightly better for you. A freshly opened find widget will use your current selection to search within.

The editor.find.autoFindInSelection is slightly better. But the issue is that the "Find in selection" state always needs to be reset in order to do another selection. The core issue I see is that a change in text selection does not update the scope of the search. I get two blocks of highlighted text (of different colors depending on the theme) when I select another block of text when find/replace selection is active: 1) the find replace selection, and 2) the keyboard or mouse selected text.
IMO the two things that need to change are: 1) editor.find.autoFindInSelection is set default to always, and 2) editor.find.autoFindInSelection when always will actually auto find in the selection when the text selection changes itself. This seems logical if one believes that the user act of the user changing the text selection while "find in selection" is activated, is the "user intent" that the scope of the "find in selection" should be updated. Or if we can't change the behavior of always, then add another option like always_update.
Is there any telemetry to know whether users are even "properly" using the current find/replace in selection? Or even if it is used in any frequency at all compared to other editor operations? If so an experiment can be done to see if this change can be done permanently remove the setting.

One extra note for anyone coming across this issue, I recommend using "editor.find.autoFindInSelection": "multiline" at the moment because always will no longer let you highlight a selection within a single line like a variable name and then press Ctrl-F or Ctrl-H to do a find or replace respectively (since the selection would be 1/1 -- just that variable instance) . Then get in the habit of pressing Esc to close the Find/Replace dropdown so that the Find in selection state gets cleared if it happens to be enabled from another action.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


question: do plugins have to hold a complete copy of the data in memory?

I've been following the project for a while and just watched your presentation and there is an issue I would like to ask about.
Because of your design of using a json-rpc server to link your plugins, it seems to me that any plugin which wants to do analysis on a whole piece of text has to either:

hold the entire piece of text in memory (i.e. it can't reference the memory already used by the rope) and update it from each update command
not hold any in memory and get the full text every time it wants to process the data

Neither of these seems particularly performant -- basically you are going to be using P * S amount of memory, where P is the number of plugins that need access to the entire text file and T is the total size of the text. It seems like you simply can't go with option2, as it would just take way too much CPU power, so you are left with using up P times as much memory as your text takes up (at least!)
Many, many plugins require access to the whole text -- autocomplete, syntax-highlighting, linting, searching... I am actually having trouble thinking of operations that don't require the whole text file -- I guess most inserting operations (i.e. snippets).
Since you have clearly thought a LOT about performance and memory efficiency, I was wondering your thoughts.
It seems to me that somehow allowing plugins to access the rope structure would be really useful... but obviously against the entire architecture goal.

I just came to this repo with the exact same question. Raph, would really appreciate an answer. Thanks!

Closing as stale. For reference, as of Early 2018 the rust plugin lib stores some portion of the document in memory, capped at some number of bytes; in the case of reasonably small files this means the whole file is in memory.
There is some discussion/intention of supporting in-process plugins at some point, which would be able to share memory with the core editor, but there's no current timeline for that.


failed

❯ cargo fullstack
    Updating crates.io index
error: failed to select a version for `create-rust-app`.
    ... required by package `my-todo-app v0.1.0 (/home/teresa/dev/rust/my-todo-app)`
versions that meet the requirements `^8.0.0` are: 8.0.0

the package `my-todo-app` depends on `create-rust-app`, with features: `database_sqlite` but `create-rust-app` does not have these features.

failed to select a version for `create-rust-app` which could resolve this conflict

here's the cargo.toml:
[[bin]]
name = "fullstack"
path = ".cargo/bin/fullstack.rs"
[[bin]]
name = "tsync"
path = ".cargo/bin/tsync.rs"
[[bin]]
name = "my-todo-app"
path = "backend/main.rs"
[dependencies]
actix-files = "0.6.0"
actix-http = "3.0.0"
actix-multipart = "0.4.0"
actix-web = "4.0.1"
futures-util = "0.3.21"
create-rust-app = {version="8.0.0", features=["plugin_dev", "plugin_storage", "database_sqlite", "backend_actix-web"]}
serde_json = "1.0.79"
tsync = "1.2.1"
[dependencies.chrono]
features = ["serde"]
version = "0.4.19"
[dependencies.diesel]
default-features = false
features = ["sqlite", "r2d2", "chrono"]
version = "2.0.0-rc.1"
[dependencies.serde]
features = ["derive"]
version = "1.0.133"
[dependencies.tokio]
features = ["full"]
version = "1"
[package]
default-run = "my-todo-app"
edition = "2021"
name = "my-todo-app"
publish = false
version = "0.1.0"
[profile.dev]
debug-assertions = true

hey @ngmisl thanks for reporting this and sorry you ran into it -- try the latest version and see if it fixes your issue!
cargo install create-rust-app_cli

gettng like 4 screens worth of errors now :(

hey @ngmisl, I just realized I forgot to reply to you. I looked into your setup and reproduced the errors too haha. The problem is that the dev plugin needs the auth plugin at the moment. This shouldn't be the case but I need to remove this requirement. Again, sorry for replying so late, I think if you remove the dev plugin, you'll be fine.
Definitely need to setup cargo-hack so we can test different feature flag combinations

Going to close this as a stop-gap solution has been put in place. Feel free to re-open :)


Could not install policy controller in disable-tuf mode

Question
Hi i wanted to deploy policy controller in disabling TUF mode. so i used the following helm command to install the policy controller.
/home/mukul>helm install policy-controller -n cosign-system --set extraArgs.disable-tuf=true sigstore/policy-controller --devel
but still after passing the disable-tuf as true in extra Args the policy controller process are not coming up as they are still trying to connect the public sigstore tuf.
Kindly update what is the correct way to install policy-controller with disabling tuf.
policy-controller-policy-webhook-5985d96f75-m6z2m   1/1     Running   1 (4m58s ago)   9m21s
policy-controller-webhook-69dc7d6b4b-4wh84          0/1     Running   2 (2m17s ago)   9m21s
/home/mukul/Desktop>kubectl logs -n cosign-system policy-controller-webhook-69dc7d6b4b-4wh84
{"level":"info","ts":1683623042.9509282,"logger":"fallback","caller":"webhook/main.go:94","msg":"Initializing TUF root from  => https://sigstore-tuf-root.storage.googleapis.com/"}
{"level":"panic","ts":1683623076.5629852,"logger":"fallback","caller":"webhook/main.go:96","msg":"Failed to initialize TUF client from  : updating local metadata and targets: error updating to TUF remote mirror: tuf: failed to download 8.root.json: Get "[https://sigstore-tuf-root.storage.googleapis.com/8.root.json\](https://sigstore-tuf-root.storage.googleapis.com/8.root.json\)": dial tcp <IP_ADDRESS>:443: i/o timeout\nremote status:

@Mukuls77 Please, use --set webhook.extraArgs.disable-tuf=true instead the set argument wasn't valid in your comment.

Closing this issue, as confirmed by the user!


Readme docker command fix

Just fixed docker command:
docker create volume postgres-data to docker volume create postgres-data
Fixes #806

Don't know what is this, closing... docker command is fixed in #806

@siinghd Sorry, silly mistake. Won't happen again. First time contributing.


Deploy a VirtualMachine fails with The  "" is invalid: : spec.template.spec.volumes.containerDisk in body is a forbidden property

We are using demo vm.yaml but it fails with below error. Kubervirt seems up and working.
[centos@ip-172-31-43-99 ~]$ kubectl apply -f https://raw.githubusercontent.com/kubevirt/demo/master/manifests/vm.yaml
The  "" is invalid: : spec.template.spec.volumes.containerDisk in body is a forbidden property
[centos@ip-172-31-43-99 kubevirt-ansible]$ kubectl get pods --all-namespaces
NAMESPACE     NAME                                                                    READY   STATUS    RESTARTS   AGE
kube-system   coredns-86c58d9df4-5gpqn                                                1/1     Running   0          25m
kube-system   coredns-86c58d9df4-j7sqz                                                1/1     Running   0          25m
kube-system   etcd-ip-172-31-43-99.eu-central-1.compute.internal                      1/1     Running   0          24m
kube-system   kube-apiserver-ip-172-31-43-99.eu-central-1.compute.internal            1/1     Running   0          23m
kube-system   kube-cni-plugins-amd64-hv2zc                                            1/1     Running   0          24m
kube-system   kube-controller-manager-ip-172-31-43-99.eu-central-1.compute.internal   1/1     Running   0          24m
kube-system   kube-multus-amd64-2ztrz                                                 1/1     Running   0          24m
kube-system   kube-ovs-cni-plugin-amd64-pqdvw                                         1/1     Running   0          23m
kube-system   kube-proxy-mggdz                                                        1/1     Running   0          25m
kube-system   kube-scheduler-ip-172-31-43-99.eu-central-1.compute.internal            1/1     Running   0          24m
kube-system   virt-api-66d49464-7bg9s                                                 1/1     Running   0          23m
kube-system   virt-api-66d49464-c9x5z                                                 1/1     Running   0          23m
kube-system   virt-controller-7fb89fd44c-j2mlc                                        1/1     Running   0          23m
kube-system   virt-controller-7fb89fd44c-xnhqm                                        1/1     Running   0          23m
kube-system   virt-handler-k6bb4                                                      1/1     Running   0          23m
kube-system   weave-net-tdxwp                                                         2/2     Running   0          25m

@haidernitc Hi, containerDisk was renamed recently from registryDisk. What KubeVirt version are you running (virtctl version)? If it's not the latest, you either upgrade KubeVirt, or downgrade the VM spec by renaming containerDisk back to registryDisk.


Consider protections against potential replay attacks

There are likely some kind of replay attacks that are possible in the auth crate. We should preemptively add some kind of protections against this.

Or remove the need for this by using an auth provider #173


Package renames for Elm 0.19

If you know of any packages that will be renamed when publishing for Elm 0.19, comment below.
Included in elm-upgrade-0.19.0-rc2

[x] NoRedInk/elm-decode-pipeline -> NoRedInk/json-decode-pipeline
[x] evancz/url-parser -> elm-lang/url

Included in elm-upgrade-0.19.0-rc6

[x] lukewestby/elm-http-builder -> lukewestby/http-builder
[x] evancz/elm-markdown -> elm-explorations/markdown
[x] elm-tools/parser -> elm-lang/parser

Included in elm-upgrade-0.19.0-rc9

[x] mgold/elm-random-pcg -> elm-lang/random
[x] ohanhi/keyboard-extra -> ohanhi/keyboard
[x] ryannhg/elm-date-format -> ryannhg/date-format
[x] elm-community/elm-test -> elm-explorations/test
[x] evancz/url-parser -> elm/url
[x] elm-tools/parser -> elm/parser

Included in elm-upgrade-0.19.0

[x] thebritican/elm-autocomplete -> ContaSystemer/elm-menu
[x] elm-community/linear-algebra -> elm-explorations/linear-algebra
[x] elm-community/webgl -> elm-explorations/webgl
[x] elm-lang/keyboard -> elm/browser
[x] elm-lang/dom -> elm/browser
[x] mpizenberg/elm-mouse-events -> mpizenberg/elm-pointer-events
[x] mpizenberg/elm-touch-events -> mpizenberg/elm-pointer-events

Included in elm-upgrade-0.19.2

[x] rtfeldman/hex -> rtfeldman/elm-hex
[x] elm-lang/mouse -> elm/browser
[x] avh4/elm-transducers -> avh4-experimental/elm-transducers
[x] dillonkearns/graphqelm -> dillonkearns/elm-graphql

Included in elm-upgrade-0.19.3

[x] elm-lang/navigation -> elm/browser
[x] BrianHicks/elm-benchmark -> elm-explorations/benchmark

Included in elm-upgrade-0.19.4

[x] elm-lang/window -> elm/browser

More

[ ] elm-lang/core#Color -> avh4/elm-color
[ ] mgold/elm-date-format -> ???

These are planned, but aren't published under the new name yet:

[ ] avh4/elm-meshes -> elm-community/meshes

BrianHicks/elm-benchmark -> elm-explorations/benchmark

elm-tools/parser -> elm-lang/parser

Published more in 0.19.0-rc6, and updated the issue description;  I'm not going to add ones that don't have the 0.19 package published yet.

I renamed ohanhi/keyboard-extra to ohanhi/keyboard

ryannhg/elm-date-format -> ryannhg/date-format

elm-lang/* -> elm/*

Also, versions changed!

@avh4 i decided not to rename http-builder after all. it's just lukewestby/elm-http-builder 6.0.0 now

@avh4 these have been published:
elm-community/linear-algebra -> elm-explorations/linear-algebra
elm-community/webgl -> elm-explorations/webgl

rtfeldman/hex -> rtfeldman/elm-hex

dillonkearns/graphqelm 11.2.0 -> dillonkearns/elm-graphql 1.1.0
This also means that all instances of Graphqelm. in people's source code should change to Graphql.. I'm not sure if there's find-and-replace magic happening for module names in source code in elm-upgrade, just thought I'd mention it in case there is!

<EMAIL_ADDRESS>and later will now produce a checklist of things the user needs to check after elm-upgrade is done.  For example:
- NoRedInk/elm-json-decode-pipeline
  - [ ] Changes uses of Json.Decode.Pipeline.decode to Json.Decode.succeed
- dillonkearns/elm-graphql
  - [ ] Changes references to Graphqelm.* to Graphql.*
- elm/core
  - [ ] Replace uses of toString with String.fromInt, String.fromFloat, or Debug.toString as appropriate
- elm/html
  - [ ] If you used Html.program*, install elm/browser and switch to Browser.element or Browser.document
  - [ ] If you used Html.beginnerProgram, install elm/browser and switch Browser.sandbox
- elm/time
  - [ ] Read the new documentation here: https://package.elm-lang.org/packages/elm/time/latest/
  - [ ] Replace uses of Date and Time with Time.Posix
- elm/url
  - [ ] Change code using UrlParser.* to use Url.Parser.*

If there are checklist items that should be added for any of the renamed packages, please make a new issue for each package, thanks!
(If there are new package renames to add, you can still comment with them in this issue.)

Due to elm compiler bug I had to rename all my packages:
1602/json-value -> json-tools/json-value
1602/json-schema -> json-tools/json-schema
1602/elm-feather -> feathericons/elm-feather


Complete migration to C++ Filters in FSTQuery

This removes FSTFilter and related classes.

Note the diffbase on #3348.


extra padding near dropdown in terminal in insider release

extra padding right to the dropdown in terminal in insider release as shown in image below

in stable release I don't see it

also the terminal action also not work when I open the insiders window
it works if create a one more terminal and then these actions start working

/duplicate #202078

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!

extra padding right to the dropdown in terminal in insider release as shown in image below
in stable release I don't see it

Looks like regression from #200980

Thanks for reporting these issues @abhijit-chikane. I created a separate issue for the second bug you mentioned here: #203630


Jwoo/release/1.2

Related Issue
Fixes #3581
Description
Vertical alignment logic's call flow didn't allow to hit the necessary condition for activating the vertical alignment.
How Verified

Ran sample cards to make sure there were no regressions.
checked test card with problem scenario worked

Microsoft Reviewers: Open in CodeFlow

Curious -- Are there existing unit tests we can add or extend for this scenario?

Can you add the test card to our list of samples in teh repo also so it's readily available for the bash?


Implement the sequence constructor

Close #945. Implement the operator Apalache!MkSeq. There are plenty of small changes to introduce the new operator ApalacheOper.mkSeq. The most interesting part is the rewriting rule MkSeqRule.

[x] Tests added for any new code
[x] Ran make fmt-fix (or had formatting run automatically on all files edited)
[x] Documentation added for any new functionality
[x] Entry added to UNRELEASED.md for any new functionality

Codecov Report

Merging #1439 (51f073a) into unstable (6402105) will decrease coverage by 0.03%.
The diff coverage is 62.96%.

@@             Coverage Diff              @@
##           unstable    #1439      +/-   ##
============================================
- Coverage     74.62%   74.58%   -0.04%
============================================
  Files           359      360       +1
  Lines         11373    11422      +49
  Branches        542      561      +19
============================================
+ Hits           8487     8519      +32
- Misses         2886     2903      +17

Impacted Files
Coverage Δ

...yte/apalache/tla/bmcmt/SymbStateRewriterImpl.scala
92.68% <ø> (ø)

...orsyte/apalache/tla/pp/CallByNameWrapHandler.scala
0.00% <0.00%> (ø)

...ir/transformations/standard/KeraLanguagePred.scala
94.38% <0.00%> (-1.08%)
:arrow_down:

...in/scala/at/forsyte/apalache/tla/imp/Context.scala
61.90% <28.57%> (-2.20%)
:arrow_down:

...at/forsyte/apalache/tla/bmcmt/util/TlaExUtil.scala
64.70% <50.00%> (-4.53%)
:arrow_down:

...t/forsyte/apalache/tla/bmcmt/rules/MkSeqRule.scala
77.77% <77.77%> (ø)

.../at/forsyte/apalache/tla/imp/StandardLibrary.scala
100.00% <100.00%> (ø)

...forsyte/apalache/tla/typecheck/etc/ToEtcExpr.scala
93.42% <100.00%> (+0.05%)
:arrow_up:

...in/scala/at/forsyte/apalache/tla/lir/Builder.scala
99.08% <100.00%> (+<0.01%)
:arrow_up:

...t/forsyte/apalache/tla/lir/oper/ApalacheOper.scala
72.72% <100.00%> (+1.57%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


Random failures in tests

As of late we are getting random failures in tests:
  52 passing (8s)
  2 pending
  2 failing
  1) user render timeout limit vector (mapnik) layergroup creation works but vector tile request fails due to render timeout:
     Uncaught Error: Invalid response status code.
     Expected: 429
     Got: 200
     Body: 9x
layer0(
cartodb_id" val" "
      at validateResponseStatus (test/support/assert.js:146:20)
      at validateResponse (test/support/assert.js:179:9)
      at Server.<anonymous> (test/support/assert.js:119:27)
      at emitCloseNT (net.js:1574:8)
      at _combinedTickCallback (internal/process/next_tick.js:77:11)
      at process._tickCallback (internal/process/next_tick.js:104:9)
  2)  "after each" hook for "layergroup creation works but vector tile request fails due to render timeout":
      AssertionError: uncaughtException:
Error: Invalid response status code.
     Expected: 429
     Got: 200
     Body: 9x
layer0(
cartodb_id" val" "
    at validateResponseStatus (test/support/assert.js:146:20)
    at validateResponse (test/support/assert.js:179:9)
    at Server.<anonymous> (test/support/assert.js:119:27)
    at emitCloseNT (net.js:1574:8)
    at _combinedTickCallback (internal/process/next_tick.js:77:11)
    at process._tickCallback (internal/process/next_tick.js:104:9)
      + expected - actual
      -1
      +0

I think it's appearing just in the PG10 run but it requires investigation.

Yes @Algunenano It appears after adding the xenial + PG10 job in Travis
Until this change, we were checking against a normal postgres 9.5 (without our custom Postgres packages). Now we are checking precise + PG9.5 and xenial + PG10.1
As I added it, I want to check it

Already merged in master


contracts: Close all srlabs findings issues

Fix all pallet-contracts pending issues reported by the SRLabs security review

PR have been merged for all the open tickets


[Auto Complete] display suggestions

Versions

Material-UI: 0.17.4
React: 15.2.1
Browser: Chrome

This is not an error. I just want to solve my problem:
I have dataSource is ['Nguyen Van A', 'Nguyễn Văn A']
If  I type on search box 'Nguy' then it show 2 suggestions, of course
But when I continue type 'Nguye' then Auto Complete only show suggetion 'Nguyen Van A'. I know it is not an error but If I want to show 2 suggestions 'Nguyen Van A', 'Nguyễn Văn A' when I type 'Nguye' then what should I do?
Thanks

You can write a filter function that  ignores accents. (the original filter:  (searchText, key) => searchText !== '' && key.indexOf(searchText) !== -1).

@wesley6j Thanks, I'll try

@wesley6j Thanks for the answer! That's a good starting point.
Also, please submit support requests and questions to StackOverflow using the tag material-ui. StackOverflow is better suited for this kind of support. The issue tracker is for bug reports and feature discussions. See also our CONTRIBUTING guidelines.


IP for handshake fix.

https://youtrack.raccoongang.com/issue/OLGA-36

Codecov Report

Merging #124 into develop will increase coverage by 0.25%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #124      +/-   ##
===========================================
+ Coverage    98.02%   98.27%   +0.25%
===========================================
  Files           22       22
  Lines          405      406       +1
===========================================
+ Hits           397      399       +2
+ Misses           8        7       -1

Impacted Files
Coverage Δ

web/olga/analytics/views.py
100% <100%> (+0.9%)
:arrow_up:

Continue to review full report at Codecov.

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


Add conversion projects

Changes
This PR creates the concept of a Conversion Project. Eventually, the application will feature other types of projects, e.g. Transfer projects, so we needed to separate out the concept of different types of projects.
The current project journey we have is for "voluntary" conversion projects. We also need to add "involuntary" conversion projects. In order to keep this PR relatively small we have decided to:

Keep the current Project journey
Introduce Conversion projects "under the hood" (users cannot currently access this journey)
Introduce Involuntary conversion projects as a subtype of conversion projects, with their own form and tasklist

Checklist

[x] Attach this pull request to the appropriate card in Trello.
[ ] Update the CHANGELOG.md if needed.

 The application supports two Converion journeys


unfurling: add settings module and dev mode conversation storage system CORE-9242

Patch does the following:
1.) Introduces a new interface ConversationBackedStorage with two simple accessor methods with the purpose if being implemented by a chat conversation.
2.) Add DevConversationBackedStorage which writes TEXT messages into DEV conversations, and always uses the contents of the latest message in the return type. Data is JSON encoded into the chat text messages.
3.) Add unfurl.Settings which manages the unfurling settings via a ConversationBackedStorage.
4.) Add simple methods and new types for tracking settings.

@joshblum ok made assertLoggedInUID change, let me know if you accept my other replies.

Going to merge since I think I hit all the feedback, let me know if there is anythign else.


Bitstream/Attachment appearance issue during submission process

Describe the bug
Bitstream/Attachment uploaded at the beginning of the submission process disappears as soon as the "terms acceptance" check box is checked and the "Deposit" button gets deactivated.
Upon re-uploading the attachment in the same submission process, both fresh and previous attachments uploaded appear. Upon selecting the "terms acceptance" check box on the second attempt, the "Deposit" button remains activated.
To Reproduce
Steps to reproduce the behavior:

Go to the item submission form, upload bitstream at the beginning of the process, or drag and drop at any time.
Click on the checkbox appearing next to "I confirm the license above". After refresh, you will notice the Deposit button will get de-activated and bitstream uploaded will not appear in the Upload files section.
Upload the same bitstream again using any of the available methods
Click on the checkbox appearing next to "I confirm the license above". After refresh, you will notice that bitstream will appear twice and the Deposit button will stay active for completing the submission.

Expected behavior
Upon clicking the checkbox appearing next to "I confirm the license above", the bitstream added in the form should not disappear and the Deposit button should get activated for finishing submission.
Related work
Link to any related tickets or PRs here.

I'm able to reproduce this on the demo site (demo7.dspace.org), but I cannot reproduce this locally (in a local.  I think though that it might be related to the other bitstream upload issues, namely #1121 .
@atarix83 : Assigning this to you for review/analysis, as I think this is the same issue as #1121, or at least caused by the same process. I've had difficulty reproducing it on my local instance, but it does appear on the demo site.

the issue is due to #1121 and it's been resolved within https://github.com/DSpace/dspace-angular/pull/1201


[stg] use image uuid

seems @kwoodson has a commit in prod and not stg?

Looks like I had a merge commit that was in prod but not stage.  I have
merged the prod branch to stg and int and it looks like it picked up that
merge commit.
Please rebase your branch with prod and it should resolve.
Thanks.
On Thu, Nov 3, 2016 at 12:56 AM, OpenShift Bot<EMAIL_ADDRESS>wrote:

Tests Passed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openshift/openshift-tools/pull/1526#issuecomment-258064602,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUlKmInSsU2ZxRD8_bedEEMiIKEmt28ks5q6WmZgaJpZM4KoAS0
.


Updated the readme file.

I have updated the readme file. @Vinay-Khanagavi
#28 issue no.

ok, but I have added the gif and demo video was it not relevant or not good?

ok, but I have added the gif and demo video was it not relevant or not good?

I didn't feel it looked good. And moreover for the demo, a few screenshots would do we don't need a whole video for it.

Ok, So tell me should I work again on it ?


Error on a "remote" rule doesn't enable the eager (onkeyup) validation mode

To reproduce:

Set up validation with a remote rule that takes some time and with the following function for the onfocusout event: function(element) { $(element).valid(); }
Type something that doesn't validate with the rule and focus out of the field.
When the rule finishes processing and error is reported, focus the field.
Modify the value so that it becomes valid.

Result: the field doesn't validate, the default onkeyup event handler doesn't process the keystrokes.
The reason for this is that the remote rule doesn't add the field to the submitted object (as opposed to e.g. the formatAndAdd method).

I'm sorry for the lack of activity on this issue. Instead of leaving it open any longer, I decided to close old issues without trying to address them, to longer give the false impression that it will get addressed eventually, especially after several years with no activity. It doesn't mean I'm abandoning the project, just that I'm unable to work through 200+ open issues with the little time I can afford to spend on this project.
To the reporter (or anyone else interested in this issue): If you're still affected by the same issue, please consider opening a new issue, with a testpage that demonstrates the issue with a current version of the plugin. Even better, make an attempt to fix the issue yourself, and improve the project by sending a pull request. This may seem daunting at first, but you'll likely learn some useful skills that you can apply elsewhere as well. And you can help keep this project alive. We've documented how to do these things, too. A patch is worth a thousand issues!


Usage with ES6 style imports

Hey, thanks for the hard work - the library is great! When I try to use the library with ES6 imports like below

import wdtEmojiBundle from "wdt-emoji-bundle";

function addEmoji(){
    wdtEmojiBundle.init('.my-selector');
}

I get the error Uncaught ReferenceError: EmojiConvertor is not defined. I'm assuming this is because I haven't imported emoji.min.js, but why wouldn't the bundle import that module itself? Why do we need to create an extra global definition for that object?
In order to use the library I had to do the following:
import EmojiConvertor from "./../../../node_modules/wdt-emoji-bundle/emoji.min";
window.EmojiConvertor = EmojiConvertor;
import wdtEmojiBundle from "wdt-emoji-bundle";

function addEmoji(){
    wdtEmojiBundle.init('.my-selector');
}

Hi @cyorobert, I guess I forgot to declare the dependency on wrapper.
I will check, thanks.

Excellent @needim thank you!

I'm having a similar, but different problem with ES6. (v0.2.0)
import wdtEmojiBundle from 'wdt-emoji-bundle'
results in:
Error: Can't resolve 'js-emoji'

Disregard - this looks to be an issue with a emoji js required package that is not compatible with webpack 3.

@DaveSanders were you able to solve the above issue, i am having similar problem.


Sync installer charts

@jwendell: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

test_operator_release-1.4
d030d6951e177b39192b0ba187112ce667d3c0db
link
/test test_operator_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@jwendell: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

test_operator_release-1.4
d030d6951e177b39192b0ba187112ce667d3c0db
link
/test test_operator_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Ok so if I sync to this PR and make mesh then run ./mesh manifest generate --set profile=demo-auth or any profile, there will be an error render error in "istio-discovery/templates/telemetryv2.yaml": template: istio-discovery/templates/telemetryv2.yaml:1:50: executing "istio-discovery/templates/telemetryv2.yaml" at <.Values.pilot.telemetry.v2.enabled>: nil pointer evaluating interface {}.enabled.
Syncing to current master(2e56cf404a00cf778ba3a98d42e43d8be1b262b5) is fine.


Add: \skillz, \project & environment hangingparlist

Can you add multicol.sty and ragged2e.sty to the branch you PR'd from, please? I carry this thing across machines often enough that I find hoping the right LaTeX packages are installed is a huge source of bitrot.

Looks good! Thank you for upstreaming these changes!


right delegation in table groupmember is only working for some users

My system is a Debian server running Apache and PHP 5.4.45.
So sabredav 3.0.9 is installed.
The clients are Thunderbird with sogo-connector and CategoryManager.
Here is my problem:
I have 3 user accounts with own calender and adressbook.
Additional i have 1 user account for a shared calender and adressbook.
I have added the calendar-proxy-read/write principal id's of the shared account for all 3 users to the groupmember table.
Thunderbird authenticate on each PC with the personal user account.
On my PC i can access and work with my personal and the shared calender and adressbook.
But the second user can only work with his personal calender and adressbook!
When i try to access with the 2. user account via browser i get
<d:error xmlns:d="DAV:" xmlns:s="http://sabredav.org/ns">
<s:sabredav-version>3.0.9</s:sabredav-version>
<s:exception>Sabre\DAVACL\Exception\NeedPrivileges</s:exception>
<s:message>User did not have the required privileges ({DAV:}read) for path "calendars/sharedaccount
"</s:message>
<d:need-privileges>
<d:resource>
<d:href>/sabredav/groupwareserver.php/calendars/sharedaccount</d:href>
<d:privilege>
<d:read/>
</d:privilege>
</d:resource>
</d:need-privileges>
</d:error>

I have double checked the groupmember entries and everything should be perfect.
The entries are correct with similar id's of the principals to my user that is working!
When i look into the principals of the user that is not working
https://<IP_ADDRESS>/sabredav/groupwareserver.php/principals/user2/
i can see that the groupmember rights are empty!
cs:calendar-proxy-read-for
cs:calendar-proxy-write-for
d:supported-report-set    d:expand-property, d:principal-property-search, d:principal-search-property-set
d:resourcetype    d:collection, d:principal

But they are defined in the table groupmember!
So sabredav has a bug and is not working. :-(

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/34557026-right-delegation-in-table-groupmember-is-only-working-for-some-users?utm_campaign=plugin&utm_content=tracker%2F294006&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F294006&utm_medium=issues&utm_source=github).

Please note that addressbook sharing does not work. The delegation / proxy stuff only applies to calendars.
That being said, it should still work. Could you share the actual database values for the relevant principals and entries in groupmembers?

Up to now i only tried to use a shared calendar.
This produced this error.
I checked the apache logfiles and no php error is logged.
It's a pity that sharing of adressbooks does not work!
Do you know if it is possible to login in Thunderbird with multiple accounts?
Or is there another workaround?
I only want to have a private and a shared calendar and adressbook for each user.
Here the anonymized content of the tables.
sabredav.sql.zip

The only way that you can log in to multiple accounts with thunderbird, is if you have multiple subdomains all running the same sabre/dav. Thunderbird will cache credentials based on an entire domain, so that's a workaround.
Looking into the zip now.

Based on the zip, I can tell that there's some errors.
You need to make sure that if user_a wants to delegate their calendars to user_b, then the principal_id in the group members table for that record needs to refer to principals/user_a/calendar-proxy-read and the member_id needs to refer to principals/user_b.
You also shouldn't add them to both the read and write group. Just write is enough.
Sorry if this is a bit annoying/complicated. I plan to add an interface at one point to the browser plugin for this.

So to conclude, only admin is currently added to the shared account. Correct?

Now i understand. Doing the things right is working! :-)
There should be a much better description with examples for it!
The principals are very important but nearly not documented.
I made it wrong because i followed this page http://publications.jbfavre.org/web/online-calendars-sabredav-php-fpm-nginx.en
So this guy did not understand it even. ;-)

I plan to add an interface at one point to the browser plugin for this.

You should do it immedeately because this is absolutely missing.
You can add an calender direct over the php of sabredav but not the users and user rights!
With two additional PHP's all problems of misunderstanding would be solved.

So to conclude, only admin is currently added to the shared account. Correct?

No - admin should not be used - i was afraid that something is not working when i delete this account.
All users should have there own calendar and additional the rights for the shared calendar.
The correct solution is:
INSERT INTO groupmembers (id, principal_id, member_id) VALUES
(1, 15, 4),
(2, 15, 7),
(3, 15, 10);
It is working now - thank you for your support!

The only way that you can log in to multiple accounts with thunderbird, is if you have multiple subdomains all running the same sabre/dav. Thunderbird will cache credentials based on an entire domain, so that's a workaround.

Here i need some additional help.
I can enter a address with my account in my private and in the shared addressbook.
Then i can see this addresses with my user in each addressbook.
With the second user on the other PC i can open the shared addressbook but it seems to be empty!
I don't understand this?
At this time i work locally in the LAN - so i address the server of it's IP like
https://<IP_ADDRESS>/sabredav/groupwareserver.php/addressbooks/shareduser/shared/
What would be the best/easiest solution to solve the problem?

I could find out that the entered shared addresses are only stored locally on my PC in Thunderbird.
This is very irritating, because there is no message that synchronizing will not work for the shared account!
So - how can i implement a shared addressbook that is working?

The only way that you can log in to multiple accounts with thunderbird, is if you have multiple subdomains all running the same sabre/dav. Thunderbird will cache credentials based on an entire domain, so that's a workaround.

This is a good hint - Thank you.
I implemented the Subdomain locally over /etc/hosts in the client.
With an entry "<IP_ADDRESS> server dav.server" i could create additional alternate Names to work with.
So i can access the shared addressbook now additionally.
But it would be fine to manage the rights for the addressbook access too.
Is it not possible to create a principal like "principals/user/addressbook-proxy-read" even it is not specified in an RFC?
Maybe it is not so much work to implement a delegation of the rights in the same way like for the calender?

Hi Karsten,
It is actually possible to allow a user full access to another user's accounts + addressbooks and calendars. In that case the user needs to be added a as  member of the account that is being shared.

It is actually possible to allow a user full access to another user's accounts + addressbooks and calendars. In that case the user needs to be added a as member of the account that is being shared.

How can i do that?
With the current rights i have full access as user to the shared user calendar - but nor address book.
I must login as shared user to access the address book.
It would be interesting to have an CardDAV account where users can read an insert addresses, but not modify or delete them.
How could that be realized?

Basically, the purpose of groups is when you add a principal as a member of any other principal, they will inherit all their privileges.
This will only allow you to do a read/write thing. Not read-only.

O.K. Where can i find a list of all possible principals?


'React/RCTViewManager.h' file not found

Hello!
Bug description:
I started to use pod in my project because i wanted to use react-native-qrcode-scanner with react-native-permissions and now I'm getting this error 'React/RCTViewManager.h' file not found I already fixed React(missing) from schemes and I don't know what I have to do to fix that error, could anyone help me please?
Screenshots:

Environment:

OS: macOS Catalina
OS version:  10.15.1
react-native version: 0.61.5
react-native-webview version: ^5.11.0

I don't know if a solution was found but i bumped into that same thing today.
Wanted to use WebView in project and so i started using cocoapods. I get the following errors while building for ios (haven't tried for android):
../node_modules/react-native-webview/ios/RNCWebViewManager.:8:9: fatal error: 'React/RCTViewManager.h' file not found
../node_modules/react-native-webview/ios/RNCWebView.:8:9: fatal error: 'React/RCTView.h' file not found
react: 16.6.3
react-native: 0.57.8
react-native-webview: ^8.1.0
My podfile looks like this:
platform :ios, '9.0'
target 'AreaRN' do
use_frameworks!
pod 'React', :path => '../node_modules/react-native', :subspcs => [
'Core',
'CxxBridge',
'DevSupport',
#The following ones are the ones taken from 'Libraries' in XCode:
'RCTAnimation',
'RCTActionSheet',
'RCTBlob',
'RCTGeolocation',
'RCTImage',
'RCTLinkingIOS',
'RCTNetwork',
'RCTSettings',
'RCTText',
'RCTVibration',
'RCTWebSocket'
]
#The following dependancies are dependancies of React native itself
pod 'yoga', :path => '../node_modules/react-native/ReactCommon/yoga/Yoga.podspec'
pod 'DoubleConversion', :podspec => '../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec'
pod 'Folly', :podspec => '../node_modules/react-native/third-party-podspecs/Folly.podspec'
pod 'glog', :podspec => '../node_modules/react-native/third-party-podspecs/GLog.podspec'
Pods for AreaRN
pod 'react-native-webview', :path => '../node_modules/react-native-webview'
end
The following is needed to ensure the "archive" step works in XCode.
It removes React from the Pods project, as it is already included in the main project.
post_install do |installer|
installer.pods_project.targets.each do |target|
if target.name == "React"
target.remove_from_project
end
end
end
Thank you

facing the same issue.

Faced same issue on React Native 0.58.1. Fixed with following steps:

Rm react-native-webview from Podfile
Perform manual linking step 1 && 2 from following tutorial: https://reactnative.dev/docs/linking-libraries-ios
Build successfull

why is this closed?
still happening

Having same issue

Any Solution so far ???

Facing same issue ? Any solution.

Re-opened via another ticket: #1789


API - Tool Product Settings

Description
POSTing to the API endpoint "tool_product_settings" with "product" parameter other than 1 (for example, using "product":2 in the body of the request) causes Dojo to ignore it and instead creating on product 1.
Also, it's required a parameter that is not listed.

Bug description
POSTing to the API endpoint "tool_product_settings" with "product" parameter other than 1 (for example, using "product":2 in the body of the request) causes Dojo to ignore it and instead creates it on product 1.
Steps to reproduce
Just using curl to create the POST request and sending it triggers this bug.
Expected behavior
Create the tool setting in the correct product.
Deployment method (select with an X)

[X] Docker Compose
[ ] Kubernetes
[ ] GoDojo

Environment information

Operating System: Amazon Linux 2
DefectDojo version (see footer) or commit message: v2.9.0

Logs
N/A
Sample scan files
N/A
Screenshots
Request:

Response:

Additional context (optional)
Add any other context about the problem here.

The issue is now fixed, thanks!


Difficult to get decorators to work

In our TypeScript + React Native project I'm trying to switch our jest tests to swc. It's working great for the part of my codebase that does not use mobx, however, as soon as the transformer comes accross mobx decorators (observer, observable, action etc) the following error is thrown:
 TypeError: decorator is not a function

      20 |   }
      21 |
    > 22 |   @observable
         |                ^
      23 |   public width = 0;
      24 |
      25 |   @observable

      at src/styles/dimension-store.tsx:22:16
          at Array.reduce (<anonymous>)
      at _applyDecoratedDescriptor (src/styles/dimension-store.tsx:21:41)
      at Object.<anonymous> (src/styles/dimension-store.tsx:134:258)
      at Object.<anonymous> (src/styles/style-constants.base.ts:3:51)TypeError: decorator is not a function

I tried to patch it by including reflect-metadata or core-js in my test setups, it does remove the error but it causes mobx to fail (properties that are decorated are no longer initialized). Can't seem to figure out what the problem is.
I have the following jest config:
transform: {
    "^.+\\.tsx?$": [
      "@swc-node/jest",
      // configuration
      {
        esModuleInterop: true,
        dynamicImport: true,
        experimentalDecorators: true,
        emitDecoratorMetadata: true,
      },
    ],
    "^.+\\.js$": `${packageRoot}/node_modules/react-native/jest/preprocessor.js`,
  }

See https://mobx.js.org/enabling-decorators.html#typescript , use Mobx decorator needs the useDefineForClassFields: true with experimentalDecorators: true which is not supported by swc.
/cc @kdy1

That makes sense, perhaps time to look into upgrading mobx at last (they dropped the decorator syntax in v6)

@Brooooooklyn  Nevermind, I just did some research, I'm not using the useDefineForClassFields at all in my TSConfig, because I'm still on mobx 5. So I suppose it's a different issue after all?

Fixed by importing reflect-metadata in my test setup.


Composing scopes idea

I'm looking at this code in pundit/policy_finder.rb and wondering if it could be changed to allow dynamic scope definitions, to support composition / multiple-inheritance, as explained in stackoverflow
def scope
  policy::Scope if policy
rescue NameError
    nil
end

def scope
  if policy
    policy::Scope || (policy.scope if policy.respond_to?(:scope))
  end
rescue NameError
    nil
end

Does this look like a viable PR (with tests etc) or is there a reason for using class definitions I haven't thought of?

Be aware that policy in this case is the policy class not an instance thereof. I can't think of anything that such a class method could do that would be in any way sensible. The only thing I can think of is doing really evil stuff with thread-locals, which is not something we should encourage.

Can you provide a user case where you would want to use this?

Hej Jonas
Thanks for looking at this!
While writing up a reply I spotted a simpler solution that doesn't need what I was talking about, so I'm closing the issue.
Method definitions and modules within the scope class work just fine. I got myself stuck on another idea somehow and missed the simple solution.
I wanted to build a whitelist for resource access, and am now doing it like this:
class ApplicationPolicy < BlankPolicy
  attr_reader :admin, :record

  def initialize(admin, record)
    raise Pundit::NotAuthorizedError, "must be logged in" unless admin
    @admin = admin
    @record = record
  end

  # this should be extended and added to by specific policy scopes as they add access
  def whitelisted_ids
    []
  end

  def resolve
    if admin.product_owner?
      scope.all
    else
      scope.where(id: whitelisted_ids)
    end
  end
end

with policy classes like this:
class PagePolicy < ApplicationPolicy
  # enable assigned work to be done
  include WorkAssignmentsSharedPolicy
  # enable section editors to update their tagged content
  include TagsAclSharedPolicy

  class Scope < ApplicationPolicy::Scope
    include TagsAclSharedPolicy::ScopeExtension
    include WorkAssignmentsSharedPolicy::ScopeExtension
  end
end

module TagsAclSharedPolicy
  module ScopeExtension
    def whitelisted_ids
      super + scope.tagged_with(admin.tags_acl, any: true).pluck(:id)
    end
  end
  # then show? edit? etc definitions
end

And this class doesn't have tags or work assignments, but inherits permission from the question it belongs to
class AnswerPolicy < ApplicationPolicy
  class Scope < Scope
    def whitelisted_ids
      # add answers in questions permitted by question policy
      super + QuestionPolicy::Scope.new(admin, Question.all)
                                   .resolve.map(&:answer_ids)
                                   .flatten
    end
  end
end

I may turn the longer writeup into a blog post or something, but it's probably not very interesting, and the crux of it is shown in the code above.


Installing puppet bolt on Ubuntu only lists bionic and focal

In the documentation on how to install Bolt on Ubuntu the Ubuntu versions listed are: 18.04 and 20.04. Suggested change: add Ubuntu 22.04

I tested the existing instructions for previous versions of Ubuntu, seeing if it would work for 22.04. After running those commands, it gave me no error, so I guess the process is the same. I'll fork and update the documentation accordingly.


Number commands not working like 01 is for payload but it is not working when we type 01 please fix in

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
https://youtube.com/c/sabrizaki

When I put the number I want in it exits. Doesn't matter what number either

You need to enter '1' instead of '01'.That fixes the problem.


dwarf: missing debug info for type used only in interface

Go version
go version go1.21.5 linux/amd64
What operating system and processor architecture are you using (go env)?

GOARCH='amd64' GOOS='linux'

GO111MODULE=''
GOARCH='amd64'
GOBIN=''
GOCACHE='/home/ajwerner/.cache/go-build'
GOENV='/home/ajwerner/.config/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='amd64'
GOHOSTOS='linux'
GOINSECURE=''
GOMODCACHE='/home/ajwerner/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='linux'
GOPATH='/home/ajwerner/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/home/ajwerner/sdk/go1.21.5'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/home/ajwerner/sdk/go1.21.5/pkg/tool/linux_amd64'
GOVCS=''
GOVERSION='go1.21.5'
GCCGO='gccgo'
GOAMD64='v1'
AR='ar'
CC='gcc'
CXX='g++'
CGO_ENABLED='1'
GOMOD='/dev/null'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=/tmp/go-build669291347=/tmp/go-build -gno-record-gcc-switches'

What did you do?
Compiled the below go program.
package main

import (
        "context"
        "fmt"
)

type contextKey struct{}
type contextValue int

func main() {
        ctx := context.WithValue(context.Background(), contextKey{}, contextValue(1))
        value := ctx.Value(contextKey{})
        fmt.Printf("%T %v\n", value, value)
}

What did you expect to see?
I expected to see debug information for the type contextValue. In particular, I was expecting to see a DW_TAG_base_type entry which would have the relevant DW_AT_GO_runtime_type which can be used to lookup the type of the interface boxed contextValue.
This means that dlv cannot unbox the value in the interface. See here:
> main.main() ./test.go:13 (PC: 0x4b6b40)
     8: type contextKey struct{}
     9: type contextValue int
    10:
    11: func main() {
    12:         ctx := context.WithValue(context.Background(), contextKey{}, contextValue(1))
=>  13:         value := ctx.Value(contextKey{})
    14:         fmt.Printf("%T %v\n", value, value)
    15: }
(dlv) print ctx
context.Context(*context.valueCtx) *{
        Context: context.Context(*context.emptyCtx) *0,
        key: interface {}(main.contextKey) {},
        val: (unreadable could not resolve interface type),}

What did you see instead?
I found no such entry in the DWARF for the program.
If I use the value in a function that does not get optimized away, or I use it in a struct field or assign it to a variable, then indeed it exists and has an entry that looks like:
0x0008a0f5:   DW_TAG_base_type
                DW_AT_name      ("main.contextValue")
                DW_AT_encoding  (DW_ATE_signed)
                DW_AT_byte_size (0x08)
                DW_AT_GO_kind   (0x02)
                DW_AT_GO_runtime_type   (0x0000000000007440)

Note that the program prints the following, so the type exists and is compiled into the binary.
main.contextValue 1

I agree with @aarzilli , same problem here.  I am going to close out as a dup, please reopen or comment if needed. Thanks.

Dup of #46670.


Installed ipcalc not detected

$ git log --oneline --no-decorate -1
741a5d5 reduced default shodan scanning threads - threads reduced from 20 to 10, Shodan introduced stricter rate limits (and longer cooldown timers)
$ dpkg -l ipcalc
Gewünscht=Unbekannt/Installieren/R=Entfernen/P=Vollständig Löschen/Halten
| Status=Nicht/Installiert/Config/U=Entpackt/halb konFiguriert/
         Halb installiert/Trigger erWartet/Trigger anhängig
|/ Fehler?=(kein)/R=Neuinstallation notwendig (Status, Fehler: GROSS=schlecht)
||/ Name           Version      Architektur  Beschreibung
+++-==============-============-============-=======================================
ii  ipcalc         0.42-2       all          parameter calculator for IPv4 addresses

$ ipcalc --version
0.5
$ ./asn github.com

╭─────────────╮
│ ! WARNING ! │
╰─────────────╯

The following tools were not found on this system:
 - ipcalc

The following features will be disabled:
 - CIDR deaggregation (due to incompatible ipcalc version - v0.41+ for Debian-based or v1.0.0+ for RHEL-based required, but you have v0.5)

Please install the necessary prerequisite packages
for your system by following these instructions:

>> https://github.com/nitefood/asn#prerequisite-packages <<

Press ENTER to continue...
Interrupted

Hi @paulmenzel! The ipcalc saga continues.
Can you please confirm if you have the -d or -r option on ipcalc v0.5? What distro and version?
Thanks.

See the output of dpkg above, but it’s Debian sid/unstable, and it only has the switch -r, and not -d apparently. (It worked up until fetch the latest changes today.)
$ ipcalc --help
[…]
 -n --nocolor  Don't display ANSI color codes.
 -c --color    Display ANSI color codes (default).
 -b --nobinary Suppress the bitwise output.
 -c --class    Just print bit-count-mask of given address.
 -h --html     Display results as HTML (not finished in this version).
 -v --version  Print Version.
 -s --split n1 n2 n3
               Split into networks of size n1, n2, n3.
 -r --range    Deaggregate address range.
    --help     Longer help text.
[…]

Ok. Apparently what should be ipcalc v0.42 actually reports itself as v0.5 for some reason which slips me (as also noted here). I had checks in place to make sure version was >=0.41 (as per issue #34) which was indeed correct, but didn't account for ipcalc's mission to drive people crazy with versions, forks and different implementations under the same name.
The initial trouble started with @odoucet's system running v0.2.x (which didn't have range deaggregation), so I'll fix it by checking if >= 0.5 instead of 0.41 - that should settle the matter for Debian-based systems, account for Redhat-based version overlapping, and hopefully mark the end of the saga :-)
Thanks for reporting!


Update

Calling attention to the maintainers of this repo that this PR seems to be an attack to execute malicious code via CI and retrieve credentials possibly available during the build execution.
You might want to make sure they were not successful /cc @clairernovotny @dnfadmin
I'm watcher on this repo and was notified of this PR, which is how I noticed it.
Similar PRs sent to other repos that I noticed before this malicious user @secopsinternal made his profile private:

https://github.com/dotnet/arcade/pull/12541
https://github.com/dotnet/xdt/pull/491
https://github.com/Azure/DaaS/pull/93

Thank you @augustoproiete! I've reported the user to GitHub for the moment and am investigating additional action.

Thanks (MUCH) for the report @augustoproiete and for the quick response @baronfel. We blocked the user this morning and reported same to GitHub. Sorry for the delay in sharing that.
Reporting these types of concerns is very much the right thing to do and we appreciate it.


Many notifications if an instanz goes on and off

Describe the bug
If a device goes on and off many times in a short delay , device watcher send many notifications.
I have seen with the broadlink adapter, that the system tried to start the instanz , it goes fail and the system try to start again and so on.
Every time a notification was send.
Also on javascript or other adapters.
It is possible to define a general timeout for all instances ?
So that the first time the notification was send immediately , but when an event occurs within the timeout, no notification should be send.
I know , that I can define a time out for each instance, but not for all globally.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '...'
Scroll down to '....'
See error

Expected behavior
only one time notification
Screenshots & Logfiles
If applicable, add screenshots and logfiles to help explain your problem.
Versions:

Adapter version: V2.8.0
JS-Controller version:
Node version:
Operating system:

Additional context
Add any other context about the problem here.

Thanks for your suggestion. I will add an option for the user to change the time global.

Ok, thx.
In the case there is an individual time out, this individual time out should be preferred.

@SurfGargano it should work with v2.8.1


Build Linux releases on big managed runners

Test run: https://github.com/iree-org/iree/actions/runs/2879203002
skip-ci

custom managed runners evidently don't really work yet. I theoretically created them, but they seem to be impossible to target and the job just hangs indefinitely

Ok fixed up the package suffix thing. Another build: https://github.com/iree-org/iree/actions/runs/2914524943

Once more: https://github.com/iree-org/iree/actions/runs/2914625750

This run almost succeeded: https://github.com/iree-org/iree/runs/8006465144

I think this is mostly ready for review

Thanks for the ping. Yes we use it downstream for our SHARK-runtime releases. We can carry a fork or if we want we can leave the "one shot" builds that we use to be self hosted.

Thanks for the ping. Yes we use it downstream for our SHARK-runtime releases. We can carry a fork or if we want we can leave the "one shot" builds that we use to be self hosted.

Not sure I understand what you mean by the second option, but I think you might actually be able to create managed runners that match ours. The option only appears for me at the org level, for some reason.
I think the URL should be:
https://github.com/organizations/nod-ai/settings/actions/custom-hosted-runners/new

oh that is a good idea to add a similar named runner. We have self-hosted runners anyway.


Add plugin to display Hitpoints and Prayer in the system tray

Closes #1340
This is a new plugin that sets and updates the system tray icon to an hp bar, prayer bar, or both. It also can display the current value for either. I chose the blocky over anything fancier version for 2 reasons: it's easy to dynamically render, and when it's small it's most readable. This icon includes an offset for mac; windows centers the icon but mac relies on transparent padding (the current icon has this problem if you compare to the default icons).
Screenshots
With hitpoints

With hitpoints over 99

With prayer (and config panel)

Showing both bars and no text (and config panel)

Possible enhancements

Make it nicer looking

I chose the colors based on what I thought looked nice, but this can use actual HP bar colors if desired
Change the font, I just picked the RS font because it looks decent low-res
Text alignments are arbitrary and can be changed as needed

Add tooltip

Might be unnecessary for a quick afk tabbed-out view (which is the idea behind this in general)
Can show things like prayer time left, poison, etc

Color differently if prayers are active
Color differently if poisoned/venomed
Re-render only on changed values

It's not at all intensive to create/set this icon per-tick (no flickering on mac at least), so checking previous tick values may be unnecessary

Unify with another plugin

This is similar to the status bars on inventory but didn't have enough in common to justify

Seeing a little weirdness with text alignment so I'll need to look at that.
New mac/windows screenshots coming.

Can someone on Windows check this looks ok?

Due to legal requirements at my new employment I’m no longer able to contribute.
If someone still thinks this is still worth adding or improving they can take this over (it looks a lot nicer than the old screenshots now, on Mac at least).


Add support for building with Clang under Windows

I was wrong, the current version of experimental/coroutine from libcxx needs some manual preprocessing to compile cleanly (or perhaps it always did and I just forgot I did so 10 months ago).
Even still, providing a coroutine_handle implementation in this library seems out-of-scope.

The first step is the cakefile work required to get a build of clang (or clang-cl) building and linking correctly.
From memory, I was running into problems trying to use clang-cl (which pulls in some MSVC headers) in conjunction with libc++. I also think clang-cl had some command-line limitations that made it difficult to override the system header include paths.
I think after working around some of these issues with some hacks I was still running into issues at link time with missing symbols, however.
I must admit it was quite a while ago that I last tried to get it working, so things may have gotten better with clang/clang-cl on windows since then.


Add and Delete Module Command

Current features:
add and rm commands now work, updating the My Modules list on the right. AddCommand checks module code against loaded module list instead of with the API. Appropriate errors are shown for:

Invalid module code format
Repeat modules upon adding
Modules with correct format but not in NUSMods e.g CS9999
Trying to remove a module not in My Modules

Commands:
add <module to add>
rm <module to delete>
Screenrecord:
https://user-images.githubusercontent.com/87590904/195651707-9b09bd4c-e44e-4b17-bf30-7ceac103479c.mp4
Todo:

[X] Merge conflicts resolved
[x] Test files: not able to do comprehensive AddCommand and DeleteCommand automated tests since we don't have a way to compare expected vs actual state of the model yet. We can update these tests later.
[x] Take screenshots and product demo video and add to project notes document v1.2

The command works on my device, thanks a lot @huzaifa1712 for helping with the integration part from AddressBook to ModuleList. LGTM 👍

Are we done with the changes?
Just tested, LGTM.  If @shenyih0ng or @SeekSaveServe has no other comments, let's merge this

LGTM 👍


investigate aiohttp in place of httpx

Investigate using aiohttp in place of httpx due to some quirks with read timeouts.

FYI - I think aiohttp works fine. I have no objections to replace httpx with it.

I've been running this for a few weeks now, and it's rock solid stable (fans still crash, but I do not believe it's related). Any chance you can send a PR, so we can move on? :-)

I filed https://github.com/jimpastos/ha-modernforms/pull/19 with everything pushed to my fork.  It's got multiple changes, I don't hve time to separate them out, but if you or anyone wants to just merge pieces, I have no objection.


Remove the purple toolbar

the purple toolbar is appearing on each and every fragment so it should appear only on folder fragment

I want to work on this issue under JWOC

If we remove this toolbar then we wouldn't be able to use option menu on fragment.

If we remove this toolbar then we wouldn't be able to use option menu on fragment.

Thats why I have mentioned that it should only visible on folder fragment not anywhere else