Adapt CLI to use Theia Manager

This PR implements the adaption of CLI in order to use Theia Manager, which includes:

Create a theia client with the ca.cert and token.
Use port-forwarder to connect to Theia Manager when running theia out of the cluster.
Add unit test.

Reviewers can focus on the changes under folder pkg/theia/commands.

Codecov Report

Merging #115 (7c8db42) into main (8a3826a) will increase coverage by 21.11%.
The diff coverage is 47.92%.

@@             Coverage Diff             @@
##             main     #115       +/-   ##
===========================================
+ Coverage   30.65%   51.76%   +21.11%
===========================================
  Files          11       11
  Lines        1442     1161      -281
===========================================
+ Hits          442      601      +159
+ Misses        978      482      -496
- Partials       22       78       +56

Flag
Coverage Δ

unit-tests
51.76% <47.92%> (+21.11%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/theia/commands/utils.go
37.03% <35.10%> (+6.52%)
:arrow_up:

pkg/theia/commands/policy_recommendation_run.go
61.77% <41.91%> (+35.84%)
:arrow_up:

...g/theia/commands/policy_recommendation_retrieve.go
54.71% <42.85%> (+30.54%)
:arrow_up:

pkg/theia/commands/policy_recommendation_delete.go
53.84% <57.14%> (+45.68%)
:arrow_up:

pkg/theia/commands/policy_recommendation_list.go
70.27% <75.00%> (+68.09%)
:arrow_up:

pkg/theia/commands/policy_recommendation_status.go
64.70% <78.37%> (+39.03%)
:arrow_up:

pkg/theia/commands/policy_recommendation.go
72.72% <100.00%> (-8.53%)
:arrow_down:

... and 2 more

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

/theia-test-e2e

/theia-test-e2e

/theia-test-e2e

/theia-test-e2e


Call retries were exceeded while build

🐞 Bug report
Command (mark with an x)

- [ ] new
- [x] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Is this a regression?

 No
Description
 A clear and concise description of the problem...
while running npm build --prod, first build done successfully, but at differential loading got the error
[error] Error: Call retries were exceeded
    at ChildProcessWorker.initialize (D:\AccFlex_Cloud_Front\accflex-erb\node_modules\jest-worker\build\workers\ChildProcessWorker.js:193:21)
    at ChildProcessWorker.onExit (D:\AccFlex_Cloud_Front\accflex-erb\node_modules\jest-worker\build\workers\ChildProcessWorker.js:263:12)
    at ChildProcess.emit (events.js:182:13)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:240:12)

🔬 Minimal Reproduction

🔥 Exception or Error

[error] Error: Call retries were exceeded
    at ChildProcessWorker.initialize (D:\AccFlex_Cloud_Front\accflex-erb\node_modules\jest-worker\build\workers\ChildProcessWorker.js:193:21)
    at ChildProcessWorker.onExit (D:\AccFlex_Cloud_Front\accflex-erb\node_modules\jest-worker\build\workers\ChildProcessWorker.js:263:12)
    at ChildProcess.emit (events.js:182:13)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:240:12)

🌍 Your Environment

     _                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 8.3.8
Node: 10.14.2
OS: win32 x64
Angular: 8.2.9
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.803.8
@angular-devkit/build-angular     0.803.8
@angular-devkit/build-optimizer   0.803.8
@angular-devkit/build-webpack     0.803.8
@angular-devkit/core              8.3.8
@angular-devkit/schematics        8.3.8
@angular/cli                      8.3.8
@ngtools/webpack                  8.3.8
@schematics/angular               8.3.8
@schematics/update                0.803.8
rxjs                              6.4.0
typescript                        3.5.3
webpack                           4.39.2

Anything else relevant?

This seems to be related to https://github.com/angular/angular-cli/issues/15493

Let's continue tracking this in #15493. Thanks.


Add mention of retain support

Is your enhancement request related to a problem? Please describe.
Retain semantics were added, however I think it would be useful to mention the support for this and any of the caveats related to this feature. One of the listed caveats is that standalone clusters hold retained messages in memory and they are lost on restart. This might be worth putting into the README.md. This is the PR for the features(#581).
Describe the solution you'd like
Add mention of the retain support in MOP, and also mention any known caveats and maybe brief notes on behavior in the README
Describe alternatives you've considered
n/a
Additional context
n/a

Yes, right. Thanks @tsturzl . Could you help push a patch for this ?
And how about putting it here ?


chore: release 2.10.4

:robot: I have created a release *beep* *boop*
2.10.4 (2024-08-21)
Bug Fixes

build/stages: Add RWMutex to ImagesTree (29855a9)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/werf/werf/releases/tag/v2.10.4 :sunflower:


Build istioctl for istio.io tests

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

Test name
Commit
Details
Rerun command

unit-tests_istio
18dc242fc59251f5af2c14c97a0ae3e8bbcb3347
link
/test unit-tests_istio

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

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

Test name
Commit
Details
Rerun command

unit-tests_istio
18dc242fc59251f5af2c14c97a0ae3e8bbcb3347
link
/test unit-tests_istio

integ-new-install-k8s-tests_istio
18dc242fc59251f5af2c14c97a0ae3e8bbcb3347
link
/test integ-new-install-k8s-tests_istio

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

/retest

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

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
18dc242fc59251f5af2c14c97a0ae3e8bbcb3347
link
/test integ-new-install-k8s-tests_istio

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


Convert token using Implicit Grant Type

Is there a way to convert tokens without exposing the client secret?
The convert-token route is potentially insecure for JavaScript Apps since JS cannot store client_secret securely. From what I've researched, OAuth applications should use Implicit Grant when dealing with JS Apps. Is there a way to convert tokens with Implicit Grant using this library?
Convert Token Route:
curl -X POST -d "grant_type=convert_token&client_id=<client_id>&client_secret=<client_secret>&backend=<backend>&token=<backend_token>" http://localhost:8000/auth/convert-token

On your server, create a new application object.
Set the client type as public and the authorization grant type as client credentials.
This will allow you to omit the client secret when converting a token.


Feature request: Download as PNG/JPEG

It would be great to be able to download as an image rather than an Excel file :)

Hi Nick,
we are planning to add the ability to download the gantt as an png image to to the next version. Meanwhile you can use the browser's default Take Screenshot functionality.

Hi Nick,
there is a new version of the QueryGantt extension out there, the requested feature is included and enables you to download the gantt as an png image.


feat: consistent labels in istio module and its resources

/kind feat
/area service-mesh
/hold
Based on the decision held in https://github.com/kyma-project/community/issues/864 adjusted labels for all resources:

add common label kyma-project.io/module and app.kubernetes.io/version
changed the purpose of a app.kubernetes.io/name label to reflect the resource name
removed app.kubernetes.io/managed-by label because it is handled by external manager
removed app.kubernetes.io/instance label as I think it also should be managed by external manager (TBD)
removed app.kubernetes.io/created-by label as it's not listed in the standard labels proposal

#548

/kind feature


What should I do if after calling extract_random_route, a Depot gets empty?

The utils.functional.extract_random_route method looks for a random route then remove it and return the removed random route.
Now if a Depot has only one route, after calling this method, the route will be removed and the Depot will no longer have any Customers in it to construct a route, actually, the Depot will be an empty object.
What is the instruction to deal with this situation?
Possible behaviours:

If we remove Depot itself, it means we are not using one of the provided Depots in the dataset or problem itself which means we will use less routes (less vehicles). It seems good but does not mean efficient. Note that creating a new Depot would be a new challenge for assigning new routes to a non-existance Depot.
Having a empty Depot won't cost us anything in term of number of routes (vehicles) or the maximum capacity restriction. By the way, adding new routes to the Depot would be much easier so I think this option is much better to be considered. [PREFERED]

It seems no need to do anything, because insert method will update it and even in the process of cross_over, we may introduce similar situation.


Infer datatype in create.df()

If create.df() receive only the column name it can try to infer the data type using the data in the first column.

Can you try with inferSchema=True?

createDataFrame() which is used behind create.df() try to infer the data type by default. But it seems that have problems with this:
import pandas as pd
from pyspark.sql.types import *
from datetime import date, datetime

cols= [
        "names",
        "height(ft)",
        "function",
        "rank",
        "age",
        "weight(t)",
        "japanese name",
        "last position seen",
        "date arrival",
        "last date seen",
        "attributes",
        "DateType",
        "Tiemstamp",
        "Cybertronian",
        "function(binary)",
        "NullType",

    ]
data=[
        ("Optim'us", 28, "Leader", 10, 5000000, 4.30, ["Inochi", "Convoy"], "19.442735,-99.201111", "1980/04/10",
         "2016/09/10", [8.5344, 4300.0], date(2016, 9, 10), datetime(2014, 6, 24), True, bytearray("Leader", "utf-8"),
         None)
    ]

op.spark.createDataFrame(data, cols)

Ir returns
ValueError: Some of types cannot be determined after inferring
I am going to try a couple of things to see if we can handle it with Optimus


App stops working after 1-2h

Describe the bug
I am using the latest version of sveltkit with adapter-node.
The app stops responding after 1-2h of inactivity, the browser keeps spinning without loading the page nor returning any error.
Reproduction
start the app with "node index.js" as usual
Logs
No response
System Info
System:
    OS: Windows 10 10.0.17763
    CPU: (64) x64 AMD EPYC 7502P 32-Core Processor
    Memory: 312.45 GB / 319.87 GB
  Binaries:
    Node: 16.15.1 - C:\Program Files\nodejs\node.EXE
    npm: 8.11.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Internet Explorer: 11.0.17763.2989

Severity
blocking all usage of SvelteKit
Additional Information
There is no apparent reason why this would happen.
The server is just ran normally at port 3000 and accessed via localhost.

Can you add a repo/stackblizz for an example that this occurs in?

this needs a reproduction. this could be anything, for example a memory leak in your application code. also, which browser?

Ok solved, unrelated to svelte.


No instance types returned for any Europe regions on AWS

Describe the bug
All European regions on AWS now return no instance types - see log message numberOfVms: 0:
{"application":"cloudinfo","environment":"production","level":"debug","msg":"getting available instance types from AWS API","provider":"amazon","region":"eu-west-1","time":"2020-08-05T09:38:42Z"}
{"application":"cloudinfo","environment":"production","level":"debug","missingAttrs":{},"msg":"instance types with missing attributes","provider":"amazon","region":"eu-west-1","time":"2020-08-05T09:38:42Z"}
{"application":"cloudinfo","environment":"production","level":"debug","missingGPU":null,"msg":"instance types with missing gpu","provider":"amazon","region":"eu-west-1","time":"2020-08-05T09:38:42Z"}
{"application":"cloudinfo","environment":"production","level":"debug","msg":"couldn't find any virtual machines to recommend","provider":"amazon","region":"eu-west-1","time":"2020-08-05T09:38:42Z"}
{"application":"cloudinfo","environment":"production","level":"debug","msg":"found vms","numberOfVms":0,"provider":"amazon","region":"eu-west-1","time":"2020-08-05T09:38:42Z"}

