Deploy a docker-compose.yml file?

Hey @jpignata, excellent work on this project!  Thank you.
Question:  what would you think about adding the ability to deploy application images and environment variables from a docker-compose.yml file?  This would make it a bit more declarative and easier to use.  You could also docker-compose up to run locally and then fargate deploy -f docker-compose.yml to be confident that you deployed the same config that you tested locally.  Would you accept a PR for this?
version: "2"
services:
  my-app:
    build: .
    image:<PHONE_NUMBER>81.dkr.ecr.us-east-1.amazonaws.com/my-app:1.0
    ports:
    - 80:3000
    environment:
      FOO: bar
      BAR: baz

fargate --cluster fargate-poc service deploy -f docker-compose.yml

I'm going to take a stab at a proposal for this feature as soon as I finish refactoring and ship 0.3.0.

We have a somewhat relevant usecase where we have pre-defined Task Definitions, each use a single ECR repo and image, but with a custom working_dir.
fargate service create allows us to specify --image, but not --task-definition.  Might be able to avoid this by using a docker-compose file.

Closing since we've released this in our fork.
https://github.com/turnerlabs/fargate/releases/tag/v0.3.0


Shorten auth cookie length to 1 day

This will mean more 'logins', so the last login date will be a more sensible value. I.e. it'd be the last day they 'logged in'.

This is already configurable with JWS_MINUTES_UNTIL_EXPIRATION_FOR_USER_TOKEN


Design: Create designs for Checkout components

Fixes #43
Description
The Checkout Component introduces essential updates to the e-commerce UI Kit, specifically focusing on the Address Form, Order Summary, Shipping Options, Payment Options, and Order Confirmation. These enhancements aim to streamline the checkout process, improve usability, and provide a seamless and intuitive experience for users finalizing their purchases.
Motivation
The motivation behind these changes is to optimize the checkout process and provide users with a smooth, convenient, and trustworthy experience when finalizing their purchases. The previous checkout components had usability issues, lacked flexibility, or provided inadequate feedback to users. By enhancing the Address Form, Order Summary, Shipping Options, Payment Options, and Order Confirmation, we aim to improve user satisfaction, minimize friction during the checkout process, and foster trust in the e-commerce platform.
Changes Made

Optimized the Address Form for simplified and error-free data entry.
Added the Order Summary for a clear and concise overview of the purchase.
Enhanced the Shipping Options to offer convenient choices for users.
Updated the Payment Options to provide streamlined and secure payment methods.
Added the Order Confirmation page for a comprehensive summary of the completed order.

Checklist

[x] Code follows the project's style guidelines.
[x] Tests have been added or updated to cover the changes.
[ ] Documentation has been updated, if applicable. Mark ☑ in the component section of readme.md file.
[x] All new and existing tests pass.
[ ] The code has been reviewed by at least one other developer.
[x] The pull request is linked to the corresponding issue, if applicable.

Screenshots
Address form
The Address Form has been optimized to simplify the input of shipping and billing information. The updates include a user-friendly layout, clearer field labels, and validation indicators to ensure accurate and error-free data entry. These improvements help users easily provide their address details and expedite the checkout process.
Address form - Light mode

Address form - Dark mode

Order Summary
The Order Summary section has been refined to present a clear and concise overview of the items being purchased. It includes item names, quantities, prices, and any applied discounts or promotions. By improving the visual presentation and organization of this information, users can quickly review their order details before proceeding to payment.
Order Summary - Light mode

Order Summary - Dark mode

Shipping Options
The Shipping Options have been enhanced to provide users with a range of convenient choices for receiving their purchases. Users can now select from different shipping methods, such as standard, express, or in-store pickup, based on their preferences and requirements. This flexibility caters to diverse customer needs and improves the overall convenience of the checkout process.
Shipping Options - Light mode

Shipping Options - Dark mode

Payment Options
The Payment Options have undergone improvements to offer a streamlined and secure payment experience. Users can now choose from various payment methods, such as credit cards, digital wallets, or bank transfers, depending on what is supported by the e-commerce platform. These updates ensure that users have convenient and trustworthy options to complete their transactions.
Payment Options - Light mode

Payment Options - Dark mode

Order Confirmation
The Order Confirmation step has been updated to provide users with a clear confirmation message and order details once the purchase is successfully completed. This confirmation page serves as a summary of the order, including the total amount paid, shipping information, and estimated delivery dates. It offers users reassurance that their order has been successfully placed.
Order Confirmation - Light mode

Order Confirmation - Dark mode

Hey @Rishika-S-V
I can see other commits in this branch
@AdityaSingh-02

Hey @Rishika-S-V I can see other commits in this branch @AdityaSingh-02

Now it is fixed @Lovelyfin00

Great
Nice job here


MAISTRA-1024: Update servicemesh-manifests with MAISTRA-929 resources…

… change

@knrc Yes. I removed the two resource sections in the second commit.

oops, wait a min. I need to delete more line. It's broken template now.

@knrc @rcernich Done. Could you help review this PR ?
The reason why I delete both the .spec.istio.global and .spec.istio.global.proxy block is : our operator could not parse block with empty content. So after I delete the .spec.istio.global.proxy.resources section, I have to delete .spec.istio.global and .spec.istio.global.proxy empty sections as well.


[Failing test] 'should read runtime metrics' test is failing when NodeLocalDNS=true

How to categorize this issue?

/area networking
/kind bug
/priority normal
What happened:
should read runtime metrics test seems to be failing when NodeLocalDNS feature gate is enabled.
What you expected to happen:
No such failure.
How to reproduce it (as minimally and precisely as possible):

Start gardenlet with

apiVersion: gardenlet.config.gardener.cloud/v1alpha1
kind: GardenletConfiguration
# ...
featureGates:
  NodeLocalDNS: true

Create a new Shoot

Run the should read runtime metrics test against the newly created Shoot.

go test -timeout=0 -mod=vendor ./test/suites/shoot \
      --v -ginkgo.v -ginkgo.progress -ginkgo.noColor \
      --report-file=/tmp/report.json \
      -kubecfg=$KUBECONFIG \
      -shoot-name=$SHOOT_NAME \
      -project-namespace=$PROJECT_NAMESPACE \
      -ginkgo.focus="Shoot application metrics testing"

Ensure that the test is failing

• Failure [654.337 seconds]
Shoot application metrics testing
/Users/i331370/go/src/github.com/gardener/gardener/test/integration/shoots/applications/metrics.go:54

  /Users/i331370/go/src/github.com/gardener/gardener/test/framework/test_options.go:56
    [BETA] [SHOOT] should read runtime metrics [It]
    /Users/i331370/go/src/github.com/gardener/gardener/test/framework/gingko_utils.go:26

    Unexpected error:
        <*retry.retryError | 0xc000898000>: {
            ctxError: {},
            err: {
                ErrStatus: {
                    TypeMeta: {Kind: "", APIVersion: ""},
                    ListMeta: {
                        SelfLink: "",
                        ResourceVersion: "",
                        Continue: "",
                        RemainingItemCount: nil,
                    },
                    Status: "Failure",
                    Message: "podmetrics.metrics.k8s.io \"gardener-e2e-vxw4c/metrics-test-7f4d6df99b-rbwqp\" not found",
                    Reason: "NotFound",
                    Details: {
                        Name: "gardener-e2e-vxw4c/metrics-test-7f4d6df99b-rbwqp",
                        Group: "metrics.k8s.io",
                        Kind: "podmetrics",
                        UID: "",
                        Causes: nil,
                        RetryAfterSeconds: 0,
                    },
                    Code: 404,
                },
            },
        }
        retry failed with context deadline exceeded, last error: podmetrics.metrics.k8s.io "gardener-e2e-vxw4c/metrics-test-7f4d6df99b-rbwqp" not found
    occurred

    /Users/i331370/go/src/github.com/gardener/gardener/test/integration/shoots/applications/metrics.go:85
------------------------------

Summarizing 1 Failure:

[Fail] Shoot application metrics testing  [It] [BETA] [SHOOT] should read runtime metrics
/Users/i331370/go/src/github.com/gardener/gardener/test/integration/shoots/applications/metrics.go:85

Ran 1 of 19 Specs in 654.338 seconds
FAIL! -- 0 Passed | 1 Failed | 0 Pending | 18 Skipped
--- FAIL: TestGardenerSuite (654.36s)
FAIL
FAIL	github.com/gardener/gardener/test/suites/shoot	655.369s
FAIL

Also with kubectl:
$ k -n kube-system top po
W1001 12:31:50.477301   37165 top_pod.go:274] Metrics not available for pod kube-system/calico-kube-controllers-68657c7d94-hwszr, age: 23m9.477288s
error: Metrics not available for pod kube-system/calico-kube-controllers-68657c7d94-hwszr, age: 23m9.477288s

$ k top no
error: metrics not available yet

