feature request: option in helm template to split output files when using {{ range }}

Hi
When using helm template to generate static yaml files, and when using --output-dir  , helm correctly generate multiple files according to source files..
would be useful if in I can add some hints (as  yaml comments  which also may include the output filename maybe ? ) to specify a different filename...
so when using a range to create a list of resources , (in one of my specific use-case it's a job to initialize/destroy resources on specific list of hostname whenre the pv/pvc with local storage would end )  those can be called/created/deleted using kubectl apply -f one by one without the "all or nothing"
because I can create some manteinance jobs/pod with proper configuration using the range but then I have to use them separatedly..
I hope to have been clear enough on why it would be useful ;)
thank you ,
Francesco

It's a bit late but, with nothing official I just decided to make something simple. Took like 10 minutes
As a script:
#!/usr/bin/env bash

# helm-fan-out

if [ -z "$1" ]; then
    echo "Please provide an output directory"
    exit 1
fi

awk -vout=$1 -F": " '
   $0~/^# Source: / {
       file=out"/"$2;
       print "Creating "file;
       system ("mkdir -p $(dirname "file"); echo -n "" > "file);
   }
   $0!~/^#/ && $0!="---" {
       print $0 >> file;
   }'

Put it in your path and run it with helm template --dry-run foo ./foo | helm-fan-out out
As a one liner:
helm template --dry-run foo ./foo | awk -vout=out -F": " '$0~/^# Source: /{file=out"/"$2; print "Creating "file; system ("mkdir -p $(dirname "file"); echo -n "" > "file)} $0!~/^#/ && $0!="---"{print $0 >> file}'

you can change the out directory to foo-out by changing -vout=foo-out

@washtubs like so thank you :)

Glad you like it @Jean-Baptiste-Lasselle . Please note I just updated it. I realized in the old version if there were multiple k8s resources in a file, the last one would clobber all the previous ones (e.g. if you have a service account followed by a deployment in the same template, the deployment would win). Now they are concatenated properly.

@washtubs I had a clean one template file / One Kubernetes Ressource Definition, but thank you so much for mentioning, and I want to mention the use case :
I have to debug a helm chart that's pretty complicated, SO I have to check all generated *.yaml components of the whole thing that Helm deploys
I mean maybe there a subject for a whole set of helm features, on the testing / debugging side... :
It'd be nice to have like a helm debug, wiht a helm template  --watch, and helm --watch so that you can modify the values.yaml of the Helm Chart under analyzis... something like that...
The best, to me, would be something similar to angular / karma / jasmine for angular coninuous testing/debugging , also with variables dynamic inspection on the fly etc...

@washtubs just to check, does this one-liner actually include your today's update ?
helm template --dry-run jbltest operations/am/ | awk -vout=operations/am/rendering -F": " '$0~/^# Source: /{file=out"/"$2; print "Creating "file; system ("mkdir -p $(dirname "file"); echo -n "" > "file)} $0!~/^#/ && $0!="---"{print $0 >> file}'

thanks @washtubs for the script. Perhaps that can be turned into a plugin.
Due to the way helm template works today, I do not foresee this becoming a feature for Helm. I do see this being a useful plugin, however. Please have a look at https://helm.sh/docs/topics/plugins/ for more details.
Thanks!

@washtubs just to check, does this one-liner actually include your today's update ?
helm template --dry-run jbltest operations/am/ | awk -vout=operations/am/rendering -F": " '$0~/^# Source: /{file=out"/"$2; print "Creating "file; system ("mkdir -p $(dirname "file"); echo -n "" > "file)} $0!~/^#/ && $0!="---"{print $0 >> file}'

This works great until you have multiple object in the same deployment file defined. it will overwrite them and keep the last one only.

This works great until you have multiple object in the same deployment file defined. it will overwrite them and keep the last one only.

That can be solved tweaking the awk script, example:
rm -fR tmp/cert-manager/; ./bin/helm-v3.8.2 template --set installCRDs=true --include-crds helm-charts/cert-manager | awk -vout=tmp/ -F": " '$0~/^# Source: /{file=out"/"$2; print "Creating "file; system ("mkdir -p $(dirname "file"); echo ""--- >> "file)} $0!~/^#/ && $0!="---"{print $0 >> file}'

For posterity here's an updated version of the script by @washtubs that puts the document separator (---) on the top and uses append (>>) to avoid clobbering existing files with the same file name:
#!/usr/bin/env bash
# Based on https://github.com/helm/helm/issues/4680#issuecomment-613201032
#
# helm-fan-out

if [ -z "$1" ]; then
    echo "Please provide an output directory"
    exit 1
fi

awk -vout="$1" -F": " '
  $0~/^# Source: / {
    file=out"/"$2;
    if (!(file in filemap)) {
      filemap[file] = 1
      print "Creating "file;
      system ("mkdir -p $(dirname "file")");
      print "---" >> file;
    }
  }
  $0!~/^# Source: / {
    if ($0!~/^---$/) {
      if (file) {
        print $0 >> file;
      }
    }
  }'

Here is another option how to split:
https://github.com/patrickdappollonio/kubectl-slice
I found it better for my cases then in comment above.
P.s. However in kubectl-slice there could be a problem with result file namings, it doesn't take it from comment '^# Source: /

It's a bit late, but with nothing official I just decided to make something simple. Took like 10 minutes

As a one liner:
helm template --dry-run foo ./foo | awk -vout=out -F": " '$0~/^# Source: /{file=out"/"$2; print "Creating "file; system ("mkdir -p $(dirname "file"); echo -n "" > "file)} $0!~/^#/ && $0!="---"{print $0 >> file}'

you can change the out directory to foo-out by changing -vout=foo-out

Thanks, it helps a lot, I change a little bit for my use case, which only get the filename not the absolute path:
helm template --dry-run foo ./foo | awk -vout=out -F"/" '$0~/^# Source: /{file=out"/"$NF; print "Creating "file; system ("mkdir -p $(dirname "file"); echo -n "" > "file)} $0!~/^#/ && $0!="---"{print $0 >> file}'