This is almost certainly caused by the AWS SDK upgrade PR - noting specifically this change in the expected region descriptions - the pricing API still uses 'EU' not 'Europe' in its location field, so this now will not match any European instances:
aws --region us-east-1 pricing get-attribute-values --service-code AmazonEC2 --attribute-name location --output table

----------------------------------
|       GetAttributeValues       |
+--------------------------------+
||        AttributeValues       ||
|+------------------------------+|
||             Value            ||
|+------------------------------+|
||  AWS GovCloud (US-East)      ||
||  AWS GovCloud (US-West)      ||
||  Africa (Cape Town)          ||
||  Asia Pacific (Hong Kong)    ||
||  Asia Pacific (Mumbai)       ||
||  Asia Pacific (Osaka-Local)  ||
||  Asia Pacific (Seoul)        ||
||  Asia Pacific (Singapore)    ||
||  Asia Pacific (Sydney)       ||
||  Asia Pacific (Tokyo)        ||
||  Canada (Central)            ||
||  EU (Frankfurt)              ||
||  EU (Ireland)                ||
||  EU (London)                 ||
||  EU (Milan)                  ||
||  EU (Paris)                  ||
||  EU (Stockholm)              ||
||  Middle East (Bahrain)       ||
||  South America (Sao Paulo)   ||
||  US East (N. Virginia)       ||
||  US East (Ohio)              ||
||  US West (Los Angeles)       ||
||  US West (N. California)     ||
||  US West (Oregon)            ||
|+------------------------------+|

Steps to reproduce the issue:
run cloudinfo with AWS enabled
Additional context
Tested on latest tagged 0.15.0

I might be mistaken, but we got bitten by this a while ago, and the SDK upgrade seemingly solved the issue. Did AWS change it back?

I might be mistaken, but we got bitten by this a while ago, and the SDK upgrade seemingly solved the issue. Did AWS change it back?

I wouldn't be surprised! It'll probably change at some point again.
mumbles something about bloody brexit breaking everything
Thanks for the super-quick fix!

mumbles something about bloody brexit breaking everything

🤣
Yeah, I'll try to add a fix that supports both versions.

Thanks for the super-quick fix!

As part of the release process I'll deploy it to our test env, test it there and then tag 0.15.1 if it works well.

See #36

My reading of the filter spec for the pricing API is that the filters are ANDed, so trying to make it support both could be tricky - I guess calling with one filter then if nothing is returned, calling with the other would solve it.
... while I was typing this, I see you've put the same concept in #367 :+1:


INP attribution: allow for LoAFs observed before interaction events

fixes #485
As described there, I added an extra condition to keep any LoAFs around that are after any existing known interactions, in case those interactions then come in later.
 I added a MAX_UNPAIRED_LOAFS of 20, so it will only keep the 20 most recent LoAFs after any known user interaction events. The number isn't hugely important, that just allows a one second delay in event-timing reporting even if the page somehow has filled the time with the shortest (50ms) long animation frames possible. LoAF entries are fairly light weight, so that's not so bad.
As I was implementing this, however, I realized that even without this change the pendingLoAFs array can grow without bound on a page with e.g. animations and a busy main thread, since LoAFs are appended as they come in but clean up is only scheduled by user interactions. Seems like we should fix? @philipwalton @tunetheweb
My only idea right now is to split the event timing and LoAF cleanups into two separate functions, and run each as their respective entries come in, since that's the only time each list of entries grows.

Oh, also I could not get a test for this working reliably. Inherently flaky and subject to future Chrome scheduler changes anyways. I verified the issue repro no longer fails, but looking for other ideas on this. Time to bust out some unit tests?

As I was implementing this, however, I realized that even without this change the pendingLoAFs array can grow without bound on a page with e.g. animations and a busy main thread, since LoAFs are appended as they come in but clean up is only scheduled by user interactions. Seems like we should fix? @philipwalton @tunetheweb

Maybe we should do this clean up in handleLoAFEntries then (behind a whenIdle call)?
At the moment we queue a clean up interactions when new interactions come in, so we should also queue LoAFs clean ups when they come in.

Updated, and also moved the whenIdle() call to within the main onINP() function (as mentioned here) to ensure that attribution is always processed synchronously with the INP dispatch.


AttributeError: 'voc_clipart' object has no attribute '_year'

In the testing process accounting the following Error:
AttributeError: 'voc_clipart' object has no attribute '_year'

resolved

@yongqiangzhang1 Dear sir,Can you tell me how to solve this problem? Thanks in davance!


BotFramework-WebChat is not support in SPFX Extension (Application Customizer)

Is it an issue related to Adaptive Cards?
No
Is this an accessibility issue?
No
What version of Web Chat are you using?
Latest production
Which distribution are you using Web Chat from?
NPM
Which hosting environment does this issue primarily affect?
SharePoint
Which browsers and platforms do the issue happened?
No response
Which area does this issue affect?
Development experience
What is the public URL for the website?
https://github.com/Hasnii501/app-extension.git
Please describe the bug
I am using SharePoint SPFX extension. When I try to use the chatbot framework, it fails to build the package.
Do you see any errors in console log?
Error - [webpack] 'dist':
./node_modules/micromark-util-decode-numeric-character-reference/index.js 23:11
Module parse failed: Identifier directly after number (23:11)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|   code > 126 && code < 160 ||
|   // Lone high surrogates and low surrogates.
>   code > 55_295 && code < 57_344 ||
|   // Noncharacters.
|   code > 64_975 && code < 65_008 || /* eslint-disable no-bitwise */
 @ ./node_modules/mdast-util-from-markdown/lib/index.js 138:0-97 1061:14-45
 @ ./node_modules/mdast-util-from-markdown/index.js
 @ ./node_modules/botframework-webchat/lib/markdown/private/iterateLinkDefinitions.js
 @ ./node_modules/botframework-webchat/lib/markdown/renderMarkdown.js
 @ ./node_modules/botframework-webchat/lib/index.js
 @ ./lib/extensions/henkelChatbotCustomizer/renderer/Chatbot.js
 @ ./lib/extensions/henkelChatbotCustomizer/HenkelChatbotCustomizerApplicationCustomizer.js