Logs of metrics-server:
I1001 09:36:04.532056       1 log.go:172] http: TLS handshake error from <IP_ADDRESS>:56384: EOF
E1001 09:36:10.285087       1 reststorage.go:160] unable to fetch pod metrics for pod kube-system/coredns-97dfb4f59-qlmhj: no metrics known for pod
E1001 09:36:10.285120       1 reststorage.go:160] unable to fetch pod metrics for pod kube-system/coredns-97dfb4f59-cp6nr: no metrics known for pod
I1001 09:36:11.924066       1 log.go:172] http: TLS handshake error from <IP_ADDRESS>:56450: EOF
E1001 09:36:15.745889       1 reststorage.go:160] unable to fetch pod metrics for pod gardener-e2e-fhp5q/metrics-test-7f4d6df99b-979bw: no metrics known for pod
E1001 09:36:17.071619       1 reststorage.go:160] unable to fetch pod metrics for pod gardener-e2e-fhp5q/metrics-test-7f4d6df99b-979bw: no metrics known for pod
I1001 09:36:19.520117       1 manager.go:148] ScrapeMetrics: time: 30.007957004s, nodes: 0, pods: 0
E1001 09:36:19.520143       1 manager.go:111] unable to fully collect metrics: unable to fully scrape metrics from source kubelet_summary:ip-10-222-0-27.eu-west-1.compute.internal: unable to fetch metrics from Kubelet ip-10-222-0-27.eu-west-1.compute.internal (ip-10-222-0-27.eu-west-1.compute.internal): Get https://ip-10-222-0-27.eu-west-1.compute.internal:10250/stats/summary?only_cpu_and_memory=true: dial tcp: i/o timeout
I1001 09:36:21.924958       1 log.go:172] http: TLS handshake error from <IP_ADDRESS>:56536: EOF

Anything else we need to know?:
Environment:

Gardener version: master
Kubernetes version (use kubectl version): v1.19.2
Cloud provider or hardware configuration: AWS
Others:

Is it related to https://github.com/gardener/gardener/issues/2745 ?

Is it related to #2745 ?

It could be related. @DockToFuture , can you confirm?

ping @DockToFuture

It seems that the issue is fixed with #2745 (or any other PR meanwhile). I ran the test multiple times with the same setup with the current gardener/gardener@master and the test passed successfully.
/close


Previews/popup info appearing behind app when "Always on top" enabled

Describe the bug
When the "Always on top" option is enabled, previews for emotes/links appear behind the app
To reproduce
Enable "Always on top" and hover mouse on top of an emote or link (nothing will seem to appear because it's behind the app)
Screenshots

Chatterino version
Chatterino 2.2.2 (commit 07fc0c3ad)
Operating system
macOS Big Sur 11.0

Someone else also confirmed that this is a bug. Highly doubt we will fix this though since it's mac only.

@gempir Could you please test this when you get the chance? :)

Yeah confirmed on Chatterino 2.2.2 (commit d56187fda)

Happens for me too on Linux Mint 20 with Chatteriono 2.2.2 (commit 98762fdfa) when opening usercards, but not on emote hover

@tintinmaster That's a completely different issue.

Highly doubt we will fix this though since it's mac only.

I could try to help with this issue. It'll help me if you'd point me to the code block that handles the tooltip.

Highly doubt we will fix this though since it's mac only.

I could try to help with this issue. It'll help me if you'd point me to the code block that handles the tooltip.

I could try to help with this issue. It'll help me if you'd point me to the code block that handles the tooltip.

Potentially interesting is the TooltipWidget class as well as this block of code: https://github.com/Chatterino/chatterino2/blob/a9b1af60de00dc22078ba58adf12bfdfa69a9310/src/widgets/helper/ChannelView.cpp#L1414-L1490

I could try to help with this issue. It'll help me if you'd point me to the code block that handles the tooltip.

Potentially interesting is the TooltipWidget class as well as this block of code: https://github.com/Chatterino/chatterino2/blob/a9b1af60de00dc22078ba58adf12bfdfa69a9310/src/widgets/helper/ChannelView.cpp#L1414-L1490


feat: decouple request timeout env variable configuration

Closes #100

@TessaIO you added this section for each node:
router:
  # Router specific configuration
  config:
    ZO_META_CONNECTION_POOL_MIN_SIZE: "0"
    ZO_META_CONNECTION_POOL_MAX_SIZE: "0"
    ZO_ACTIX_REQ_TIMEOUT: "30"

recently, i need add more ENV for separate node, so i just flow it, then i found a bit duplicated, i need add it for each node, and when i add a new ENV i need update the template again.
So, i found a new solution for this:
router:
   extraEnv:
    - name: RUST_LOG
      value: "info,openobserve=debug,actix_web=warn"
    - name: ZO_META_CONNECTION_POOL_MIN_SIZE
      value: "0"

if you want custom something, just add an extra env, it can support custom anything. so i am think if we can use the new patten, then we can remove the separate config for each node. it is a bit duplicated.
What do you think?


High Contrast Menu Over Laps On a Menu

When using High Contrast Mode, your menu bar overlaps with the tabs that scroll behind it, causing it to be rather confusing for a few seconds.
Example:

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

vscode remains in high contrast after back in windows normal contrast (#65554)

It is not related to that issue.


Fix Travis badge .org -> .com

Also put it on one line.
I dont really get the difference between travis-ci.com and .org but maybe thats just me...

org is for public, com is for private iirc

Okay.. strange. Because https://travis-ci.com/bitwalker/swarm seems to work for this repo. Anyway.


Add Find node API to the syntax API

Description:
Add a new API to get the node when given a position

I think what we need is an API to return a Node instance for a given TextRange.
public NonTerminalNode findNode(TextRange textRange);


[BUG] I reported an error when building with the apktool B command, ple help me

Information

Apktool Version (apktool -version) -2.7.0
Operating System (Mac, Linux, Windows) -mac
APK From? (Playstore, ROM, Other) - Development debug
Java Version (java --version)  "1.8.0_151"

Stacktrace/Logcat
W: invalid resource directory name: /Users/jerry/jerryking/Jerry/GoveeHome/Util/Decompilation/file/res navigation
brut.androlib.AndrolibException: brut.common.BrutException: could not exec (exit code = 1): [/var/folders/td/zkkf43sx4zd2pp85pwz0795m0000gp/T/brut_util_Jar_19065783164599488865285493720240900239.tmp, p, --forced-package-id, 127, --min-sdk-version, 21, --target-sdk-version, 33, --version-code, 580, --version-name, 5.8.00, --no-version-vectors, -F, /var/folders/td/zkkf43sx4zd2pp85pwz0795m0000gp/T/APKTOOL8242752269328417561.tmp, -e, /var/folders/td/zkkf43sx4zd2pp85pwz0795m0000gp/T/APKTOOL3250995606889694169.tmp, -0, arsc, -I, /Users/jerry/Library/apktool/framework/1.apk, -S, /Users/jerry/jerryking/Jerry/GoveeHome/Util/Decompilation/file/res, -M, /Users/jerry/jerryking/Jerry/GoveeHome/Util/Decompilation/file/AndroidManifest.xml]
Include stacktrace here

Steps to Reproduce

apktool

Frameworks
If this APK is from an OEM ROM (Samsung, HTC, LG). Please attach framework files
(.apks that live in /system/framework or /system/priv-app)
APK
If this APK can be freely shared, please upload/attach a link to it.
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything? y
If you are trying to install a modified apk, did you resign it?
Are you using the latest apktool version? y

Generally when you see that res/navigation error - it means the previous application was built using aapt2 and thus not compatible with building with aapt1.
So just add --use-aapt2 during build command.


Refactor DocumentationBundle.Info

Summary
This work refactors the existing logic for handling DocumentationBundle Info to make it easier to add new configuration options moving forward.
Details

Use DocumentationBundle.Info to pass around bundle info.
Currently we pass around DocumentationBundle Info as function parameters in a multitude of places. This means that when we add a new property to the DocumentationBundle.Info model, we have to add new function parameters in several places across the codebase. It's easy to miss some of these and I think likely to introduce bugs in the future.
We already have a model for handling documentation bundle info, so this just updates a number of calling sites to rely on passing around the DocumentationBundle.Info directly instead of its properties.
This change is isolated in this commit c357b35 with another commit here (1c850a6) that addresses deprecation warnings.

Use Codable to parse the Info.plist
I believe when this code was originally written, PropertyListEncoder was macOS only so we relied on using PropertyListSerialization directly instead.
PropertyListEncoder is now supported on Linux so I refactored the DocumentationBundle.Info models to be Codable which simplifies a lot of our logic here.
That change is isolated here 2e5658d.

Use a strongly-typed API for creating bundle discovery options
The bundle discovery options are intentionally stringly-typed so that we can easily add new info plist fields and have clients adopt them immediately. However, there's no reason for us to be using the stringly-typed API within DocC itself. This led to some unnecessary duplication of the Info.plist keys.
This change is isolated to this commit 22c6c80 and just adds some convenience initializers to the bundle discovery options to reduce duplication of the Info.plist keys in DocC itself.

Dependencies
None.
Testing
Perform basic regression testing. No functionality change is expected.
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[x] Updated documentation if necessary

@swift-ci please test


Update configure

b/31910117
correctly finds python3 bin libraries

Can one of the admins verify this patch?

From what I can see, this is already merged into master. am I missing something?


chore: randomize the selection of nodes to balance the load

Also update the sentry DSN to add it to a sep project in Santry

yolo!


az aks get-versions leaves a \r character in the output

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug
running the az aks get-versions command leaves a \r in the output when running it on unbuntu using the windows subsystem for linux
To Reproduce

run

VERSION=$(az aks get-versions \
> --location $REGION_NAME \
> --query 'orchestrators[?!isPreview] | [-1].orchestratorVersion' \
> --output tsv)

at the command prompt in Ubuntu using Windows Terminal
2) set REGION_NAME=eastus, RESOURCE_GROUP=akscostsaving, and AKS_CLUSTER_NAME=akscostsaving-$RANDOM so the next command will work (These were the values being used in the tutorial)
3) run
az aks create \
> --resource-group $RESOURCE_GROUP \
> --name $AKS_CLUSTER_NAME \
> --location $REGION_NAME \
> --kubernetes-version $VERSION \
> --node-count 2 \
> --load-balancer-sku standard \
> --vm-set-type VirtualMachineScaleSets \
> --generate-ssh-keys

