element-desktop window is blank

Describe the bug
After upgrading on nixos-unstable, element-desktop opens but the window is totally blank - just white.
Steps To Reproduce
Steps to reproduce the behavior:

Run element-desktop in the terminal.

It's possible that you need to have already signed in and used an older version of  element desktop.
On terminal, I see the following output:

$ element-desktop
/home/jluttine/.config/Element exists: yes
/home/jluttine/.config/Riot exists: no
[4581:1208/110154.863969:ERROR:object_proxy.cc(590)] Failed to call method: org.freedesktop.portal.Settings.Read: object_path= /org/freedesktop/portal/desktop: org.freedesktop.DBus.Error.ServiceUnknown: The name org.freedesktop.portal.Desktop was not provided by any .service files
No update_base_url is defined: auto update is disabled
Fetching translation json for locale: en_EN
Changing application language to en-us
Fetching translation json for locale: en-us
Could not fetch translation json for locale: 'en-us' Error: Cannot find module './i18n/strings/en_US.json'
Require stack:
- /nix/store/lad0w66r4vxw5cgz4mk80zwvipvgzxws-element-desktop-1.11.50/share/element/electron/lib/language-helper.js
- /nix/store/lad0w66r4vxw5cgz4mk80zwvipvgzxws-element-desktop-1.11.50/share/element/electron/lib/tray.js
- /nix/store/lad0w66r4vxw5cgz4mk80zwvipvgzxws-element-desktop-1.11.50/share/element/electron/lib/settings.js
- /nix/store/lad0w66r4vxw5cgz4mk80zwvipvgzxws-element-desktop-1.11.50/share/element/electron/lib/ipc.js
- /nix/store/lad0w66r4vxw5cgz4mk80zwvipvgzxws-element-desktop-1.11.50/share/element/electron/lib/electron-main.js
- /nix/store/52mgy3lkjv664sk1cnmvs2vr4azwqinm-electron-unwrapped-26.4.0/libexec/electron/resources/default_app.asar/main.js
-
    at node:internal/modules/cjs/loader:1084:15
    at Function._resolveFilename (node:electron/js2c/browser_init:2:117576)
    at node:internal/modules/cjs/loader:929:27
    at Function._load (node:electron/js2c/asar_bundle:2:13327)
    at Module.require (node:internal/modules/cjs/loader:1150:19)
    at require (node:internal/modules/cjs/helpers:110:18)
    at AppLocalization.fetchTranslationJson (/nix/store/lad0w66r4vxw5cgz4mk80zwvipvgzxws-element-desktop-1.11.50/share/element/electron/lib/language-helper.js:86:20)
    at /nix/store/lad0w66r4vxw5cgz4mk80zwvipvgzxws-element-desktop-1.11.50/share/element/electron/lib/language-helper.js:99:39
    at Array.filter (<anonymous>)
    at AppLocalization.setAppLocale (/nix/store/lad0w66r4vxw5cgz4mk80zwvipvgzxws-element-desktop-1.11.50/share/element/electron/lib/language-helper.js:98:39) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/nix/store/lad0w66r4vxw5cgz4mk80zwvipvgzxws-element-desktop-1.11.50/share/element/electron/lib/language-helper.js',
    '/nix/store/lad0w66r4vxw5cgz4mk80zwvipvgzxws-element-desktop-1.11.50/share/element/electron/lib/tray.js',
    '/nix/store/lad0w66r4vxw5cgz4mk80zwvipvgzxws-element-desktop-1.11.50/share/element/electron/lib/settings.js',
    '/nix/store/lad0w66r4vxw5cgz4mk80zwvipvgzxws-element-desktop-1.11.50/share/element/electron/lib/ipc.js',
    '/nix/store/lad0w66r4vxw5cgz4mk80zwvipvgzxws-element-desktop-1.11.50/share/element/electron/lib/electron-main.js',
    '/nix/store/52mgy3lkjv664sk1cnmvs2vr4azwqinm-electron-unwrapped-26.4.0/libexec/electron/resources/default_app.asar/main.js',
    undefined
  ]
}
Resetting the UI components after locale change
Resetting the UI components after locale change

While the window is open, the following message repeats in the terminal:

Errors:
link failed but did not provide an info log
[4614:1208/111100.941175:ERROR:shared_context_state.cc(83)] Skia shader compilation error
------------------------
// Vertex SKSL
#extension GL_NV_shader_noperspective_interpolation: require
uniform float4 sk_RTAdjust;uniform float2 uAtlasSizeInv_S0;in float2 inPosition;in half4 inColor;in ushort2 inTextureCoords;noperspective out float2 vTextureCoords_S0;flat out float vTexIndex_S0;noperspective out half4 vinColor_S0;void main() {// Primitive Processor BitmapText
int texIdx = 0;float2 unormTexCoords = float2(inTextureCoords.x, inTextureCoords.y);vTextureCoords_S0 = unormTexCoords * uAtlasSizeInv_S0;vTexIndex_S0 = float(texIdx);vinColor_S0 = inColor;float2 _tmp_1_inPosition = inPosition;sk_Position = inPosition.xy01;}
// Fragment SKSL
#extension GL_NV_shader_noperspective_interpolation: require
uniform sampler2D uTextureSampler_0_S0;
noperspective in float2 vTextureCoords_S0;flat in float vTexIndex_S0;noperspective in half4 vinColor_S0;void main() {// Stage 0, BitmapText
half4 outputColor_S0;outputColor_S0 = vinColor_S0;half4 texColor;{ texColor = sample(uTextureSampler_0_S0, vTextureCoords_S0).rrrr; }half4 outputCoverage_S0 = texColor;{ // Xfer Processor: Porter Duff
sk_FragColor = outputColor_S0 * outputCoverage_S0;}}
// Vertex GLSL
#version 300 es

#extension GL_NV_shader_noperspective_interpolation : require
precision mediump float;
precision mediump sampler2D;
uniform highp vec4 sk_RTAdjust;
uniform highp vec2 uAtlasSizeInv_S0;
in highp vec2 inPosition;
in mediump vec4 inColor;
in mediump uvec2 inTextureCoords;
noperspective out highp vec2 vTextureCoords_S0;
flat out highp float vTexIndex_S0;
noperspective out mediump vec4 vinColor_S0;
void main() {
    highp int texIdx = 0;
    highp vec2 unormTexCoords = vec2(float(inTextureCoords.x), float(inTextureCoords.y));
    vTextureCoords_S0 = unormTexCoords * uAtlasSizeInv_S0;
    vTexIndex_S0 = float(texIdx);
    vinColor_S0 = inColor;
    gl_Position = vec4(inPosition, 0.0, 1.0);
    gl_Position = vec4(gl_Position.xy * sk_RTAdjust.xz + gl_Position.ww * sk_RTAdjust.yw, 0.0, gl_Position.w);
}

// Fragment GLSL
#version 300 es

#extension GL_NV_shader_noperspective_interpolation : require
precision mediump float;
precision mediump sampler2D;
out mediump vec4 sk_FragColor;
uniform sampler2D uTextureSampler_0_S0;
noperspective in highp vec2 vTextureCoords_S0;
flat in highp float vTexIndex_S0;
noperspective in mediump vec4 vinColor_S0;
void main() {
    mediump vec4 outputColor_S0 = vinColor_S0;
    mediump vec4 texColor = texture(uTextureSampler_0_S0, vTextureCoords_S0, -0.5).xxxx;
    mediump vec4 outputCoverage_S0 = texColor;
    {
        sk_FragColor = outputColor_S0 * outputCoverage_S0;
    }
}

Errors:
link failed but did not provide an info log
[4614:1208/111100.941799:ERROR:shared_context_state.cc(83)] Skia shader compilation error
------------------------
// Vertex SKSL
#extension GL_NV_shader_noperspective_interpolation: require
uniform float4 sk_RTAdjust;uniform float3x3 umatrix_S1_c0;in float2 inPosition;in half4 inColor;in float4 inCircleEdge;noperspective out float4 vinCircleEdge_S0;noperspective out half4 vinColor_S0;noperspective out float2 vTransformedCoords_3_S0;void main() {// Primitive Processor CircleGeometryProcessor
vinCircleEdge_S0 = inCircleEdge;vinColor_S0 = inColor;float2 _tmp_0_inPosition = inPosition;float2 _tmp_2_inPosition = inPosition;sk_Position = _tmp_0_inPosition.xy01;{
vTransformedCoords_3_S0 = float3x2(umatrix_S1_c0) * _tmp_2_inPosition.xy1;
}
}
// Fragment SKSL
#extension GL_NV_shader_noperspective_interpolation: require
uniform float3x3 umatrix_S1_c0;uniform sampler2D uTextureSampler_0_S1;
noperspective in float4 vinCircleEdge_S0;noperspective in half4 vinColor_S0;noperspective in float2 vTransformedCoords_3_S0;half4 TextureEffect_S1_c0_c0(half4 _input) {
return sample(uTextureSampler_0_S1, vTransformedCoords_3_S0);}

half4 MatrixEffect_S1_c0(half4 _input) {
return TextureEffect_S1_c0_c0(_input);
}

half4 DisableCoverageAsAlpha_S1(half4 _input) {
_input = MatrixEffect_S1_c0(_input);
half4 _tmp_0_inColor = _input;
return half4(_input);
}

void main() {// Stage 0, CircleGeometryProcessor
float4 circleEdge;circleEdge = vinCircleEdge_S0;half4 outputColor_S0;outputColor_S0 = vinColor_S0;float d = length(circleEdge.xy);half distanceToOuterEdge = half(circleEdge.z * (1.0 - d));half edgeAlpha = saturate(distanceToOuterEdge);half4 outputCoverage_S0 = half4(edgeAlpha);half4 output_S1;output_S1 = DisableCoverageAsAlpha_S1(outputColor_S0);{ // Xfer Processor: Porter Duff
sk_FragColor = output_S1 * outputCoverage_S0;}}
// Vertex GLSL
#version 300 es

#extension GL_NV_shader_noperspective_interpolation : require
precision mediump float;
precision mediump sampler2D;
uniform highp vec4 sk_RTAdjust;
uniform highp mat3 umatrix_S1_c0;
in highp vec2 inPosition;
in mediump vec4 inColor;
in highp vec4 inCircleEdge;
noperspective out highp vec4 vinCircleEdge_S0;
noperspective out mediump vec4 vinColor_S0;
noperspective out highp vec2 vTransformedCoords_3_S0;
void main() {
    vinCircleEdge_S0 = inCircleEdge;
    vinColor_S0 = inColor;
    highp vec2 _tmp_0_inPosition = inPosition;
    highp vec2 _tmp_2_inPosition = inPosition;
    gl_Position = vec4(_tmp_0_inPosition, 0.0, 1.0);
    {
        vTransformedCoords_3_S0 = mat3x2(umatrix_S1_c0) * vec3(_tmp_2_inPosition, 1.0);
    }
    gl_Position = vec4(gl_Position.xy * sk_RTAdjust.xz + gl_Position.ww * sk_RTAdjust.yw, 0.0, gl_Position.w);
}

