ConditionExpression in Entity Update

Hi there,
I said it before but I will say it again: great library! Thank you very much.
I currently have a case where I need to pass in a ConditionExpression into an update call. As far as I can tell this is not yet possible, but should be easy to expose since you are already using ConditionExpression internally as documented here.
Any thoughts on this?

Yep, that's correct, there is already a condition expression builder which takes care of building clear and concise condition expressions. I will look into getting this exposed to the entity manager.

@silberistgold this is now available to use in 1.12.0-beta.1, please refer to Condition based updates
 for more info.


rsa/rsa_eay.c: cache MONT_CTX for public modulus earlier. [1.0.2]

Blinding is performed more efficiently and securely if MONT_CTX for public
modulus is available by the time blinding parameter are instantiated. So
make sure it's the case.
Reviewed-by: Matt Caswell<EMAIL_ADDRESS>Reviewed-by: Richard Levitte<EMAIL_ADDRESS>(Merged from https://github.com/openssl/openssl/pull/7586)
(manually cherry picked from commit 2cc3f68cde77af23c61fbad65470602ee86f2575)

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

ping @dot-asm @levitte @mattcaswell

Thanks!

My pleasure. Merged as 23bfb5b556a4e534fd61fb30719851d4b7b1fc82.


CORTX-32738: rgw:draft changes to tryout Gconf migration for rgw

Signed-off-by: Sachitanand Shelake<EMAIL_ADDRESS>Problem Statement

Problem statement

Design

For Bug, Describe the fix here.
For Feature, Post the link for design

Coding
Checklist for Author

[ ] Coding conventions are followed and code is consistent

Testing
Checklist for Author

[ ] Unit and System Tests are added
[ ] Test Cases cover Happy Path, Non-Happy Path and Scalability
[ ] Testing was performed with RPM

Impact Analysis
Checklist for Author/Reviewer/GateKeeper

[ ] Interface change (if any) are documented
[ ] Side effects on other features (deployment/upgrade)
[ ] Dependencies on other component(s)

Review Checklist
Checklist for Author

[ ] JIRA number/GitHub Issue added to PR
[ ] PR is self reviewed
[ ] Jira and state/status is updated and JIRA is updated with PR link
[ ] Check if the description is clear and explained

Documentation
Checklist for Author

[ ] Changes done to WIKI / Confluence page / Quick Start Guide

started custom build : https://eos-jenkins.colo.seagate.com/job/GitHub-custom-ci-builds/job/generic/job/custom-ci/7146/

retest this please

https://eos-jenkins.colo.seagate.com/job/GitHub-custom-ci-builds/job/generic/job/custom-ci/7186/

retest this please

Tested the patch,
a)  1 Node deployment with yaml based Gconf : 1Node-success.PNG
b) RGW setup logs with yaml based Gconf : rgw_Setup_with_yaml_gconf.log
c) 1 Node deployment with consul based Gconf : Consul_based_deployment.PNG
(Note: here I have updated local docker images by replacing gconf path from yaml to consul so that only rgw will use consul based Gconf and rest of the component will use yaml based Gconf.)
Reference code  : https://github.com/Seagate/cortx-rgw-integration/blob/main/src/rgw/setup/rgw_setup#L42
d) RGW setup logs with Consul based Gconf : rgw_setup_with_consul_gconf.log


Votes column is too wide

The Title column should be closer to the Votes column to see as much of the title as possible.

This is how it should look like. Sadly I could only do that through editing the CSS when inspecting the page 😅

We really tried to read the documentation but it just didn't work...
Maybe this thread can help you @ifaoji: https://github.com/TanStack/table/issues/5115


sarama consumer client can not return to leader when the client fetch from follower which is outed of isr

now, there maybe have a problem, when the  sarama client configured the rackid, it will fetch message from the matched follower replica.  When the consumer After the replica subscribed by the end falls out of the ISR, the sarama client can not notice. Once the replica  falls out of the ISR and does not return to the ISR for a long time, resulting in a backlog of client lag.
I think we can remove followers who have fallen out of ISR from the subscription by listening to ISR on the client side.When the dropped replica returns to the ISR, the leader will return to the corresponding follower for the client to pull data.
What needs to be guaranteed is that the client version > 3.2.3.  This version guarantees that the replica returned by the leader is in the isr.  https://github.com/apache/kafka/pull/12877/files

收到


Standard Streaming Endpoint in Preview?

This document and the pricing page appear to indicate that the Standard Streaming Endpoint is still in Preview, is this correct?  The document is a little ambiguous and it seems strange to make the default something that is not generally available.

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

ID: 8a6e1018-20f7-a898-4f0c-e1b42736fb90
Version Independent ID: 8203dbb7-fc92-a136-4997-fa645be0e829
Content: Streaming Endpoints (Origin) - Azure Media Services
Content Source: articles/media-services/latest/streaming-endpoint-concept.md
Service: media-services
GitHub Login: @Juliako
Microsoft Alias: juliako

@bowlerma Thanks for the feedback! We are currently investigating and will update you shortly.

@BryanTrach-MSFT any news on this one ? Is Standard Streaminf Endpoint still in preview ?

@jkovacev Standard Streaming Endpoint is still considered preview. Even though it's in preview, availability is still guaranteed at 99.9% since it's a purchased streaming unit. At moment, there isn't any tentative timeline for GA to share.


Does not compile on MacOS

On MacOS in a fresh clone, I tried cargo run, but the compilation failed with:
error[E0432]: unresolved import `winit::platform::windows`
  --> src/context.rs:7:34
   |
7  |     dpi::PhysicalSize, platform::windows::IconExtWindows, window::{Icon, Window, WindowAttributes}
   |                                  ^^^^^^^ could not find `windows` in `platform`
   |
note: found an item that was configured out
  --> /Users/byron/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.30.5/src/platform/mod.rs:20:9
   |
20 | pub mod windows;
   |         ^^^^^^^
note: the item is gated here
  --> /Users/byron/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.30.5/src/platform/mod.rs:19:1
   |
19 | #[cfg(any(windows_platform, docsrs))]
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

warning: unused import: `GenericImageView`
 --> src/context.rs:4:13
  |
4 | use image::{GenericImageView, ImageBuffer, Rgba};
  |             ^^^^^^^^^^^^^^^^
  |
  = note: `#[warn(unused_imports)]` on by default

For more information about this error, try `rustc --explain E0432`.
warning: `cleave` (bin "cleave") generated 1 warning
error: could not compile `cleave` (bin "cleave") due to 1 previous error; 1 warning emitted

My recommendation is to setup CI to create builds on Linux at least.

Issue also applies to Linux, exact same type of error message, with cargo version as: 1.82.0 (8f40fc59f 2024-08-21)

Same error on Linux as well.
$ uname -a
Linux skyfall 6.8.0-47-generic #47-Ubuntu SMP PREEMPT_DYNAMIC Fri Sep 27 21:40:26 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

I do not have a linux machine but i've removed the trait as it wasn't needed, also put in some rounding fixes so please let me know if that has fixed the issue

@Exotik850 The new patch compiles fine, but the invalid selection size error persists.

