murrine engine not found when using XFCE

Issue description
GTK+-2 applications running in the XFCE desktop with a theme that depends on the murrine engine fails to load the engine. For instance, when launching gvim on a terminal with the theme set to Arc-Dark, I am getting the following messages:
$ gvim ~/.gvimrc

(gvim:4640): Gtk-WARNING **: Unable to locate theme engine in module_path: "murrine",
[...]

With other window managers (like openbox) this does not happen.
Looking at the XFCE module one can see that the environment variables GTK_PATH and GTK_DATA_PREFIX are explicitly set and some engines and themes are also explicitly included in the environment.
I have noticed that when the GTK_DATA_PREFIX variable is unset after login in XFCE, the murrine engine is found.
So, why is GTK_DATA_PREFIX set in the XFCE module? Is it really needed? If so, how to fix the issue of not finding the murrine engine?
Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)
16.09pre85363.886c03a (Flounder)
Nix version: (run nix-env --version)
nix-env (Nix) 1.11.2
Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)
"16.09pre85363.886c03a"

I too use the Arc theme and this happens to me as well. I don't have a GTK_DATA_PREFIX set and I'm not using a DE though.

I enabled both gtk2 and gtk3 themes next way:
    extraInit = ''
      # GTK2 theme
      export GTK_PATH=$GTK_PATH:${pkgs.gtk-engine-murrine}/lib/gtk-2.0:${pkgs.pkgsi686Linux.gtk-engine-murrine}/lib/gtk-2.0https://ru.wikipedia.org/wiki/WebP
      export GTK2_RC_FILES=${pkgs.writeText "iconrc" ''gtk-icon-theme-name="Numix-Light"''}:${pkgs.numix_solarized-gtk-theme}/share/themes/Numix\ Solarized/gtk-2.0/gtkrc:$GTK2_RC_FILES
      # GTK3 theme
      export GTK_THEME="Numix Solarized"
      # export XDG_DATA_DIRS="${pkgs.numix_solarized-gtk-theme}/share:$XDG_DATA_DIRS"
      # export XDG_CONFIG_DIRS="/etc/xdg:$XDG_CONFIG_DIRS"

      # SVG loader for pixbuf (needed for svg icon themes)
      export GDK_PIXBUF_MODULE_FILE=$(echo ${pkgs.librsvg.out}/lib/gdk-pixbuf-2.0/*/loaders.cache)
      # Set GTK_DATA_PREFIX so that GTK+ can find the system themes.
      export GTK_DATA_PREFIX=$GTK_DATA_PREFIX:${config.system.path}:${pkgs.numix_solarized-gtk-theme}
    '';

It works for me.

Oh, and yes, I use custom theme:
      numix_solarized-gtk-theme = with pkgs; stdenv.mkDerivation rec {
        version = "0.0.1";
        name = "numix_solarized-gtk-theme-${version}";

        src = fetchFromGitHub {
          owner = "Ferdi265";
          repo = "numix-solarized-gtk-theme";
          rev = "b6fce07c95a84dd618ed8594d441d0543192c1bb";
          sha256 = "0gix5gx0n2bqyig3ngc9ky4296kqhnjr2vzjkidsjlkqp8raalmw";
        };

        # should be build dependency, maybe, but as buildDependency it doesn't see sass executable in PATH
        buildInputs = [ gdk_pixbuf glib sass libxml2 ];
        buildPhase = ''
          make DESTDIR=$out
        '';

        prePatch = ''
          sed -i 's/Numix/Numix Solarized/g' index.theme
          export bkgrn=586e75
          export white=eee8d5
          export black=002b36
          sed -i 's/selected_bg_color: #[^\\n]*;/selected_bg_color: #'$bkgrn';/g' gtk-3.0/scss/_global.scss
          sed -i 's/selected_bg_color: #[^\\n]*;/selected_bg_color: #'$bkgrn';/g' gtk-3.20/scss/_global.scss
          sed -i 's/white: #fff;/white: #'$white';/g' gtk-3.0/scss/_global.scss
          sed -i 's/white: #fff;/white: #'$white';/g' gtk-3.20/scss/_global.scss
          sed -i 's/black: #000;/black: #'$black';/g' gtk-3.0/scss/_global.scss
          sed -i 's/black: #000;/black: #'$black';/g' gtk-3.20/scss/_global.scss
          sed -i 's/nselected_bg_color:#[^\\n]*\\n/nselected_bg_color:#'$bkgrn'\\n/g' gtk-2.0/gtkrc
        '';

        installPhase = ''
          install -dm 755 $out/share/themes/Numix\ Solarized
          cp -dr --no-preserve='ownership' {LICENSE,CREDITS,index.theme,README.md,gtk-2.0,gtk-3.0,gtk-3.20,metacity-1,openbox-3,unity,xfce-notify-4.0,xfwm4} $out/share/themes/Numix\ Solarized/
        '';

        meta = {
          description = "Numix Solarized GTK theme";
          homepage = http://bitterologist.deviantart.com/art/Numix-Solarized-417575928;
          license = stdenv.lib.licenses.gpl3;
          platforms = stdenv.lib.platforms.all;
        };
      };

Thanks, I'll try this way. Anyway, since murrine it's specified as a depencency of Arc it should be nix to correctly setup the environment.

OT @Pitometsu: would you mind PR numix-solarized-gtk? :)

@nico202, I'm not using NixOS for now, sorry, feel free to reuse any my source code snippets.

Can confirm, I'm using xfce with the Arc-dark theme

Installing gtk-engine-murrine in systemPackages made the error go away for me. Unfortunately, my themes are still broken in the exact same way...
I'd imagine that the same applies for other missing theme engines.

Nix can't propagate global environment variables from installed packages, so it is not currently possible to fix this issue for user-installed packages. However, it is rather easy to do for environment.systemPackages case via propagatedUserEnvPkgs.
I'll switch all themes that use custom engines to propagatedUserEnvPkgs.


Add support for value of type JTokenType.Date

Hi there,
I'm happy to accept this but could you fix the indenting please before I do?

Sure. Incoming ;)

That looks good to me, thank you.
I'll try get a new release later this evening.

Awesome. Thanks.


readFile Strips Out Conditional Formatting

Hello. Many, many thanks for your hard work on this library.
I have a workbook with two sheets, one that contains a few columns of validation data, and sheet another for inputting new data. Some columns on the latter sheet have conditional formatting, and use the validation data sheet to validate new inputs.
When I use XLSX.readFile(myWorkbook), the parsed file doesn't appear to contain conditional formatting data. I wanted to dynamically write in the validation data and save the workbook, but after reading the workbook, the conditional formatting is gone.
Thanks again.

We offer this in the Pro compendium.  Since companies have paid for the features already, it would be unfair to them if we turned around and made it available as open source.  We have a longer comment in a gist.


Thread 1: signal SIGABRT

I tried to implement this framework into swift project.
But when I call start function, I'm getting crash.
Could you help me?

The Bare thread will abort on uncaught exceptions by default so check the logs for more information. If there's nothing there, post your code and I'd be happy to take a look.

See https://github.com/holepunchto/bare-ios/blob/main/app/App.swift for a reference on how to call Bare Kit from Swift.


Feature Request: send file then exit

Hi,
I'm very happy with 2 small shell functions I made to send and receive files without thinking about it too much. Usage is very simple: edge_send path/to/file and edge_receive (assuming you set EDGEVPNTOKEN in both env).
edge_send () {
	local name=$(echo $EDGEVPNTOKEN| sha256sum)
	local name=${name:0:10}
	local pathoffile=$1
	echo "Edge token:\n$EDGEVPNTOKEN\nName: $name\nPath: $pathoffile\n\n"
	if [[ -z "$pathoffile" ]]
	then
		echo "Must supply a path"
	else
		edgevpn file-send --name "$name" --path "$pathoffile"
	fi
}

edge_receive () {
	local name=$(echo $EDGEVPNTOKEN| sha256sum)
	local name=${name:0:10}
	local pathoffile=$1
	if [[ -z "$pathoffile" ]]
	then
		pathoffile="edgevpn_transfer.$(date +'%Y%m%d_%H%M%S').file"
	fi
	echo "Edge token:\n$EDGEVPNTOKEN\nName: $name\nPath: $pathoffile\n\n"
	edgevpn file-receive --name "$name" --path "$pathoffile"

But the thing is, there does not seem to be a way for edge_receive to stop as soon as it's done sending the file once. That would be nice as it would for example allow me to send multiple files in a loop or whatever.
Edit: I shared an improved version at #545

Actually, if you add thiq feature then it would be easy to set a sync between two machines. I'm thinking a simple script that uses inotify on both ends, as soon as it's triggered then store the diff in a CRDT, and send it to the other machine, which then reverses the step.
For that, it seems the only missing part is the fact that send-file never stops after sending. Maybe using expect could solve this? Quite unelegant though

Well, actually with claude sonnet I was able to make expect work and that's enough for me. I'm closing this now but still think it would be nice to have this as a default feature!


vehicle.Acceleration memory offset issue

This may of been fixed in a pending version, if so my apologizes.
Apparently the new gunrunning update has changed the vehicle.Acceleration memory offsets.
Example:
using Me.Player.CurrentVehicle.Acceleration.ToString())
Pre-Gunrunning
it returns 1 when going forward
0 when stopped
-1 in reverse
After-Gunrunning
it returns higher than 0.2 when going forward
0.2 when stopped
higher than 0.2 in reverse

This should be soon fixed by a later release. I do know that @kagikn was working on fixing some helicopter memory address offsets, but I'm not sure if he knows about any other ones such as this.

I wish I had the knowledge to find the offsets myself.


Update README.md

added readme notes on how to output the extracted content using the parser interface.

Coverage remained the same at 40.038% when pulling affea80c89c94dc8d7a333706b493677c7de6eb0 on lathakris:patch-1 into 80af8d2f12d081cdc9bb742795d87a1b431df9f5 on chrismattmann:master.

LGTM thanks @lathakris !


CI Script: Check if Issuer Country is set

Please insert a checkup function which checks if the issuing country is correctly set and follows iso standard.

@SchulzeStTSI I assume issuing country is cbor claim key1?

Cbor claim key 1 from 2.6.3 Common Payload Values in https://ec.europa.eu/health/sites/default/files/ehealth/docs/digital-green-certificates_v3_en.pdf
[r/v/t]/is from Certificate type specific information in https://ec.europa.eu/health/sites/default/files/ehealth/docs/covid-certificate_json_specification_en.pdf

@chrloch please note https://github.com/eu-digital-green-certificates/dcc-quality-assurance/commit/315ea3064a810dd3ec7145c3887dd61a1b072efd

@mkubicek-dtc Yes CBOR Claim Key one. It should be set to the right ISO code matching to the uploading folder.  the field "co" within the certificate should be just a valid iso code, because this country can be a different one as the issuing country.

Implemented, but if merged, will fail 4 DCCs:
FAILED tests/test_qrqualitycheck.py::test_country_in_path_matches_issuer[P:\cwa\dcc-quality-assurance\EL\1.0.0\REC.png] - AssertionError: assert 'EL' == 'GR'
FAILED tests/test_qrqualitycheck.py::test_country_in_path_matches_issuer[P:\cwa\dcc-quality-assurance\EL\1.0.0\TEST.png] - AssertionError: assert 'EL' == 'GR'
FAILED tests/test_qrqualitycheck.py::test_country_in_path_matches_issuer[P:\cwa\dcc-quality-assurance\EL\1.0.0\VAC.png] - AssertionError: assert 'EL' == 'GR'
FAILED tests/test_qrqualitycheck.py::test_country_in_path_matches_issuer[P:\cwa\dcc-quality-assurance\FR\1.3.0\VAC.png] - AssertionError: assert 'FR' == 'CNAM'
FAILED tests/test_qrqualitycheck.py::test_country_code_formats[P:\cwa\dcc-quality-assurance\FR\1.3.0\VAC.png] - ValueError: Invalid country code: CNAM


[Theme] inconsistencies w.r.t Material Design

The current raw themes and theme-manager are not totally consistent with Material Design Color recommendations.
Some color choices and corresponding assignments in theme-manager need to be re-thought to adhere to Material Design guidelines.

@callemall/material-ui : Do you guys have an idea about this? Is this resolved or still a WIP?

Closing this.


Fix SAR model to accept integer type for userId and itemId

Fixes #2274
Update SAR model to accept userId and itemId as integer types (LongType).

SAR.scala

Update calculateUserItemAffinities method to handle userId and itemId as LongType.
Update calculateItemItemSimilarity method to handle userId and itemId as LongType.

test_ranking.py

Add test case test_adapter_evaluator_sar_with_long to verify userId and itemId as LongType.

Smart Adaptive Recommendations.md

Update documentation to reflect that userId and itemId can be of LongType.

For more details, open the Copilot Workspace session.

Acrolinx Scorecards
A minimum total score of 80 is required.
Select the total score link to review all feedback on clarity, consistency, tone, brand, terms, spelling, grammar, readability, and inclusive language. You should fix all spelling errors regardless of your total score. Fixing spelling errors helps maintain customer trust in overall content quality.

Article
Total score(Required: 80)
Words + phrases(Brand, terms)
Correctness(Spelling, grammar)
Clarity(Readability)

:white_check_mark: docs/Explore Algorithms/Other Algorithms/Smart Adaptive Recommendations.md
78
100
65
70

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx


[proto] having gRPC service names PascalCase'd and others.

I think there are minor quirks with some of our protobuf files and definitions. Nothing is broken, but some best practices and guidance to be consistent within the project could be applied. Some rules especially apply to ensure multi-language compatibility. These are a few points to consider, there are others:
a) Servicenames you should use PascalCase: https://developers.google.com/protocol-buffers/docs/style#services
b) Enums should be prefixed with TYPE_ Uber Styleguide, Enums , Google Style Guide, Enums
c) Enums should have a zero value with _UNSPECIFIED / _INVALID prefixed
d) gRPC method options (like google.api.http) use historical optional delimiters when they use the aggregate syntax (protobuf TextFormat) to set option messages values.
Those rules and others are from both Uber Styleguides and Googles Protocolbuffer Style Guide:
https://developers.google.com/protocol-buffers/docs/style
https://github.com/uber/prototool/blob/dev/etc/style/uber1/uber1.proto
https://github.com/uber/prototool/blob/dev/style/README.md#enums
As always "consistency is key" and we might like to define our own style and best practices for cloudstate. Perhaps our "Formal Spec" #119 Draft-PR could be used to address these issues. Therefore I leave the PR #511 as a Draft-PR to track changes and collaborate on succeeding changes on other places in the code or documentation.
Also, we could use a linter to ensure consistency (buf).

Yes, good ideas. And using a linter.

Yes, good ideas. And using a linter.


Create MultivariateCoupledNormal distribution class

Create a MultivariateCoupledNormal distribution class that takes in coupling value kappa and number of dimensions dim, and includes the following functions:

log_prob. See our latest nsc code here.

sample_n. See our latest nsc code here.

entropy. See our latest nsc code here.

kl_multcouplednormal_multcouplednormal. Or the KL-Divergence function for two MultivariateCoupledNormal distributions.

Use the respective CoupledNormal functions as a foundation to build the MultivariateCoupledNormal ones.

 This has been completed.


[release-2.6] update Strict-Transport-Security header

Signed-off-by: Subbarao Meduri<EMAIL_ADDRESS>

/test sonarcloud


Add vep versions

Moved the VEPed context paths out of vep utils and into resources, also turning it into a versioned resource. If we add a VEPed context HT for v101 when fixed then we will need to have versioning. Also changed vep_or_lookup_vep to handle the versioning and if a version is not present it will warn the user that it is going to VEP everything from scratch.

I'm not sure if we can implement or how often it will come up but if an individual spins up a hail cluster with VEP, the default config file will be copied to "file:///vep_data/vep-gcloud.json". If hail updates to VEP 101, someone running this script can request the VEP 95 context HT and if any variants get filtered to the revep_ht, they would then have a HT with VEP'd variants from two different versions of VEP. Is this something we want to try to prevent? Or maybe a just print a warning reminding people to make sure, the VEP context table is the same as the version of VEP they are using?

Yeah, I thought about that too, but I am not sure how to grab the VEP version that was used in an init script for the running cluster. A warning would be easiest, but much less satisfying. @nawatts do you know an easy way to grab the version of VEP that will be used with hl.vep?

I'm not sure if we can implement or how often it will come up but if an individual spins up a hail cluster with VEP, the default config file will be copied to "file:///vep_data/vep-gcloud.json". If hail updates to VEP 101, someone running this script can request the VEP 95 context HT and if any variants get filtered to the revep_ht, they would then have a HT with VEP'd variants from two different versions of VEP. Is this something we want to try to prevent? Or maybe a just print a warning reminding people to make sure, the VEP context table is the same as the version of VEP they are using?

Yeah, I thought about that too, but I am not sure how to grab the VEP version that was used in an init script for the running cluster. A warning would be easiest, but much less satisfying. @nawatts do you know an easy way to grab the version of VEP that will be used with hl.vep?

A similar issue occurs if someone uses a different VEP configuration that the one used to generate the context Table (see #165).
Maybe we could store the VEP version and configuration in the globals of the context Table and assert that they match the version / configuration used when vep_or_lookup_vep is run? It's still possible that the same VEP command could refer to different data files, but there's only so much we can do. We could also add a warning in the docstring.
It looks like VEP doesn't have a CLI option to print version info. The version is included in the output of vep --help. However, the format has changed between VEP 85 and 95.
VEP 85 has:
version 85

while VEP 95 has:
Versions:
  ensembl              : 95.4f83453
  ensembl-funcgen      : 95.94439f4
  ensembl-io           : 95.bd1a78d
  ensembl-variation    : 95.858de3e
  ensembl-vep          : 95.1

The easiest thing may be to store the entire output of vep --help vs trying to parse version(s) out of it. For getting the VEP help, this should work for Dataproc clusters created with hailctl dataproc start --vep. It's not very robust though, it assumes that the command is like /path/to/vep ... vs perl /path/to/vep.
import json
import os
import subprocess

def get_vep_help():
    with hl.hadoop_open(os.environ["VEP_CONFIG_URI"]) as vep_config_file:
        vep_config = json.load(vep_config_file)
        vep_command = vep_config["command"]
        vep_help = subprocess.check_output([vep_command[0]]).decode("utf-8")
        return vep_help

A similar issue occurs if someone uses a different VEP configuration that the one used to generate the context Table (see #165).
Maybe we could store the VEP version and configuration in the globals of the context Table and assert that they match the version / configuration used when vep_or_lookup_vep is run? It's still possible that the same VEP command could refer to different data files, but there's only so much we can do. We could also add a warning in the docstring.
It looks like VEP doesn't have a CLI option to print version info. The version is included in the output of vep --help. However, the format has changed between VEP 85 and 95.
VEP 85 has:
version 85

while VEP 95 has:
Versions:
  ensembl              : 95.4f83453
  ensembl-funcgen      : 95.94439f4
  ensembl-io           : 95.bd1a78d
  ensembl-variation    : 95.858de3e
  ensembl-vep          : 95.1

The easiest thing may be to store the entire output of vep --help vs trying to parse version(s) out of it. For getting the VEP help, this should work for Dataproc clusters created with hailctl dataproc start --vep. It's not very robust though, it assumes that the command is like /path/to/vep ... vs perl /path/to/vep.
import json
import os
import subprocess

def get_vep_help():
    with hl.hadoop_open(os.environ["VEP_CONFIG_URI"]) as vep_config_file:
        vep_config = json.load(vep_config_file)
        vep_command = vep_config["command"]
        vep_help = subprocess.check_output([vep_command[0]]).decode("utf-8")
        return vep_help


Adding rake:publish

This will allow for a release of what Tyler Ball want's as a standard
release process.

Bumps the minor version
runs github_changelog_generator
commits the things to github
will push to rubygems...after everyone is cool with this PR.

Looking good so far!  Can I help with anything?


Bypass Store from FindEventHandler

Hello,
It looks like FindEventHandler does not support any return except error. Is there any way to implement caching to bypass going to the store?
It seems like the only way to do this at present is to implement your own storage extension.
Is there any plans to support 'tiered' storage extensions? I think the ES storage extension is very good, but in most cases ES is not used as a canonical store. A tiered storage extension would allow for a canonical store that can utilize ES (or redis) as a secondary index which can be guaranteed 'up to date' with the final storage backend through use of hooks.

If I understand your question correctly, you would like to return a ItemList response from the FindEventHandler without going through the data storage, am I right? Do you have a practical exemple of the use of such behavior?
What you could do is to return a custom error in the FindEventHandler and implement a FoundEventHandler that set this specific error to nil and provide your own ItemList. The error in the "pre" hook Find…will skip the storage request, but the "post" hookFound…` is always called to let you modify the final response. But the best option would be certainly to wrap your storage handler.
That second option lead me to your second question about tired storage handlers. The best way to implement that would be to create a storage handler wrapper. I plan on providing a generic wrapper that can route requests to two distinct storage handlers. The routing could be based on methods, like reads on ES and writes on Mongo or some more complexe logic that you could customize via custom code.


Can't download update v3.19.6

Can't download update v3.19.6, it never stops "downloading", anyone else having this issue?

Dupe of #822


Lack of reproducibility despite using a consistent seed

Hello,
We’ve been having some discussion concerning a lack of reproducibility despite using a consistent seed in xgboost when employing Bayesian optimization for the hyperparameters. After some digging, the issue seems to be in the xgboost package itself, but I’m still unclear as to what’s driving this. Could you please take a look? a sample syntax to reproduce the problem is available here: https://github.com/AnotherSamWilson/ParBayesianOptimization/issues/43 2
Thank you!

I think I got the result now.  It's caused by non-deterministic openmp reduction  Same cause as https://github.com/dmlc/xgboost/issues/5019 .


Feat/add synapse fixes

Overview
Adds two bug fixes - throttling rate limit for bot users and a fix for the prometheus port on worker units.
Rationale
Fixes from trunk, propagating to 1/ channels.
Juju Events Changes

n/a
Module Changes

n/a
Library Changes

n/a
Checklist

[x ] The charm style guide was applied
[x ] The contributing guide was applied
[x ] The changes are compliant with ISD054 - Manging Charm Complexity
[x ] The documentation is generated using src-docs
[x ] The documentation for charmhub is updated.
[x ] The PR is tagged with appropriate label (urgent, trivial, complex)

You are also removing enable_irc_bridge here, right?

You are also removing enable_irc_bridge here, right?

Yes, it had to be removed as some checks were failing, This was an effort to test the branching strategy, so I'm closing this one out.

Out of curiosity, what bug does this fix and what does irc-bridge removal do in this case?

This PR is a test against the branching strategy. The irc code had to be removed for the fix to make it through, so we have to draw conclusions from that..I am closing this PR.

Closing.


Scoop does not find binaries from Yarn

$ yarn global add DanielRuf/phantomjs-mocked
yarn global v1.13.0
[1/4] Resolving packages...
[2/4] Fetching packages...
info<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check.
Excluding it from installation.
[3/4] Linking dependencies...
[4/4] Building fresh packages...

success Installed<EMAIL_ADDRESS>with binaries:
      - phantomjs
Done in 19.10s.

Daniel@Daniel-MSI MINGW64 ~/scoop/apps/scoop/current (master)
$ phantomjs
/bin/sh: /c/Users/Daniel/scoop/apps/yarn/current/bin/../../../../../AppData/Local/Yarn/Data/global/node_modules/.bin/phantomjs: No such file or directory

While /c/Users/Daniel/AppData/Local/Yarn/Data/global/node_modules/.bin/phantomjs works.
But one thing is weird. pwd shows me that /c/Users/Daniel/scoop/apps/yarn/current/bin/../../../../../ is /c/Users/Daniel but ll shows a different path.
ll /c/Users/Daniel/scoop/apps/yarn/current/bin/../../../../../
total 65
lrwxrwxrwx 1 Daniel 197121  14 Apr 12  2018 'All Users' -> /c/ProgramData/
drwxr-xr-x 1 Daniel 197121   0 Feb  9 19:21  Daniel/
drwxr-xr-x 1 Daniel 197121   0 Mai  6  2018  Default/
lrwxrwxrwx 1 Daniel 197121  16 Apr 12  2018 'Default User' -> /c/Users/Default/
drwxr-xr-x 1 Daniel 197121   0 Okt 10  2016  Default.migrated/
drwxr-xr-x 1 Daniel 197121   0 Mai  6  2018  DefaultAppPool/
-rw-r--r-- 1 Daniel 197121 174 Apr 12  2018  desktop.ini
drwxr-xr-x 1 Daniel 197121   0 Mai  6  2018  Public/

So there is something very wrong.

Also when I remove the package again with yarn global remove phantomjs-mocked, it is not deleted in the bin directory of scoop but in the bin directory of Yarn.

Duplicate of https://github.com/lukesampson/scoop/issues/1641

Yep, checked other binaries now and Yarn generated a different content. But still weird that the other binary is not deleted and that there are two different ones.

But still weird that the other binary is not deleted and that there are two different ones.


[Dependency Scanner] Always add NonPathCommandLine arguments from Clang scan-deps result

When building a set of command-line options required to build a Clang module, also add NonPathCommandLine from Clang's ModuleDeps. These flags are mandatory in order to build modules discovered by the scanner. For example, they will include -fsystem-module for system modules, which will then alter ClangImporter's behaviour for such modules.
This is a part of the solution for: rdar://70212660

@swift-ci please test

@swift-ci please smoke test

Build failed
Swift Test Linux Platform
Git Sha - a09ae489994d1d3f9e7879fc1493d97e28e3de5d

@swift-ci please test Linux Platform


Add byte cursor read/write utility functions.

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I consider the error returns here to be too much of a 'normal' condition to be an error code that requires the whole thread-local-and-callback error return framework; bringing that framework in has significant overhead.
Also, the test list was updated in the prior commit; the subsequent change is to add in a missing file. I'll squash them when I rebase on master.

Yes, that is what the change uses now.


didSelect not firing when programatically selecting date in 2.7.9

Description:
Same issue as #467 that was fixed in v2.5.0 is now present in 2.7.9.
Programatically selecting date with
let now = Date()
calendar.select(now)

does not fire calendar(_ calendar: FSCalendar, didSelect date: Date, at monthPosition: FSCalendarMonthPosition) but selecting date in the UI does fire this function.
Cocoapods,
Swift 4.0,
iphone 6 -> iOS 11.0 & iphone 5 -> iOS 8.1.3,
Xcode 9.0 GM,
FSCalendar 2.7.9

The same problem as here under objective-c.

I shouldn't, when you do it programmatically no event is taking place. In fact, @WenchaoD already gave an answer about this. Look here https://github.com/WenchaoD/FSCalendar/issues/99#issuecomment-144270662

@mchirino89, thanks, somehow I missed that. Closing this now.


Add Support for nullishCoalescingOperator

Is your proposal related to a problem?
TypeScript 3.7 released today which includes https://github.com/tc39/proposal-nullish-coalescing
After turning it on with TS, CRA throws a babel error Support for the experimental syntax 'nullishCoalescingOperator' isn't currently enabled (56:22):
Add @babel/plugin-proposal-nullish-coalescing-operator (https://git.io/vb4Se) to the 'plugins' section of your Babel config to enable transformation.
Edit:
Also would like to see Support for the experimental syntax 'optionalChaining' isn't currently enabled (18:17): to be fixed as well
Describe the solution you'd like
Add the babel plugin to CRA since this is now in typescript's stable version. That we it can be used without ejecting.

There have been several PRs and issues opened for this already. You can follow the progress here: #7438.
Even though this is now supported in TypeScript and Babel, it's not fully supported by other tools like ESLint and Prettier.

Sorry about that. I promise I searched for it but githubs search leaves much to be desired...


Support artifacts section

Closes #519

Thanks @daavoo!
Questions I have:

Not a blocker, but this reminds me we need to revisit the lightning callback since it's most popular but doesn't auto-log the model artifact.
Do we know what the artifact names look like for each framework? Should we be inferring them from the path or trying to come up with some better name for them?
If we log the artifact at the end of each step/epoch, will it raise an error because another artifact exists with the same name?

Do we know what the artifact names look like for each framework? Should we be inferring them from the path or trying to come up with some better name for them?

This would be a breaking change, so can we at least see what the artifact names look like and make a decision on which way to go here?

@daavoo We agreed that we would drop the callback support and make this depend on explicit calls to log_artifact for now, right? What's the status?

@daavoo We agreed that we would drop the callback support and make this depend on explicit calls to log_artifact for now, right? What's the status?

Dropped from the P.R.

Sorry for so many iterations here @daavoo. The blocker left for me is what to do with attrs if there isn't type=model. IMO we should write any attrs to artifacts. It's simple and even if it doesn't do much now, neither does anything else we do with those attrs (at least it could be useful in the future and an artifact registry would work with the existing artifacts written to artifacts).

Pushed a new version

Don't see much point in merging until we test end-to-end against Studio.

@daavoo, looks like it will be working, but if you want me, I can test this against what I have now. WDYT?

@daavoo, looks like it will be working, but if you want me, I can test this against what I have now. WDYT?

Sounds good, what just saying that we can wait for merge and release until it works with Studio prod

Can we merge and push people to start using it now, letting them know what's coming? We have people today asking to get started with the model registry. I'd like to tell them to use this and their models will appear once Studio is updated. Otherwise, I can only tell them do it the old way but it will break soon, or they can't start using it at all.


test commit for new fiddles

This is a test commit to see how some updated jsfiddles appear on the dev site for prebid.org (and hopefully bypass the cookie issues seen when locally host the server).

@jsnellbaker , @MartianTribe - is this ready to be merged?


Error: You must provide the URL of lib/mappings.wasm by calling SourceMapConsumer.initialize

 [webpack-dev-server] 404s will fallback to '/index.html'
10% building 0/11 entries 16/26 dependencies 0/16 modulesC:\Software Development\open-graph-outlook-auth\Office-Add-in-samples\Samples\auth\Office-Add-in-Microsoft-Graph-React\node_modules\source-map\lib\read-wasm.js:8
throw new Error("You must provide the URL of lib/mappings.wasm by calling " +
^
Error: You must provide the URL of lib/mappings.wasm by calling SourceMapConsumer.initialize({ 'lib/mappings.wasm': ... }) before using SourceMapConsumer
at readWasm (C:\Software Development\open-graph-outlook-auth\Office-Add-in-samples\Samples\auth\Office-Add-in-Microsoft-Graph-React\node_modules\source-map\lib\read-wasm.js:8:13)
at wasm (C:\Software Development\open-graph-outlook-auth\Office-Add-in-samples\Samples\auth\Office-Add-in-Microsoft-Graph-React\node_modules\source-map\lib\wasm.js:25:16)
at C:\Software Development\open-graph-outlook-auth\Office-Add-in-samples\Samples\auth\Office-Add-in-Microsoft-Graph-React\node_modules\source-map\lib\source-map-consumer.js:264:14
Node.js v20.9.0

Note: This template is only for reporting bugs about samples in this repo. If you have a product bug to report about Office web Add-ins, please submit the issue at https://github.com/OfficeDev/office-js/issues. This will help us respond to your issue faster.
(DELETE THIS PARAGRAPH AFTER READING)

URL of sample
URL linking to the sample that has the bug.
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

Platform [PC desktop, Mac, iOS, Office Online]: ______
Host [Excel, Word, PowerPoint, etc.]: ______
Office version number: ______
Operating System: ______
Browser (if using Office Online): ______

Additional context
Add any other context about the problem here.

Hi @RA-Davies,
Could you please fill out the issue template? That will help us triage and understand where things are going wrong in the sample.


Update the CLM and MLM examples

What does this PR do?

This PR updates the CLM and MLM examples to add support for streaming datasets.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Disable HTTP/2

This fixes potential errors when server uses HTTP/2
error:
stream error: stream ID 1; PROTOCOL_ERROR

#145168591

This is the bug on our backlog #145135295


Refactor into a library

I'd like to use hq in one of my personal rust projects, but I don't want to use the binary.
Could we refactor hq into a library and make the binary use that lib instead?

That's a great idea - I don't have time to work on it right now, but when I next get round to making changes to this project I will give it a go.
Can you describe what kind of interface you'd like for the library?

I'd  love to have something like a pub fn hq -> Result<serde::Value, HqError>

I was having more thoughts about this and figured I'd also love to have pub fn validate_query(query: impl ToString) -> Result<(), HqError>

Yep! This is my next project to show some love to. I’m going to be making this into a library, changing a few things around and making it a bit more easy to use.


Restores test and refactors component and test for layout-super-navigation-header

What
These changes restores a test that was temporarily removed and refactors both that and the component it relates to.
(Trello ticket: Refactor and improve recent change to layout-super-navigation-header component)
Why
When we made the changes in PR#3918 (Remove "Popular" links from super navigation header) we lost a recently added accessibility improvement, specifically to close the search menu when a keyboard user tabs out of that element. This was because this addition relied on the presence of links that were removed in the previous PR. We decided at the time to go ahead with that change and return to this to restore the lost behaviour.
This restores that by refactoring the original method dealing with that so that it doesn't rely on the presence of links but broadens the selector to include other tab-able elements. It also makes a small change to the original code for the search menu to make both more consistent with each other.

Nice work on adding this functionality back in to the super navigation menu.
The code itself looks good to me, just made some small suggestions around the naming of things.

Thanks @MartinJJones - I've made those changes and updated the CHANGELOG for this PR. (I'll squash the commits if it's all OK). Could you have another look over it? Cheers.


command doesnot work

below command does not work in powershell
Set-MpPreference -PUAProtection
correct commands are as follows
Set-MpPreference -PUAProtection disable
Set-MpPreference -PUAProtection enable
Set-MpPreference -PUAProtection auditmode

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

ID: 2b27975a-fa49-d9cd-d500-792448c57f9e
Version Independent ID: 21c3eb7b-065a-b68a-8a88-dfce45571b0d
Content: Block potentially unwanted applications with Windows Defender Antivirus - Windows security
Content Source: windows/security/threat-protection/windows-defender-antivirus/detect-block-potentially-unwanted-apps-windows-defender-antivirus.md
Product: w10
Technology: windows
GitHub Login: @denisebmsft
Microsoft Alias: deniseb

@illfated .Please review the changes of PR #5796

I presume these command parameters have been tested in their appropriate scenario, so my review is based on that presumption.

Closing the issue as the PR has been merged. Thank you for your continued engagement with the community, @RAJU2529. Much appreciated.


Data processing

I am very interested in your work, I now have some doubts：
(1)  In your paper, the image size  is 153X193X50, and The image size I used is 240X240X155, will the image size affect the result?
(2)  In your paper, the MR size is 32X32X32, the CT size is 16X16X16, the image MR size obtained in the code is 64X64X64, the CT size is 48X48X48, but your note is THE MR size is 32X32X32, the CT size is 24X24X24, What dimension size should I take?

The image size doesnot affect much. Donot worry about it.
In my later works, I begin to use 2D big patches, for example, 2402405 for input MRI, and 2402401 for output CT. Since this kind of pseudo 3D (actually 2D network) can be much faster for training and also easier to get converged, I encourage you to use such settings.


Update location.yaml

Added Multiple Device in Verify Location

Added Multiple Device in Verify Location

What is the purpose of this PR to  Device Location API. Did you intend to contribute to Device Status API instead? It sound to me that that response model is being discussed there

Added Multiple Device in Verify Location

@chia2017 What is the purpose of this PR to Device Location API. Did you intend to contribute to Device Status API instead? It sounds to me that that response model is being discussed there

I wanted to first internally review that then apply for pull request, I close this pull request.


fix(graphql-docs/build): rollup error thrown during build

during the dts transformation, the css import was not recognized, hence marking it as external in the dts transform didn't throw the error

"extract: true" in postcss plugin makes sure the CSS file gets extracted to the final bundle as well

I think the pre-commit caused stylistic changes 🤔 lmk if anything needs to be changed or added

Nice! Merged!
Thanks @sadn1ck !


[987] setup users endpoint

Context
Originally started as a spike to get a JSON API endpoint, and to provide a /users endpoint at the same time.
Changes proposed in this pull request
Add the /users endpoint with only one action, to show user info. This is the first endpoint to use JSON API.
Guidance to review
We technically may not actually need the /users endpoint for any of the stories currently on the board, however we do think that we'll be building endpoints off of /users for user-scoped data and that we'll want to have the canonical endpiont /users/:user_id for user info at some point. So we've decided to proceed with creating this endpoint.

I guess that the tests for the JSON output of /api/v2/users are a bit different from what some of us are used to, but I'm trying it out since I'm not a big fan of doing comparisons of large hash structures like we've done with the other endpoints. What do people think?

I guess that the tests for the JSON output of /api/v2/users are a bit different from what some of us are used to, but I'm trying it out since I'm not a big fan of doing comparisons of large hash structures like we've done with the other endpoints. What do people think?

I personally find the large hash structure specs better as documentation of what the response looks like, but your approach is readable enough.

 Setup users endpoint on rails API:V2

@benilovj I've changed the test to test the JSON verbatim.


go: Switch to CometBFT v0.34.27

This PR switches from our fork of Tendermint to our fork of CometBFT, which is a drop-in replacement.
Here is our fork of CometBFT that contains the exact same patches as our Tendermint fork: https://github.com/oasisprotocol/cometbft/tree/v0.34.27-oasis1

Running go mod tidy returns an error.

That's strange, we run this as part of CI?

Running go mod tidy returns an error.

I have no idea how to fix this, it seems like a problem in an upstream package or go mod tidy itself.

Running go mod tidy returns an error.

I have no idea how to fix this, it seems like a problem in an upstream package or go mod tidy itself.

This will probably fix it: https://github.com/golang/go/issues/60352
But I'm not waiting for 1.20.5 to be released just for this, going to go ahead with the merge now :)

That's strange, we run this as part of CI?

We should probably fix common.mk (replace ; with &&):
$(GO) mod tidy; \

We should probably fix common.mk (replace ; with &&):
$(GO) mod tidy; \

We can do this after Go 1.20.5 is released, which should have a fix for this issue, otherwise all our builds will keep failing...

I have no idea how to fix this, it seems like a problem in an upstream package or go mod tidy itself.

I think you need to specify which version to use. I guess that should be 1.0.1, but I'm not sure if this breaks anything.
One solution is to clear the require section in the go.mod file and then run the following commands:
go get<EMAIL_ADDRESS>go mod tidy


Mixed case XML ID and Routing Name results in failed TLS handshake

Behavior Seen: When an HTTPS delivery service is created with a mixed case XML ID and/or mixed case Routing Name, the SNI sent by a client appears to cause Apache Traffic Server to respond with an empty X509 server certificate chain.  DNS A/AAAA records are considered to be case insensitive via RFC4343.  TLS Extension Server Name Indication (SNI) is also supposed to be case insensitive, but RFC 4366 and RFC 6066 seem to be in contradiction with respect to proper SNI string matching (case sensitive vs. case-insensitive).
Possible Solution: Delivery Service XML ID and Routing Name should only contain lowercase alpha characters due to the ambiguity of how SNI is sent by HTTPS/TLS clients and the comparison algorithm used by HTTPS servers to process an SNI match.
Impacted Components: Apache Traffic Server - Delivery services in DNS routing mode appear to be impacted by this bug when the SNI is sent directly to the edge cache. No X509 certificate chain is sent unless the SNI matches the original remap/hostname exactly (comparison algorithm appears to be incorrectly case-sensitive).
Example Request using original mixed case Delivery Service URL:
$ curl -o /dev/null -Lvs https://edge.DeMo1.mycdn.ciab.test

* About to connect() to video.DeMo1.mycdn.ciab.test port 80 (#0)
*   Trying <IP_ADDRESS>...
* Connected to video.DeMo1.mycdn.ciab.test (<IP_ADDRESS>) port 80 (#0)
> GET / HTTP/1.1
> User-Agent: curl/7.29.0
> Host: video.DeMo1.mycdn.ciab.test
> Accept: */*
>
< HTTP/1.1 302 Found
< Location: http://edge.DeMo1.mycdn.ciab.test/
< Content-Length: 0
< Date: Wed, 16 Jan 2019 23:34:01 GMT
<
* Connection #0 to host video.DeMo1.mycdn.ciab.test left intact
* Issue another request to this URL: 'http://edge.DeMo1.mycdn.ciab.test/'
* About to connect() to edge.DeMo1.mycdn.ciab.test port 80 (#1)
*   Trying <IP_ADDRESS>...
* Connected to edge.DeMo1.mycdn.ciab.test (<IP_ADDRESS>) port 80 (#1)
> GET / HTTP/1.1
> User-Agent: curl/7.29.0
> Host: edge.DeMo1.mycdn.ciab.test
> Accept: */*
>
< HTTP/1.1 200 OK
< Content-Type: text/html
< Accept-Ranges: bytes
< ETag: "3812539862"
< Last-Modified: Thu, 10 Jan 2019 22:53:33 GMT
< Cache-Control: public, max-age=300
< Access-Control-Allow-Origin: *
< Access-Control-Allow-Headers: Accept, Origin, Content-Type
< Access-Control-Allow-Methods: GET, POST, PUT, OPTIONS
< Content-Length: 1881
< Date: Wed, 16 Jan 2019 23:34:01 GMT
< Server: ATS/7.1.4
< Age: 2
< Via: http/1.1 mid.infra.ciab.test (ApacheTrafficServer/7.1.4 [uScMsSfWpSeN:t cCMi p sS]), http/1.1 edge.infra.ciab.test (ApacheTrafficServer/7.1.4 [uScMsSfWpSeN:t cCMi pSs ])
< Connection: keep-alive
<
{ [data not shown]
* Connection #1 to host edge.DeMo1.mycdn.ciab.test left intact

Example Request with a fully lowercased URL:
$ curl -o /dev/null -vs https://edge.demo1.mycdn.ciab.test/

* About to connect() to edge.demo1.mycdn.ciab.test port 443 (#0)
*   Trying <IP_ADDRESS>...
* Connected to edge.demo1.mycdn.ciab.test (<IP_ADDRESS>) port 443 (#0)
* TCP_NODELAY set
* ALPN, offering http/1.1
* Cipher selection: ALL:!EXPORT:!EXPORT40:!EXPORT56:!aNULL:!LOW:!RC4:@STRENGTH
* successfully set certificate verify locations:
*   CAfile: /usr/local/etc/openssl/cert.pem
  CApath: /usr/local/etc/openssl/certs
* TLSv1.2 (OUT), TLS header, Certificate Status (22):
* TLSv1.2 (OUT), TLS handshake, Client hello (1):
* TLSv1.2 (IN), TLS header, Unknown (21):
* TLSv1.2 (IN), TLS alert, handshake failure (552):
* error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure
* Closing connection 0

This issue has been fixed by PR(s): #3380, #3382, and #3331.
Issue will be closed.


Load balancing Kibana requests [Question]

The suggested method for load balancing of Kibana requests (from the documentation):

If you have multiple nodes in your Elasticsearch cluster, the easiest way to distribute Kibana requests across the nodes is to run an Elasticsearch client node on the same machine as Kibana.

Is this method supported in Elassandra, i.e. how can I run an Elassandra/Elasticsearch client node in my Elassandra cluster?
If not supported, is there an intention to support this in future, or any other suggested way of accomplishing the same result?

Sorry for this late response. You could likely use HAProxy on your kibana server to achieve http connexion failover to elassandra nodes.

Good idea. Thanks for your answer and best of luck with this interesting project.


Corrected label for RHCLOUD Jira

Corrected the label that users need to add to the RHCLOUD Jira (platform-experience-services ).
Plus a few tiny things, like the latest openshift release version on a link, some punctuation.
@Hyperkid123 I think this is all good to merge, but please let me know if you want me to create a Jira ticket for you too :)

/ok-to-test


Correct .showHelp docs

Corrects the .showHelp docs by changing the parameter from a function to a console function, to match what the code is doing:
https://github.com/bcoe/yargs/blob/master/lib/usage.js#L209

thanks for the contribution @robatron, you rock.


Adding new/updated lab for cgroups

Signed-off-by: Nigel<EMAIL_ADDRESS>

Thank you so much for updating this lab - LGTM!
I'll follow up with the action items you've raised and also with any Forum/Video resources 👍


Add an ADR for server-side rendering

Type

documentation

Related JIRA Issue(s)
https://www.ebi.ac.uk/panda/jira/browse/ENSWBSITES-763
Description
An Architecture Decision Record capturing the state of our conversation about server-side rendering. It seems that we are generally in agreement that server-side rendering is a desirable goal for the new Ensembl site, but are still unsure which solution will suit us the best.

Is it possible to estimate the amount of work involved with implementing a custom rendering server versus a Next.js-based implementation of server-side rendering? How much does the opinionated nature of Next.js tip the balance towards a custom renderer?
Intuitively, it feels like implementing our own renderer should be more difficult and more time-consuming than using a third-party tool. It will also add presumably a significant volume of code that will require on-going maintenance. However, if we could find, six months after implementing Next.js, that we can't do something essential, maybe that extra workload is justified.

Is it possible to estimate the amount of work involved with implementing a custom rendering server versus a Next.js-based implementation of server-side rendering? How much does the opinionated nature of Next.js tip the balance towards a custom renderer?

For each of those, there is its own set of known unknowns and unknown unknowns.
For Next.js, the unknowns are clustered around the question, what happens when you want your rendering server to do something else as well. E.g. proxy a subset of requests to other servers, add custom headers to response, run custom logger at every request. There's a section on using Next.js with a custom server in the docs, but I have never before tried that. In any case, I expect the experiments around Next to only take several days.
For a custom implementation, the unknowns are around the best ways to code split (React's stock solution doesn't work server-side at the moment, but a blessed alternative exists), to fetch data server-side, and achieving good developer experience with proper code updates without having to rebuild the project. It should probably take around a week to get a clearer picture.


Cannot read mouse and keyboard in the same program

Likely because I'm calling them from two different threads, there are workarounds, but I don't expect to use the mouse.

Didn't fix it but we're doing all the keyboarding through Tk anyway.


feat: add side panel

Please check if the PR fulfills these requirements

[x] The commit(s) message(s) follows our guidelines
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
What is the new behavior?
New component that displays a menu within a panel
Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Review changes pushed. Please review/approve :)


🐛 Bug Report: ensure CRDs are installed first

📜 Description
I am using an organization's windows machine and there were a few issues with permissions of the service account token that were preventing the discovery of project's K8 cluster in Backstage I/O.
So I migrated everything to WSL-2 running Ubuntu 24.04.01 instance.
Now the issue/error I am facing is two folds.
👍 Expected behavior
I was expecting it to be a smooth sail since I have shifted everything to Linux but unfortunately that is not the case.
👎 Actual Behavior with Screenshots
First issue is: (This error was not present before installing WSL-2)
catalog-info.yaml
apiVersion: backstage.io/v1alpha1
kind: Component
metadata:
  namespace: gmx-app
  annotations:
    github.com/project-slug: mohammad-ali_uics/gmx-app
    backstage.io/kubernetes-id: gmx-app-service
  name: gmx-app
  description: GMX App for user management
  tags:
    - app
    - gmx
  relations:
    - type: ownedBy
      targetRef: user:default/mohammad-ali_uics
      target:
        kind: user
        namespace: gmx-app
        name: mohammad-ali_uics
spec:
  type: website
  owner: user:mohammad-ali_uics
  lifecycle: experimental

kubectl apply -f catalog-info.yaml
error: resource mapping not found for name: "gmx-app" namespace: "gmx-app" from "catalog-info.yaml": no matches for kind "Component" in version "backstage.io/v1alpha1"
ensure CRDs are installed first

I believe that it is not able to read this line apiVersion: backstage.io/v1alpha1
Second issue:
deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: gmx-app
  namespace: gmx-service-account
  labels:
    app: gmx-app
    backstage.io/kubernetes-id: gmx-app
spec:
  replicas: 3
  selector:
    matchLabels:
      app: gmx-app
  template:
    metadata:
      labels:
        app: gmx-app
    spec:
      serviceAccountName: gmx-service-account
      containers:
        - name: gmx-app
          image: gmx-app
          imagePullPolicy: IfNotPresent
          ports:
            - containerPort: 3000
      imagePullSecrets:
        - name: regcred

The pods are not working, this issue was persistent before as well.
Behavior:
kubectl get pods -n gmx-service-account
NAME                       READY   STATUS              RESTARTS   AGE
gmx-app-6444dd44f6-6jrkp   0/1     ErrImageNeverPull   0          39m
gmx-app-6444dd44f6-cgrc6   0/1     ErrImageNeverPull   0          39m
gmx-app-6444dd44f6-qkq22   0/1     ErrImageNeverPull   0          39m
gmx-app-644dd949bc-5j7nf   0/1     ImagePullBackOff    0          46s

👟 Reproduction steps
my steps

minikube status
minikube start
kubectl get namespace
kubectl create namespace gmx-service-app
kubectl apply -f service.yaml
kubectl apply -f service-account.yaml
kubectl get serviceaccount --all-namespaces
kubectl apply -f role.yaml
kubectl apply -f role-binding.yaml
kubectl apply -f cluster-role.yaml
kubectl apply -f cluster-role-binding.yaml
kubectl apply -f deployment.yaml

When I do this and run the command
minikube service gmx-service-account -n gmx-service-account it works but the webpage is totally blank and says
This page isn’t working right now
<IP_ADDRESS> didn’t send any data.

📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Gonna close this one as inactionable

Bit of extra point of clarifications: it is not the intention and the Backstage project does not provide any way to load catalog-info.yaml files into kubernetes. The catalog info files use the same base schema as kubernetes manifests, but they are intended to be consumed through Backstage's software catalog, see https://backstage.io/docs/features/software-catalog/life-of-an-entity

I am aware of the fact that Backstage only uses catalog info to fetch information about a component or a service. The issue arises when I run the command kubectl apply -f catalog-info.yaml while I am in my repo which I want to be discovered by the backstage app


brk syscall implementation.

Hi, I have a question regarding this piece of code:
https://github.com/aemmitt-ns/radius/blob/1a6d76a9b04eb3f6913404e2c28ca7c39f87c813/radius/src/sims/syscall.rs#L235-L244
Is this part really implementing the brk syscall?
Isn't it instead implementing the glibc wrapper function?
Please have a look at the section "C library/kernel differences" in the man page:
https://man7.org/linux/man-pages/man2/brk.2.html

However, the actual Linux system call returns the new
program break on success.  On failure, the system call returns
the current break.  The glibc wrapper function does some work
(i.e., checks whether the new break is less than addr) to provide
the 0 and -1 return values described above.

(I'm currently working on some kind of user-space emulator and also have to implement the brk and sbrk syscalls and I remembered your project from Twitter so I thought I'd look at how you implemented it :))

Yes thank you it looks like you are right! This should be fixed with https://github.com/aemmitt-ns/radius/commit/4e5097cd1841c2657f67b48d56be3827c094cc5e but i definitely need to test syscall stuff more. Thanks for pointing this out


modify engine.py for formatting

fix #2666

sorry, I think I wrongly committed changes to another branch and create a new pull request #2764. I'll close this PR and please approve the new one intead. thanks


signalr/bearer-auth New topic on bearer authentication

As per this comment by @davidfowl, we could use a doc on bearer authentication specifically with server side examples of consuming the token using the JWT auth handler.

This issue is old and might pre-date a lot of the doc work that's been done. Bearer auth is covered in https://docs.microsoft.com/aspnet/core/signalr/authn-and-authz?view=aspnetcore-3.1#bearer-token-authentication. If something is missing feel free to file a new issue.


Add support for PostLinks and Comments

The current scripts don't support the importing of PostLinks nor Comments.

Closed by pull request #2


Show social proofs in search results

r? @chrisnojima @MarcoPolo

I haven't heard anything back in a few days, so I'm mergin'.


Event Managers can view list of Managed Events

Breakout from #1803
Update event lister if possible or add a new lister page where an event manager can see links to all events(s) they have access to manage.
Clicking the link should take them to the admin event page.

A couple of options for this:

We allow event managers to view the campaign details page for the campaigns owning events they are allowed to manage. This would require us to hide buttons on those pages and possibly limit the information they can see.

We develop a new lister specifically for event managers. This would list all events they are able to manage, which in most cases will be limited to one event.

Now that I think this through I believe option 2 is probably easier since it prevents any risk of event managers having access to campaign data they do not need and have not been granted access to.
@tonysurma @schuback Unless you have any objections I'm going to start on the code for option 2.

thanks @stevejgordon lets go with option 2 and I will connect with emily to see if there is a variation on that from Jim/Pascal/Others

cc @EmilyLuijbregts


Fix bytes encoding issue when ignoring GOWORK

What does this PR do?

Fix a bytes encoding error when disabling GOWORK in invoke tasks.
The function currently always prints the warning because res.stdout is a bytes and is not stripped...
Introduced by https://github.com/DataDog/datadog-agent/pull/22334
Motivation

We don't want to print a warning or export GOWORK=off if there is no workspace anyway.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

/merge


Google Forms Data Extraction

Purpose/implementation Section
What changes are being implemented in this Pull Request?
This is a set of functions that retrieve and get data from google forms.
Usage
Here's how these work. You can pass in a single URL or form id to a form and retrieve info:
authorize("google")
form_info <- get_google_form("https://docs.google.com/forms/d/1Z-lMMdUyubUqIvaSXeDu1tlB7_QpNTzOk3kfzjP2Uuo/edit")

### OR You can give it a direct form id
form_info <- get_google_form("1Z-lMMdUyubUqIvaSXeDu1tlB7_QpNTzOk3kfzjP2Uuo")

You can extract multiple forms using get_multiple_forms() function
authorize("google")
form_list <- googledrive::drive_find(shared_drive = googledrive::as_id("0AJb5Zemj0AAkUk9PVA"), type = "form")
 multiple_forms <- get_multiple_forms(form_ids = form_list$id)

get_google_form() may need to be examined as form_info$title contains NULL.
get_multiple_forms() looks good to me.

This is a more universal problem that I want to address for all these functions #32
I'm thinking the solution should be 1) Make a specialized data class type to handle these data as well as a 2) set of handler functions. But this will be a much bigger set of work than will be done in this PR.


Add simple implementation for fetching Packages from the Organisation

Because the endpoint 'CreatePackage' does not yet take into account what user or organisation the package belongs to; It's more difficult to test this feature.

Because the endpoint 'CreatePackage' does not yet take into account what user or organisation the package belongs to; It's more difficult to test this feature.

Because it cant be tested right now. This issue will be a draft until #7 is made


Browser Devlopment key ?

@Laimiux
hi , m new to development and my need to embed some YouTube videos in my application, i read your library and i think it's a great idea. but the init function asks for a browser_development_key , what's that about ? how do we get that ?

 In that project, enable Youtube Data API. Then, go to "APIs & Auth -> Credentials",  create new api key, select browser key and in the form you can leave the http refferers empty.

Got it Sir !
Cheers,
Vinit
On Oct 24, 2015 5:08 PM, "Laimonas Turauskas"<EMAIL_ADDRESS>wrote:

Go to https://console.developers.google.com/ and create a project. In
that project, enable Youtube Data API. Then, go to "APIs & Auth ->
Credentials", create new api key, select browser key and in the form you
can leave the http refferers empty.
—
Reply to this email directly or view it on GitHub
https://github.com/Laimiux/android-youtube-listview/issues/1#issuecomment-150793949
.

Hi! I can not get the browser Okay. In https://console.developers.google.com/ everything changed. He is now in the mail come? Help me please.


operator project-quay (3.10.0-nightly.20230824)

Add 3.10.0-nightly.20230824 to community operators

/merge possible

/merge possible


Fixes error message passthrough

Fixes #2

Raises error with correct, passed message

Thanks. I released the update as 1.0.0 (I've been trying to move from 0.x based versioning to x.0, so whenever I release something I change the scheme if the module used 0.x)

Awesome, thank you :)


Testing Stripe with Jest-Puppeter

First of all thanks for this package. Its saving me a lot of fo time and I'm looking forward to the next implementations (such as an Element that I can customize rather than being stuck with the predefined ones). But that is a problem for another day.
Right now I am trying to test Stripe with jest-puppeter and I can't seem to find a way to fill the form.
Simple methods are provided to accomplish those tasks such as await expect(page).toFillForm(${selector}, { key: value } where the key is the input name and the value is what I wish to fill the form with.
This normally works on other forms and you have provided a name for the inputs as well, yet I cannot find a way to fill the CardElement.
Does anyone have experience or is there a way to accomplish this??
THanks a lot.

Hi @santaclauze I don't have any direct experience with writing the tests you're describing, but there's another issue that has an example of a puppeteer test. Perhaps that will help you! https://github.com/stripe/react-stripe-elements/issues/137

Closing this as this project has migrated to React Stripe.js. If you believe this is still important, please re-open it there.


Minor corrections

Correction to outdated comments describing function usage.
Corrections to English grammar.
Opinionated changes to sentence structures.
This PR only makes changes to line comments.
No changes to code or doc comments have been made.

@CABrinegar Thank you! Many of developers(including me) are not native speakers, so help in English grammar corrections is very welcome, and for the doc comments too! :)


Issue with Signature Verification When 'Transforms' Tag is Absent in 'Reference' Element

Good Day
I have this signture on a XML Document that it content has been deliberately modified to fail the verification
<Signature xmlns="http://www.w3.org/2000/09/xmldsig#">
	<SignedInfo>
		<CanonicalizationMethod Algorithm="http://www.w3.org/TR/2001/REC-xml-c14n-20010315"/>
		<SignatureMethod Algorithm="http://www.w3.org/2000/09/xmldsig#rsa-sha1"/>
		<Reference URI="#T33F19384429">
			<DigestMethod Algorithm="http://www.w3.org/2000/09/xmldsig#sha1"/>
			<DigestValue>uU+OaebbefWVOIXXP4q8FV7F9JQ=</DigestValue>
		</Reference>
	</SignedInfo>
	<SignatureValue>q7ENUbjtG9WauSnjDM7jwDePwnQVVeQDrsaL6ZitDLfJ+dtPi833JubqjD4on8TU+xSDAjVHYV9s
EByiiOvWLw981QICjG3g+N8AA2xpjx8nZhbZDIXvkFKmFsaII651Te/Pe+qYH6ONUYWs6Hf9cCnx
933q9A/YanGR4bRfWhyxMXyRKe+rysDCOZA/7gx2jclnaDkGVcC4Cfe28cxaHVw9xDMMZk0MEroQ
YvkJ1BttKHx8BfjIaT17aands2rW6meUH5etkMEoET19QCZO4Ez/7PI7Xr379azgrqX9QuBr+Lg9
nIGjS0Iz7E7AUgaHz6fErYug7P4/PEcNN8uAZQ==</SignatureValue>
.....
</Signature>

As the Reference tag does not contain a Transforms tag, when I apply the checkSignature function to that document, the result is true. However, this is due to the function performing verification solely on the signature's digest, without recalculating the digest itself.
To reproduce the problem, take any verified document that doesn't has the Transforms tag, and then make any modification to the content that would cause the digest calculation to fail to match the original. You'll observe that the signature is still verified.
Let me know if I need to provide more details

Is this program logic correct?

Some preliminary testing I've done indicates that it isn't correct, but I don't have a test case for it yet.
As for the other item reported here, I've put up #379 to address it. Thanks for your thorough analysis @srd90.

@srd90, thanks for your superb work on this issue.
The issue with the skip of digest calculation seems to be rectified. However, at first glance, I'm still experiencing some problems with the canonicalization. So, allow me to verify a little more thoroughly with my testing documents


fix: make delete standout

What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[x] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Description
Makes the Delete in the delete popup stand out.
Related Tickets & Documents
Fixes #1020
Mobile & Desktop Screenshots/Recordings

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

LGTM


Your app isn't using AndroidX.     To avoid potential build failures, you can quickly migrate your app by following the steps on https://goo.gl/CP92wY.

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

Hi @Melisa1-gif
Follow the steps on https://goo.gl/CP92wY (or create an app anew, it will use AndroidX by default)
Closing, as this isn't an issue with Flutter,
if you disagree please write in the comments and I will reopen it


Questions for Update Management on Azure

for the packages listed on update management pages, how update management classify package as "security" and "critical" on CentOS or Ubunt ? based on what database?
when new CVE was issued, how fast/quick can we see on update management page?
when will the post-script be executed while VM need to reboot after update?

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

ID: db47901d-1664-058d-9407-47e932dc9661
Version Independent ID: e90ec9ee-e7da-4f19-248f-4c825aaa8b9f
Content: Update Management solution in Azure
Content Source: articles/automation/automation-update-management.md
Service: automation
GitHub Login: @georgewallace
Microsoft Alias: gwallace

@waynemsft Thanks for your question. Can you please share the url, if there is a particular Azure doc you have been following?

Hi @Karishma-Tiwari-MSFT ,
Sure. I came from this URL:

https://docs.microsoft.com/en-us/azure/automation/automation-update-management#data-collection

@georgewallace any insights on this?

@waynemsft To Answer your questions. For classification of patches we enrich the patch data on our side to be able to provide the additional information such as classification that is not available normally, on an OS like CentOS.  Each operating system has it's own enrichment data provided by us.
When a new CVE is issued, for Linux machines it will show up within 2-3 hours of release.  For Windows it can take 12-15 hours before it shows up in Update Management.
If a reboot is scheduled and a post-script is defined.  The post-script will run after the reboot.
I will be sure to add this additional information in the docs to help clear this up for others as well.
Hopefully this answers your questions.

Hi @georgewallace,
Yes. Those information are good and valuable.
Thank you for the information.
Cheers

@waynemsft I added the information to the docs. The updates based on your feedback will be live today.. Thanks for reaching out and making our docs that much better.


Using nz-checkbox in reactive form - patchValue with { emitEvent: false,  onlySelf: true } (ngModelChange() still firing)

Reproduction link
https://fxrq8g--run.stackblitz.io
Steps to reproduce
Create reactive form with nz-checkbox. Try to patchValue with { emitEvent: false,  onlySelf: true }.
What is expected?
Using nz-checkbox - ngModelChange() should not fire when using patchValue with { emitEvent: false,  onlySelf: true }.
What is actually happening?
ngModelChange() is firing, not taking patchValue options into account.

Environment
Info

ng-zorro-antd
15.1.0

Browser
Chrome

I used undocumented nzCheckedChange event listener, and it works fine.


feat: disable blob restore policy by default for standard Blob storage

azurerm_storage_account.this.blob_properties[0].restore_policy[0] should not be created when account_kind is BlobStorage.

fixed by #153


Intl polyfill

Also a fix for smn text because rolling two disparate things together is totally my style.
Should resolve #113

Pushed up a fix, it should now load our messages.json and the intl polyfill (if required) at the same time.
Unfortunately, I can't include the polyfill locale data in with that grouping, as it makes calls to the main polyfill at import time.


Create interhigh.txt

Online education school based in the UK. Offers courses and learning curricula for different years and key stages (Up to sixth form, for 16-18 year olds), including several IT-related courses such as Computer Science (2 years GCSE, https://interhigh.co.uk/key-stage-4-subjects/computer-science/ ) and Computer Game Development (2 years, https://interhigh.co.uk/key-stage-4-subjects/computer-game-development/ )
School can be contacted through the following link if needed:
https://interhigh.co.uk/contact/
I am a student at InterHigh and I am willing to prove that the school provides an email on the same domain name for every student if needed. Verification by email is OK.

@LvlAndFarm Pull request merged. Thank you!


fix: static page generation issues with language

LIIKUNTA-372.
Currently, the Sports app did not fail, but the Hobbies and the Events did.
This was because there was a different kind of locale handling:
The locale which was destructed from the context was set to null.
Now there is a fallback language set when the context locale is null.
The issue was easier to find and understand,
when I understood where the error was printed.
The error was our own message from the Apollo-Client's ErrorLink.
If the operation-variable from the onError-handler was printed in the console,
the actual error place was shown!

How the issue was found:

Set a new console error printer in the eventsFederationApolloClient.ts, because the onError -handler was printing the earlier messages also.
Run yarn build

Result: The real cause for error was shown!
The errorLink in the eventsFederationApolloClient.ts:
  const errorLink = onError(
    ({ graphQLErrors, networkError, operation, forward, response }) => {
      graphqlClientLogger.error(
        JSON.stringify({ operation, forward, response })
      );
      if (graphQLErrors) {
        graphQLErrors.forEach(({ message, locations, path }) => {
          const errorMessage = `[GraphQL error]: Message: ${message}, Location: ${locations}, Path: ${path}`;
          graphqlClientLogger.error(errorMessage);
          Sentry.captureMessage(errorMessage);
        });
      }
      if (networkError) {
        graphqlClientLogger.error(networkError);
        Sentry.captureMessage('Network error');
      }
    }
  );

Events-Helsinki branch is deployed to platta: https://tapahtumat-pr236.dev.hel.ninja :rocket::rocket::rocket:

Hobbies-Helsinki branch is deployed to platta: https://harrastukset-pr236.dev.hel.ninja :rocket::rocket::rocket:

Hobbies-Helsinki branch is deployed to platta: https://harrastukset-pr236.dev.hel.ninja :rocket::rocket::rocket:

Events-Helsinki branch is deployed to platta: https://tapahtumat-pr236.dev.hel.ninja :rocket::rocket::rocket:

Sports-Helsinki branch is deployed to platta: https://liikunta-pr236.dev.hel.ninja :rocket::rocket::rocket:

Events-Helsinki branch is deployed to platta: https://tapahtumat-pr236.dev.hel.ninja :rocket::rocket::rocket:

Hobbies-Helsinki branch is deployed to platta: https://harrastukset-pr236.dev.hel.ninja :rocket::rocket::rocket:

Sports-Helsinki branch is deployed to platta: https://liikunta-pr236.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://tapahtumat-pr236.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://harrastukset-pr236.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://liikunta-pr236.dev.hel.ninja :laughing::tada::tada::tada:

Hobbies-Helsinki branch is deployed to platta: https://harrastukset-pr236.dev.hel.ninja :rocket::rocket::rocket:

Events-Helsinki branch is deployed to platta: https://tapahtumat-pr236.dev.hel.ninja :rocket::rocket::rocket:

Sports-Helsinki branch is deployed to platta: https://liikunta-pr236.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://harrastukset-pr236.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://liikunta-pr236.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://tapahtumat-pr236.dev.hel.ninja :laughing::tada::tada::tada:


Bundle CartoCSS (Builder) fonts with SDK

Currently SDK bundles fonts used in basemaps, but not the ones what can be used in CARTO Builder visualizations. Therefore for CARTO visualizations app has to bundle own additional fonts.

Currently only bundled styles use bundled fonts. Change SDK so custom cartocss decoder would try to find fonts also from SDK bundle;
Review selection of fonts - maybe basemaps and CARTO fontset can be unified more
add common CARTO cartocss selectable fonts to SDK package. Adding all would be probably way too big in terms of bytes. E.g. first default "Dejavu Sans Book"

Bundling all the fonts used by Builder would make the SDK binary probably too large. Besides, only limited users of the SDK would benefit from this. A better idea is to make it easy to add custom fallback fonts to VectorTileDecoder, for example.

Closing this, as SDK 4.2 has option to specify fallback fonts.


Add an autocomplete form input

What are you trying to accomplish?

This PR adds an autocomplete form input to the forms framework.
Screenshots

Input with caption and validation message
With dropdown list

Integration

No changes necessary in production.
Risk Assessment

[x] Low risk the change is small, highly observable, and easily rolled back.
[ ] Medium risk changes that are isolated, reduced in scope or could impact few users. The change will not impact library availability.
[ ] High risk changes are those that could impact customers and SLOs, low or no test coverage, low observability, or slow to rollback.

What approach did you choose and why?

I chose to re-use the existing Primer::Beta::AutoComplete component.
Accessibility

No new axe scan violation - This change does not introduce any new axe scan violations.

Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[x] Added/updated previews (Lookbook)
[x] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

Depends on: https://github.com/github/auto-complete-element/pull/116

Now depends on: https://github.com/github/auto-complete-element/pull/118


Fix latent casting bug in BKDWriter

This commit fixes a latent casting bug where int multiplication could roll-over to the negatives.
new byte[Math.toIntExact(numSplits * config.bytesPerDim)];
toIntExact does nothing from what I understand. Since both numSplits and config.bytesPerDim are already int values, their multiplication will be an int multiplication that will return an int. Consequently, Math.toIntExact does no work here and we can still get negative numbers.
I will venture to assume that the original author actually wanted Math.multiplyExact(numSplits, config.bytesPerDim) which will multiply the two values and throw if there is overflow.

@iverase you might be interested in this.

Actually, I think there are more occurrences of this multiplication without check, could we add it? for example: https://github.com/apache/lucene/blob/3210a42f0958e395930d2259e155a7149fb475eb/lucene/core/src/java/org/apache/lucene/util/bkd/BKDWriter.java#L518


Add Books

[x] need to refactor code in search bar (remove  booleans shown at bottom or result)   NICOLE/COLIN?

[x] put the book information on the side of the book cover image + add description saying ‘book added’ instead of viewing 1 result thing  NICOLE

finished!


":tmap  N", the mapping doesn't work in terminal buffer.

Steps to reproduce

start gvim.exe
:tmap <ScrollWheelUp> <C-W>N
:term
mouse wheel in terminal buffer, nothing happens.

Expected behaviour
It should go to normal mode.
Version of Vim
8.2
Environment
OS: windows 11
gvim 8.2
Logs and stack traces
No response

I see that it is not supported by GUI vim on windows.


Read() doesn't remove whitespaces in variable (parameter)

When reading variables from a SQL-Server table they contain some whitespaces, as shown here.

Maybe it is better to Trim() excess characters or use an Regex as described here.
Using Trim() produces a readable address and doesn't lead to WrongVarFormat error.

Best regards.
Edit: Some more investigation on the whitespaces: It is a Unicode non-breaking space (bytes C2A0).

My opinion: this doesn't belong in a communications library. I don't know how you get your data in the database, but logically I'd say you forgot about input sanitizing, you should've made sure the data is valid before storing it in the database.
My proposal for this issue is to close as invalid.

You are right, this should not be part of this library. Since it is it should work as expected.
Closing this as it is not a "normal" whitespace.


fix(ui): show query result along with error message if both exist

Issue

when running a sql query, if the api result contains both exceptions as well as resultTable then ideally UI should show both
instead UI is showing only exceptions
Eg- when API response contains both result and exception then UI only shows error alert

curl -H "Content-Type: application/json" -X POST \
        -d '{"sql":"select count(*)  from airlineStats "}' \
         http://localhost:8000/query/sql

{
  "resultTable": {
    "dataSchema": {
      "columnNames": [
        "count(*)"
      ],
      "columnDataTypes": [
        "LONG"
      ]
    },
    "rows": [
      [
        9343
      ]
    ]
  },
  "exceptions": [
    {
      "errorCode": 305,
      "message": "null:\n1 segments unavailable: [airlineStats_OFFLINE_16072_16072_0]"
    }
  ],
  "numServersQueried": 1,
  "numServersResponded": 1,
  "numSegmentsQueried": 30,
  "numSegmentsProcessed": 30,
  "numSegmentsMatched": 30,
  "numConsumingSegmentsQueried": 0,
  "numConsumingSegmentsProcessed": 0,
  "numConsumingSegmentsMatched": 0,
  "numDocsScanned": 9343,
  "numEntriesScannedInFilter": 0,
  "numEntriesScannedPostFilter": 0,
  "numGroupsLimitReached": false,
  "totalDocs": 9343,
  "timeUsedMs": 7,
  "offlineThreadCpuTimeNs": 0,
  "realtimeThreadCpuTimeNs": 0,
  "offlineSystemActivitiesCpuTimeNs": 0,
  "realtimeSystemActivitiesCpuTimeNs": 0,
  "offlineResponseSerializationCpuTimeNs": 0,
  "realtimeResponseSerializationCpuTimeNs": 0,
  "offlineTotalCpuTimeNs": 0,
  "realtimeTotalCpuTimeNs": 0,
  "segmentStatistics": [],
  "traceInfo": {},
  "numRowsResultSet": 1,
  "minConsumingFreshnessTimeMs": 0,
  "numSegmentsPrunedByBroker": 0,
  "numSegmentsPrunedByServer": 0,
  "numSegmentsPrunedInvalid": 0,
  "numSegmentsPrunedByLimit": 0,
  "numSegmentsPrunedByValue": 0,
  "explainPlanNumEmptyFilterSegments": 0,
  "explainPlanNumMatchAllFilterSegments": 0
}

Solution
show both query result as well as error message if both exist

Codecov Report

Merging #10071 (c8a2626) into master (97678f8) will increase coverage by 0.09%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #10071      +/-   ##
============================================
+ Coverage     70.33%   70.42%   +0.09%
+ Complexity     5694     5693       -1
============================================
  Files          1996     1996
  Lines        107805   107805
  Branches      16388    16388
============================================
+ Hits          75825    75927     +102
+ Misses        26672    26568     -104
- Partials       5308     5310       +2

Flag
Coverage Δ

integration1
25.13% <ø> (+0.03%)
:arrow_up:

integration2
24.57% <ø> (+0.12%)
:arrow_up:

unittests1
67.90% <ø> (+<0.01%)
:arrow_up:

unittests2
13.60% <ø> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...tream/kafka20/server/KafkaDataServerStartable.java
72.91% <0.00%> (-6.25%)
:arrow_down:

...l/segment/index/readers/OnHeapFloatDictionary.java
86.36% <0.00%> (-4.55%)
:arrow_down:

...gregation/function/StUnionAggregationFunction.java
73.52% <0.00%> (-2.95%)
:arrow_down:

...inot/core/operator/filter/FilterOperatorUtils.java
87.50% <0.00%> (-2.50%)
:arrow_down:

...roller/helix/core/relocation/SegmentRelocator.java
63.57% <0.00%> (-1.43%)
:arrow_down:

...ery/optimizer/filter/MergeEqInFilterOptimizer.java
92.59% <0.00%> (-1.24%)
:arrow_down:

...x/core/realtime/MissingConsumingSegmentFinder.java
86.02% <0.00%> (-1.08%)
:arrow_down:

...re/accounting/PerQueryCPUMemAccountantFactory.java
69.15% <0.00%> (-1.02%)
:arrow_down:

...core/startree/operator/StarTreeFilterOperator.java
87.50% <0.00%> (-0.66%)
:arrow_down:

...core/query/pruner/SelectionQuerySegmentPruner.java
85.88% <0.00%> (ø)

... and 26 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jayeshchoudhary what happens if there are multiple rows to be displayed in the result? 1) Is that a valid use case? 2. If yes, is there a chance that the exception gets pushed down and not visible unless the page is scrolled?
In my opinion, an exception should be at the top, above the query response stats.

Ok, so code wise exception is at the top (first thing handled) except that the logic seems to be display nothing else if there is an exception. Let's keep that structure and also reflect it in the UX.

comments addressed


script to update bcfishpass layers in background_layers.gpkg

actually https://github.com/NewGraphEnvironment/dff-2022/commit/79598bbf26df5d04defff6f5dfdd4bd16d1226a8 closes this piece of the puzzle
closing https://github.com/NewGraphEnvironment/dff-2022/issues/124 should do the rest

Looks like we have a ton layers that are missing info related to this project tracking https://github.com/NewGraphEnvironment/fish_passage_fraser_2023_reporting/issues/1
Not sure why but will try rebuilding the background_layers.gpkg with latest rfp_* scripts.  Guessing this is an issue with the scripts in https://github.com/NewGraphEnvironment/dff-2022/tree/master/scripts/qgis at the time of update mergin v29 .  Looks like the aws stored layers were updated but all the bcdata layers are empty.  Perhaps the gpkg was essentially erased with a flag such as overwrite in the ogr2ogr calls when update_aws script was run and the update_bcdata was not.
Another anomoly occurs in the v35 version of the Fraser project and that is that the observespp_* layers do not come through to QGIS.  Its extra weird because the species values are present in the gpkg but for some reason they are not visible in QGIS (even though othe colon seperaed arrayss such as  barriers_anthropogenic_* do come through. so odd.

update went well:

anomoly in the the Fraser project that  the observespp_* layers do not come through to QGIS is related to .qgs styles somehow. Reimporting the crossings layer from the geopackage eliminates the disappearing values...


Fix deployments

We're having issues deploying due to a change in the way ARM templates are handled. For some reason, running string against a created array now appears to flatten out the array into a comma seperated list, so we need to change how we build the SERVICE_ENTITY_IDS JSON array. This builds up the array using a combination of a [, an (escaped) single quote '', the VSP entity ID and a ]. The means if the value of VSP.VERIFY_ENTITY_ID was http://example.com, the resulting JSON string would be ['http://example.com'].
I've also updated the command we use to create deployments, as we were getting a deprecation warning.

Yep, works like a charm 👍


feat: Github Actions to publish Helm Charts.

This uses Helm Chart Releaser Action to publish the helm chart.
This requires the following:

GitHub pages set to gh-pages branch.
Token called GHP_TOKEN with at least Contents permissions. (feel free to use another one here as needed)

This has been tested and working in my fork.
I see the v0.0.10 chart released but didn't find any GHA workflows in any branch. So if this is redundant, please feel free to close this PR.

Thank you @kaiwalyajoshi


PDF files in repository

Case

HELP QUESTION

Issue
Is it necessary to have the default exported pdf files in the repository or can they be gitignored?

We need to keep the pdf files, that there is no need to checkout the repo to preview the pdfs.


Add Microsoft login but delete mojang login

I hope that the multimc starter can add a Microsoft account without deleting the mojang account, so that players with two different login modes can enjoy the charm and function of this great starter. Mojang account will be migrated to Microsoft sooner or later. First, design a Microsoft account to log in, and then wait until the day of migration, won't it save a bit of work? I'm not a player who is slandering mojang's account. I just hope that the production team of the multimc starter can see that there is such a big disadvantage in the multimc starter. I hope that the multimc starter will be successful

I'm not sure what this is, and have no intention of looking at the contents of the PR.

Plus its dumb. People are still gonna use mojang accounts until thry cant


The notebook widget can't add widget into it's frame

The notebook's frame is also include in the titlebar...

Will fix it , it is really a bug.
Emm, maybe i will rebuild this widget...?
BTW, i also need rebuild the menu


BUG: Convert pyperclip & webbrowser errors to warnings

[x] Closes #25
[x] Tests added

Thanks @justingruca :+1:


ImportError: cannot import name 'Self' from 'typing'

Whenever I try to run a simple script, I encounter this problem
❯ python3.10 /Users/Gokul/Desktop/ssl_auto_update.py
Traceback (most recent call last):
  File "/Users/Gokul/Desktop/ssl_auto_update.py", line 1, in <module>
    from sitesniffer import SiteSniffe
  File "/opt/homebrew/lib/python3.10/site-packages/sitesniffer/__init__.py", line 18, in <module>
    from ._sitesniffer import SiteSniffer
  File "/opt/homebrew/lib/python3.10/site-packages/sitesniffer/_sitesniffer.py", line 10, in <module>
    from typing import Any, Literal, Optional, Self
ImportError: cannot import name 'Self' from 'typing'<EMAIL_ADDRESS>

I did try pip3 install typing-extensions --upgrade, but still facing the issue

You need to install Python 3.11 or higher. Self is a new typing feature.

The issue has been resolved after adding the python_requires option in the setup.py file. Now, it requires a version of Python 3.11 or higher to run the package.


May be add method getTree()?

Get the complete tree of descendants - a common task when working with hierarchical structures. Is it possible to add method getTree($depth = null) that works same as getDescendants() but returns tree instead plain list? I mean this will be very useful, but this method returns not ActiveQuery as other methods.
What do you think about that?

getDescendants() work as relation. If you need plain list of descendants (not ActiveQuery), simple use getter from model:
$descendants = $model->descendants;

Sorry, I did not understand right away. Looks good.

Good. I'll add this.

I thought about this functionality, and can not think of the normal structure to return the result.
I believe that the structure in the form of:
$result = [
    [
        'item' => $model,
        children' => [/* ... recoursive ... */],
    ],
    [
        'item' => $model,
        children' => [/* ... recoursive ... */],
    ],
    /* ... more ... */
];
$title = $result[1]['children'][2]['item']->title;