./node_modules/micromark-util-sanitize-uri/index.js 86:22
Module parse failed: Identifier directly after number (86:22)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|     }
|     // Astral.
>     else if (code > 55_295 && code < 57_344) {
|       const next = value.charCodeAt(index + 1)
|
 @ ./node_modules/micromark/lib/compile.js 44:0-55 770:10-21 781:10-21 1089:8-19 1105:22-33
 @ ./node_modules/micromark/index.js
 @ ./node_modules/mdast-util-from-markdown/lib/index.js
 @ ./node_modules/mdast-util-from-markdown/index.js
 @ ./node_modules/botframework-webchat/lib/markdown/private/iterateLinkDefinitions.js
 @ ./node_modules/botframework-webchat/lib/markdown/renderMarkdown.js
 @ ./node_modules/botframework-webchat/lib/index.js
 @ ./lib/extensions/henkelChatbotCustomizer/renderer/Chatbot.js
 @ ./lib/extensions/henkelChatbotCustomizer/HenkelChatbotCustomizerApplicationCustomizer.js
./node_modules/htmlparser2/lib/esm/index.js 48:9
Module parse failed: Unexpected token (48:9)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|  * They should probably be removed eventually.
|  */
> export * as ElementType from "domelementtype";
| import { getFeed } from "domutils";
| export { getFeed } from "domutils";
 @ ./node_modules/sanitize-html/index.js 1:19-41
 @ ./node_modules/botframework-webchat/lib/markdown/renderMarkdown.js
 @ ./node_modules/botframework-webchat/lib/index.js
 @ ./lib/extensions/henkelChatbotCustomizer/renderer/Chatbot.js
 @ ./lib/extensions/henkelChatbotCustomizer/HenkelChatbotCustomizerApplicationCustomizer.js

How to reproduce the issue?

Create new extension of SharePoint Framework (Application Customizer)
Add chatbot framework code in file and install the package for example "botframework-webchat": "^4.9.1"
Call the chatbot file and use the code in extension
Build the project.

What do you expect?
The project should build sucessfully
What actually happened?
Build Failed with error mentioned.
Do you have any screenshots or recordings to repro the issue?

Adaptive Card JSON
No response
Additional context
Below is my Package.JSON
{
  "name": "chatbot-extension-v-1",
  "version": "0.0.1",
  "private": true,
  "main": "lib/index.js",
  "scripts": {
    "build": "gulp bundle",
    "clean": "gulp clean",
    "test": "gulp test"
  },
  "dependencies": {
    "tslib": "2.3.1",
    "@microsoft/sp-core-library": "1.15.2",
    "@microsoft/decorators": "1.15.2",
    "@microsoft/sp-dialog": "1.15.2",
    "botframework-webchat": "^4.9.1",
    "@microsoft/sp-application-base": "1.15.2"
  },
  "devDependencies": {
<EMAIL_ADDRESS>"0.2.2",
    "@rushstack/eslint-config": "2.5.1",
    "@microsoft/eslint-plugin-spfx": "1.15.2",
    "@microsoft/eslint-config-spfx": "1.15.2",
    "@microsoft/sp-build-web": "1.15.2",
    "@types/webpack-env": "~1.15.2",
    "ajv": "^6.12.5",
    "gulp": "4.0.2",
    "typescript": "4.5.5",
    "@microsoft/sp-module-interfaces": "1.15.2"
  }
}

Looks like SharePoint Framework (SPFx) is using an outdated version of toolchain that doesn't support compiling ES Modules and newer code under /node_modules/ folder.
Today, increasing number of packages are ESM-only. For example, the mdast-family package which we are using in Web Chat are ESM-only. These packages expect the package user to either use an ESM-compatible host (e.g. Node.js > 12) or ESM-compatible bundler (e.g. ESBuild, tsup, Webpack 5, etc.)
In SPFx dependencies under https://www.npmjs.com/package/@microsoft/gulp-core-build-webpack?activeTab=code. They are using<EMAIL_ADDRESS>Despite 4.47.0 is released recently, it's a major version behind the current one and is 3 years behind the current version.
Could you file an issue to their repo and ask them to update their toolchain?
To workaround their issue, you can try either:

(Recommended) Compile your package using ESBuild/tsup before using SPFx to compile the final bundle
Overriding their Webpack configuration to include compiling stuff under /node_modules/


Error during Deployment: "metrics-server" version "5.11.7" not found

Hi everyone,
I am trying to deploy Viya, but I get the error below.
Deployment command:
sudo docker run --rm \
--group-add root \
--user $(id -u):$(id -g) \
--volume /deployments:/data \
--volume /home/ec2-user/viya4-deployment/ansible-vars.yaml:/config/config \
--volume /home/ec2-user/viya4-iac-aws/terraform.tfstate:/config/tfstate \
--volume $HOME/.ssh/id_rsa:/config/jump_svr_private_key \
viya4-deployment --tags "baseline,viya,install"

The error I am getting:
TASK [baseline : Deploy metrics-server] ****************************************
fatal: [localhost]: FAILED! => changed=false
  command: /usr/local/bin/helm --version=5.11.7 --repo=https://charts.bitnami.com/bitnami/ show chart metrics-server
  msg: |-
    Failure when executing Helm command. Exited 1.
    stdout:
    stderr: Error: chart "metrics-server" version "5.11.7" not found in https://charts.bitnami.com/bitnami/ repository
  stderr: |-
    Error: chart "metrics-server" version "5.11.7" not found in https://charts.bitnami.com/bitnami/ repository
  stderr_lines: <omitted>
  stdout: ''
  stdout_lines: <omitted>

The Ansible summary:
PLAY RECAP *********************************************************************
localhost                  : ok=57   changed=4    unreachable=0    failed=1    skipped=38   rescued=0    ignored=0

Wednesday 18 January 2023  16:38:03 +0000 (0:00:01.838)       0:00:17.410 *****
===============================================================================
baseline : Deploy metrics-server ---------------------------------------- 1.84s
jump-server : jump-server - lookup groups ------------------------------- 1.32s
baseline : Check for metrics service ------------------------------------ 1.01s
baseline : Retreive K8s cluster information ----------------------------- 0.99s
jump-server : jump-server - create folders ------------------------------ 0.97s
baseline : Remove deprecated efs-provisioner namespace ------------------ 0.96s
baseline : Retrieve K8s cluster information ----------------------------- 0.88s
Gathering Facts --------------------------------------------------------- 0.86s
baseline : Retreive K8s cluster information ----------------------------- 0.83s
baseline : Deploy ingress-nginx ----------------------------------------- 0.70s
baseline : Deploy ebs-csi-driver ---------------------------------------- 0.68s
common : tfstate - export kubeconfig ------------------------------------ 0.67s
baseline : Deploy nfs-subdir-external-provisioner ----------------------- 0.67s
baseline : Deploy cluster-autoscaler ------------------------------------ 0.58s
baseline : Remove deprecated nfs-client-provisioner --------------------- 0.41s
global tmp dir ---------------------------------------------------------- 0.34s
baseline : Remove deprecated efs-provisioner ---------------------------- 0.30s
baseline role install --------------------------------------------------- 0.10s
jump-server role -------------------------------------------------------- 0.08s
common : Load config file ----------------------------------------------- 0.07s

I have previously provisioned the AWS infrastructure for the deployment using Terraform using this project https://github.com/sassoftware/viya4-iac-aws, and I am deploying Viya on that infrastructure.
Would anyone be able to help me resolve this error?
Thank you

I think last chart version is 3.8.3 (check here: https://artifacthub.io/packages/helm/metrics-server/metrics-server).
But it does not work well if kubernetes version is < 1.25 - https://github.com/kubernetes-sigs/metrics-server/issues/1141.
In that case use chart version 3.7.0.

Hi @momiji I changed the variable METRICS_SERVER_CHART_VERSION to use version 3.7.0, but it still can't find it:

Hi again @momiji I managed to resolve the issue.
The deployment was getting the chart version from here https://artifacthub.io/packages/helm/bitnami/metrics-server, so the latest chart version there is 6.2.6. I have set the METRICS_SERVER_CHART_VERSION variable to the default value suggested here https://github.com/sassoftware/viya4-deployment/blob/main/docs/CONFIG-VARS.md#cert-manager which is 6.2.4 and then that allowed me to resolve the error.
Thank you

Great to hear it works, I didn't thought it was a bitnami chart, but you figured it out.
Beware the constraint on the application version, that must not be 0.6.* so far, if you have a kubernetes 1.23 like me.

Based on last comment, user managed to resolve this issue.


Change raw latex to math so equations display correctly

Hello! First and foremost: thank you for the great library.
Some equations seem to not display correctly in the examples, see this page for an example
Before
Here's what it looks like in Firefox:

After
Changing the lines to .. math:: seems to do the trick:

I also have a question. I see that the files below are identical:
./examples/notebooks/WWW/robust_kalman.ipynb
./doc/source/examples/applications/robust_kalman.rst

Question. Is the .rst the result of a manual export from the notebook, or is it automated? If it's not automated, I think it would be better to have the code in one place and automate this.

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

It is automated. Thanks for fixing this!

@SteveDiamond If it's automated, won't this PR be without effect once the notebooks overwrite the .rst files? Where is the automation step? The only file that I found that seems to be exporting notebooks is examples/machine_learning/generate_rst.sh.

We convert the .ipynb files to .rst files using a jupyter function. So it's not generated each time, only when we call the function.

Alright! Where is the jupyter function located?
If I want to read through the examples and do some proofreading as I go, it is meaningless to do it in the .rst files since they will be overwritten? In other words: If I make changes, should I do it in the .rst files or .ipynb files?

Changing the .rst files is correct. They would be overwritten if we changed the .ipynb file and needed to generate the  .rst, but this is only done manually. It's not automated when we build cvxpy or something. We make the  .rst files using ipython nbconvert --to rst notebook.ipynb.


Duplicate plugin 'autoprefixer' warning although there is no duplicate

I use postcss-cssnext with the postcss-loader in webpack. This is my postcss.config.js script:
// PostCSS configuration

const postcssConfig = {
    plugins: {
        'postcss-cssnext': {},
        //'autoprefixer': {} // <-- already included in postcss-cssnext
    }
};

// Compress CSS with cssnano in production
if (process.env.NODE_ENV === 'production') {
    postcssConfig.plugins['cssnano'] = {};
}

postcssConfig.plugins['doiuse'] = {};

module.exports = postcssConfig;

When cssnano is added (in production mode), I get the message "Warning: postcss-cssnext found a duplicate plugin ('autoprefixer') in your postcss plugins. This might be inefficient. You should remove 'autoprefixer' from your postcss plugin list since it's already included by postcss-cssnext." If I comment out the line postcssConfig.plugins['cssnano'] = {};, the message goes away.
Is this a bug, or do I miss something?

cssnano has prefixes optimization which use autoprefixer (http://cssnano.co/optimisations/autoprefixer/) I see that warning is ok.

Alright. So I can savely ignore the warning?

Yeah.

@Semigradsky why would cssnano or cssnext include autoprefixer by default instead of letting me include it when I want to include it?

@yordis because they depend on autoprefixer.

@gflohr right, my point is that autoprefixer is not a have to have, I don't understand from high level perspective why the plugins uses autoprefixer.


Posters & Email Templates Link incorrect

Under "Onboarding related resources", the link for Posters, email templates  directs to https://support.microsoft.com/office/create-and-use-site-templates-in-sharepoint-server-versions-60371b0f-00e0-4c49-a844-34759ebdd989
That link is for how to create and use site templates in SharePoint and does not link to the stated resource.

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

ID: 9ee6b3fe-cc73-cf65-8f91-aa75b2bda8f8
Version Independent ID: a49a2330-4e5b-922e-870c-26aca6f63892
Content: Migrate file shares to SharePoint and OneDrive - Migrate to Microsoft 365
Content Source: migration/Fileshare-to-ODSP-migration-guide.md
Service: sharepoint-online
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

Thanks @fns-it


Issues/added offsite check

This looks good and I'm happy to merge this, but it looks like it was based off the 2.0 branch orginally so it would need to be merged into that. Or you could cherry-pick your last commit there into another branch which is based on master. Either is fine with me.


Improve gradle configuration

Improving Gradle configuration to make build faster
1. First improvement is to update all tasks declaration by using tasks.register()
The proposal will defer task creation when is needed
Here 2 build scans where you can see the improvement

the first for master branch (commit 92173ba6dbc3daadea68e4348e760e99d1e79783), where you can see that during the Configuration, tasks for build.gradle are Created immediately.

on the other side for a scan on my PR branch skurtzemann/improve-gradle-configuration (commit 711d770a2c496f2deb8c8d7426ecd0c13754b4e2) all build.gradle tasks are Created during configuration

2. Second improvement is to update how checkstyle is configure
Instead of using getConfigDirectory() to retrieve checkstyle-suppressions.xml file which will retrieve an absolute path for this file.
The proposal is to inline the configuration, use {config_loc} that will give a relative path. Using that configuration be become Cacheable for next builds

LGTM code wise
I would switch the bullet points in the description as checkstyle modification is the important change.
I wouldn't provide some screenshots to demonstrate how task configuration is deferred, just mention it linking to the doc. You can also mention that the code was not aligned in that regards.
For Checkstyle, I would just say that cache is not relocatable due to config file defined with an absolute path.


Passing batch scheduler through python breaks

I've been working with the websocket script in python for a few months now and have had great success in generating through workflows via scripting. After digging more into animation workflows, I'm running into an issue and I'm hoping someone has a solution.
When passing prompt data, it's written as a frame number and the prompt that occurs at that frame number followed by a comma. That continues until all desired frame numbers are entered each on their own line (return character) and the last line does not need a comma.
For reference:
"0" :"graveyard ",
"16" :"dark cold road, dead trees"

When the data from this node is passed using the websocket python example, the console shows errors that something is wrong with the JSON load due to expected characters. I'm assuming it does not like all the extra ( " , ; ) that exists and breaks the load injected.
I guess my question is does ComfyUI support a different method of listing "frame:prompt," data possibly with characters that won't break the rules of python? Included is some errors pulled from the generation.
data.json

Python parsing issue when using argv's to inject prompt data. Found a workaround.


[Monitor Query] Add notebook ML sample

Detect anomalies in Azure Monitor log data using machine learning techniques.
Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.

Thank you, Paul!
Pls go ahead and push to my branch
Pls let me know once done (and then I need to merge to main branch, right?)
Thanks!
From: Paul Van Eck @.>
Sent: Friday, June 9, 2023 2:56 AM
To: Azure/azure-sdk-for-python @.>
Cc: Ilana Waitser @.>; Mention @.>
Subject: Re: [Azure/azure-sdk-for-python] [Monitor Query] Add notebook ML sample (PR #30682)
@pvaneck commented on this pull request.
Thanks, @IlanaWaitserhttps://github.com/IlanaWaitser. I think this is almost ready to be merged. It's a bit tricky to leave review comments in GitHub because the diff does not render on account of the file being too large.
Some comments:

Some of the table of contents links do not work.
AZURE_CLIENT_SECRET should be used when defining constants in the ingestion section (instead of AZURE_SECRET_ID)
All cell outputs should be removed. The rendered graphs are greatly inflating the size of the notebook, making the size > 5MB. The size is ~34KB after clearing the outputs.

If you don't mind, I can push a commit to your branch to address these. WDYT?

Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-sdk-for-python/pull/30682#pullrequestreview-1470922462, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AY7KXRHH4V5JW2UVGYLCXWDXKJRB3ANCNFSM6AAAAAAY7GMQLA.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


History page: better distinction between different users

The history page shows a random picture that should depict the user in future releases (when the backend has image upload).
However, the random picture does a pretty bad job to make a distinction between users. A better idea could be to make the initials of the user into an image (e.g. Trello does that). Or we simply write down the name of the user.

Solved in #513


BarChart: highlight x-axis value when tapped

What did you expect to happen?
I need to handle action on x-axis value and highlight x-axis value
What happened instead?
Touch in x-axis value will highlight the bar, not the x-axis value
DGCharts Environment
DGCharts version/Branch/Commit Number: 5.1.0
Xcode version: 14.0
Swift version: 5.0
Platform(s) running DGCharts: iOS
macOS version running Xcode: Monterey 12.6.3
Demo Project
ℹ Please link to or upload a project we can download that reproduces the issue.

Hi!
I came across this issue and am interested in working on it. I believe I can contribute by solving this problem. Do you think I can pick up this issue?
Thank you!


API

No issue just a quick question and perhaps a hand. Love the project!
I working on a project via python. Only working with 1 specific ECU and wanted to use a line of code you have in your project as an API to auto-calculate the ecukey.
Using the DaimlerStandardSecurityAlgo, ECU is MED1775
example:
access_level = "05"
seed = "ABCDEFGH"
ecuKey = ECUSeedKeyDLL(seed)
send ecukey  //--- answer

Hello,
Thanks for checking out the project. You may wish to look into ConsoleUnlockECU (Windows: ConsoleUnlockECU.exe), which is UnlockECU as a command line tool. The compiled binaries are available in the releases page.
Set up your definition (e.g. name, parameters) first in db.json, then run the command line utility with a format similar to this:
ConsoleUnlockECU --database=db.json --name=MED1775 --level=9 --seed=1122334455667788

which should then return a hex value like:
0453FD7A

As this isn't an issue, discussions would be a better place to seek community help.

any help on this?

Hello,
Thanks for checking out the project. You may wish to look into ConsoleUnlockECU (Windows: ConsoleUnlockECU.exe), which is UnlockECU as a command line tool. The compiled binaries are available in the releases page.
Set up your definition (e.g. name, parameters) first in db.json, then run the command line utility with a format similar to this:
ConsoleUnlockECU --database=db.json --name=MED1775 --level=9 --seed=1122334455667788

which should then return a hex value like:
0453FD7A

As this isn't an issue, discussions would be a better place to seek community help.

Awesome! So upon doing this, I see MED1775 is already in the DB with a level of 5.

Hello,
Thanks for checking out the project. You may wish to look into ConsoleUnlockECU (Windows: ConsoleUnlockECU.exe), which is UnlockECU as a command line tool. The compiled binaries are available in the releases page.
Set up your definition (e.g. name, parameters) first in db.json, then run the command line utility with a format similar to this:
ConsoleUnlockECU --database=db.json --name=MED1775 --level=9 --seed=1122334455667788

which should then return a hex value like:
0453FD7A

As this isn't an issue, discussions would be a better place to seek community help.

A way to make this cmd work on linux? I see its an exe file.

A way to make this cmd work on linux? I see its an exe file.


updated spinglass engine to 1.4.0

We have an old version of the engine right now, and engine tests are failing (the rest is ok).

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

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

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls


feat: re add swipe bottom

https://github.com/Skipperlla/rn-swiper-list/assets/68515357/5c962461-405f-45a3-9dfe-16b9773c180c

referenced by #22


Clean up Dockerfile a bit

Currently ubi8 has golang 1.19 so there is no point in reinstalling it
from upstream. Leave the code commented out in there so we can move
to another version in case the requirements change over time.

/lgtm


Handle newlines and normalize the HTML

Brings the number of passing tests up to 409

I chose not to normalize the HTML first time around, because it was relatively straightforward to match the cmark output byte-for-byte. Has that significantly changed? I haven't dug into this patch too carefully, but it's actually unexpected that the insignificant whitespace has changed, as that's mostly controlled by HTML generation and not parsing.
I'm totally fine with the general thrust, just want to make sure I understand what's going on with the whitespace. Probably that means I should dig into it myself, but I'm a bit busy the next couple of days...

There are two reasons why I think it's worthwhile to normalize HTML like this:
First of all, the official test harness does it. This tells me that cmark output doesn't match the specification byte-for-byte. If the reference implementation doesn't have to do it, then why do we have to do it?

The HTML produced by the code, as I have it written now, looks like this:

<!-- markdown -->
Example\
Code
<!-- html -->
Example
<br>Code

I could fix that, by doing some manual fiddling with the tree structure (it's the SynthesizeNewline item that's getting inserted into the wrong place) but it seems pointless.

@raphlinus Should I remove the HTML normalization stuff?

Sorry for not responding earlier. I didn't know that cmark did normalization, that's a good argument. I'll merge this.


[Add] Pulumi IaC

Site URL
https://www.pulumi.com/
Description of the above provided site
Infrastructure as Code tool that is open source and allows you to define just about any cloud resources (infrastructure) in TypeScript/JavaScript, Python, Go, C#, Java, or YAML.
Category
DevOps
Sub Category
IaC
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

Looks great @desteves! :)


metric/system/cgroup/cgv2: ensure Rdev is correct width

What does this PR do?

On mips platforms, Stat_t.Rdev is uint32 (for example https://cs.opensource.google/go/go/+/refs/tags/go1.20.1:src/syscall/ztypes_linux_mips64.go;l=107) resulting in a type error. So explicitly convert to uint64.
$ grep 'Rdev\s*uint32' *.go
syscall_linux_mips64x.go:	Rdev       uint32
ztypes_dragonfly_amd64.go:	Rdev     uint32
ztypes_freebsd_386.go:	Rdev          uint32
ztypes_freebsd_amd64.go:	Rdev          uint32
ztypes_freebsd_arm.go:	Rdev          uint32
ztypes_freebsd_arm64.go:	Rdev          uint32
ztypes_linux_mips.go:	Rdev    uint32
ztypes_linux_mips64.go:	Rdev    uint32
ztypes_linux_mips64le.go:	Rdev    uint32
ztypes_linux_mipsle.go:	Rdev    uint32

Please inform on need for backport.
Why is it important?

The current code will not build on mips platforms.
Checklist

[ ] My code follows the style guidelines of this project
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added an entry in CHANGELOG.md

Author's Checklist

[ ]

Related issues

:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

@rdner What is the backport situation here? Do I manually refer to this PR in beats. The code in question was removed from beats in elastic/beats@ae50f3a6d740be84e2306582ec134ae42c6027b7.

@efd6

The code in question was removed from beats in https://github.com/elastic/beats/commit/ae50f3a6d740be84e2306582ec134ae42c6027b7.

That code was moved from Beats to this repository.

What is the backport situation here? Do I manually refer to this PR in beats.

Normally, you tag a new release (e.g. v0.4.6) and update the dependency in Beats' go.mod. This dependency update could be backported if needed.

Actually, it looks like it was fixed there in https://github.com/elastic/beats/commit/5a9fdafee4766475987bae6cd4dfad56b5082f78.

But I thought you had reproduced the problem before opened this PR? It's up to you what to do with this PR then.

Actually, it looks like it was fixed there in https://github.com/elastic/beats/commit/5a9fdafee4766475987bae6cd4dfad56b5082f78.

But I thought you had reproduced the problem before opened this PR? It's up to you what to do with this PR then.

I think it that must have happened after the lib here was forked. This should go in here.

Normally, you tag a new release (e.g. v0.4.6) and update the dependency in Beats' go.mod. This dependency update could be backported if needed.

Yeah, I was wondering about 7.17 which that won't affect. I'll make a PR against 7.17 that reflects the state of the file here. Is it OK if I ping you on that?


(maint) Add note about ubuntu 18.04 install

Test FAILed

Test PASSed

CLA signed by all contributors.

:thumbsup:


can't load image in server but local's fine

i have uploaded image but its not showing the image in server

Did you run php artisan storage:link on the server?

my server doesnt support php artisan

To make them accessible from the web, you should create a symbolic link from public/storage to storage/app/public
See: https://laravel.com/docs/6.x/filesystem#the-public-disk
If I'm correctly you can do ln -s ../storage/app/public public/storage in the root of your project. If that is not possible you need to have another way to let your webserver server storage/* from storage/app/public.


PYI-481: Minimal lambda and terraform to setup aws gateway and empty lambda

Proposed changes
What changed
Add new terraform file to create a new empty lamdba for the dcs credential and wire it up with the aws api gateway.

Why did it change
This is the terraform infrastructure required to deploy this new lambda needed for the cri-passport oauth flow.

Issue tracking

PYI-481

Are the 2 Security Hotspots in the Sonarcloud report worth noting?


Debugger raycast params configuration

Allows customizing the RaycastParams used by the Debugger mouse highlight. Also configures default raycast params to ignore terrain water, which is likely not helpful for any users.
Use case:
I have door entities in my game. They are under the "Door" collision group that is not collidable with the "Default" collision group (which, the absence of RaycastParams uses). Therefore, they are not currently selectable with the Matter debugger. By making this change, I can create a "Debug" collision group that is collidable with everything, and set the debugger's collision group accordingly.

not sure if @LastTalon has anything else

No, just wanted clarification on docs. Changelog would also be good. Other than that I'm good to approve.


Cli Repl: Automatically resolve promises and add last result variable

Context
The CLI repl does not support the await keyword at the lop level.
This PR turns the bug into a feature by adding the following behaviour

If the repl command returns a promise the. repl will block until the promise is resolved and returns the fulfilment value of the promise
The result of the last run command, or the fulfilment value in the case of a promise, is stored in a variable _ for easy reuse.

Related Tasks: REPL: Resolve promises automatically
Manually testing the PR
cargo run -- repl
> Contract.create("export default () => { console.log(`Hello from ${Ledger.selfAddress} 👋`); return new Response() }");
> new Request(`tezos://${_}`);
> Contract.call(_);

I tried running it, but the example doesn't seem to work properly:
`Using mock self-address tz4RepLRepLRepLRepLRepLRepLRepN7Cu8j.

Contract.create("export default () => { console.log(Hello from ${Ledger.selfAddress} 👋); return new Response()) }");
[📜] Contract created: tz4RVBAHakYgHx7XYK1nacqCGikzWBGGm6Ld
tz4RVBAHakYgHx7XYK1nacqCGikzWBGGm6Ld
new Request(tezos://${_});
[object Object]
Contract.call(_);
Evaluating: "export default () => { console.log(Hello from ${Ledger.selfAddress} 👋); return new Response()) }"
Uncaught EvalError: JsError
`

I tried running it, but the example doesn't seem to work properly:

There was an extra bracket in the example contract. It should work now

LGTM, please clean up your git history before merging (merge commits aren't allowed in main)

Changes look good. I'm not entirely happy that we automatically resolve promises (since I don't expect this from most REPLs). But for now it is a good workaround the issues we've had with promises in the REPL.


Use jshint-cli instead of grunt-jshint

This reduces the use of grunt and make it easy to manage module dependencies.

Codecov Report

Merging #2064 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2064   +/-   ##
=======================================
  Coverage   64.08%   64.08%
=======================================
  Files           1        1
  Lines        4455     4455
=======================================
  Hits         2855     2855
  Misses       1600     1600

Continue to review full report at Codecov.

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

Thanks!


build_library/toolchain_util.sh: Install matching versions of toolchain packages

When adding a mask or accept keywords entry for some version of a toolchain package (gcc, libc, gdb, binutils or kernel headers), it can't be done by just doing it, for example, for sys-devel/gcc. Both cross-{x86_64,aarch64}-cros-linux-gnu/gcc needs to be masked/keyworded, otherwise crossdev will pick up the latest stable version for cross-{x86_64,aarch64}-cros-linux-gnu/gcc and this choice is not affected by masks or accept keywords of sys-devel/gcc.
This situation does not happen all that often, but when it happens, it's usually hard to remember to handle also the cross toolchain packages. Forgetting to do so leads to weird issues. So instead of telling crossdev to use the latest stable versions of cross toolchain packages, we will tell it to use specific versions that match the version of plain packages.
CI: http://jenkins.infra.kinvolk.io:8080/job/container/job/sdk/597/cldsv (together with weekly updates)

CI passed.

This can be merged separately from the weekly updates.

To clarify - this can be merged separately from weekly updates, but weekly updates are blocked on this PR.


Cleanup remote tracebacks

Adds a utility function shorten_traceback, and wraps sys.excepthook to apply it to unhandled exceptions.
The behavior of the function is controlled by several settings:

admin.traceback.shorten
admin.traceback.allowlist
admin.traceback.skiplist

If admin.traceback.shorten: true,  and any of the stack frames match a pattern listed in admin.traceback.allowlist, stack trace is shortened. Frames that match a pattern listed in admin.traceback.skiplist are omitted. The method always preserves the first and last stack frame though.
For IPython, instead of sys.excepthook,  a custom exception handler is provided.
The second part - adding worker info to the remote exception - is implemented in https://github.com/dask/distributed/pull/7847.
Issue: https://github.com/dask/distributed/issues/4880.

[x] Tests added / passed
[x] Passes pre-commit run --all-files

As discussed with @crusaderky.

@crusaderky Finally got the test passing. Lesson learned: never expect ipython to behave the same on different envs. This is now ready for review.
One more thing I was thinking about adding to this PR: exposing a function to install the excepthook, instead of modifying it without the user participation. It might be an overkill. What do you think?

exposing a function to install the excepthook, instead of modifying it without the user participation. It might be an overkill. What do you think?

I don't think I follow. What function are you referring to?

@j-bennet see my review commit Merging as soon as you confirm you're happy with my changes

when and what sounds fine.

I don't think I follow. What function are you referring to?

In the PR at the current state, as soon as the user imports something from dask, sys.excepthook is wrapped, even if the wrapper does nothing when admin.traceback.shorten is false.
Alternatively, wrapping excepthook could be something that's triggered by the user, by calling a method, let's say, dask.instal_hooks.

In the PR at the current state, as soon as the user imports something from dask, sys.excepthook is wrapped, even if the wrapper does nothing when admin.traceback.shorten is false.
Alternatively, wrapping excepthook could be something that's triggered by the user, by calling a method, let's say, dask.instal_hooks.

I see your reasoning, but I feel that if we added a completely different API it would make things worse for the UX rather than better.
This (and quite a few other situations) could potentially resolved by being able to install callbacks in dask.config.set. Definitely out of scope though.

Thank you!


docs: Suggested technical writing changes for BYOB.

@laurentsimon, @olivekl asked me to take a look at the BYOB docs. These are my suggested changes, and I am open to feedback.

The failing linter seems to jut be because the title and actual section names don't match


[SPARK-43252][SQL] Replace the error class _LEGACY_ERROR_TEMP_2016 with an internal error

What changes were proposed in this pull request?
Replace the legacy error class _LEGACY_ERROR_TEMP_2016 with an internal error as it is not triggered by the user space.
Why are the changes needed?
As the error is not triggered by the user space, the legacy error class can be replaced by an internal error.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing test cases.
Was this patch authored or co-authored using generative AI tooling?
No.

ping @MaxGekk

Waiting for CI.

@dengziming Could you resolve conflicts, please.

+1, LGTM. Merging to master.
Thank you, @dengziming.


Update Appendix.md

As "PromiseKit has perhaps capriciously switched between the names always and ensure", this page just needs an update to refer to ensure instead of always :)

Thanks!


[RESTEASY-1721] ResourceMethodInvoker checks if @Cont…

…ext HttpServletResponse was written to.

Replaced by https://github.com/resteasy/Resteasy/pull/2752.


hi guys its ur boi alia

Pull request

[ ] I have read and agreed to the CODE OF CONDUCT.
[ ] What I'm implementing was assigned to me or was not being worked on and is a somewhat requested feature. For reference see our trello.
[ ] I have tested all of my changes.
[ ] I have followed the style of the rest of the project.

Added/Changed
Cat Command, and remove some imports from RolesCommand
Feature
If this is a new feature, why should we add it to the bot.

Can one of the admins verify this patch?

Master 👀
One cat? This isn't a good enough command to have just one cat in it.


Added shim to replace svg functionality removed in Chrome [#116982211]

Also replaced a hotlink to jquery.anythingzoomer.min.js on GitHub to a link to a local version of the file.  The hotlinked file was not loading because GitHub was returning the incorrect mimetype for the file.

@sfentress Can you review and deploy when you get back?

👍


GraphQL Playground to develop a query

Is there a public gui website?

It's recommended to access this on a local instance. If you run the docker stack it will be available at http://localhost:3100


RK2023 stuck at 88% charge

I've tried USB 2.0 ports on my PC as well as only using the provided charger with the device as Powkiddy instructed, and they blamed the OS "supposedly" being the issue. It charges but gets stuck at 88% and won't charge past that.

Either open your device and unplug then replug the battery or let the battery drain completely then recharge again.

Any update on this issue?

Closing due to lack of update from the op.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #380 (a73f3cf) into main (2974dfe) will increase coverage by 12.36%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##             main     #380       +/-   ##
===========================================
+ Coverage   83.33%   95.69%   +12.36%
===========================================
  Files           2        1        -1
  Lines         126      395      +269
  Branches       31       42       +11
===========================================
+ Hits          105      378      +273
+ Misses         21       17        -4

Impacted Files
Coverage Δ

src/index.ts
95.69% <100.00%> (+13.69%)
:arrow_up:

example/src/main.ts

Continue to review full report at Codecov.

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


Surface bundle permissions

What is the context of this PR?
Bundles can have permissions assigned, but this isn't exposed to groups.
How to review

@MebinAbraham that is fixed in 6.2 release notes
< Allow access to snippets and other model viewsets to users with “View” permission (Sage Abdullah)


fix: support space in file name

According to this issue comment, GitHub will rename the asset, which causes the name-based check to fail for file names containing spaces. We can change the file name to the same as GitHub before checking to solve this problem.

thanks for looking into this @MystiPanda. nice catch

Yes, I noticed what the documentation said, I initially wanted to use a regex to match all the special characters the documentation said, but I didn't find this character list in the documentation, so I just replace space which is most common.

I think I've got something here. It seems that all characters except letters, numbers and @ - _ + are not supported, and consecutive unsupported characters will be replaced with a ., so simple space replacement may have problems, If a txt file is named with unsupported characters except for the extension, GitHub will rename it to default.txt

thanks. we can expand this list as we go. just a heads up, your changes have been incorporated into the v2 which now includes these changes


[ Golang ] 8. Introduction to DOC Concurrency

Description 📜
Introduction to DOC Concurrency:-
A small description about-

Goroutines and how to create Goroutine.
Channels
Buffered Channels
Range and Close
Select
Default Selection
What is Mutex?
Fixes #1334

Type of change 📝

[x] Doc#umentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] Go#lang

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@1anshu-56: I am already done with changes. Thanks for your contributions! :partying_face:

Thank You so much @amino19 ✨ for your guidance and mentoring me.


Sends default Go useragent

The useragent for all requests is Go's default. A better option would be to send something like Chrome or Firefox (or even allow the user to choose).

I have set it to user's User-Agent for now. As aim for this project is not an anonymizing proxy but a proxy which just allows you to access unnecessarily blocked websites.


API Docs

This is a complete rewrite of our API Documentation for gulp 4.  We'd really appreciate some thorough review - as we've stared at these for months.
A few things to note:

Each API is in it's own document because things were really unwieldy with a single doc previously
Tables are used extensively to pack a lot of data densely but still be readable.
API Concepts was created because we had a lot of terminology that kept needing to be defined or linked - I don't believe it is complete but added the things I could come up with.
Everything should be completely updated for the shipped 4.0 - If not, that's a mistake that should be reported.
Not all error cases/messages are documented, especially for src()/dest() because they touch the filesystem and might error in untested ways based on a user's filesytem/configuration.
We pulled in all the external docs we could hunt down and rewrote them to match our style - so you'll see massive tables of options for src() and watch() and maybe some others.

I'll comment inline with a few other questions we had on specific parts.
cc @stevelacy @contra @terinjokes @callumacrae @jonschlinkert @doowb @erikkemperman

Hey all, we're hoping to do the cleanup pass next week so please send some more feedback that we can work on.  I'd love to get these landed soon.

I'm going to call this done once CI wraps up (it won't affect anything but whatever) - If we need to make any further changes, we can do small PRs against master.


FR: add GUI-lite

https://github.com/idea4good/GuiLite

Just fixed the recipe here so it builds now.


SQL Query Errors tooltip is ambiguous

Piyush Singh commented:
@timveil had a question about tooltips for the SQL query errors graph:

the tooltip says "The number of statements which returned a planning or runtime error." but the docs say " SQL statements that returned a planning, runtime, or retry error".  so first question, does this graph include retry errors?  second question, are these errors that would be returned to a client?

Jordan confirmed that these are only errors that are surfaced to clients, and it includes errors of any kind.
See full slack thread for all context
cc @thtruo and @awoods187
Jira Issue: DOC-1003

cc @kevin-v-ngo this would be a product change and is mistakenly a docs issue.

Thanks for the heads up - let's close this docs issue as no-op.
Heads up @kevin-v-ngo this might be a small ui copy change that your team can take up for the SQL Statement Errors graph tooltip


Add @SuppressWarnings("deprecation") annotation to generated finalize methods

The use of finalize is a big can of worms (see StackOverflow Why is the finalize() method deprecated in Java 9?), but it would be nice to at least have a djinni option to have the generated Java classes annotate the finalize generated methods with @SuppressWarnings("deprecation") so I don't get Java compile warnings.

I wonder if there is an alternative to use finalize .
Didn't find one, but I am also not a java expert so chances I have overseen something are huge

I think the correct way to handle this would be to use PhantomReferences.
See also here: https://www.baeldung.com/java-phantom-reference
I've implemented something like this many moons ago, but I really don't remember any details.


Add isValid(), fix security issues in dependency

hi @perry-mitchell patches pushed :+1:
hope I can use the new version of ulidx from npm instead of my local build hahaha

Thanks, I'll check this in a few hours.. but I suspect it'll be fine and I'll do a release then.
Cheers!


Static code analysis _037 (Coercion Alters Value - phdly)

Short summary (details in attached file):
GptpIniParser::iniCallBack()
.../common/gptp_cfg.cpp
            parser->_config.phy_delay[LINKSPEED_1G].set_tx_delay( phdly );

phdly is coerced from int to unsigned short.
phdly evaluates to strtoul(value, &pEnd, 10)gptp_cfg.cpp:158.
Coercing them to unsigned short can cause data loss or sign change.

gptp_037.txt

Fix was merged


Segmentation fault (core dumped) in tf.data.experimental.service.DispatchServer

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.18.0-dev20240925
Custom code
Yes
OS platform and distribution
Linux Ubuntu 22.04.3 LTS (x86_64)
Mobile device
No response
Python version
3.9.13
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
tf.data.experimental.service.DispatchServer triggered a crash, when the start parameter is false.
Standalone code to reproduce the issue
import tensorflow as tf

def test():
    tf.data.experimental.service.DispatchServer(start=False)
test()

Relevant log output
2024-09-28 20:20:16.983171: I tensorflow/core/util/port.cc:153] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable `TF_ENABLE_ONEDNN_OPTS=0`.
2024-09-28 20:20:17.013139: E external/local_xla/xla/stream_executor/cuda/cuda_fft.cc:485] Unable to register cuFFT factory: Attempting to register factory for plugin cuFFT when one has already been registered
2024-09-28 20:20:17.063675: E external/local_xla/xla/stream_executor/cuda/cuda_dnn.cc:8454] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered
2024-09-28 20:20:17.087886: E external/local_xla/xla/stream_executor/cuda/cuda_blas.cc:1452] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2024-09-28 20:20:17.147364: I tensorflow/core/platform/cpu_feature_guard.cc:210] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: SSE4.1 SSE4.2 AVX AVX2 AVX512F AVX512_VNNI AVX512_BF16 AVX512_FP16 AVX_VNNI AMX_TILE AMX_INT8 AMX_BF16 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
Segmentation fault (core dumped)