Actual Behavior
output of:
The behavior of this command has been altered by the following extension: aks-preview
SSH key files 'C:\Users\charl\.ssh\id_rsa' and 'C:\Users\charl\.ssh\id_rsa.pub' have been generated un
der ~/.ssh to allow SSH access to the VM. If using machines without permanent storage like Azure Cloud
 Shell without an attached file share, back up your keys to a safe location
Operation failed with status: 'Bad Request'. Details: Error to parse agent pool version "1.19.7\r": In
valid character(s) found in patch number "7\r"

Expected behavior
output of:
The behavior of this command has been altered by the following extension: aks-preview
SSH key files 'C:\Users\charl\.ssh\id_rsa' and 'C:\Users\charl\.ssh\id_rsa.pub' have been generated un
der ~/.ssh to allow SSH access to the VM. If using machines without permanent storage like Azure Cloud
 Shell without an attached file share, back up your keys to a safe location

followed by lots of json

Environment summary

installed via MSI
version:
{
 "azure-cli": "2.19.1",
 "azure-cli-core": "2.19.1",
 "azure-cli-telemetry": "1.0.6",
 "extensions": {
   "aks-preview": "0.5.0"
 }
}

Windows 10 Home ver 2004 os build 19041.804
shell bash running inside of Ubuntu Linux installed via Windows Subsystem for Linux. All of that was running inside of Windows Terminal
Additional context

I installed the Azure CLI onto windows before I installed the Windows Subsystem for Linux, it was visible inside of my Ubuntu install without any additional steps.
I have run these command together about a week ago on macos using the bash shell and this problem does not occur.

Please see the explanation at https://github.com/Azure/azure-cli/issues/13573#issuecomment-888791313.


gitignore: remove vim swap files.

[ ] Added an entry to CHANGELOG_VULKANO.md or CHANGELOG_VK_SYS.md if knowledge of this change could be valuable to users
[ ] Updated documentation to reflect any user-facing changes - in this repository
[ ] Updated documentation to reflect any user-facing changes - PR to the guide that fixes existing documentation invalidated by this PR.
[ ] Ran cargo fmt on the changes

@prataprc Merged. Thanks!


Standardize Lite's filenames

What does this PR do?
Lite's naming scheme is inconsistent:

Accelerator, Strategy and Precision do not include the suffix. For example, we use cpu.py instead of cpu_accelerator.py
ClusterEnvironment and Collective do include it. For example lsf_environment.py and single_device_collective.py
CheckpointIO has _plugin. For example torch_plugin.py

This PR removes the suffix from all filenames with the exception of torch_collective.py and torch_checkpoint_io.py so they are not mistaken for actual torch.
Does your PR introduce any breaking changes? If yes, please list them.
None
cc @carmocca @justusschock @awaelchli

Can we resolve these inconsistencies the other way around, i.e. putting suffix to ALL files? That way, although the names will be quite long, we won't have to have exceptions to the rule.

I don't mind either way. Suffix just has the annoyance of longer and repeating imports. Let's vote in your comment


Only one of the four ambiguous monomers is implemented.

It is possible to define a * monomer, but not X, N or _. See the extract below from the HELM specification.
5.1.8. Monomer Ambiguity
Unknown monomers can represented by four different characters:
•   * The character “*” represents 0..n unknown monomers.
•   X The character “X” represents one single unknown amino acid in a PEPTIDE polymer.
•   N The character “N” represents one single unknown base in a RNA polymer.
•   _ The character “_” represents a deleted or missing single monomer. This is typically used for list elements (see  section List Elements)

fixed in 2017-09-12 update

I can't work out how to add X, N or _.
I can add * using the 'monomer attributes' box, but don't know how to add the others.
This method of adding ambiguous monomers is not clear even for *. We need to have a discussion about how the UI works for this example.

Fixed in 2017-12-1 update

I can see *, _ and X as peptide monomers, but nothing under nucleotide or chem.

Not quite sure what should be used for the ambiguity of nucleotide and chem.

Nucleotides should have *, _ and N
Chem should have *.

Are there some example HELM notations?  So we know how to generate the HELM string.

RNA1{R()P.R(C)P.(N)P.R(G)P}$$$$V2.0
CHEM1{*}|CHEM2{[sDBL]}|CHEM3{[SMPEG2]}$CHEM2,CHEM3,1:R2-1:R1|CHEM1,CHEM2,1:R1-1:R1$$$V2.0

2017-12-09 update: * is added to CHEM

Confirmed in today's meeting:
Base: _, N, *
Ribose: *
P: *
Combo: *

Fixed in 2017-12-13 update

Confirmed fixed.


Closes #25. Set up Travis CI and add in a Gradle wrapper.

CI build failed due to test failure fixed in PR #18. Hadoop 2.9 build failed because the proper fsimage files are not set up; I will update. But it looks that overall the Travis integration is working as expected. Very cool! And it builds all of the Hadoop versions in parallel. Great stuff.

have you considered using CircleCI? We have free trial for OSS. The Travis queue is pretty congested.

@oliverhu Is there any benefit to using CircleCI over TravisCI? I know of other LinkedIn projects (Azkaban) using TravisCI, and the integration was exceedingly easy. They also have a committed to be always free for OSS. Does CircleCI have something similar?
re: your congestion comment, I don't know if I just got lucky, but all of my tests started in only a few seconds after pushing.

Looks like in the free tier on CircleCI we cannot take advantage of any parallel execution because only a single container is granted, is that right?
This is from CircleCI's website:

While Travis CI is focused on the open source market, CircleCI focuses on performance and service designed to solve the needs of professional engineering organizations looking to scale.

Based on this this sentence alone, it seems to me that TravisCI is a batter fit.

Not sure, we only used single execution for macOS containers and the performance was really good. Yeah, just want to point out there are other available CI solutions available.

Thanks for the info @oliverhu ! I suspect that the TravisCI queue with Linux containers instead of macOS may perform better. If we do see issues I'll definitely look into CircleCI.

Also, if their status page is to be trusted, at no point in the last month has there ever been a backlog of Linux open source jobs: https://www.traviscistatus.com/#month


docs: doc-markdown and some typos

Noticed a typo, checked with cargo-spellcheck and added clippy::doc_markdown as a lint.

cargo-spellcheck creates a bunch of false positives pretty quickly but seems to checks different places than typos. Stuff like impl or struct is written sometimes in the docs and cargo-spellcheck isn't happy about it. Personally I haven't added it to my CI stuff and only use it on demand as its just simpler to go through them when relevant. It's also not as good as something like LanguageTool (and I don't know a tool which works with Rust code there)

Okay, let's keep it out of CI for now. I also use Grammarly although painfully manually to spellcheck the markdown files in the website directory. They discontinued their support for the SDK that their unofficial VSCode extension was using..

I use LTeX on VSCode with LanguageTool installed on the same device for Markdown checking which works well enough for me currently.

Thanks! I'll check it out


Tenancy management/PIM operational documentation

Describe the Issue
Create PIM structure & operational documentation
Additional Context
Add any other context, attachments or screenshots
Acceptance Criteria

Demo PIM initially around structure and how to elevate
Document purpose and structure of PIM within an Operations context
Describe how to use PIM for management of the tenancy
Create
Work with SecOps to have the SecOps documentation integrate with our own
Put all final documentation in Ops repo

PIM operations (access control) draft document for management use of PIM to control access was completed.  A Demo of PIM for management to the PO was also done.  The draft document was put into our Operations repo and shared for any further refinement


Silence Didn't Work

The following alert triggered:
http://goo.gl/r8HDgZ

Even though there was an active silence for this:

@kylebrandt We're running a self compiled Bosun -- pulled this morning 451c58823c9b9ffa22f3dec63f1b36c0d1803a6a.
I'm seeing emails notifications being generated for already acknowledged alerts. That might be related. I'm happy to send you more information, if helpful.