Ah, these lines here are most likely the issue
   let width = width.floor() as usize;
   let height = height.floor() as usize;
   // ...
   if selection.area() as usize != image_data.len() / 4 {

Uploaded another patch

@Exotik850 works on/off, depends on the area selected.
Copying to the actual clipboard will require separate implementations, for macos and both graphical backends on linux (x11 and wayland)
$ ./target/release/cleave
Invalid selection size (936, 436, 407160) (w h p)
$ ./target/release/cleave
Invalid selection size (1318, 554, 728301) (w h p)
$ ./target/release/cleave
Invalid selection size (659, 491, 322910) (w h p)
$ ./target/release/cleave
$ ./target/release/cleave
$ ./target/release/cleave
$ ./target/release/cleave
Invalid selection size (724, 360, 260280) (w h p)
$ ./target/release/cleave
Invalid selection size (1040, 608, 630673) (w h p)
$ ./target/release/cleave
Invalid selection size (992, 384, 379936) (w h p)
$ ./target/release/cleave
Invalid selection size (924, 663, 611688) (w h p)
$ ./target/release/cleave
$ ./target/release/cleave
Invalid selection size (1000, 501, 499500) (w h p)

Well, since the original issue is resolved I'll close this, please open a new issue with your OS info as well if you don't mind

Well, since the original issue is resolved I'll close this, please open a new issue with your OS info as well if you don't mind

Yeah, no worries, I'll open an issue and fork the repository and try to implement a decent solution in regards to the multi-platform clipboards later in the evening as well.


Link broken

At this page
https://developer.mozilla.org/pt-BR/docs/Mozilla/Projects
Spider monkeys recent releases link are no working
(https://developer.mozilla.org/pt-BR/docs/Mozilla/SpiderMonkey/Releases)

Thanks for reporting this. The link was incorrect on the SpiderMonkey page (pulled in via the LandingPageListSubpages macro). I corrected it and updated the Projects page.


@conda-forge-admin,please add bot automerge

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/yaqc-feedstock#15.


CDEBUG_WORKSPACE points to the sidecar filesystem root

Motivation
I have a kind cluster with at least 1 worker and I'd like to use cdebug to instrument a sidecar with enough tooling to run the kubelet in debug mode.
The sidecar is built from a custom image with the following things:

A systemd service for kubelet-debug, this is similar to the kubelet systemd service but instrumented to run the kubelet through delve
An entrypoint script that's run through cdebug exec, when the entrypoint script is executed:

it copies all the tooling to the target image, in this case dlv and the new systemd service
stops the kubelet service and starts the kubelet-debug service

At this point a target container (e.g. kind-worker) is running the kubelet through dlv, then editors can connect to it to start a debugging session.
Steps:

Create an image for the sidecar with tooling to provide debugging tools

FROM golang:bullseye AS go
RUN apt update && apt install git -y
RUN go install github.com/go-delve/delve/cmd/dlv@latest
RUN git clone https://github.com/garabik/grc.git /go/src/github.com/garabik/grc

FROM debian:bullseye
RUN apt update && apt install --only-upgrade bash -y
RUN mkdir -p /app/bin
COPY --from=go /go/bin/dlv /app/bin/dlv
COPY --from=go /go/src/github.com/garabik/grc /app/grc

WORKDIR /app
ADD . .
RUN (cd ./grc && ./install.sh)

ENTRYPOINT ["sleep", "infinity"]

In the Dockerfile workspace there are additional files related to the kubeadm setup of the kubelet
10-kubeadm.conf

Click me
# Note: This dropin only works with kubeadm and kubelet v1.11+
# from the worker node /etc/systemd/system/kubelet.service.d
#
# Override: ExecStart runs the kubelet through delve

[Service]
Environment="KUBELET_KUBECONFIG_ARGS=--bootstrap-kubeconfig=/etc/kubernetes/bootstrap-kubelet.conf --kubeconfig=/etc/kubernetes/kubelet.conf"
Environment="KUBELET_CONFIG_ARGS=--config=/var/lib/kubelet/config.yaml"
# This is a file that "kubeadm init" and "kubeadm join" generates at runtime, populating the KUBELET_KUBEADM_ARGS variable dynamically
EnvironmentFile=-/var/lib/kubelet/kubeadm-flags.env
# This is a file that the user can use for overrides of the kubelet args as a last resort. Preferably, the user should use
# the .NodeRegistration.KubeletExtraArgs object in the configuration files instead. KUBELET_EXTRA_ARGS should be sourced from this file.
EnvironmentFile=-/etc/default/kubelet
# On cgroup v1, the /kubelet cgroup is created in the entrypoint script before running systemd.
# On cgroup v2, the /kubelet cgroup is created here. (See the comments in the entrypoint script for the reason.)
ExecStartPre=/bin/sh -euc "if [ -f /sys/fs/cgroup/cgroup.controllers ]; then create-kubelet-cgroup-v2; fi"
# on WSL2 (and potentially other distros without systemd) /sys/fs/cgroup/systemd is created after the entrypoint, during /sbin/init.
# This eventually leads to kubelet failing to start, see: https://github.com/kubernetes-sigs/kind/issues/2323
ExecStartPre=/bin/sh -euc "if [ ! -f /sys/fs/cgroup/cgroup.controllers ] && [ ! -d /sys/fs/cgroup/systemd/kubelet ]; then mkdir -p /sys/fs/cgroup/systemd/kubelet; fi"
ExecStart=
# I got these args by looking at how skaffold sets up debugging.
ExecStart=dlv exec --headless --continue --accept-multiclient --listen=:56268 --api-version=2 /usr/bin/kubelet-debug -- $KUBELET_KUBECONFIG_ARGS $KUBELET_CONFIG_ARGS $KUBELET_KUBEADM_ARGS $KUBELET_EXTRA_ARGS --cgroup-root=/kubelet --v=2

kubelet-debug-entrypoint.sh

Click me
#!/bin/bash

set -euo pipefail

install() {
  # setup systemd kubelet-debug.service unit
  systemd=/etc/systemd/system
  mkdir -p "${systemd}/kubelet-debug.service.d"
  cp $CDEBUG_WORKSPACE/app/kubelet-debug.service "${systemd}"
  cp $CDEBUG_WORKSPACE/app/10-kubeadm.conf "${systemd}/kubelet-debug.service.d/10-kubeadmin.conf"

  # copy kubelet to a debug file if it doesn't exist already
  if [[ ! -f /usr/bin/kubelet-debug ]]; then
    cp /usr/bin/kubelet /usr/bin/kubelet-debug
  fi

  # copy dlv if not already there
  if [[ ! -f /usr/bin/dlv ]]; then
    cp $CDEBUG_WORKSPACE/app/bin/dlv /usr/bin/dlv
  fi

  # start kubelet-debug unit and disable kubelet unit
  systemctl daemon-reload
  systemctl disable kubelet && systemctl stop kubelet
  systemctl enable kubelet-debug && systemctl start kubelet-debug
}

restore() {
  # restore kubelet systemd unit
  systemctl disable kubelet-debug && systemctl stop kubelet-debug
  systemctl enable kubelet && systemctl start kubelet
}

install
trap restore exit

bash

kubelet-debug.service

Click me
# slightly modified from:
# https://github.com/kubernetes/kubernetes/blob/ba8fcafaf8c502a454acd86b728c857932555315/build/debs/kubelet.service
[Unit]
Description=kubelet: The Kubernetes Node Agent
Documentation=http://kubernetes.io/docs/
# NOTE: kind deviates from upstream here to avoid crashlooping
# This does *not* support altering the kubelet config path though.
# We intend to upstream this change but first need to solve the upstream
# Packaging problem (all kubernetes versions use the same files out of tree).
ConditionPathExists=/var/lib/kubelet/config.yaml

[Service]
ExecStart=/usr/bin/kubelet
Restart=always
StartLimitInterval=0
# NOTE: kind deviates from upstream here with a lower RestartSecuse
RestartSec=1s

[Install]
WantedBy=multi-user.target

Build the image: docker build -t kubelet-debug:latest .
Run cdebug targeting the kind-worker container cdebug exec --image kubelet-debug:latest -it docker://kind-worker '$CDEBUG_WORKSPACE/app/kubelet-debug-entrypoint.sh'
Exec into the kind-worker container and check the status of the systemd services

○ kubelet.service - kubelet: The Kubernetes Node Agent
     Loaded: loaded (/etc/systemd/system/kubelet.service; disabled; vendor preset: enabled)
    Drop-In: /etc/systemd/system/kubelet.service.d
             └─10-kubeadm.conf
     Active: inactive (dead)
       Docs: http://kubernetes.io/docs/

root@kind-worker:/# systemctl status kubelet-debug
● kubelet-debug.service - kubelet: The Kubernetes Node Agent
     Loaded: loaded (/etc/systemd/system/kubelet-debug.service; enabled; vendor preset: enabled)
    Drop-In: /etc/systemd/system/kubelet-debug.service.d
             └─10-kubeadmin.conf
     Active: active (running) since Thu 2023-01-26 05:02:50 UTC; 13s ago

Check that the dlv server started

root@kind-worker:/# journalctl -u kubelet-debug
...
Jan 26 05:02:50 kind-worker dlv[632591]: API server listening at: [::]:56268
...

At this point we can start a debugging session

At this point is where I'd sync the kubelet compiled in debug mode (I wrote the instructions on how to do that here: https://github.com/mauriciopoppe/kubernetes-playground/blob/master/docs/kubelet.md#normal-workflow) and I'd sync it to the directory created by the sidecar and restart the kubelet-debug service

To end the session hit ctrl-c on the terminal where the sidecar is running as part of the teardown most of the instrumentation code will be removed and it'll leave the original kubelet running as it was before

Implementation details
This is possible by creating a special envvar CDEBUG_WORKSPACE that points to the location of the sidecar root filesystem, the sidecar image can use this var to install tooling on the target container as seen above in the cdebug exec execution

Friendly ping, what do you think about this proposal?

Sorry, was (and still am) really short on time for the past few weeks. From first sight, having this flexibility makes perfect sense. But I'll have to give it another look with a clearer head. Hopefully, this weekend. Thanks for your contribution!

Pinging back, what do you think about this proposal?

@iximiuz Sure, I've changed the env var to be CDEBUG_ROOTFS I also added some instructions about how I tested it in the description.


[Event Request] Codeunit 31032 "Prepayment Links Management".CollectSalesLetters

Please add these events.
    procedure CollectSalesLetters(var AdvanceLinkBuf: Record "Advance Link Buffer")
    var
        SalesAdvanceLetterLine: Record "Sales Advance Letter Line";
        TotalAmount: Decimal;
        AdjAmount: Decimal;
    begin
        with AdvanceLinkBuf do begin
            SalesAdvanceLetterLine.SetRange("Bill-to Customer No.", AdvanceLinkBufDefEntry."CV No.");
            SalesAdvanceLetterLine.SetRange("Currency Code", AdvanceLinkBufDefEntry."Currency Code");
            SalesAdvanceLetterLine.SetFilter("Amount To Link", '>%1', 0);
            //---------------------------------------OnBeforeFindSalesAdvanceLetterLine:BEGIN
            OnBeforeFindSalesAdvanceLetterLine(SalesAdvanceLetterLine);
            //---------------------------------------OnBeforeFindSalesAdvanceLetterLine:END
            if SalesAdvanceLetterLine.FindSet then
                repeat
                    Init;
                    "Entry No." := SalesAdvanceLetterLine."Line No.";
                    if AdvanceLinkBufDefEntry."Entry No." = 0 then
                        if AdvanceLinkBufDefEntry."Document No." = "Document No." then
                            AdvanceLinkBufDefEntry."Entry No." := SalesAdvanceLetterLine."Line No.";
                    "Entry Type" := "Entry Type"::"Letter Line";
                    "Document No." := SalesAdvanceLetterLine."Letter No.";
                    "CV No." := SalesAdvanceLetterLine."Bill-to Customer No.";
                    Type := Type::"G/L Account";
                    "No." := SalesAdvanceLetterLine."No.";
                    "Currency Code" := SalesAdvanceLetterLine."Currency Code";
                    "Remaining Amount" := SalesAdvanceLetterLine."Amount To Link";
                    "Posting Date" := SalesAdvanceLetterLine."Advance Due Date";
                    "Due Date" := SalesAdvanceLetterLine."Advance Due Date";
                    Description := SalesAdvanceLetterLine.Description;
                    if AdvanceLinkBufDefEntry."Link Code" <> '' then begin
                        if SalesAdvanceLetterLine."Link Code" = AdvanceLinkBufDefEntry."Link Code" then begin
                            TotalAmount := TotalAmount + Abs(SalesAdvanceLetterLine."Amount Linked To Journal Line");
                            AdjAmount := Abs(TotalAmount) - Abs(AdvanceLinkBufDefEntry."Remaining Amount");
                            if AdjAmount > 0 then
                                TotalAmount := TotalAmount - AdjAmount
                            else
                                AdjAmount := 0;
                        end
                    end else
                        if SalesAdvanceLetterLine."Applies-to ID" = AdvanceLinkBufDefEntry."Document No." then begin
                            TotalAmount := TotalAmount + Abs(SalesAdvanceLetterLine."Amount Linked To Journal Line");
                            AdjAmount := Abs(TotalAmount) - Abs(AdvanceLinkBufDefEntry."Remaining Amount");
                            if AdjAmount > 0 then
                                TotalAmount := TotalAmount - AdjAmount
                            else
                                AdjAmount := 0;
                        end;

                    "Amount To Link" := SalesAdvanceLetterLine."Amount Linked To Journal Line" - AdjAmount;
                    if "Amount To Link" <> 0 then
                        if AdvanceLinkBufDefEntry."Link Code" <> '' then
                            "Link Code" := SalesAdvanceLetterLine."Link Code"
                        else
                            "Links-To ID" := SalesAdvanceLetterLine."Applies-to ID";

                    "Source Type" := "Source Type"::Customer;

                    Insert;
                until SalesAdvanceLetterLine.Next = 0;
        end;
    end;

[IntegrationEvent(false, false)]
local procedure OnBeforeFindSalesAdvanceLetterLine(var SalesAdvanceLetterLine: Record "Sales Advance Letter Line")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Script to detect characters not in subsets

To solve https://github.com/google/fonts/issues/1285 I need a small script that will compare all the characters in all the fonts in the GF repo against the bunch of GF subsets, and against the bunch of glyph sets, and tell us what characters and not in each bunch :)

What's the difference between this request and #1466 ? Can this be closed as a duplicate and the description of this issue be posted there as an additional comment ?

Yep, its a duplicate!


Update Release image to go 1.22

Updating Go version image to go 1.22 in release pipeline

Changes

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you review them:

[ ] Has Docs included if any changes are user facing
[ ] Has Tests included if any functionality added or changed
[ ] Tested your changes locally (if this is a code change)
[ ] Follows the commit message standard
[ ] Meets the Tekton contributor standards (including functionality, content, code)
[ ] Has a kind label. You can add a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ ] Release notes block below has been updated with any user-facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contain the string "action required" if the change requires additional action from users switching to the new release

Release Notes

/cherry-pick release-v0.12.x


Feature/cloud uploader validator no reader data

Three separate (breaking) commits.
Together they make a major version (after the merge).

(breaking) CloudUploader: do not provide file.readerData to the validator: this allows the validator to use web-workers and do the file-reading on the worker
util/image: window.atob() -> atob() for workers: window is unnecessary here and breaks when running in a worker
util/uploadValidator: use native Promise: the nbd/Promise async depends on window, using natively provided Promise instead (all clients use a polyfill at this point).

👍

👍


typo: acess -> access

One image wasn't referenced, but I updated it anyway

Thank you for your contribution. We have had an issue with the public vsts-docs repo, but they are resolved now and I am incorporating the outstanding changes. For some PRs (such as this one) that were opened after the issue began, I am closing the PR, and manually incorporating the changes in our private repo, with a link to this PR in the commit message. Any new PRs will be merged in the regular fashion. Thank you for your contribution.


Update dynamicsummary.md

Change name from Azure Sentinel to Microsoft Sentinel

Learn Build status updates of commit 1473ce8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azure-monitor-ref/tables/dynamicsummary.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@bwren
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


support branch parallel for evoformer

support Branch Parallelism in paper Efficient AlphaFold2 Training using Parallel Evoformer and Branch Parallelism

Thank you, I will review this in the weekend.


fixes for syntax errors in basic usage example

There were some missing closing square brackets on the arrays of the route definitions in the basic usage example.

Thanks for noticing and fixing this


Update for liquidprompt theme v2.0 support

When using the theme liquidprompt v2.0 or later with bash-it, bash issues an error message when the prompt is displayed with the current working directory in a git repository that is not in detached head state.
Description
In v2.0 (released Feb 5, 2021) of liquidprompt, the function _lp_escape() was renamed to __lp_escape(). Therefore, calling the function by the old name results in an error message from bash. By using the new name, the problem goes away.
The shim used by bash-it still uses the old name.
Motivation and Context
The annoying error message goes away with this change.
How Has This Been Tested?
Change made. Issued bash-it reload. Error message no longer appears.
Screenshots (if appropriate):
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] My code follows the code style of this project.
[X] If my change requires a change to the documentation, I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[ ] If I have added a new file, I also added it to clean_files.txt and formatted it using lint_clean_files.sh.
[ ] I have added tests to cover my changes, and all the new and existing tests pass.