Hi @x0w3n ,
I tried running your code on Colab using TensorFlow v2.17.0 & the nightly version and faced the same issue. I also tried an alternative approach. If we start the server manually, then it works fine. Here, I am providing the gist for your reference. I hope it will be useful for you.
Thank you!

Tensorflow team how are you letting these segfaults through and not responding even after user has give you detail colab. Fix the problem. Fix your unit tests


Documentation

Where ican find the docs on how to use the library?
Not sure how i can define a directory where i want the packages to be installed insted of vendor dir

We haven't really documented this library yet because it's still in active development. However, I would imagine you can change the vendor directory to something else by adding the following to your composer.json file:
"config": {
    "vendor-dir": "<custom-vendor-path>"
}


Add 'new' & 'updated' badges to docs to show what's changed

I thought it would be a good idea to add a small badge next to the docs navigation, next to each component/utility etc to show what's new and what's been updated with each new version of Bootstrap starting with 4.2
I think this will help people find new and updated features at a glance, the idea being that these badges could stay around until the next iteration (new major & minor releases).
e.g:

@MartijnCuppens  The release notes absolutely would cover new changes, I was just thinking for those who don't really look at the release notes/Github, but rather just want to quickly see what's new with a visual indicator (a badge next to the component), maybe instead of two types, consider just having a NEW badge for new features, or if a component has extra modifier classes, consider then adding an UPDATED badge?