// Fragment GLSL
#version 300 es

#extension GL_NV_shader_noperspective_interpolation : require
precision mediump float;
precision mediump sampler2D;
out mediump vec4 sk_FragColor;
uniform highp mat3 umatrix_S1_c0;
uniform sampler2D uTextureSampler_0_S1;
noperspective in highp vec4 vinCircleEdge_S0;
noperspective in mediump vec4 vinColor_S0;
noperspective in highp vec2 vTransformedCoords_3_S0;
void main() {
    highp vec4 circleEdge = vinCircleEdge_S0;
    mediump vec4 outputColor_S0 = vinColor_S0;
    highp float d = length(circleEdge.xy);
    mediump float distanceToOuterEdge = circleEdge.z * (1.0 - d);
    mediump float edgeAlpha = clamp(distanceToOuterEdge, 0.0, 1.0);
    mediump vec4 outputCoverage_S0 = vec4(edgeAlpha);
    mediump vec4 _0_input = outputColor_S0;
    _0_input = texture(uTextureSampler_0_S1, vTransformedCoords_3_S0, -0.5);
    mediump vec4 output_S1 = _0_input;
    {
        sk_FragColor = output_S1 * outputCoverage_S0;
    }
}

Expected behavior
Element desktop should show normal content in the window.
Screenshots
Additional context
Notify maintainers
Maintainers: @Ma27 @fadenb @mguentner @Ralith @sumnerevans @NickCao @dali99
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.4, NixOS, 24.05 (Uakari), 24.05.git.a54ab23b588a`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - nixpkgs: `/etc/nixpkgs`

Add a :+1: reaction to issues you find important.

Please check if this is due to https://github.com/NixOS/nixpkgs/issues/244742

Ran these commands but it didn't help (the directories did exist and were removed):
$ rm ~/.config/chromium/GrShaderCache -rf
$ rm ~/.config/chromium/ShaderCache -rf
$ rm ~/.config/chromium/Default/GPUCache/ -rf

And after that this command doesn't find anything anymore so I suppose everything relevant should be removed:
$ find ~ -name "GrShaderCache"

It should be rm -rf ~/.config/Element/GPUCache, as in https://github.com/NixOS/nixpkgs/issues/244486#issuecomment-1643916313

That fixed it, thanks!


Test Certificates - "Date of Collection" and "Date of Test Result" values not displayed

This relates to https://github.com/eu-digital-green-certificates/dgc-testdata/issues/30 opened by @martin-lindstrom
Verification App (Android, potentially iOS also) - Seems the "dr" and "sc" fields are not displayed by the Android Verification App (did not test with iOS).
Issuance Service - The labeling used in the CBOR is not fully compliant with RFC8949 3.4.1 (https://www.rfc-editor.org/rfc/rfc8949.html#name-standard-date-time-string)
To reproduce

Scan a test-type certificate with the template verifier app (tested with Android currently),
e.g. https://github.com/eu-digital-green-certificates/dgc-testdata/blob/main/BE/png/3.png

Currently

The "Date of Collection" and "Date of Test Result" are displayed empty, although the data is present in the QR.

Note that currently in the QRs generated, the CBOR includes the "dr" and "sc" fields non-tagged, while RFC8949 (3.4.1) specifies tag 0 to be used.
See https://www.rfc-editor.org/rfc/rfc8949.html#name-standard-date-time-string
Tested also with tag 0. If tag 0 IS used in the CBOR/QR (according to RFC8949), then Verification App behavior is still not correct.
Input QR:

Verification app:

Expected
Verification app

The "Date of Collection" and "Date of Test Result" are displayed correctly with the template dgca-verifier-app-android app, when a QR is scanned that
-- conforms to the schema (and references) at https://github.com/eu-digital-green-certificates/ehn-dgc-schema
-- was created with dgca-issuance-service

Issuance Service

QRs are generated with datetime string fields tagged with "0", as specified in RFC8949 3.4.1 (https://www.rfc-editor.org/rfc/rfc8949.html#name-standard-date-time-string)

Points of contact at the Estonian development teams

Issuance (Guardtime Estonia) - @sokkphoto - Jaan Sokk; @martti007 - Martti Marran
Verification (Nortal) - @tanelkaar - Tanel Käär

@sokkphoto
The problem is due to a java.text.ParseException: Unparseable date: "2021-05-19T08:20:00Z"

Fixed by PR: #46

Thanks!

Fixed with PR: https://github.com/eu-digital-green-certificates/dgca-verifier-app-android/pull/53


Custom default label -> value to labels

Do you know an option to export a static label -> value in all metrics?

You can use labeling in Prometheus to add static labels (during scrape) to all metrics that are scraped from Kong by Prometheus: https://prometheus.io/docs/prometheus/latest/configuration/configuration/#relabel_config

Thx @hbagdi , I know that but it creates a dependency with how Prometheus is reading my metrics

That is the only way possible today.

it creates a dependency with how Prometheus is reading my metrics

I don't understand the concern here. Could you please elaborate to help me understand the problem better?

I need to configure prometheus to rewrite labels just for my project. It's a dependency... I'm looking this code and it's really not supporting custom metrics. Do you think about support it?

My knowledge of Prometheus might be lacking but are there other exporters which do support this?
Is it a common pattern that is followed in the Prometheus community?

ping here

I really don't know about other exporters, but you can be the first one ;)
I'm already using relabel, but I think it's not the best practice because my server need to overwrite my configs.
I'll close this issue


Enable projectile-locate-dominating-file on tramp

Essentially this makes projectile work properly (again?) when using tramp.
This causes the following (all of it on tramp):

If projectile-locate-dominating-file is disabled then projectile-find-file doesn't work, which also makes +ivy/projectile-find-file only able to search in the current directory.
It allows projectile-vc to work again (only on machines with the same OS).
Projects are now added properly: they now appear with counsel-projectile-switch-project (accessible in Open project) in the dashboard.
LSP autocompletion now includes files from the whole project (LSP still doesn't work properly though, see #3390).

I tested it with Emacs 26.3 and 27.0.91 , both with ssh and sshx.  The infinite loop was no longer present.

projectile-vc doesn't work for me using emacs locally on MacOs and a remote Linux server.
I guess this was because magit-git-executable is set in modules/tools/magit/config.el with (executable-find magit-git-executable).  For MacOs this evaluates to /usr/local/bin/git, but in Linux (Debian) it evaluates to /usr/bin/git.  Setting it manually to "git" allows to run magit with projectile through tramp properly.

I'm unable to test this at the moment, so I'll take your word for it until I can. I've also fixed the magit-git-executable issue in 58fa2b1. Thanks for the PR and bringing it to my attention!


[7.x] Publish console stub

This PR adds the console.stub to the files that will be published when running php artisan stub:publish.
If there is a published stub, make:command will make use of it.

Looks like the DummClass, dummy:command, etc. need to be replaced with the published stub style {{ }} replacements.

Updated to use the right style of placeholders

Thank ya.


@Moocow9m would you be calm when you find out you have a virus which is using your GPU as an ASIC Miner? and when you find out and trace it back to the owner to find out the program is open source.... would you be calm

The miner is opensource, not the virus :-)

@TehMerkMods that's why I build my own version rather than just blindly downloading one.

@Moocow9m i downloaded a mod menu for FiveM with this inside

@TehMerkMods then please look where the mod got it or replace the mod's version with one built from the repo as I have not had any problems.

@Moocow9m Well like i said I'm going to reverse the application in my spare time and get the twats bitcoin


[FLINK-30157] Trigger Events Before JM Recovery and Unhealthy Job Restart

What is the purpose of the change
To emit specific events along the following cases:

JM recovery
Unhealthy Job Restarts

Brief change log
ApplicationReconciler has been updated with the extra event logic
Verifying this change
Unit tests where added to ApplicationReconcilerTest that covers the functionality.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changes to the CustomResourceDescriptors: no
Core observer or reconciler logic that is regularly executed: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

cc @gyfora


Add support for writable values

Add support for writable holding registers

Hi. Any progress on this? I have a few devices with writable holding and coil registers I wanted to add

Yes this is in progress aim to be complete within the next couple of weeks

Awesome!

This has now been merged.


Fix sros license handling

The uuid mangling doesn't seem to be needed for all license files. Get a bigger sampling base and potentially remove or fix logic to only invoke for given versions!?

I dunno but I haven't had this issue with any new licenses that I've got so I'm going to assume this is just associated with some older crappy license files or whatnot. Closing.


tf-trt converted deep models int8 inference crashes

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock
example script provided in TensorFlow):  NO
OS Platform and Distribution (e.g.,
Linux Ubuntu 16.04):
Version: tf-gpu.1-15.m45 (dlvm)
Based on: Debian GNU/Linux 9.12 (stretch) (GNU/Linux 4.9.0-12-amd64 x86_64\n)
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if
the issue happens on mobile device:
TensorFlow installed from (source or
binary): - TensorFlow version (use command below):  v1.15.2-1-g61ff2cb 1.15.2
Python version: - Bazel
version (if compiling from source): n/a
GCC/Compiler version (if compiling from
source): n/a
CUDA/cuDNN version: - GPU model and memory:
Cuda compilation tools, release 10.0, V10.0.130
#define CUDNN_MAJOR 7
#define CUDNN_MINOR 6
#define CUDNN_PATCHLEVEL 5

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
I am using tf-trt to convert some of the models to different precision mode.
it works fine for fp32 and fp16  (gets converted and inference runs fine)
but int8  only get converted but on inference gives the following error
2020-04-12 10:20:31.123734: I tensorflow/compiler/tf2tensorrt/kernels/trt_engine_op.cc:812] Starting calibration thread on device 0, Calibration Resource @ 0x7fd990004ea0
2020-04-12 10:20:31.123883: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libnvinfer.so.5
2020-04-12 10:20:31.124498: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libnvinfer_plugin.so.5
2020-04-12 10:20:58.770669: I tensorflow/compiler/tf2tensorrt/kernels/trt_engine_op.cc:812] Starting calibration thread on device 0, Calibration Resource @ 0x7fd96c004e80
2020-04-12 10:21:20.543550: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10.0
2020-04-12 10:29:21.108202: F tensorflow/compiler/tf2tensorrt/kernels/trt_engine_op.cc:349] Check failed: t.TotalBytes() == device_tensor->TotalBytes() (1832000 vs. 21467376)
Aborted
Describe the expected behavior
model should run faster with int8 ?
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
n/a
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.
already attached

@prateekpatelsc,
In order to expedite the trouble-shooting process, could you please provide the complete code to reproduce the issue reported here. Thanks!

@amahendrakar : unfortunately the code is confidential and private to snap . don't think would be able to share those.

@prateekpatelsc,
Is it possible to share a minimal code sample to replicate the error you are facing? Thanks!

from tensorflow.python.compiler.tensorrt import trt_convert as trt
converter = trt.TrtGraphConverter(
	input_graph_def=frozen_graph,
        precision_mode="INT8",
       max_batch_size=2,
	nodes_blacklist=['logits', 'classes'])
frozen_graph = converter.convert()

This is 4 lines of code that i use to take my original graph def and convert it to tf-trt optimized graph for precision mode int8 .
the conversion works fine. During inference it takes huge time and then the above stacktrace,
NOTE : if i use precision mode fp32 or fp16 , things work completely fine.
i have been running this tesla t4 and v100 gpu

@prateekpatelsc,
On running the above code, I'm facing an error stating NameError: name 'frozen_graph' is not defined. Please find the gist of it here. Thanks!

@amahendrakar : can you please look at the code ,
i sent you a snippet of the code , not the full code right , you definitely need to provide frozen graph for some model

Hi There,
We are checking to see if you still need help on this, as you are using an older version of tensorflow which is officially considered end of life . We recommend that you upgrade to the latest 2.x version and let us know if the issue still persists in newer versions. Please open a new issue for any help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need help with this issue, please provide us with more information.


Smoother stop and restart for games.

currently, SSM stops service. Can try using MCRcon to send a command to stop to restart the server for source games. other games may not be possible from PS. Telnet for 7D2D might be possible. Conan requires Ctl+C in the console window.

Looks like close window and send keys are going to be the best.currently  7D2D close window with Alt+Y

updated ssm with close window instead of stop process. 7d2d and conan smooth stop.


Notice Warn -> Notice Info

Issue #, if available:
Fix Bugs
Description of changes:
Fix Document Bugs
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Automatically Merged


Usage Instructions for auditjs

I don't know what I am missing. I installed nodejs for Windows, then installed auditjs with npm install auditjs -g command. Then I rebooted the PC and then I run auditjs -v and it says "auditjs is not recognized as a in internal or external command, operable program or batch file". What am I missing?
I am using Windows 7.

I am using Windows 8.1 which should be close enough.
I just tried uninstalling then installing my auditjs, and have the exact same results as you do. I am reasonably sure I know the reason and am planning on having a new version published tonight which will hopefully resolve the issue. I will be introducing a new command name auditjs-win which will be intended to target windows.
The issue seems to be a fix that was put in to handle some aggravating issues with some Linux distributions, which apparently have broken Windows installation.
Sorry about that. I will write again as soon as we have a fix published.

Thank you. For your honesty and hardwork.

Alright, I have added an auditjs-win binary/shortcut which should allow things to work on windows again. I will keep looking for a better cross-platform solution for these problems, but at least there is now a workaround.
Thank you for the issue!


scriptmgr-server failing readiness and liveness probes

Describe the bug
I am trying to run a self-hosted pixie in a 3 node cluster. Here's how my env looks like:
Kubernetes Version: v1.28.2
OS-Image: Rocky Linux 8.9 (Green Obsidian)
Kernel Version: 5.4.266-1.el8.elrepo.x86_64
Container Runtime: containerd://1.6.26
Pixie Cloud Version: 0.1.7
$ kubectl get nodes -o wide
NAME                     STATUS   ROLES           AGE   VERSION   INTERNAL-IP     EXTERNAL-IP   OS-IMAGE                           KERNEL-VERSION                CONTAINER-RUNTIME
rdev5-rocky8-control-1   Ready    control-plane   43d   v1.28.2   <IP_ADDRESS>   <none>        Rocky Linux 8.9 (Green Obsidian)   5.4.266-1.el8.elrepo.x86_64   containerd://1.6.26
rdev5-rocky8-worker-1    Ready    <none>          43d   v1.28.2   <IP_ADDRESS>   <none>        Rocky Linux 8.9 (Green Obsidian)   5.4.266-1.el8.elrepo.x86_64   containerd://1.6.26
rdev5-rocky8-worker-2    Ready    <none>          43d   v1.28.2   <IP_ADDRESS>   <none>        Rocky Linux 8.9 (Green Obsidian)   5.4.266-1.el8.elrepo.x86_64   containerd://1.6.26

To Reproduce
Steps to reproduce the behavior:
https://docs.px.dev/installing-pixie/install-guides/self-hosted-pixie/#1.-deploy-pixie-cloud
Expected behavior
Pixie self-hosted cloud gets deployed.
Logs
Please attach the logs by running the following command:
$ kubectl -n plc describe pod/scriptmgr-server-56d97c78c7-q6s4m
Events:
  Type     Reason          Age                 From               Message
  ----     ------          ----                ----               -------
  Normal   Scheduled       18m                 default-scheduler  Successfully assigned plc/scriptmgr-server-56d97c78c7-q6s4m to rdev5-rocky8-worker-2
  Normal   AddedInterface  16m                 multus             Add eth0 [<IP_ADDRESS>/32] from k8s-pod-network
  Normal   Created         15m (x2 over 16m)   kubelet            Created container scriptmgr-server
  Normal   Started         15m (x2 over 16m)   kubelet            Started container scriptmgr-server
  Normal   Killing         15m                 kubelet            Container scriptmgr-server failed liveness probe, will be restarted
  Warning  Unhealthy       15m (x12 over 16m)  kubelet            Readiness probe failed: Get "https://<IP_ADDRESS>:52000/healthz": dial tcp <IP_ADDRESS>:52000: connect: connection refused
  Warning  Unhealthy       15m (x6 over 16m)   kubelet            Liveness probe failed: Get "https://<IP_ADDRESS>:52000/healthz": dial tcp <IP_ADDRESS>:52000: connect: connection refused
  Normal   Pulled          11m (x7 over 16m)   kubelet            Container image "gcr.io/pixie-oss/pixie-prod/cloud/scriptmgr_server_image:0.1.7" already present on machine

I've faced the same issue described here #1838
I got these errors in logs scriptmgr-server
time="2024-02-14T00:29:10Z" level=error msg="Failed to update store using bundle.json from gcs." bucket=pixie-prod-artifacts error="rpc error: code = Internal desc = failed to download bundle.json" path=script-bundles/bundle-oss.json
time="2024-02-14T00:30:40Z" level=error msg="Failed to get attrs of bundle.json" bucket=pixie-prod-artifacts error="Get \"https://storage.googleapis.com/storage/v1/b/pixie-prod-artifacts/o/script-bundles%2Fbundle-oss.json?alt=json&prettyPrint=false&projection=full\": dial tcp <IP_ADDRESS>:443: i/o timeout" path=script-bundles/bundle-oss.json

workaround for me is to add failureThreshold and failureThreshold equal to 5
---
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization

resources:
- yamls/cloud_deps_elastic_operator.yaml
- yamls/cloud_deps.yaml
- yamls/cloud.yaml
- yamls/cloud_ingress_grpcs.yaml
- yamls/cloud_ingress_https.yaml

patches:
- target:
    group: apps
    version: v1
    kind: Deployment
    name: scriptmgr-server
  patch: |-
    - op: add
      path: /spec/template/spec/containers/0/livenessProbe/failureThreshold
      value: 5
    - op: add
      path: /spec/template/spec/containers/0/readinessProbe/failureThreshold
      value: 5


Validation accuracy more than train

Hi, I am getting validation accuracy greater than training and validation loss lesser than train. Can you tell why? Maybe due to class imbalance? I am using different dataset

Hi, validation loss can be lower than train, e.g. using dropout during training. Best practice is to the model with early stopping on validation loss. We cannot say anything about your accuracy, especially if you are using different dataset. However, we recommend using F-score instead of accuracy with imbalanced datasets.


add paradym wallet

@maaikevanleuken I don't see a place to add an url for the open source codebase. Is that correct? The url is https://github.com/animo/paradym-wallet

@maaikevanleuken I don't see a place to add an url for the open source codebase. Is that correct? The url is https://github.com/animo/paradym-wallet

That's correct in the current version. After PR3 has been merged, there will be a place for that. I will update it then. Thank you for adding your wallet!


Unwanted navbar scroll on Cordova app

Using the kitchen-sink example I've created an example iOS Cordova App. All works fine, but when I try the form example and I focus an input field, the entire page scrolls up and the navbar disappears. How I can fix on to the navbar scrolling only the page content?
Thanks!

This is default Cordova behavior. You need to use Keyboard plugin https://github.com/apache/cordova-plugins/tree/master/keyboard take a look at disableScrollingInShrinkView and shrinkView methods

Thanks!


Write tests

https://docs.pytest.org/en/7.2.x/
https://realpython.com/pytest-python-testing/

can only be performed after https://github.com/obervinov/_templates/issues/51


HTTPS support for Feed URLs

Now that ShowRSS finally supports HTTPS connections, everyone should (be able to) use them.

Thanks for this!
Two nitpicks:

Since it's no longer a prefix, can you rename the constant to end with Regex instead of PrefixRegex?
Could you use NSRegularExpression instead of NSPredicate?

PR has been updated based on your feedback.

Looks great, thanks!


Cannot select language for southpark videos

When downloading videos from southpark.de, I have no way to select which language the video should be in. The site offers both German and English through a menu, but youtube-dl only shows the following formats — and I can't select one because it errors out with ERROR: requested format not available (but that's not the purpose of this report).
youtube-dl --verbose -F http://www.southpark.de/alle-episoden/s03e09-beschnittene-pfadfinder-teil-3
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', '-F', 'http://www.southpark.de/alle-episoden/s03e09-beschnittene-pfadfinder-teil-3']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2014.11.16
[debug] Python version 2.7.6 - Darwin-14.0.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 2.4.3, ffprobe 2.4.3, rtmpdump 2.4
[debug] Proxy map: {}
[southpark.de] s03e09-beschnittene-pfadfinder-teil-3: Downloading webpage
[southpark.de] 600fd1d8-ecfd-11e0-aca6-0026b9414f30: Downloading info
[southpark.de] bumper: Extracting information
[southpark.de] bumper: Downloading video urls
[southpark.de] e61e5062-ed00-11e0-aca6-0026b9414f30: Extracting information
[southpark.de] e61e5062-ed00-11e0-aca6-0026b9414f30: Downloading video urls
[southpark.de] e61e9fcc-ed00-11e0-aca6-0026b9414f30: Extracting information
[southpark.de] e61e9fcc-ed00-11e0-aca6-0026b9414f30: Downloading video urls
[southpark.de] e61ef2ba-ed00-11e0-aca6-0026b9414f30: Extracting information
[southpark.de] e61ef2ba-ed00-11e0-aca6-0026b9414f30: Downloading video urls
[southpark.de] e61f5304-ed00-11e0-aca6-0026b9414f30: Extracting information
[southpark.de] e61f5304-ed00-11e0-aca6-0026b9414f30: Downloading video urls
[info] Available formats for bumper:
format code extension resolution  note
400         mp4       512x288
[info] Available formats for e61e5062-ed00-11e0-aca6-0026b9414f30:
format code extension resolution  note
400         mp4       480x360     (worst)
700         x-flv     640x480
600         mp4       640x480
800         mp4       640x480     (best)
[info] Available formats for e61e9fcc-ed00-11e0-aca6-0026b9414f30:
format code extension resolution  note
400         mp4       480x360     (worst)
700         x-flv     640x480
600         mp4       640x480
800         mp4       640x480     (best)
[info] Available formats for e61ef2ba-ed00-11e0-aca6-0026b9414f30:
format code extension resolution  note
400         mp4       480x360     (worst)
700         x-flv     640x480
600         mp4       640x480
800         mp4       640x480     (best)
[info] Available formats for e61f5304-ed00-11e0-aca6-0026b9414f30:
format code extension resolution  note
400         mp4       480x360     (worst)
700         x-flv     640x480
600         mp4       640x480
800         mp4       640x480     (best)

Apparently, the language is set via Flash variables from the HTML page:
var m = document.cookie.match(/SPS_video_language=([a-z][a-z])/);
  if (m && m.length > 1) flashvars.lang = m[1];

Any idea how this could be carried over to the extractor?

Any updates on this issue?

I'm having the same issue. Including this feature would be great.

Related: #10758

Nevermind, it is working fine for southpark.de

I don't have the means to confirm it now, but if you say it works, I'll close this then.

I would like to know how is the language selection supposed to work. How do I get the English episodes from southpark.de with youtube-dl? I can get a list of the different formats with the -F option, but none of those is in English.

@Emgalai You can select the language by modifing the address:
Eng:
"http://southpark.cc.com/full-episodes/s" + currSeason + "e" + currEpisode;
Ger:
"http://southpark.de/alle-episoden/s" + currSeason + "e" + currEpisode;

I know that, but southpark.de has better maximum resolution, 1920x1080, while southpark.cc.com has just 1280x720. That's why I would like to get the English version from the German site.

You could download the episodes from .de and then replace the m4a with the English version. But as far as i know there is no possibility to download English episodes directly from southpark.de

I know it is possible. Maybe not (yet) with youtube-dl, but I downloaded English episode from southpark.de with Video DownloadHelper Firefox addon.
But it seems that only the German versions have the higher resolution. So, merging two sources would be required to get a 1920x1080 English video from there as well...

Well, language selection on the german site does NOT work - on the .de website this is done with a cookie (SPS_video_language en). A mixture of .de and .cc episodes won't work anymore (season 20+), as the domain .cc is distributing NTSC @ 23,976 fps and .de PAL @25 fps, so each act has different lengths - and you have to evade geoblock.
Any ideas? Passing cookies did not do the job, and -F does not list another audio stream.

@khoschi that's true, mixing the two sources wont work anymore. @slhck i think this issue should be reopened.

I think it'd be better to open a new issue with the respective logs, as I cannot follow up on it anymore.

@khoschi There is no geoblocking. You can download from any country (I think) by using the episodes address, either http://southpark.cc.com/full-episodes/s21e01 or http://www.southpark.de/alle-episoden/s21e01 for example.
And I succesfully mixed the German and English episodes by MKVToolNix GUI. The episodes have different lenghts but only because the German episodes don't have the repeat part at the end.

Geoblocking applies only for surfing the website, you're right. I've found different runtime only for episodes from season 20, so if you try s20e01 act1 you will find that both files have the same amount of frames, but .cc framerate is 23.976 and .de is 25 - and this time you have to fix the audio duration before mixing. Other seasons use 23.976 fps  for the .de domain as well.


Implement macOS signing and notarization

PLAN:

[x] Collect and share documentation links about the topic (many of which I've been reading as of recently).
[x] Get a developer certificate from Apple.
[ ] Explore the process manually / via ad-hoc scripts, confirming that it seems to work.
[ ] Try to validate it in collaboration with someone else, running macOS Catalina or Big Sur.
[ ] Transpose the process into pup code.

NOTE:

From the investigation so far, it is a very good idea to have a properly layed out, as small as possible macOS Application Bundle to submit for notarization.
With that in mind, #38 becomes a relevant dependency to this issue's "correct" implementation.

RESOURCES and LINKS:

This Mu issue, nearly one year old (!!!).
These articles by @glyph:

Mac Python Distribution Post Updated for Catalina and Notarization
Tips And Tricks for Shipping a PyGame App on the Mac

Apple sourced documents:

WWDC 2019 "All About Notarization" session
Code Signing Tasks
Customizing the Notarization Workflow
Resolving Common Notarization Issues

More:

Notarize a Command Line Tool
How to notarize your software on macOS

CONCEPTS (that need better understanding):

Hardened runtime (and variations thereof?).
Secure timestamps.
Entitlements.

WORK IN PROGRESS:
signing script
#!/bin/bash
set -e

# Use keychain to allow "codesign" to always access the code signing certificate's private key.

cd ./build/pup > /dev/null 2>&1

BASE_DIR=./mu-editor.app/Contents/Resources/Python/

SIG_ID='<apple signing certificate id -- 10 alphanumeric digits>'
CSOPTS="--deep --force --verify --verbose --sign $SIG_ID --options runtime --timestamp"

# --------------------------------------------------------------------------------
# SIGN all shared libraries

find ${BASE_DIR}/lib -type f -name "*.so" -o -name "*.dylib" | while read SHLIB_TO_SIGN
do
    echo "SIGNING ${SHLIB_TO_SIGN}"
    codesign ${CSOPTS} ${SHLIB_TO_SIGN}
done

# --------------------------------------------------------------------------------
# SIGN binaries

find ${BASE_DIR}/bin -type f | while read BIN_TO_SIGN
do
    echo "SIGNING ${BIN_TO_SIGN}"
    codesign ${CSOPTS} "${BIN_TO_SIGN}"
done

# --------------------------------------------------------------------------------
# SIGN the application bundle itself

echo "SIGNING ./mu-editor.app"
codesign ${CSOPTS} ./mu-editor.app

cd -  > /dev/null 2>&1

WORK IN PROGRESS:
notarization request script
#!/bin/bash

cd ./build/pup > /dev/null 2>&1

BID='com.github.mu-editor.mu'
USER='<apple-id>'
MUAS='<application-specific-password>'

/usr/bin/ditto -c -k --sequesterRsrc --keepParent ./mu-editor.app/ mu-editor.zip
xcrun altool --notarize-app --primary-bundle-id ${BID} --username ${USER} -p ${MUAS} --file mu-editor.zip

# will print the Notarization RequestUUID

cd -

notarization status script:
#!/bin/bash

USER='<apple-id>'
MUAS='<application-specific-password>'

if [ -z "${RID}" ]
then
    echo "Set RID to the Notarization RequestUUID"
    exit
fi

xcrun altool --notarization-info ${RID} -u ${USER} -p ${MUAS}

PROGRESS:

Preliminary scripts created and shared above.
All scripts work (up to a point).
Notarization result is unsuccessful:

2020-09-17 17:00:21.395 altool[78612:1135309] No errors getting notarization info.

   RequestUUID: ...
          Date: 2020-09-17 15:43:31 +0000
        Status: invalid
    LogFileURL: ...
   Status Code: 2
Status Message: Package Invalid

Just dropping in here due to the @-mention… (I am so glad that my writing is helping with this, annoying as it is!)
The status message rarely contains enough information to debug notarization failures; you'll really want to look at the log file in that elided LogFileURL.
How did you generate the .zip?  IIRC "Package Invalid" is an error you can get when forgetting to pass the -y option.

@glyph, thanks for chiming in,
Agree and undestand that the actual log holds details that the status cannot convey - the note was posted with brevity and progress sharing/noting in mind. For completeness, it did note that several files were not signed which, somehow, may point at a .zip file creation issue (is it including said signatures? I will investigate) which, again, may be somewhat consistent with the "Package Invalid" status...
Wondering what you mean by the -y option... The .zip file was produced with /usr/bin/ditto -c -k --sequesterRsrc --keepParent <path-to-app-bundle> <name-of-zip-file> which, per the man page, does not seem to support such option. Maybe you are refering to the zip command's option, WRT to symlinks?
Regardless of that, again, thanks for your articles and this message. I know I'll have to keep reading, undertanding, exploring, until I have a clear enough mental model of how every bit and piece should fit together: then things should work! :)

Agree and undestand that the actual log holds details that the status cannot convey - the note was posted with brevity and progress sharing/noting in mind. For completeness, it did note that several files were not signed which, somehow, may point at a .zip file creation issue (is it including said signatures? I will investigate) which, again, may be somewhat consistent with the "Package Invalid" status...

As I understand it, code signatures are just regular files so they should be included with just about any archive creator...
If you're creating binary files with extensions other than .so and .dylib you might need to modify my simple signing script.  (What files is it complaining about?)

Wondering what you mean by the -y option... The .zip file was produced with /usr/bin/ditto -c -k --sequesterRsrc --keepParent <path-to-app-bundle> <name-of-zip-file> which, per the man page, does not seem to support such option. Maybe you are refering to the zip command's option, WRT to symlinks?

Ah, yes, sorry I didn't realize that's what ditto was doing; I've never used it to create zip archives.  No idea if it has problems with this process!

Regardless of that, again, thanks for your articles and this message. I know I'll have to keep reading, undertanding, exploring, until I have a clear enough mental model of how every bit and piece should fit together: then things should work! :)

Good luck!

PROGRESS:

Read a bit more.
Brought in yet another article on the subject.

THOUGHTS:

Previous (failed and, admittedly, somewhat rushed) experiences go along with recent readings, boiling down to "probably the signing I've been doing isn't 100% as it should".
Need to specify entitlements, which I haven't done up until now.
Want faster iteration cycles:

For that I'll create a minimal GUI app, tkinter-based.
No Qt dependencies (or any other, from PyPI) so smaller in size and with much fewer shared libraries.
Should also make it easier to digest any notarization failure reports.

PROGRESS:

Created minimal GUI app at https://github.com/tmontes/puppy.
Adjusted hand-hacked scripts:

Now signing with entitlements.
Now subitting a .zip file.

Notarization is still failing, but now down to a single, common error, once for each of the 9 files that were signed within puppy.app. It reads: "The binary is not signed with a valid Developer ID certificate.".
Wonderful. I want my 100 EUR back! :P

PROGRESS (and preliminary SUCCESS!):

Figured out I had been using the wrong kind of certificate for signing after reading more and with more attention (thanks again to @glyph, who very carefully pinpoints that in his artice).
Successfully signed, notarized, and stapled puppy.
Will share the binary with the Mu development team to gather feedback.

It's been such an inspiration to read this thread, thank you for sharing these updates!
But…

😢

Wow @glyph, thanks a lot for trying it out! Even though it's a failure, it's the best possible feedback I could have had: an attempt from a completely foreign system, that in no possible way could trust my system.
INVESTIGATED:

Figured out that the packaged bundle included a soft-link pointing nowhere.
The output of spctl --assess --type exec <path-to-app-bundle> helped pinpointing it.

ALSO:

Rememberd I have a Catalina VM (I'm packaging on Mojave) which failed as above.

THUS:

Hand fixed the dangling soft-link, signed, notarized puppy again, producing puppy-1.0.0-macos-post1.zip.
Tried out the newly packaged variant on my Catalina VM which ran it with no warnings: looks promising.
Published it alongside the previously published release.

🎉 🎉 🎉

An interesting side-note though, which is probably unrelated to code-signing: when I first downloaded it, it unarchived to "puppy 2.app" because of the existing "puppy.app" in my downloads folder.  If the app's name has spaces in it, it fails to launch, due to a buggy shell script.
The shell script is written as:
#!/bin/sh

exec "$(dirname $0)/../Resources/Python/bin/python3.7" -m puppy "$@"

which our trusty ol' friend ShellCheck which you should always be using if you're ever writing a shell script tells us:
In ./Downloads/puppy.app/Contents/MacOS/puppy line 3:
exec "$(dirname $0)/../Resources/Python/bin/python3.7" -m puppy "$@"
                ^-- SC2086: Double quote to prevent globbing and word splitting.

Did you mean:
exec "$(dirname "$0")/../Resources/Python/bin/python3.7" -m puppy "$@"

For more information:
  https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ...

Modifying it in the suggested way fixes the launch, and oddly enough does not seem to invalidate the code signature, which seems quite bad; this "main executable" really ought to be covered by the codesigning check; if attackers can rewrite it and it launches anyway then codesigning provides no protection!
However, this strategy of using a shell (or Python or Ruby) "launcher", while a popular trick that worked in the past, seems to be breaking down on more recent macOS releases, particularly catalina.  See this blog post about how one Emacs mac build's too-clever binary-selector launching script effectively broke the ability to launch the application multiple times.  puppy.app seems to have the same problem - launching it from the Downloads folder while it's open doesn't bring it back to the front.  By contrast, py2app applications use an executable shim and do not have this problem, since they don't re-exec; testing with my own pygame apps validates this.  So you might want to figure out some way to build a C executable that invokes Python as a library.
Finally I think using -m like this is kind of risky, as it skirts close to the vulnerabilities discussed in https://glyph.twistedmatrix.com/2020/08/never-run-python-in-your-downloads-folder.html.  This won't be a problem for most users given the semantics of app launching, but -m seems like a mechanism that is unnecessarily risky for final distribution of a binary artifact.

An interesting side-note though, which is probably unrelated to code-signing: when I first downloaded it, it unarchived to "puppy 2.app" because of the existing "puppy.app" in my downloads folder. If the app's name has spaces in it, it fails to launch, due to a buggy shell script.
The shell script is written as:
#!/bin/sh

exec "$(dirname $0)/../Resources/Python/bin/python3.7" -m puppy "$@"

which our trusty ol' friend ShellCheck which you should always be using if you're ever writing a shell script tells us:
In ./Downloads/puppy.app/Contents/MacOS/puppy line 3:
exec "$(dirname $0)/../Resources/Python/bin/python3.7" -m puppy "$@"
                ^-- SC2086: Double quote to prevent globbing and word splitting.

Did you mean:
exec "$(dirname "$0")/../Resources/Python/bin/python3.7" -m puppy "$@"

For more information:
  https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ...

Modifying it in the suggested way fixes the launch, and oddly enough does not seem to invalidate the code signature, which seems quite bad; this "main executable" really ought to be covered by the codesigning check; if attackers can rewrite it and it launches anyway then codesigning provides no protection!

Good catch and thanks for the reference to ShellCheck which I didn't know about! Logged #44 to track and improve this.

However, this strategy of using a shell (or Python or Ruby) "launcher", while a popular trick that worked in the past, seems to be breaking down on more recent macOS releases, particularly catalina. See this blog post about how one Emacs mac build's too-clever binary-selector launching script effectively broke the ability to launch the application multiple times. puppy.app seems to have the same problem - launching it from the Downloads folder while it's open doesn't bring it back to the front. By contrast, py2app applications use an executable shim and do not have this problem, since they don't re-exec; testing with my own pygame apps validates this. So you might want to figure out some way to build a C executable that invokes Python as a library.

Interesting, and confirmed that mis-behaviour on my Catalina VM. On my dev/packaging system, running Mojave, "relaunching" does bring the running instance's window to the front... That's Catalina changes for us! sigh :-)
I wanted to keep things as simple and minimal as possible... But maybe a native executable launcher will be required. Let's see what the future holds. Logged #45 to track this.
Side-side-note: maybe also related to the fact that running the pup-packaged puppy results in a python3.7 application name in the menu bar (or, then again, maybe this can be customized from within the app bundle's Info.plist).

Finally I think using -m like this is kind of risky, as it skirts close to the vulnerabilities discussed in https://glyph.twistedmatrix.com/2020/08/never-run-python-in-your-downloads-folder.html. This won't be a problem for most users given the semantics of app launching, but -m seems like a mechanism that is unnecessarily risky for final distribution of a binary artifact.

Agreed and, again, thanks for that article which I skimmed before and now read in more detail. :-)
...I guess #42 (partially?) addresses this concern. To learn a bit more, a created (and published puppy 1.1.0 that now displays both the CWD and sys.path.
What I get on both my Mojave dev/packging system and the Catalina VM differs slightly, but is not very frightening:

os.getcwd():

/ on both systems, probably due to the way the Python Standalone Build is built.

sys.path:

Does include / on both systems, again, which is silly and (relatively) safe.
Includes no entries outside .../puppy.app on Catalina, which looks pretty good.
Includes a dangerous entry on my dev/packaging system: ~/.local/lib/python3.7/site-packages. I suppose this is site.py doing its thing and that the local site-packages is there (empty, but there) because at some point pip may have created it for me.

Again, thanks a lot for your help! :-)

Again, thanks a lot for your help! :-)

Super glad to see folks attacking the problem of proper app distribution with Python, so I'm happy to do so.

Closing:

Basic, (very) minimal implementation in place.
Other identified issues created and linked here.
Let's release and have the team experiment to gather feedback early on.


[cmake] Refactor: Update CMakeLists.txt

Little CmakeLists.txt  refactoring.

This just adds some whitespace, I'm not sure that there is much gained from this change. CC: @etcwilde

This just adds some whitespace, I'm not sure that there is much gained from this change.

I agree that they can be small changes, but I don't think that this is improving readability. We have plenty of cases of the dense CMakeLists, and in general do have dense C++ (as per the LLVM style). This just adds whitespace to one location which also diverges from the overall style of the project.

I agree that there is plenty of cases of the dense CMakeLists, but in this case my idea was in separeting blocks, which is also often used in many projects.

Sure, other projects do use the less dense form and liberally add newlines. I don't think that we generally do that in this project.

See this for example. You are also using blocks style.

We don't really have a hard rule for style in the CMake. @compnerd and I will usually comment on stylistic improvements if we see them in PRs that are changing the code, but without an actual formatting guide and verification in place to ensure that we're adhering to those rules, I'm reluctant to accept a change that is solely adding newlines for an arbitrary style change. If you find a bug in that region of the script and you added a newline there, then I would be fine with that.

My idea was not a global change in the style of the project, but a small improvement. To be honest, I didn’t think that such a change could raise questions and doubts, because, in my opinion, it is quite intuitive.

Ping

People obviously have different opinions on good style and taste. In fact, I prefer an empty line here too. I think the point Evan is making is that a small improvement is not going to be useful unless we all acknowledge that it is an objective improvement and devise a way to keep that improvement in place. Otherwise, this is just one person cleaning up what they think is inferior formatting style after another person, in an already and still inconsistently formatted code base, which does not sound like a wholesome activity. Improvement or not, reformatting code in such small volumes is also going to pollute the commit history.


Add GLM

Hi, this PR adds GLM to the assets list.
Token Address: 0x7DD9c5Cba05E151C895FDe1CF355C9A1D5DA6429
Token Name (from contract): Golem Network Token
Token Decimals (from contract): 18
Token Symbol (from contract): GLM
SushiSwap pair of Token: 0x56820f78efd1061d174f7a460cfd9d8e4283069b
Link to the official homepage of token: https://golem.network/
Link to CoinMarketCap or CoinGecko page of token: https://coinmarketcap.com/currencies/golem-network-tokens/

Hey @mat7ias,
It looks like GLM was added in #12.
Can you confirm this is acceptable or would rather use this PR?
Thanks

@matthewlilley Yes, that's perfect, thanks! Closing this PR.


Serializing Public Key and Context

I understand that a Ciphertext can be serialized using the Ciphertext::save(std::ostream &) method. I also saw a test for the same. However, I did not see one for the keys or the context.
Are they serialized along with the ciphertext?
To encrypt a plaintext or perform operations on a ciphertext at a different machine, I will require context and the public key. Is there a procedure to serialize them?
Moreover, is there an example or a test that demonstrates this usecase (serializing various components of the scheme)?

The keys are not serialized with the Ciphertext. You can serialize them separately using *::save(std::ostream &) methods just like the Ciphertext. The context itself cannot be serialized, but the EncryptionParameters that are used to create the context can be serialized through EncryptionParameters::Save(const EncryptionParameters&, std::ostream&).

I would like to know how to send serialized Ciphertext (which is in ostream) by socket, and how to receive and deserialize in the remote computer, thanks.

I would like to know how to send serialized Ciphertext (which is in ostream) by socket, and how to receive and deserialize in the remote computer, thanks.

You can use Ciphertext::save(std::ostream &) to save your ciphertext to a file and then send that file, receive it on the remote machine, and use Ciphertext::load(context , std::istream &) and you are good to go. However, I found the save method to be a very expensive operation and I am trying to see if writing to memory which is another way to save the ciphertext would make any difference here.

I found the save method to be a very expensive operation

@uomprogrammer You are right. One thing is that if ZLIB compression is turned on, this serialization includes compression. Another thing is that we are optimizing the serialization itself, since it always unnecessarily wipes the memory buffer after serialization.

One thing is that if ZLIB compression is turned on

@WeiDaiWD I appreciate pointing this out, I just tried compr_mode_type::none and it does make a huge impact. Indeed, I was able to get a high speed to save the ciphertext when I turned the compression off. Save was taking 130 seconds for 1.6 GB of Ciphertext and now it takes 0.8 of a second with a bit larger Ciphertext at only 1.9 GB which is amazing in my case.


Multivarainte fit

ENH: Implement multivariate_normal.fit.
Implement multivariate_normal.fit, MLE was the method of choice
for the Implemention also "Closes #15676".

Reference issue

What does this implement/fix?

Additional information

last test had sample size which way too small, i've increased it's size from 500 to 1000 which on my machine fixed the assertion error

i'll have to look into why this error occurred
'isfinite' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''

the problem boils down to in ability to access cov of a multivariate_normal_frozen during test, do you know how could it be done?
the error given to when i tested the method
AttributeError: 'multivariate_normal_frozen' object has no attribute 'cov'


Update broken link in README.md

Updated link (highlighted line number).
It was pointing to Line number of a old version of the file.

Can one of the admins verify this patch?


Add support for black chords

This is the first pass at adding support for the "black chords" (#8).
This switches us over to Tone.js, to allow more easily expanding the chords. The Tone.js sampled piano sounds a little different from the original recorded versions, so we leave that available as a "legacy" mode.
Still to-do in this PR:

[x] Add chord names (notes) to the flags when on a "black chords" level or greater
[x] Improve the UI in the  "music trainer" grid, which is now expanded to 14 chords.
[x] Figure out why the hidden "Red" chord option is still showing up on iOS

I will leave it to another PR to figure out a better UI for training with the 10 "black chord" inversions.

Made some progress on the UI for displaying chord names. This is what I have so far:

I'm not 100% satisfied with it, but it is a good start I think.

Figure out why the hidden "Red" chord option is still showing up on iOS

OK, this seems to be an issue in Safari. I will either just leave it as is and the Safari users can just always have a weird "Level 0" option in the drop-down or I can try to go back to the old way where the option is added and removed programmatically by the Easter egg.

I decided to add shapes to each of the note names, to help the kids remember them as sequences (you can turn off the shapes or the letters):

I'm not totally satisfied with how this came out, but I think it's good enough to ship at this point. I would like to put a border around the shapes, but the way I added the shapes was using clip-path, and it seems like it's annoyingly difficult to get nice-looking borders around clipped paths and I don't want to burn half a day on this.

OK, so I know why the red chord thing is showing up on iOS, but I'm going to leave that for later to fix. I think everything else is ready to go.


instructions

Thank you for your research, I was wondering if there are any instructions on how to use the code. Thank you

Hi @floeshak, thank you for your comment. We have provided the code used to develop our EGNN models in the 'src' directory. The 'egnn_model.py' script calls the relevant scripts for eg graph generation and the E_GCL layers. Do you have a specific question about how to use a certain part of the code?

Hi @floeshak, thank you for your comment. We have provided the code used to develop our EGNN models in the 'src' directory. The 'egnn_model.py' script calls the relevant scripts for eg graph generation and the E_GCL layers. Do you have a specific question about how to use a certain part of the code?

Well, I have some doubts. When I was trying to run your code, there were various errors related to version mismatching, I had to make a fresh environment and install every package manually, and you definitely have to remove unnecessary packages from the environment 'yaml' file. Also, the packaging was not good, even after adding directories to SYSPATH, PYTHONPATH, adding os.system(<ADDING CURRENT DIR TO PATH CMD>) in script, the egnn_model.py doesnt import modules, so i have to move the scripts out of directories and sub directories. Well, coming to the main issue, when I tried to write my custom training script I faced all these issues aforementioned, I want to ask you,

When will you release the training script?
Can I use the Data provided in the repo as my training data? (because we don't posses a large storage space to store all of the synthetic dataset)
Can you release a mini data subset, so people can have a lite or small version of the model trained?

Hi  @ELMIAR-0642, I am sorry to hear you had difficulties, thank you for letting me know.
I have addressed your questions in the separate issue you raised.


add stopping and changing VMs

@minorczyka did you mean to use this embedding https://www.geeksforgeeks.org/embedding-interfaces-in-golang/ on the meeting? cause I'm not sure if that's it

@minorczyka did you mean to use this embedding https://www.geeksforgeeks.org/embedding-interfaces-in-golang/ on the meeting? cause I'm not sure if that's it

I meant that you can embed interfaces in structs: https://golang-examples.tumblr.com/post/132927916289/embedded-interface

I changed the return types of ChangeMachineType and StopInstance so that would return error rather than crash immediately. I should probably do the same for list recommendations?

I changed the return types of ChangeMachineType and StopInstance so that would return error rather than crash immediately. I should probably do the same for list recommendations?

Yes you can do it right now.

If you could also add license headers to all files, I forgot about it in the previous PR and I think we're supposed to have them.


Alamofire 5: Incorrect documentation instructions for default HTTP headers

[x] I've read, understood, and done my best to follow the *CONTRIBUTING guidelines.

The documentation has this paragraph which talks about a non existing property of URLSessionConfiguration
If you need to customize these headers, a custom URLSessionConfiguration should be created,
the defaultHTTPHeaders property updated, and the configuration applied to a new Session instance.
Use URLSessionConfiguration.af.default to customize your configuration while keeping Alamofire's default headers.

What did you do?
let configuration = URLSessionConfiguration.af.default
configuration.defaultHTTPHeaders

What did you expect to happen?
I expected the defaultHTTPHeaders property to be present in the configuration
What happened instead?
Got an Xcode error saying Value of type 'URLSessionConfiguration' has no member 'defaultHTTPHeaders'
Alamofire Environment
Alamofire version: 5.2.2
Xcode version: 11.6
Swift version: 5.3
Platform(s) running Alamofire: iOS
macOS version running Xcode: Mac OS X 10.15.5
Demo Project
N/A

Feel free to file a PR to correct the documentation.


New Tractus-X project contributor

Before requesting the project contributor role, please make sure, that you:

Make yourself familiar with the Eclipse Foundation project roles: https://eclipse-tractusx.github.io/docs/oss/contributor-committer
Make ourselves familiar with prerequisites, like a signed ECA: https://eclipse-tractusx.github.io/docs/oss/getting-started
Check the committer list of our project and highlight at least one of them on this issue, since every committer
can add project contributors: https://projects.eclipse.org/projects/automotive.tractusx/who

Necessary Information
Email address to add:
Committers, that could take over:
Please make sure, that you accept the invitation you receive from GitHub, after you have been added as project contributor.

Hi  @Gitleena  and @klaudiaZF  do you know maybe a committer in your ZF round?
Because on boarding a new contributor could also be done by any committer

and we need an email adress from you @Gitleena  😉

@FaGru3n  Hi Fabian, we followed the steps from email that we've got from Kelaja. It supposed to add Gitleena to Tractus-X Organization and not to create committer ticket...I think there was a mistake in those steps... sorry for confusion. We will delete this :)

Hi @klaudiaZF jep your totally right here but just want to inform that more people can add @Gitleena as a Contributor into the Eclipse Tractus-X GitHub Organisation.
if you have the generated Eclipse Foundation Accounts even i can add her, but the isse templates mentioned a committer who can do that ... and it was missing. 😉 so... some information are needed an we can onboard...

@FaGru3n  Last time I just asked Angelika through email to add me to Tractus-X Org and that was it. The steps we've got from email was quite misleading;)  what information do you need?
Here's an email of @Gitleena<EMAIL_ADDRESS>

@FaGru3n Last time I just asked Angelika through email to add me to Tractus-X Org and that was it. The steps we've got from email was quite misleading;) what information do you need? Here's an email of @Gitleena<EMAIL_ADDRESS>
you can find the real process on our web-page https://eclipse-tractusx.github.io/docs/oss/getting-started

So @Gitleena you will get an invite to join Eclipse Tractus-X GitHub organisation.
will look like

For next time please ❤

Check the committer list of our project and highlight at least one of them on this issue, since every committer
can add project contributors: https://projects.eclipse.org/projects/automotive.tractusx/who

Because as already stated every committer can invite new contributors.

Please verify @Gitleena and @klaudiaZF and close the issue afterwards.
Greetings

Hello @FaGru3n , I didn't received any email regarding invitation to join Eclipse Tractus-X GitHub organisation. What to do next please let me know.

removed and added you again, i guess the process search for leenagujre9 in github and found nothing... maybe you can check you Eclipse Account if it is linked to you GitHub account @Gitleena

Add your GitHub ID to your Eclipse profile (Section: Social Media Links)

Hello @FaGru3n,
I signed the agreement for Eclipse Contributer Foundation. I think now you need to add me again. My username for Eclipse Foundation is leenagujre9

Hi @Gitleena added you again.

Hi @Gitleena
opened an Gitlab Issue for you that Eclipse Foundation can also Check on further details.
https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues/3994
Generall Info: If you Eclipse Account is working you can ask the eclipse foundation also on Gitlab.
Did you also checked out your Spam Mailfolder? Happen sometimes.

Hi @FaGru3n
I checked Junk folder but there is no email for invitation. I am able to login Eclipse account.

Info from Eclipse Foundation

I can't find any github handles in the eclipse profile. Can you ask the contributor to add this information?

Could you please add your GitHub Handle @Gitleena  into you Eclipse Foundation account see https://github.com/eclipse-tractusx/sig-infra/issues/356#issuecomment-1818933482

Add your GitHub ID to your Eclipse profile (Section: Social Media Links)
https://eclipse-tractusx.github.io/docs/oss/getting-started

Hi @FaGru3n
I have added my GitHub ID to Eclipse profile (Section: Social Media Links)

Hi @FaGru3n and @klaudiaZF
I got the invitation email and now I am part of eclipse-tractusx organization.
Thank you


Add option for arrow-key-based user controls

The follow-up after feedback on #299.
Adds what I called "shortKey" commands (a single string, instead of using spaces for parsing commands vs arguments. Essentially:
'turn left 250' --> '[left arrow]250'
'climb 5' --> '[up arrow]5'
etc...
All old commands are still intact, but this method can be enabled through a game options setting ("Control Method" --> "Classic" or "Arrow Keys").
In total, new commands include:
'[up arrow]' for climbing
'[down arrow]' for descending
'[left arrow]' for turn left
'[right arrow]' for turn right
'fh' for fly-heading
'[numpad +]' for increase speed to xxx
'[numpad -]' for decrease speed to xxx
'*' for cleared ILS XX approach
I tested it thoroughly to make sure nothing was broken, and it all seems to be in order.
Feel free to try it out: http://erikquinn.github.io/atc/

Looks good, I especially like additions to the tutorial :+1:

I don't see a good quick way to make the short keys work well given the distinction between left and right.  Given it's a code issue rather than gameplay I see no reason to wait for the merge.  Great work @erikquinn :+1:


clean plotting

Removes duplicate of box and whisker plot definition

@fritjof-b , Something is wrong with the formatting


Admins don't have admin access on staging

@thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox @thatoddmailbox

@medensqui SORRY I FIXED IT YAY


Not working

Not working on Storyboards.
2016-09-03 12:44:44.988 DMF[64902:2375215] Unknown class MultiStepRangeSlider in Interface Builder file.
2016-09-03 12:44:44.989 DMF[64902:2375215] Failed to set (trackLayerHeight) user defined inspected property on (UIView): [<UIView 0x7feede04cde0> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key trackLayerHeight.
2016-09-03 12:44:44.990 DMF[64902:2375215] Failed to set (trackTintColor) user defined inspected property on (UIView): [<UIView 0x7feede04cde0> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key trackTintColor.
2016-09-03 12:44:44.990 DMF[64902:2375215] Failed to set (trackHighlightTintColor) user defined inspected property on (UIView): [<UIView 0x7feede04cde0> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key trackHighlightTintColor.
2016-09-03 12:44:44.990 DMF[64902:2375215] Failed to set (trackCurvaceousness) user defined inspected property on (UIView): [<UIView 0x7feede04cde0> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key trackCurvaceousness.
2016-09-03 12:44:44.990 DMF[64902:2375215] Failed to set (thumbSize) user defined inspected property on (UIView): [<UIView 0x7feede04cde0> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key thumbSize.
2016-09-03 12:44:44.991 DMF[64902:2375215] Failed to set (thumbTintColor) user defined inspected property on (UIView): [<UIView 0x7feede04cde0> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key thumbTintColor.
2016-09-03 12:44:44.991 DMF[64902:2375215] Failed to set (shadowEnabled) user defined inspected property on (UIView): [<UIView 0x7feede04cde0> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key shadowEnabled.
2016-09-03 12:44:44.991 DMF[64902:2375215] Failed to set (thumbCurvaceousness) user defined inspected property on (UIView): [<UIView 0x7feede04cde0> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key thumbCurvaceousness.

And cannot manually instantiate it given the fact that ALL properties are private inside this POD.

Have you set module as "MultiStepSlider"?


[Feature Request] Start without indicator

Hi there. I'd like the ability to start the program without having it create an indicator. I want to use a keyboard shortcut to just open and close the window, be it with a script or a toggle argument.

Ability to run without indicator added in version 2.2.0


Pinned classnames dependency causes bloat

Can we add a caret to this dep? https://github.com/react-bootstrap-table/react-bootstrap-table2/blob/master/package.json#L83
react-bootstrap-table-next is the only dependency in the application I'm currently debugging that decides to use its own special version of classnames:
# yarn why classnames
yarn why v1.16.0
[1/4] Why do we have the module "classnames"...?
[2/4] Initialising dependency graph...
[3/4] Finding dependency...
[4/4] Calculating file sizes...
=> Found<EMAIL_ADDRESS>info Has been hoisted to "classnames"
info Reasons this module exists
   - Hoisted from "react-bootstrap-table#classnames"
   - Hoisted from "react-date-range#classnames"
   - Hoisted from "rc-drawer#classnames"
   - Hoisted from "recharts#classnames"
   - Hoisted from "react-big-calendar#classnames"
   - Hoisted from "react-big-calendar#react-overlays#classnames"
info Disk size without dependencies: "32KB"
info Disk size with unique dependencies: "32KB"
info Disk size with transitive dependencies: "32KB"
info Number of shared dependencies: 0
=> Found<EMAIL_ADDRESS>info This module exists because "react-bootstrap-table-next" depends on it.
info Disk size without dependencies: "40KB"
info Disk size with unique dependencies: "40KB"
info Disk size with transitive dependencies: "40KB"
info Number of shared dependencies: 0

It's fixed, I will publish soon, thanks

Thanks @AllenFang  I just noticed there is already an issue for this https://github.com/react-bootstrap-table/react-bootstrap-table2/issues/1110
so I'm closing this issue.


[DNM] Chore/map acceptance test

This PR suffers from the same problems in #27 - we still can't run more than a single acceptance test.
In addition, by trying to actually test the map, we now get the following error every time in PhantomJS (but not in Chrome):
Global error: Script error. at , line 0

JSHint | unit/services/esri-loader-test.js: global failure
    ✘ Script error.
        :0
Script error. at , line 0

NOTE: that error was happening in ad5c133 - when the acceptance test didn't even try to wait for and inspect the map. In other words, the error occurs simply by loading the JSAPI modules, not even using them. Which sucks, b/c it was such a royal PIA to write a test helper to wait for the map to load.
I'm so f'ing over Ember acceptance tests and the JSAPI.

As of 0d9bd076c4e1022dea105a3f1c3546dda0084eeb we can run more than one acceptance test.
I tried DRYing up the error handling code but was having problems. Besides, I ultimately want to move that code upstream into esri-loader in https://github.com/Esri/esri-loader/issues/28
I'm still seeing the global script error in phantom. I'm going to try moving the loadModules code into the route model() hooks in hopes that that fixes it. If that doesn't I'll configure the tests to only run in Chrome, maybe FF or headless Chrome in CI and we can merge this.

I would have had to re-arrange most of the dummy app to run loadModules() in the route hook, so I abandoned that, and any hope of running tests in PhantomJS. Phantom's so 2016 anyway.
I configured the tests to run in FF in CI now. Both ember t and npm test pass locally in FF and they've passed in Travis, so I'm going to merge this PR and assume that will update #27 and we can merge into master from there.

@tomwayson I wouldn't worry about phantom. It's a deprecated library and I'd expect most folks to be moving towards headless chrome at this point.

@dmfenton speaking of... #30 has DMF written all over it. 😉


rubocops/lines: allow a runtime Python when supporting multiple Pythons

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Draft for now as unable to test out on Sonoma and not too sure about RuboCop logic. Maybe there is better way to handle.
Main thing is to handle recent changes to various formulae where multiple Python libraries were built and runtime dependency was dropped resulting in non-functioning executables installed/linked into user PATH, e.g. pyinvoke (https://github.com/Homebrew/homebrew-core/issues/151248), python-tabulate, python-argcomplete, etc...

This makes sense in isolation.
Though it does seem problematic when we have any dependents using it as we'll get mixed runtime Pythons again and they will now need to be migrated as one.
Alternative solutions could be:

Have some shim that automatically picks whatever supported Python it can find in the PATH.
Separate executables into a separate formula (perhaps tricky given we don't support subpackages)

Though it does seem problematic when we have any dependents using it as we'll get mixed runtime Pythons again and they will now need to be migrated as one.

As I recall, we ignore mixed Pythons in dependency tree (#13714) so migration should still be possible separately.

Alternative solutions could be:

Introduce a shim executable that automatically picks whatever supported Python it can find in the PATH.

I've thought on this before but seemed like it would end up a bit hacky. Open to discussion if there is a reasonable way. However, some formulae will still be non-functional out-of-the-box unless user installs a python separately (which may require propagating caveats to impacted formulae).

Separate executables into a separate formula (perhaps tricky given we don't support subpackages)

We've tried this before and reverted those formulae now (e.g. python-tabulate + libpython-tabulate). It often required various workarounds to support.

As I recall, we ignore mixed Pythons in dependency tree (#13714) so migration should still be possible separately.

It's not a great user experience though for a formula to depend on two versions of Python.

It's not a great user experience though for a formula to depend on two versions of Python, particularly when only one is really needed.

Ideally, should only happen for short time during migration.
We probably should discuss/document further on Python package distribution as we don't seem to be following our original guidelines, e.g. https://docs.brew.sh/Python-for-Formula-Authors

I'm fine with this change if it's preferred, but it does feel like there's better solutions.

I am open to any other ideas. Just that during current<EMAIL_ADDRESS>--><EMAIL_ADDRESS>migration we have impacted more formulae than previous migrations so may end up seeing more user issues.

An opposite approach would be preventing "multiple pythons" for formulae whose functionality depends on the bin scripts.
For example,

pyinvoke workflow needs bin scripts - https://docs.pyinvoke.org/en/stable/getting-started.html#defining-and-running-task-functions (https://github.com/Homebrew/homebrew-core/issues/151248)
I think docutils also commonly uses bin scripts - https://docutils.sourceforge.io/README.html#usage

On other hand, there are some formulae that have bin scripts that are not commonly used (e.g. python-tabulate).

We probably should discuss/document further on Python package distribution as we don't seem to be following our original guidelines, e.g. https://docs.brew.sh/Python-for-Formula-Authors

If we can turn any of those guidelines into code/audits/rubocops: that would be ideal.

An opposite approach would be preventing "multiple pythons" for formulae whose functionality depends on the bin scripts.

This makes sense to me 👍🏻

Will close for now until we finalize on how we want to handle Python formulae moving forward (especially given adoption of PEP 668 in<EMAIL_ADDRESS>and other changes ongoing in homebrew-core).


Fix issue with misc/tools/survey.py causing multiple copies of bulk_atacseq records

We know that the actual records in the database contain a double link in the associated Neo4J graph.  Somehow this seems to trigger the survey.py graph walk to massively replicate the resulting record.

Fixed by https://github.com/hubmapconsortium/ingest-pipeline/pull/326 (actually in new_dataset_survey).


Basic path fixes and app_label changes

Coverage remained the same at 98.744% when pulling 7be2c02f0080b0500b208140d0bad06f904131e9 on hiteshgarg14:master into b38b6ddb992f16c5d6aab656206e686ca9b1a48f on opencivicdata:master.

@jamesturk
This PR is ready for review.

that directory name should be opencivicdata, my mistake!

actually, the directory was renamed for core and legislative apps both. so I got confused. :+1:

Coverage remained the same at 98.744% when pulling 8ba7365cfbe4ae7b21b7cd22e8a7a71048b73ef3 on hiteshgarg14:master into b38b6ddb992f16c5d6aab656206e686ca9b1a48f on opencivicdata:master.


Java grpc auto tune is disabled sets transport window size to 1mb

Java grpc has auto tune transport window disabled and its default window size is 1 mb. While c grpc has auto tune enabled and adjust the initial window to around 4mb or more.

FYI @dpcollins-google

@ejona86, do you have any perspective on why the gRPC Java default and the C settings would be different (as per the OP)?

"Because they are different implementations."
Before 1.0 benchmarks showed a larger window (~1 MB) was necessary for proper cloud bigtable performance. Thus Java increased the default window size, as that's all there was time to do.
Later, I designed the auto tuning and it was implemented in Java. But GFE has some silly PING restrictions that require working around, so it was left disabled. C had some users complain about performance, so the auto-turning was implemented in C. C tracks GFE's state and avoids sending PINGs if it would conflict with GFE's restrictions; however, this was implemented improperly and has caused issues in the past; it may be right now. Go was last, and used a technique to "trick" (it can happen quite naturally, which is why I say the behavior was silly) GFE into resetting its PING restrictions.
But nobody has complained about Java performance or memory overhead or push-back delay (to my surprise) and there have been active fires elsewhere, so there's not been effort to finish the work. It may be that users that are impacted by these negative things are simply content with changing the window size in NettyChannelBuilder.

@ejona86, thanks for the explanation.
I think we should revisit the Cloud Bigtable performance question relating to the window settings.
FYI, right now, there are concerted efforts to improve performance of Pub/Sub and we'll probably do more testing in the near future.

@sduskis, I'm not sure what you're saying. Are you saying that some further performance investigations are showing the window size may need to be larger than 1 MB? Or are the problems that the current value is too large?
In either case, instead of changing the current value we'd probably need to try to enable window auto-tuning. That would start as 64 KB and increase the size as necessary (like Go and C).

@ejona86, I guess there isn't something for you to do with the window size.  I guess I'm just musing out loud to see if we ought to test different window sizes in our internal tests.
I think that we shouldn't continue discussing this on google-cloud-java.


KeyBindPress and KeyBindDown will fire only for ct KeyBinds. You have to use the newly added '.equals' method to compare the returned key with your KeyBind, or you can use the 2nd argument which is the keycode, again this is only for ct KeyBinds.
(note: none of these triggers will fire inside GUIs)
Example usage:
let bind1 = new KeyBind("Bind 1", Keyboard.KEY_G);
let bind2 = new KeyBind("Bind 2", Keyboard.KEY_H);

register('keyBindPress', (key, keycode) => {
	print(keycode);
	if (key.equals(bind1)) print("Bind 1 has been pressed");
	else if (key.equals(bind2)) print("Bind 2 has been pressed");
});

register('keyBindDown', (key, keycode) => {
	print(keycode);
	if (key.equals(bind1)) print("Bind 1 is held down");
	else if (key.equals(bind2)) print("Bind 2 is held down");
});

register('keyPress', (keycode) => {
	print("Key Pressed: " + keycode);
});

register('keyDown', (keycode) => {
	print("Key Down: " + keycode);
});

Closing pull request as @Squagward made a much better implementation. Squagward's PR


Documentation cleanup

Our docs are in pretty bad shape: poorly organized and outdated. This does some significant cleanup, and imports (and organizes) several of our example notebooks.

[x] Bring current (and useful!) examples into the openpathsampling.org docs
[x] Modify the website so that examples are very easy to find
[x] Move a few notebooks that are pure documentation from examples to the docs
[x] Move less useful "miscellaneous" docs to the background (a "miscellaneous examples" page)
[x] Remove things (e.g. Langevin example) that have no pedagogic value (or instead shift to miscellaneous)
[x] Start developer documentation: explanations of the code to help new devs. Might also be useful in writing paper 2.

Must merge after #571.
The first few tasks address @jchodera's remaining concerns about the examples directory (after the stuff from #588 is merged), so this PR closes #490.

This is ready for review. Only affects the docs and reorganizes some examples (added some docstrings to the code, too.)
The docs are still far from perfect, but I suspect that will be an eternal goal. However, this is enough to move me away from my previous statement to people who were interested in OPS, which was "uh, go to the GitHub site, not openpathsampling.org." Now I think openpathsampling.org will be a better place to start than the GitHub examples.
Among the changes: most examples are now on the website, organized as "introductory", "advanced", "special topics", and "miscellaneous." Of course, we can switch examples from one to another within those.
Our sidebar used to contain a bunch of specific API stuff (e.g., "Storage Functions", "PathMover functions", etc.) I moved all of that to the heading "OpenPathSampling API". Now our sidebar consists of: "Installation," "Examples," "Guides," "Topics," "What's New?," "Frequently Asked Questions," "Overviews for Developers," and "OpenPathSampling API." All the old information is still available; I just tried to minimize what the user sees on first sight.
The "Overviews for Developers" is a new addition, which I've only started. I will also need help from @jhprinz to complete it (especially the storage and CV sections). These are explanations of the code aimed at, e.g., our ESDW developers. The "Code structure" section, which is a broad overview, is pretty close to done -- others, less so.

Can you compile the docs into a PDF and upload that? I think it's just make latexpdf.

Looks like it doesn't work. I think LaTeX doesn't like our notebook directive:
Exception occurred:
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/sphinx/writers/latex.py", line 2015, in unknown_visit
    raise NotImplementedError('Unknown node: ' + node.__class__.__name__)
NotImplementedError: Unknown node: notebook_node
The full traceback has been saved in /var/folders/n1/lvkvzv_j57d92rnh6d4qn_zh0000gn/T/sphinx-err-ceMq__.log, if you want to report the issue to the developers.

The full log wasn't much more informative than that. I think our notebook directive is something that @jhprinz wrote/found, so it might have only included HTML support.
(BTW, I wasn't entirely clear on what you meant @jchodera : I assume you meant to attach it to a GitHub comment, since reading PDF is a lot easier than these miscellaneous RST diffs -- is that correct?)

The full log wasn't much more informative than that. I think our notebook directive is something that @jhprinz wrote/found, so it might have only included HTML support.

This was inspired by what was used in mdtraj and it was a quick afternoon hack to include the notebooks. Maybe its time to make this a little better and robust against these changes.
Otherwise I will merge this now. I will make a quick issue about the PDF docs.


Gatt exchange mtu event not work

I am trying to use this lib to create gatt server, it works. But no matter how I send the exchange MTU event, its MTU always remains at 517.
And I find in application.c line:1035  has one todo : deal with the offset & mtu parameter
So is this a known issue, and is there any simple solution?

I have been looking into this but I don't think Bluez has an API for this.

Yes. I found a link discussing this thing : Determine ATT MTU size over DBUS
Now I have another problem. How can I get the negotiated MTU size from GattCharacteristic?
A simple solution is get from ReadValue or WriteValue method options. But it's not a good solution. Do you have any idea?

Using the MTU from the ReadValue and WriteValue options parameter is totally fine to use. However, that doesn't enforce a MTU. It simply says what the current MTU is....


Issue - Skills show extra dash

Get rid of extra dashes
Screenshots

fixes https://github.com/hidaytrahman/rezume/pull/15/commits/71fd9f58b864621ed9f88cdad8fa1b3ef3e92091


Add some minor improvements.

Added an ImportConfig function which will accept a parsed configuration
file as a struct.
Added a SetHTTPTransport function which will enable setting a HTTP
transport for tweaking low-level HTTP functionality.
Added a SetCookieJar function which enables using a custom cookie jar
implementation.

Perfect, Thank you.


gem tinkerer disappears

When Conjuring (1.0.12) is used on the modpack "Medieval Minecraft". Using the gem tinkerer with shift+right click made it disappear.

Update to latest version, this is a duplicate of #20 and has been fixed.


Compatibility with Angular-Translate

When you use ng-busy with Angular-Translate or any kind of variable from scope inside the  tag it prints the raw content instead of value of the variable.  This commit fix that issue allowing variables and filters inside the button.
Best regards

Well, this broke some stuff, so we need to back up here. Can you provide a sample or test case highlighting the issue?

Sorry about that, we're running a project and just intent to collaborate.  We're using render this template:

 {{'please-wait' | translate}}
{{'button-signup' | translate}}

With the original version what appears in button is {{'button-signup' | translate}} instead "Sign up" for example.


e2e Tests: CI: Setup test to run for PR pointing to master

Summary
Fixes: #4181
Integrate E2E tests with Cypress on PR to master.
What was fixed, or what changes have occurred
Merge to master will be prevented if tests are not passing. It may be skipped if necessary with [skip-e2e]
Cypress E2E have been fixed, including race conditions for navigation tests:

Since now in the sidebar all the accordions within a cluster will collapse on navigation to another group, the test is unfolding it prior interaction to the menu
In the sidebar tests we did not wait to complete navigation before opening and click to the menu item

Technical notes
Tests

Fixed breaking E2E tests

Use Restore hidden cards while verifying the home screen, so it works for already existing instances
Updated behaviors for side navigation to re-open the menu every time we access to the URL
Corrected redirection for login to ignore ?local as parameter
Removed flaky burger open condition
Corrected race condition issue while opening side navigation by introducing manual waiting time It seems like in CI the amount of waiting time is over 4s

Created

Command cy.getId() (check docs for details)
Welcome and first login tests
Cards restore
PO

Checkbox
Radio
Radio group
Page actions

Updates

Extended form PO
Replaced classes over data-testid to avoid need of rewriting tests due restyling adjustments
Replaced form item PO selection with specific data-testid value
Renamed test suits with better and aimed descriptions
Renamed main side menu test file
Renamed methods with actual meaning
Added documentation and types to the methods

Disabled tests
Every test which resulted to be unreliable have been set as skip.
Most of the disabled tests are related to the navigation, as it seems like some resources are not loaded in time to allow interaction with it (example with worse case) and I am not able to identify the issue.
As the tests were already failing and flaky and blocking the integration to CI, including possibility for other PR to be closed, these tests will require refactoring in a separated ticket.
Issue to reintegrate them: #5966
Project configuration

Created GitHub Action configuration for PR request, generating check
Allowed to manually trigger E2E tests to a specific environment
Updated Cypress related libraries
Added E2E related scripts in package.json (check docs and file changes for details), for production DEV_PORTS=true NODE_ENV=production yarn start --spa
Changed DEV_UI to TEST_BASE_URL for Cypress base URL
Added env var TEST_SKIP_SETUP to allow DO instances to run E2E by skipping bootstrapping tests
Added command for Docker bootstrap
Allowed to skip entirely E2E tests using label ci/skip-e2e in the pull request (log)

Global

Made checkbox input component visible and able to interact
Allowed radio groups to pass IDs to children options
Extended E2E documentation and adjusted commands/env comments

Known issues

Custom side navigation, e.g. NeuVector case https://github.com/rancher/dashboard/issues/5948

Areas or cases that should be tested

This very PR request should pass all the new checks, which means pass all the current E2E tests
E2E tests should pass on a local client with both remote and local Rancher instance
Test check should not be present after adding label ci/skip-e2e; it seems not possible to trigger the check without push
Re-run CI jobs several times to ensure lack of flaky tests, link in the details of the check below

What areas could experience regressions?
None.
Screenshot/Video
Videos are generated as (zipped) artifacts from the tests.

@richard-cox FYI at the current state, if I test the E2E to master I have this result:

       Spec                                              Tests  Passing  Failing  Pending  Skipped
  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ ✖  tests/home.spec.ts                       00:17        2        1        1        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✖  tests/login.spec.ts                      00:14        2        1        1        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✖  tests/side-nav/burger-side-menu.spe      00:26        5        4        1        -        - │
  │    c.ts                                                                                │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✖  tests/side-nav/product-side-nav.spe      01:20        5        3        2        -        - │
  │    c.ts                                                                                │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘
    ✖  4 of 4 failed (100%)                     02:18       14        9        5        -        -

Login optimization in the tests to be handled with https://github.com/rancher/dashboard/issues/5955

@richard-cox I've not found a way to make these tests work. This is the first time I managed to get the green check for the E2E after multiple runs. If you have any idea on how to integrate back the tests we can do so.

Created issue to review flaky tests: #5966

Seems like artifacts have been switched to default on success only, not sure how's possible 🤔
However now it's set on failure only.

Seems like we had a change in the packages and now the report is visually broken and a test is failed.

@richard-cox they released Cypress 10 just last week.
It may be convenient to take a look to update to the latest version already, while we are still in progress.
The new feature worth mentioning is the configuration extended from JSON to JS and even TS.
https://cypress.io/blog/2022/06/01/cypress-10-release/
https://docs.cypress.io/guides/references/migration-guide#Configuration-File-Changes