bad idea, it's better this way to access:
$title = $model->children[1]->children[2]->title;

We can set children for node by \yii\db\BaseActiveRecord::populateRelation(). And after that we can get childrend of node by using $parent->children.

Good idea!

Should this method returns self node as tree's root? Or he must returns list of next level children?  What code is prefered?
$node = Model::find()->byId(1)->one();
$tree = $node->getTree(); // $tree is same as $node but with children
$children = $tree->children;

or this:
$node = Model::find()->byId(1)->one();
$children = $node->getTree();

I think that it should return itself.


Support for Elasticsearch 7.17.9

Hi,
Elasticsearch has released a few new versions with security updates, for which there is no compatible prometheus-exporter version. Could a newer version for 7.17.9 please be released?
Thank you

@vvanholl or any other collaborators - I created a PR for 7.17.8 and 7.17.9 - is there anything I can do to get it reviewed and merged?


Remove unnecessary null pointer checks

An extra null pointer check is not needed in functions like the following.

OSNacl::finalize_core
OS_Unix::finalize_core
OS_Windows::finalize_core

I guess the fix is simply to drop the ifs? Does it apply only for delete or also  memdelete? Feel free to propose a pull request to fix it.

@elfring  @akien-mga  I did some investigation and it seems most people recommend doing null check anyway.
If you are trying to delete a null pointer is likely you might have a bug somewhere. Also say you don't check for null you may have a bug that will spam delete on many null pointers affecting performance and you will never notice it without a check.