We could automate it, not sure if it's worth the trouble though.
Anyway, if anyone wants to submit a simple solution, glad to review it.

Per last comments, could be cool, but currently not on our radar given the release notes and more.


Dungeons

Build it using phaser.
Ideas:

[ ] Shoot NPCs
[ ] NPCs get angry if you interact with them too much
[ ] Accuracy, critical, effectiveness
[x] Portal Teleport
[ ] Mini Map
[x] Slider quadratic ease into step (used step since it's simple and beautiful)

Great idea here about feature resolvers, we can use this to split out the annoying logic for handling player input in the world scene, and each class will implement AFeatureResolver and handleInput(justDownInput, input), then we can implement the priority as well with AFeaturePriorityListResolver, we can declare the priority in which we want to handle the resolvers, and the input can return a boolean to indicate whether the input was handled successfully

This requires ES decorators to be implemented and will require some research for feasibility

Stopped at ep 128, vue-phaserjs isn’t compatible with latest vue reactivity system… hope it fixes itself later


model的bug问题

[
private Byte stockWarning;
private Integer stockWarningNum;
]
i have two object with properties like this,when i set stockWarningNum as Integer.MAX，its a error happend ；
Value  is too large for java.lang.Byte
does  modelmapper  mapped by perfix???
why this error happend ?

What is the matching strategy of modelmapper?
if Standard,you should change Strict
modelMapper.getConfiguration()
  .setMatchingStrategy(MatchingStrategies.STRICT);

refer to http://modelmapper.org/user-manual/configuration/#matching-strategies


GraphQLTestAutoConfiguration tries to autowire TestRestTemplate regardless of WebEnvironment

Describe the bug
When the GraphQLTestAutoConfiguration class attempts to initialize the graphQLTestUtils bean, a NoSuchBeanDefinitionException is thrown because TestRestTemplate could not be autowired.
To Reproduce
Steps to reproduce the behavior:

Create a simple working Spring application
Add tests using @SpringBootTest (aka. with webEnvironment = WebEnvironment.MOCK) and check that they run
Add dependency to graphql-spring-boot-starter and graphql-spring-boot-starter-test
Notice how existing tests now fail to start due to NoSuchBeanDefinitionException

Expected behavior
Adding the graphql dependencies should not break existing tests.
Additional context
A workaround is to explicitly disable graphql for the existing tests, using eg. @SpringBootTest(properties = "graphql.servlet.enabled=false")

I have this exact same issue, however your workaround doesn't work for me as I get another error:
Caused by: org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'graphQLWebsocketServlet' defined in class path resource [graphql/kickstart/spring/web/boot/GraphQLWebsocketAutoConfiguration.class]: Unsatisfied dependency expressed through method 'graphQLWebsocketServlet' parameter 0; nested exception is org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type 'graphql.kickstart.execution.subscriptions.GraphQLSubscriptionInvocationInputFactory' available: expected at least 1 bean which qualifies as autowire candidate. Dependency annotations: {}

@streibeb have you tried also setting graphql.servlet.websocket to false? That property is used by the GraphQLWebsocketAutoConfiguration class.
@SpringBootTest(properties = {
        "graphql.servlet.enabled=false",
        "graphql.servlet.websocket.enabled=false"
})

+1
Issue might be also be caused by
@ConditionalOnProperty(value = "graphql.servlet.enabled", havingValue = "true", matchIfMissing = true)
Slightly nicer approach for now is to disable auto config in general configuration file:
@EnableAutoConfiguration(exclude = {GraphQLTestAutoConfiguration.class})
After that you could create local config with GraphQL enabled

You don't need to make graphql.servlet.enabled=false.
TestRestTemplate is only auto-configured when @SpringBootTest has been configured with a webEnvironment.
So the simplest way to fix this is adding webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT to your @SpringBootTest.

with com.graphql-java-kickstart.graphiql-spring-boot-starter:11.0.0 and com.graphql-java-kickstart.graphql-spring-boot-starter-test:11.0.0 we only need to add
@SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT)
Voila it works as expected. We can have something else done either. Not sure why the above just work without the property change.


