Need smarter contextmenu items

People working on the context menu of JupyterLab have run into various problems (see jupyterlab/jupyterlab#4529 for a good overview). After spending a while thinking about it, I believe that all of the issues can be resolved by making the contextmenu items in @phorsphor/widgets a little bit smarter/more aware of the circumstances under which they were invoked. Specifically, I think that two changes are required:

There needs to be a way to pass information contained in the target of a contextmenu event (ie the node that the user actually right-clicked) on to the command of a contextmenu item.

There needs to be a way to decide whether to hide/show a contextmenu item based on the result of a callback that takes the contextmenu event target as an argument. This would be in addition to (or possibly instead of) the existing selector property.

I've already written a PR (#355) addressing the first point.

For 2, you can use CSS dataset selectors in the registered context menu item.

Hmm, as long as the context menu handler is scoped to an appropriate selector (not body), I think it is fair to give it a chance to show or not based on the current event target (performance-wise).

I agree with the thesis of this thread. We've known this for a while. So far nothing has hit me as the "right" way to provide that information. I agree with @blink1073 that CSS selectors should be enough to control visibility, provided you tag your node with enough information.
My current thinking is I'd like to just find a clean way to pass the DOM node to command which is executed, but there may be some unintended consequences to that, such as what to pass when the command is invoked directly?

At this point we are not getting rid of selectors, since that would be a breaking change.

but there may be some unintended consequences to that, such as what to pass when the command is invoked directly??

My thoughts exactly. When I set out to write #355 I did originally intended to just have it pass the target node to a command, but then I realized that would make it awkward to write commands that could both be used in contextmenu items and invoked directly. The design I settled on is intended to make invocation simple, since this way the names of the dataset properties you'd set when invoking a command via a context menu would be the same as the names of the args you'd pass when invoking the command directly.
Still, if you think it's better to pass the whole node, I guess it wouldn't be that big a deal? You would just need to test for/branch on the presence of an optional node arg (or static property, or whatever) within the body of relevant commands.

What is the status of this issue? I am looking for a way to select only certain file according to their extension ('.zip', 'tar,bz2', etc). See https://github.com/hadim/jupyter-archive/issues/19 for details.
In CSS you can add multiple selectors with a comma: const selectorOnlyArchive = '.jp-DirListing-item[title$=".zip"], .jp-DirListing-item[title$=".tgz"]';
But it does not seem to work in Jupyter. And actually the documentation is pretty clear about it: https://phosphorjs.github.io/phosphor/api/widgets/interfaces/contextmenu.iitemoptions.html
The selector must not contain commas.
So is there is a way with the current API to achieve this kind of selection only with the selector attribute?

Indeed this is what I ended up doing. See https://github.com/hadim/jupyter-archive/issues/19

provided you tag your node with enough information.

@hadim Following this piece of advice from @sccolbert was ultimately how I ended up fixing all of my selector issues. By being careful with classnames and dataset contents, I was able to solve ~90% of my contextmenu item selector issues.
The trickiest cases all involved situations in which I needed to distinguish between two different areas, but for which there were no mutually exclusive selectors. For these cases, I either rearranged my existing DOM, or I added an extra invisible div to help distinguish one area from another.


Fix identity fuse for oneDNN

Description
During testing https://github.com/dmlc/gluon-nlp/pull/1581 it occurred that identity fuse doesn't work properly on big graph, but works in unit test - that was caused by lack of stop condition in subgraph selector.
This PR fix this issue and expand test to cover such cases

@mxnet-bot run ci [centos-gpu, unix-cpu, unix-gpu]

@mxnet-bot run ci [centos-gpu, unix-cpu]

@mxnet-bot run ci [centos-gpu]

@mxnet-bot run ci [centos-gpu]

@mxnet-bot run ci [all]


missing public IP in examples/logging-demo

Hi @satnam6502 @nikhiljindal , I followed the instructions in examples/logging-demo, and there was no "external IP" field for elasticsearch-logging or kibana-logging, as shown below:
../../_output/local/bin/linux/amd64/kubectl get services kibana-logging -o json
{
    "kind": "Service",
    "apiVersion": "v1beta3",
    "metadata": {
        "name": "kibana-logging",
        "namespace": "default",
        "selfLink": "/api/v1beta3/namespaces/default/services/kibana-logging",
        "uid": "9dc6f856-f358-11e4-a58e-42010af09a93",
        "resourceVersion": "31",
        "creationTimestamp": "2015-05-05T18:57:45Z",
        "labels": {
            "kubernetes.io/cluster-service": "true",
            "name": "kibana-logging"
        }
    },
    "spec": {
        "ports": [
            {
                "name": "",
                "protocol": "TCP",
                "port": 5601,
                "targetPort": "kibana-port"
            }
        ],
        "selector": {
            "name": "kibana-logging"
        },
        "portalIP": "<IP_ADDRESS>",
        "sessionAffinity": "None"
    },
    "status": {}
}

I start up my cluster via cluster/kube-up.sh. Did I miss something or is it a bug? Thanks.

Correct. This demo does not create an ELB so there is no external IP address.
Access is via service proxy at the master.
Type:
$ kubectl cluster-info

and you should the URLs for Elasticsearch and Kibana.

@caesarxuchao please let us know whether:
a. Satnam's comment resolves your issue
b. the demo should be updated to prevent this misunderstanding
c. the issue isn't resolved
Thank you!

Yes, Satnam's comment resolve the issue. I updated the demo in #7824.

Awesome, thanks for sending the patch! We'll discuss there. Always fantastic to have docs updated regularly by folks trying them.


Check if the Path Exists

HI,
In the documentation it was mentioned that if we are fetching multiple elements from different paths then instead of parsing JSON again and again at each step, it's better to create an object like below,
Object document = Configuration.defaultConfiguration().jsonProvider().parse(json);
But if I want to check at each stage whether the Path exists or not before extracting the element, right now I think the only way is to parse the JSON again and again, is there a way we can avoid parsing again but to check if element exists or not before extracting it ?
Configuration conf1 = Configuration.defaultConfiguration();
Configuration conf = conf1.addOptions(Option.DEFAULT_PATH_LEAF_TO_NULL);
JsonPath.using(conf).parse(json). read("$.path_to_check")

I am interested in a alternative approach as well.

You can test for existence by add the SUPPRESS_EXCEPTIONS option:
Configuration conf = Configuration.defaultConfiguration().addOptions(Option.SUPPRESS_EXCEPTIONS); String result = JsonPath.using(conf).parse(json).read(path); if (result != null) { ... }

I actually did this. But in this case:
DocumentContext outJson = JsonPath.parse("{}", jsonPathConfiguration);
// Whereas the jsonPathConfiguration has been set in the constructor of my class with the suppress exception option
String jsonPath = "$.items.item[*].ean";

String[] jsonPathBranches = jsonPath.split("\\.");

for(String currentBranch : jsonPathBranches) {
    // currentBranch will get concated with the previous value
    // e.g.
    // 1. loop -> $
    // 2. loop -> $.items
    // 3. loop -> $.items.item[*]
    // and so on
    if(outJson.read(currentBranchModified) == null) {
    // create the path manually
    }
}

But outJson.read(jsonPath) returned [] which is not null.
So this check if null wont work in my case. Or am I missing something?
My goal is to create the json structure from given jsonpaths.

public boolean isPathValid(String path){
    try {
    String value = JsonPath.read(this.userData, path).toString();
    } catch(PathNotFoundException e) {
            return false;
    }
    return true;
}

Would you accept a PR which adds @anandujjwal13 's  implementation as an method for JsonPath?

It will nice if resolve Null problem in path syntax.
like:
aaa?.bbb[0]?.ccc?.
mean if aaa not null then get aaa.bbb[0], and aaa.bbb[0] not null, then get aaa.bbb[0].ccc.

That is especially useful for stuff like GraphQL fetching, when we want to stop the nested search if an object is NULL.
E.g. user { child { firstname } }. If the child is NULL, we want this result: user { child: NULL }, if not but the firstname is NULL, we want user { child { firstname: NULL } }. But today, we'll have the latest result, whether the child is NULL or not. We therefore lose an information (whether the child exists or not).

Why not simply add a path function 'exists()'?
That would be so much more convenient than shutting down exception throwing for the sake of one optional path.

hi, I create a PR for this refers to @anandujjwal13 's implementation.

any update on this issue?

@ChrisLeeBare if the path is not definite then you would get an empty array as reponse, if it is finite the response would be empty. In your example, the path has a * hence you got an empty array


Ruby/C++ Interop Flake timeout_on_sleeping_server

Warning! PATH is not properly set up, '/usr/local/rvm/gems/ruby-2.1.8/bin' is not available,
         usually this is caused by shell initialization files - check them for 'PATH=...' entries,
         it might also help to re-add RVM to your dotfiles: 'rvm get stable --auto-dotfiles',
         to fix temporarily in this shell session run: 'rvm use ruby-2.1.8'.
Using /usr/local/rvm/gems/ruby-2.1.8
stdin: is not a tty
I0708 00:03:56.766125924       1 ev_epoll_linux.c:84]        epoll engine will be using signal: 36
D0708 00:03:56.766162041       1 ev_posix.c:106]             Using polling engine: epoll
src/ruby/pb/test/client.rb:83:in `assert': timeout_on_sleeping_server: status was wrong (AssertionError)
	from src/ruby/pb/test/client.rb:343:in `rescue in timeout_on_sleeping_server'
	from src/ruby/pb/test/client.rb:337:in `timeout_on_sleeping_server'
	from src/ruby/pb/test/client.rb:473:in `call'
	from src/ruby/pb/test/client.rb:473:in `main'
	from src/ruby/pb/test/client.rb:478:in `<main>'