Greetings @cdoenges !
I'm guessing you didn't see #2175 which is also trying to fix this.
Maybe you can collab with @DarrenBishop  to review each other's changes and decide on the best way forward.

please see the conflict with master and decide if this still relevant?


[PE-D][Tester C] Feature that don't fit well with the product

There is this feature where financial advisors can enable or diable visuals by clicking on the tick button.

However, the application is supposed to be optimized for CLI. I would suggest that this functionality being controlled by command typed in the CLI instead. This is because switching from CLI to mouse back to CLI again can be very frustrating for users that prefer CLI.

Labels: type.FeatureFlaw severity.Medium
original: IanCheah/ped#12

Thank you for your suggestion! We can plan this as a feature enhancement.
Our current design consideration is that as we do not foresee the users constantly toggling this additional function, it is reasonable to make it clickable, and leave the commands to be centric around the core functions that operate the contact book. Additionally, as it is a one-click step, this is in line with the original copy URL for help designed for AB3. This is something we believe is well within constraints.


feat: Automatic creation of namespaces on control plane for autonomous agents

Closes #109

Codecov Report
Attention: Patch coverage is 66.66667% with 15 lines in your changes missing coverage. Please review.

Project coverage is 48.80%. Comparing base (aefc4b6) to head (27741d2).

Files with missing lines
Patch %
Lines

cmd/principal/main.go
0.00%
7 Missing :warning:

principal/event.go
73.91%
2 Missing and 4 partials :warning:

principal/options.go
77.77%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #177      +/-   ##
==========================================
+ Coverage   48.75%   48.80%   +0.04%
==========================================
  Files          52       52
  Lines        3384     3424      +40
==========================================
+ Hits         1650     1671      +21
- Misses       1589     1604      +15
- Partials      145      149       +4

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


Use the new AutoInject attribute to remove all constructor injections

After developing the attribute and Source Generator, it is required to use this new feature in the projects.

After developing the attribute and Source Generator, it is required to use this new feature in the projects.

Which projects?
@msynk

@maryamhdr
I think all projects?
@ysmoradi

@maryamhdr I think all projects? @ysmoradi

Yes, Components Playground, Todo Template, and even Bit Platform Web Site
Both Backend and frontend (Including razor files)

@maryamhdr I think all projects? @ysmoradi

Yes, Components Playground, Todo Template, and even Bit Platform Web Site Both Backend and frontend (Including razor files)

So isn't it better to submit separate issues for each project?
@ysmoradi @msynk

@maryamhdr
ok, I'm closing this one and will submit new ones for each project.


the poopins

the cronitons are misilinged, the taxes arent filed, it's been 79 years, and we have yet to build satern.

satern can wait.


Zoom level resets after using windows lock screen

Windows Terminal version
1.17.11461.0
Windows build number
10.0.19045.0
Other Software
No response
Steps to reproduce

Open terminal
zoom using ctrl+mouse scroll wheel
lock windows (WIN + L)
unlock windows
observe that the zoom in terminal has been reset to the level it was at step 1

Expected Behavior
I expected that the zoom level stays the same as I set it to before I unlocked windows. For me it's good if zoom level resets when terminal is closed, but I don't expect it to change after only locking windows.
Actual Behavior
After returning from the windows lock screen, the zoom level did not stay where I set it before I locked windows.

I'm fairly confident that this is the same root cause as /dup #11522. Thanks!


fix(harness)!: Compile with new libtest-mimic

Looks like I got frustrated with GH not responding in #23 and didn't notice the root cause; github disabled the actions because they were also scheduled and I hadn't touched this repo enough.


Error -36: Not Implimented

I got to run the script once, and it completed on exporting 16 layers (visible only, no trim, png-24), but it didn't actually create any files in when I checked after running it.  The directory I used was  "F:/art/output" if it matters
After this, I now get the following message when I run the script, and restarting photoshop CS4 doesn't help, nor does redownloading the script:
Error -36: Not implemented.
Line: 1143
->
formatOptRoot.quality.notify("onChanging");
(Thank you for the script by the way, this is going to be immeasurably helpful!)
edit: I commented out this one line and the script now runs as it should, wunderbar!

I get the same behavior after running once in CS4 64bit.

I get this also

I had the same experience: it worked once beautifully, then produced an error when running on the same file. Please advise, thanks.

Another person here with the same issue. Only ran the script once successfully before this error appeared.