Estimator of signal multiplier based on perturbation theory

This PR adds a function to alea.utils that can estimate the best-fit signal multiplier given signal background model and data histogram. Compared to the iminuit fitting, it totally gets rid of the initial guess, which could cause fitting failure if it's too ridiculous. This function can be used as an initial guess of the multiplier when there are more nuisance parameters in the likelihood. The math is summarized here
When testing it, it already gives quite good fit

To reproduce it,
import numpy as np
import multihist as mh
import matplotlib.pyplot as plt
import inference_interface as ii

from alea.utils import signal_multiplier_estimator

np.random.seed(1)

@np.errstate(invalid='ignore', divide='ignore')
def loglikelihood(lam, data, bkg, sig):
    exp = bkg + lam * sig
    assert np.all(data[exp <= 0] == 0)
    return np.sum(np.where(exp > 0, data * np.log(exp) - exp, 0))

bkg = ii.template_to_multihist('/home/zihaoxu/alea/alea/examples/templates/er_template_0.ii.h5', hist_name='er_template')
sig = ii.template_to_multihist('/home/zihaoxu/alea/alea/examples/templates/wimp50gev_template.ii.h5', hist_name='wimp_template')

data = (bkg + sig*1e-1).get_random(size=np.random.poisson(bkg.n))
data = mh.Histdd(*data.T, bins=bkg.bin_edges)