@trollworkout: Where do you see recommendations for extra null pointer checks before calling the delete operator?
@touilleMan: Thanks for your small source code improvement.

@elfring  I just simply researched online for opinions, forum threads, reddit posts etc from C++ devs based on their experience with the said issue. I formed my own post comment and opinion based on what experienced C++ devs have posted.
for example http://stackoverflow.com/questions/615355/is-there-any-reason-to-check-for-a-null-pointer-before-deleting
The consensus while not required to do null checks before delete it is useful to do it anyway because trying to delete null pointers may be a bug sign.

How useful is it really to repeat unnecessary null pointer checks?
Can it be a class like "std::unique_ptr" should be reused more often nowadays (instead of even calling the delete operator explicitly)?

@elfring  Well they say the issue is delete simply does nothing if the pointer is NULL. The issue is delete returns nothing. Sending a null pointer to delete may be a symptom that your code is bad somewhere. So you do the check for null to catch a possible flaw in the code.

How do you think about to improve the affected error detection by reusing appropriate C++ exceptions instead of leaving questionable checks over?

Sending a null pointer to delete may be a symptom that your code is bad somewhere. So you do the check for NULL to catch a possible flaw in the code.

But that's not the case here. There was no error message raised in the case of a null pointer, only:
if (pointer)
    delete pointer;

We wouldn't have removed those extraneous checks if the code were:
if (pointer)
    delete pointer;
else
    print_line("Oh shi--- pointer was null, somebody messed up!);

;)

@elfring  see here
http://www.cplusplus.com/reference/new/operator delete/
returns nothing
You could be sending null pointers to delete and would not even know it.
C++ devs say is useful tool in preventing certain bugs from seeping in

@trollworkout: Are you looking for a customised debug memory management system?

You could be sending null pointers to delete and would not even know it.
C++ devs say is useful tool in preventing certain bugs from seeping in

In the stackoverflow topic you linked, those "C++ devs" were actually saying that such checks are not needed. Then some added that in some specific cases, they can be useful if one needs to intercept the fact that the pointer was null when an attempt to delete it was made. It does not mean that we're in this case, and in this same thread there is a clear consensus that checking for null pointer before deleting in C++ is not needed generally.

@elfring i don't know :)
I was reading about the issue and found most people do the checks in their own application because sending null pointers to delete may be a sign of bugs. So is more of a tool like you said
if (pointer)
delete pointer;
else if (debugmode)
print_line("you're trying to delete a null pointer);
Is not necessary but may be an useful tool

@akien-mga  correct :)

Fixed by #3230.