Keep in mind that the awk approach proposed in the comments won't work if you have multi-document templates. In such case, file names in # Source:  will be duplicated. Therefore, manifests will override each other.
If this is the case for you, you can use yq utility (`v4.x). This utility is YAML-aware, so it will split documents properly. See the docs.
Unfortunately, you cannot generate fancy names (or I haven't found how).
helm template foo ./foo | yq -s '"prefix-" + $index'

Small bugfix, this version did in some cases leave out the --- between multiple resources in one Yaml file.
#!/usr/bin/env bash
# Based on https://github.com/helm/helm/issues/4680#issuecomment-1152289297
# helm-fan-out

if [ -z "$1" ]; then
    echo "Please provide an output directory"
    exit 1
fi

awk -vout="$1" -F": " '
  $0~/^# Source: / {
    file=out"/"$2;
    if (!(file in filemap)) {
      filemap[file] = 1
      print "Creating "file;
      system ("mkdir -p $(dirname "file")");
    }
    print "---" >> file;
  }
  $0!~/^# Source: / {
    if ($0!~/^---$/) {
      if (file) {
        print $0 >> file;
      }
    }
  }'

why not use kubesplit

chatgpt helped escape this into a bash alias
alias yaml_split='awk -vout=out -F"/" '"'"'$0~/^# Source: /{file=out"/"$NF; print "Creating "file; system ("mkdir -p $(dirname "file"); echo -n "" > "file)} $0!~/^#/ && $0!="---"{print $0 >> file}'"'"''

helm template --dry-run foo . | yaml_split
I was kind of surprised helm didn't have this already

We have a similar situation. What worked for us so far:
helm template foo . --dry-run | yq -s '.kind + "-" + .metadata.name'


Hard code survey title into survey pages

Changes
Hard codes the survey title into the survey views, leaving the error pages blank
How to test

Checkout the branch
View the survey title next to the beta button on each survey page
Cause an error and see the survey title is missing from next to the beta button

Who can test
Anyone except @weapdiv-david

A few code style check failures

This title is already in the schema so why not use that rather than hard coding it everywhere? 👎

Rebasing

Can we remove markup from the tests before this is merged? I don't think the tests should be concerned with a particular class being used.


Add Guild.max_stage_video_users and Guild.safety_alerts_channel

Summary
Add the currently missing Guild.max_stage_video_users attribute and Guild.safety_alerts_channel property
What is the feature request for?
The core library
The Problem
In https://github.com/Rapptz/discord.py/pull/9318, support for Guild.max_stage_video_users and Guild.safety_alerts_channel have been added. It'd be nice to get that in d.py-self as well.
The Ideal Solution
I'm not familiar enough with the codebase (and running a diff of the guild.py files of discord.py and discord.py-self yielded this monstrous diff which was too intimidating for me) so I haven't dared to touch it myself, but in an ideal world the changes made in the PR should be able to be merged into here?
The Current Solution
I don't believe there is any.
Additional Context
No response

Implemented in the latest master!


Missing Notifier option under Cluster tools

I was looking in the documentation about adding the notifier but couldnt find it under Cluster tools as per following documentation:
https://ranchermanager.docs.rancher.com/v2.0-v2.4/explanations/integrations-in-rancher/notifiers
We are using v2.7 and in the latest documentation there is no mention about the steps to add notifiers.
https://ranchermanager.docs.rancher.com/reference-guides/rancher-project-tools#notifiers-and-alerts
Also there is no way to add notifiers in the latest version

From 2.5 the new Rancher UI was made default. Documentation regarding the UI pre 2.5 refers to the old UI.
Notifiers are part of the logging process, in the new UI they can be found after installing that app and going to the Logging section on the target cluster

Can we add Slack as notifier in version 2.7? Where can I find the documentation?


feat(configservice-configrule): remove remediation config

If you attempt to delete a config rule that has a remediation config it will fail, this fixes it so that if a remediation config is present it deletes that first.

:tada: This issue has been resolved in version 3.13.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


widen version bound for cfg-if

This will allow cargo to use a more recent version of cfg-if, and use fewer versions of cfg-if if it's used elsewhere in the crate graph

Whoops, these version bounds should be equivalent. Nvm!


Remove contentPadding approach

Codecov Report

Merging #231 (ce183a6) into main (492a7fe) will increase coverage by 0.10%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main     #231      +/-   ##
==========================================
+ Coverage   73.45%   73.55%   +0.10%
==========================================
  Files          66       66
  Lines        2976     2961      -15
  Branches      477      472       -5
==========================================
- Hits         2186     2178       -8
+ Misses        523      521       -2
+ Partials      267      262       -5

Impacted Files
Coverage Δ

...o/composelife/ui/InteractiveCellUniverseOverlay.kt
68.30% <72.72%> (+0.77%)
:arrow_up:

...yo/composelife/ui/action/CellUniverseActionCard.kt
72.38% <100.00%> (+0.48%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Gracefully handle problem in fetching stream partition count for consumption rate limiter

Setting up consumption rate limit requires knowing the stream partition count so that each partition rate limit can be calculated based on the topic consumption rate limit config. Connecting to stream for fetching the partition count can time out or face any other issue. This PR handles that connection failure by using the previously fetched value.

Codecov Report

Merging #9249 (c3d7a22) into master (6c5d85e) will decrease coverage by 2.81%.
The diff coverage is 53.33%.

:exclamation: Current head c3d7a22 differs from pull request most recent head e6eaa85. Consider uploading reports for the commit e6eaa85 to get more accurate results

@@             Coverage Diff              @@
##             master    #9249      +/-   ##
============================================
- Coverage     69.91%   67.10%   -2.82%
- Complexity     4764     4815      +51
============================================
  Files          1858     1381     -477
  Lines         99187    71954   -27233
  Branches      15087    11521    -3566
============================================
- Hits          69346    48282   -21064
+ Misses        24930    20169    -4761
+ Partials       4911     3503    -1408

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
67.10% <53.33%> (+0.01%)
:arrow_up:

unittests2
?

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

Impacted Files
Coverage Δ

...nager/realtime/RealtimeConsumptionRateManager.java
69.56% <53.33%> (-6.12%)
:arrow_down:

...va/org/apache/pinot/core/routing/RoutingTable.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/org/apache/pinot/common/config/NettyConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/pinot/common/metrics/MinionMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...g/apache/pinot/common/metrics/ControllerMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/BrokerQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/MinionQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ache/pinot/server/access/AccessControlFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/common/messages/SegmentReloadMessage.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/common/messages/TableDeletionMessage.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 714 more

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

We had discussed about a metadata cache. is that something that is coming in another PR (and this one is to get us through the immediate problem)?
thanks

Yes that will be another PR. This PR is just to make sure we properly handle stream connection exceptions.


Backport nix update to 0.24 for wayland-* 0.29

Would a PR to update nix to 0.24 for wayland-* 0.29 be acceptable?

I don't have any objections a-priori. You can make a PR to the v0.29.x branch for this.

Thanks for the quick work!


[#152150260] Add Datadog monitors for high CPU usage

What
We would like to know if any of our EC2 instances has high CPU usage as it
might affect our services. We introduce two alerts:

a warning if an EC2 instance has at least 50% CPU usage on average for two
hours or more
a critical alert if an EC2 instance has at least 80% CPU usage on average
for 30 minutes or more

Please note that there is at least 10 minutes delay for AWS metrics.
In the warning alert we still have to specify a critical threshold, but it will never fire. We have to do this because we want to check different time windows for the two alerts.
How to review

Enable Datadog in your dev environment, update the pipelines and run the create-cloudfoundry pipeline
BRANCH=cpu_usage_monitors_152150260 ENABLE_DATADOG=true SELF_UPDATE_PIPELINE=false make dev pipelines
The cf-deploy job should successfully create all your Datadog monitors (datadog-terraform-apply task)
Search for "#DEPLOY_ENV# CPU load" monitors and check whether the monitor parameters make sense and the original data is valid
To test the alerts then you can change the deploy_env filter to staging or prod and set the thresholds to a low value and the alerts should be present in 5-10 minutes. Please check if the alerts have the right title and message.

Who can review
Not me or @chrisfarms

Looks good 👍


Bash Prompt tweak doesn't apply

Describe the Bug
The bash prompt tweak does not apply on Raspberry Pi OS Lite due to the missing "fontconfig" package.
To Reproduce
Steps to reproduce the behavior:

Use a system that does not include the "fontconfig" package by default.

Expected Behavior
It should check if the package exists.
Checklist

[x] I have checked for duplicate issues.
[x] I have reviewed existing discussions.
[x] This issue is not included in the roadmap.
[x] This issue is present on both stable and development branches.

There already is a check present. Are you sure the issue is present on the dev release?


buildah push  to dir: creates .tar file which is not in tar format

buildah push  to dir: creates .tar file which is not in tar format

Steps to reproduce the issue:

./buildah from registry.fedoraproject.org/fedora:rawhide
mkdir fedora-rawhide
./buildah push registry.fedoraproject.org/fedora:rawhide dir:fedora-rawhide
ls -l fedora-rawhide
tar tvf fedora-rawhide/feec4342ed84f31082461f08a31662af85d61177bb83936ed5cc5a0ceb5755d4.tar

Describe the results you received:
# ./buildah from registry.fedoraproject.org/fedora:rawhide
Getting image source signatures
Copying blob sha256:7ecc8a985be6848a7ee66b80217c168de6e79dcbda3c01ee36a45367c3a3a6bc
 86.23 MiB / 86.23 MiB [====================================================] 9s
Copying config sha256:feec4342ed84f31082461f08a31662af85d61177bb83936ed5cc5a0ceb5755d4
 1.27 KiB / 1.27 KiB [======================================================] 0s
Writing manifest to image destination
Storing signatures
fedora-working-container
# mkdir fedora-rawhide
# ./buildah push registry.fedoraproject.org/fedora:rawhide dir:fedora-rawhide
Getting image source signatures
Copying blob sha256:02a71b4f9b794b507208390fac92f88382f10ca62be5073397a82cde9c6a586f
 251.90 MiB / 251.90 MiB [==================================================] 6s
Copying config sha256:feec4342ed84f31082461f08a31662af85d61177bb83936ed5cc5a0ceb5755d4
 1.27 KiB / 1.27 KiB [======================================================] 0s
Writing manifest to image destination
Storing signatures
# ls -l fedora-rawhide
total 257964
-rw-r--r--. 1 root root 264140800 Feb 16 20:12 02a71b4f9b794b507208390fac92f88382f10ca62be5073397a82cde9c6a586f.tar
-rw-r--r--. 1 root root      1299 Feb 16 20:12 feec4342ed84f31082461f08a31662af85d61177bb83936ed5cc5a0ceb5755d4.tar
-rw-r--r--. 1 root root       430 Feb 16 20:12 manifest.json
-rw-r--r--. 1 root root        33 Feb 16 20:12 version
# tar tvf fedora-rawhide/feec4342ed84f31082461f08a31662af85d61177bb83936ed5cc5a0ceb5755d4.tar
tar: This does not look like a tar archive
tar: Skipping to next header
tar: Exiting with failure status due to previous errors

Describe the results you expected:
If feec4342ed84f31082461f08a31662af85d61177bb83936ed5cc5a0ceb5755d4.tar has .tar suffix, I'd expected it to be in tar format.
Since it seems to be JSON, it should be named feec4342ed84f31082461f08a31662af85d61177bb83936ed5cc5a0ceb5755d4.json.
Output of rpm -q buildah or apt list buildah:
I'm running buildah built from master (09e7ddf5441f1734e5d920c354b8d2f6d1ac21a9)
Output of buildah version:
# ./buildah version
Version:       0.12
Go Version:    go1.9.3
Image Spec:    1.0.0
Runtime Spec:  1.0.0
Git Commit:    09e7ddf
Built:         Fri Feb 16 19:00:00 2018
OS/Arch:       linux/amd64

Output of cat /etc/*release:
# cat /etc/*release
Fedora release 27 (Twenty Seven)
NAME=Fedora
VERSION="27 (Twenty Seven)"
ID=fedora
VERSION_ID=27
PRETTY_NAME="Fedora 27 (Twenty Seven)"
ANSI_COLOR="0;34"
CPE_NAME="cpe:/o:fedoraproject:fedora:27"
HOME_URL="https://fedoraproject.org/"
SUPPORT_URL="https://fedoraproject.org/wiki/Communicating_and_getting_help"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="Fedora"
REDHAT_BUGZILLA_PRODUCT_VERSION=27
REDHAT_SUPPORT_PRODUCT="Fedora"
REDHAT_SUPPORT_PRODUCT_VERSION=27
PRIVACY_POLICY_URL="https://fedoraproject.org/wiki/Legal:PrivacyPolicy"
Fedora release 27 (Twenty Seven)
Fedora release 27 (Twenty Seven)

Output of uname -a:
Linux redacted.example.com 4.14.18-300.fc27.x86_64 #1 SMP Thu Feb 8 01:35:31 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Output of cat /etc/containers/storage.conf:
# storage.conf is the configuration file for all tools
# that share the containers/storage libraries
# See man 5 containers-storage.conf for more information

# The "container storage" table contains all of the server options.
[storage]

# Default Storage Driver
driver = "overlay"

# Temporary storage location
runroot = "/var/run/containers/storage"

# Primary Read/Write location of container storage
graphroot = "/var/lib/containers/storage"

[storage.options]
# AdditionalImageStores is used to pass paths to additional Read/Only image stores
# Must be comma separated list.
additionalimagestores = [
]

# Size is used to set a maximum size of the container image.  Only supported by
# certain container storage drivers.
size = ""

# OverrideKernelCheck tells the driver to ignore kernel checks based on kernel version
override_kernel_check = "true"

Urvashi can you take a look at this?

Fixed in #489


Usage statistic for site is wrong

For example: using LET for https://krasotkina.com 38.24% + 0.64% = 38.88%
But google statistic (see image) is
Chrome for Android | 42.3%
Chrome 77 | 22.02%
Android 76 browser | 8.55%
Opera 63 | 3.38%
Safari & Chrome for iOS 12.2-12.4 | 3.15%
hover over cell of browser table show 0% for Chrome for Android (78) and 0.96% for Android 76 browser

I found another example of wrong Google Analytics data usage calculation.
The data shows that almost every used browser has support but the summarized percentage shows only ~65%.

@larsbo Every time I see incorrect Google analytics data (low percentage), I delete it and import it again and the data from my sites is correct.

@zimorodokan wow, ok 😄
will try that!

I assume that the data somehow "decays". I.e. when you import it it is correct and the older the import is the lower are the values. That should definitely be corrected.
In addition to that, I suggest that the date range of the analytics data is shown somewhere. This will provide a hint to the user that they should update it regularily.
Alternatively, if possible, there should be an option to provide relative dates, i.e. automatically use data from the last x months.
Sebastian


Adding to LinkedIn munges dates--different dates in credential json and LinkedIn add credential

From LCW, one can create a public link. Those public links can be added to a LinkedIn profile using LinkedIn's shortcut.
Adding to the LinkedIn profile is changing or not respecting the Issue Date and Expiration Date from LCW.
Date time formats in JSON from LCW
Issue Date: 2020-05-22T00:00:00.000+00:00
Expiration Date: 2024-01-01T00:00:00.000+00:00
Corresponding dates that show in Linked In are April 2020 and November 2023.
https://verifierplus.org/credentials/eb6fb2cb-4c09-41a8-af52-cad6f16e13d0
{
  "@context": [
    "https://www.w3.org/2018/credentials/v1",
    "https://purl.imsglobal.org/spec/ob/v3p0/context.json",
    "https://w3id.org/security/suites/ed25519-2020/v1"
  ],
  "id": "urn:uuid:2a28c227-ebc6-4dab-9226-d3c541149057",
  "type": [
    "VerifiableCredential",
    "OpenBadgeCredential"
  ],
  "name": "Emerging Tech Entrepreneurship Level 1",
  "issuer": {
    "type": "Profile",
    "id": "did:key:z6MkpcJENgsDNoVct5MnzM49sydB35GvewXEdJeWdy68yi4j",
    "name": "San José City College",
    "url": "https://www.sjcc.edu",
    "image": {
      "id": "https://www.eslstation.net/wp-content/uploads/2020/05/sjcc-square.jpg",
      "type": "Image"
    }
  },
  "issuanceDate": "2020-05-22T00:00:00.000+00:00",
  "credentialSubject": {
    "type": [
      "AchievementSubject"
    ],
    "id": "did:key:z6Mkus9Gw6paSW638N1Qmv1Tc9L7cZJSQ7qF37KWjbmYN6q4",
    "name": "SJCC Jaguar",
    "achievement": {
      "id": "urn:uuid:24c58959-4549-405b-a4de-fb341ce94c95",
      "type": [
        "Achievement"
      ],
      "name": "Emerging Tech Entrepreneurship Level 1",
      "description": "Pilot digital credential for SJCC's Emerging Tech Entrepreneurship Level 1.",
      "criteria": {
        "narrative": "Students who complete the Emerging Tech Entrepreneurship certificates will be exposed to and become proficient in the coursework to become entrepreneurs in a technical field. This program combines the business skills necessary to make sound business decisions with the technical know-how to survive in the 21st century. Students will learn how to build and manage successful modern technical businesses."
      }
    }
  },
  "expirationDate": "2024-01-01T00:00:00.000+00:00",
  "proof": {
    "type": "Ed25519Signature2020",
    "created": "2023-03-15T14:44:32Z",
    "verificationMethod": "did:key:z6MkpcJENgsDNoVct5MnzM49sydB35GvewXEdJeWdy68yi4j#z6MkpcJENgsDNoVct5MnzM49sydB35GvewXEdJeWdy68yi4j",
    "proofPurpose": "assertionMethod",
    "proofValue": "z2oBzoV62Ft7h1Wcsyebw8jCSQX19wc8YJR6XxK8Bi7z6LTZs5H48afb4k3sq5MvLUMaYr9LMv1U2JsZQXWBA3ypX"
  }
}

If one adds a license or certification via the native UI on mobile, LinkedIn has the user select month and year but not day.
(Even if it's a date format issue, LinkedIn is basically subtracting a month in the display...which makes no sense.)

Dmitri working on this. Fixed in Android. Will be published to Apple store soon.

Does not work as expected for Android LCW Build 62.
Works as expected for iOS LCW Build 62.

When testing fix for Android please test again in IOS. Thanks!

This appears to work as intended in v2.0.9-build 64 in Android and iOS--correct dates are displayed in LinkedIn


Running multiple aimsun instances while training

Pull request information

Status: in development
Kind of changes: new feature
Related PR or issue: #761

Description
This PR allows for running of multiple Aimsun instances through rllib and stable_baselines.

Pull Request Test Coverage Report for Build 4699

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 90.869%

Totals

Change from base Build 4696:
0.0%

Covered Lines:
9215

Relevant Lines:
10141

💛  - Coveralls


With watch-demo, every time there's a new compile, a new webserver starts up.

Caused by #82

Can you please describe a little more on this? What's the expectation for this?


[OpenAI] Update Assistants sample

To handle more message types

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


SpotLight3d contribution to volumetric fog inconsistant when shadows are enabled or disabled

Godot version
4.0.alpha15.official (432b25d36)
System information
Windows 11 Pro (OS Build 22000.918), Vulkan Clustered, i74770k x86_64, GTX 1080 SLI x2 (driver version 516.94)
Issue description
Spotlight's fog contribution is inconsistant in the latest version of godot alpha with a fully updated device.
With shadows disabled, the light contributes to fog volume's lighting as expected. With shadows enabled, the Spotlight will only contribute to the fog volume's light under specific instances of distance from view and view angle.
Factors I have tested that influence contribution:

Enabling and disabling a spotlight's shadows.
Distance from view, but the light will not fade out, it's contribution will come and go at different distances.
Angle of light in view.

Factors I have tested that do not appear to influence contribution:

Scene geometry.
Environmental Volume Fog.
A Light's Shadow settings besides Enabling and disabling.
Fog Volume resolution.
Shadow atlas size.
Disabling SLI.

This occurs both in the editor and when playing the project.
Below demonstrates the issue, the light on the left has shadows disabled, the right has shadows enabled.

Steps to reproduce

Create a new scene with a world environment, enable volumetric fog and insert some fog into the scene, either globally, or with a fog volume.
Insert a Spotlight3D, enable shadows on it.
Reorientate the view around until you find an angle and distance where its contribution disappears.
Disable shadows on the created light and note that contribution returns.

Minimal reproduction project
spotlight-reproducton.zip

Duplicate of https://github.com/godotengine/godot/issues/57722.

Closing as dupicate.

closing as duplicate


Object tracker sample code not working

Object tracker sample: https://docs.luxonis.com/en/latest/pages/samples/object_tracker/ is not working.
In the pipeline creation the blob file seems to point out to resource which does not exist:
p = device.create_pipeline(config={
"streams": ["previewout", "object_tracker"],
"ai": {
#blob compiled for maximum 12 shaves
"blob_file": "/path/to/depthai/resources/nn/mobilenet-ssd/mobilenet-ssd.blob.sh12cmx12NCE1",
"blob_file_config": "/path/to/depthai/resources/nn/mobilenet-ssd/mobilenet-ssd.json",
"shaves" : 12,
"cmx_slices" : 12,
"NN_engines" : 1,
},
'ot': {
'max_tracklets': 20,
'confidence_threshold': 0.5,
},
})
Also if the file is pointed to for example:
"ai": {
#blob compiled for maximum 12 shaves
"blob_file": "/depthai/resources/nn/mobilenet-ssd/mobilenet-ssd.blob.sh14cmx14NCE1",
"blob_file_config": "/depthai/resources/nn/mobilenet-ssd/mobilenet-ssd.json",
"shaves" : 14,
"cmx_slices" : 14,
"NN_engines" : 1,
},
'ot': {
'max_tracklets': 20,
'confidence_threshold': 0.5,
},
})
Then i get this error: ERROR: requested CNN resources overlaps with RGB camera :( Any help?

Hi @jonathanleinola ,
Sorry about the trouble here.  I had been trying to figure out what is going wrong but didn't notice at first.  So I think you need to also specify metaout in your streams.  I think that is what is missing.
I think this is it, as if I specify metaout along with previewout and object_tracker in depthai_demo.py it does succesfull track my boat (the ceiling of the garage from which I am working):

This command was used:
depthai_demo.py -s metaout previewout object_tracker
But if I run without requesting metaout then there is no neural network data, and so no tracker (as the tracker works off the results of the neural network object detector output):

This was run with:
python3 depthai_demo.py -s previewout object_tracker
And notice no boat and no tracking/etc. and the NN FPS shows as 0.  In other words, not requesting the metaout from the neural network tells the firmware to not run the neural network.
So I think the only thing you need to change (keeping the sh12cmx12 model, your first attempt) is to add "metaout" to the streams, as below:
p = device.create_pipeline(config={
"streams": ["metaout", "previewout", "object_tracker"],
...

So the python3 depthai_demo.py -s metaout previewout object_tracker command is the CLI-equivalent of that.
Thoughts?
Thanks,
Brandon

Thanks! I think i'm now one step closer, but still i'm missing this file: mobilenet-ssd.blob.sh12cmx12NCE1? The script can not find it / (me neither)

Thanks! I think i'm now one step closer, but still i'm missing this file: mobilenet-ssd.blob.sh12cmx12NCE1? The script can not find it / (me neither)

Run python3 depthai_demo.py -sh 12, it will generate the required model : mobilenet-ssd.blob.sh12cmx12NCE1 at depthai/resources/nn/mobilenet-ssd/
Alternativel if you would follow the steps Brandon described previously, it would work as it is.
python3 depthai_demo.py -s metaout previewout object_tracker

Oh, thanks!! It's working now :)

Any ideas on how could I get the spatial data from the tracklet object? (x,y,z position)

Hi @jonathanleinola ,
So unfortunately we do not have that functionality implemented yet.  We had intended to have it out in December but are behind.  We are planning on implementing it in our Gen2 API (examples here), and the issue for it is https://github.com/luxonis/depthai/issues/293.
Largely we've ended up spending a lot more time then we expected on unknown unknowns discovered in functionality we already wrote in Gen2 and then fixing/refactoring/etc. to make this work properly/better.  Fortunately, a good number of these actually do involve making it so that tracking in 3D coordinates should work better and should be easier to implement.
I'm not sure on timeline for it yet, but @szabi-luxonis has been doing a lot of underlying work recently to allow this.  So he might have an idea as to when we will have this out in Gen2.  My gut is we'll probably have it in March.
In the meantime, it is actually possible to use Gen2 to grab the depth region of interest from a detected object or any ROI.  So you could actually use something like mosse (e.g. here run on the host, and our depth-averager node to accomplish 3D position of a tracked object.
So the flow would be: take ROI from object detection in Gen2, pass this to mosse iteratively, use the mosse ROI and feed this into depthai to grab the depth from the ROI, and connect this to the depth averager (another node) to get the 3D position.  (Alternatively you could just have the depth of the ROI returned to the host so you could do more sophisticated processing on it).
I know the depth averager node works (as we have several customers using it over SPI interface), but I don't know if we have an example yet in Python.  So I'll ask and circle back.
Thoughts?
Thanks,
Brandon

Hi Brandon,
What is the current situation with the development? Is there some experimental branch on the spatial data from the tracklet object? Also, thanks for the tips, I'll need to do some research on those.

Hi @jonathanleinola ,
So the unfortunate news is that we had intended to have this implemented in December, but just ended up behind.  So what is blocking it in Gen2 right now is a crash that occurs with our depth-averager node.  This is what would be used with tracking to give the XYZ data of a tracked object.  Once that is working (not sure on ETA, but @szabi-luxonis is actively on it), we will be able to shift to porting the tracker implementation from the Gen1 API to Gen2 and then integrating it with this depth-averager node.
Not sure on ETA... thinking this month.
In the meantime, we could probably make an example that runs the tracker and the averaging on the host (which would still run decently actually, even on a Pi, probably still 30FPS).  Would that be helpful?  We'd likely use mosse and the crop-node on device to get the ROI of the mosse tracklet, and then use averaging on the host to get the depth and then XYZ.
Thoughts?
Thanks,
Brandon

Hi @Luxonis-Brandon,
Yes, that would be really helpful and good luck with the development!
-Jonathan

Thanks and will do!

Hi, I'm also looking for Object tracking to work (ideally together with depth perception).
Currently I'm adopting CentroidTracker to work with Gen2 object detection.
But best solution would be to combine object detection and tracking in single NN. Is this also in plan?

Thanks @austy81 .  So actually @szabi-luxonis is working now on integrating an improved object tracker as a node in the Gen2 API.
So this would work separately with the NN (so that any NN that produces bounding boxes could be used with it).  I'm not sure how far that work is yet, but I suspect we'll probably at least have a PR with this functionality in the coming weeks.
And on an NN that has tracking built-in, yes, I think that's a great solution.  And such NNs should theoretically already be runnable, but we haven't yet tried them.
I know AlexeyAB has done a YOLO-LSTM implementation that does this.  Note sure if PINTO0309 has any of the object detection + tracking networks already converted.
https://github.com/PINTO0309/PINTO_model_zoo
Thanks,
Brandon


Add OpenDyslexic font as alternative user setting

From the forum: Adding the ability to use a font such as OpenDyslexic to improve the readability of the web interface would benefit app inventors. I don't think we want to make it the default, but having an option to switch it on (and store the state in UserSettings) would make the platform more accessible.

This would also benefit from #1593, since it moves the loading of UserSettings earlier in the app initialization, which means we can also apply a stylesheet with the font at that time.

I wanted to give it a try, know how to do in android studio.
Can u please show me where to make the required changes.
I m new in here.

If i m not wrong,there must be a folder for fonts , and there the Open Dyslexic font is to be added but i cannot figure out where is that fonts folder

@27vinayvyas Fonts are located in appengine/war/fonts/.
The real challenge here is the design for how this is presented to the app developer. We've tried to keep discovery somewhat easy by putting most things in the designer view. But one might want to turn this feature on before they start a project, or even before they've created an account (in the case of code.appinventor.mit.edu). A simple first pass might be to add an entry to the Help menu that can be enabled/disabled in the current session but doesn't persist. A second pass could add persistence.
To add menu items, please see TopToolbar.java in the appengine module.

I m having problem understanding how to do this
What does these two arguments in bold mean and what would be there value in this case??
fileItems.add(new DropDownItem(WIDGET_NAME_EXPORTPROJECT, MESSAGES.exportProjectMenuItem(),
new ExportProjectAction()));

@ewpatton
Sorry for the delay but its finally working(changing font to opendyslexic on a button click)
What i have done is that i have REMOVED  all the font-family attributes  from Ya.css and gwt.css EXCEPT from the body tag which has Roboto as default.
On clicking the Dyslexic font button, the Roboto font attribute is removed and the opendysleic font is added to the body tag.
All the other elements ,being body's child , gets the new font.
I am facing an issue, the font inside the main design screen doesnot seems to be controlled by Ya.css and gwt.css files. It sfont is unaffected when the above changes are made.
Which css files control their font??
I just wanted to clarify if what i am doing is right or not.
If so, i can send the pr

As there was little activity in pull request #1639 I started working on my own version. I used #1639 as a starting point. I now made it that the user's preferred font gets saved to the UserSettings object. I did run into an issue. When changing the font, the blocks' font gets changed, but the padding doesn't. It does render the blocks properly when reloading the page.

Before reload
After reload

I can solve this by reloading the window when a new font is selected. This is also done when selecting a new language, but this doesn't feel like it is the right way to do it.
I was also thinking about the user interface of the font switcher. Pull request #1639 made it that there were always two options in the Settings dropdown. I want to make it that when the regular font is selected the DropDownItem has a caption of Enable OpenDyslexic and when OpenDyslexic is selected the DropDownItem has a caption of Disable OpenDyslexic. What do you think?


Are there plans to open source the training data?

Are there plans to open source the training data?

Hey @wwfcnu! All the training data is open sourced on the Hugging Face Hub, c.f. https://huggingface.co/collections/parler-tts/open-source-speech-datasets-annotated-using-data-speech-661648ffa0d3d76bfa23d534


Render subviewport in editor

So, I just began using viewports and I was confused, that they didn't render anything in the editor. However, in a demo of Godot 2 I saw that someone did exactly that. So: Is it possible in Godot 3 or not anymore?

I see, you need to use ViewportTextures now.


Milestone: Created our own Enumerable

At this Milestone we,

Created a class MyList that has an instance variable @list.
In MyList class we implemented a method #each that yields successive members of @list and uses the MyEnumerable module.
We also Created a module MyEnumerable that implements
all?
, any?
, filter
Each class and module  has a separate .rb file.
We also verified our solution.

@BertrandConxy Thank you very much for reviewing our PR and approving it 👍


node-agent image tag in OperatingSystemConfig uses shoot kubernetes version instead of gardener version

How to categorize this issue?

/area control-plane
/kind bug
What happened:
After update to gardener v1.92.0, Workers don't apply updated OperatingSystemConfig. We found they try to download a non-existing node-agent image tag. Their OperatingSystemConfig -original contains the following lines
  - content:
      imageRef:
        filePathInImage: /gardener-node-agent
        image: europe-docker.pkg.dev/gardener-project/releases/gardener/node-agent:v1.27.12
    path: /opt/bin/gardener-node-agent
    permissions: 493

with the tag always being the shoot kubernetes version.
What you expected to happen:
The tag should be node-agent:v1.92.0
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
My best guess so far
https://github.com/gardener/gardener/blob/2dbec5f2a69f8855b00baf53aae1db574dff324b/pkg/gardenlet/operation/botanist/operatingsystemconfig.go#L53
is where the tag should be set to the gardener version, but the tag has been set already in
https://github.com/gardener/gardener/blob/2dbec5f2a69f8855b00baf53aae1db574dff324b/pkg/gardenlet/operation/botanist/operatingsystemconfig.go#L49
FindImages -> FindImage -> ToImage
ToImage applies the given <targetK8sVersion> to the source to produce an output image. If the tag of an image source is empty, it will use the given <targetVersion> as tag.
and so it is not nil anymore.
WithOptionalTag will set the 'Tag' field of the 'Image' to <tag> in case it is nil. If 'Tag' is already set, nothing happens.
Environment:

Gardener version: v1.92.0
No custom gardenlet IMAGEVECTOR_OVERWRITE at all
os-ubuntu v1.25.0, os-gardenlinux v0.24.0

/cc @rfranzke since you where the last to touch that code ❤

/assign


I am receiving zsh: command not found: dotnet and I am in a new terminal window what to do moving forward.

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/microservice-tutorial/install
Operating System: macos
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

What mac do you have and what version did you install?

Good evening
I have Macbook Air and version 12.
On Thu, 15 Sep 2022 at 02:11, Maira Wenzel @.***> wrote:

What mac do you have and what version did you install?
—
Reply to this email directly, view it on GitHub
https://github.com/dotnet/core/issues/7795#issuecomment-1247421584, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ARBAUZQ7MFXDEY5SLTOI773V6JSTLANCNFSM6AAAAAAQMBTFI4
.
You are receiving this because you authored the thread.Message ID:
@.***>


Finance Badges

Currently we have Google to sponsor the badge itself, Deutsche Telekom to sponsor one adapter each, and the GDG Berlin Android to contribute the code and the idea.
For the material goods, @mariobodemann volunteered to talk to the invested parties, get the money needed and order the parts online.

I just got the final confirmation from big G. As soon as the money is on my account I'll order the badges.

A truckload of money arrived on my account.


Strict mode on code generation

Strict mode should sound when:

[ ] invalid tag with go type
[x] invalid tag with go marshaler and unmarshaler (e.g binary)
[x] duplicate primary key
[x] duplicate column name

https://github.com/si3nloong/sqlgen/issues/33 is related to this


Refactor the Timeline to meet coding standards.

The Timeline widget is awesome and people love it; however, it's some of the oldest UI code in Cesium and is radically different than the rest of the code base.  We need to bring it up to snuff so that it meets our coding standards.  There are also additional features we would like to add but cannot do so until we have a way to validate and

[ ] - Refactor to use Knockout and view models and meet general coding standards.
[ ] - Document the API (there is currently no entry at all in the built Documentation)
[ ] - Add specs (we currently don't have any)]

Related issues: #773, #1768, #2066, #3555

Some discussion on the fourm

Thanks Matt. There's also an issue with mousewheel: when you can scroll the page Cesium is on, you can't mousewheel the timeline because the page scrolling moves the timeline away from the pointer, and the pointer needs to be hovering over the element to register mousewheel (for keyboard events only focus is needed, not hover over.) So when the timeline is hovered over page scrolling should be disabled (which seems to already be the case for the 3D canvas.)

+1
A standalone application would be ideal...

idea from the forum: add a way to select the current date/time by typing it in or pick from a calendar instead of having to use the slider

@hpinkos I had the same issue and used bootstrap's datepicker to create a solution:
var start_julian = clock.startTime;
var end_julian = clock.stopTime;

$("#start-datepicker").datepicker({
    onSelect: function(dateText, inst) {
        var dateAsObject = $(this).datepicker( 'getDate' ); //the getDate method
        start_julian = new Cesium.JulianDate.fromDate(dateAsObject);
        clock.startTime = start_julian;
        clock.currentTime = start_julian;
        timeline.zoomTo(start_julian, end_julian);
    }
});

$("#end-datepicker").datepicker({
    onSelect: function(dateText, inst) {
        var dateAsObject = $(this).datepicker( 'getDate' ); //the getDate method
        end_julian = new Cesium.JulianDate.fromDate(dateAsObject);
        clock.stopTime = end_julian;
        timeline.zoomTo(start_julian, end_julian);
    }
});

This is unlikely to be a priority anytime soon. If someone does want to address it, something along the lines of https://github.com/CesiumGS/cesium/issues/10876 would be more in line with current coding standards.
Tracking this isn't particularly useful because of its low priority, so I will close it.
Thanks!


Infer Package URL from references and other references issues for "commitish" URLs

In some case it may be possible to infer new package URLs from collected references. In this
CVE-2014-1904.pdf
we have these:

https://github.com/spring-projects/spring-framework/commit/741b4b229ae032bd17175b46f98673ce0bd2d485
https://github.com/spring-projects/spring-framework/commit/75e08695a04980dbceae6789364717e9d8764d58
https://github.com/spring-projects/spring-framework.git/741b4b229ae032bd17175b46f98673ce0bd2d485
https://github.com/spring-projects/spring-framework.git/75e08695a04980dbceae6789364717e9d8764d58

these are duplicates (even though they look different) and the two later ones are obsolete/dead
they represent a Package URL:
https://github.com/spring-projects/spring-framework/commit/741b4b229ae032bd17175b46f98673ce0bd2d485 means pkg:github/spring-projects/spring-framework@741b4b229ae032bd17175b46f98673ce0bd2d485

We may be able to get the Package URL (both the python packageurl and @TG1999 FetchCode may help for this inference.
And this is also a clear source commit information (though I am not sure if these are the commit that fixed of introduced the vuln... and as explained in #326 the data provenance is hard to trace.

I cannot easily debug this issues because of the lack of logging and history trail. I would need to be able to trace for each record the original data source and data it came fro. That does not need to be easy, but would need to be possible. Today it is neither easy nor possible AFAIK.

@pombredanne  this makes sense, but I'm not sure how to infer relationship between  the inferred commitish package and the vulnerability. For eg, there could be multiple commits which  fix the vulnerability incrementally, so  we can't be sure whether a particlular commit fully fixed a vulnerability .

@pombredanne  this makes sense, but I'm not sure how to infer relationship between  the inferred commitish package and the vulnerability. For eg, there could be multiple commits which  fix the vulnerability incrementally, so  we can't be sure whether a particlular commit fully fixed a vulnerability .

more about improving rather than strictly "data quality".


proc/variables: prefetch of target process memory

Prefetch the entire memory of structs and arrays and cache it instead
of issuing readMemory calls only when we get down to primitive types.
This reduces the number of system calls to ptrace that variables makes.
Improves performance in general, greatly improving it in some
particular cases (involving large structs).
Benchmarks without prefetching:
	BenchmarkArray-4         	      10	 132189944 ns/op	   0.06 MB/s
	BenchmarkArrayPointer-4  	       5	 202538503 ns/op	   0.04 MB/s
	BenchmarkMap-4           	     500	   3804336 ns/op	   0.27 MB/s
	BenchmarkGoroutinesInfo-4	      10	 126397104 ns/op
	BenchmarkLocalVariables-4	     500	   2494846 ns/op

Benchmarks with prefetching:
	BenchmarkArray-4         	     200	  10719087 ns/op	   0.76 MB/s
	BenchmarkArrayPointer-4  	     100	  11931326 ns/op	   0.73 MB/s
	BenchmarkMap-4           	    1000	   1466479 ns/op	   0.70 MB/s
	BenchmarkGoroutinesInfo-4	      10	 103407004 ns/op
	BenchmarkLocalVariables-4	    1000	   1530395 ns/op

Improvement factors:
	BenchmarkArray				12.33x
	BenchmarkArrayPointer		16.97x
	BenchmarkMap				 2.59x
	BenchmarkGoroutinesInfo		 1.22x
	BenchmarkLocalVariables		 1.63x

This is great, I'm very excited about this patch. Is there any reason this is on top of #315? That PR has a legitimate failure on OSX, which is causing CI to fail on this PR.
I will review the code tomorrow morning.

This is great, I'm very excited about this patch. Is there any reason this is on top of #315?

No, it's a mistake. I also didn't notice the issue with #315

Decoupled from #315

One small nit, otherwise LGTM.


MVP Relayer Tracking Issue

Summary
The QBG is significantly simpler than the normal gravity bridge, and can therefore fit into a significantly simpler design. Notably, since the bridge is only one way, it only requires one or more relayers.
Core

[x] Create an RPC endpoint for DataCommitment #620

App

[x] Copy over portions of the gravity module #197
[x] Add a parameter that indicates the frequency for which the validators are expected to sign DataCommitments #198
[x] Add MsgSubmitDataCommitment and corresponding functionality #199
[ ] Document the new qgb functionality #217

QGB

[x] celestiaorg/celestia-app#327
[x] celestiaorg/celestia-app#328
[x] celestiaorg/quantum-gravity-bridge#9
[ ] Write ADR for MVP of the QGB

Even though the gravity bridge supports certain types of slashing, I didn't include this in MVP as it is not strictly necessary.

I think we can close this imo. We continue to write specific ADRs which are the only tasks left, and then we can write specific issues for the new relayer design that are tbd


🐛 Bug Report: EntityLayout.Route does not work with subroutes

📜 Description
When a component inside EntityLayout.Route has its own subroutes attached and when navigating to subroute, clicking the tab does not return to the original tab content. This leads to a problem that every subpage inside needs to have a separate back button to return the user to the root.
👍 Expected behavior
When clicking a tab that is already active, it should redirect to the root page of the content in case the root points to a different location than the current one.
👎 Actual Behavior with Screenshots
Nothing happens
👟 Reproduction steps

Add Jenkins plugin according to instructions (https://github.com/backstage/backstage/blob/2c91a2bb60d44c6522f2818cc43a68b38422c54d/plugins/jenkins/README.md)
Add annotation to add a job for some entity and necessary UI changes to add the CI/CD entity layout route
Run Jenkins build so that it's visible in the Backstage
Click on the build in Backstage to go to the DetailedViewPage
Click the CI/CD tab again -> Nothing happens

📃 Provide the context for the Bug.

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

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Yep that doesn't seem like desirable behavior, let's fix it! 😁 It seems like the tab change callback is only called when changing tab, I think the best fix here might be to call it any time a tab is clicked, even if it's already the main one? https://github.com/backstage/backstage/blob/0870577bd4ac6922ff999a0207c2aed71512d499/packages/core-components/src/components/TabbedLayout/RoutedTabs.tsx#L83

Yeah, that sounds like a good fix. Now we always have to have a back button if creating subviews inside the tabs and it's not very good for the UX.

@drodil @Rugvip help me understand what we want to achieve here with example.
Like if im on todo tab and click todo tab again onTabchange  does not run but header tab onchange runs.

Suggested the above fix in #20228


chore: Update docs with working sample

Pull request

What this PR does / why we need it
Updating docs with working and referenced sample.
Which issue(s) this PR fixes

Fixes #622
Describe testing done for PR

Additional information or special notes for your reviewer

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (58783ca) 70.10% compared to head (830f9f4) 70.10%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #642   +/-   ##
=======================================
  Coverage   70.10%   70.10%
=======================================
  Files          17       17
  Lines         950      950
=======================================
  Hits          666      666
  Misses        263      263
  Partials       21       21

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Kanboard behind the proxy

Guys from our IT department are a bit paranoid, so Kanboard is deployed to the server behind proxy.
Everything works fine except integrations which rely on webhooks, for example Slack integration.
I didn't find any configuration options or even code related to configuring proxy server for outgoing webhooks.
Is this possible, to use integrations while server is behind proxy?

I just added this feature in the dev version: https://github.com/fguillot/kanboard/blob/master/doc/config.markdown#http-client-proxy
The next stable release will include that.

Thanks, Frederic! I will test this code tonight.

Work like a charm :)


Support multiple init files

If I have some generic setup code (helpers) and some contract-specific code (see #148) it would be nice to import both of them without merging the source files. This could be done if we support multiple instances of --init in packages/cli/src/cli.ts
export function main(originalArgs: readonly string[]): void {
  const parser = new ArgumentParser({ description: "The CosmWasm REPL" });
  parser.addArgument("--version", {
    action: "storeTrue",
    help: "Print version and exit",
  });
  parser.addArgument("--init", {
    metavar: "FILEPATH",
    help: "Read initial TypeScript code from file",
  });
  // ...
    if (args.init) {
    init += fs.readFileSync(args.init, "utf8") + "\n";
  }

@webmaster128 can you take a look at this? Since you wrote these cli arg parser.
I will be updating some examples (#147)

Sure, sounds like a simple change

Thanks


⚠ Bofrost.at has degraded performance

In 9f2a2b0, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1295 ms

Resolved: Bofrost.at performance has improved in cf9a0ba after 13 minutes.


bug: recommended not working after removing specs for some plugins

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.10.0
Operating system/version
Fedora 38
Describe the bug
After removing specs for some plugins in a lua module that returns a table containing recommended keyword,  during the startup of Neovim, an error is reported as:

...local/share/nvim/lazy/lazy.nvim/lua/lazy/core/plugin.lua:67: attempt to call method 'find' (a nil value)

Steps To Reproduce

Enable dot
Edit plugins/extras/util/dot.lua, comment out the specs for nvim-lspconfig and mason.nvim (Check the Repro section to see my dot.lua)
Restart Neovim.

Expected Behavior
It should work without error raised, but only losing some features. In my case, I only need treesitter highlighting but not all lsp features.
Repro
My `plugins/extras/util/dot.lua` (This is the only file modified, and everything worked well before the modification):

---@type string
local xdg_config = vim.env.XDG_CONFIG_HOME or vim.env.HOME .. "/.config"

---@param path string
local function have(path)
  return vim.uv.fs_stat(xdg_config .. "/" .. path) ~= nil
end

return {
  recommended = true,
  desc = "Language support for dotfiles",

  -- add some stuff to treesitter
  {
    "nvim-treesitter/nvim-treesitter",
    opts = function(_, opts)
      local function add(lang)
        if type(opts.ensure_installed) == "table" then
          table.insert(opts.ensure_installed, lang)
        end
      end

      vim.filetype.add({
        extension = { rasi = "rasi", rofi = "rasi", wofi = "rasi" },
        filename = {
          ["vifmrc"] = "vim",
        },
        pattern = {
          [".*/waybar/config"] = "jsonc",
          [".*/mako/config"] = "dosini",
          [".*/kitty/.+%.conf"] = "bash",
          [".*/hypr/.+%.conf"] = "hyprlang",
          ["%.env%.[%w_.-]+"] = "sh",
        },
      })

      add("git_config")

      if have("hypr") then
        add("hyprlang")
      end

      if have("fish") then
        add("fish")
      end

      if have("rofi") or have("wofi") then
        add("rasi")
      end
    end,
  },
  --[==[
  {
    "neovim/nvim-lspconfig",
    opts = {
      servers = {
        bashls = {},
      },
    },
  },
  {
    "williamboman/mason.nvim",
    opts = function(_, opts)
      opts.ensure_installed = opts.ensure_installed or {}
      vim.list_extend(opts.ensure_installed, { "shellcheck" })
    end,
  },
  --]==]
}

You shouldn't customize the source paths of the plugins you want. Just put your customizations in /lua/plugins/ directory.

That error can only happen if you just keep the treesitter spec in there.
Either way, recommended and desc are not support by lazy.nvim. That's a LazyVim thing. You shouldn't use that in your own specs.


[JS, TS] Add Monarch support for private identifiers

Fixes #3254
This is a very simple solution that just allows the # character as the start character of identifiers for all identifiers. I don't know Monarch enough to know if there is a way to constrain this to identifiers in classes, but I doubt there is. In any case if you use the # character at the start of e.g. a variable name the language service raises an error diagnostic, so that this might not be a problem? Maybe this simple solution is enough for this issue. Let me know what you think, or feel free to discard this.

Local Playground Link: https://microsoft.github.io/monaco-editor/playground.html?sourceLanguages=http%3A%2F%2Flocalhost%3A5002%2Fout%2Flanguages%2Famd-tsc#XQAAAAJoAQAAAAAAAABBqQkHQ5NjdMjwa-jY7SIQ9S7DNlzs5W-mwj0fe1ZCDRFc9ws9XQE0SJE1jc2VKxhaLFIw9vEWSxW3yscwzblk-_1uhs0-3YGTBOsVJ4S4FwfCDQb5-2uUHXTVAefbVSd9IzaZ_vv9VR9KugsKikU_wkLK9jT_o8SJxwvSkZPE3r8N8C4O8JIRyP6PwIhuEzac9gRZJKV5O9_hRAHvieANI4sR5MKuH1inQaY1HbVLv0DvpOrvrDW6_vt1nHUY5cvnsy0iVQoS8hn2gWkAT4c2hWqFP24-1CdV2IAbzISiaOAa-IE1cC042gJ9VmprZ1KuCo6ic8uz7shuG3cxVt_qoOHa5r6Qz6qCXoH-gAs_coxt1vAO8lWJWMLtOTMA3__77F3Y

Thanks!
Also: I see you used the playground to debug/develop your PR. Did you run into any issues?

Also: I see you used the playground to debug/develop your PR. Did you run into any issues?

No issues! I really like this experience and it helped me immensely with the other PR for link and editor openers I contributed to the VS Code repo, because you could see and debug the changes immediately. The only thing I had to do for my machine was to change ports, but that was easy enough.


Add compact density for Icon

With the addition of separate compact icons we need to update Icon to support this. How this will be implemented in code will depend on #1826

[ ] React

[ ] Design

[ ] Documentation

Edit:
Notes:

Default size will need to be adjusted to 18x18 for small icons when density is compact, or else they become larger than default in a 24x24 size.

After meeting on 19 jan, we decided that the small icon size would be used whenever sizes 16 or 18 are used.


Class docstring does not need blank line before

The PEP 257 recommendation for a class docstring states, in part:

Insert a blank line after all docstrings […] that document a class -- generally speaking, the class's methods are separated from each other by a single blank line, and the docstring needs to be offset from the first method by a blank line.

That is covered (correctly) by the D204 check.
There is no mention of blank line before the class docstring. The D203 check is incorrect to require a blank line.
Please remove the D203 check as it is not a PEP 257 recommendation.

I would argue, further, that Pydocstyle should check that there is no blank line between the class statement and the class docstring.

Hi @benf-wspdigital,
Pydocstyle contains checks for more than just PEP 257. Regarding D203 - PEP 257 used to recommend a blank line before a class docstring. It has been edited to remove this recommendation and therefore D203 is not checked by default in pydocstyle. It still exists so projects with the old standard can still remain consistent. To support the new recommendation, D211 was a added to make sure no blank lines exists before a class docstring and it is enabled by default.

PEP 257 […] has been edited to remove this recommendation and therefore D203 is not checked by default in pydocstyle.

Which version of Pydocstyle disabled D203 by default?

How can a user interrogate pydocstyle to know which checks are default, which are disabled?

Version 0.7.0, you can see the release notes here: http://www.pydocstyle.org/en/latest/release_notes.html#october-9th-2015

Version 0.7.0, you can see the release notes here […]

Thanks, I see the release notes there.
I was getting the D203 check by default, with Pydocstyle version “1.1.1”. What could have caused that?
How can a user interrogate Pydocstyle directly, to know which checks are default, which are disabled?

@benf-wspdigital Unfortunately, there is currently no way to interrogate pydocstyle as to the defaults. In you case, have you use the --ignore flag for some other error code? If so, it would bring back D203 if you haven't specifically ignored it. You can use the --add-ignore instead if that's the case.


[Admission] Forbid to remove the Template object if one or more Deployment references it

I'd like to clarify a few details.
We have the following hierarchy:
for AWS
MachineDeployment
  - K0sWorkerConfigTemplate
  - AWSMachineTemplate

for Azure
MachineDeployment
  - K0sWorkerConfigTemplate
  - AzureMachineTemplate

So, we need to wait for MachineDeployment termination before K0sWorkerConfigTemplate, AWSMachineTemplate/AzureMachineTemplate (respectively to provider type).

@slysunkin It will be enough to check the ManagedCluster existence that references (spec.template) the Template that was triggered for the removal and forbid it if at least one ManagedCluster is found. Similarly, it's currently done for Management object removal: https://github.com/Mirantis/hmc/blob/main/internal/webhook/management_webhook.go#L65 No need to check internal cluster objects.


Tidy map rummager code

Following on from https://github.com/alphagov/finder-frontend/pull/944.
A bit of code tidying from PR comments.

 Mapping Rummager request/response in Finder Frontend


[bn] Localizes content/bn/docs/tutorials/kubernetes-basics/update/update-intro.html

Signed-off-by: Asif Zubayer Palak<EMAIL_ADDRESS>Localizes content/bn/docs/tutorials/kubernetes-basics/update/update-intro.html  to Bengali
This is an initial PR as may need some improvements!
Tracking issue: #31677

/label tide/merge-method-squash


adjacency_matrix methods added (GH25)

Codecov Report

Merging #43 (fa126b6) into master (59ea94d) will decrease coverage by 1.26%.
The diff coverage is 84.61%.

@@            Coverage Diff             @@
##           master      #43      +/-   ##
==========================================
- Coverage   97.28%   96.02%   -1.27%
==========================================
  Files          12       13       +1
  Lines         553      578      +25
==========================================
+ Hits          538      555      +17
- Misses         15       23       +8

Impacted Files
Coverage Δ

piso/graph.py
78.94% <78.94%> (ø)

piso/__init__.py
76.19% <100.00%> (-18.81%)
:arrow_down:

piso/accessor.py
98.50% <100.00%> (+0.06%)
:arrow_up:

piso/docstrings/accessor.py
100.00% <100.00%> (ø)

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 59ea94d...fa126b6. Read the comment docs.


No push token

I have following the documentation and trying to implement to my project, but keep getting "no push token" in my onesignal user dashboard. So i decided to clone the example by creating new project, but no luck, always "no push token". I see in event listener log, device.pushToken always null. Did anyone also have similar issue with me? Thank

Are you using real device? Simulators won't get push token.

This is wat i get in my verbose log : https://github.com/OneSignal/OneSignal-Android-SDK/issues/229

Hi @avishayil . So is it safe to assume that the pushToken would always be returned in a real device? I also don't get a pushToken in simulator which according to me is justified. I am just worried how sure we can be, about getting a pushToken in real device!


Persistent storage

We should store the open documents and the artwork within each document in localStorage or IndexedDB. Other state, like the current tool and working colors, would be a nice bonus. This helps restore previous sessions if the user accidentally closes the browser tab (for example, while intending to use a keyboard shortcut to close a document tab).

A thing to keep in mind is that, as far as I know, localStorage is usually restricted to 5MB, so once documents can contain bitmap images that might become a problem.

That's true, and localStorage is also unfortunately restricted to the string data type only, so you have to convert it to base64 or JSON which takes up even more storage space. But I think bitmaps are still a bit further away on the roadmap.

I am looking at this issue and had a question about which component should be responsible for doing this. Since I think additional work is needed based on the decision we make, I wanted to bring it up.
From the UI side, the Document component has the contents of the canvas but doesn't know about other tabs. The Workspace component knows about the tabs but it only knows the index and the name of a tab, not the content.
So it makes it tricky to decide who should be responsible for persisting the content. A simple solution would be to make sure that the Document knows what its index is and let each document store its content based on index. Another solution would be to expose the content of each tab to the Workspace component and let it handle persistence.

I believe the plan is for the backend to serialize the state of its documents, then send a message to the frontend which the Response Handler receives and calls the localStorage API.
Perhaps the UX could work based on File > Save and File > Open. We may want to provide the user a way of naming unsaved documents also, but I can worry about the frontend implementation of that and we can stick to the default document names for now.
I think the first step for this might be implementing a way of downloading and uploading Graphite documents, perhaps with a .gsvg file format ("Graphite SVG"). This format will be used until Graphite 0.2 when we have a proper node graph and the .gdd ("Graphite Design Document") format. Once that step is implemented, then we can worry about including that in localStorage.

@mfish33 you might be interested in this as the next logical step in your work on the web/Rust document save status infrastructure. Feel free to self-assign if you're interested in doing it next. If it's not interesting to you, or you don't have time in the next few weeks, no big deal if you prefer not to (this is merely a suggestion).


some import paths, filenames updated and json output added

some imports and file mismatches corrected.

I have tested the project after the above changes, works fine.


Create fr.json

Thanks for the contribution! I will include it in the next release (probably 2.1.0)

No problem ! Btw seasons are already translated in the game, you should probably get them directly from the game ToDrew mod do.
For the unified version you also have
  "new-reminder.recurring-label": "Récurrent",
  "new-reminder.interval-label": "Intervalle",
  "new-reminder.interval-explaination": "1: journalier, 2: tous les 2 jours, etc...",


How use custom Query for filter facet ?

I want to have my filter list and when I click on one of them query the facet . Is there any way to do that ?
I have a refinementlist that list my facets but I need another menu to query into facets.

Hey @mahdi-gh1373 do you mean searchForFacetValues: true? I'm not sure if I completely understand the use case you had in mind. Maybe a small diagram would do wonders?

Hi @Haroenv ,

I have a nav menu like this. each of SubCat items are my facets.I want to filter my result when I click on each of them.

For that you can make your own version of a hierarchical menu using connectHierarchicalMenu. Read more about how to use the connectors in this guide

Thanks @Haroenv  I will check that.


SIGPIPE should kill now/cancel a deployment

SIGPIPE is raised with a process if an output pipe for one of the standard file descriptors breaks during execution.
We need to cancel/roll back a deployment if SIGPIPE or EPIPE are raised via a signal or fd operation, respectively.
@rauchg pointed out epipebomb as an example for how some of these are handled.

Thank you for creating this issue!
Please note that – in order for us to be able to handle each feature request with care – we have to consume all of them through a unified pipeline that makes it easier for us to prioritize, track and progress on the features our user base is interested in.
In turn, the Issues tab on this repository is best only be used for reporting bugs, which we can immediately act on.
For well-explained feature requests (like the one you just posted), please contact us at<EMAIL_ADDRESS>so that we can consider adding it to our roadmap.


Fixed double-JSONing; incorrect name translation

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Ack, forked wrong branch; redoing.


Awesome packages, but where is the saf_util repository?

Hi, first of all thank you so much for these packages, they are very well designed and work perfectly! I was desperate to find a way to use SAF with Flutter and it took me a long time to find these gems.
I was wondering however where the GitHub repository of saf_util went, as I get a 404 error when trying to access it at the address specified on pub.dev ?

@maelchiotti thanks for reporting this. I just realized saf_util was a private repo on my side. Updated it to public now :D


[API] Set server time as UTC

The default server time must be UTC, and the client can change it to the user's timezone.
So, it will be great to fix every datetime.now() to UTC in the API server.

close at #992


Fix infer module for uppercase extensions

Fix the infer_module_for_data_files function when passed a data file name with uppercase extension, e.g. filename.TXT.
Before, None module was returned.

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


[VL]Offlaod expand with bigger project size

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
Allow to offload expand when it's projection head size > child output size + 1.
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
manual tests

@JkSelf PTAL

Can you add the unit test for this case?


Close Button

Would be nice to have a close Button for easy usage on iOS

Hi @guenth39, sorry for the late response.
Because our maintainers don't have iOS platform to test, could you help add the close button on iOS and create a pull request?


Multiple rows & responsive issues

Unable to change the number of rows in the responsive settings.
http://jsfiddle.net/jpvnufcL/
Steps to reproduce the problem:

Scale your browser window down from a desktop size to a mobile size or load the page at a mobile size.

What is the expected behavior?
That the number of rows and visible slides would change to the correct settings once the breakpoint is hit. In this code example (scaling from desktop to mobile), it should go from 3x2 to 2x2 to 1x1.
What is observed behavior?

If your browser window is 750px wide and you load the page, you'll see 6 slides (3x2) and no arrows.
Resizing the browser down from 1920px wide to 750px wide, you end up with 4 slides (2x2) and no arrows.

More Details

Which browsers/versions does it happen on?
The above functionality is from FF 50 on Windows 7. However, the functionality is similar in Chrome 54 on Windows 7.

Which jQuery/Slick version are you using?
jQuery 2.1.4
Slick 1.6.0

Did this work before?
This has never worked, it's new code.

Any update on a solution to this?


⚠ Samsung has degraded performance

In 6b905bb, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2306 ms

Resolved: Samsung performance has improved in db23d31.


feat: ✨ initial run parameters are added

Description

Run parameters  added to table to show via mouse hover tooltip
RT-DETRs are moved to single folder

List any dependencies that are required for this change.
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] This change requires a documentation update

How has this change been tested, please provide a testcase or example of how you tested the change?
YOUR_ANSWER
Any specific deployment considerations
For example, documentation changes, usability, usage/costs, secrets, etc.
Docs

[ ] Docs updated? What were the changes:

@onuralpszr, this PR should come before YOLO-NAS, as we're not quite sure about the results.

@onuralpszr, this PR should come before YOLO-NAS, as we're not quite sure about the results.

I read it. I am also working on other run params for yolov9 If you notice as well.

Final looks


Not recognizing bootstrap-settings.json at runtime

I have a project correctly setup as instructed - During the build phase, the plugin detects the client/bootstrap-settings.json file and correctly populates it. On the client, however, I get a warning message on the console saying "Bootstrap disabled. Create a file named 'bootstrap-settings.json' to enable.".
I don't really know what to do in this situation, as the build phase is occurring (apparently) ok.
Does anyone have any idea of what could be happening?
Thanks in advance,
/Edson Carlotti

Correction: Bootstrap CSS files are being generated and I can locate them in the generated APP. Bootstrap JS files is what I'm unable to find and that is the main reason why my application is failing to run.

I forced the initialization of the JavaScript library by configuring the explicitly setting the Javascript expose setting to TRUE and adding an explicit import of the generated JavaScript library in my project's client main code (e.g. import './stylesheets/bootstrap.js'). After doing this, I still get the console message telling me that Bootstrap is disabled (because the default initialization is not working) but I get Bootstrap running OK.


undefined method `model_name' for ProveKeybase::Configuration:Class

I'm following the starting guide and calling https://localhost:3000/prove_keybase/config.json yields this result since the Configuration class is not an ActiveModel object.
A workaround is to add the following lines to the prove_keybase.rb initializer:
ProveKeybase::Configuration.class_eval do
  extend ActiveModel::Naming
  include ActiveModel::Conversion
  def persisted?
    false
  end
end

thanks. i'm on it. looks like something with a dependency update causes this behavior to change. fix coming...

hopefully fixed now 🤞
lemme know either way when you get a chance.

@xgess it's working, thanks a lot!


Memory corruption after constructing an absl::BitGen

Describe the bug
Constructing an absl::BitGen can corrupt memory.
Built with Clang using Bazel. The memory corruption is easiest to see using msan.
Steps to reproduce the bug
Clone this repo, which has a minimal reproducing example: https://github.com/gumwreath/absl-bitgen-broken
Then, run this command:
bazel run -c dbg --copt=-fsanitize=memory --cxxopt=-fsanitize=memory --linkopt=-fsanitize=memory :random_test

msan will show a use-of-uninitialized-value error. This happens on the line after the absl::BitGen is constructed, specifically when the std::endl is written to std::cout. Seeing as std::cout and std::endl presumably aren't buggy, absl::BitGen must have corrupted some data used by cout or endl.
(However, more complex examples cause segfaults even without msan).
This is the minimum reproducer code included in that repo:
#include <iostream>

#include "absl/random/random.h"

void MRandom() {
  absl::BitGen gen2(std::seed_seq{1, 2, 3});
  std::cerr << "A X" << std::endl;
  return;
}

int main(int argc, char** argv) {
  MRandom();
  return 0;
}

What version of Abseil are you using?
Tested with several versions, including the latest commit hash 5fa65f28e46e86c44966a1ca8a727a329d9c1ff8.
What operating system and version are you using
Ubuntu:
Linux MS-7A34 5.15.0-50-generic #56-Ubuntu SMP Tue Sep 20 13:23:26 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

What compiler and version are you using?
Ubuntu clang version 14.0.0-1ubuntu1
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin
Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/11
Selected GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/11
Candidate multilib: .;@m64
Selected multilib: .;@m64

What build system are you using?
bazel 5.3.1

Uses the toolchain included in the repo I provided. This is a minimal toolchain to use clang instead of gcc.

https://github.com/google/sanitizers/wiki/MemorySanitizerLibcxxHowTo
Correct use of MemorySanitizer requires that all code, including the standard library, be built with MemorySanitizer, otherwise you can get false positives from the standard library. This unfortunately means it is nearly impossible to use distro provided toolchains with MemorySanitizer.
I tested your example with Clang using a MemorySanitzer built libc++ and didn't get any errors. Unfortunately I don't have an MemorySanitizer built libstdc++ available.
You can try this yourself a Docker container:
docker run -it gcr.io/google.com/absl-177019/linux_hybrid-latest:20220217 /bin/bash

cd
git clone https://github.com/gumwreath/absl-bitgen-broken
cd absl-bitgen-broken

export CC="/opt/llvm/clang/bin/clang"
export BAZEL_CXXOPTS="-std=c++17:-nostdinc++"
export BAZEL_LINKOPTS="-L/opt/llvm/libcxx-msan/lib/x86_64-unknown-linux-gnu:-lc++:-lc++abi:-lm:-Wl,-rpath=/opt/llvm/libcxx-msan/lib/x86_64-unknown-linux-gnu"
export CPLUS_INCLUDE_PATH="/opt/llvm/libcxx-msan/include/x86_64-unknown-linux-gnu/c++/v1:/opt/llvm/libcxx-msan/include/c++/v1"
/usr/local/bin/bazel --noworkspace_rc test ... \
  --compilation_mode=dbg \
  --copt="-fsanitize=memory" \
  --copt="-fno-sanitize-blacklist" \
  --features=external_include_paths \
  --linkopt="-fsanitize=memory" \
  --test_env="MSAN_SYMBOLIZER_PATH=/opt/llvm/clang/bin/llvm-symbolizer"

Please let me know if you disagree with this assessment or can reproduce this with an instrumented standard library.

Thanks.
I'll reopen this if I can reproduce the segfault with a minimal example. If not, the segfault was probably my fault somewhere.


cs complete-dep doesn’t support :: before version

In Scala CLI and mill this is a valid dependency format <org>::<name>::<version>, but complete-dep returns no results in this case.
I would expect that
cs complete-dep com.lihaoyi:pprint_3:0.7
gives the same results as
cs complete-dep com.lihaoyi:pprint_3::0.7

I think you need to provide the --scala-version option in addition,

It works correctly for cs complete-dep --scala-version 3.3.0  com.lihaoyi::pprint:0.7 but not if there is a double colon before version

To make more clear what you expect: In Mill, the double colon before the version denotes the platform, e.g. _sjs1 for Scala.js. Is this what you expect?

In Scala CLI this syntax is valid //> using dep com.lihaoyi::pprint::0.8.0 and it will work on both jvm and js/native. So instead of using  com.lihaoyi::pprint_sjs1:0.8.0 we can use :: before version


Custom Login

how to customize login to LDAP login/ SSO

Hi @belutz, It's easy. Basically, badaso uses headless concept with JWT. In order to customize your login with LDAP/SSO or whatever you want, you only need to override badaso auth controller and the auth route endpoint. Of course you need to understand how existing badaso auth logic work first. Thanks


Recipes Issues (Ender Alloy (Advanced), Glitch Boots)

Modpack v2.0.0 (updated from beta..)
I have no recipe for Ender Alloy (Advanced), so i can't progress and make the "Mana Infused Ingot" (or i have no clue how i should do).
And the boots and Helmet from the Glitch Armor Set have the same recipies.

Fixed in 2.0.1


update maxlength for names, use constants to revert/change easily

Please check if the PR fulfills these requirements:

[ ] After these changes, the app was run and still works as expected
[ ] Unit tests for these changes were added (if applicable)
[x] All existing unit tests were run and still pass

Please specify the type of change this PR introduces (Bug fix, feature addition, content update, chore, etc.):
chore
Additional Notes:
Updating firstname and middlename maxlength requirements. Refactored to use a variable since this will likely change again and be shared across all the user info pages

It's middlename and firstname that had issues (20 char limit in db), lastname is okay at 30, FPC rules are weird. If we will have to change lastname too though I would extend the logic to it to use a variable.


Fix PATH variable

This PR updates the changes made in #449 to re-order the PATH values. Before this change, there were some issues occurring on the devel images where conda was assuming the base environment was /opt/conda/envs/rapids instead of /opt/conda.

checks are expected to fail on this PR


Fix 100% scroll problem

What
Fix a problem with the scroll tracker, specifically:

in some cases the percentage scroll mode doesn't work for the 100% scroll position
this seems to be because the calculated percentage positions are sometimes decimal and not whole numbers, although the problem seems to be somewhat sporadic and hard to reproduce
for the avoidance of doubt, floor (and therefore reduce) each calculated percentage scroll position
the test is a bit weird too, ideally we'd set the height of the page to the height of the viewport, but for some reason it comes out as 14px less than the viewport, so the 100% event doesn't trigger, but manual testing confirms it to be working

Why
Sometimes the 100% scroll track event doesn't fire, because even if you're at the bottom of the page the calculated position is slightly further away.
Visual Changes
None.

It might be safer to both minus 1 from pageHeight and round up the scroll value, as I've encountered a scenario where the bottom for 100% is 3981 but the scroll + this.windowHeight value is 3979.888916015625

Thanks @andysellick - it's a lot more reliable now. I'll approve it but there's still some rare cases where it goes wrong.
I think using ceil in your last commit may still result in scenarios of not hitting 100%. For example I encountered a scenario (albeit they're very rare now, it seems to be very specific page widths/zooms) where scroll was 7601 but bottom was 7602 because bottom was getting rounded up. If it was rounded down they would both be 7601. Annoyingly though, even when doing Math.floor, I zoomed out my browser in one scenario and that also results in a mismatch between scroll and bottom. I got scroll: 7265 vs bottom: 7266.
The main reason for all of this inconsistency is because var pageHeight = Math.max(body.scrollHeight, body.offsetHeight, html.clientHeight, html.scrollHeight, html.offsetHeight) - 1  is trying 5 different ways to calculate the browser height. Each one factors different things into how it calculates height. See this: https://stackoverflow.com/revisions/46429828/3
Other height calculations such as window.innerHeight factor in different things too. For example, in the test that you need to subtract 15px from, you have to do that because window.innerHeight includes the horizontal scrollbar in its calculation, which is 15px high. It's possible that Jasmine has a horizontal scrollbar on its test page. So the horizontal scrollbar is being added to the body height when you do setPageHeight. Then the scroll tracker is grabbing the body height which has that added extra 15px on it, but is also grabbing the 15px scrollbar from the browser, resulting in a 15px mismatch.
And for the 100% scroll not firing issue, it seems that Math.max favours body.scrollHeight,  body.offsetHeight or html.offsetHeight when the 100% scroll calculation goes wrong. The problem is likely that these calculations don't include margin values in their height calculation. There is probably some margin: [negative number]px in our CSS at certain widths on certain pages which is making the scroll height of the page smaller by 1 or 2 pixels, but those other height calculations don't include margin so it doesn't get factored in.
The safest thing to do might be to give us a lot more buffer for 100%, e.g. psuedocode
when making percentDetails for 100%:
      minus 10px from the 'bottom' value


wip: cleanup(tests/falco): remove dropped CLI options test.

https://github.com/falcosecurity/falco/pull/3311 is dropping the deprecated -t,-T,-D options.
This PR removes TestFalco_Legacy_DisabledAndEnabledRules1 and TestFalco_Legacy_DisabledAndEnabledRules2 tests since they are no more needed.
Also, it bumps all other tests regarding rule disable/enable tags to the newly introduced in Falco 0.38 rules: config key: https://github.com/falcosecurity/falco/pull/3178

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: FedeDP
Once this PR has been reviewed and has the lgtm label, please assign leogr for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

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

/unhold


Android demo compile fixes

Thank you for fixing! That was strange the issue was not found out by the CI; will check once again why

Thank you for fixing! That was strange the issue was not found out by the CI; will check once again why

Thank you! That's strange that the build issue was not found by CI; will check why

Thank you! That's strange that the build issue was not found by CI; will check why

@dr-orlovsky this doesn't completely solve the issue though, it only gets the app to build. I still get this crash report mentioned at the bottom of this issue: https://github.com/LNP-BP/rgb-sdk/issues/47

@dr-orlovsky this doesn't completely solve the issue though, it only gets the app to build. I still get this crash report mentioned at the bottom of this issue: https://github.com/LNP-BP/rgb-sdk/issues/47

Yes, looking into that. Can I merge this PR? (I will do another one for the second half of the issue)

Yes, looking into that. Can I merge this PR? (I will do another one for the second half of the issue)

Thanks! Sure go for it.

Thanks! Sure go for it.


Don't truncate links in select text view

Long links in comments are truncated and end with '...'.
Though in the select text view they are still truncated, this means if you select text it will not include the whole link.
e.g. https://news.ycombinator.com/item?id=40292401

This is indeed a bad behavior, took a little dirty fix but should work fine https://github.com/SimonHalvdansson/Harmonic-HN/commit/d2a1746009d21db98e458ae8d86c4725f3a340c4


/health endpoint not working properly

I'm planning on using the endpoint /health for liveness and readiness probes for my kubernetes deployments, but I've been running into issues.
This is the deployment that I'm testing

apiVersion: apps/v1
kind: Deployment
metadata:
  name: tgi-tpu
spec:
  replicas: 1
  selector:
    matchLabels:
      app: tgi-tpu
  template:
    metadata:
      labels:
        app: tgi-tpu
    spec:
      nodeSelector:
        cloud.google.com/gke-tpu-topology: 2x4
        cloud.google.com/gke-tpu-accelerator: tpu-v5-lite-podslice
      volumes:
        - name: data-volume
          emptyDir: {}
      containers:
      - name: tgi-tpu
        image: {my_image}
        args:
        - --model-id=google/gemma-2b
        - --max-concurrent-requests=4
        - --max-input-length=32
        - --max-total-tokens=64
        - --max-batch-size=1
        securityContext:
            privileged: true
        env:
          - name: HF_TOKEN
            valueFrom:
              secretKeyRef:
                name: hf-secret
                key: hf_api_token
        ports:
        - containerPort: 80
        volumeMounts:
            - name: data-volume
              mountPath: /data
        resources:
          limits:
            google.com/tpu: 8
        livenessProbe:
          httpGet:
            path: /health
            port: 80
            scheme: HTTP
          initialDelaySeconds: 300
          periodSeconds: 120

---
apiVersion: v1
kind: Service
metadata:
  name: service
spec:
  selector:
    app: tgi-tpu
  ports:
    - name: http
      protocol: TCP
      port: 8080
      targetPort: 80

However, I get this error related to the /health endpoint
$ kubectl describe pod tgi-tpu
...
Warning  Unhealthy  109s (x13 over 41m)  kubelet            Liveness probe failed: Get "http://<IP_ADDRESS>:80/health": context deadline exceeded (Client.Timeout exceeded while awaiting headers)

I also tested out if I could reach the endpoint using curl. When I do a /generate request first, it returns successfully:
$ curl <IP_ADDRESS>:8080/generate     -X POST     -d '{"inputs":"What is Love?","parameters":{"max_new_tokens":40}}'     -H 'Content-Type: application/json'
{"generated_text":"\n\nLove is a feeling of affection for someone or something.\n\nLove is a feeling of affection for someone or something.\n\nLove is a feeling of affection for someone or"}
$ curl <IP_ADDRESS>:8080/health     -X GET
$

However, if I don't do a /generate request beforehand, the /health request never returns.
Looking at the router code, looks like this path is not working properly on Optimum TPU https://github.com/huggingface/text-generation-inference/blob/main/router/src/infer/health.rs#L27

Hi here @Edwinhr716, AFAIK this should have been fixed as of https://github.com/huggingface/optimum-tpu/pull/66, so any of:

https://github.com/huggingface/optimum-tpu/releases/tag/v0.1.2
https://github.com/huggingface/optimum-tpu/releases/tag/v0.1.3

should work if you rebuild the containers, thanks for flagging! Also see the original issue where this was listed at https://github.com/huggingface/optimum-tpu/issues/65#issuecomment-2196871340, and kudos to @tengomucho for solving those 👏🏻

Hi @Edwinhr716, @alvarobartt is right we have put some effort in improving TGI robustness with optimum-tpu. Latest release should be the most solid one, let us know if you still see the issue.

Upgrading it worked! Thanks for the help, I'll close the issue


Update to use asc v0.8.0 and near-bindgenas

Use near-bindgen-as's compiler api for projects that already use the gulp-utils.

Needs to not fail CI.
Does studio need any corresponding changes?

I think we can merge this when the studio update is also ready @vgrichina

@bowenwang1996 looks like it's ready https://github.com/nearprotocol/NEARStudio/pull/229
just need to test to make sure all stuff works fine

@willemneal @bowenwang1996 merged, but we also need to update https://github.com/nearprotocol/create-near-app templates now I think.

I think you just need to update it to use the new near-shell since the
gulp-utils just uses the bindgens compiler interface.
On Mon, Jan 6, 2020, 17:27 Vladimir Grichina<EMAIL_ADDRESS>wrote:

@willemneal https://github.com/willemneal @bowenwang1996
https://github.com/bowenwang1996 merged, but we also need to update
https://github.com/nearprotocol/create-near-app templates now I think.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nearprotocol/near-shell/pull/192?email_source=notifications&email_token=AALKD3FQAYSIP4BLXBC7CPLQ4PZDLA5CNFSM4JO2KJF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIHSFBQ#issuecomment-571417222,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALKD3GFYJBUZQCFETSJ5ULQ4PZDLANCNFSM4JO2KJFQ
.


Major bug

version:3.0.1   3.0.0

The problem in tomcat9 (M9) tomcat8 (8.5.4), tomcat7 (7.0.70) no problem.

Verified working on 10.1.10.  Please re-open if this is still an issue.


reduce bigint2 log level

This log is very verbose, assume meant to be debug (could be trace?)

I was intending to keep this in until I finish a way to print metrics on a per-op and per-accelerator grouping is done.

I was intending to keep this in until I finish a way to print metrics on a per-op and per-accelerator grouping is done.

Cool, I'll close this for now then, feel free to open back if you want