x = np.logspace(-3, 0.3, 100)
plt.plot(x, [loglikelihood(_x, data.histogram, bkg.histogram, sig.histogram) for _x in x])
best_fit_x = signal_multiplier_estimator(sig.histogram, bkg.histogram, data.histogram)
plt.axvline(best_fit_x, color='r', ls='--')
plt.title(f'Best fit multiplier = {best_fit_x:.1e}')
plt.xscale('log')
plt.ylabel('log likelihood')
plt.xlabel('WIMP multiplier')
plt.show()

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 15 (6.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.8%) to 90.253%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

alea/utils.py
1
15
6.67%

Totals

Change from base Build<PHONE_NUMBER>:
-0.8%

Covered Lines:
1463

Relevant Lines:
1621

💛  - Coveralls

Added
See the following figure. Note that the new function is just an estimator, which could differ from model.fit() if there are nuisance parameters or ancillary term. The advantage is the good convergence performance of the function, thus can be used as the estimator but NOT the true fitting.

Added
I think as long as the data is described normally by the templates, i.e. no data point with pdf = 0, the function will always converge. Not sure if a flag is needed.

The code to make the above plot is here. Maybe there's a better way to hack into the templates?
import numpy as np
import matplotlib.pyplot as plt
import scipy.stats as stats
from copy import deepcopy
import alea
from alea import BlueiceExtendedModel

# Just some plotting settings
import matplotlib as mpl

mpl.rcParams["figure.dpi"] = 200
mpl.rcParams["figure.figsize"] = [4, 3]
mpl.rcParams["font.family"] = "serif"
mpl.rcParams["font.size"] = 9

# initialize
config_path = "unbinned_wimp_statistical_model_simple.yaml"
model = BlueiceExtendedModel.from_config(config_path)

np.random.seed(1)
truth = {
    'wimp_rate_multiplier': 2.0,
    'er_rate_multiplier': 1.0,
    'er_band_shift': 0,
}
data = model.generate_data(**truth)
model.data = data

bkg = model.likelihood_list[0].base_model.get_source('er')
num_bkg = bkg.events_per_day * model.parameters['livetime'].nominal_value
bkg = bkg._pdf_histogram / bkg._pdf_histogram.n * num_bkg

sig = model.likelihood_list[0].base_model.get_source('wimp')
num_sig = sig.events_per_day * model.parameters['livetime'].nominal_value
sig = sig._pdf_histogram / sig._pdf_histogram.n * num_sig

obs = sig.similar_blank_hist()
obs.add(data['science_run']['cs1'], data['science_run']['cs2'])

x = np.geomspace(1e-1, 20, 100)
plt.plot(x, [model.ll(wimp_rate_multiplier=_x) for _x in x])
plt.axvline(model.fit()[0]['wimp_rate_multiplier'], color='k', label='model.fit')
plt.axvline(alea.utils.signal_multiplier_estimator(sig.histogram, bkg.histogram, obs.histogram), color='k', ls='--', label='signal_multiplier_estimator')
plt.xscale('log')
plt.xlabel('wimp_rate_multiplier')
plt.ylabel('log likelihood')
plt.legend()


Add requested resources metrics per priority

Issue link

What changes have been made

This PR adds custom metrics (Prometheus Gauges) for the current consumed resources (CPU/memory/GPU) per priority.
I preferred measuring the consumed resources, rather than the available capacity as logged here because summarizing over the consumed resources in PromQL has meaning while summarizing over the available capacity doesn't.
Verification steps

I created 3 AppWrappers with 2 different priorities and requested resources as follows:
  ...
  priority: 1
  ...
      custompodresources:
      - replicas: 1
        requests:
          cpu: 100m
          memory: 128M
---
  ...
  priority: 3
  ...
      custompodresources:
      - replicas: 2
        requests:
          cpu: 200m
          memory: 256M
---
  ...
  priority: 3
  ...
      custompodresources:
      - replicas: 1
        requests:
          cpu: 300m
          memory: 512M
          nvidia.com/gpu: 0

While running MCAD, the new metrics are:
curl -s localhost:8080/metrics | grep mcad_requested

# HELP mcad_requested_cpu Requested CPU per priority
# TYPE mcad_requested_cpu gauge
mcad_requested_cpu{priority="1"} 0.1
mcad_requested_cpu{priority="3"} 0.7
# HELP mcad_requested_gpu Requested GPU per priority
# TYPE mcad_requested_gpu gauge
mcad_requested_gpu{priority="3"} 0
# HELP mcad_requested_memory Requested memory per priority
# TYPE mcad_requested_memory gauge
mcad_requested_memory{priority="1"} 1.28e+08
mcad_requested_memory{priority="3"} 1.024e+09

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] Testing is not required for this change

@ronensc please rebase

@ronensc please rebase

done

@ronensc This PR has a minor conflict with #82. Please fix.

@ronensc This PR has a minor conflict with #82. Please fix.

Rebased and fixed


Add Strfry install script

Completes #4717
config to be tested

some fun on update to v1.0.0:

Sep 13 10:53:46 raspberrypi systemd[1]: Started strfry.service - strfry relay service.
Sep 13 10:53:46 raspberrypi strfry[1421847]: date       time         ( uptime  ) [ thread name/id ]   v|
Sep 13 10:53:46 raspberrypi strfry[1421847]: 2024-09-13 10:53:46.168 (   0.032s) [main thread     ]INFO| arguments: /home/strfry/strfry/strfry relay
Sep 13 10:53:46 raspberrypi strfry[1421847]: 2024-09-13 10:53:46.168 (   0.032s) [main thread     ]INFO| Current dir: /
Sep 13 10:53:46 raspberrypi strfry[1421847]: 2024-09-13 10:53:46.168 (   0.032s) [main thread     ]INFO| stderr verbosity: 0
Sep 13 10:53:46 raspberrypi strfry[1421847]: 2024-09-13 10:53:46.168 (   0.032s) [main thread     ]INFO| -----------------------------------
Sep 13 10:53:46 raspberrypi strfry[1421847]: 2024-09-13 10:53:46.168 (   0.032s) [main thread     ]INFO| CONFIG: Loading config from file: /etc/strfry.conf
Sep 13 10:53:46 raspberrypi strfry[1421847]: 2024-09-13 10:53:46.180 (   0.044s) [main thread     ]INFO| CONFIG: successfully installed
Sep 13 10:53:46 raspberrypi strfry[1421847]: 2024-09-13 10:53:46.180 (   0.044s) [main thread     ] ERR| Database version too old: 2. Expected version 3
Sep 13 10:53:46 raspberrypi strfry[1421847]: 2024-09-13 10:53:46.180 (   0.044s) [main thread     ] ERR| You should 'strfry export' your events, delete (or move) the DB files, and 'strfry import' them
Sep 13 10:53:46 raspberrypi strfry[1421847]: strfry error: aborting: DB too old
Sep 13 10:53:46 raspberrypi strfry[1421847]: 2024-09-13 10:53:46.181 (   0.045s) [main thread     ]INFO| atexit
Sep 13 10:53:46 raspberrypi systemd[1]: strfry.service: Main process exited, code=exited, status=1/FAILURE
Sep 13 10:53:46 raspberrypi systemd[1]: strfry.service: Failed with result 'exit-code'.

Works well on RPi and server since the update.

Tested install on my RP5 - got a "502 Bad Gateway" when calling https://<IP_ADDRESS>:7701


[dv/flash] Update erase-suspend test with read

Hi,
This PR is adding a read after the erase-suspend to make sure the erase is suspended.
Follow-up to PR #17365
Thanks!

@eshapira do you mind helping me rebase and repush here? i think that would solve the ci issue.


Images in Transcluded Notes are missing

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Transclude a note (containing at least one image) into another "Parent Note"
Copy the Parent Note to HTML, then paste into empty Google Docs document

Expected behavior
Images from the transcluded note will be visible.  Actual behavior, images appear as Obsidian path text placeholders
Screenshots
Obsidian Source Mode

Obsidian Live Preview

Google Docs or Gmail

Desktop (please complete the following information):

OS: Windows 10
Version: Obsidian 1.3.7 w/ Copy document as HTML v0.5.0

Additional context
Thanks for the awesome plugin!  Hard to imagine using Obsidian without it!
Edit
Added Obsidian Source Mode Screenshot

Hi @senorvalenz, thanks the detailed report ! I can confirm the issue, I will try to look into it, no guarantees about delays unfortunately :)

I tried to reproduce this on my dev computer, with an up to date Obsidian (1.4.12), and the image is included correctly, both through a direct link and in a transcluded child document.
If you can still reproduce this, feel free to reopen the issue !

Indeed, I just updated to the latest release of your plugin and obsidian, all is well, thanks :)


Bluebubbles fails to run on Asahi Linux

When attempting to run the Blueblubbles aarch64 flatpak on Asahi Linux (M1 Pro aarch64 hardware), it returns this error:
bluebubbles: error while loading shared libraries: libpdfium.so: ELF load command address/offset not page-aligned
This is likely due to the 16k pages that Asahi Linux uses

https://github.com/AsahiLinux/docs/wiki/Broken-Software#:~:text=Fixed since 1.5.0-,pdfium,-https%3A//bugs.chromium
Looks like it's been fixed upstream. I'll see if I can upgrade the dependency.

@lacamar A new version has been published to the flathub beta repo. Could you let me know if the issue is resolved? Thanks.

I can't seem to find the beta flathub package, is that publicly available?
I managed to manually update libpdfium for my install of the most recent public flathub package and I was able to run bluebubbles, though it was very slow. I would guess that's due to the GPU drivers not being fully functional yet.

You should be able to install the beta after adding the beta repo.
flatpak remote-add --user flathub-beta https://flathub.org/beta-repo/flathub-beta.flatpakrepo

I installed the beta from that repo but I'm getting the same error. It looks like it isn't pulling the latest version.
`$ flatpak info app.bluebubbles.BlueBubbles
BlueBubbles - BlueBubbles client for Linux
      ID: app.bluebubbles.BlueBubbles
     Ref: app/app.bluebubbles.BlueBubbles/aarch64/beta
    Arch: aarch64
  Branch: beta
 Version: <IP_ADDRESS>
 License: Apache-2.0
  Origin: flathub-beta

Collection: org.flathub.Beta
Installation: user
Installed: 56.2 MB
Runtime: org.gnome.Platform/aarch64/43
Sdk: org.gnome.Sdk/aarch64/43
  Commit: 031eb8cf8ff74a1d744a19eb53213d6457e5fb01cb0f4e3185603e5396f4f02e
  Parent: 6e7f4633378e96d62bb64e814b12a7727d3c2c0fbaa6f25b7ace63d21cc0a4db
 Subject: Update to <IP_ADDRESS> Beta (e3046959)
    Date: 2023-04-09 13:11:41 +0000

$ flatpak run app.bluebubbles.BlueBubbles
bluebubbles: error while loading shared libraries: libpdfium.so: ELF load command address/offset not page-aligned`