@lukaspustina: You should get another email for acknowledge alert if there is a state increase ( http://bosun.org/documentation.html#severity-states ).
And can you look at the history for one of the alerts that is doing this, you get to history via a dashboard panel:

And it will look something like this:

No, neither severity nor state changed -- see history screenshot.

@lukaspustina : Do you have any notification chains set up?

Yes, a tiny one:
notification default {
  email =<EMAIL_ADDRESS>  next = default
  timeout = 6h
}

Did you do any restarts of bosun during this 6 hour period?
On Wed, Feb 11, 2015 at 11:01 AM, Lukas Pustina<EMAIL_ADDRESS>wrote:

Yes, a tiny one:
notification default {
email =<EMAIL_ADDRESS>next = default
timeout = 6h
}
—
Reply to this email directly or view it on GitHub
https://github.com/bosun-monitor/bosun/issues/607#issuecomment-73906461.

Unfortunately, no.

@lukaspustina Can't repro, I tested with a short timeout and kept getting emails until I ack'd the alert. Have you been using Bosun for a while? If you have I wonder if deleting your state file will fix this.

@kylebrandt So I deleted the state file and had no luck. Still the same behavior. So I tried to silence the alarms at least which also had no effect. Alarm notification emails keep coming.
Any ideas?

@kylebrandt I restarted all of bosun components including OpenTSDB etc. and that seemed to have fixed the problem.

Oh good to know, sorry for forgetting to respond to your last comment
On Tue, Feb 17, 2015 at 11:46 AM, Lukas Pustina<EMAIL_ADDRESS>wrote:

@kylebrandt https://github.com/kylebrandt I restarted all of bosun
components including OpenTSDB etc. and that seemed to have fixed the
problem.
—
Reply to this email directly or view it on GitHub
https://github.com/bosun-monitor/bosun/issues/607#issuecomment-74701344.

Assuming silence bug isn't real - haven't noticed any issues in a long time.


Docker example

A docker example would be great to go with devshell template for golang. Along with the instructions on how to upload them inside docker registry.

I can work on this one 🖖🏻


No response

I added your plugin in hoobs service and was so glad that it worked as soon as I input the token.
Since the first check the vacuum (roborock S5) is not responding whatever I do.
I reset wifi, I reinstalled, uninstalled, installed..
What am I doing wrong?

Hi @costispavlou ,
What do you mean with “it worked as soon as I input the token” and then “is not responding”?
Can you please elaborate? If you could provide some logs as well, it would be amazing!

Hi thanks for the quick reply. I mean it only worked once. Now it shows on home.app as not responding

According to the logs it looks like it cannot connect to your robot. It usually is due to the IP or the token being wrong. Can you make sure both are correct?
We’ve seen in the past, as well, some routers have different subnetworks between the 2.4GHz and the 5GHz (my BT router was one of them) despite being named the same. When that happens, it’s not possible to find the device.

If token can somehow change then yeah, i need to find it again but i really doubt thats the case. Also the wifi is 2,4ghz.
I did change the roborocks’ name and was experimenting with more plugins but since it worked the first few times, i’m really struggling to find what went wrong..

The token should be static overtime unless you reset/reconfigure the robot through the app. In the past, when I reconfigured the WiFi connection in my robot, I had to retrieve the token again. Maybe that will help?

Yeah maybe that’s it. I’ll recover the token again using a new backup. There’s no other easier way right?
I’ll let you know tonight

🤞

Yup. Token is different. Thanks so much.
Hope this helps others too!

Awesome! Thank you for confirming. Closing the issue then, if you don’t mind.


Upload pipeline from file - PermissionDenied

Error message came up in pre-release of 1.7, when creating a pipeline from file
Experiment creation failed
{"error":"Failed to authorize the request: Failed to authorize with API resource references: PermissionDenied: User 'test' is not authorized with reason:  (request: \u0026ResourceAttributes{Namespace:test,Verb:create,Group:pipelines.kubeflow.org,Version:v1beta1,Resource:experiments,Subresource:,Name:testexp,}): Unauthorized access","code":7,"message":"Failed to authorize the request: Failed to authorize with API resource references: PermissionDenied: User 'test' is not authorized with reason:  (request: \u0026ResourceAttributes{Namespace:test,Verb:create,Group:pipelines.kubeflow.org,Version:v1beta1,Resource:experiments,Subresource:,Name:testexp,}): Unauthorized access","details":[{"@type":"type.googleapis.com/api.Error","error_message":"User 'test' is not authorized with reason:  (request: \u0026ResourceAttributes{Namespace:test,Verb:create,Group:pipelines.kubeflow.org,Version:v1beta1,Resource:experiments,Subresource:,Name:testexp,})","error_details":"Failed to authorize the request: Failed to authorize with API resource references: PermissionDenied: User 'test' is not authorized with reason:  (request: \u0026ResourceAttributes{Namespace:test,Verb:create,Group:pipelines.kubeflow.org,Version:v1beta1,Resource:experiments,Subresource:,Name:testexp,}): Unauthorized access"}]}

Output of kubectl get ClusterRole -n test | grep pipeline I got:
aggregate-to-kubeflow-pipelines-edit                                   2023-02-27T11:35:27Z
aggregate-to-kubeflow-pipelines-view                                   2023-02-27T11:35:27Z

Output of kubectl describe ClusterRole aggregate-to-kubeflow-pipelines-edit:
Name:         aggregate-to-kubeflow-pipelines-edit
Labels:       app.juju.is/created-by=kubeflow-roles
              app.kubernetes.io/component=ml-pipeline
              app.kubernetes.io/name=kubeflow-pipelines
              application-crd-id=kubeflow-pipelines
              rbac.authorization.kubeflow.org/aggregate-to-kubeflow-pipelines-edit=true
...

kubeflow-pipelines-edit ClusterRole is missing, same goes for kubeflow-pipelines-view


1 pixel line between refresh view and collection view

Hi,
A one-pixel line appears at the moment when when releasing - as can be seen in the attached image. It flashes for the briefest moment in time, but my app's contrast between the two colours is very high so it's quite obvious.
Is this in the library or could it be something weird in the way my UI is put together?

Update - doesn't do it on any other collection views in the app. What's different about the screen where this occurs is that there is a header cell with a white background. The top insets are all zero. However with pull to refresh the 1 pixel line must be creeping in somehow when released.

Any solution for this? I found that the color of the line is controlled by the tableView.backgroundColor

I my case the tableView.backgroundColor is white and the screen background is dark blue. In some cases when bouncing a 1 pixel gap appeared between the tableView and the pullToRefreshView above it.
I eventually solved it by making pullToRefreshView 1 pixel higher than it theoretically needed to be, so that this line would not sometimes appear:
DGElasticPullToRefreshView.swift line 370:

frame = CGRect(x: 0.0, y: -height - 1.0, width: width, height: height)

frame = CGRect(x: 0.0, y: -height - 1.0, width: width, height: height + 1)


Merge from downstream to upstream

Introducing GitHub Actions
Remove travis / landscape
Introducing dedicated caching service
Fixing user-agent page
Making w3schools state page obsolete + removal
Clean-up/refactoring code+ better coding standard (black)
Update requirements.txt

So now I only need to debug this issue: https://github.com/danger89/fake-useragent/actions/runs/3381208500/jobs/5614863529

Build is green (no tests are disabled!). See: https://github.com/danger89/fake-useragent/actions/runs/3381327412/jobs/5615116933

Hi!
Will 0.1.13 be downloadable through pip?

Hi!
Will 0.1.13 be downloadable through pip?

Yes it will! We are migrating to a separate GitHub organization. After which I will add a deployment job towards PyPi 🤟🏽

Version 0.1.13 is deployed to PyPi (which you can install using pip).
New GitHub repo location: https://github.com/fake-useragent/fake-useragent
Deployment CI/CD:


High cpu usage

Hi. I have high cpu usage using this library (mac os and debian using pypy 3.7 and python 3.9). I would like to run it on pypy, but with pypy I have a biggest problem (other parts of my code works very well on pypy). It's like polling with cpu, some noop signal handler is called again and again.
Here is a sample example I have done for the pypy teams https://github.com/romikforest/asyncio_streams_high_cpu_usage.
Thank you a lot in advance if you take a look and say what mistake I have done or what may be the reason of such a behavior.

Hi. Here is the same example with the original nonpathced library https://github.com/romikforest/asyncio_streams_high_cpu_usage2

Thanks for the report and the repro!
What kind of CPU usage are you seeing? Running locust like you described, I am seeing approximately 10% on both client.py and server.py using CPython (admittedly I'm not sure what the expected usage should be, but I don't see anything alarming in qcachegrind). I will try with PyPy as well soon.
In my case I see _sighandler_noop called only once, with uvicorn version 0.13.4 and starlette version 0.14.2.

It's a simple example, but in profiler file you can see how selectors work. Depending on the response time and additional logic cpu usage will change and grow. I'm trying to use it with graphql-core, and a low rps grows up to 100% cpu usage in my case, but looking in the profiler I see all the time had been lost in noop_handler as you can see in the example: there were most time spent in selectors library. In the case of example it may be lower, I didn't spent a lot time on parameters, but can be seen easily.

I used it on macos 11.2.3 and in docker containers python-3.9.2-buster and pypy-3.7-7.3.3-buster. Also have tried some other python versions. On mac os method control of select.kqueue took 96% of the program time and it was 96% of the self executing. Inside it noop possible onece. Sorry. It should be both really precompiled. And it shows only noop handler inside the select control. I thought it was the reason first.

I used it on macos 11.2.3 and in docker containers python-3.9.2-buster and pypy-3.7-7.3.3-buster. Also have tried some other python versions. On mac os method control of select.kqueue took 96% of the program time and it was 96% of the self execution. Inside it noop possible once. Sorry. It should be both really precompiled. And it shows only noop handler inside the select control. I thought it was the reason first.

I am also seeing very high self time for control. It seems like the control method is where Python waits for kernel events (this happens here, and there is more information about kevent here). I think this means that it's spending all that time waiting on IO. On Mac OS and Linux I think the select-based event loop is the default.
I think using uvloop might help with CPU usage (though it doesn't seem to work with PyPy 3.7).
If you don't use asyncio-connection-pool, do you see less CPU usage?

Closing due to inactivity, feel free to reopen if the issue persists.


windows compatability

Creates a .bat file that works on windows
Use --make-install option to handle both 'make' commands from Configure.pl (which chooses make or nmake appropriately)

Note that --make-install is only implemented for the 'build' command; I did not implement it on 'triple'

Awesome, thanks!

@ugexe can you check if e15753d796d791b994451c9a7ae895a036b4aec1 also needs some similar fix in the windows code branch?

It does. I tried the non-windows solution on windows but it didn't work.


updated airdroid (3.3.0)

http://blog.airdroid.com/post/first-major-update-in-2016-desktop-3-3-0-android-3-2-0/

Thank you for the contribution. It was merged directly as 9d6415167fb639746d3eb0fc43a1288a46476d2e to keep commit history cleaner. Your contribution is still credited to you.


Support Xcode 11.2

Is this gonna be available any time soon?

We are having trouble with CI (jobs not starting), once those are resolved we'll cut an 11.2 release. In the meantime I tested #392 locally and it works.

Alright, cool 👍


Hammerdb LSO support

This PR is including:

Adding LSO support for the following databases for pod/kata/vm:
a. PostgreSQL
b. MariaDB
c. MSSQL
Adding storage_type field indication: odf, lso, ephemeral
Enable PostgreSQL pod/kata/vm in functional CI.

RobertKrawitz

I prefer not to change it in action because it will affects all other workflows.


refactor: a new empty sentinel

WIP
The current Empty and EmptyType sentinel values and type are unable to be appropriately narrowed away, resulting in a lot of calls to cast(), particularly when accessing cached attributes on our core types.
This PR introduces a new empty sentinel, the adoption of which will allow us to properly narrow down from unions with itself.
This first stage implements the sentinels for our internal attribute caching only, and on transition to 3.0, Empty would alias _EMPTY and EmptyType would alias ``_LiteralEmpty`, so there shouldn't much downstream disruption.

Pull Request Checklist

[ ] New code has 100% test coverage
[ ] (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
[ ] (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR
[ ] Pre-Commit Checks were ran and passed
[ ] Tests were ran and passed

Description

Close Issue(s)

Superseded by #2734


helmfile: init at 0.16.0

Motivation for this change
Adds https://github.com/roboll/helmfile, a useful tool for managing Helm charts.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg  build helmfile

Success on x86_64-darwin (full log)
Attempted: helmfile
Partial log (click to expand)
/private/tmp/nix-build-helmfile-0.16.0.drv-0/go /private/tmp/nix-build-helmfile-0.16.0.drv-0
/private/tmp/nix-build-helmfile-0.16.0.drv-0
post-installation fixup
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/p4imbrdwi30hai4459vpn4lj1dfvqnv3-helmfile-0.16.0-bin/bin
patching script interpreter paths in /nix/store/p4imbrdwi30hai4459vpn4lj1dfvqnv3-helmfile-0.16.0-bin
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/x6vk1k6xaxa21wayj9h358zin3746yhy-helmfile-0.16.0
/nix/store/x6vk1k6xaxa21wayj9h358zin3746yhy-helmfile-0.16.0/share/go/src/github.com/roboll/helmfile/test/integration/run.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/r8bx3qf1bpncb14i9gzma4vr089pc3pv-bash-4.4-p19/bin/bash"
/nix/store/p4imbrdwi30hai4459vpn4lj1dfvqnv3-helmfile-0.16.0-bin

Success on x86_64-linux (full log)
Attempted: helmfile
Partial log (click to expand)
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/wb85zlzvm3fq0jmsbky2j9j4gsgm25bp-helmfile-0.16.0-bin/bin
patching script interpreter paths in /nix/store/wb85zlzvm3fq0jmsbky2j9j4gsgm25bp-helmfile-0.16.0-bin
checking for references to /build in /nix/store/wb85zlzvm3fq0jmsbky2j9j4gsgm25bp-helmfile-0.16.0-bin...
shrinking RPATHs of ELF executables and libraries in /nix/store/f9fbr90bb7qhqki7rgswi2999xzkk1ky-helmfile-0.16.0
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/f9fbr90bb7qhqki7rgswi2999xzkk1ky-helmfile-0.16.0
/nix/store/f9fbr90bb7qhqki7rgswi2999xzkk1ky-helmfile-0.16.0/share/go/src/github.com/roboll/helmfile/test/integration/run.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/bash"
checking for references to /build in /nix/store/f9fbr90bb7qhqki7rgswi2999xzkk1ky-helmfile-0.16.0...
/nix/store/wb85zlzvm3fq0jmsbky2j9j4gsgm25bp-helmfile-0.16.0-bin

Success on aarch64-linux (full log)
Attempted: helmfile
Partial log (click to expand)
strip is /nix/store/jk6j4lh9v5mvjdbdc35sj0zffhhf6s56-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/xh29qys6vcpy8c8yjzp19hrsjmy33rab-helmfile-0.16.0-bin/bin
patching script interpreter paths in /nix/store/xh29qys6vcpy8c8yjzp19hrsjmy33rab-helmfile-0.16.0-bin
checking for references to /build in /nix/store/xh29qys6vcpy8c8yjzp19hrsjmy33rab-helmfile-0.16.0-bin...
shrinking RPATHs of ELF executables and libraries in /nix/store/9irxnbq1hjahm82p75vrqwf0izhmh0s7-helmfile-0.16.0
strip is /nix/store/jk6j4lh9v5mvjdbdc35sj0zffhhf6s56-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/9irxnbq1hjahm82p75vrqwf0izhmh0s7-helmfile-0.16.0
/nix/store/9irxnbq1hjahm82p75vrqwf0izhmh0s7-helmfile-0.16.0/share/go/src/github.com/roboll/helmfile/test/integration/run.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/l4w7xwjy2nmk31fl5kgyy7gg8z7l9n8z-bash-4.4-p19/bin/bash"
checking for references to /build in /nix/store/9irxnbq1hjahm82p75vrqwf0izhmh0s7-helmfile-0.16.0...
/nix/store/xh29qys6vcpy8c8yjzp19hrsjmy33rab-helmfile-0.16.0-bin


Error: OpenAPI - Swagger - Cannot use 'in' operator to search for '$ref' in undefined

I am trying to use graphql-mesh with the [Orbit API}(https://docs.orbit.love/reference) https://docs.orbit.love/reference and its Swagger definition with the following config:
sources:
  - name: Orbit
    handler:
      openapi:
        source: https://app.orbit.love/api-docs/v1/swagger.json

When trying the serve, I receive the following error:
Unable to serve mesh:  Cannot use 'in' operator to search for '$ref' in undefined {"stack":"TypeError: Cannot use 'in' operator to search for '$ref' in undefined
{
  "swagger": "2.0",
  "basePath": "/api/v1",
  "info": {
    "title": "Orbit API",
    "version": "v1",
    "description": "Please see the complete Orbit API documentation at [https://docs.orbit.love/reference](https://docs.orbit.love/reference)."
  },

Noticed similar issues:

https://github.com/Urigo/graphql-mesh/issues/424
https://github.com/Urigo/graphql-mesh/issues/261

Is this related to #3550 ?
Im biting it too.

same issue, have been fixed ?

tsoa spec

There was a problem resolving type of 'Booking'.
Generate swagger error.
TypeError: Cannot use 'in' operator to search for 'typeParameters' in undefined

We are currently rewriting the OpenAPI handler and releasing it as newOpenapi
https://github.com/Urigo/graphql-mesh/blob/master/packages/handlers/new-openapi/yaml-config.graphql
But still after this PR, this should be working in the existing handler as well;
https://github.com/Urigo/graphql-mesh/pull/3551
Also see;
https://github.com/Urigo/graphql-mesh/tree/master/examples/openapi-orbit


[FEAT] stdin support

markdownlint-cli supports read from stdin, and markdownlint itself supports --stdin option to parse stdin input. Is it possible to make markdownlint-cli2 support stdin?

I'll look into whether it is straightforward to do this by supporting - as with cat and tar. but if it is not, I will leave things as they are and recommend those other tools for this scenario.
FYI, I've asked before, and have not heard a particularly compelling scenario for why standard input was necessary versus creating and using a temporary file.

I'll look into whether it is straightforward to do this by supporting - as with cat and tar. but if it is not, I will leave things as they are and recommend those other tools for this scenario.
FYI, I've asked before, and have not heard a particularly compelling scenario for why standard input was necessary versus creating and using a temporary file.

I would not say standard input is necessary for linting, but it does help editors like neovim to update diagnostics after changing the active buffer (without saving it to disk). Currently, nvim-lint uses markdownlint and markdownlint-cli2 without stdin support, and because of that, users have to save the buffer to see diagnostics update. I am also trying to customize my configuration to make markdownlint in nvim-lint work from standard input, in that case, I would like to stick with markdownlint if it works well.
Anyway, thank you for developing an amazing linting/formatting tool for markdown!


dart2 upgrades

@pq

Yeah, I think I'm just waiting for appveyor to support a 2.0.0 sdk? Or, are we there already?

I think they do... At least the linter builds are passing...
https://ci.appveyor.com/project/pq/linter/branch/master

@devoncarew - going to publish?


Tutorial on c-t.org for verifying CT in Chrome is out of date

here: https://www.certificate-transparency.org/certificate-transparency-in-chrome

Would be helpful to know which bits are :)
Also, the detailed SCT info in DevTools change has landed, I should update the page with that.

The original reporter was a bit vague :)
"I've just realised that the tutorial to verify Certificate Transparency on certificate-transparency.org is not up to date with the latest Chrome UI update which modified a lot of things."

I've added a screenshot that shows the new DevTools UI for CT.


Requirements for Mac need update or clarification

Hello
We could not get the add-in to work on Mac. After working with Premier Support we got information that for Mac Add in to work there is a need to have OAuth configured between local Exchange and Office 365 as well as there are requirements regarding matching the UPN, user sign-ing info to Teams and Office apps and e-mail address of the user. Can you please verify this information with the product owners and update the documentaiton if needed to make sure all prerequisites are included?

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

ID: 37e10f52-d726-d29a-e0a8-93729a32dfae
Version Independent ID: 83555f80-a8ca-ee85-c1fc-e39891029758
Content: Use the Microsoft Teams Meeting add-in in Outlook - Microsoft Teams
Content Source: Teams/Teams-add-in-for-Outlook.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@Larryngolog Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

Hello @scanum as I mentioned I have already opened the ticket with Microsoft's premier support and I have provided information what was the outcome - there is a need to have OAuth configured between local Exchange and Office 365 as well as there are requirements regarding matching the UPN, user sign-ing info to Teams and Office apps and e-mail address of the user.


Add values_at

Besides the standard usecases from Ruby, given #683 was accepted, this can return a tuple and thus be used as a splat:
def add(a, b)
  a + b
end

data = [1, 2, 3, 4]
add(*data.values_at(0, 3)) #=> 5
add(*data.values_at(3, 1)) #=> 6

Awesome! And thanks for the doc comments.
I think I wanted to have/use values_at many times but didn't want to pay the price of the array allocation. Tuple#map was the solution :-)
Maybe we should also add overloads for receiving an Enumerable, so you can pass an array of keys, for example. What do you think? (although the times I remember I used values_at, I wanted a fixed set of values)

Yeah, I think I'd prefer to leave it like this, those overloads would return an Array instead of a Tuple, so just increase the inconsistency for little benefit.
Btw. looking at this, maybe we should think about a Indexable?

Oh, I also forgot MatchData I guess...

+1 for Indexable(I) That wouldn't have left MatchData away. Makes total sense.
I will declara just a abstract def [](index : I) I guess. Pretty much as Enumerable(T).

I'm thinking more of as def fetch(index : I, &fallback : -> V) : V (whether to name it fetch or at, I'm not sure) and Indexable(I, V). That could provide generic implementations for a bit more stuff (rough version):
def fetch(index : I, &fallback : -> V) : V

module Indexable(I, V)
  def fetch(index : I)
    fetch(index) { raise IndexOutOfBounds.new }
  end

  def at(index : I, &fallback : -> V)
    fetch(index) { yield }
  end

  def at(index : I)
    fetch(index) { raise IndexOutOfBounds.new }
  end

  def [](index : I)
    at(index)
  end

  def []?(index : I)
    at(index) { nil }
  end

  def [](range : Range)
    range.map {|index| at(index) }
  end

  def [](start : I, count : Int)
    self[start..(start+count)]
  end

  def values_at(*indexes : I)
    indexes.map {|index| at(index) }
  end
end

Of course more efficient implementations can always be provided at the the target class, Ruby does that a lot with Enumerable, Array and Hash.

Yeap, better, that will provide consistency. The Enumerable has the nice thing that the method the user will need to definie y pretty same he/she types for doing the basic operation #each. But is not a huge difference and force the developer to think about "out of bounds".

On second thought Indexable(I) is better, but keeping &fallback : -> V with an in method free var, updated above.


Question: How to properly add intl-polyfill to support Safari browsers

I'm using react-intl for handling app translation. I know that Safari does not support Intl API and so a polyfill has to be used to make the app load on said browser.
I've found an example of how such Intl polyfill code could look like, here:
https://github.com/andyearnshaw/Intl.js/#intljs-and-browserifywebpack
but I'm not sure how to properly add it to the webpack's config in Slingshot.
So I kindly ask if anyone could explain to me how to use the code above in react-slingshot case? Thanks.

You should be able to put this section below in your App.js
if (!global.Intl) {
    // line below allows for code splitting
    require.ensure([
        'intl',
        'intl/locale-data/jsonp/en.js'
    ], function (require) {
        // requires like this will just initialize them, and the modules set the globals
        require('intl');
        require('intl/locale-data/jsonp/en.js');
    });
}

I added it the same way webpack-public-path.js is added, as suggested.
It's working fine, thank you @kwelch!


Version Controlling with Git in Azure Repos

Hi @AMITH2205
If it is the beginning of any issue, kindly provide more info.
Thanks


remove TAU, TWO_PI

I found that the parameter TAU is the same as TWO_PI and both of them have never been used in sculptToGLSL function, so I remove both of them.

These constants are not used by the library itself internally, but are defined so that the user code can reference them. They are not conventionally exported like they would be from a normal library which can be confusing, but this was a tradeoff we've made to make the user code be as minimal and as quick to script with as possible. See #59

Thanks, I will read it later.

For extra context, the way we run use code is by using eval near the bottom of the file. This allows maybe variables to automatically be in scope without having to import everything, which makes it faster/easier for quickly scripting in the live editor, but more challenging for building standard applications with the library. P5.js also has two different modes that it can operate in for this reason, that linked issue I mentioned is one possible compromise.

eval function here

Hi, PWhiddy
Do you mean P5.js itself also use eval function for live editor and use another method for building the application library? Is there any further information for this two different modes in P5.js codebase (e.g. Github link)? Thank you.

Here is some documentation of p5's "instance mode" vs "global mode"
https://github.com/processing/p5.js/wiki/p5.js-overview#instantiation--namespace


Add Naive UI

Name with link
Naive UI
Description
A Vue 3 Component Library. Fairly Complete. Customizable Themes. Uses TypeScript. Not too Slow.
Link to the vector file, preferably from an official source
Github
Logo

Added in 80c549990a0f513dcbaf3c9eab6a1a620df76088
Thanks


Fix regression in BufferedConsole after TS migration

Summary
This PR is to add an end to end test for #8020.
Fixes #8020

Thanks @natealcedo! This exposed a regression in my migration of jest-util, great we caught it before release 🙂
I pushed a fix and cleaned up the tests

Great Stuff @SimenB! Was working on feature I've been thinking about implementing and noticed that the local build was failing. I'll open an early PR on that feature to get feedback :)

Codecov Report

Merging #8021 into master will decrease coverage by <.01%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master    #8021      +/-   ##
==========================================
- Coverage   64.11%   64.11%   -0.01%
==========================================
  Files         262      262
  Lines       10170    10171       +1
  Branches     1818     1818
==========================================
  Hits         6521     6521
- Misses       3250     3251       +1
  Partials      399      399

Impacted Files
Coverage Δ

packages/jest-util/src/BufferedConsole.ts
70% <33.33%> (-1.43%)
:arrow_down:

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 392a815...3012e44. Read the comment docs.


Update IoT Onboarding Application so that IoT devices are Onboardable from Windows 10

This change modifies the IotOnboarding background task to conform to the IoT Device Wi-Fi Beacons Partner Spec.  This change will be compatible with upcoming Windows 10 changes that will be accessible from the "Settings->Devices->Connected Devices->Add a Device" console.  This change does not affect onboarding from other devices.

Hi @jslobodzian, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Hi @jslobodzian, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


Height is 100%

Hey,
i included your js in my rails project. It works fine except that the height of the notification seems to be 100%.

Maybe somebody knows a fix for that

right click on the notification and inspect the element. Verify that no css classes are setting the height to 100%.
This issue is complicated to resolve without having access to the page causing the issue.


Notification for failed dependency installation

on hold while developing a "universal installer" for python connectors

https://github.com/specklesystems/speckle-qgis/commit/d67d29828f863dffc8c1347838f23ae4c06fe2cd


Segmentation Fault due To build conflicts

Hi, I'm observing a segmentation fault as described in https://github.com/jupyter/qtconsole/issues/585 of which I believe basemap is the root cause.
This is due to underlying conda c-library conflicts.
It seems the underlying issue with basemap support is the build process for libgeos. I wonder if life would be easier by making calls to https://pygeos.readthedocs.io/en/stable/ instead?

To replicate this issue, this is the minimal build script that results in failure.
conda env remove nept -y
conda create -n nept -c conda-forge  python=3.9.12 -y
conda activate nept
conda install -c conda-forge basemap qtconsole -y
jupyter qtconsole #This fails with segmentation fault.

If the issue at hand is C library conflicts, that requires two packages both requiring the same C library, but vendoring/shipping different versions of it.  I can see raising the issue here to get help re-compiling basemap with whatever version of geos the other library uses, but that doesn't seem to be what you're after.
Since the problem as stated is that conda is installing an invalid environment, might I ask why you did not move the issue to the conda repo?  There is a decent chance conda is doing the best it can with the information it has, in which case, have you tried raising this issue with the conda-forge maintainers and asking them to use the same version of geos for building basemap and the other package?
Here's where conda-forge declares basemap needs geos to build, though it looks like it specifies that the geos version installed at runtime must match that used at build time, so conflicts would be on the other package pulling in geos (presumably without that specification). It sounds like you think qtconsole is the other package that depends on geos, but it looks like the only C library qtconsole directly depends on is python.  Of the other packages installed in the log on the previous post, the only ones that look likely to link against geos are dependencies of basemap (build dependencies, even).
If you're just looking for workarounds, have you checked the versions of the installed packages on the working version, then attempted to install exactly those versions (except for the different python, since you want that to be 3.9)?  There is an excellent chance this won't work, but using <${version} instead of ==${version} might.  What about the other way around?  Does installing the package versions used by the broken installation with the newer python fix things?
I'm not seeing any obvious reason you can consistently crash qtconsole while everyone helping you can run qtconsole just fine, so could you try installing just basemap (no qtconsole) and seeing if that produces decent plots in the normal python console, then just qtconsole (without basemap) to try your test, then qtconsole and matplotlib (still no basemap), and seeing which versions cause problems?
What does pip list --user show? Is there anything there that would have a different version of geos than that used for the py39 build of basemap?
I'm suspecting you haven't set LD_PRELOAD, but echo "${LD_PRELOAD}" could check that.  Does it mention a libgeos.so*?  What of your LD_LIBRARY_PATH, is there a directory with a libgeos.so* before the conda environment's lib directory (echo "${LD_LIBRARY_PATH} | tr ':' '\n')?
Perhaps I am missing something.  Could you explain why you think "the underlying issue with basemap support is the build process for libgeos"?  I do remember libgeos conflicts causing problems, when two packages that link against geos bundled it as part of their wheels, but you're using conda, not pip, and the tools to avoid that problem have gotten better in the last decade.
Do other jupyter front-ends (jupyter console and jupyter notebook) cause the same problem?  What about using the QtAgg or QtCairo backends for matplotlib when creating a figure using basemap?
Out of curiousity, what version of conda are you using?  conda env remove nept -y doesn't work for me (with conda 23.11) unless I add -n before nept.  What happens if you try running conda clean -a between deleting the old environment and creating the new one?

Hi I was able to solve this issue by reinstalling conda and cleaning its installation first. Sorry for not replying sooner.
I recall made a number of mistakes mixing repos from pip and the conda forge install which I think made it harder to debug.


Flood Waits due to heavy requests

pyrogram.errors.exceptions.flood_420.FloodWait: [420 FLOOD_WAIT_X]: A wait of 26 seconds is required (caused by "messages.GetFullChat")
ERROR:pyrogram.dispatcher:[420 FLOOD_WAIT_X]: A wait of 14 seconds is required (caused by "messages.GetFullChat")
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/pyrogram/dispatcher.py", line 217, in handler_worker
    await handler.callback(self.client, *args)
  File "/app/DaisyXMusic/modules/play.py", line 465, in play
    administrators = await get_administrators(message.chat)
  File "/app/DaisyXMusic/helpers/admins.py", line 32, in get_administrators
    administrators = await chat.get_members(filter="administrators")
  File "/usr/local/lib/python3.9/site-packages/pyrogram/types/user_and_chats/chat.py", line 862, in get_members
    return await self._client.get_chat_members(
  File "/usr/local/lib/python3.9/site-packages/pyrogram/methods/chats/get_chat_members.py", line 109, in get_chat_members
    r = await self.send(
  File "/usr/local/lib/python3.9/site-packages/pyrogram/methods/advanced/send.py", line 77, in send
    r = await self.session.send(
  File "/usr/local/lib/python3.9/site-packages/pyrogram/session/session.py", line 425, in send
    return await self._send(data, timeout=timeout)
  File "/usr/local/lib/python3.9/site-packages/pyrogram/session/session.py", line 395, in _send
    RPCError.raise_it(result, type(data))
  File "/usr/local/lib/python3.9/site-packages/pyrogram/errors/rpc_error.py", line 78, in raise_it
    raise getattr(
pyrogram.errors.exceptions.flood_420.FloodWait: [420 FLOOD_WAIT_X]: A wait of 14 seconds is required (caused by "messages.GetFullChat")

How to fix this

pyrogram.errors.exceptions.flood_420.FloodWait: [420 FLOOD_WAIT_X]: A wait of 26 seconds is required (caused by "messages.GetFullChat")
ERROR:pyrogram.dispatcher:[420 FLOOD_WAIT_X]: A wait of 14 seconds is required (caused by "messages.GetFullChat")
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/pyrogram/dispatcher.py", line 217, in handler_worker
    await handler.callback(self.client, *args)
  File "/app/DaisyXMusic/modules/play.py", line 465, in play
    administrators = await get_administrators(message.chat)
  File "/app/DaisyXMusic/helpers/admins.py", line 32, in get_administrators
    administrators = await chat.get_members(filter="administrators")
  File "/usr/local/lib/python3.9/site-packages/pyrogram/types/user_and_chats/chat.py", line 862, in get_members
    return await self._client.get_chat_members(
  File "/usr/local/lib/python3.9/site-packages/pyrogram/methods/chats/get_chat_members.py", line 109, in get_chat_members
    r = await self.send(
  File "/usr/local/lib/python3.9/site-packages/pyrogram/methods/advanced/send.py", line 77, in send
    r = await self.session.send(
  File "/usr/local/lib/python3.9/site-packages/pyrogram/session/session.py", line 425, in send
    return await self._send(data, timeout=timeout)
  File "/usr/local/lib/python3.9/site-packages/pyrogram/session/session.py", line 395, in _send
    RPCError.raise_it(result, type(data))
  File "/usr/local/lib/python3.9/site-packages/pyrogram/errors/rpc_error.py", line 78, in raise_it
    raise getattr(
pyrogram.errors.exceptions.flood_420.FloodWait: [420 FLOOD_WAIT_X]: A wait of 14 seconds is required (caused by "messages.GetFullChat")

How to fix this

Floodwaits happen due to heavy requests for userbot or bot
"remember that the API is designed to be used by official apps, by real people; anything above normal usage could be limited." - Pyrogram Docs
Exceptions are set already in places. There is no more places to fix
This happens due to

Too many userbot join requests
Too many admin updates
(2nd one negligible)

And I hope it will get fixed automatically within few hours. and also userbot join floods also get reduced with time

Thanks for the help. Have a good day.


Fix move navigation over castles and promotions

When playing / stepping through a PGN, pygn-mode-next-move might skip one or more moves in certain cases:

The move is a pawn capture, e.g., cxd4.
This PGN fragment: 1. e4 e5 2. Nf3 Nc6 3. Bb5 a6 4. Ba4 Nf6 5. O-O Be7 6. Re1 b5 7. Bb3 O-O 8. c3 d5
If you place the cursor on '5'  (the start of "5. O-O") and invoke next-move. "O-O" is not shown as the next move.

The pull request fixes these issue.
The side effect of this change is that when you reach the end (for next-move), or beginning (for previous-move) of a game, you will nolonger get the error message indicating there is no more move -- which is annoying in my opinion.
The change also simplify the next-move previous-move logic, and make the code more direct.
Visually, when doing next-move, the cursor will landed on the last character of the move.  And when doing previous-move, the cursor landed on the beginning of the move.

Hi!  Thanks, and thanks for revising!
This PR still does too many things at once, some of which are desirable, and some of which are not.
The diagnosis of skipping over pawn captures and castles is correct — thanks — and also applies to pawn promotions.  @dwcoates had noticed the skipping but we hadn't tracked it down.
Just the fix for pawn captures in pygn-mode--strict-legal-move-pat would be great in a separate PR, if you are willing: \\(?:[a-h]x?\\)?[a-h][1-8] .
The fix for skipping over castles should actually be changing these ranges from A-Za-z to a-h:
https://github.com/dwcoates/pygn-mode/blob/503ebb29de55d90e60bc7682f821d915cdbd0ae4/pygn-mode.el#L651-L660 .  I cannot remember why I wrote A-Za-z in the first place.
Some of the other things this PR does include

changing the location of the point when navigating moves
removing the feature of skipping over variations by default
changing pygn-mode--relaxed-legal-move-pat even though that pattern remained unused in the changed code
including annotations as a part of pygn-mode--strict-legal-move-pat, which is intended to be "strict"
removing the message when there are no more moves

most of which would be better in a local defun if these are your preferences.  Did you diagnose a separate problem with annotations?  Let us know.
Ultimately the plan is to use https://github.com/rolandwalker/tree-sitter-pgn for both highlighting and navigation, which will be a lot more straightforward than regexes.

Side note: usages of \s- in the original code may be incorrect, or incorrectly escaped, and need to be checked.

I see.  I haven't used the comment and variation features and totally missed that.  I will rework the pull request to fix the skipping only focus on the regexp.

In case you missed this commit message:
Found this example where after 49. h8=Q, it doesn't land on 49...Rxh8

1. e4 e5 2. Nc3 Bc5 3. g3 h5 4. h3 h4 5. g4 Ne7 6. d3 Ng6 7. Be3 d6
8. Qd2 Be6 9. Nge2 Nc6 10. f4 Bxe3 11. Qxe3 exf4 12. Nxf4 Qg5 13. Nfd5
Qxe3+ 14. Nxe3 Nd4 15. O-O-O Kd7 16. Ne2 Nf3 17. Bg2 Ng5 18. d4 f6
19. Nf5 Bxf5 20. exf5 Ne7 21. Nf4 c6 22. c4 Rae8 23. Rhe1 b6 24. Ne6
Nc8 25. Nxg7 Rxe1 26. Rxe1 c5 27. dxc5 dxc5 28. Ne6 Nd6 29. b3 Rg8
30. Nxg5 Rxg5 31. Bd5 b5 32. Re6 bxc4 33. bxc4 Nxf5 34. gxf5 Rxf5
35. Re4 Rf2 36. a4 Ra2 37. Rxh4 Rxa4 38. Rf4 Ke7 39. h4 Ra2 40. Be4
Rh2 41. Bg6 Kd6 42. h5 Ke5 43. Re4+ Kd6 44. Re8 Rh4 45. Kb2 f5 46. Rf8
Rxc4 47. h6 Rh4 48. h7 Kc6 49. h8=Q Rxh8 50. Rxh8 Kd5 51. Bxf5 a5
52. Bc2 Kc4 53. Rh4+ Kb5 54. Ba4+ Kb6 55. Kc3 c4 56. Kxc4 Kb7 57. Kb5
Kb8 58. Rh7 1-0

but it has no problem with 50. Rxh8 (the very next move with the same
textual representation - Rxh8). So it appears there are a bug some
where else in the scanning code.

Yes, changing the ranges elsewhere from A-Za-z to a-h fixes skipping over castling and promotions.

Yes, changing the ranges elsewhere from A-Za-z to a-h fixes skipping over castling and promotions.

You are right.  Sorry I didn't think about that after reading your earlier comments.

Not sure why the test fail.

Not sure why the test fail.

I believe it's failing lint because your last PR featured an unused variable val-string. I missed this when I merged it. It's not a big deal; if that that unused variable, tests might pass again.

Found an issue with long castles in this game:

e4 c6 2. Nc3 d5 3. Nf3 dxe4 4. Nxe4 Nf6 5. Qe2 Nxe4 6. Qxe4 Nd7 7. Bc4 e6 8. d4 Nf6 9. Qe2 Bd6 10. Bg5 a6 11. O-O-O Qc7 12. Kb1 Bd7 13. Ne5 Rf8 14. Rhe1 O-O-O 15. Rd3 Kb8 16. Rb3 Ka7 17. Qf3 Be7 18. Bf4 Bd6 19. Bg3 Nd5 20. Qh5 Bc8 21. Qxh7 Nb6 22. Qxg7 f6 23. Qxc7 Bxc7 24. Ng6 Bxg3 25. Rxg3 Nxc4 26. Nxf8 Rxf8 27. b3 Nd6 28. Kb2 Kb6 29. c4 Kc7 30. Rg7+ Bd7 31. Rxe6 Nf5 32. Rh7 Nxd4 33. Ree7 Rd8 34. h4 Nf5 35. Ref7 b6 36. Rxf6 1-0

On the board, it first shows short castle, and then on subsequent next-move shows long castle.
Added another change to the strict san regexp.

Thanks very much for the revisions!


Add Conway's Game of Life compute shader example.

Conway's Game of Life compute shader example featuring various ways to use SSBO and Compute Shaders ("transfert" shaders, rendering from a SSBO, ...).

@TSnake41 Added some points to review.

It seems like there are some errors in the cmake run. I can’t see what currently since I’m on mobile

@TSnake41 Thanks for this great example! I'm merging but I'll move the example to others, I think it fits better on that category.


[3D] Graphical artefacts when Ambient Light, Omni-Light and Spot light on the same scene

Bugsquad edit: Initially seen on 1.1-stable
When on the scene there are Ambient Light (from WorldEnviroment) and omni-light together with spotlights then depending on camera position some graphical artefacts will appear, video reference here:
http://youtu.be/JG0xGrrzBaQ
If we add on the scene also directional light the artefacts will disappear (even if energy of directional light is 0).
Here is a project that replicate this issue:
https://dl.dropboxusercontent.com/u/168251/godot/some/issues/Omni_Spot_lights_issue.7z

guess all this will be fixed in 3D renderer rewrite for 3.0

should no longer happen, everything is rendered on one pass now, closing


The pedestrian 3D model not spawn

Hi, @zzuxzt
I compiled these packages and launch the simulator successfully. But no pedestrian model spawned, I noticed that the ros logger were logging the spawning of the agents without any problem, but no agent ever appear in gazebo, I think maybe it is because I do not have the 3D model of the avatar.

This happens when Gazebo is not completely shut down. Did you try to restart the simulation? Restarting can solve most spawning issues. A 3D model of a pedestrian is already included in this package.
Can you see the pedestrian visualization in Rviz? If not, you may launch an empty Gazebo world without pedestrians.

I read the pedsim_ros code and now I am debugging your code. The visualizer does not appear at all though the launch log said it has been launched successfully, it is weird. The agents spawning phase persists about 5 minutes during launching the eng_hall.xml config and eng_hall.world which is the default option and there is still no pedestrian in the world, besides the gzclient kept inactive during the spawning phase. Here are the whole launching logs. I think the problem maybe happened when spawning the Turtlebot2 base, there were two warnings about the parameters of Kobuki:

the Kobuki base does not have RosDebugLevel
the Kobuki base does not have update rate and default to 0

In rviz, there is an error about loading the urdf of the kobuki base_link, the pedestrians shows normally but in gazebo there is only the robot.

The rviz goes like this,  the robot spawner has no problem. And it seems the wrapped pedsim_ros works as well, I can see the social forces.

You also need to get our robot_gazebo package to load the Turtlebot2 robot. The detailed usage of our simulator can be found in our drl_vo_nav package.  You can do this and run the simulation by following these steps:
cd ~/catkin_ws/src
git clone https://github.com/TempleRAIL/robot_gazebo.git
git clone https://github.com/TempleRAIL/pedsim_ros_with_gazebo.git
cd ..
catkin_make
source ~/catkin_ws/devel/setup.sh

roslaunch pedsim_simulator robot.launch
roslaunch robot_gazebo view_navigation.launch

By the way, did you install all the packages for the Turtlebot2? If not, please install them following our drl_vo_nav package.
A GPU is required to make the simulator run fast (especially with our default setting of 34 pedestrians).


Added ahanniga links

Thank you and apologies, I spoke too soon without careful review.

No worries, thanks Saq


Updated the Google Places Api key with a paid account

Issue Reference
This PR addresses the Issue : Fixes #289
Summarize
I have included a new key with billing enabled which is sponsored by my company CloudxLab which will solve the issue of places api not responding

Why does this PR also seem to remove this CSS? https://github.com/IEEEKeralaSection/rescuekerala/pull/378/files#diff-8944b57466f08564caa53a1988261ae0L29

this css was no longer requred so i removed it, i included this file in #289

no it had just one css line which wasn't required

Currently the enter location manually field doesnt do anything, did you exhaust the api ?

Yes it's exhausted hence I added a new key

Its not showing local location names

I think it should suggest local locations.
About local names I'm not sure