I, [2016-07-08T00:03:56.951403 #1]  INFO -- : ... connecting securely to localhost:34451
W, [2016-07-08T00:03:57.953452 #1]  WARN -- : bidi: read-loop failed
W, [2016-07-08T00:03:57.953526 #1]  WARN -- : 13:{"created":"@1467936237.953122416","description":"RST_STREAM","file":"src/core/ext/transport/chttp2/transport/frame_rst_stream.c","file_line":107,"http2_error":2} (GRPC::BadStatus)
/var/local/git/grpc/src/ruby/lib/grpc/generic/active_call.rb:47:in `check_status'
/var/local/git/grpc/src/ruby/lib/grpc/generic/bidi_call.rb:212:in `block (2 levels) in start_read_loop'
/var/local/git/grpc/src/ruby/lib/grpc/generic/bidi_call.rb:200:in `loop'
/var/local/git/grpc/src/ruby/lib/grpc/generic/bidi_call.rb:200:in `block in start_read_loop'

https://grpc-testing.appspot.com/job/gRPC_interop_pull_requests/6184/testReport/junit/(root)/tests/cloud_to_cloud_ruby_c___server_timeout_on_sleeping_server/
This was on a python PR, I'm not sure if its been seen in master as well.

This is somewhat blocking PR submitted by external developers. There is an issue with Jenkins where hitting "Rebuild" will just fail immediately for those PRs. So there is no easy way to retry these flakes.

This appears to no longer be happening. I think it was fixed by #7291. If we see it again, we can reopen this issue.


Where to give input data to generate ground truth samples from?

I'm trying to generate ground truth examples, where are we providing the data here?

It's set up to pull documents from an Azure AI Search index, but I failed to document the environment variables. Will do now!

Here you go:
https://github.com/Azure-Samples/ai-rag-chat-evaluator?tab=readme-ov-file#running-the-generator-script
Let me know if you have follow-up questions!


Round E Competition

Hi there. I'm new to Kickstart and I was wondering if you did go through Round E? I'd love to learn from your code so that I can grow as a programmer. Thank you very much.

Just updated. Thank you very much!


v2.1 Add ability to version installations of docker-ce and docker-ce-cli

@Fizzadar Currently tested on ubuntu, haven't got round to testing on CentOS yet.

I've now also created a draft PR: https://github.com/Fizzadar/pyinfra-docker/pull/5
It modernizes the example (out of scope here) and also adds the usage of the docker_version option created here.

Apologies for the delay guys, busy times! Thanks for the changes 🙏, LGTM!


Better setup help for CLI

Adjust the inquirer code in index.js to be more descriptive about AWS credentials, region etc.
Tied to: https://github.com/williamparry/cloudRIG/issues/36 and https://github.com/williamparry/cloudRIG/issues/40

CLI is for maintenance.


[FEATURE] - Featured content layout make responsive

We should make layout responsive as I see a big usage of this

Sure, we can explore this for the future. There's the auto mode until then, which will cover 90% of use cases 😄


Add toggle to hide/show summary

This PR adds the ability to hide or show the summary feature via the config.yaml file.
Example:
corpus_id: 1
customer_id:

search_title: "Ask Paul Graham"

enable_source_filters: False
source: ""

enable_summary: True

By default, the enable_summary value is set to True.
When enable_summary is on, the search header text is set to "References." When it's off, the text is set to "num Search Results", where "num" is the number of search results displayed. However, I'm not sure if it's necessary to show the number of search results since it's always a fixed number.

I think we can just show a fixed title "Search Results" without the number.
Also, note that in sendSearchResults.ts the number of results is really "10" if reranking is on, or rerankNumResults if not (could be 50 or 100 or whatever it's set to). I think that's okay and if reranking is turned on and rerankNumResults is 100 then we should show 100 results, but in some way it feels like not the best user experience to see so many results without "paging". Perhaps we can add pagination later on for search only (no summarization) use case, but I would not prioritize it so high.


Add support for Simulator

Switches to BepInEx and adds support for build 93.
Closes #9.
Implementation differences:

Controllable locomotives are now determined by the presence of the RemoteControllerModule component, which only the DE2 and DH4 have.
Control dirty state is now updated by ControlBaseImpl's ValueChanged event.
The configuration now has a "Save Permissions" button. This is required since the config is only saved when a ConfigEntry's Value is set to a new value.

Looks like the mod has abandoned this, could you possibly compile this and make it available in DLL format?

It's only been three days. Zeibach is a busy guy, you can either compile it yourself (The Mapify documentation may help) or wait for him to update it.

It's only been three days. Zeibach is a busy guy, you can either compile it yourself (The Mapify documentation may help) or wait for him to update it.

Alright, on my end it shows he hasnt logged into Github since November hence my comment, eitherway thanks for the files Insprill, I install Visual Studio and compile it myself temporarily, by any chance do you know if he has a contribution page, I already sent you over a few bucks but want to send him something to for the amazing program

I have seen Zeibach is now developer  so he had to fix main game issues before coming back here.


"Stejari" is listed in the drop-down menu

Description:
"Stejari" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Stejari" in the search bar.

Expected results
The location is listed in the drop-down menu as "Stejari, Argeș (Băiculești)".


isolation:  close file when exits

The file is not closed，and the sock connection has not been truly closed, and it occupy connections for libvirtd

/approve
Hi @grass-lu,
nice catch! Can you please keep the templated description and make sure you use the release note section? It is worth noting the bug fix.

/lgtm
/ok-to-test
Thank you!

/release-note-none

/cherrypick release-1.0

/retest


improve SEO

As an admin I need to drive more traffic to the site to save money and maximize visibility

Resources
https://hackernoon.com/how-we-grew-our-seo-to-70k-visitors-a-month-e6ea34ce1555
Tasks
TBD
Definition of Done
TBD

not a priority at this time.


fix: upgrade to latest builders

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: jstrachan
If they are not already assigned, you can assign the PR to them by writing /assign @jstrachan in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/test all

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

Test name
Commit
Details
Rerun command

ng
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test ng

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.

@jstrachan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

ng
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test ng

gitops
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test gitops

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.

/test ng

@jstrachan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gitops
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test gitops

ng
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test ng

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.

/test ng

@jstrachan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gitops
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test gitops

ng
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test ng

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.

/test tekton

/test ng

@jstrachan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gitops
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test gitops

ng
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test ng

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.

/test ng

@jstrachan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gitops
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test gitops

ng
168b6d72d362d4b0d9658dd1978c5e764efdbe66
link
/test ng

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.

/test ng

@jstrachan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gitops
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test gitops

ng
007dfb867f732944c3f468c864a499044bba4fde
link
/test ng

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.

/test ng

@jstrachan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gitops
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test gitops

ng
65686f68504d61a475388a185a496066d0249f77
link
/test ng

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.

/test ng

@jstrachan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gitops
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test gitops

ng
65686f68504d61a475388a185a496066d0249f77
link
/test ng

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.

/test ng

@jstrachan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gitops
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test gitops

ng
c5c87e5a88b598969fdffae601d3cfd39223c4ea
link
/test ng

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.

/test ng

@jstrachan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gitops
5bfb92a318711701ac653e7c38bd034ec3aa650e
link
/test gitops

ng
2aa9f5f7b78799e22f9f7bffd7daa4bedd41d40f
link
/test ng

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.

/test all


Update from iandees/aws billing to slack

I didn't have access to create branches in this repo, so I just added it to my repo.
But this should make us able to use amortized costs instead of unblended costs
I'm having a bit of trouble deploying and testing this, I think because my python is 3.11, and my pipenv installation is a newer version, which has some backward incombatibilities, that hasn't been addressed in the version of serverless-python-requirements yet.
So if you could deploy and test it for me, that would be great! If there are any issues, we can just revert, and look into it together, whenever you have time. The changes aren't urgent :)

@MarleDK I just added you as a maintainer :p I'll deploy and test it 😊


Sorting by severity is not performed by severity value

Sorting by severity is not performed by severity value

which page/url is this? For me it's working on open/closed/all findings list.

which page/url is this? For me it's working on open/closed/all findings list.

I did not test in all, but in Open Findings it starts loading by severity but then it "breaks".
In my case it starts with Critical High Medium on load. When it loads becomes Medium High Critical.

Ok. I've found the issue. if you click that little icon instead of clicking on severity text, it correctly sorts

Not exactly sure what is in control of the column sort here.  Clicking on the arrow with the stairstep icon appears to do a sort via the UI (didn't see any calls to the server), verses clicking on the 'Severity' column name makes a call to the server to retrieve new data.
In both cases, the severity appears to be ascending (or descending) with regard to the Severity strings of 'Critical', 'High', 'Low', etc versus a numeric/ordinal version of Critical High Medium Low, etc.  Not sure if the data retrieved is sorted correctly or if the UI elements are automatically sorting the elements incorrectly (aria-sort = ascending or descending).
I've uploaded some screenshots showing the improper ordering of the Severity (which appear to be sorted based on literal String of Critical/High/Medium/Low/Info.

Ok, this occurs because we recently fixed the datatables sorter that was broken. And it now sorts by severity but the javascript only knows the text labels, so it sorts them by label even though the server initially sent them ordered by numerical severity. We'll have to check if the datatables library supports sorting by something other than the text label. Or we should remove client side sorting for severity. Or remove client side sorting alltogether when we're in paged mode :-)

I fixed the default sorting on initial page load on #2311. I opened #2312 as a new bug report to fix the remaining part as the issue here is already mixing 2 things :-)


Various errors when compiling on armv7h using qemu-arm-static

Hi there. For some time now I have had a couple of errors pop up when running with qemu-arm-static inside an armv7h chroot for crosscompiling armv7h software.
Most notable and annoying ones are:
QT
I get missing modules (these vary depending on project) when compiling QT applications:
Project ERROR: Unknown module(s) in QT: core gui widgets
Project ERROR: Unknown module(s) in QT: core

The exact same procedure, with qemu-aarch64-static in an aarch64 chroot works fine.
It also builds fine on an armv7h device without qemu at all.
GTK
I get unrecognised image formats when compiling some GTK apps. Like this one:
[3/71] Generating manager_resources_h with a custom command.
FAILED: resources/manager_resources.h
glib-compile-resources ../resources/pamac.manager.gresource.xml --sourcedir ../resources --internal --generate --target resources/manager_resources.h
failed to load "../resources/package-available.png": Couldn?t recognize the image file format for file ?../resources/package-available.png?
../resources/pamac.manager.gresource.xml: Child process exited with code 1.
[4/71] Generating manager_resources_c with a custom command.
FAILED: resources/manager_resources.c
glib-compile-resources ../resources/pamac.manager.gresource.xml --sourcedir ../resources --internal --generate --target resources/manager_resources.c --dependency-file resources/manager_resources.c.d
failed to load "../resources/package-available.png": Couldn?t recognize the image file format for file ?../resources/package-available.png?
../resources/pamac.manager.gresource.xml: Child process exited with code 1.
[11/71] Compiling Vala source ../src/common_daemon.vala ../src/package.vala ../src/pamac_config.vala ../src/alpm_config.vala ../src/aur.vala ../src/system_daemon.vala.
ninja: build stopped: subcommand failed.

Again, it builds fine with aarch64 instead and on the armv7h device directly.
I am at a loss here, I even tried different versions of the qemu-arm-static file, but they all had the same issues.
I'm using qemu 2.12, qemu-user-static 2.12 and systemd 239 (for systemd-nspawn).

Hi @Strit Could you try the latest multi/qemu-user-static? The latest version is 4.0.0.
We also released new image multiarch/qemu-user-static that can build and run standard architecture image instead of multiarch compatible images.
$ uname -m
x86_64

$ docker run --rm -t arm64v8/ubuntu uname -m
standard_init_linux.go:211: exec user process caused "exec format error"

$ docker run --rm --privileged multiarch/qemu-user-static --reset -p yes

$ docker run --rm -t arm64v8/ubuntu uname -m
aarch64

Thanks.

Sorry. We have since dropped armv7h support, so it is no longer an issue for us.

Sure. Okay. Thanks for sharing it.


Remove public feature flags

The current implementation of usdt (v0.3.2) uses the public feature flag "asm" to decide what code to generate inside the probe macros. We initially did this for a few reasons. First, the Rust asm feature was required, and it's pretty common practice to gate nightly features in a crate with a public feature flag. Secondly, it was very useful for development to pick a real or no-op implementation of the probes.
The time has come to remove this feature flag, however. The asm feature has been stabilized. The asm_sym feature, required for macOS, is on the path to stabilization. More importantly, a feature flag for this doesn't make a whole lot of sense. First, the "real" version of the probes is the whole point of the crate, so turning them off seems silly. Second, feature flags are additive, which I didn't quite appreciate when I built it this way initially. This is a big problem, since it means it's possible for trouble when crate uses two other crates, both of which use usdt. If one enables the probes, it doesn't matter whether the other crate chose to enable them, the probes are there due to feature unification.
This issue tracks removing all public feature flags. The crate will choose the code to be generated in the probe macros solely on the basis of the target OS.

It may be a good time to tackle this. The last feature we really need, asm_sym, stabilized some time ago. If we're willing to push the MSRV to 1.66, then we can use a stable toolchain on all supported platforms and avoid any nightly messiness or feature checks. That MSRV is not too far away from the 1.65 currently implied by some of our dependencies. (E.g., see this checksuite run.)
It would be very nice to get rid of all that cruft and the attendant build.rs files. 1.66 is pretty old now, so this feels like a reasonable choice.


Disallow some parameters for completion commands

<@142041528875876352> ahahahahahahha it has been broke you're not smart enough

Original message by @Przegrany

This seems to actually be fixed already with the new completion query


feat(configs): setup app configuration

may not be the best setup, suggestions? @MurphyMarkW

This looks generally good. I do want to poking at it this afternoon before merging, though, if that's ok. The main thing I want to look at is side-effects of using the python import system for configuration management, since there is technically caching going on when importing modules. I'm not entirely sure how this affects things like reload behavior under apache, if at all.

Yeah I never check how Apache reload works, but the default behavior will reload the settings

Ack, sorry, forgot to respond to this after my testing. This seems to work fine, so I'd say go ahead and merge it.


Add Romain DUCHADEAU project to the website

Adding my project page:

I added my project page in "project/sleep-detection-using-fmri/"
Inside of this folder : The index.md and another folder for pictures

Thanks,
Romain DUCHADEAU (hub polytechnique Montréal - session été 2024)

LGTM!! merging.


Updated and now the server crashes...

Issue Description
Updated King Phisher and now the server will not start. SO many errors I do not even know where to begin...
Reproduction Steps

Attempt Start King Phisher Server

Environment Details

Host OS: Debian, Parrot Security 4.6, 64bit
Hardware: Newer Dell laptop
King Phisher Version: 1.13.1
Error Details / Stack Trace

I start the server with ./KingPhisherServer server_config.yml
Then I get this error:
Loading .env environment variables…
WARNING  automatically updating the database schema from version 8 to 9
CRITICAL database schema upgrade failed with exception: sqlalchemy.exc.OperationalError
Traceback (most recent call last):
  File "/home/hwac121/Pentesting/Exploits/king-phisher-master/.venv/lib/python3.7/site-packages/sqlalchemy/engine/base.py", line 1244, in _execute_context
    cursor, statement, parameters, context
  File "/home/hwac121/Pentesting/Exploits/king-phisher-master/.venv/lib/python3.7/site-packages/sqlalchemy/engine/default.py", line 552, in do_execute
    cursor.execute(statement, parameters)
sqlite3.OperationalError: duplicate column name: credential_regex_username
---------------------------
Which in turn of course causes a buttload of additional errors.

ALSO: MUST I allow it to now install the PostrgreSQL stuff? Because I already have that installed on Parrot with a LOT of stuff configured to work with it...on any Parrot Security Install.

You can't upgrade the database schema for SQLite3, that's why we recommend you use Postgresql. You must have upgraded from a version of King Phisher that used an older database schema. You're only options are to downgrade, wipe the database and or manually export your campaign data and reimport it.
SQLite3 is really only good for testing King Phisher. It shouldn't be used for long term server deployments for this reason. This limitation and these recommendations should be documented in the database section of the wiki.

Ok, fair enough that makes good sense. So my next question is...I am worried that allowing king phisher to install PostgreSQL (when I already have it installed) will screwup my current PostgreSQL install. So in other words, will the KP install detect that and use the already installed DB or do I need to do something special to make sure it uses my currently installed one? What's the best way to handle that. I have absolutely no issue changing what DB it uses if I need to I just need to know what would be the best route for me to take in your opinion please.
PS - Don't be afraid to get a little technical I am a software system engineer, I can take it. LOL

I would recommend skipping the PostgreSQL database install / setup when running the install script. Then setup the database yourself and update the server_config.yml file to replace the database: setting with your connection string. The connection string would take your database name, username, and password that you've configured yourself.
That will be the best way to ensure it doesn't tamper with anything.

Perfect! Thanks for that recommendation. That is exactly what I was thinking but wanted to be sure I was on the right track.

Just FYI as a user of Parrot Security OS:

I made the pre-installed postgres server accept logins/passwords traditionally rather than use ident.
Set a new password.
Edited the server_config.yml file. (Server now starts without a hitch)
Deleted previously made campaigns.
Made a new campaign and all works fine.

Feel free to close this after reading, just wanted to touch base with you on what worked in case you get this question in the future. I was determined to get this app to work again to provide a good tutorial for it and because I LOVE the app! Appreciate all the hard work you guys put into it. =)


Object Detection React App not showing up in browser

I am trying to run the 'Real-Time Custom Object Detection with TensorFlow.js (for Web)' but it does not show in the browser. The localhost page opens but its blank.
Here's the message i got.
Compiled successfully!

You can now view tensorflowjs-object-detection in the browser.

  Local:            http://localhost:3000/
  On Your Network:  http://<IP_ADDRESS>:3000/

Note that the development build is not optimized.
To create a production build, use yarn build.

yarn build

Hello, anyone has suggestion on how to fix this?
Thanks,
CE

check your console maybe its a problem with your Webcam

I thought so too, but I am not getting an error message about webcam.
How exactly should i go about checking webcam settings?
Thanks in advance
-CE

    if (navigator.mediaDevices && navigator.mediaDevices.getUserMedia) {       const webCamPromise = navigator.mediaDevices         .getUserMedia({           audio: false,           video: {             facingMode: 'user'           }         })         .then(stream => {           window.stream = stream           this.videoRef.current.srcObject = stream           return new Promise((resolve, _) => {             this.videoRef.current.onloadedmetadata = () => {               resolve()             }           })         }).catch(err =>           console.log(err.message)         )

Hi Saadpasta, thanks for the reply. But could you elaborate a little more on this?
could provide more details on how do i run this script?
Thanks
-CE

There are couple of reasons
1 -> You are downloading model from internet which may take some time to load it in the browser first time. (Once downloaded TensorFlow catch it on disk)
2 ->I think you must be trying to access url from another system or mobile in intranet.
Camera won't be visible without https.
Try following option to run the code.
in package.json (make changes in start command.)
"scripts": {
    "start": "set HTTPS=true&&react-scripts start", // do not use **set** for mac or linux
}

more details on  & ReactJS https development
run the program using
HTTPS=true npm start
I tried this when i was tried at first time. issue solved.


User Guide | cmd for policy-library to Forseti bucket

Regarding this command:
export POLICY_LIBRARY_PATH=path/to/local/policies/library
I think it would be clearer to finish the command as '/policy-library/'
And for the next command:
gsutil cp -r ${POLICY_LIBRARY_PATH} gs://${FORSETI_BUCKET}/policy-library/
do we actually mean this?
gsutil cp -r ${POLICY_LIBRARY_PATH} gs://${FORSETI_BUCKET}/
I accidentally created nested /policy-library/policy-library/

Done.


Updating src_translated_ip and dest_translated_ip within Network Traffic DM

Updated the fields src_translated_ip and dest_translated_ip in network traffic to meet the new requirements.

:tada: This PR is included in version 1.3.13-develop.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.6.0-develop.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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


Support template package type

Is your feature request related to a problem? Please describe.
See https://github.com/NuGet/NuGetGallery/pull/7548
Package has a content/.template.config/template.json file entry. Package has a Template entry in the packageTypes nuspec property.
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.

Addressed in 81a43f9ae3962fc4290a40436237358a40f669c2


unable to pick up color when trying to set it from a dataframe

Hi there,
I was trying to make cufflinks pick up color from a column and assign it to entry point when plotting
i have been unsuccessful so far and have different behaviour like xaxis change its unit or color not properly changing the example below give more detail. Can any one look into it?
Thanks
exple:
data = {
    'time': ['10:00:00.001001','10:00:00.020020','10:00:00.003030','10:00:00.040015','10:00:00.050025','10:00:00.060036'],
    'color': ['red','orange', 'blue', 'orange', 'orange', 'red'],
    'duration': ['0.01020642', '0.01332378', '0.01018166', '0.01882410', '0.01899997', '0.01169282']}

df = pd.DataFrame.from_dict(data)

df.time= pd.to_datetime(df.time,format='%H:%M:%S.%f')
df.duration = df.duration.astype(float)

df.iplot(kind='scatter', mode='markers', x='time', y='duration')

if i try to use categories to differentiate different color, it will messed up the xaxis
df.iplot(kind='scatter', mode='markers', x='time', y='duration', categories='color')
(i saw that there is a ticket fr that already
now if i use the following:

df.iplot(kind='scatter', mode='markers', x='time', y='duration',colors='color')
I will get

CufflinksError: Not a valid color: color

now if use the column color to get the color i want using
df.iplot(kind='scatter', mode='markers', x='time', y='duration',colors=df.color)
i get

ValueError: The truth value of a Series is ambiguous. Use a.empty, a.bool(), a.item(), a.any() or a.all().

since color can take a a value or list I then use this
df.iplot(kind='scatter', mode='markers', x='time', y='duration',colors=df.color.tolist())
i will get the image below

the color will changed to red (first dataframe entry) but will not iterate to the list and change other entry points to their respective color
Am i doing something wrong or there is a way to achieve this (plot every single point in the dataframe with specific color)?

Hey there again,
tried now using command:
df.iplot(kind='scatter',mode='markers',symbol='circle-dot',colors=df.color.tolist(),size=10)
got this

and that is not looking to what i am trying to achieve as it is not picking up the right x and y axis and the color is still not link to each entry row

This works fine to me:
df.iplot(kind='scatter',x='time',y='duration',mode='markers',colors=df['color'].to_list(),categories='color')

Hello, Thanks for looking into this but if you check the x-axis it has changed from datetime to a scientific format.
My question now is how to do it but conserve the datetime x-axis format?
regards

Ok the last commit should fix this.

same version on Windows as well (Plotly '3.7.0' and cufflinks '0.14.6')

@santosjorge

Working as expected after downloading the version from Github neither pip nor pipy  have the latest.
Issue solved
Thanks


INF-1816: refactor, cleanup, fmt, lint

udnssdk.go: still fails golint due to necessary underscores in variable names

I like this.


Support for angular 7

This issue will capture things in progress for getting angular 7 working. Here's the POC / early work on things: https://github.com/joeldenning/angular7-experimentation

[x] Support ng build
[x] Support ng serve (#32)
[x] Make ng serve more useful by opening up the single-spa root config html file (#37)
[x] Auto generate a root config for people (#29)
[x] Allow for configuration of webpack externals (#21)
[x] Have ng build call enableProdMode(). (See #30)
[x] Get bundle size down for ng build (See #31 and #12)
[ ] Support for multiple angular applications (#41)
[ ] SystemJS interop (#44)
[x] Properly load global styles (#46)
[ ] Navigating between two single-spa applications (#47)

I created this issue to attempt to contact the angular team in an effort to collaborate with them: https://github.com/angular/angular/issues/28809

<EMAIL_ADDRESS>just released that begins support of ng build and schematics

<EMAIL_ADDRESS>now supports ng serve, but in a limited way. See #37 for next step with ng-serve.

<EMAIL_ADDRESS>resolves #37 and now ng serve is working well

<EMAIL_ADDRESS>fixes some issues with ng serve on Windows.

<EMAIL_ADDRESS>reduces bundle size, but had to introduce a breaking change in the process.

<EMAIL_ADDRESS>fixes global styles loading and allows for the user to override webpack configs.

beta.12 allows for multiple angular applications to be active at once.

beta.13 has fixes for angular 6 and also removes need for passing in ApplicationRef to single-spa-angular.

beta.19 makes it easy to use SystemJS and multiple Angular applications.
I am going to close this issue now in favor of just using the github issue queue for any remaining things before single-spa-angular@3 is released as "latest" instead of "beta"


EN_increasing_depth shortcut

@s-good, would it make sense to immediately flag all levels in EN_increasing_depth for profiles where every level has depth = 0? This is a practical performance concern, since there are profiles with thousands of levels in them and no depth info; looking at the test as written, I believe that in this case it will construct an NxN matrix, reject the last level, construct an (N-1)x(N-1) matrix.... which gets pretty painful for N O(1000).

That would be fine to do as all levels should end up flagged if they are all 0, so no point in running through the full algorithm in this case.


when Feign Service is closing,throwing a exception:BeanCreationNotAllowedException

when I use STS develop a Feign-Service,if I do what it is a run-as Spring-boot-App。the console will show a error this it is a exception:
org.springframework.beans.factory.BeanCreationNotAllowedException: Error creating bean with name 'eurekaAutoServiceRegistration': Singleton bean creation not allowed while singletons of this factory are in destruction (Do not request a bean from a BeanFactory in a destroy method implementation!)
but I do  what it is a run-as Java Application，the error is nothing.
I don't understand what the reason is.,please help me

Please learn how to format code on GitHub.
Not sure either, sounds like it might be something corrupt in your STS environment, especially is the same app runs fine elsewhere.  If you can provide a project to reproduce the problem I would be happy to try it on my machine as well.


Update Post “README”

Automatically generated by Netlify CMS

👷 Deploy Preview for hotplate-docs processing.

Name
Link

🔨 Latest commit
57b7a4ad4011af7ca35fb609c6597b1a177e6fb8

🔍 Latest deploy log
https://app.netlify.com/sites/hotplate-docs/deploys/62bcf551c45a3700083b447d


ATX: FHIR Feed Option should be explained/linked or referenced

Section: 1:52.1.1 Actor Descriptions and Actor Profile Requirements refers to the ATX: FHIR Feed Option without reference to what that is or where to find it.  That should be added as a link to the PDF.

I don't disagree, but by linking to the PDF this puts a need to update this when the PDF is either converted or integrated. I was trying to avoid a link in this case.

committee review: point them at the supplement PDF - https://www.ihe.net/uploadedFiles/Documents/ITI/IHE_ITI_Suppl_RESTful-ATNA.pdf

create an open issue to track this future fix needed


Loop invariant code motion initial implementation

This patch provides an initial implementation of a loop invariant code motion pass.
The pass expects eliminate-local-multi-store to have been run on the module first.
The pass works by performing the following algorithm on each loop in the module.

Add each invariant instruction in the loop to a queue of invariants
For each invariant instruction in the queue
2.1. Hoist it to the loops preheader
2.2. For each of the instructions users, check if they are now invariant, and if so add them to the invariants queue

And instruction is invariant iff

It is an opcode with no side effects
All instructions contained in its operands are outside the loop

As for scheduling, we may need to do more testing, but generally if you fully unroll a loop you can expose many more opportunities for SROA so we may want to re-run the mem2reg set of passes (SROA and 3 mem2regs) afterwards. I agree this needs to come after the initial round mem2reg so the induction variables are promoted.
Probably we should file another issue or two. Determine an ordering and (separately, determine whether all three mem2regs are really necessary).

On Wed, Feb 7, 2018, 13:30 Steven Perron<EMAIL_ADDRESS>wrote:

@s-perron commented on this pull request.
In source/opt/cfg.cpp
https://github.com/KhronosGroup/SPIRV-Tools/pull/1242#discussion_r166711344
:

@@ -58,7 +58,6 @@ void CFG::RemoveNonExistingEdges(uint32_t blk_id) {
pred_blk->ForEachSuccessorLabel([&has_branch, blk_id](uint32_t succ) {
if (succ == blk_id) {
has_branch = true;

   return;

If we had a "WhileEach*" version, we could return false to indicate that
we should stop.

D'oh, of course. Thanks. I kept forgetting that the iteration is not
affected by the lambda. Apologies for the noise.
—

You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/KhronosGroup/SPIRV-Tools/pull/1242#discussion_r166711344,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG9RZwD3yc4UFUhlEYospmVBkesa4A5Fks5tSevAgaJpZM4Rwhld
.

I think this is good to merge once all of the bots have passed.  Please rebase, and then I can merge it.

LGTM

Please fix the test failures.

I'll try to merge these right now.

Could you do a rebase and squash?  Not a merge.  We like keeping a linear history to make problem identification easy.  I tried rebasing, but it is not obvious how to do it.

Rebased and merged as 84ccd0b9.


nixos/restic: add wrapper scripts that set parameters for backup

Description of changes
For added convenience.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Initially i was confused about the usecase of the new package, until I saw the changes in the test file.
I think that addition deserves proper Documentation, once as a comment where it is added, and another inside an Option Description (probably the top-level services.restic.backups)

Great idea!

This seems to be missing /run/wrappers/bin from $PATH which allows restic-<host> mount to work. Otherwise I'm getting the error:
$ sudo restic-laptop mount --allow-other .local/tmp/mnt/
repository 5451d26a opened (version 2, compression level auto)
[0:04] 100.00%  11 / 11 index files loaded
fusermount: exec: "fusermount": executable file not found in $PATH
unable to umount (maybe already umounted or still in use?): exec: "fusermount": executable file not found in $PATH

Should this be a separate bug report? Thanks! Otherwise it's a great PR!

Experiences the same but in my case because the wrapper sets PATH and does not append it.

Hm, why does this work in the test then?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/restic-backups-on-b2-with-nixos-agenix/36196/3

The test is not mounting a backup with restic mount which only requires that executable.


fix(calendar): fix Issue with events now showing in calendar for negative time zone offsets

About the changes

Fix Issue with events not showing in calendar for negative time zone offsets.
The date sent to graphql was getting constructed as '<date>+-7:00' instead of '<date>-07:00' when the time zone offset is negative.

Fixes #186


Fixed link to hibernate client installation instructions

Fixes https://github.com/cockroachdb/docs/issues/8006

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/d00f85b6fca669f583da60a1815646ea3cdb6862/
Edited pages:

v19.2/build-a-java-app-with-cockroachdb-hibernate.md
v20.1/build-a-java-app-with-cockroachdb-hibernate.md
v20.2/build-a-java-app-with-cockroachdb-hibernate.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/250b342c16a88af8a27b549c8c315d74f467f705/
Edited pages:

v19.2/build-a-java-app-with-cockroachdb-hibernate.md
v20.1/build-a-java-app-with-cockroachdb-hibernate.md
v20.2/build-a-java-app-with-cockroachdb-hibernate.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/fc6eefe02bf89b4a2ff696e9242f20d8444157e1/
Edited pages:

v19.2/build-a-java-app-with-cockroachdb-hibernate.md
v20.1/build-a-java-app-with-cockroachdb-hibernate.md
v20.2/build-a-java-app-with-cockroachdb-hibernate.md
v21.1/build-a-java-app-with-cockroachdb-hibernate.md


Python 3.8 doesn't work with Werkzeug 0.15.3 in current requirements.txt

Run this tool with Python 3.8.6 from Debian testing this error shows:
TypeError: required field "type_ignores" missing from Module

This link seems to be the issue report in werkzeug repo. Updating to 0.15.5 it works fine. It seems the dependency has always updated by bot, please maintainers take a look and update werkzeug? Thanks.

I can confirm that the issue still exists on latest master on Ubuntu Focal, and the same fix also worked for me.

I can confirm that the issue still exists on latest master on Ubuntu Focal, and the same fix also worked for me.


Handler Vars Should Not Include $_GET and $_POST

Initially reported by: @chrismeller as #TRAC1366
Merging $_GET and $_POST with values obtained from the URL into handler_vars (which may then be fetched with Controller::get_var()) essentially emulates $_REQUEST (without including $_COOKIE).
This is bad practice and we should stop relying on handler_vars to include everything we need. There should be better separation between sources of information and we should always know where our data is coming from.
In addition to being bad practice, it's significantly confusing. If you saw $_POST['foo'] in code you would know where it came from. $this->handler_vars['foo'] on the other hand tells you nothing of how the value was obtained. This may not be important from a strictly code perspective, but it can be very valuable information when you're trying to backtrack the source of a value.

I agree with Chris, you should always know where the data is coming from.


Unable to see Code Coverage Results from VS 2017

Copied from https://github.com/Microsoft/vstest-docs/issues/56 on behalf of @SrivatsaMarichi
Hi Team,
I have added the package for Code Coverage from the Package Manager Console by passing Install-Package Microsoft.CodeCoverage -Version 1.0.3.
I then ran Analyze Code Coverage but I don't see any results in Code Coverage Results window.
I am trying this for .NetCoreApp 1.0, Can anyone help me out here?
Visual Studio Enterprise 2017 Version- 15.2(26430.6)
Also I want to know is there a support for the same on VSTS CI builds? If so, how?
Thanks,
Srivatsa

Any help here?

@mayankbansal018 Can you please take a look.

@SrivatsaMarichi can you please share csproj of test project?

Steps to enable code coverage is detailed in https://github.com/Microsoft/vstest-docs/blob/master/docs/analyze.md#setup-a-project.
In CI, it is possible to use this command line: https://github.com/Microsoft/vstest-docs/blob/master/docs/analyze.md#collect-coverage-with-command-line-runner

Connected with @SrivatsaMarichi offline. This issue is resolved now.

I still can't see results.  The menu item is there but disabled (vs2017 build 26430.15).  I have included the all the things in the "Steps to enable code coverage" link mentioned above.  I have even restarted vs2017, dotnet restore, etc with no luck.
What was the fix?

@Allann 26430.15 build is VS 2017 15.2. The coverage feature requires VS 2017 15.3+ build: https://www.visualstudio.com/vs/preview/.


Make secure crate weldable

About the PR
The secure crate is now weldabale.
Why / Balance
Steel crates should be all weldable.
Media

https://github.com/space-wizards/space-station-14/assets/74560659/8df4f013-e7f9-48d5-9111-9b414817d9f1

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
this is technically a breaking change
Changelog

fix: Secure crates are now weldable!

add -  before fix

added -  before fix

and :cl: in the line above it
:trollface:

what does :cl: do?

lets the changelog bot parse a changelog to put ingame


Evolution X (A13) 10/4 22 - OP8T - Problems with Assistant

The following bugs/issues occur on the newest release of Pixelfy (2.1) with the actual Evolution X ROM:

Module leads to not accepting "Hey, Google", I just can't activate assistant by voice anymore, no matter what I try.

When opening with buttons, the "old assistant" is opening instead of new one (German language), even if - when installing Pixelfy - NGA is not set to be installed.

When uninstalling the module and resetting Google App (delete cache and storage) NGA works again and options for NGA voice type remain (however, greyed out, although German language should be supported - at least on Pixel 6 on A13 it works)

The interesting fact is that the features (e.g. extended smart space in Pixel launcher) as well as the (greyed out) option for NGA voice type remain active after uninstalling the module.

I can provide logs if helpful. Thanks very much!

Your Google app version?

Package version is <IP_ADDRESS>.arm64
The Problem with the old assistant got resolved, it was the Bitdefender accessibility addon.
However, the issue with "Hey Google" persists if the module is installed on EvoX. And NGA voice typing can't be enabled for German (language not yet supported message and greyed out switch).


speed up list of slow models to try to add back to test set

These models are not run currently in the est job, because they take too long to run:
https://github.com/nmfs-stock-synthesis/workflows/blob/4eb01e69d71dac765310d90b547666830b2e5d35/.github/workflows/run-ss3-with-est.yml#L52-L57
rm_mods <- c("BlueDeaconRF_2017", "BlackRf_2015_WA", "China_2015_Central",
                     "Spinydogfish_2011", "YellowtailRF_2017", "Bocaccio_2015",
                     "Darkblotched_2015", "POP_2017", "seasonal_with_size_comp",
                     "Widow_2015", "YelloweyeRF_2017", "CanaryRf_2015",
                     "Petrale2015", "three_area_double_normal_selex",
                     "Sablefish2015")

It would be great to try to speed these up if possible so that they are run with estimation.
List of models:

~BlueDeaconRF_2017~ (removed b/c no unique features)

~BlackRf_2015_WA~ (removed b/c no unique features)

~China_2015_Central~ (removed b/c no unique features)

Spinydogfish_2011 - sped up to under 5 min by changing start year from 1916 to 1980 and some simple modifications.

~YellowtailRF_2017~ (removed b/c no unique features)

~Bocaccio_2015~ (removed b/c no unique features)

~Darkblotched_2015~ (removed b/c no unique features)

~POP_2017~ (removed b/c no unique features)

~seasonal_with_size_comp~ (removed due to lack of unique features)

~Widow_2015~ (removed because added maturity option 2 to simple lorenzen instead.)

~YelloweyeRF_2017~ (removed b/c no unique features)

~CanaryRf_2015~ (removed b/c annual deviations are already used in other models for other parameters)

~Petrale2015~ (removed b/c no unique features)

~three_area_double_normal_selex~ (don't see this model in test set, renamed?)

Sablefish2015 (has special survey and unique sel. patterns; could maybe remove some or all CAALs to speed up?) - was able to speed up by reducing length of ts; still runs for 15 min, so may need more work.

deleted three_area model due to lack of unique features

TODO

Make sure selectivity time block captured in test set? (already have q time blocks tho)
Make sure -year and -fleet values are included in datasets.
make sure lambdas used.
Add catch multiplier to one of the test set.
All prior types used.

more wc gfish files are less likely to have unique features.  Better to look for a tuna setup (areas, generalized size comp), or a Max setup (devs, env, predation)

Most of these slow models were removed b/c they don't have any different features compared to faster models already in the test suite. There were a few that did have unique features, so I will focus next on either speeding up the models or adding those unique features into a faster model within the test set.

the est job passes and now runs all models in the test set. Run time is 1 hour, while before it was 35 min. I think we are actually running more models with est than we were previously, though. Continuing to work on reducing run times would be helpful in the future.

If run with -noest fails, then it is a waste of time to proceed to run with est.  So, can the gha abort when it gets a fail?

I thought about linking them like that, but I think decided not to because then the est run won't start until the noest one finishes. I think I also struggled with how to implement it technically at the time, but might be able to now.
I will open an issue on this in the workflows repo.

Ahh.  I forgot about aspect of them running in parallel


I want to get the appendix of this article, where can I find it

I want to get the appendix of this article, where can I find it.

You could find it in this CVPR html page: https://openaccess.thecvf.com/content/CVPR2023/html/Hu_Efficient_Semantic_Segmentation_by_Altering_Resolutions_for_Compressed_Videos_CVPR_2023_paper.html


QSortFilterProxyModel can't be imported

Hi Marcus, trying now to port some complex stuff on Qt.py,  but started hitting some issues:
from Qt.QtGui import QSortFilterProxyModel
ImportError: cannot import name QSortFilterProxyModel

any idea ?
Cheers.
p.s
from PySide.QtGui import QSortFilterProxyModel
print QSortFilterProxyModel
>> <type 'PySide.QtGui.QSortFilterProxyModel'>

print Qt
<module 'PyQt5' from '/usr/lib64/python2.7/site-packages/PyQt5/__init__.pyc'>

I'll try forcing on PySide

Hey @hdd, thanks for this. Looks like a bug. I'll have a PR ready in the new few mins.

Note that this class should be accessed via QtCore.

https://doc.qt.io/qt-5/qsortfilterproxymodel.html

Arriving on PyPI in a few minutes.

https://travis-ci.org/mottosso/Qt.py/jobs/149454277

Thanks man, really is appreciated!
I'll give it a whirl as soon as it land.


Important Fossil Area Resource Model edits

[x] Change Name to “Area Name and Description”
[x] Keep “Area Name” - mandatory
[x] Delete Name Type
[x] Add Header “Description” - mandatory
[x] Delete Description Type
[x] Add Header “Notes” with instruction “Add rationale for creating the area”
[x] #911
[x] Under Potential Ranking, create LOV with 6 choices: - mandatory

High
Medium-High
Medium
Low-Medium
Low
Low-High

@LeannePyle - The ranking code is common to all locations. Do do you want those 6 values everywhere or do we need to create a new LOV?

this is a new LOV just for this model where a range is possible

@LeannePyle - These values don't match the data that is being imported (eg Medium to High). Do you want to use the existing format or convert the existing data to the format listed above?

Sorry, yes, please match existing format "Medium to High", "Low to Medium" and "Low to High"

Complete however LOV out of order due to Arches Core bug


Prompt error message if too many FCP paths

If there are too many FCP devices, return the error message.


Compatibility with<EMAIL_ADDRESS>

Hi, is this plugin compatible with<EMAIL_ADDRESS>?

When I try to compile, I get the error:
:javaPreCompileDebug
/Users/..../platforms/android/src/com/customtabplugin/ChromeCustomTabPlugin.java:46: error: cannot find symbol
        final Context context = this.cordova.getContext();
                                            ^
  symbol:   method getContext()
  location: variable cordova of type CordovaInterface
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.

CordovaInterface.getContext() was only added in Cordova Android 7.0.0, so you will need to upgrade to at least 7.0.0.

Thanks @AndrWeisR


Warning on Azure Functions Package

Plugin name and version
azure-functions-maven-plugin:1.6.0
Plugin configuration in your pom.xml
 <plugin>
                <groupId>com.microsoft.azure</groupId>
                <artifactId>azure-functions-maven-plugin</artifactId>
                <configuration>
                    <resourceGroup>${functionResourceGroup}</resourceGroup>
                    <appName>${functionAppName}</appName>
                    <region>${functionAppRegion}</region>
                    <appSettings>
                        <property>
                            <name>WEBSITE_RUN_FROM_PACKAGE</name>
                            <value>1</value>
                        </property>
                        <property>
                            <name>FUNCTIONS_EXTENSION_VERSION</name>
                            <value>~3</value>
                        </property>
                    </appSettings>
                </configuration>
                <executions>
                    <execution>
                        <id>package-functions</id>
                        <goals>
                            <goal>package</goal>
                        </goals>
                    </execution>
                </executions>
            </plugin>

Expected behavior
No warnings on mvn package
Actual behavior
[WARNING] The POM for com.microsoft.azure.applicationinsights.v2015_05_01:azure-mgmt-insights:jar:1.0.0-beta is invalid, transitive dependencies (if any) will not be available, enable debug logging for more details appears in the logs
Steps to reproduce the problem
Run mvn package with version 1.6.0 on azure-functions-maven-plugin
Seems to be caused by this: https://github.com/microsoft/azure-maven-plugins/blob/a6160f31b5f80b2398b8cd0ce6bbefb43bb5cdd7/azure-tools-common/pom.xml#L85-L89
Perhaps it can be updated or removed?

@EwanNoble Thanks for your report,we are using application insights mgmt sdk for resource creation in function deployment, so the dependencies can not be removed, we are waiting for the sdk updates and i'll update here once there are updates.


View / Export chats from Application Insights

[ ] bug report -> please search issues before submitting
[ ] feature request
[x] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

How can I view past chats using Application Insight ? I have it enabled and can access it on Azure > Application Insights > Investigate > Transaction search but can't seem to find the actual messages anywhere. Also it would be amazing if there was an easy way to export all past chat histories with time stamp and which user called as a csv, json, or similar. Thanks for any help!

The Application Insights configuration in the demo app is for tracing errors and performance of the backend service and dependant APIs. It doesn't capture the chat request messages mainly because that would include things like API keys and other secure data. Have a look in the Performance tab and find a POST to /chat then use the Drill into samples button to see all the related data.

I think you could achieve this result by setting a Diagnostic Log using this guide https://learn.microsoft.com/en-us/azure/ai-services/diagnostic-logging


Add Rudimentary Custom Colour Support and add Extra Regions

Hi,
Hopefully if I have some time I'll be able to push some more commits to this, in the meantime this should allow arbitrary RGB colours to be used instead of being limited to the predefined ones, for blink and color, will investigate modes tomorrow. Based on the C++ in the ticket from earlier.
Thanks

Slick - thanks, merged!


Training LibreASR LM, Cudnn Error happens

hi, When I was training LibriSpeech ASR Language model, this error is happening, What might be the issue, Thanks
python train.py RNNLM.yaml
speechbrain.core - Beginning experiment!
speechbrain.core - Experiment folder: /asrltrc/ganesh/spb_exp/Libre/LM/results/RNNLM/
root - generating datasets...
datasets.builder - Using custom data configuration default-410ea46aba2d7db9
datasets.builder - Reusing dataset text (/home/asr/.cache/huggingface/datasets/text/default-410ea46aba2d7db9/0.0.0/e16f44aa1b321ece1f87b07977cc5d70be93d69b20486d6dacd62e12cf25c9a5)
speechbrain.core - Info: ckpt_interval_minutes arg from hparam file is used
speechbrain.core - Exception:
Traceback (most recent call last):
File "train.py", line 255, in
lm_brain = LM(
File "/home/asr/spb/speechbrain/core.py", line 476, in init
self.modules = torch.nn.ModuleDict(modules).to(self.device)
File "/home/asr/.local/lib/python3.8/site-packages/torch/nn/modules/module.py", line 673, in to
return self._apply(convert)
File "/home/asr/.local/lib/python3.8/site-packages/torch/nn/modules/module.py", line 387, in _apply
module._apply(fn)
File "/home/asr/.local/lib/python3.8/site-packages/torch/nn/modules/module.py", line 387, in _apply
module._apply(fn)
File "/home/asr/.local/lib/python3.8/site-packages/torch/nn/modules/rnn.py", line 186, in _apply
self.flatten_parameters()
File "/home/asr/.local/lib/python3.8/site-packages/torch/nn/modules/rnn.py", line 172, in flatten_parameters
torch._cudnn_rnn_flatten_weight(
RuntimeError: cuDNN error: CUDNN_STATUS_NOT_INITIALIZED

Hi, this looks like a problem with your GPU. Are you able to simply run the following code without errors?

import torch
torch.rand(10, device='cuda')

Hello,
I ran those commands and they worked fine. I have given the output below.
(spb) asr@ltrcgpu:~$ python
Python 3.8.10 (default, May 19 2021, 18:05:58)
[GCC 7.3.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import torch
>>> torch.rand(10, device='cuda')
tensor([0.2570, 0.3983, 0.1486, 0.8994, 0.7859, 0.0431, 0.0474, 0.6916, 0.1371,
        0.1126], device='cuda:0')

Did you fix that? This looks a GPU problem more than a speechbrain one


[SPARK-29518][SQL][TEST] Benchmark date_part for INTERVAL

What changes were proposed in this pull request?
I extended ExtractBenchmark to support the INTERVAL type of the source parameter of the date_part function.
Why are the changes needed?

To detect performance issues while changing implementation of the date_part function in the future.
To find out current performance bottlenecks in date_part for the INTERVAL type

Does this PR introduce any user-facing change?
No
How was this patch tested?
By running the benchmark and print out produced values per each field value.

Test build #112316 has started for PR 26175 at commit b361b51.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/17296/
Test PASSed.

Test build #112316 has finished for PR 26175 at commit b361b51.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/112316/
Test PASSed.

Merged to master.


add core::stream

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 92.85714% with 2 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

src/core/stream.rs
92.85%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ

src/core/stream.rs
92.85% <92.85%> (ø)


an error: Shapes are not in standard layout

I use a model to do inference,but an error occurs. the output:
amdmigraphx/src/include/migraphx/check_shapes.hpp:136: standard: softmax: Shapes are not in standard layout
the model link: https://pan.baidu.com/s/1BZ32JKuSt3r0VUZfTgM5Dw
password：lz2l

I tried the latest develop branch, it works well. Could you please try again?

@scxiao ，what is the reason ?

@qianqing13579 I cannot reproduce the problem, so no idea what is the reason..
Can you try the latest develop branch to see whether it works for you now?

@scxiao ，I tried the latest develop branch, it works well. Thanks

issue is not reproducible, so close this issue.


Do not automatically resolve response Location header values

node-fetch appears to be transforming response headers and resolving relative URLs against the request URL which means in order to assert that a correct relative URL is returned by the server one cannot use fetch. 😞
Consider the following HTTP messages:

Request:GET / HTTP/1.1
Host: localhost:3000

Response:HTTP/1.1 200 OK
X-Powered-By: Express
Location: /
Content-Type: text/html; charset=utf-8
Content-Length: 170
ETag: W/"aa-z+ebXSEdArbZ+EXlN/WQjf6HV8c"
Date: Thu, 08 Mar 2018 18:18:38 GMT
Connection: keep-alive

<!DOCTYPE html><title>Express</title><link rel="stylesheet" href="/stylesheets/style.css">
Express

Welcome to Express

Using node-fetch to request the resource and extract the Location response header might look like the following:
const response = await fetch("http://localhost:3000/")
const location = response.headers.get("Location");
assert.equal(location, "/");

The assertion, however, will fail because the Location response header value is in fact "http://localhost:3000/" instead of simply "/" even though the server is in fact simply returning "/".
A Location response header value of "/" is accessible through fetch in Google Chrome (I tested it via the DevTools console); it is also available when using Node.js's http module directly. From what I can tell node-fetch is transforming these response headers for some reason but now the raw values are inaccessible.
It would be very helpful to either not transform these values and return them in Response objects as the server sent them.
Thank you.

What version of node-fetch are you using? I suspect this might have been fixed in v2.1.1.

Oh. "node-fetch": "^1.7.3". I'll try 2.1.1+.

I just checked v2.1.1 and I'm seeing the same issue. I also found some what I believe is the relevant node-fetch code for this: https://github.com/bitinn/node-fetch/blob/780598ad72d7205aa5616be2215692785270eda3/src/index.js#L79-L80
It references HTTP fetch step 5.3 which says "If location is a value, then set location to the result of parsing location with actualResponse’s url." I'm not sure what that means but it almost makes it sound like Google Chrome should be behaving more like node-fetch but isn't? Not sure.

I suspect we can't fix this issue for a few reasons:
https://github.com/bitinn/node-fetch/blob/780598ad72d7205aa5616be2215692785270eda3/src/index.js#L87-L92

Too many people rely on the behavior where manual flag giving the full url in location header.
node-fetch itself isn't origin aware so relative path means nothing to it (it will deny a relative url unlike browsers)

So I think we could either expose an additional header like X-LocationRaw as a workaround, or you might just take the base url and parse the full url back to relative path as you wish.

Ah yes, those seem like very legitimate reasons not to "fix" this issue directly.
I wonder if instead of injecting a new header which the server didn't actually send if it would make more sense to have a node-fetch extension for this as this seems very specific to the Location header. Here are some ideas:

Make an extension property on Headers for the raw/original/unmodified Location header value. e.g.:

res.headers.location
res.headers.rawLocation
res.headers.originalLocation
res.headers.unmodifiedLocation

Make res.headers.raw() return a map containing the "raw" Location header value. This would probably require some work to pipe the resolved locationURL around too. Not sure exactly what that looks like but that way res.headers.raw() would give back only whatever was received from the server. I haven't found yet where raw() is documented yet (if it is). When I originally found it I was hoping it would give me what I wanted.

Speaking of documentation, regardless of the how this issue is resolved I wonder if the way Location is handled should be added to the Known Differences.
Thank you.

@mfulton26 I don’t see a redirect: "manual" in your fetch() call. Did you miss adding it? If the response status code is 200, the Location header should not be modified in any way.

@TimothyGu check the code I linked, I think it shows Location header is being modified, and it was a specific node-fetch behaviour.

And yeah, I was hoping @mfulton26 is aware of manual flag.

@bitinn I meant in the OP, the response code is shown to be 200. I suspect before v2.1.0 the header was changed for the 200 case as well.

@TimothyGu got that, though OP still won't have the raw location header, but maybe OP doesn't really need raw location anyway :)

Thank you. Yes, I am aware of the redirect manual flag. The example I gave isn't real. My actual use case is that of a service level API test that a Location header response value is a relative URL in a 3xx status code response which I cannot currently validate using node-fetch.

(because the raw Location response header is no longer available so there's no way to determine if the server did indeed return a relative URL which is important in this use case because the service in question isn't called directly but via zuul)

I think we can possibly just do away with the URL resolution. The user should have access to the request URL in the first place, and they can always do the resolution themself. Targetting v3.x since this is a breaking change.

https://github.com/node-fetch/node-fetch/blob/8c197f8982a238b3c345c64b17bfa92e16b4f7c4/src/index.js#L126
As of today, if anyone's looking for an interim solution (before v3), you guys can pass whatever value to redirect so node-fetch doesn't automatically convert the response location to request host + response location. Like:
fetch(url, { redirect: 'manual-dont-change' });

This is not ideal, but this works for now until we wait for v3 ^


Upgrade to AFNetworking 4.0

I've updated Podspec and Cartfile to point to new AFNetworking 4.0 release. Also updated tests and upgraded to latest Xcode version.

Can this PR be merged? Thanks!

@kylef could you please have a look if this meets the criteria to be merged? Thanks a lot!

@cnoon @SlaunchaMan @kylef is there anything blocking this? Thanks.

AFNetworking 4 has been out for months now, there's a PR to support it, so what's the holdup? Is there anybody actually maintaining this project anymore?


Update Discord classes

Discord updated quite a few classes, breaking some parts of this theme. This PR switches the old classes for the new classes in place using ClassUpdate.

Classes changed again


GPSPRINGSECURITYCORE-300: @Secured not working for namespaced controllers with same name

Original Reporter: flextron
Environment: grails-2.4.2, spring-security-core:2.0-RC4
Version: Not Specified
Migrated From: http://jira.grails.org/browse/GPSPRINGSECURITYCORE-300
I have 2 DashboardController. However both take annotation of the controller without the namespace.
The first can't be redirected to login. I tried using the annotation permitAll unsuccessfully.
DashboardController.groovy(with namespace):
class DashboardController {

    static namespace = 'XXX'

}

DashboardController.groovy (without namespace):
@Secured('IS_AUTHENTICATED_FULLY')
class DashboardController {
}

This already works. See https://github.com/grails-plugins/grails-spring-security-core/blob/2.x/functional-test-app/test/functional/NamespaceSecuritySpec.groovy#L116:L130


Uses yield from in _handle_generator instead of a lookalike that doesn't work anymore

Dear @kevin1024,
PEP479 is active in python3.7 now, and this breaks _handle_generator. I've filed an issue to report this, and worked on finding a fix.
Essentially, this fix consists in using yield from coroutine in _handle_generator, but as this pattern is not python2.7 compliant, I designed a full patch.
This PR would close issue #396

it looks like the linter is failing

@P-EB few change requests here

@graingert I changed my approach. Essentially I just added a catch of StopIteration in the handle_generator method. Do you wish me to post it?


Add support for the graphics accelerator

The latest Linux SDK from TI supports the 1.17.* series kernel module and user mode driver to run the Power VR graphics accelerator on the Beaglebone am335x systems. This PR builds in support for these driver / modules and also enables Weston and libglfw 3.

Just annotated some stuff I didn't recognize, LGTM. Great work Justin.


[SMALLFIX] Fix a memory leak

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13031/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13035/
Test PASSed.


Declarative

@pixelswapnil13 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


OCPBUGS-18179: UPSTREAM : update build images to rhel9

/cherry-pick openshift-4.13

/lgtm
/approve

/jira refresh

/jira refresh

/jira refresh

@geliu2016 can you CPA here too, please? that's somewhat urgent sadly as it breaks manual FIPS etcd backups.

/label backport-risk-assessed

/retest

/retest

/payload 4.14 nightly blocking


Less redirects

Adds a forward-slash to the end of urls in Service.svelte - some scratch urls redirect you if you look closely

LGTM!


PT Run, sort order for previously selected not always working.

Microsoft PowerToys version
0.59.1
Running as admin

[X] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
Some items in the list will appear before previously selected/clicked items.  Previously selected items should always be before never-selected items. We need to do this because the "Score" is not uniform (across plugins) and thus it might require a lot more clicks to put something at the top of the list.
We might want to also put a time box on the amount of time it can be considered "clicked". For instance, if it's last click was a long time ago, use the current "Score * 5" system, else use the new system.
E.g.:
Search results have this makeup.
Item 1: Score 900
Item 2: Score 200
Item 3: Score 100
Item 4: Score 3
Here, "Item 4" has to be clicked "a lot" and it's an unknown (to the user) why it's not at the top. Because we're using (Score + (ClickedCount * 5)). We need to click "Item 4" 182 times to get it to the top.
✔ Expected Behavior
Clicked items in the list always above never-clicked items.
❌ Actual Behavior
Clicked items can be shown below never-clicked items, because the score for them is higher.
Other Software
No response

There's likely something in the middle. If the query I've used to search for items ends up being more relevant for that search than previously clicked items, it might make sense to show the former. I imagine this having other side-effects like always getting usually clicked items much higher consistently than what we are actually looking for.

I'm guessing this would be something like https://github.com/microsoft/PowerToys/issues/12971 or https://github.com/microsoft/PowerToys/pull/17879
Perhaps a option to allow using different weight for the clicks might make sense, because I don't think there's a "one size fits all" here.

I'm guessing this would be something like #12971 or #17879
Perhaps a option to allow using different weight for the clicks might make sense, because I don't think there's a "one size fits all" here.

I like both of those really, but they 1) don't directly solve the issue with "previously selected" (unless there is also way to manage these as a virtual group) and 2) I don't really want to manage anything.
So, I also agree with you in that there's not a "one size fits all" solution. I think any "one size fits all" will involve updating the UI though, since all users might want something different, and that I don't know how to do or have time to learn.
So, using the 80/20 rule will have to do here. Also, I 100% agree with your "something in the middle". There will always be debate as to if something I clicked is more relevant that something I never clicked. The sad fact is that the scores are not uniform and clicked weight it arbitrary. We know that unless there is something like #12971 or #17879, we'll need something arbitrary, just the way it is.
And of course the risk going to be either A) clicked items never rise to top and that looks broke or B) clicked items seem to stick to the top.
I coded this update to have the clicked count and the last clicked time stored so that more elaborate but still arbitrary schemes can be created. What about this...
Current model:
`
public void Sort()
{
foreach (var result in Results)
{
result.Result.Score += result.Result.SelectedCount * 5;
}
        var sorted = Results.OrderByDescending(x => x.Result.Score).ToList();
        Clear();
        Results.AddRange(sorted);
    }

`
Suggested new model:
`
public void Sort()
{
// very recent, big bump
foreach (var result in Results.Where(x => x.Result.LastSelected > DateTime.UtcNow.AddMinutes(-5)))
{
result.Result.Score += result.Result.SelectedCount * 100;
}
        // somewhat recent, medium bump
        foreach (var result in Results.Where(x => x.Result.LastSelected > DateTime.UtcNow.AddMinutes(-30)))
        {
            result.Result.Score += result.Result.SelectedCount * 50;
        }

        // recent, small bump
        foreach (var result in Results.Where(x => x.Result.LastSelected > DateTime.UtcNow.AddHours(-12)))
        {
            result.Result.Score += result.Result.SelectedCount * 10;
        }

        // last week, old *5 bump, clicked more than a week ago, will not get a bump and are 'aged out'
        foreach (var result in Results.Where(x => x.Result.LastSelected > DateTime.UtcNow.AddDays(-7)))
        {
            result.Result.Score += result.Result.SelectedCount * 5;
        }

        var sorted = Results.OrderByDescending(x => x.Result.Score).ToList();
        Clear();
        Results.AddRange(sorted);
    }

`
My goal is something that is just a little tiny bit smarter than what's there.
The two issues with the current model are:
A) clicked items never age out
B) the bump of *5 is too small to overcome the Score disparities among plugins.

Something here is also thinking how to make this behavior configurable by the users.  For example, making the *5 value configurable would be something that could be added to Settings, but wouldn't be as smart as you suggest. I think this could be also a toggle for "Show recent results first".
WDYT?
@crutkas @htcfreek @davidegiacometti

This feature exists by default. But because of the different scoring behaviors it doesn't work .

Something here is also thinking how to make this behavior configurable by the users. For example, making the *5 value configurable would be something that could be added to Settings, but wouldn't be as smart as you suggest. I think this could be also a toggle for "Show recent results first".
WDYT? @crutkas @htcfreek @davidegiacometti

I have moved this over to #18986 and have a screen for this and I can add "Show recent results first" but we get almost the same affect by what I did, which is allow changing the weight of the clicked items to something higher (I am using 50).
However, I had orientally coded this as an option for "Show recent results first" and I do really like that also and will make that an option ALSO.
I am closing this and switching to issue #18986.


Update release cheat sheet

Changes
This commit updates the release cheat sheet with some things I ran into
when doing the most recent release.
/kind documentation
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
n/a Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
n/a Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
n/a Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/lgtm


Storybook: Career Quiz Result Page Component

User story
Who: As a user
What: I want to be open the Career Quiz Result Page
Why: so that I can see the results of the QUIZ.
Acceptance criteria
Create a storybook component for the Career Quiz Result Page where we aggregate all the components below:

Robot logo and a label below with the text BASED ON QUIZ RESULTS
HEADER - https://github.com/HackYourFuture-CPH/Guide-IT/issues/13
CARD for YOUR PERSONAL PROFILE - https://github.com/HackYourFuture-CPH/Guide-IT/issues/16
CARD for YOUR PROFESSIONAL PROFILE - https://github.com/HackYourFuture-CPH/Guide-IT/issues/16
TEASER - https://github.com/HackYourFuture-CPH/Guide-IT/issues/17
BUTTON - https://github.com/HackYourFuture-CPH/Guide-IT/issues/18

These are the 3 different options for the career quiz result page can look like

Check the wireframes here: https://www.figma.com/file/9SJRaxPSuPI3lSl8ULNyJ8/GuideIT-For-Developers?node-id=0%3A1
Implementation details
Use the image Vector.svg located in this folder /src/client/assets/images/ as a background image and find a way for half of the it to have a layer of opacity like we can see in the screenshot above.

@dpfernandes which dummy page in containers should we use as our starting point? We want to avoid duplication where possible.

Hi Aijaz i have run your storybook in that i saw like 'your professional profile is going out of the page and also that card ,could you please check.


Release 0.3.0

Changes -

Specific Commit interval per time window, fix #70
Initial move to confluent 4.1.0 and kafka 1.1.0, fix #71
UUID in source statistic monitor client id, fix #59
Attempts to fix #60

I agree about the configuration. We should investigate and make each stream individually configurable as much as possible.


Feature request: Backlisting email adresses

Since I would like to make my dispose me instance public, I would like to see a kind of blacklist for emails.
At best it would be a list with wildcards org regex configurable.

@shokinndo That would be doable, though I'm not sure what would be the benefit of having it in place? There are two things to note:

Even if the API and web are key protected, anyone can send you an email as this part doesn't require any authentication.
Regardless if such filtering would be on or off it would still trigger Lambda processing the email payload to determine if it should be processed further.

What's the end goal you're trying to achieve?

Hey,
the benefit would be to blacklist addresses like hostmaster@, postmaster@, webmaster@, admin@, administrator@, ...
These addresses can be quiet destructive since maybe someone can try to steal the domain.
This would particular threatening, when you make your dispose me instance public (#9).

@shokinn Thanks! That's something I didn't took into consideration since now. I'm wondering if it would make more sense redirect such emails to real email of the service owner rather than just dropping 🤔

Ideal would be to be able to configure this 🙂.
Even better to be able to to this for each email.
Maybe two arrays out of regex patterns (one for dropping, one for forwarding)?

@shokinn sorry it take soo long, would you like to take a look at #13 and share some fedback?


UpdateEmitter drops MultipleInstancesResponseType updates due to type checking.

Basic information

Axon Framework version: 4.6.0-SNAPSHOT and earlier
JDK version:  11/17
Complete executable reproducer if available (e.g. GitHub Repo):

Steps to reproduce
Add to MultipleInstancesResponseTypeTest:
   @Test
    void testListMatchesArrayList() {
        List<QueryResponseInterface> testResponse = new ArrayList<>();
        Class<?> clazz = testResponse.getClass();
        ResponseType<?> sut = ResponseTypes.multipleInstancesOf(QueryResponseInterface.class);
        assertTrue(sut.matches(clazz));
    }

Expected behaviour
A MultipleInstancesResponseType should return true for an Iterable.class.
Actual behaviour
A MultipleInstancesResponseType would return true for an Iterable.class.
org.opentest4j.AssertionFailedError: expected: <true> but was: <false>
	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
	at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
	at org.junit.jupiter.api.AssertTrue.failNotTrue(AssertTrue.java:63)
	at org.junit.jupiter.api.AssertTrue.assertTrue(AssertTrue.java:36)
	at org.junit.jupiter.api.AssertTrue.assertTrue(AssertTrue.java:31)
	at org.junit.jupiter.api.Assertions.assertTrue(Assertions.java:179)
	at org.axonframework.messaging.responsetypes.MultipleInstancesResponseTypeTest.testListMatchesArrayList(MultipleInstancesResponseTypeTest.java:365)

Proposed solution:
I think this is caused by:
https://github.com/AxonFramework/AxonFramework/blob/6616b5fce22107221bfac1e2d423ef74c981a665/messaging/src/main/java/org/axonframework/messaging/responsetypes/AbstractResponseType.java#L84
    protected boolean isIterableOfExpectedType(Type responseType) {
        Type iterableType = TypeReflectionUtils.getExactSuperType(responseType, Iterable.class);
        return iterableType != null && isParameterizedTypeOfExpectedType(iterableType);
    }

The value of iterableType evaluates to Iterable.class.
Then in:
https://github.com/AxonFramework/AxonFramework/blob/6616b5fce22107221bfac1e2d423ef74c981a665/messaging/src/main/java/org/axonframework/messaging/responsetypes/AbstractResponseType.java#L113
The instanceof ParameterizedType always returns false for Iterable.class.
Proposed fix: remove the && isParameterizedTypeOfExpectedType(iterableType) from isIterableOfExpectedType.
I did not want to create a pull-request, as I am not sure of any side-effects that this may have in other parts of the framework.
Currently, this bug has the consequence that if the type check that has been added in https://github.com/AxonFramework/AxonFramework/issues/2331 is in place, a subscription query update message with multiple instances of type and a List payload will not be delivered to the client.

So this change would break a few other tests and scenarios. Will take a closer look, how this could be resolved for the UpdateEmitter specifically.

So I figure the added type check introduced the bug. As far as I understand, the type-checking mechanisms of the ResponseType classes are primarily for checking method parameters and not for updates at runtime.
So I propose to extend the type checking in the UpdateEmitter. Due to type erasure, you can only check if the object is an Iterable etc. but not if the content of the Iterable matches the type. The trivial "matches" approach via the ResponseType does not work reliably for multiple instances and for Optionals.

So I figure the added type check introduced the bug. As far as I understand, the type-checking mechanisms of the ResponseType classes are primarily for checking method parameters and not for updates at runtime.
So I propose to extend the type checking in the UpdateEmitter. Due to type erasure, you can only check if the object is an Iterable etc. but not if the content of the Iterable matches the type. The trivial "matches" approach via the ResponseType does not work reliably for multiple instances and for Optionals.

Just to be certain, but PR #2343 introduces the fix as you've suggested, correct?

Yes. However, I am unsure if this fix covers all the types you expect to be emitted via the emitter. It only checks for .isArray, Optional or Iterable. In all other cases, its checks for isAssignableFrom between the expected type of the query and the class of the object to be emitted.

And, of course, you cannot check for the type contained in the update. A way to make this reliable in the face of type erasure would be an additional emit method taking the generics class or a ResponseType as an explicit parameter. But that would be an API change.

Thanks for the effort once more, @heutelbeck. We've approved the pull request and I've merged it just now.


Add TypeRefHasher v1.0.3

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

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

Microsoft Reviewers: Open in CodeFlow

Single pr should only contain 1 manifest and directory of manifest and version should match

I give up...


split test off

cc @lynzrand

Pull Request Test Coverage Report for Build 1695
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.608%

Totals

Change from base Build 1692:
0.0%

Covered Lines:
2629

Relevant Lines:
2967

💛  - Coveralls

Pull Request Test Coverage Report for Build 1703
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.675%

Totals

Change from base Build 1702:
0.0%

Covered Lines:
2623

Relevant Lines:
2958

💛  - Coveralls


Admin client has no support to launch a server

The admin client has support for configuring a server.
However once configured, there is no ability to then control (including start)  the server that has been configured
ie the equiv of posting to
"/open-metadata/platform-services/users/" + user + "/server-platform/origin"

identified whilst working on #426

This is still needed.

This is still needed.

The platform origin request is part of the platform services rather than the admin services (see module name in URL).
It is available through the PlatformServicesClient.
However, I think this is still a valid issue as the commands to start and stop servers is owned by the admin services and currently is not included in the admin clients.  I have codes an new Admin Services client called OMAGServerOperationsClient and this starts and stops servers -as well as adds open metadata archives to a running server.
The new client should be in my next PR