That is the latest version, so seems like its still an issue

Well pdfium appears to download the latest version every time I build according to its CMakeLists.txt, and according to the link I sent earlier, it should be fixed since chromium 108. The latest release is on chromium 114 now....
So I'm not sure what else I can try. How did you manually update the package for your install @lacamar ?

I downloaded and extracted "pdfium-linux-arm64.tgz" from this repo: https://github.com/bblanchon/pdfium-binaries/releases
After that I deleted the libpdfium.so file located at /var/lib/flatpak/app/app.bluebubbles.BlueBubbles/current/active/files/bluebubbles/lib/, then I copied libpdfium.so downloaded from the repo to that directory.
After that bluebubbles just runs as usual via flatpak run app.bluebubbles.BlueBubbles.

That's what the CMake should be doing. I can probably replicate what you did in the flatpak config. I'll let you know when I release another beta version to flathub in a bit

@lacamar Just published <IP_ADDRESS> to flathub beta which downloads and places the libpdfium.so file like you did. Lmk if it works.

Just tested it, that worked. The flatpak just runs now.


Add router option

Hi @alecbcs @hyprspace, nice project!
This PR adds possibility to optionally set one of the remote nodes as router / gateway and so allows to reach LAN networks behind the router or even use it as default gateway to forward Internet traffic via it.
In short it adds traditional VPN functional in addition to existing mesh functional.
Related issue #33
Examples
hs0.yaml
interface:
  address: <IP_ADDRESS>/24
  router: <IP_ADDRESS>
...

Access LAN network <IP_ADDRESS>/24
$ ip route add <IP_ADDRESS>/24 dev hs0
$ ping -c 1 <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=64 time=7.63 ms

Access Internet (remote node should have configured forwarding and NAT)
# <IP_ADDRESS> is the public address of remote node
# <IP_ADDRESS> is the local router
$ ip route add <IP_ADDRESS> via <IP_ADDRESS>
$ ip route add default dev hs0 metric 1
$ curl http://2ip.ru
<IP_ADDRESS>

ping @alecbcs

@dukzcry (Not a dev) Just to ask, does this pull request account for if router: <IP_ADDRESS>, ::1 is set so both ipv4 and ipv6 packets are routed? Also, huge thanks for making this pull request!

@LynHyper As I see current code of Hyprspace only works with IPv4 packets https://github.com/hyprspace/hyprspace/blob/main/cli/up.go#L191
Also you set router address to one of the remote peers address, it has not been confused with your local router address

@dukzcry It seems you're right, I didn't realise the ipv6 patch from a while ago only dealt with the ipv6 ip-addresses of the Hyprspace devices and not the internal ip-addresses, I'll probably open an issue for internal ipv6 support soon.
The reason I brought this up is because https://www.wireguard.com mentions that setting a peer as an ipv6 router, even in an ipv4-only VPN, is important to prevent ipv6 packets bypassing the router peer. Supporting ipv6 router addresses, even if Hyprspace's peers can't actually have an ipv6 address, would help address this problem.


Too many established connections

I'm using 0.6.1 version and after a very short time I get several hundred ESTABLISHED connections on my server between mattermost service nad matterircd, which causes my mattermost service to hang because of too many connections.
Has anyone seen this behaviour?

Probably related to https://github.com/42wim/matterircd/issues/48?
Fixed in 0.7

It looks so!
Thanks


Configure pipenv for python tests.

LGTM, @funbringer any comment?

I don't actually know what this does or why it's better-- could someone explain it to me?
Also, should this change the way we build the code in CI?

I don't actually know what this does or why it's better-- could someone explain it to me?
Also, should this change the way we build the code in CI?

Well, installing python packages system (or user) wide is pain, as dependencies of different projects conflict with each other. Python has virtual environments which solve this problem, and pipenv is wrapper around them making the work a bit more comfortable. It appears to me that pipenv is the main community accepted way of managing dependencies in Python now. For instance, with this we get list of dependencies (including exact versions known to be working in .Lock file) in a documented format and easy way to install them.


Micro places compiled Java programs into incorrect directory when auto-compiling

Description of the problem or steps to reproduce
Upon saving any Java program that throws no compiler errors, Micro will automatically compile and save a .class file with the same name as the .java file that was saved. However, when certain .java programs are saved as part of a package, Micro will place the compiled file in the wrong directory.
To reproduce:

Create a directory to serve as your project source. I'm using ~/Documents/JavaProjects/ProjectSource, but you can use anything, and I'll just refer to this as src from now on.
Create a folder such as src/MyPackage.
Navigate to src/MyPackage and, using Micro, create a file such as src/MyPackage/PackagedClass.java. The class should include package MyPackage; as the first line, and the contents can probably be anything, as long as the result is a valid Java class. Upon saving, notice that Micro has taken the liberty of creating a new directory, src/MyPackage/MyPackage, and within this new directory, the compiled .class file can be found, src/MyPackage/MyPackage/PackagedClass.class. This is the crux of the issue. When correctly compiled, the resulting compiled class should be found in the same directory as the original .java file. (The reason this is an issue is because a developer may wish to structure their .java files in accordance with their package structure, and so having Micro create the compiled .class files in the wrong directory creates hassle and confusion).

I've also been able to create situations where a .java file in the default package causes the same issue on a much larger scale. For example, I had a situation where I had a file in /home/[User]/Documents/JavaProjects/TestCore/Test.java that, when saved after being edited with Micro, created a class in /home/[User]/Documents/JavaProjects/TestCore/home/[User]/Documents/JavaProjects/TestCore/Test.class, and I have... no idea how or why this happens. I'm not able to consistently reproduce it, with some projects it happens and with some it doesn't
I apologize if I have made any errors in my explanation, and please let me know if there is anything I should clarify or change.
Specifications

Commit hash: 68d88b57
OS: Linux
Terminal: Konsole (KDE's Terminal Emulator)

The reason this happens is that you have the built-in linter plugin enabled, and that calls javac -d CURRENTDIRECTORY CURRENTFILE to get diagnostics. This means that if your working directory is src/MyPackage/ instead of src/ it will create a new directory. I'm not sure why it's even doing full compilation instead of just checking for errors like it does for other languages (maybe javac can't do that?).
Anyway it seems you can work around the issue by adding this to your ~/.config/micro/init.lua to override the default linter for java:
function preinit()
    linter.makeLinter("javac", "java", "javac", {"%f"}, "%f:%l: error: %m")
end

I'm hesitant to make a pull request to change the default as I don't know java or how javac works – maybe there was a good reason to explicitly specify the output dir?

Thank you for the quick response. Removing the -d flag from the javac call in the linter plugin would probably fix the issue, but you bring up a good point about how there might be some reason behind setting it up the way that it currently is that would break if changed. I also appreciate the workaround that you provided. Thank you so much for your help.


core: Migration to remove all legacy GCS versions

Description
We had kept the legacy GCS version as part of moving to a JSON format. It is now time to delete them all so that we stop handling them when scrubbing data source documents.
Risk
Low paths are pretty explicit.
Deploy Plan

deploy prodbox
Run from prodbox: cargo run --release --bin migration_clean_legacy_gcs

@flvndvd closing in favor of a PR on the other PR branch


Keep confidence channel as unsignec char (Reliability in TS)

Simple change of config.

In the LAS file at IA/Analyses/20221208_PredictedClassification_as_Reliability, with a readable Reliabilioty channel, type of channel was unsigned char with LASinfo.

Check this file for reading in TerraScan (obtained via simple pdal translate in CLI, from the readable file)

pdal_translation-confidence_as_uchar.zip


[eSteem mobile 2 iOS] Need to close and re-launh the app to view updated upvoter numbers & Values

Describe the bug
I was getting multiple upvoting notifications for my recent post and thought to launch the app to see them but when i launch the app i saw that there is only one upvote. I tried to pull down to refresh the page but it gave no result.
To Reproduce
Steps to reproduce the behavior:

Create a new post.
Check that it is visible in the profile page.
minimize the app pressing the home button of iphone.
Wait for a moment and maximize the app using double tapping on home button.
See the error. You can try to refresh the page by pulling down.

Expected behavior
If i am getting proper notifications of upvote then the upvoter numbers and its upvote value should be updated on post simultaneously.
Screenshots or Video
Smartphone (please complete the following information):

Device: [ iPhone6]
OS: [e.g. iOS 12]
Version

Additional context
Add any other context about the problem here.

It is fixed, you can test with next release (v2.1.1)


dockerfile  rbenv cannot install ruby

Hi,
dockerfile build install rbenv successfully and RUN rbenv  show
Usage: rbenv <command> [<args>]

Some useful rbenv commands are:
   commands    List all available rbenv commands
   local       Set or show the local application-specific Ruby version
   global      Set or show the global Ruby version
   shell       Set or show the shell-specific Ruby version
   install     Install a Ruby version using ruby-build
   uninstall   Uninstall a specific Ruby version
   rehash      Rehash rbenv shims (run this after installing executables)
   version     Show the current Ruby version and its origin
   versions    List all Ruby versions available to rbenv
   which       Display the full path to an executable
   whence      List all Ruby versions that contain the given executable

See `rbenv help <command>' for information on a specific command.
For full documentation, see: https://github.com/rbenv/rbenv#readme

but when I Run rbenv install 2.2.3
I got error
Downloading ruby-2.2.3.tar.bz2...
-> https://cache.ruby-china.org/pub/ruby/2.2/ruby-2.2.3.tar.bz2
Installing ruby-2.2.3...

WARNING: ruby-2.2.3 is nearing its end of life.
It only receives critical security updates, no bug fixes.

BUILD FAILED (Ubuntu 14.04 using ruby-build 20170914-5-gf41e847)

Inspect or clean up the working tree at /tmp/ruby-build.20171031090758.39
Results logged to /tmp/ruby-build.20171031090758.39.log

Last 10 log lines:
compiling prelude.c
compiling array.c
linking miniruby
generating encdb.h
./tool/mkconfig.rb:199:in `block in <main>': undefined method `dump' for nil:NilClass (NoMethodError)
	from ./tool/mkconfig.rb:198:in `each'
	from ./tool/mkconfig.rb:198:in `<main>'
make: *** [.rbconfig.time] Error 1
make: *** Waiting for unfinished jobs....
encdb.h updated

if i installed old-package or missed installing something

Hi, I checked each step, nothing found
finally  I  docker pull ubuntu:14.04  re pull and it works
installed 2.2.3 and 1.9.2-p320 successfully
- -)


Awesome tool, deserves more attention!

Thank you for writing such a great tool! Everything works like expected. Keep it up!

Thanks a lot. I very much appreciate such a comment.
Written for my own needs and kept small, I guess that's the key.