Should be fixed by catching the error that happens when applying stored settings, but the settings will revert to default every time :-(

Same issue again, different line (1372) I just commented that line to keep working.
Looks like "formatOptRoot.quality.notify("onChanging");" is not supported on CS4


[MastodonBridge] Add support for various platforms

Add support to various platforms, within orthodox limits of ActivityPub.
The bridge is tested to fully support: Castopod (podcasts only), Friendica, GNU Social, Mastodon, Misskey, Mobilizon, Peertube, Pixelfed[^1], Pleroma, Plume, and WriteFreely.
The bridge should now partially support:

BookWyrm, only "finished reading" updates. Reviews could be supported but it'll require extensive formatting, so it may be preferable to just use the RSS provided by the instance.
diaspora, without boost filtering (since they're typed as Notes like everything else). I could filter the recycle symbol on the content if the maintainers wish.
Funkwhale, if the requests are sent without signature (see https://github.com/superseriousbusiness/gotosocial/issues/538). As such nosig option is added.

The bridge does not support:

Hubzilla and Socialhome, since it does not seem to display outbox
Lemmy, because @username@instance could refer to both the user username and the community username
Nextcloud Social, since it did not implement webfinger

[^1]: If the instance allows outbox requests

Haven't tested but trust that you have. Those comments about support of software should go in bridge description maybe? To help users.


[BUG no downloads available for every game

Describe the bug
This started a few days ago. I updated my Hydra and no downloads were available for any game. I don't know what happened and if this is a general issue.
Steps to Reproduce
Just open the Hydra launcher, I guess.
Expected behavior
No response
Additional information and data

Operating System
Windows 10.
Hydra Version
3.0.5 "Skyscraper"
Before opening this Issue

[X] I have searched the issues of this repository and believe that this is not a duplicate.
[X] I am aware that Hydra team does not offer any support or help regarding the downloaded games.
[X] I have read the Frequently Asked Questions (FAQ).

Have you read the FAQ?
The answer is in the FAQ

Have you read the FAQ? The answer is in the FAQ

Thanks, but I'm having issues with the .json file. How can I get it (my .json file) working correctly?

you should not upload a json file, instead, you need to find a community hydra font and use its raw json url

Have you read the FAQ? The answer is in the FAQ

Where's the FAQ? It's hard to read something when it's hidden.


Add react-native-web-css-in-js

Example of https://github.com/necolas/react-native-web/

Very interesting one, thanks @necolas!


MS Edge, almost working colors

It looks like Edge engine (not the new Chromium edge) almost has support for colors now, 3 years after #417
Could we revisit enabling them / If I submit a PR would it be considered?

I don't use or have Edge so if you can prove the colors work now then absolutely :)

4.3.4


Enable SBAS for UBlox

Looking at the code it looks like UBX_CONFIGURE_SBAS is not defined, so it seems for the ublox sbas is disabled. What is the reasoning behind this?
https://github.com/PX4/GpsDrivers/blob/master/src/ubx.cpp#L221

Flaky performance in some reasons of the world, e.g. Australia.


Feature: Features for the RPC mod

Two cases of rpc mods for producer.

If None is returned from the consume function, I would like the publish function to return None instead of an empty byte string.

I would like to be able to put pydantic model on publish with rpc=True so that the RPC response is automatically validated.

# ------------- SERVICE 1 -------------

from faststream.rabbit import RabbitRouter, RabbitBroker, RabbitQueue

broker = RabbitBroker()
service_1 = RabbitRouter()
broker.include_router(service_1)

class DTO(BaseModel):
    user_id: UUID | None

    queue: ClassVar[RabbitQueue] = RabbitQueue('v1.get', auto_delete=True)

class UserDTO(BaseModel):
    id: UUID
    name: str

@service_1.subscriber(
    queue=DTO.queue
)
async def handler(body: DTO) -> UserDTO | None:
    if body.user_id is None:
        return None
    return UserDTO(id=body.user_id, name='name')

# ------------- SERVICE 2 -------------

from faststream.rabbit import RabbitBroker, RabbitQueue

class DTO(BaseModel):
    user_id: UUID | None

    queue: ClassVar[RabbitQueue] = RabbitQueue('v1.get', auto_delete=True)

class UserDTO(BaseModel):
    id: UUID
    name: str

broker = RabbitBroker()

async def send():
    # 1 CASE
    message = DTO(user_id=None)
    result = await broker.publish(message, message.queue, rpc=True)
    # result => b''
    # but I want to:
    # result => None

    # 2 CASE
    message = DTO(user_id=uuid4())
    result = await broker.publish(message, message.queue, rpc=True)
    # result => {'id': *some UUID*, 'name': 'name'}
    # but I want to:
    # result = await broker.publish(message, message.queue, rpc=True, response_model=UserDTO)
    # result => UserDTO(id=*some UUID*, name='name')

I think, we should implement it via Generic: broker.publish[int](..., rpc=True)
Or even with broker.publish[RabbitMessage](..., rpc=True) to return raw message with all headers and other params

Well, the concept changed. I think, we will make decoder lazy in 0.6.0, so u will be able to serialize RPC response to any time right from bytes (also, we should merge #1649 first)


Displaying points outside of innerView

It looks like there were some recent changes to chartSettings that messed up my project. In the Multi-chart touch tracking, how do you get the point to not get cutoff near the bottom edges (or any edge)?
In my project I had just increased the bottom/leading/top/trailing settings to make sure the point never gets cuts cutoff. However, with the new update, it now gets cutoff. Can you advise on the solution? I thought it had something to do with the customClipRect setting, but maybe I'm doing it wrong. If we can fix it in the Multi-chart example, I can apply it to my project.
Also related: When my line is touching the bottom (or top) of the chart, part of the thickness of the line is getting cutoff. Not sure if that is related to this issue, but that didn't happen to me before (because adjusting the chartSettings solved it).

Hey.
It should be like this a while already, because the content has to be clipped for zooming and panning (otherwise when you zoom everything will be displayed outside of the inner boundaries).
I added a containerViewUnclipped to be able to add overlays that can display outside of the chart inner frame... I'm thinking now, also, that in general we probably should disable the clipping when the chart doesn't support zooming and panning. What do you think?

So before the recent updates it used to always be unclipped? How does contrainerViewUnclipped work in practice? I'm just not able to figure out what the actual solution is. I don't think there's an example I could follow.
Is there something I have to change here?
                let containerView = U(frame: chart.containerView.bounds)

                let point = ChartPointEllipseView(center: chartPointModel.screenLoc, diameter: 12)
                point.fillColor = tintColor.withAlphaComponent(0.5)
                containerView.addSubview(point)`

It should have stopped working in master since Jan. 23 which is when I merged the zooming and panning.
I added a setting to disable the clipping.
There's still something incorrect though, when you start tracking the background view of the bottom labels doesn't hide completely the axis labels but this should be easy to fix. I'll give it a look later when I have time again.

I forgot. If you want to use multi-tracker with zooming, yes, you'd have to use containerViewUnclipped. In RangedAxisExample there's an example for this. The 3 markers on the right side use it. Let me know if you have any questions!

Perfect! chartSettings.clipInnerFrame = false fixed it for me. You might want to consider setting that as the default. Thanks again!

Ok, nice!
I think having it on as default is more suitable as when using zooming and panning, which probably many users will do it would seem broken. And you should be able to enable zooming and panning without having to care about internals like the clipping.
I also thought about coupling it to the zooming and panning, i.e. if clipToBounds = zoomPan.zoomEnabled, and having an additional setting forceClipping with which you can enable clipping when zooming is disabled. But the current approach seems easier to understand, so I went with it.


add explicit_unsteady and implicit_unsteady

adds methods for time marching simulations (e.g. discrete adjoint)

We can remove UnPack as a dependency right?  I didn't see anywhere you actually use it.  You do use it in the tests so we can leave it as a test dependency (alternatively we could require 1.7, where similar functionality is built in)


New atmos interface and off-load from Transparency

New types for more flexible atmospheres, can be ingested from different codes. New machinery for continuum opacities and Voigt profile, off-loaded many parts from Transparency.jl. Added new sample scripts for computing intensity from populations.

Codecov Report
Patch coverage: 79.45% and project coverage change: -8.61% :warning:

Comparison is base (c9c6390) 98.22% compared to head (f750263) 89.61%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
- Coverage   98.22%   89.61%   -8.61%
==========================================
  Files           5       10       +5
  Lines         450      751     +301
==========================================
+ Hits          442      673     +231
- Misses          8       78      +70

Files Changed
Coverage Δ

src/read_atmos.jl
51.65% <51.65%> (ø)

src/incline.jl
96.15% <96.15%> (ø)

src/background.jl
100.00% <100.00%> (ø)

src/formal_solvers.jl
100.00% <100.00%> (ø)

src/intensity.jl
100.00% <100.00%> (ø)

src/line.jl
100.00% <100.00%> (ø)

src/lte.jl
100.00% <100.00%> (ø)

src/read_utils.jl
99.10% <100.00%> (+2.69%)
:arrow_up:

src/types.jl
100.00% <100.00%> (ø)

src/utils.jl
100.00% <100.00%> (ø)

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


The refresh icon does not disappear sometimes

Sometimes I found the app has the refresh icon at the bottom when I scroll up. It will not disappear until I scroll down to bottom again to refresh it.
If I try to reproduce it "deliberately", I can never reproduce it.
But it does happen from time to time. I have seen it several times in the past few days.
Rarely seen but it  happens time to time to give weird user experience.

Did you find a solution for this issue?

No. Did you reproduce the same issue?

Yes. I didn't find how to fix this issue in library, but I found another solution :
`override func viewDidAppear(animated: Bool) {
self.tableView.bottomRefreshControl = self.bottomRefreshControl
self.bottomRefreshControl.beginRefreshing()
self.bottomRefreshControl.endRefreshing()
}
override func viewDidDisappear(animated: Bool) {
    self.tableView.bottomRefreshControl = nil
}`

@abereghici nil it in viewDidDisappear is not helping for me because it means the end of my app.

@joe528 Try this :
override func viewDidAppear(animated: Bool) {
        self.bottomRefreshControl.beginRefreshing()
        self.bottomRefreshControl.endRefreshing()
    }

Why would calling these two functions in viewDidAppear resolve my issue? I don't get it. The issue happens when I scroll up at the bottom of a table view. Most times, the refresh icon will disappear as designed, and only in rare condition, the refresh icon does not disappear.

@joe528 Oh, okay. I didn't understand right your question. This piece of code is working when you have 2 controllers, and when you navigate back, refresh controller appears on bottom of the screen.

I think I had the same issue.
Changing if (offset > 0) into if (offset >= 0) inside - (void)brc_setContentOffset:(CGPoint)contentOffset seems to fix the issue.

@joonorbertsv  thanks for the suggestion, I will give it a try and observe it for a while to see if I will never see this issue again (because I can't reproduce it "deliberately")

I have the same issue. This helped me
`- (void)viewWillAppear:(BOOL)animated {
[super viewWillAppear:animated];
self.tableView.bottomRefreshControl = nil;
}

(void)viewDidAppear:(BOOL)animated {
[super viewDidAppear:animated];
self.tableView.bottomRefreshControl = self.bottomRefreshControl;
}`

 lazy var refreshControl: UIRefreshControl = {             let refreshControl = UIRefreshControl()             refreshControl.triggerVerticalOffset = 100             refreshControl.addTarget(self, action:   #selector(handleRefresh),for:UIControl.Event.valueChanged)        return refreshControl     }()
 override func viewDidLoad() {         super.viewDidLoad()     tableView.bottomRefreshControl = refreshControl }
`   @objc func handleRefresh(refreshControl: UIRefreshControl) {
print("handleRefresh","handleRefresh")
refreshControl.endRefreshing()
}`


ModuleNotFoundError: No module named 'tkinter' #92022

https://github.com/microsoft/vscode/issues/92022#issue-575558607

Unfortunately there is not enough detail to debug your issue, but based on your traceback it looks like you're on Linux. If you happen to be using Ubuntu, know that tkinter is not included with the system install and you will need to apt-get it separately.

Yes, I'm on Linux (Debian 10).
All is correctly installed and tkinter works perfectly everywhere else and with otther IDEs
apt policy python3 python3-tk python3-venv
python3:
  Installed: 3.7.3-1
  Candidate: 3.7.3-1
  Version table:
 *** 3.7.3-1 500
        500 http://deb.debian.org/debian buster/main amd64 Packages
        100 /var/lib/dpkg/status
python3-tk:
  Installed: 3.7.3-1
  Candidate: 3.7.3-1
  Version table:
 *** 3.7.3-1 500
        500 http://deb.debian.org/debian buster/main amd64 Packages
        100 /var/lib/dpkg/status
python3-venv:
  Installed: 3.7.3-1
  Candidate: 3.7.3-1
  Version table:
 *** 3.7.3-1 500
        500 http://deb.debian.org/debian buster/main amd64 Packages
        100 /var/lib/dpkg/status

No way to use tkinter with this plugin (using gui or terminal integrated in vscode)
Which detailed information do you need to better check this bug?

Just an update with another package:
apt policy python3-matplotlib
python3-matplotlib:
  Installed: 3.0.2-2
  Candidate: 3.0.2-2
  Version table:
 *** 3.0.2-2 500
        500 http://deb.debian.org/debian buster/main amd64 Packages
        100 /var/lib/dpkg/status

Python code:
import os
import sys
import tkinter
import matplotlib

Preoblems result in vscode:
Unable to import 'tkinter'
Unable to import 'matplotlib'
Unused import os
Unused import sys
Unused import tkinter
unresolved import 'tkinter'
Unused import matplotlib
unresolved import 'matplotlib'

We don't manipulate imports in any way so chances are you selected a Python environment/interpreter in VS Code which doesn't have tkinter installed. Check that the Python interpreter down in the status bar matches what you expect.

I believe so and I tried many times on differnt venv and on OS environment.
Probably the problem is related on the way i decided to install vscode-oss. I used Flatpak.

Does it work if you import tkinter in the REPL of the Python interpreter specified in "python.pythonPath"? If it fails does the exact same interpreter succeed in the import in the terminal outside of VS Code?

Install software on real Debian 10 machine
sudo apt install python3 python3-pip python3-venv
mkdir ~/VEs
python3 -m venv ~/VEs/python3_env
source ~/VEs/python3_env/bin/activate
pip install --upgrade pip
pip --version && python --version && which python

result:
(python3_env) user@pc:~$ pip --version && python --version && which python
pip 20.0.2 from /home/user/VEs/python3_env/lib/python3.7/site-packages/pip (python 3.7)
Python 3.7.3
/home/user/VEs/python3_env/bin/python

Create TkInter test file
vi ~/VEs/python3_env/tkinter_test.py
+++ from tkinter import *
+++ window = Tk()
+++ window.title("Hello World!")
+++ window.mainloop()

Run from Debian terminal (python3_env activated)
(python3_env) user@pc:~$ python3 ~/VEs/python3_env/tkinter_test.py
result: ok
Run from VSCode-OSS terminal (python3_env activated)
sh-5.0$ source ~/VEs/python3_env/bin/activate
(python3_env) sh-5.0$ pip --version && python --version && which python

result:
pip 20.0.2 from /home/user/VEs/python3_env/lib/python3.7/site-packages/pip (python 3.7)
Python 3.7.4
/home/user/VEs/python3_env/bin/python

test python file
(python3_env) sh-5.0$ python3 ~/VEs/python3_env/tkinter_test.py
result: FAIL
ModuleNotFoundError: No module named 'tkinter'
Run from VSCode-OSS GUI (python3_env selected and activated)
result: FAIL
ModuleNotFoundError: No module named 'tkinter
Get python path from VSCode-OSS Terminal using virtual environment
sh-5.0$ source ~/VEs/python3_env/bin/activate
(python3_env) sh-5.0$ pip --version && python --version && which python

result:
pip 20.0.2 from /home/user/VEs/python3_env/lib/python3.7/site-packages/pip (python 3.7)
Python 3.7.4
/home/user/VEs/python3_env/bin/python

check python sys.path and tkinter import
(python3_env) sh-5.0$ python
>>> import sys
>>> sys.path
['', '/usr/lib/python37.zip', '/usr/lib/python3.7', '/usr/lib/python3.7/lib-dynload', '/home/user/VEs/python3_env/lib/python3.7/site-packages']
>>> from tkinter import *
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'tkinter'
>>> quit()
(python3_env) sh-5.0$ deactivate
sh-5.0$

result: FAIL

There's a Python version discrepancy there. In your first example it's Python 3.7.3 but in all your other examples it's 3.7.4 which suggests you're using two different interpreters. That would explain why in one case you have tkinter installed and in all the others you do not.
You can also import tkinter in the case where it's succeeding, print out tkinter.__file__ and then print out sys.path on the environment that's failing to see if that path is being set.

Oh, you're right! That's really strange.
It seems VSCode-OSS cannot see what the system installed using venv
I decided a double check installing Virtual Enviromnent in the old way
From Debian Terminal
user@pc:~$ virtualenv -p /usr/bin/python3 ~/VEs/python3-bis_env
user@pc:~$ source ~/VEs/python3-bis_env/bin/activate
(python3-bis_env) user@pc:~$ pip --version && python --version && which python
pip 20.0.2 from /home/user/VEs/python3-bis_env/lib/python3.7/site-packages/pip (python 3.7)
Python 3.7.3
/home/user/VEs/python3-bis_env/bin/python
(python3-bis_env) user@pc:~$

From VSCode-OSS Terminal
sh-5.0$ source ~/VEs/python3-bis_env/bin/activate
(python3-bis_env) sh-5.0$ pip --version && python --version && which python
pip 20.0.2 from /home/mnnnld/VEs/python3-bis_env/lib/python3.7/site-packages/pip (python 3.7)
Python 3.7.3
/home/mnnnld/VEs/python3-bis_env/bin/python
(python3-bis_env) sh-5.0$

And now they match!
Really strange.
Anyway, even without that issue, TkInter still cannot be found.
After run the script (I provided in the last post) I got the same error.
Here is the result from the VSCode-OSS Terminal
sh-5.0$ source /home/mnnnld/VEs/python3-bis_env/bin/activate
(python3-bis_env) sh-5.0$ /home/mnnnld/VEs/python3-bis_env/bin/python /home/mnnnld/VEs/python3_env/tkinter_test.py
Traceback (most recent call last):
  File "/home/mnnnld/VEs/python3_env/tkinter_test.py", line 1, in <module>
    from tkinter import *
ModuleNotFoundError: No module named 'tkinter'
(python3-bis_env) sh-5.0$
(python3-bis_env) sh-5.0$ pip --version && python --version && which python
pip 20.0.2 from /home/mnnnld/VEs/python3-bis_env/lib/python3.7/site-packages/pip (python 3.7)
Python 3.7.3
/home/mnnnld/VEs/python3-bis_env/bin/python
(python3-bis_env) sh-5.0$

And now the results about the path (sys and tkinter)
From Debian Terminal:
(python3-bis_env) user@pc:~$ python
Python 3.7.3 (default, Apr  3 2019, 05:39:12)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>>
>>> import tkinter
>>> print(tkinter.__file__)
/usr/lib/python3.7/tkinter/__init__.py
>>>
>>> import sys
>>> print(sys.path)
['', '/home/user/VEs/python3-bis_env/lib/python37.zip', '/home/user/VEs/python3-bis_env/lib/python3.7', '/home/user/VEs/python3-bis_env/lib/python3.7/lib-dynload', '/usr/lib/python3.7', '/home/user/VEs/python3-bis_env/lib/python3.7/site-packages']
>>>
>>> quit()
(python3-bis_env) user@pc:~$

From VSCode-OSS Terminal:
(python3-bis_env) sh-5.0$ python
Python 3.7.3 (default, Apr  3 2019, 05:39:12)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>>
>>> import tkinter
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'tkinter'
>>>
>>> import sys
>>> sys.path
['', '/home/user/VEs/python3-bis_env/lib/python37.zip', '/home/user/VEs/python3-bis_env/lib/python3.7', '/home/user/VEs/python3-bis_env/lib/python3.7/lib-dynload', '/usr/lib/python3.7', '/home/user/VEs/python3-bis_env/lib/python3.7/site-packages']
>>>  quit ()
(python3-bis_env) sh-5.0$

Conclusion:
I have to say I tried also another IDE (same script and same Virtual Environment) and all was fine.
It's a pity because I would like to use VSCode-OSS
By the way TkInter fails even without Virtual Environments
Thanks for your time and I hope we will find where how to solve.
I will keep you informed in case of success

Why are the prompts different? In your Debian example it's user@pc:~$ and in the test example it's sh-5.0$. Are you using different shells in each case?
You can also try running Python with -v to get verbose import logging to see if any differences show up there. Lastly, you can try setting "terminal.integrated.inheritEnv": false to try a more clean environment.
And what is "VSCode-OSS"? Are you using a fork of VS Code?

Why are the prompts different? In your Debian example it's user@pc:~$ and in the test example it's sh-5.0$. Are you using different shells in each case?

No. The first is the OS Terminal, the second is the tetminal inside vscode. I change nothing from the original vscode settings.
I'm using bash in my OS and it seems VSCode is using sh (I tryed to change to bash in VSCode too, but same result.

you can try setting "terminal.integrated.inheritEnv": false

in that way I disabled the terminal inside vscode. but still the same issue

And what is "VSCode-OSS"? Are you using a fork of VS Code?

No fork. Name taken just from the following link and from the header of the installed software itself "Visual Studio code - OSS"
https://github.com/microsoft/vscode#visual-studio-code---open-source-code---oss

You can also try running Python with -v to get verbose import logging

After activating veng in OS and VSCode, the return was totally the same
This is a very strange issue
I can only ask if there is an official way to install on Linux. Maybe the problem is really there.
I used flatpak (I took the OSS)
https://flathub.org/apps/search/visual%20studio%20code
Thanks for your time.
And, if I'm the only one with this issue, feel free to close the ticket.
If I will find a way, I will share here

It is a very strange issue and unfortunately you appear to be the only sufferer of it, so I suspect it has something to do with your setup somehow. As I said you can try running Python with -v to see what the import logging emits for a potential clue.
I don't know how far you want to dive into this to try and figure out where the differences are occuring, but import in Python is implemented via https://docs.python.org/3/library/importlib.html which is all in Python so you could theoretically use importlib.import_module() and if need be debug through that code to try and find out what's going on. You can also look at https://docs.python.org/3/library/importlib.html#approximating-importlib-import-module which is a rough approximation of how import works and see where the failure is occurring.
As for how to officially install VS Code on Linux, https://code.visualstudio.com/#alt-downloads has the downloads.

I made a snap install, as an available option from the suggested website.
Now it works, so the flatpak (at least for the OSS version) has issues.
Could you please tell me if all installed way (spap included) available from https://code.visualstudio.com/#alt-downloads are fully Open Source and related to this github project?
Thanks for all!

No, VS Code as shipped from Microsoft is not entirely open source. The vast majority of it is, but there are bits which are not for various reasons.
As for our extension, the vsix that gets installed is currently made up entirely of open source.


docs: fix footer bug on pages with navigation

At breakpoints 1200px and smaller, the footer starts to be overlapped by the navigation because it exists outside the main region (the main region accommodates for the navigation).
This PR fixes that.

Ready-for-merge Checklist

[x] Expected files: all files in this pull request are related to one request or issue (no stragglers or scope-creep).
[x] Documentation updates are clear and concise.

Merging
Please squash when merging and ensure your commit message uses conventional commit formatting.  Documentation updates do not require a CHANGELOG entry.
Be sure to share your updates with the<EMAIL_ADDRESS>mailing list!

Preview: https://deploy-preview-1546--patternfly-elements.netlify.app/components/


No Frames Received for Intel RealSense D435 via RealSense Viewer

I need to be occasionally testing both FRAMOS D435e and Intel RealSense D435 for my applications on the same Ubuntu 22.04 laptop.
RS D435 camera was working but my FRAMOS stopped working which means I had to install the updated firmware. However, now, my Intel RS D435 is not working.
I am getting this error in $ realsense-viewer GUI

$ realsense-viewer
 23/08 13:08:54,240 WARNING [132948055557696] (model-views.cpp:4229) Exception caught trying to detect presets: could not open directory: /home/mona/Documents/librealsense2/presets/
 23/08 13:09:20,397 WARNING [132947659195968] (backend-v4l2.cpp:1132) Frames didn't arrived within 5 seconds

Upon opening realsense-viewer, I get this error

Confirming that I am also not getting any RGB frame in rviz (used to work previously in a docker I created), so I am not sure if this may be a HW problem. Tried different USB port and still same problem.

Hi @monajalal  The camera firmware driver that should ideally be used with SDK 2.50 is <IP_ADDRESS>, whilst you have <IP_ADDRESS> installed.
The newer firmwares after <IP_ADDRESS> contained a change to the Auto Gain Limit Toggle feature that could cause an error like the one that you encountered when using those firmwares with older SDK versions such as 2.50.
Firmware <IP_ADDRESS> can work with SDK 2.50, so updating again to that firmware should hopefully resolve your issue.

Thanks for your notes.
So, I did this:
2024-08-23 13:31:50 [mona@DOS Signed_Image_UVC_5_16_0_1]$ rs-fw-update -r -f Signed_Image_UVC_5_16_0_1.bin

Update to FW: Signed_Image_UVC_5_16_0_1.bin

No recovery devices were found!

But the firmware wasn't updated. Did I do it wrongly?

I got the bin file from here https://dev.intelrealsense.com/docs/firmware-releases-d400
exact file link https://www.intelrealsense.com/download/23422/?tmstv=1713899242&_ga=2.187605931.286611782.1724434091-1350462228.1723576954

If your camera has not had a firmware update failure that puts it into Recovery mode then you do not need to include the -r parameter in the rs-fw-update command.

BTW I just remembered about this StarTech.com USB extension device. I used it and problem was resolved however, I am still getting those errors
https://github.com/IntelRealSense/librealsense/issues/11973#issuecomment-1631188216

Removed the -r flag and it seems working. Should I be concerned about not using USB3 and having to use USB2?
2024-08-23 14:06:26 [mona@DOS Signed_Image_UVC_5_16_0_1]$ rs-fw-update -f Signed_Image_UVC_5_16_0_1.bin

Warning! the camera is connected via USB 2 port, in case the process fails, connect the camera to a USB 3 port and try again

Updating device:
Name: Intel RealSense D435, serial number:<PHONE_NUMBER>32, update serial number:<PHONE_NUMBER>37, firmware version: <IP_ADDRESS>, USB type: 2.1
 23/08 14:06:46,287 ERROR [132530506251328] (librealsense-exception.h:52) set_xu(...). xioctl(UVCIOC_CTRL_QUERY) failed Last Error: Protocol error

Firmware update started

Firmware update progress: 100[%]

Firmware update done

Device<PHONE_NUMBER>32 successfully updated to FW: <IP_ADDRESS>

Also, confirming that with new firmware, I still get an error upon opening realsense-viewer.

I know that firmware <IP_ADDRESS> can work with librealsense 2.50.0 in the RealSense ROS wrapper.  This may not be the case with the Viewer though.  As mentioned earlier, firmware <IP_ADDRESS> is meant to be used with 2.50.0.
librealsense <IP_ADDRESS> can also sometimes work with the ROS1 wrapper 2.3.2 even though it is not meant to work.
If you are happy to use the 2.50.0 Viewer and you also need 2.50.0 for the 2.3.2 ROS1 wrapper then using firmware <IP_ADDRESS> is likely to solve your problems.


bugfixes in the ways data is returned from datasets and imaging transforms

transforms were slightly broken for ITK kind of data. Fixed.
Stack images channel-wise was creating channel dimension last.

maybe some explanation about the changes to datasets?

adjusted datasets such that they return file paths that are relative to the "base_path" of the dataset.


[Impeller] Better optimizations for subpasses which contain only a clip and texture contents.

Flutter's API favors composition over loading on properties on a single widget. That is, while applying a color filter directly to an image is faster:
Image.asset(..., colorBlendMode: BlendMode.colorDodge, color: Colors.red)

Its much more common for folks to nest this under a ColorFiltered widget:
ColorFiltered(colorFilter: ..., child: Image.asset(...)

Additionally, there are many such operations which are not made easy by the current framework API: Namely other types of color filters and image filters cannot be directly applied.
We should be able to avoid the subpass in these cases and provide the texture snapshot directly to the filter.

For the most part, the Skia backend gets away with not having this optimization as the raster cache will replace this with a texture after frame 1.


Azure Lighthouse + Resource Lock (error)

Azure Lighthouse
I sucessfully attached my subscription1 from Tenant1 to my Tenant2. Now, I can manage from Tenant 2 my subscription1.
The problem is about Resource Lock. From my Tenant1, my subscription1 had a lock inside one of the resource groups. Trully, doesn't matter if you have your lock inside either a resource group or resource, from your tenant2 you can't see them.
So, I though: OK. But since I've my lock from Tenant1, he will protect my resource, right? But don't. I could bypass this security from Tenant2 using Lighthouse.
So, I tried to create a lock from tenant2 inside my subscription1 but I found an error bizarre! He didn't do anything and keep all the infos I put, such as Name, Level and Description. Sounds like a cache when you couldn't do anything in a page.
So, I tried to monitoring using devtools and I realized that the problem was like that:

Can someone help me to prevent my team to delete any resource from my client's subscription?

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

ID: 85dc331a-0f34-f9a2-43d9-8ab7e0e635c6
Version Independent ID: 96c07c81-f84c-f47b-659c-b3d055aefb03
Content: Onboard a customer to Azure delegated resource management - Azure Lighthouse
Content Source: articles/lighthouse/how-to/onboard-customer.md
Service: lighthouse
GitHub Login: @JnHs
Microsoft Alias: jenhayes

Hi @lcarli - Thank you for your feedback. Since this channel is more for driving improvements towards MS Docs, could you provide us with a URL of the documentation that your feedback is most applicable to? That way, we're able to locate the right team that's better suited to address your feedback.

Hi, @mike-urnun-msft, Thank you!
I totally agree! Sorry for my mistake to forget that.
I used this documentation (https://docs.microsoft.com/en-us/azure/lighthouse/how-to/onboard-customer) to onboard my costumer.

@JnHs Can you please provide your inputs on this?

Hi, @SwathiDhanwada-MSFT ,
Here is my json parameters
{     "mspOfferName": {         "value": "MyOfferName"     },     "mspOfferDescription": {         "value": "MyOfferName conneted in Client"     },     "managedByTenantId": {         "value": "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"     },     "authorizations": {         "value": [             {                 "principalId": "5baa72c8-f81a-4722-a3c1-cd078788d207",                 "principalIdDisplayName": "ClientManager",                 "roleDefinitionId": "18d7d88d-d35e-4fb5-a5c3-7773c20a72d9",                 "delegatedRoleDefinitionIds": [                     "b24988ac-6180-42a0-ab88-20f7382dd24c",                     "92aaf0da-9dab-42b6-94a3-d43ce8d16293"                 ]             },             {                 "principalId": "5baa72c8-f81a-4722-a3c1-cd078788d207",                 "principalIdDisplayName": "ClientManager",                 "roleDefinitionId": "b24988ac-6180-42a0-ab88-20f7382dd24c"             }         ]     } }

Hi @lcarli - the issue you've experienced with resource locks sounds like the current limitation described here: https://docs.microsoft.com/en-us/azure/lighthouse/concepts/cross-tenant-management-experience#current-limitations
As noted there, you can use system-assigned deny assignments to protect system-managed resources, but resource locks on delegated subscriptions do not prevent actions from users in a managing tenant.

Hi, @JnHs!
Thak you for the explanation and sorry for do not found this before.
One question is: To me, if I really understood, system-assigned deny assignements is only available thru Blueprints. So, I can't apply to Resources or Resources Groups that already exists...
Did I missunderstood something?

Yes @Icarli AFAIK, you understanding is quite correct.
@JnHs your inputs please?

That is indeed correct.

Hi, Guys.
Thanks for the explanation. I'll close this issue.


fix(product-import): allow import of masterVariant

[x] commit messages are commitizen-friendly
[x] code is unit tested
[ ] code is integration tested
[ ] public code is documented
[ ] code is reviewed by at least one person
[x] code satisfies linter rules

fixes #161

Coverage increased (+0.006%) to 97.24% when pulling 4fe2da0f3793162a4fbb26aa4a24d2d92f001650 on 161-product-sync-variants into 6e5c429736e364d3e82b1b256bf2aa57c1000dcb on master.

LGTM


Should snap.hold_refresh change it's functionality to allow for infinite holds?

Currently setting snap.hold_refresh(days=0) will remove the hold on the snaps. What value would be reasonable to allow for indefinite holds?

I don't think there is a way currently, as the hold_refresh function checks that days is less than or equal to 90 and raises a ValueError if not. However, as of Aug 2022 (this snapd commit), snap allows the refresh.hold config option to be set to "forever" (a user can do this with snap refresh --hold or sudo snap set system refresh.hold=forever) to allow indefinite holds.
I'd be in favour of a PR that allows hold_refresh(forever=True) option that enabled this. @zmraul Would you be willing to put up a PR with tests?
I though about hold_refresh(days=None) but I think it's a bit confusing that it would allow days=0 to mean no hold and days=None to mean indefinite (especially when both 0 and None are falsey in Python).

Agree with the proposal for hold_refresh(forever=True) - there are some other bits that landed recently like snap install by revision too

(I should also mention that a really important feature is the per snap hold you can now do - we should think about implementing that)


Add possibility to filter yaml keys

We just now noticed that the fix:structure Command seems to be removing the filtered entries from the result file. I will have a look how we can fix that, but so long this should not be merged and released.


update env with qualimap dependency

Adding qualimap dependency to prepare the test to the full integration.
Many thanks to contributing to nf-core/rnaseq!
Please fill in the appropriate checklist below (delete whatever is not relevant). These are the most common things requested on pull requests (PRs).
PR checklist

[X] This comment contains a description of changes (with reason)
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If necessary, also make a PR on the nf-core/rnaseq branch on the nf-core/test-datasets repo
[ ] Ensure the test suite passes (nextflow run . -profile test,docker).
[x] Make sure your code lints (nf-core lint .).
[ ] Documentation in docs is updated
[X] CHANGELOG.md is updated
[ ] README.md is updated

Learn more about contributing: https://github.com/nf-core/rnaseq/tree/master/.github/CONTRIBUTING.md

Please also add which packages you updated to the CHANGELOG, to make it 100% transparent with a simple look at that :-) Otherwise looking good 👍

sure! do i need to add the version to the CHANGELOG or just the names of the packages?

I saw where to update the versions.thanks!

awesome! I am an official contributor :P
Let me know when I can do the qualimap PR (to address #202), not sure when the docker image will be available with the new versions!

That shouldn't take longer than ~1 hour.


Top down action sample

The next sample will be a top down action game sample. It should be fairly simple, and inspired by some V rising gameplay.
This will also serve as a benchmark as we can simply spawn more units and see where the bottlenecks are.

[ ] Camera: Top down, can rotate and move with the player
[ ] Expose unreal parenting/attachments to Rust
[ ] Add Camera shakes
[ ] Find Attack animations
[ ] Add animation notifies to Rust. With this we can do gameplay things during an animation like axe throwing.
[ ] Access to skeletal mesh sockets, so we know where our hands/weapon is during an animation.
[ ] Change spawning actor to accept a custom UClass*, so we can spawn enemies which are defined in the editor
[ ] Expose some navigation api to Rust. Most of the api seems to use shared ptr and we need to find a way to have those in ffi.
[ ] Enemies should be turned into ragdolls on death.
[ ] Add spells/abilities.
[ ] Bigger levels with streaming?
[ ] ...

https://user-images.githubusercontent.com/1994306/194226627-9a78a5d8-9056-4e44-9af7-23b35fabb581.mp4

https://user-images.githubusercontent.com/1994306/194774612-d6d54bc9-afa9-4d1f-aebf-9df19bef6c02.mp4


Should warn when assigning to catch clause's error or trace parameters

Take a look at:
main() {
  try {
    throw "!";
  } catch (error, trace) {
    error = null; // <--
    trace = null; // <--
  }
}

According to the spec (§17.11), the parameters to catch are final. Given that, the two marked lines should have static warnings.

This is recently enforced by the analyzer; those parameters are now considered final.


Vault wallet migration between currencies

We'll need a companion to https://github.com/interlay/interbtc/pull/1116 that migrates bitcoin wallets

In the case of https://github.com/interlay/interbtc/pull/1116 since we migrate the entire Vault I'm wondering if it is sufficient to simply rename the wallets? We will have a problem with re-scanning however since the issue keys will always be re-imported for the old_vault on restart. What is the advantage again for having separate wallets? I'm wondering if that was only relevant when we had theft reporting enabled. Going back to a single wallet would solve this issue and reduce some complexity.

I think the primary reason we went with separate wallets was the theft checking indeed. But I guess it also helps with the (prometheus) monitoring. I think we also have a lock per wallet to ensure that tx creation is atomic with the sending, but that could be refactored. Rescanning logic would also need to be refactored to fetch issues for all vaults.
Having said that, I'm not necessarily against merging the wallets, but I'd probably ask Dom to sign off on it

IMO it would be good to merge wallets for the following reasons:

One wallet should help with averaging out BTC fees. Technically, users pay on the upper end for redeem fees, but in extreme cases, the vault might have insufficient due to fee fluctuations. Averaging out over multiple wallets should help with that.
Reducing complexity is much needed. Hopefully, this might also help move to descriptor wallets soon (see #457).

Technically the downside is that vaults will need to ensure that they have enough BTC left once they withdraw some in case of liquidations: if the same account has a DOT and USDT vault with the DOT portion being liquidated, then they can withdraw parts but not all of the BTC.

What about the monitoring though?


Resolve iterations bug

I'll check why the test passes now (it may have solve a problem).
I accept the PR and incorporate it to a major refactor
Thanks


git project load --commit does not error if commits are missing

In the steno3d deploy, we download repos via SSH and then do a git project load --commit to tack them to the correct version of code. If that commit does not exist in the tree, the repo will still be on master:HEAD. Since we do not error, the build continues, and potentially puts the wrong version online.
Need to error in this case!

Fixed in #28.
TODO: Test coverage


core[patch]: Remove some usage of .copy() in favor of .model_copy()

Address under place where deprecated functionality is used.

Failed twice due to a flaky test -- we need to increase the time allowance for the rate limiter tests


section.calculate_warping_properties(solver_type='cgs') fails due to NameError

Describe the bug
Calculation of warping properties fails when using solver_type='cgs'
To Reproduce
section.calculate_warping_properties(solver_type='cgs')
sectionproperties\analysis\section.py", line 402, in ilu_decomp
(self.num_nodes, self.num_nodes), linalg.spilu(k).solve
NameError: name 'k' is not defined
Additional context
It seems that issue due partially made refactoring as assemble_torsion was modified to return only lagrangian version of k (k_lg) and no k at all.
Fixing

remove usage of k  and k_precond in calculate_warping_properties will be provided

Thanks for picking this up @simo-11. I have resolved this for the upcoming release, see here.
Fixed by #277.


Add dtr to project

I added an updated DTR document for review.

Added dtr.md

I like the hash syntax of the doc


Strange symbols at the beginning of each console message

I'm using Detekt with Gradle and getting console output like this:
Ruleset: complexity
Ruleset: empty-blocks
Ruleset: exceptions
Ruleset: naming

Please remove these strange prefixes. If it's for colored output, please make "if the console supports colors" detection smarter, like it's done in Gradle, for example.
I'm running Gradle from IDEA.
Your Environment

Version of detekt used: Gradle plugin 1.0.0-RC15
Version of Gradle used (if applicable): 5.4.1
Operation system: Windows 10

Thank you for reporting this.

Thanks for reporting this bug.
Please answer the following questions to help us reproduce the problem.
Which default system shell are you using on Windows?
Which shell does IntelliJ's terminal emulator emulate?

Which default system shell are you using on Windows?
Basically I don't use system shell. But if I have to, I use ёсьвёю I'm not sure if I answered correctly...

Which shell does IntelliJ's terminal emulator emulate?
I don't know.

As I can see in Gradle sources, Gradle uses Ansi library to colorize the output. As I understand, the library checks if current OS is Windows and if it's so, it does some magic.
If I'm right, I suggest choosing between these options:

Omit colors on Windows systems (you can check if os.name system property contains win)
Use Ansi library to colorize output. Personally I don't like this solutions, as it requires to add one more dependency. But you can shade this dependency :)

@marschwar @schalkms I've submitted 2 PRs. Suggest we merge #1702 and close the other.

@remal big thanks 🥇


⚠ Bofrost.es has degraded performance

In 43ff72f, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 2104 ms

Resolved: Bofrost.es performance has improved in d554d9e after 8 minutes.


Avoid creating extra Seq when adding label to mention

This won't save as much CPU time as it cost to write it.


FC-100 data loader

I have found out the FC-100 data from links of your paper, and Could you please share the data loader and detailed setting for FC-100 experiments?

Hi Liu, I want to run your code on CIFAR-FS, which has a 32* 32* 3 shape of Image, and cannot fit to your code. I try to change the parameter img_size from 84 to 32, but it has error with  "Negative dimension size caused by subtracting 5 from 2 for 'pretrain-model/AvgPool' (op: 'AvgPool') with input shapes: [64,2,2,512]". Please help address this issue.

I have replyed to you in this issue.

thanks


Build failure: tere

Steps To Reproduce
Steps to reproduce the behavior:

build tere

Build log
failures:

---- app_state::tests::test_case_sensitive_mode_change stdout ----
thread 'app_state::tests::test_case_sensitive_mode_change' panicked at src/app_state.rs:1711:9:
assertion `left == right` failed
  left: [2]
 right: [1]
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    app_state::tests::test_case_sensitive_mode_change

test result: FAILED. 90 passed; 1 failed;

Additional context
after 49f83b701e7939079c529f378c79fa8544f4db72
Notify maintainers
@ProducerMatt
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.22, NixOS, 24.05 (Uakari), 24.05.20240319.b06025f`
 - multi-user?: `no`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - nixpkgs: `not found`

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

This bug was apparently fixed here, but upgrading to the latest release (or git commit) just gives new errors, which have been mentioned but not fixed here.
I assume that these errors starting, especially in the Nix build system, show that it's some kind of issue with upgrades to dependencies.
running 4 tests
test output_on_exit_without_cd ... FAILED
test first_run_prompt_accept ... FAILED
test first_run_prompt_cancel ... FAILED
test basic_run ... FAILED

failures:

---- output_on_exit_without_cd stdout ----
thread 'output_on_exit_without_cd' panicked at tests/cli.rs:76:5:
assertion `left == right` failed
  left: "Error: Io(Os { code: 2, kind: NotFound, message: \"No such file or directory\" })\r\n"
 right: "tere: Exited without changing folder\r\n"
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- first_run_prompt_accept stdout ----
thread 'first_run_prompt_accept' panicked at tests/cli.rs:126:5:
assertion failed: ptn.find(&output).is_some()

---- first_run_prompt_cancel stdout ----
thread 'first_run_prompt_cancel' panicked at tests/cli.rs:96:5:
assertion failed: ptn.find(&output).is_some()

---- basic_run stdout ----
thread 'basic_run' panicked at tests/cli.rs:61:5:
assertion `left == right` failed
  left: "Error: Io(Os { code: 2, kind: NotFound, message: \"No such file or directory\" })\r\n"
 right: "/build/.tmpbctZr0\r\n"

failures:
    basic_run
    first_run_prompt_accept
    first_run_prompt_cancel
    output_on_exit_without_cd

test result: FAILED. 0 passed; 4 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.11s

Will be fixed by https://github.com/NixOS/nixpkgs/pull/298527


Looking For Maintainers!

If you are interested in helping maintaining this plugin, you can comment here or contact @ThomasV on the Capacitor Slack. to help you get set up on the Capacitor Community Github org 😄
This plugin is only Capacitor 2.x compatible and will not work with the upcoming Capacitor 3

Maybe @robingenz can help 👍 . He has quite a good track record in maintaining Capacitor plugins 😎.

Thank you for the flowers! 😄  @Boosten
Currently I have many private projects and therefore not too much time.
One project should be "finished" in the next few weeks.
So I will keep an eye on this issue.

@bazuka5801 look to have the most advanced fork ! he did the capacitor v3 support already

@riderx Thanks!)
@thomasvidas Now I worked for project what need this plugin work correctly for cap3.
I can maintain it with fresh docgen from capacitor 😉

@bazuka5801 has reached out to me and I've added him as a maintainer for this repo 🎉  So I'll go ahead and close this issue

Thanks @thomasvidas @bazuka5801 !
Cannot wait to use the next update :)


feat(build): binary opt using codegen-units

This would be an "easy" addition to optimize some binaries.
By default codegen-units is set to 16, this could do it just all at once over everthing.
Additional binary opt flags:

strip affects what symbols are found in the binary
lto for link time optimizations (didn't get it working)

Quickly tested it on 2 crates you seem to build, I explicitly chose 2 smaller crates as they should show to effect in even small cases. the following byte measures are from stat --printf="%s" xyz

cargo-all-features (specificall cargo-build-all-features)

No flags # 4'041'456 bytes
-C codegen-units=1 # 3'982'752 bytes
-C codegen-units=1 -C strip="symbols" # 497'984 bytes

minifier (one you've just added)

No flags # 362'816 bytes
-C codegen-units=1 # 362'816 bytes
-C codegen-units=1 -C strip="symbols" # 362'816 bytes

Now minifier is very very (and I can't stress this enough) very small crate with only 2 dependencies and some hunderd lines of code. But cargo-all-features shows that there can be a difference in size.
To point this out clearly, these flags are usually used to have some small perf gains. (which yes, it does)
So whats the down side? Build time. You have longer builds to be faster at execution.
Why not strip? I'm personally not sure about this. cargo-all-features seems to benefit well from it interms of size, but loosing symbols might be counter productive. Then again, if one would like to control them, they could provide binaries themselves and an almost 10x reduction in size is huge, not only for execution but also for downloads speed and the amount of transfer happening.
It's a simple proposal, but I'm happy to hear your opinion

Not sure about strip, seems to be only required for debugging.
BTW, if you enables strip, then I think it is also reasonable to set panic behavior to abort.

Does abort have an effect on panic::catch_unwind%20is%20undefined%20behavior.)?

Thanks for the PR. The codegen-units tweak sounds reasonable and is also very low risk.
strip is something that I would prefer to avoid. Same with panic=abort. I would prefer to ship binaries that are as good as (or better than) the ones that cargo install builds by default. Strip makes debugging harder, so let's avoid it. We already provide orders of magnitude space savings by not requiring the user to download and compile the source code. I don't think we should go hunting after marginal savings if they are going to cause user visible regressions.
If you manage to get lto working, I would be interested in this too, but only if it is not fragile. Fundamentally, cargo-quickinstall is optimised to be as low maintenance for me as possible, so I probably won't accept patches that cause random build failures.


UI issues tab not showing on fragment_explore

UI issues tab not showing on explore fragment

Active Campaigns and Active Course Tabs not showing

Screenshot

How to Fix it

Constraint the layouts properly

Would you like to work on the issue?
yes

@ujjwalagrawal17 I have fixed this issue. Please check the PR here #201


Does Bramble support resolving one field from multiple federated services?

Hi, as you mentioned in the doc:

With the exception of namespaces and the id field in objects with the @boundary directive, every field in the merged schema is defined in exactly one federated service.

Does this mean any field (even it's a boundary object) should be resolved by only one service? Or in other words, does Bramble support resolving a boundary object with unlimited recursion?
Here is the use case for your reference:
## foo service

type Foo @boundary {
  id: ID!
  title: String!
  bar: Bar!
}

type Bar @boundary {
  id: ID!
}

## bar service

type Bar @boundary {
  id: ID!
  subtitle: String!
}

The schema above works well in Bramble, as we could resolve Foo.bar within the bar service.
However, it starts failing when we add a new field in Bar, which can only be resolved in the third service:
## foo service

type Foo @boundary {
  id: ID!
  title: String!
  bar: Bar!
}

type Bar @boundary {
  id: ID!
}

## bar service

type Bar @boundary {
  id: ID!
  subtitle: String!
  baz: Baz!
}

type Baz @boundary {
  id: ID!
}

## baz service

type Baz @boundary {
  id: ID!
  status: String!
}

The query fails when trying to resolve Foo.bar.baz.status: Cannot query field status on type Baz.

Hi @ihac
This should work, bramble currently has no protections against query complexity other than a timeout on execution time, so recursing down to dependent services should be fine.
I'm assuming you've cut out the fields on Query for looking up the @boundary types in the example schemas.
Can you share these example services so we could run a test with them?

Hi @pkqk here is the repo: https://github.com/ihac/graphql-poc-playground/tree/master/bramble-federation if you wanna run a test.
And below are some failed queries (we expected them to work though):
query {
  randomContentWithScoreCard { # resolved by content service.
    id
    title
    score { # resolved by social service
      teamAScore
      teamBScore
      ads { # should be resolved by ads service
        id
        message # cannot query field "message" on type "TinyAdsBanner"
      }
    }
  }
}

# Same issue as above
query {
  masthead_v2(count: 10) { # resolved by masthead service
    __typename
    contentItemsWithScoreCard { # resolved by content service
      id
      title
      score { #should be resolved by social service
        id
        teamAScore # cannot query field "teamAScore" on type "ScoreCard"
      }
    }
  }
}

Also, we tried the federation on union types, but it didn't seem to work. Is this as we expected as well?
query {
  masthead(count: 5) {
    __typename
    ... on ContentItem {
      id
      title
    }
    ... on ContentItemWithScoreCard {
      id
      title
    }
    ... on ScoreCard {
      id
    }
  }
}

The query above even caused a panic in Bramble: runtime error: invalid memory address or nil pointer dereference when it tried to marshal the result into response.
Thanks for help and also your great work!

Any updates here?

@ihac – I suspect that your report about abstract type failures (unions) may be related to https://github.com/movio/bramble/pull/81 based on your query. I haven't been able to trigger panic, but panic itself may be a side effect of your logging.

@ihac — the original issue that you reported with deeply-nested fields not delegating properly was a big problem that I also ran into fairly quickly. It here’s a fix in the works here https://github.com/movio/bramble/pull/85.

@gmac Thanks for the quick fix. Yes the issue title might be ambiguous as we're actually discussing about the graph traversal here.
As for the panic, it happened only when we queried the federated union types. I'll try once again to see whether I could reproduce it with your recent patches included.


On Faster Blocktimes

There is a desire to have faster blocktimes in the OP Stack. From the point of view of the protocol, what is a blocktime exactly?
The only thing that the protocol sees is the incrementing value in the block header. An OP Stack chain with a 2 second blocktime doesn't mean that it must create a block every 2 seconds - it just means that every block that is produced must have its timestamp incremented by 2. In theory a sequencer could build blocks extremely fast, incrementing the timestamp by 2 seconds until it reaches the maximum its allowed to be ahead of the L1 timestamp.
The most simple way to think about supporting 1 second blocktimes is to simply make it so that the timestamp increments by 1 and change nothing else about how time is managed within the protocol. This could be introduced via a hardfork and chains that opt in could have their blocktime change to 1 second. There are some chains that support OP Stack based L3s, which would break if they moved to 1 second. There have not been any hardforks yet that chains have opted out of and introducing this sets a precedent. If we wanted to prevent the need for backwards compatibility concerns, then we can allow a chain operator to set their blocktime via the SystemConfig. This guarantees backwards compatibility and allows chains to modify their blocktime as they see fit, but is much more complex then simply hardforking in 1 second blocktime.
Another way to think about supporting 1 second blocktimes is to change the way that time is managed in the OP Stack. Right now every block timestamp is deterministic and known ahead of time. This is similar to L1 after proof of stake. During Ethereum proof of work, the timestamp could be chosen by the miner but it had constraints on what it could be. Allowing for this could enable a more flexible design space with blocktime where its not necessarily enshrined into the protocol. We could change the gas limit as defined in the SystemConfig to be the epoch level gas limit rather than the block gas limit. The sequencer could have the right to produce blocks with the same timestamp and choose the timestamp for its blocks but the timestamp must be within a range so that it doesn't fall too far ahead or behind. The block building software now has flexibility with choosing the blocktime now, it becomes a configuration parameter of the sequencer rather than a protocol level config.
Having an epoch level gas limit doesn't play well with when the sequencer loses its connection to L1. That would mean that the sequencer has a liveness failure more more quickly compared to the existing max sequencer drift rules. The epoch gas limit gets used up and the sequencer must observe the next L1 origin to be able to produce more gas. Perhaps a way around this would be to make the epoch level gas limit instead be a timestamp level gas limit and keep the existing timestamp drift rules. With a cap on the max number of blocks that can be produced with the same timestamp, it forces the same progression of timestamps, which can still move forward building on the same L1 origin until the sequencer drift kicks in.
Allowing the sequencer to choose the timestamp means that it can bias RNG. Application developers should already be aware at this point that the timestamp isn't good for creating on chain RNG.

Allowing the sequencer to control the timestamp will mean the timestamp will need to be batch submitted. This could be optimized by submitting a timestamp diff rather than the full timestamp. This would require a new batch version and we would want to keep the semantics of the current batch version the same, where the timestamp auto increments by a hardcoded amount specific to the network

Note that using an epoch level gas limit would mean restricting the total gas for the epoch to something that can be proven in a single cannon/op-program run (ie the ephoch gas limit could not be larger than the maximum block gas limit that could be safely supported).
Not a blocker, just something to factor in to designs to ensure it remains fault provable.

Another consideration here is "lazy block production". It should be possible for a sequencer to build blocks more slowly if they are not seeing a lot of throughput. Decoupling the timestamps helps enable this. To be able to build blocks slower than the base chain, we would need https://github.com/ethereum-optimism/specs/issues/416. The tradeoff with allowing the sequencer to build blocks arbitrarily slowly is that it could delay deposits. There would need to be a rule where blocks can be produced lazily when there are no deposits and then progression is forced when a deposit is observed.
After thinking about this more, I am much more in favor of a timestamp based gas limit than an epoch based gas limit. We could keep the exact same timestamp rules with sequencer drift and have a cap of 4 blocks with the same timestamp and overload the definition of gas limit to mean "gas limit of blocks with same timestamp". This guarantees progression of the timestamp and opens up innovation at the block builder layer, which we are already seeing with rbuilder/rollup boost. Its a complex optimization problem to build blocks that are cheap to batch submit and also earn more fees.
We would also need a way to determine the total ordering of blocks with the same timestamp, to which @protolambda has previously suggested we could overload the block header's nonce field for this.
The op-node could at first implement a naive block building process that updates the timestamp in a very simple way and we can then build blocks faster in the future without needing to introduce any protocol changes.


feature: How to deploy dashboard quickly with APISIX Ingress ?

Having a dashboard can greatly reduce the barrier to use and maintenance.

Having a dashboard can greatly reduce the barrier to use and maintenance.

We may integrate the dashboard for viewing the objects in APISIX first.


Dbeaver issue with Official Java SE 9 release

Hi,
I have installed Java SE 9 (JDK 9) general availability release on my Mac OS. After that latest version of DBeaver (4.2.1) is not working. I am getting java.lang.NoClassDefFoundError: java/sql/SQLException. I have added --add-modules=java.se.ee,java.sql in my dbeaver.ini file and still I am not able to run it. Please let me know the fix.

Looks like Eclipse platform issue for Neon, please see https://bugs.eclipse.org/bugs/show_bug.cgi?id=507417

I am using the latest version of STS, which is based on Eclipse Oxygen. So I dont think so it's neon issue.

The Eclipse Oxygen platform should include the fix of https://bugs.eclipse.org/bugs/show_bug.cgi?id=507417
The DBeaver 4.2.1 is based on Eclipse Neon platform that requires Java 8 JRE/JDK according to https://wiki.eclipse.org/Eclipse/Installation#Eclipse_4.6_.28Neon.29
So, I do not expect DBeaver 4.2.1 to work with Java 9 out-of-the-box.
Now it looks like DBeaver should be adopted for Eclipse Oxygen to resolve the issue.
Thanks for the report!

So, is there any way that I can run current version of dbeaver with java 9? I have both version installed (Java 8 and 9). Please let me know the fix, if I can still use Dbeaver with java 9.

We planning to migrate on Eclipse Oxygen in October.
To run DBeaver with non-default JRE:

dbeaver -vm <path-to-java-binaries>

path-to-java-binaries is where java executables reside.

As framework developer, sometimes Java migration is a little bit awful :/

Yes, it is awful. But it's good that we always have some other way around to deal with issues.

Thank you @serge-rider. I will try with java 8 path.

It works with commond line from terminal. Thanks @serge-rider  @ruspl-afed

Eidt /Applications/DBeaver.app/Contents/Info.plist,
Add somethins like this :

then you don't need to start dbeaver from command line every time

@livelazily  I tried adding this. I am getting this error:

I have also tried adding the vm args in dbeaver.ini file

@harshbhavsar I  don't know why, maybe it trigger something apple's  security checking😕.
I install dbeave by runinng
brew cask install dbeaver-community

maybe you can have a try

@livelazily  It worked with brew cask. I don't know how, but it did. Thanks alot :D

'brew cask install dbeaver-community' does not work for me. Same issue with JDK 9

DBeaver 4.2.2 is bundled with E4.7 which supports Java 9. Check #2247 for details.


Feat!: improve field label behavior

Description
Improve field label behaviour and simplify label color computation.
Until now, TextField was applying conflicting rules to its Label, for instance:

variant: 'red' + invalid when it was invalid
variant: 'subtle' + disabled when it was disabled

Expectations where mislead by this and the result did not match them.
How to test
Check Label, Text, TextField, Textarea and Switch stories
:warning:  Breaking changes
Text
Reworked text variants related to label behavior inside forms:

disabled label
invalid label

Instead of having separate variants, I merged them together inside variant

Added stories.
Label
Added stories to follow Text changes and display some implementations we made in other places.
Changes
TextField
Updated TextField label props accordingly
Textarea
Update Textarea label props accordingly
Switch
I noticed I didn't write stories to show examples of having a Switch with a label.
I wrote 2 stories to follow our mockups:

For example with 2 labels, the behaviour for invalid and disabled states and their compound is a design proposal I made, to try following a similar behaviour to that of TextField. I hope we'll have this validated or changed by a mockup one day :pray:
If you have opinions, I'm up to applying them.
I suppose I could move this part to a separate PR if you prefer. I kept it here simply because I don't add features to the component, but related documentation.

:information_source: Adding no merge as this PR should not be in release 1.0.0-9

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: