GitHub Runner Scaler does not trigger execution if workflow has a sleep statement and waits for the previous execution to complete

Discussed in https://github.com/kedacore/keda/discussions/5604

Originally posted by anandanthony March 14, 2024
We have the following GitHub workflow:
name: GitHub Actions Demo
run-name: ${{ github.actor }} is testing out GitHub Actions 🚀
on: [push]
jobs:
  Explore-GitHub-Actions:
    runs-on: self-hosted
    steps:
      - run: echo "🔎 The name of your branch is ${{ github.ref }} and your repository is ${{ github.repository }}."
      - name: Wait for 60 seconds
        run: sleep 60
      - name: Check out repository code
        uses: actions/checkout@v3
      - name: List files in the repository
        run: |
          ls ${{ github.workspace }}
      - run: echo "🍏 This job's status is ${{ job.status }}."

We have a scaled job that gets triggered in the subsequent polling interval when we the workflow is queued. Though, if we add another workflow when the previous execution on step -sleep 60 then the new workflow does not trigger an execution until the previous one completes. Additonally, we noticed that if we start don't have the sleep and just run additional workflows then executions are created in the subsequent polling intervals correctly.
To isolate the issue, we also created a scaled job with trigger type as azure-queue and could see that the scaled job created executions every polling interval (if there is a message in the queue), irrespective of whether the previous execution is still running or is waiting due to a sleep operation.
Is this behavior very specific to GitHub runners and could it be addressed with settings for the scaled job.
Our scaling strategy is default, with min-max executions as 0-10.

do you see any error in KEDA operator logs? @anandanthony ?
@Eldarrin , have you seen this behaviour previously?

@JorTurFer I checked the KEDA operator logs and could not see any errors in it.

@anandanthony Could you share your scaledjob spec please? I'll take a look and see if I can replicate it on my sandbox.
Cheers, Eld

Attaching the scalejob spec here:
ghrunner-scaledjob-spec-keda.txt
Also, we are using the image for our container from - https://github.com/Azure-Samples/container-apps-ci-cd-runner-tutorial

Good news, managed to reproduce it :) Bad news, not sure why yet. Will investigate but it seems like GitHub itself is reporting ok when I check the APIs.

So the scaled job is able to detect the workflow queued in GitHub, but not able to pick it up and trigger an execution?

Should be fixed now for the next release, it was only detecting on queued and ignoring in_progress jobs

@Eldarrin thank you for looking into this and fixing it. Is there an ETA for the release which would have this fix and if there is workaround till then so that the queued workflows get picked up.

You can check the next planned releases on ROADMAP.md


[question] Lost connection with battery powered devices after docker restart

I am running zwavejs2mqtt on docker (running on Ubuntu) everytime the docker restarts I lose the connection to all battery powered sensors and I need to re-join them to the network - is there a way to avoid this?

Hi, have you given them time to reappear in the network? Also depending on the battery devices. If they are "sleeping" device they can be interviewed only when they are awake and the interview might not end in one wake up period. The frequency of waking up might be configurable on the device so you can make them wake up more often or give them more time to finish the interview.
Also you can share some longs to show if there are errors in communication or simply the interview process is taking too long.

I was seeing this behavior as well but after waiting for a bit (10-20 minutes), the devices appeared normally.  I am running zwavejs2mqtt 1.1.1 and zwavejs 6.2

I am using mostly Fibaro motion sensors and a few Sensitive strips door sensors, it doesn't take 10-20 minutes it takes several hours and some just stay undetected, I looked at the configuration options but didn't find something that seems to me relevant to the wake up interval.
Are you familiar with these modules? Can you point me in the right direction for optimal configuration options for these modules?

Same behavior - just got all of my devices to show up after a week and decided to try a reboot of the docker image, but now all of the battery powered devices that wake once a day are missing (Aeotec Dry Contact, Ecolink PIR Motion Sensors, Aeotec Minimote). Should the default behavior be to cache the device info after discovered?

Usually you have to wait even some hours before them come back

Please be aware that "waking up" isn't always the same as triggering a device.
A motion sensor, for example, will not wake up when it detects motion. You'll have to check the manual, though ususally a "re-associate" action (i.e. press button 3 times) will awaken the device.

@zeddski - this is not very practical I have about 20 motion sensors in different locations in my house - I cannot run around resetting them every time a docker restarts (which happens on power outages or code updates).
The sensative strips modules are great for window\door state detection but they have no buttons so you have to pass a magnet on them to wake up - I can't even start to describe how many hours I spent rubbing these sensors with magnets trying to guess when they are expected to work :-(
I used to run the integrated home-assistant z-wave module and the states of all the modules were kept in the entities file so whenever the network started it took forever to start it but at least I didn't have to waken up anything- the data was still there.
When I restart zwavejs2mqtt - the list of nodes shows me all the nodes but without the data it should have already known about them (which is everything).
I noticed the data is saved into json files - I wonder why the zwavejs2mqtt doesn't try to read the relevant modules from the files.

I moved from Home Assistant's integrated Z-Wave to this and none of my battery operated devices have shown up even after 2 weeks.  Just last night I excluded and secure included one of the door sensors and still nothing came in.  I'm very disappointed this is the path Home Assistant has chosen to go because this is a move backwards because like @unehab said Home Assistant would not lose the manufacturer information on a reboot.  A lot of that data will never change, and the nodes close to it won't change often enough to warrant a re-interview on reboot.  If you need that, flag that node as requiring an interview but have the data cached so it's not useless until some point way down the line where the Docker finally has all the information it needs.

I have the same issue..
After docker restart, some of my battery powered still not available. Shouldn't this being loaded from the cache??

After docker restart, some of my battery powered still not available. Shouldn't this being loaded from the cache??

With version 6.4.0 of zwavejs this should be fixed. Could you try touse dev tag if you are using docker?

@robertsLando I'm running Docker on unRAID and I switched to the dev tag.  After 11 minutes the web UI is still spinning and saying Loading.  Looking at the logs, it's interviewing everything and hit it's max failed interviews on the battery operated nodes.  It's doing partial interviews on the nodes it can reach.  It will log a lot of lines, then stay idle for a while, then log several more.  I'm not sure what it's doing in the idle time when it's not logging, but I've let it run for 14 minutes and it's not yet gone past the Loading screen in the UI.

@robertsLando I haven't tagged the dev branch but I have version 6.4.0 running now at my setup for the last 4 hours and still most of my battery powered sensors are undetected.
I reduced their wakeup interval to 15m (instead of 2h) but it doesn't seem to be very helpful 😞

@AlCalzone ?

Good thing I have this reply saved:
Please make a zwave-js log, loglevel debug or silly and attach it here as a file. Make sure to mention which Node IDs are affected.
If nothing else is going wrong, battery-powered nodes should be immediately available for use after a reboot if they have been interviewed prior. They will still be queried for updated values, but that happens in the background.

running 6.4.0 and still experiencing the behavior in question

Please make a zwave-js log, loglevel debug or silly and attach it here as a file.

On the latest release, everything is scanned immediately from the cache, So currently my issue is solved!
And I don't have to wait anymore for the scan to complete.

Closing for now so


Added characters

Hi!
I added a couple of special characters: ;(),':?!+/=@&€*$"°É£%À<>[]{}~
8 years later, it still is an amazing project, thanks a lot!
For your information, we're using it to make a simple message board with Glitch.com.
Really fun!

🙇 thank you


Please stop bumping PRs twice per day for no reason

Ok, seems that the repo where should I report this. Unfortunately, @jsf-clabot has no contact information — it took time for me to locate this repo. I suggest adding the link to it to @jsf-clabot profile, instead of a generic link to https://js.foundation.
@jsf-clabot bumps https://github.com/webpack/loader-runner/pull/25 about twice per day and pollutes my «Recent activity» feed on GitHub.

That's 344 edits in 150 days, each one bumps the issue in the «Recent activity» feed:

Please make it stop doing that, the actual content of that message does not have to be changed, I have mo idea why it keeps bumping it.

That should definitely not be happening. We will look into this.

In the edit history, GitHub says @jsf-clabot edited on 26 Mar (most recent) — that is not correct. GitHub truncates the edit history after 100 entries, apparently displaying only the first 100 edits.
The full edit history has 344 entries, last of them was yesterday.

It keept doing that. I closed the offending PR because of this.
You can reopen it if you would like after the CLA bot gets fixed. :smiley:


Offering up some monster data

I responded on reddit as well, but I have been working on something similar but I started from the monster data side:
https://github.com/devonjones/PFSRD2-Data/tree/schema1.2
I have tooling that can generate this data off of AoN, and I've been slowly enriching it.  We might find some synergy if we collaborate.

Hi!  Thanks for reaching out!  I was actually considering doing something similar with the SRD repos from the Obsidian TTRPG Community (pre-remaster, remaster).  It's unlikely that I'll get around to it anytime in the near future, but I'd be happy to include compatibility for your schema if I end up taking that on.  You might even have Schema 1.3 ready by then.


Newsletters: add 204 (2022-06-15)

[x] Add lede, update releases/RCs, add topic links @harding
[x] Bitcoin 24171 sync chain from inbound peers @Xekyo
[x] LDK 1505, BDK 628, and BDK 593 @harding

(Sorry for being a bit late this week.)

Added remaining merge summaries, updated RCs, added topic links, and reviewed @Xekyo's commit (thanks!).  Also addressed all review comments, thanks @adamjonas @Xekyo and @glozow!

Thank you @harding @Xekyo @adamjonas @nechteme !

A heads up, we are about to release Wasabi Wallet 2.0, the mainnet release.
WasabiWallet.io
Github.com/zkSNACKs/WalletWasabi
Sorry for the very short notice.


Problem with Horizontal line

Issue:
I don't understand why this code does not work.
I tried to plot a horizontal line at y-value of 5.0, and it doesn't work.
I want to plot my data in two parts: yesterday's data colored blue, and today's data colored black.
ScottPlot Version:
5.0.47
Code Sample:
using ScottPlot;

public class Program
{
    public static void Main()
    {
        var plt = new ScottPlot.Plot();
        plt.ScaleFactor = 2;
        plt.Clear();

        double[] X_scatter = { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 };
        double[] Y_scatter = { 10, 20, 30, 40, 50, 60, 70, 80, 90, 100 };

        // Part 1: Plot yesterday's data, colored blue.
        var yesterdaysX = X_scatter.Take(5).ToArray();
        var yesterdaysY = Y_scatter.Take(5).ToArray();

        var scatter1 = plt.Add.Scatter(yesterdaysX, yesterdaysY);
        scatter1.Color = Colors.Blue;
        scatter1.LineWidth = 0;
        scatter1.MarkerShape = MarkerShape.FilledCircle;
        scatter1.MarkerSize = 5;
        scatter1.Axes.YAxis = plt.Axes.Right;
        plt.Axes.Left.IsVisible = false;

        // Part 2: Plot today's data colored black.
        var todaysX = X_scatter.Skip(5).ToArray();
        var todaysY = Y_scatter.Skip(5).ToArray();

        var scatter2 = plt.Add.Scatter(todaysX, todaysY);
        scatter2.Color = Colors.Black;
        scatter2.LineWidth = 0;
        scatter2.MarkerShape = MarkerShape.FilledCircle;
        scatter2.MarkerSize = 5;
        scatter2.Axes.YAxis = plt.Axes.Right;

        plt.Add.HorizontalLine(5.0, width: 1, color: Colors.Green);

        plt.SavePng("../../../horizontal_line.png", 800, 800);
    }
}

When I run this code, I get:

Notice the left side of the plot is cut off.
Charles

Thanks, that solved the horizontal line problem.
I'm still concerned that the left size of the plot is cut off.


[VS Code] Request was cancelled

Error:
Request was cancelled
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.735

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.27.2

platformName
MacOS

platformVersion
17.7.0

Closing as a dupe of 469.


Cosmetic bug : layer button is placed over search bar

Seen this on my S10

Strange... I will investigate.

This seems to be caused by the higher-than-usual status bar due to the camera cutout in the display. I fixed it now, but only remembered to do so after releasing an update... so this will have to wait for the next one.

This seems to be caused by the higher-than-usual status bar due to the camera cutout in the display. I fixed it now, but only remembered to do so after releasing an update... so this will have to wait for the next one.

So maybe a specific Samsung problem because for the camera .
Thanks for fixing 😊

Working now 👌


clean harbor containers in CI

Coverage decreased (-0.008%) to 54.82% when pulling 79e6d3c3ef7d575d202719816bace0598e36a420 on wy65701436:add-clean-ci into 89ee3c784888e07cee063765f1beec3adf47cbcb on vmware:master.


Hang on merge conflicts

VS Code Version:

Version: 1.71.0
Commit: 784b0177c56c607789f9638da7b6bf3230d47a8c
Date: 2022-09-01T07:36:10.600Z
Electron: 19.0.12
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22000
Sandboxed: No

OS Version:
Windows 11.

Steps to Reproduce:

Open and try to merge conflicts. Only hang for this specific one:

Not repro in Visual Studio:

Conflict
Incoming:
https://github.com/xiaomi7732/ApplicationInsights-Kubernetes/blob/f00d06cc228dace35adc1f2f7c806361a292fa5e/src/ApplicationInsights.Kubernetes/Extensions/KubernetesServiceCollectionBuilder.cs
Current:
https://github.com/microsoft/ApplicationInsights-Kubernetes/blob/faf5ef526fa432caab6926b336aaffc6b299a5d4/src/ApplicationInsights.Kubernetes/Extensions/KubernetesServiceCollectionBuilder.cs

Hi there, it has been a while, is there any update to this?
For people ran into the same problem, a work around is to disable the new merge editor:
"git.mergeEditor": false

Thanks for the bug report! Can you share the base version of that conflict?

Sorry, what do you mean by the base version of the conflict? I am trying to merge Incoming into Current (both provided), what's the difference between base and the current? Thanks.

Base is the common ancestor file of both commits.

I see. I am not 100% sure this is accurate, but this is my best bet for the base:
https://github.com/xiaomi7732/ApplicationInsights-Kubernetes/blob/6b19fd5f414dbc8653a3b8ab445c6576d186575f/src/ApplicationInsights.Kubernetes/Extensions/KubernetesServiceCollectionBuilder.cs#L1


Errortests

Implemented HTTP Exception Handling test cases for Error Codes 400, 401, 403, 404, 409, 500 and 503 in following scenarios .

Catalog(GET)
Schema(GET) - NonHTTP - throws NotFoundException
Table(GET) - NonHTTP - throws NotFoundException
Entity(GET, PUT, POST, DELETE)
Attributes(GET)
Aggregate(GET)

The test-cases mock HTTP requests to simulate respective error scenarios.

@karlcz @robes Can you guys take a look at the test scenarios?

@chiragsanghvi it reports having a conflict. Do you have to resolve the conflict first?

@hongsudt  There have been commits in master, so I will need to resolve them and push my changes.

In the future, it would be better to keep distinct changes in different branches/PRs. This PR says it is about 'Errortests' but includes changes to the packaging and installer, which don't seem directly relevant to the primary purpose of this change.

This looks good to me. You could merge it any time, in my opinion.


Drop support for outdated PHP versions

As of today (September 2019)  PHP versions  up to 7.0 don't get support any longer.
I suggest that we drop support for older PHP versions as well (maybe allowing to be one major release behind).
Migrating to PHP 7+ would allow us use great features like strict mode, argument and return type hinting and many more.

As this breaks backwards compatibility a new major release (2.0) would be required.


correct_colours acts strangely in mouth area

I've noticed that the correct_colours routine is misbehaving in the mouth area. Here's an example with Kanye West’s body and a replacement face image:

You will get this output, where the lips appear white:

This shows just the transformed output by setting the output of get_face_mask() to all white:

And here is what the faceswap looks like if the correct_color step is skipped:

Here is a second example with Gwyneth Paltrow's body and a replacement face.

The colour correction this time causes a crazy bloody mouth look:

Turning off colour correction fixes this:

I might look into this more and be able to offer a pull request to tweak the correct_colours() routine to fix this, but wanted to go ahead and report the issue.

Hi, @dribnet,
I face the same problem with you, have you work out any solution for this?  Thank you!

I have not - my only solution so far has been to simply turn off colour correction - but I would be still be interested if anyone had time to propose better colour correction routine. One thought could be to make a new mask for the color routine to limit it to the boundary region between the two images.


Kernel crash with stacktrace

I just had the kernel crash on OPi5 Plus.
I booted, edited flake.nix with Vim and ran nixos-rebuild --flake ., and after a minute or two the kernel crashed with this log. Should this be reported in armbian?
I had the kernel spam as in #37 and disabled it with dmesg -n 1 at boot. The log shows spam getting through after the stacktrace, maybe related or not — I don't know.
I'm using the current master head b95bb58af82e65af8f2ecdec829f74143958a49c.
Ivan

My understanding of the Linux kernel is not deep, and I can't quite figure out the cause of your issue either.
However, I just merged #60, which includes updates to the kernel and nixpkgs with numerous improvements. You can try updating your flake inputs to see if it helps.


fixes #2265 : adds scroll to preferences

Description
Changed preferences.xml as there is no scroll view in it.
Related issues and discussion
fixes #2265
Screen-shots, if any
Checklist
Make sure you've done all the following (You can delete the checklist before submitting)

[x] Apply the AndroidStyle.xml style template to your code in Android Studio.
[x] Code is well documented
[x] Include unit tests for new functionality
[x] All user-visible strings are made translatable
[ ] Code passes Travis builds in your branch
[ ] If you have multiple commits please combine them into one commit by squashing them.
[x] Read and understood the contribution guidelines .

Do you have a video of this working?

Do you have a video of this working?

Do you have a video of this working?

@Karljoones I think there is alredy a scroll view involved in it..But it is not visible

No change. It is same as before.
Also create a new branch for opening a PR.

Going to close this off because there were no changes in this.


Is Windows Forms still a first-class citizen in the ReactiveUI framework?

I unfortunately cannot use WPF because of "reasons" and have to stick with Windows Forms, but would like to benefit from a framework like ReactiveUI.
I cannot find any good examples of usage with Windows Forms and commits in the main repository are all about mobile stuff, so I'm not sure ReactiveUI is the right tool for me - I'm wishing it is.
Any advice?

Yes it is

Hey @AidanCondon Windows.Forms is still first-class. You're not going to see much activity on the Windows.Forms front as it's pretty rare to see greenfield Windows.Forms in my experience. However we would love to take in anything you can contribute, documentation or sample apps.

Thank you @paulcbetts and @ghuntley. I don't really know how to use ReactiveUI with Windows Forms just yet to contribute samples... I was actually trying to find some good ones for reference so I can decide to use RXUI or not with my project.

Thanks @gardnerbp !

Thanks @gardnerbp !


Cannot execute command graphql-codegen init

Describe the bug
Cannot run graphql-codegen init due to an error on Windows 10.
To Reproduce
Steps to reproduce the behavior:

open a PowerShell prompt
install with yarn global add @graphql-codegen/cli
run graphql-codegen init

Expected behavior
The command to execute correctly ...
Environment:

OS: Windows 10
@graphql-codegen/...: graphql-codegen/cli
NodeJS: v10.9.0

Additional context
Error with powershell:
At C:\Users\adrie\AppData\Local\Yarn\bin\graphql-codegen.ps1:5 char:13
+     *CYGWIN*) basedir=`cygpath -w "$basedir"`;;
+             ~
Unexpected token ')' in expression or statement.
At C:\Users\adrie\AppData\Local\Yarn\bin\graphql-codegen.ps1:8 char:3
+ if [ -x "$basedir/pwsh" ]; then
+   ~
Missing '(' after 'if' in if statement.
At C:\Users\adrie\AppData\Local\Yarn\bin\graphql-codegen.ps1:8 char:5
+ if [ -x "$basedir/pwsh" ]; then
+     ~
Missing type name after '['.
At C:\Users\adrie\AppData\Local\Yarn\bin\graphql-codegen.ps1:9 char:20
+ ... edir/pwsh"  "$basedir/../Data/global/node_modules/.bin/graphql-codege ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Unexpected token '"$basedir/../Data/global/node_modules/.bin/graphql-codegen.ps1"' in expression or statement.
At C:\Users\adrie\AppData\Local\Yarn\bin\graphql-codegen.ps1:9 char:84
+ ...  "$basedir/../Data/global/node_modules/.bin/graphql-codegen.ps1" "$@"
+                                                                      ~~~~
Unexpected token '"$@"' in expression or statement.
    + CategoryInfo          : ParserError: (:) [], ParseException
    + FullyQualifiedErrorId : UnexpectedToken

Error with powershell core 6:
At C:\Users\adrie\AppData\Local\Yarn\bin\graphql-codegen.ps1:4 char:6
+ case `uname` in
+      ~~
The Unicode escape sequence is not valid. A valid sequence is `u{ followed by one to six hex digits and a closing '}'.
At C:\Users\adrie\AppData\Local\Yarn\bin\graphql-codegen.ps1:5 char:13
+     *CYGWIN*) basedir=`cygpath -w "$basedir"`;;
+             ~
Unexpected token ')' in expression or statement.
At C:\Users\adrie\AppData\Local\Yarn\bin\graphql-codegen.ps1:8 char:3
+ if [ -x "$basedir/pwsh" ]; then
+   ~
Missing '(' after 'if' in if statement.
At C:\Users\adrie\AppData\Local\Yarn\bin\graphql-codegen.ps1:8 char:5
+ if [ -x "$basedir/pwsh" ]; then
+     ~
Missing type name after '['.
At C:\Users\adrie\AppData\Local\Yarn\bin\graphql-codegen.ps1:9 char:20
+ ... edir/pwsh"  "$basedir/../Data/global/node_modules/.bin/graphql-codege ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Unexpected token '"$basedir/../Data/global/node_modules/.bin/graphql-codegen.ps1"' in expression or statement.
At C:\Users\adrie\AppData\Local\Yarn\bin\graphql-codegen.ps1:9 char:84
+ ...  "$basedir/../Data/global/node_modules/.bin/graphql-codegen.ps1" "$@"
+                                                                      ~~~~
Unexpected token '"$@"' in expression or statement.
+ CategoryInfo          : ParserError: (:) [], ParseException
+ FullyQualifiedErrorId : InvalidUnicodeEscapeSequence

I found a "workaround". By using graphql-codegen.cmd init it works (using the cmd script instead of the ps1 script).

@pellea the .ps script is created by Yarn, so I'm not sure it's related to the codegen (maybe we missed something?).
Anyway, I don't have Windows environment, so it might be difficult for me for to fix that soon :(

@pellea It seems like there are some issue with Yarn and PowerShell.
Can you please try to take a look at this issue?
https://github.com/yarnpkg/yarn/issues/6902
I think they already solved it in the latest version.

The graphql-codegen init command is working with Yarn 1.15.2. Thx for pointing out the other issue.

I'd recommend using WSL to avoid issues like this.

npx  graphql-codegen init


Fix testKeyBasics and testEverything

Some crypto backends (e.g. CNG and OpenSSL with SymCrypt) fail to create RSA keys that are smaller than 512 bits. Some tests use keys lower than 512 bits, so we have to skip them. We were previously doing so for the CNG backend. This PR conditionally skips the failing tests when using any backend.
We lose a bit of coverage, as the OpenSSL backend do support small keys when using the built-in providers. There is no easy way to know which key sizes are supported by a given OpenSSL provider, and our implementation doesn't depend on the key size, so I think is makes sense to just skip the test rather than trying to be too clever.
⚠ Don't merge until #1384 has been merged. ⚠

@qmuntal - merge conflicts

@qmuntal - merge conflicts

Fixed.


[Feature Request] Programmatically set validation field to invalid state

Existing Component
Yes
Component Name
el-form
Description
At the moment there is no way to set validation field to invalid state programmatically. We could trigger 'validateField' method for specific field, but it will only trigger existing validation rules.
It would be nice if we had some method that accepts desired field to validate and error message as string (e.g. form.invalidField(field, 'Some error message')).
Currently as a workaround its possible to do this hack that achieves desired effect:
const falsyValidator = {
    validator: (rule, value, callback) => callback(new Error(errorMessage)),
    trigger: 'manual'
  };
  validationRules[field].push(falsyValidator);
  form.validateField(field, () => validationRules[field].splice(-1, 1));

I'm adding 'falsy' validator that will always return error to validation rules for specified field, than we need to trigger validateField method and afterwards remove this validator. As a result user will see error message and invalid field on UI.

You can set FormItem error attribute. https://jsfiddle.net/04dmf85w/ It works for you.
Right now, we have no plan to support this.


Short solution needed: "Backup a table" (clickhouse)

Please help us write most modern and shortest code solution for this issue:
Backup a table (technology: clickhouse)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

Backup single table using TSV dump


⚠ Jenkins Plugins Artifactory has degraded performance

In 8d5a063, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 2518 ms

Resolved: Jenkins Plugins Artifactory performance has improved in c69e36f after 20 minutes.


Fixes F.affine and F.rotate to support rectangular tensor images

Description:
Currently, F.affine does not work correctly on rectangular tensor images due to normalized output of affine_grid.
In this PR there is an attempt to fix the issue. Code is not nice due to dispatch in two part of implementations.

[x] Added tests on square and rectangular images for F.affine and F.rotate

@fmassa PR is ready to review. What is done:

F_t.affine and F_t.rotate uses common method _gen_affine_grid to generate a grid which supports square and rectangular images (due normalization issue with torch native affine_grid).
Code is tested on square/rect images for F.affine and F.rotate
_gen_affine_grid mimicks affine_grid C++ implementation and optimized such that on CPU it is almost same execution time as for affine_grid.


[WIP] Added flannel backend config flag

Proposed Changes

Create a flag to add additional flag to the flannel backend configuration using the one defined here https://github.com/flannel-io/flannel/blob/master/Documentation/backends.md
flannel-backend-config is added on the server configuration and the same config is inherited by all the agents. With the current implementation to configure additional flag to the backend it's required to manually create a net-conf.json file and pass it with flannel-conf to all the K3s instances of the cluster.
Types of Changes

New Feature
Verification

flannel-backend-config flag is added on the server configuration.
Testing

A sample config could be as follow
flannel-backend: vxlan
write-kubeconfig-mode: 644
flannel-backend-config:
  - MTU=1200
  - Port=8477

With this configuration Flannel will use vxlan as backend and will configure the port and the MTU accordingly.
Linked Issues

User-Facing Change

Further Comments

How does this integrate with https://github.com/k3s-io/k3s/blob/master/docs/adrs/flannel-options.md
We explicitly don't want to add more flannel flags.

It's not. The flannel options flag is only for the boolean flag as far I understood. Or we can use the type=option1=value,option2=value implementation on flannel-backend but I don't like it much.

@dereknola and I decided that we don't need an additional flag. The flannel backend configuration could be done using a yaml file with --flannel-conf


TANH/Sigmoid 16-bit activation functions using LUT

We think the reference functions for 16-bit activation are too complex for efficient implementation on resource constrained platforms and propose to replace the functions with a lookup table approach as follows:
First rescale the input data to fixed range of -10.7 to +10.7. Then use a 256-entry lookup table for Sigmoid followed by linear interpolation to efficiently derive the result.
The Sigmoid LUT table is used for the TANH function, because tanh(x) = 2sigmoid(2x) -1 and we take into account the symmetry.
The proposed reference kernel implementation also has higher accuracy than the existing one.
On the current functions we measure a difference of up to 6.3 for sigmoid and 11.7 for tanh in quantized units compared to the floating point reference implementation over the 16-bit input range (representing -8.0 to +8.0). For the implementation of this patch we see the error reduced to less than 1.5 quantized units compared to floating point reference for both tanh and sigmoid.

can you share benchmark results showing how this compares to the current 16bit fixedpoint implementations on various ARM CPUs?
because pure arithmetic implementations scale linearly with SIMD widths and lookup tables do not, we have typically found that despite looking expensive in source code, pure arithmetic implemenations can actually perform well. As ARM CPUs SIMD width increases, they keep looking better. For example, on a Cortex-A76 CPU (e.g. Pixel4) we can issue 2 128-bit multiplications each cycle, each doing 8 16-bit fixed-point multiplications, so each fixed-point multiplication effectively only costs one sixteenth of a cycle.
as this varies greatly from one CPU to another, we don't expect one approach to consistently outperform the other (arithmetic vs LUT).  Instead, we have started to let both coexist in source code.  We still need to pick one to be used by default but at least having both in source code makes it easy to switch.
i don't remember the latest but some the the following people might: @renjie-liu @abattery @jianlijianli

Thanks Elena for the pr. Like Benoit pointed out, lut is not necessarily desired from the performance point of view.
Another thing is I suspect the accuracy measurement is "correct", do you have any real model to benchmark the accuracy? Like you said, your method will use range from -10.7 to 10.7, but should that based on the real scenario?
Jaesung can probably comment more, but if you want to cover more range for your scenario, probably just add one more bits to the integer parts? (where currently it's 3 bits, so represents -8 to 8) for sigmoid?

Hi, Elena.
Thank you for your contributions :)
For the performance, I have seen that LUT-table methods for Tanh/Sigmoid are practically faster than fixed point arithmetic based calculations in the modern mobile ARM CPUs for integer 8 bit cases. However, it's worth double-checking again for the integer 16 bit cases. Could you share the benchmark results before and after?
It seems that you are trying to increase accuracy based on the idea that it takes an absolute value since tanh function is symmetric and does LUT where the table covers little larger than 8-bit fixed point arithmetic range. In my first looking at it, it makes sense to me. However, As Renjie said, the Sigmoid case has the same expression range [-8, 8] with the current fixed point arithmetic kernels for Sigmoid. I am not sure that this approach improves Sigmoid case. Could you share your accuracy difference results for both Sigmoid and Tanh to us and how did you measure the accuracy?
Best regards,
Jaesung

Cache aliasing also plays key role in the performance of LUT and that is really hardware dependent. Seems to be the best option is for them to "coexist", like Benoit mentioned in his comment.

Hi guys,
Thank you for the review @bjacob @abattery @jianlijianli @renjie-liu  !
I attached the file accuracy_test.cpp that demonstrates how the accuracy was tested:
https://gist.github.com/wwwind/0b7413833cf8a7596bee084403224fa5
The test code reports the range and total sum of the error over the whole signed 16-bit integer range. Below are results of the testing.
For sigmoid:
tensorflow   sigmoid implementation error -6.392090 to 5.392578, error sum -32767.042969
proposed     sigmoid implementation error -1.015625 to 1.015625, error sum -0.043635
For tanh:
tensorflow   tanh    implementation error -11.791016 to 11.791016, error sum 0.992188
proposed     tanh    implementation error -1.476562 to 1.476562, error sum 0.992188
Regarding the performance benchmark, we do not have such measurements at hand. When we refer to resource constrained platforms, apart from the world of CPUs, MCUs and GPUs, we also consider neural network accelerators, the dedicated HW where small look-up table is likely to be a preferred implementation path. However, even just for CPUs, for the non-optimised reference code, we expect the proposed solution to be more efficient than the gemmlowp one due to the number of operations involved in the iterative approach used by the latter.
In the proposed solution, to compute a value, we need about 14 operations. The existing implementation is based on the iterative Newton-Raphson division algorithm to find a reciprocal, which takes around 12 operations + computation of exponent, which needs Taylor approximations with 4 terms. So, it looks a lot heavier, at least for a single result computation.
We agree that, for vectorised implementation, the performance advantage may or may not shift towards the iterative approach. Therefore, it seems that the approach proposed by @bjacob, @abattery and @jianlijianli , where both implementations will co-exist, is a sensible compromise. I would suggest that we keep a more accurate, LUT-based implementation, as a default one. Should I proceed to make the corresponding changes in this PR? What would be the best way to implement the switching mechanism between the two implementations? Could you please point to an example in the code ?
Thanks!

Hi,
thanks a lot for the testing! and thanks a lot for the work!
I wonder should we sum the absolute value instead of the raw value? also the large sigmoid value makes me wonder it looks really look like a direct overflow: -32767.042969 really looks like -32768 plus some number (maybe we can just try with the extreme number?) or maybe just systematically shifted -0.5 for every value?
also given you're verifying the int16 value, judging from the number both have good results: +- 12/(1 << 15) for tanh
I think it's fine to keep both approach, you can refer here for prepare: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/kernels/activations.cc#L395-L422
and here for eval: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/kernels/activations.cc#L813-L824

@wwwind Could you please check renjie-liu's comments and keep us posted. Thanks!

Hi @renjie-liu,
Thanks a lot for your review. I pushed an updated version. Could you please take a look ?
Also, I think that this block is not relevant anymore for LUT approach https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/kernels/activations.cc#L515 Should it be removed ?
Thanks,
Elena

@wwwind Could you please check reviewer comments and keep us posted. Thanks!

@renjie-liu, thanks for the review, while Elena is away, I replied where I could. Please, let us know what you think.

@renjie-liu, @jdduke, following our off-line discussion, can we try to move this PR forward? As the proposed LUT-based INT16 implementation improves the accuracy, it seems reasonable to keep it for both kReference and kGenericOptimized paths. It would also make INT16 path in the corresponding Eval and Prepare calls symmetric to the current INT8 and UINT8 ones.

Hi @jianlijianli I moved implementation of Tanh/Sigmoid into integer_reference_ops per discussion. Could you please review this PR ?
Thanks!

Hi @jianlijianli , Could you please re-approve this PR ? I had to push a fix for the warning.
Thanks!

Hi @gbaned, @liufengdb  ! I pushed a small fix for the error reported by buildifier.
Could you please re-approve this PR ?
Thanks!

Hi @rthadur ! This PR has been approved 10 days ago and all checks are green.
Is it stuck internally with some errors ? Could you please take a look?
Thanks!

@wwwind thanks for your patience , @jianlijianli is working internally on this , we will get back to you asap.


Fix command format in tutorial

Closes: #8788

/lgtm

Hi @tengqm -- almost, but this still isn't right for continuing numbered lists. Sigh. There's a PR against the smoke test page here that explains what to do, #8622 if you want to read everything. TL;DR:

Numbered lists can have paragraphs or block elements within them.
 Just indent the content to be one tab stop beyond the text of the bullet
 point. **This paragraph and the code block line up with the `m` in
`Numbered` above.**

(note from Jennifer: actually, they line up with the N, but I'm quoting from the PR)
If you could fix one more time, that would be great. Thanks!

@Bradamant3 Thanks for the feedback. Will give it another stab.

@Bradamant3 Ready for review now. The hack is ugly.

@tengqm unfortunately, it's not a hack. It's the result of the very loose markdown spec and the differences between Kramdown, which we've been used to with GitHub and Jekyll, and BlackFriday, which Hugo uses. It doesn't look as though anything will change, either, unless a large and effective group puts pressure on Hugo. BlackFriday maintainers seem uninterested in these issues. (I did some digging into the issues that Misty links to from the PR cited in comments ^^)
Thanks for your patience -- I guess we all have to get used to The New Way(s) of Writing Markdown now.
/lgtm
/approve


V0.13.0/documentation

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@adcbd64). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #73   +/-   ##
=========================================
  Coverage          ?   35.13%
=========================================
  Files             ?       68
  Lines             ?    10146
  Branches          ?      787
=========================================
  Hits              ?     3565
  Misses            ?     5794
  Partials          ?      787

Continue to review full report at Codecov.

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


Practice: Classes and Collections: Android Basics in Compose

Class is great! Here's a MINOR issue. In the practice for classes and collections, task 2 says to define the enum Daypart as having NIGHT. In task 3, one of the events assigns daypart as Daypart.EVENING. Again, thanks for these great classes.

Hello @williamFerrante,
We appreciate your feedback and the issue report. The typo in the Task 2 is fixed:


Docker image?

Hey, thanks for working on this!
Is there a public docker image available?
The readme includes 'aaw3/ytdl-server:latest' but this does not seem to be public.

Never mind, I realized I'd need to update the Dockerfile to work with my caprover setup, so I just forked the repo and had caprover build the image from the repo.

Yeah sorry for any confusion, the image aaw3/ytdl-server:latest must be built locally. I currently do not have any images posted to docker hub.


NullPointerException in MouseMoveToLocation.java

Hi all,
I am on 2.45.0 and trying to use mouseMove(element) following the instructions. However, I am getting this error:
16:11:32.331 INFO - Executing: [mousemove: unused false])
16:11:32.331 WARN - Exception thrown
java.lang.NullPointerException
at org.openqa.selenium.remote.server.handler.interactions.MouseMoveToLocation.call(MouseMoveToLocation.java:52)
at org.openqa.selenium.remote.server.handler.interactions.MouseMoveToLocation.call(MouseMoveToLocation.java:1)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at org.openqa.selenium.remote.server.DefaultSession$1.run(DefaultSession.java:168)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)
16:11:32.332 WARN - Exception: null
I see that in :
https://github.com/SeleniumHQ/selenium/blob/master/javascript/node/selenium-webdriver/CHANGES.md
it says "FIXED: Correct serialize target elements used in an action sequence." for 2.46.0-dev. I am wondering if there is the same issue? If so, can you point me to the issue for this? I am also wondering what's the milestone for 2.46.0 release in that case?
Thanks a ton!

That sounds like the issue you mentioned. I don't know what the timeframe for 2.46.0 is, but we will update as soon as it is available!

selenium-webdriver 2.46.1 is released two weeks ago and it seems to fix this issue.

Thanks for the update! Will try to verify

Should be fixed in 2.3.0 and higher.


Selected column header issue

Hi,
I am facing a issue as mentioned in the attached docx. Please go through the docx as I have explained in detailed step with images.
Use Case :

Navigate through context browser till the last assest(px-context-browser)
Now click on open button as seen in an above Image. After open is clicked there will be px-modal that contains px-data-table(selectable and single-select property is true)  (selected header is as Selected(0))
Now select stage1 is selected  (Stage1 is highlighted and selected header is changed to Selected(1))
Now I click on Ok and again navigate through context browser and select another asset and follow step 2 again
(Now when the user has clicked on the open button of another asset, I have cleared the table-data value and assign all-together new data then also the selected header is shown as Selected(1)  as seen below.

Expected Behaviour : Selected header should be reset or since it is single select there shouldn't be a number indicating how many items are selected.
Actual Behaviour : Selected header remains as Selected(1) even after table data is newly assigned
(I believe this is happening since only the table data is getting changed and the headers don't change when this happens so.)
px-data-table issue docx.docx

Duplicate of #80 and #51 - see discussion and possible workaround there.


The hasgroups claim is present when Group Claims are missing

In this related issue it's mentioned that by design the groups being configured here will not be emitted when a user belongs to more than a few groups.
Documentation should describe the hasgroups claim which is emitted instead in this scenario and describe the recommended path of make a separate call to the Graph API to query for the user's group membership.

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

ID: 1d8e93d6-413b-34aa-3359-41f5baa442d3
Version Independent ID: 0d657318-8058-a76a-c219-393ab3a4920d
Content: Learn how to provide optional claims to your Azure AD app - Microsoft identity platform
Content Source: articles/active-directory/develop/active-directory-optional-claims.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

Related issues

https://github.com/MicrosoftDocs/azure-docs/issues/42370
https://github.com/MicrosoftDocs/azure-docs/issues/42371

@KSchlobohm Thanks for your feedback! We will investigate and update as appropriate.

Reassigning to the article author @billmath  #reassign:billmath

I have already created a work item for this issue.  This is a duplicate.
#please-close


[Enhancement] optimize memcpy for large json

What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes:

Fixes #
Problem Summary(Required):

Optimize the copy into move for CSV loading
Optimize the copy into move for TableSink

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

@Mergifyio rebase

@Mergifyio rebase


[stdlib] Add overload to String(describing:) for metatypes

Currently, String(describing:) goes through a very slow path to eventually get the textual representation of Any.Type. String interpolation on the other hand has an overload that calls directly into _typeName to make "\(someType)" faster than String(describing: someType).
Resolves: rdar://71192147

@swift-ci please smoke test

Should we add a comment (or something) to clarify why we've added this overload?


Bundled libjpeg-turbo linkage seems to be hard-coded

I get this failure when trying to build on MSYS2 UCRT64:
ninja: error: 'C:/msys64/home/kmilos/MINGW-packages/mingw-w64-libultrahdr/src/libultrahdr-1.0.0/third_party/build/libjpeg-turbo/src/libjpeg-turbo-build/libjpeg.a', needed by 'ultrahdr_app.exe', missing and no known rule to make it

Sorry, I see now things might be improved on main, while I was trying to package the 1.0.0 release...
Please tag and release these improvements when ready.

On latest main this issue should not be seen. we are working on few more changes that further improve build and install process on different platforms. We will publish them in the earliest. For now I am closing this bug.


video size is wrong

This is a good project.  ID consistency is great. Quality wise this project is much better than EchoMimic or MimicMotion, etc. And it‘s faster also. Just noticed a small issue. The case-1 images and related  ref image all have size 512x512, but the generated gif has size 576x1024 so the figure is distorted due to wrong aspect ratio. After resizing back to 512x512, the gif looks nice.
Thanks for the fantastic job！

Hi, you can modify --width and --height parameters  in command_basic_infer.sh. StableAnimator currently supports image animation in two resolutions: 512×512 and 576×1024. For more details, please refer to the Model Inference section.


[DOC] Documentation for builtin mice function

This pull request adds Documentation for builtin mice function.

Thank for working on this @stanp7.
In your local repository (in PowerShell):

Checkout master branch: git checkout master
Create a new branch from master: git checkout -b mice
Pick only the commits related to mice function: git cherry-pick dde6a84319468869d902da4818ef1cb059c3a1eb
Now check the mice function is cleanly added: git log
Push changes to stanp7:mice-doc : git push --set-upstream-to=p****/mice-doc


[Addon] Add EnvoyProxy Gateway as a experimental addon

Envoy Gateway manages Envoy Proxy as a standalone or Kubernetes-based application gateway.

Currently EG does not add support to helm chart, but provided yaml to install. In this case, how can I add it as an addon?
cc @wonderflow : )

@Xunzhuo Good job, I already find your pull request. Helm chart is not necessary, KubeVela support various kinds of artifacts.

Thank you! @wonderflow I am figuring out how a extension is embedding and running in vela, and the behaviors behind vela addon enable.
Any source codes explanations?
I have a quick view and find that vela converts the extensions into Application CR, and add yamls to k8s-objects.

Yes, addon will be converted into Application CR and run. The main function is here: https://github.com/kubevela/kubevela/blob/master/pkg/addon/addon.go#L911


Collapsing nav

Summary
This PR brings in 2 changes:

Add accordion-like "side" nav for mobile-sized about pages

The side nav we have currently looks pretty good on a desktop browser, but on mobile it is basically just in the way

Prevent visually-hidden text from being selectable

Test instructions

Open up the Heroku PR app
Click any of the about pages
Resize your browser

gif
Here's how it looks.

Hey @amazingphilippe, I've tried with two different colours and it's hard to choose between them.

lighter
darker

Hey @amazingphilippe, I've tried with two different colours and it's hard to choose between them.

lighter
darker

Ah! We could flip a coin!
I lean towards the darker because it feels more noticeable. And the ultra light shades of gray sometimes don’t display well on some poorly calibrated screens.
Is it me or does the blue also change? We should check if they at least have 7:1 contrast on hover.
Also wanted to ask if the menu toggle card for the actual nav or only the sidebar?
Philippe Caron
Canadian Digital Service (CDS) | Service numérique canadien (SNC)
Treasury Board of Canada Secretariat | Sécretariat du Conseil du Trésor du Canada
<EMAIL_ADDRESS>|<PHONE_NUMBER>
N'hésitez pas à répondre dans la langue officielle de votre choix.
Please do not hesitate to reply in the official language of your choice.

Le 4 janv. 2021 à 10:58, Paul Craig<EMAIL_ADDRESS>a écrit :
Hey @amazingphilippe https://github.com/amazingphilippe, I've tried with two different colours and it's hard to choose between them.
lighter	darker
https://user-images.githubusercontent.com/2454380/103553578-a15b6f00-4e7b-11eb-98ea-95d132cc2fd0.gif https://user-images.githubusercontent.com/2454380/103553590-a5878c80-4e7b-11eb-949a-47aa2994bdd3.gif
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/cds-snc/covid-alert-portal/pull/348#issuecomment-754058129, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABH5BAG34RLUMW6DO2KAMCTSYHQRPANCNFSM4VPX7UDQ.

Ah! We could flip a coin!
I lean towards the darker because it feels more noticeable. And the ultra light shades of gray sometimes don’t display well on some poorly calibrated screens.
Is it me or does the blue also change? We should check if they at least have 7:1 contrast on hover.
Also wanted to ask if the menu toggle card for the actual nav or only the sidebar?
Philippe Caron
Canadian Digital Service (CDS) | Service numérique canadien (SNC)
Treasury Board of Canada Secretariat | Sécretariat du Conseil du Trésor du Canada
<EMAIL_ADDRESS>|<PHONE_NUMBER>
N'hésitez pas à répondre dans la langue officielle de votre choix.
Please do not hesitate to reply in the official language of your choice.

Le 4 janv. 2021 à 10:58, Paul Craig<EMAIL_ADDRESS>a écrit :
Hey @amazingphilippe https://github.com/amazingphilippe, I've tried with two different colours and it's hard to choose between them.
lighter	darker
https://user-images.githubusercontent.com/2454380/103553578-a15b6f00-4e7b-11eb-98ea-95d132cc2fd0.gif https://user-images.githubusercontent.com/2454380/103553590-a5878c80-4e7b-11eb-949a-47aa2994bdd3.gif
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/cds-snc/covid-alert-portal/pull/348#issuecomment-754058129, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABH5BAG34RLUMW6DO2KAMCTSYHQRPANCNFSM4VPX7UDQ.

Okay! Updated to use "About the Portal" instead of "Topics", so the french translation is already there.

Okay! Updated to use "About the Portal" instead of "Topics", so the french translation is already there.

LGTM as well. For archival purposes :)

LGTM as well. For archival purposes :)


[FR] Python 3.10 Support

So far as I know, only Python 3.10 wheels for fiftyone-brain must be added. Barring any other dependencies missing support, 3.10 support will be added in the next release.

Resolved with fiftyone-brain==0.8.2


Add Impl for UVecs?

Per title. Happy to do it if you are good with a PR.
Cheers,
--Joseph

A PR would be appreciated. You can probably just do it like the Vec4 impl here: https://github.com/jakobhellermann/bevy-inspector-egui/blob/2a7a0c1f64fd311955d14b8384a42738d38debb8/src/impls/vec.rs#L168

Done: https://github.com/jakobhellermann/bevy-inspector-egui/commit/4c2e505d350a61f69c07d3e928a04fa48a865af4


Request: Soulreaper axe

I do not know how to find animation ID.

I apologize, it is apparently only the crush attack style not working which has limited uses.


[FLINK-13819][coordination] Introduce State for RpcEndpoint

What is the purpose of the change
To better reflect the lifecycle of RpcEndpoint, we can introduce its running state.
We can use the non-running state e.g. to make decision about how to react on API calls if it is already known that the RpcEndpoint is terminating.
Brief change log

Introduce RpcEndpoint.isRunning flag and getter
Introduce non-overridable internalCallOnStart and internalCallOnStop methods in RpcEndpoint to set the running flag
add unit tests

Verifying this change
run RpcEndpointTest
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

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

Thanks for the reviews @tillrohrmann @TisonKun
I have updated the PR

Thanks @tillrohrmann @TisonKun
Merging the PR when the Travis is green

merged into master by aabde88e5280d9fef9c911265e270ebb101b58e4


Fix ...IndexOutOfBoundsException if device don't have orientation sensor

What steps will reproduce the problem?

Run app on Lenovo S820
Get java.lang.IndexOutOfBoundsException: Invalid index 0, size is 0 in 260 line MainActivity.java:
orientSensor = sensorManager.getSensorList(Sensor.TYPE_ORIENTATION).get(0);
This occurs because the list does not include items that are returned getSensorList.

Good catch, thanks!


Popover in Materialize

Does Popover feature is available on Materialize? I cannot find anything related on the documentation.

There is no such component, cause its not a part of Material Design. Use modals and tooltips instead. You may also made some custom tooltip for your project.


Unable to obtain file via archive::nexus with Puppet 4

Just to note this works correctly on Puppet 3.x.
This seems to have something to do with the faraday_middleware and the checksum that the archive::nexus calls for the archive type.
If you comment out the checksum_url and checksum_type parameters your able to successfully obtain the file from a nexus store.
I've checked and the two variables for urls are generating valid urls that I can manually download both the file and the checksum (md5) file.
$artifact_url = assemble_nexus_url($url, delete_undef_values($query_params))
$checksum_url = regsubst($artifact_url, "p=${packaging}", "p=${packaging}.${checksum_type}")

On a vagrant instance with Puppet 4 I have the following gems installed
# gem list

*** LOCAL GEMS ***

faraday (0.9.1)
faraday_middleware (0.10.0)
json (1.8.3)
multipart-post (2.0.0)

On a host with Puppet 3 installed the only difference seems to be the version of the json gem is 1.5.5, but i've even downgraded that gem and still get the same issue.

I'm guessing this is related to the AIO packaging, and maybe the AIO gem v.s. system gem. FYI, I'll be offline for a while so will not be able to respond or provide further feedback.

I'll keep trying to dig and do some testing for the time being I've set the $checksum_type to none to effectively disable the checking as a work around for anyone else having this issue

Fixed with merge of 70


How to remove logger instances?

Hello, I tried your plugin and I really like it.
I have log statements like this.logger.log('Saving Session...'); and they get all removed. The only thing that is not removed is this call:
this.logger = new Logdown({ prefix: 'MyLogger', alignOutput: true });

How can I remove such logger initializations?
My current configuration is this:
.pipe(
  gulp_remove_logging({
    namespace: ['this.logger', 'Logdown']
  })
)

Best from Berlin,
Benny

Hi,
AFAIK the regex in https://github.com/valeriansaliou/gulp-remove-logging/blob/master/lib/removelogging.js#L17 only removes statements and not assignments.
Thus, as the line that doesn't get removed is an assignment, it is not matched by the regex.
I do accept PRs if you want to add support for assignments 😄
Best from 🇫🇷


BUG-000099009  The Coordinate Conversion ArcMap 1.0.1 add-in fails to populate the DMS, DD, and DDM fields if the region is set to use a comma as the decimal separator.

Salesforce ID: BUG-000099009
Salesforce Submitter: Ciro Romero
Salesforce Submit Date: 8/30/2016 7:03 AM
Salesforce Bug Type: Usability
Salesforce Severity: Medium
Repro Data: (n/a)
Work Around: Ensure that the region setting is configured to use a period as the decimal separator.
Product: ArcGIS for Defense
Functional Category: ArcGIS for Defense
Client Platform: (n/a)
Version Found: 10.3.1
Planned Version Fixed: (n/a)
Comment: (n/a)

On my windows 8 machine, I am also not able to reproduce the bug.  See the graphic below with the commas highlighted.

@nfeuerstein since Kevin and I are both unable to reproduce the issue, it will be closed.


Short solution needed: "Rust lang throw" (rust)

Please help us write most modern and shortest code solution for this issue:
Rust lang throw (technology: rust)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to throw an error in Rust
How to throw an exception in Rust
How to throw custom error in Rust


Filter SSL Ciphers list to those supported by Java and Erlang R16

Added test to verify this list in the future / on new javas/erlangs.
Added test to print supported/unsupported lists for future reference.

Just out of curiosity, did anyone run into the term_to_oid error with the Java client?

Not that I know of.

On Mar 5, 2016, at 12:15 PM, Luke Bakken<EMAIL_ADDRESS>wrote:
Just out of curiosity, did anyone run into the term_to_oid error with the Java client?
—
Reply to this email directly or view it on GitHub.

Since Java seems to correctly negotiate ciphers with Erlang there's nothing to merge anymore, so I'm going to close this PR.  I'll keep the cipher test code around for newer erlangs/javas though instead of deleting the branch.
I'm also going to keep the if(supportedProtocols includes TLS1.2) use TLS1.2 code in, Java 7 will use TLS 1.0 by default locally in tests. Can remove it when we drop 7 support.
So... no code changes right now.


Weighted Round Robin load balancing does not work

Describe the bug
I need to use Weighted Round Robin load balancing algorithms. For now, as a load balancer, Ribbon is used and Ribbon rule WeightedResponseTimeRule.
WeightedResponseTimeRule works, but because of missing statistics, RoundRobin is used as a fallback.
The same behavior with or without Eureka. The cause of the problem - missing information about calls in ServerStats (com.netflix.loadbalancer) - all statistics have value 0.
springCloudVersion: "Hoxton.SR1"

spring-cloud-starter-gateway
spring-cloud-starter-netflix-ribbon
spring-cloud-starter-netflix-eureka-client
spring-cloud-starter-circuitbreaker-reactor-resilience4j
org.springframework.boot: '2.2.2.RELEASE'

Sample
Main class:
@SpringBootApplication
@EnableDiscoveryClient
public class DemoApplication {

		@Bean
		public RouteLocator routeLocator(final RouteLocatorBuilder routeLocatorBuilder) {
			return routeLocatorBuilder.routes()
					.route("test Route", p -> p.path("/test-route")
							.filters(f -> f.rewritePath("/test-route", "/v1/test-route"))
							.uri("lb://test-service")).build();
		}
}

Config:
server.port: 8080
test-route.ribbon.NFLoadBalancerRuleClassName:com.netflix.loadbalancer.WeightedResponseTimeRule

See https://github.com/spring-cloud/spring-cloud-commons/issues/674 for spring cloud loadbalancer statistics.
I'm not quite sure what to do here. There's really no way to call LoadBalancerClient.execute() which ribbon needs for stats. You'd have to implement a LoadBalancerClientFilter that was specific to ribbon, something we're not going to do.
We could do something with the reactive one being worked on now.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


[0.8.0Beta17] Issues

The latest BETA release of 0.8.0 is Beta 17. This issue is for reporting issues with this specific version of MeerK40t.
Please do all further testing on this release. Please do not report any further bugs with previous beta releases. Do give any other types of feedback, questions, concerns or comments.
Personal note: I admire the optimism expressed in the release notes of this beta: "Hopefully last beta" and "Known errors are down to effectively nothing." I am looking forward to confirming all previously reported issues are fixed and that this version does not have any bugs when I try to use it in production.

This comment has been created as a holding place for the development team to use for a list of any issues not yet corrected (if there are any).

This is not new and don't think it's a big deal.
If you edit an objects Properties and change the fill color and uncheck "Immediately classify after colour change", the color of the object in the Operation, assuming it is in one, does not change.  The color in the Elements list does change.  If you leave the "classify" check box checked it will classify it and change the color accordingly. If, after you change the color and do not allow it to classify and it doesn't change the color, "Assign Operation" to the object, even it it is the same one, the color will update and change to the correct one.  It just appears that if it doesn't classify, it doesn't refresh the color.

Not new again: If you "Collapse all children" in the Operations panel, everything disappears, it does not show the Operations.  I assumed that this collapse would simply not show any of the  objects listed under a give operation, not hide the operation itself.

Not sure I can reproduce that - if you click on an operation it just closes this single operation, if you click on the root element of operations it will close all. (BTW same goes for elements and group nodes).

Correct.  But right-click on "Operations" (or Elements") and select "Collapse children " and everything disappears under Operations or Elements.

Ah - glad it wasn't me that pointed out that "Known errors are down to effectively nothing." was a bit over optimistic.

[ ] Group nodes do not show Inkscape labels.
[ ] Node labels are inconsistent. Images do not show the SVG id but paths do.
[ ] Path node labels - can we please have the Inkscape label before the SVG id if it exists as this is the name the user will primarily know it by. (Perhaps we can also have Inkscape-Label(id) in this case.)
[ ] Previously reported issue with confusing Raster Wizard functionality still exists. Ability to see expected results when Image ops Raster script settings are applied (aka 0.7 Raster Wizard preview) is still missing. Default 500 DPI which (maybe) is equivalent to step 2 is different from 0.7 default of step-1 if you don't use Raster Script and default of 3 for a raster script.
[ ] Camera - select a different camera in the dropdown on the button bar and then click the camera icon - correct camera is often not opened in the camera window.
[ ] Camera Frames per second slider does nothing except when set to 0. Does it make sense to set as low as zero? Frame rate on my i5 Surface Pro is about 1 per 2 secs. Setting is not persistent either.
[ ] Discord invite no longer works.
[ ] Path properties - small X under the stroke/fill colour is not great UI. Can  we use a much bigger border for the colour button to indicate this. Mouse over on colour button changes the border (good) but clears the colour (bad).
[ ] Path properties - Retrieve button takes a long time or starts an infinite loop (didn't want to wait to find out which).
[ ] Step repeat functionality in Execute Window still missing.
[ ] Automated begin and end special ops functionality still missing from Execute Window.

Stopping testing at this point.

Previous beta issues entries still have loads of reported issues unticked. It is possible that these have been fixed and no checked off. Perhaps someone can review these and create a list of unfixed issues carried forwards.

Automated begin and end routines are now device specific and can be managed in the device configuration window

[ ] Gridlines do not recognise flipped x- or y-axis fully
[ ] changes to device dimensions / orientations are only recognised after a restart

Camera Frames per second slider does nothing except when set to 0. Does it make sense to set as low as zero? Frame rate on my i5 Surface Pro is about 1 per 2 secs. Setting is not persistent either.

It should provide a new picture every 5 seconds. The tooltip has been extended to indicate this

Previous beta issues entries still have loads of reported issues unticked. It is possible that these have been fixed and no checked off. Perhaps someone can review these and create a list of unfixed issues carried forwards.

As far as bugs were concerned they should have been put forward, will validate it over the next couple of days

Path properties - Retrieve button takes a long time or starts an infinite loop (didn't want to wait to find out which).
Depending on the shapes complexity this can indeed take some time, as the shapes area will be calculated as well, which uses polygon representation of the shape. All the samples I had were working, so any 'rogue' pattern would be good to have.

[ ] Grid not properly centered around device origin
[ ] Guide not following all combinations of device axis mirroring
[ ] Moshi-Device-Config incomplete

Correct.  But right-click on "Operations" (or Elements") and select "Collapse children " and everything disappears under Operations or Elements.

This has become now a two-stage-collapse. First it will collapse all open (direct) children, if called a second time it will even hide these.
I am not seeing that is is two stages and not sure why it would collapse all the way to showing nothing.  Would there be a use case where you don't want to show anything under "Operations"? If I have 3 operations; Raster, Engrave, and Cut and then put some paths under each of those

Correct.  But right-click on "Operations" (or Elements") and select "Collapse children " and everything disappears under Operations or Elements.

This has become now a two-stage-collapse. First it will collapse all open (direct) children, if called a second time it will even hide these.

I guess we should define what is considered a child and what is consider the parent.  I assumed that the operation under "Operations" is the parent and the element under "Elements" was the parent then anything under those were children.  If Meerk40t considers Operations and Elements themselves the parent then it would probably collapse the actual Operations or Elements themselves since they are children.  I don't agree with this but that is my opinion, I just think you would always want to see the operations or elements, otherwise, it is miss-leading, to me, and looks like there are no Operations or Elements when they are collapsed.
I also do not see how this is two-stage.  I put 3 operations of Engrave, Cut, and Raster in the Operations, put some paths in those operations, manually expand them all then right-click on "Operation" and select "Collapse children" and everything disappears.  It does not just collapse the paths under the Operations.  It collapses everything to nothing.   If it is supposed to be two-stage, I don't know that it is working like that.  I guess it would depend on what is the parent and child as I mentioned above.  If the parent is the operation itself, then it is collapsing that and not showing it when I "Collapse children" once.  I can never use Collapse children twice since there is never a time when I can get more than one child under an operation.  Maybe there is a situation where this can occur but I can't get it to do that in Operations.   However, I can do this in Elements.  I can have an SVG, then a Group under that, and then paths under the group.  I would think this is two children (Group and paths)  but maybe it is 3 (SVG, Group, paths).  It is treating it like "Operations" and "Elements" are the parents and I don't think it should ever collapse the main one (i.e. the SVG element).  Regardless, if I expand the SVG and the group and then right-click on Elements and select "Collapse children", it all disappears, it does not just collapse the path(s) or not just the Group.
It might help to explain how I expected it to work and I think this is consistent with other software and even how Meerk40t works.  If something in the tree has a child and is collapsed, a plus "+" sign appears next to the parent.  The "+" icon indicates it has a child and can be expanded to show it.  If it is expanded it shows the "-" icon to be able to collapse it.  This also works the same as my example above for Elements if there are more children of the children, you still get a plus "+" icon to expand a collapsed item and a minus "-" on an expanded item to collapse it.  If we agree that these icons represent the ability to expand and collapse items, then "Operations", "Elements", and "Regmarks" themselves are not parents should not be collapsible by Meerk40t because they never show a "+" or "-" icon next to them when they have items within them.  If I cannot collapse them manually then why is MeerK40t allowed to when I use Collapse or Expand children?  I expected Meerk40t to only collapse or expand the children that should be expandable or collapsible manually.  And note that I think that is how it should work and you should not be able to expand or collapse "Operations", "Elements", or "Regmarks".  I personally always want to see if there is anything within them.  However, is there is some odd reason that these should be collapsible and expandable, then I think they should have the "+" and "-" icons next to them for consistency.

I guess we should define what is considered a child and what is consider the parent. I assumed that the operation under "Operations" is the parent and the element under "Elements" was the parent then anything under those were children. If Meerk40t considers Operations and Elements themselves the parent then it would probably collapse the actual Operations or Elements themselves since they are children. I don't agree with this but that is my opinion, I just think you would always want to see the operations or elements, otherwise, it is miss-leading, to me, and looks like there are no Operations or Elements when they are collapsed.

I get your point and i don't fundamentally disagree, there are some edge cases though, where you might want to have better view of certain parts of the tree and want to reduce the ops /elements even more (eg having quite a lot of groups/root objects under the elements tree)
We could even make it user configurable... But we have a agreed a feature freeze until the next official release, which is not far away imho, so for now this needs to go to the backlog (but noted)

Path node labels - can we please have the Inkscape label before the SVG id if it exists as this is the name the user will primarily know it by. (Perhaps we can also have Inkscape-Label(id) in this case.)

The ability to have user-defined node-formatting patterns is in the backlog for the next release after a stable 0.8

Step repeat functionality in Execute Window still missing

True this has been temporarily disabled as the underlying functionality of re-defining homes is currently revised to allow more flexibility.
If you are dependent on this feature then you need either circumvent the issue by just copy / paste the to be repeated elements or continue using v 0.77 until this feature has been reintroduced

Path properties - Retrieve button takes a long time or starts an infinite loop (didn't want to wait to find out which).
Depending on the shapes complexity this can indeed take some time, as the shapes area will be calculated as well, which uses polygon representation of the shape. All the samples I had were working, so any 'rogue' pattern would be good to have.

It was a combined path with a half-dozen tiny circles - so nothing that complicated.

Path node labels - can we please have the Inkscape label before the SVG id if it exists as this is the name the user will primarily know it by. (Perhaps we can also have Inkscape-Label(id) in this case.)
The ability to have user-defined node-formatting patterns is in the backlog for the next release after a stable 0.8

I am not asking for user-defined node-formatting patterns - IMO this would be a unnecessary feature where there are much bigger issues to work on. I am simply asking for consistency of fixed formatting patterns - and I consider inconsistency to be a minor bug.

I guess we should define what is considered a child and what is consider the parent. I assumed that the operation under "Operations" is the parent and the element under "Elements" was the parent then anything under those were children. If Meerk40t considers Operations and Elements themselves the parent then it would probably collapse the actual Operations or Elements themselves since they are children. I don't agree with this but that is my opinion, I just think you would always want to see the operations or elements, otherwise, it is miss-leading, to me, and looks like there are no Operations or Elements when they are collapsed.

I get your point and i don't fundamentally disagree, there are some edge cases though, where you might want to have better view of certain parts of the tree and want to reduce the ops /elements even more (eg having quite a lot of groups/root objects under the elements tree) We could even make it user configurable... But we have a agreed a feature freeze until the next official release, which is not far away imho, so for now this needs to go to the backlog (but noted)

NO - this is NOT a feature - this is a bug (and one that was fixed in 0.7 if memory serves me correctly). When you close all, the Operations, Elements and Registration Marks nodes should NOT be closed i.e. the next level down should remain visible just like in 0.7. BUT if you do decide that the Operations/Elements/Registration nodes should be closed, then they definitely need a +/- symbol next to them.

Camera Frames per second slider does nothing except when set to 0. Does it make sense to set as low as zero? Frame rate on my i5 Surface Pro is about 1 per 2 secs. Setting is not persistent either.

It should provide a new picture every 5 seconds. The tooltip has been extended to indicate this. Persistence works for me.

So what is the purpose of this slider then (as so far I have not been able to decide what its purpose is by experimentation)?

Price to pay: about 10 pixels of screen estate

Camera Frames per second slider does nothing except when set to 0. Does it make sense to set as low as zero? Frame rate on my i5 Surface Pro is about 1 per 2 secs. Setting is not persistent either.

It should provide a new picture every 5 seconds. The tooltip has been extended to indicate this. Persistence works for me.

So what is the purpose of this slider then (as so far I have not been able to decide what its purpose is by experimentation)?

It decides the refresh rate of the camera picture, the more you go to the right, the more 'realtime' the picture will become (at the price of CPU consumption) - I personally use the lowest framerate that is still providing 'acceptable' (to me) results, 1 fps, 2 fps max

Previously reported issue with confusing Raster Wizard functionality still exists. Ability to see expected results when Image ops Raster script settings are applied (aka 0.7 Raster Wizard preview) is still missing. Default 500 DPI which (maybe) is equivalent to step 2 is different from 0.7 default of step-1 if you don't use Raster Script and default of 3 for a raster script.

This is done already it's actually visible in scene. That's what replaced raster-wizard scene, actual scene. The image was modified to apply that stuff during pre-processing since it could be sure to be right otherwise with different devices and different dpi. The default DPI is step 2 (on a 1mil-step device). 0.7 was switched to step 2 too. That was default for most because 1000 dpi was too harsh and burned things more. The raster-wizard I think doesn't mess with the DPI it was inconsistent before with the scripts where some had 3 and others had 2 by default.

I am not asking for user-defined node-formatting patterns - IMO this would be a unnecessary feature where there are much bigger issues to work on. I am simply asking for consistency of fixed formatting patterns - and I consider inconsistency to be a minor bug.

I disagree, there are more usecases and personal preferences than just a single one. I e.g. have no use for Svg-ids or labels in my workflow - they take too much space and hide more relevant information for me - I could consider this an annoyance (as I do) or as a 'minor bug' (which it is not). Others will find them very useful, ymmv.

I am not asking for user-defined node-formatting patterns - IMO this would be a unnecessary feature where there are much bigger issues to work on. I am simply asking for consistency of fixed formatting patterns - and I consider inconsistency to be a minor bug.

I disagree, there are more usecases and personal preferences than just a single one. I e.g. have no use for Svg-ids or labels in my workflow - they take too much space and hide more relevant information for me - I could consider this an annoyance (as I do) or as a 'minor bug' (which it is not). Others will find them very useful, ymmv.

So you refuse to accept a reported issue as a bug (when it is removing existing function) because it is not your personal preference?

Previously reported issue with confusing Raster Wizard functionality still exists. Ability to see expected results when Image ops Raster script settings are applied (aka 0.7 Raster Wizard preview) is still missing. Default 500 DPI which (maybe) is equivalent to step 2 is different from 0.7 default of step-1 if you don't use Raster Script and default of 3 for a raster script.

This is done already it's actually visible in scene. That's what replaced raster-wizard scene, actual scene. The image was modified to apply that stuff during pre-processing since it could be sure to be right otherwise with different devices and different dpi. The default DPI is step 2 (on a 1mil-step device). 0.7 was switched to step 2 too. That was default for most because 1000 dpi was too harsh and burned things more. The raster-wizard I think doesn't mess with the DPI it was inconsistent before with the scripts where some had 3 and others had 2 by default.

AFAIK 0.7.7 still has default Raster Wizard step as 3. But this is still as confusing as heck. In image previously saved at step 3 in 0.7.7 still shows up as 500 DPI. And in 0.7 the Wizard shows you a reasonable size image regardless of the scene zoom whereas  on 0.8 you need to zoom the scene in on the image to see the consequences. I also asked for some sort of reset button - you can abort the 0.7 wizard before executing it - there is no way to experiment with settings in 0.8 and then revert them to the way they were before.

Camera Frames per second slider does nothing except when set to 0. Does it make sense to set as low as zero? Frame rate on my i5 Surface Pro is about 1 per 2 secs. Setting is not persistent either.

It should provide a new picture every 5 seconds. The tooltip has been extended to indicate this. Persistence works for me.

So what is the purpose of this slider then (as so far I have not been able to decide what its purpose is by experimentation)?

It decides the refresh rate of the camera picture, the more you go to the right, the more 'realtime' the picture will become (at the price of CPU consumption) - I personally use the lowest framerate that is still providing 'acceptable' (to me) results, 1 fps, 2 fps max

Framerate does not change when you move the slider (except when on 0 when the picture freezes) - I get 1 frame every 5 seconds as stated above, and the framerate doesn't change. That is the bug report.

No I don't deny your request to have a different display, the solution you are proposing is not the one I see as there are different possibly conflicting demands that require a different approach..

Framerate does not change when you move the slider (except when on 0 when the picture freezes) - I get 1 frame every 5 seconds as stated above, and the framerate doesn't change. That is the bug report.

Thank you, confirmed due to an old setting in place that may override this behaviour if in the cfg file. If you want to see its effect look for fps=1 in the config file. Replace it to something like fps=40.

The step rate in 0.7.7 for raster wizard actually depends on the script applied.

Type set -p camera/0 fps 60 to undo the current setting. It got split from the other one but is still stuck low for the scene. The slider controls the camera fps not the scene fps.

No I don't deny your request to have a different display, the solution you are proposing is not the one I see as there are different possibly conflicting demands that require a different approach..

No - you are refusing to accept it as a bug (which will be fixed in 0.8) and treating it as a feature request for 0.9.

Ah - glad it wasn't me that pointed out that "Known errors are down to effectively nothing." was a bit over optimistic.

Known serious errors are, there are some cosmetic issues but actual kick you in the teeth with not working are actually reduced pretty heavily. Some cosmetic issues that don't produce crashes will probably be swept under the rug. Though I'll likely need to go through them all and actually decide to dismiss them, before a full release is issued. There's still some scut housekeeping like updating the translation templates etc. Rereviewing the crash logs and some other stuff.
Known errors are down to nothing. Unknown errors are of an unknown quantity.

The fixing of the failure to start from the current location identified another error where the HomeCut was being optimized. When it was close to 0,0 it would jot back for a home and then continue with the laser. This from having Home Before set. Both should be fixed in Beta18.

Known serious errors are, there are some cosmetic issues but actual kick you in the teeth with not working are actually reduced pretty heavily. Some cosmetic issues that don't produce crashes will probably be swept under the rug. Though I'll likely need to go through them all and actually decide to dismiss them, before a full release is issued. There's still some scut housekeeping like updating the translation templates etc. Rereviewing the crash logs and some other stuff.
Known errors are down to nothing. Unknown errors are of an unknown quantity.

Cosmetic issues ARE important and IMO should not "be swept under the rug".
As I have said elsewhere, 0.8 is getting to be close to production ready - but IMO the confusing UI for the new image rasterisation functionality is so confusing as to be a significant usability issue.
(But I accept that Step Repeat can wait until 0.8.1.)

AFAIK 0.7.7 still has default Raster Wizard step as 3. But this is still as confusing as heck. In image previously saved at step 3 in 0.7.7 still shows up as 500 DPI. And in 0.7 the Wizard shows you a reasonable size image regardless of the scene zoom whereas on 0.8 you need to zoom the scene in on the image to see the consequences. I also asked for some sort of reset button - you can abort the 0.7 wizard before executing it - there is no way to experiment with settings in 0.8 and then revert them to the way they were before.

The default depended on the script you use. It would give you whatever the script said to use. In 0.8.x it does not set a default. It actually just uses the DPI you give it. So it doesn't have a clear requirement here.
A bit of background. The RasterWizard would actualize the image in a concrete fashion. So the image in the scene was actually that the actual end result pixels of the actual thing you would raster. This required that X steps equaled 1 pixel. So a step of 2 would be sized such that 2 mils would equal 1 pixel. Then it would lock that in place. In 0.8.x because we fully support other drivers for the laser, we do not actually perform the action that rasterwizard says. We just do it in the scene as a soft action. The image nodes always have access to the original image and each time you mess with something it applies the modified script to the original image and gives you the result. This is a simulated result you see in the scene and it's directly related to the view of the image you would see in RasterWizard 0.7.x/0.6.x it actually applies this real script during the cut planning. When you send this to the laser rather than just do "actualize" if it needed to be resized at that point it performs the rasterwizard script inside the cutplanning process. This is because only during cutplanning do we know exactly what backend we are planning for so we know the actual sizes things need to be.
This is also why we prefer DPI to steps. Steps implies discrete elements of distance like steps of a stepper motor, but we don't actually know how big that is. We lost the 1 mil = 1 step assumption. So 2 steps on a galvo laser may actually be basically any size (depends on the lens), but 2 steps on a K40 is about 2 mils (belt stretching aside). In order to accommodate different lasers we needed units that were directly references to physical units. Dots-Per-Inch relates to Inches which have a concrete size (exactly 2.54 cm). We cannot actually use steps on a galvo laser though. During testing Sandor had to set Steps to about 20 to get something the size of a postage stamp. The reason is that given the units there a step was about 0.02 mils. However, a DPI of 500 for a similarly sized image gives us the same quality regardless what the native units are for the device.
I am quite happy to discuss how to make the GUI more user-friendly and I get that some of the required changes are confusing. But, the reasons for doing this are numerous and quite required. The old version 0.7.x could not actually account for stretched belts. Since 1 step = 1 mil was hard-baked into things. But, if you want an image that is correctly sized regardless that your belts are 10% stretched and your 1 step goes 1.1 mils, it needed to be coded this way.

I did the accounting of previous versions of these errata and checked for the ones that were completed and asked. See #1332 to make sure nothing falls through the cracks. A couple of them I fixed while doing the list like the control+mousewheel zoom.

So you refuse to accept a reported issue as a bug (when it is removing existing function) because it is not your personal preference?

Point of fact, that's actually just a defect if anything at all. It's not broken it works the way it's designed but it may be designed sub-optimally which is defect not bug. But, he wrote a whole you can go ahead and make those labels say anything you want thing so it's likely fine in the long run. I sort of considered this a bit feature creep since it's moving stuff that is generally dynamic into letting people theme their nodes.
This was done because Galvo lasers actually use kHz and have frequencies. This is a fundamental part of the operation for Galvo lasers but would be utterly pointless to a K40 user, it could even be confusing to them. So this was generally changed so in Galvo/Balor you were given frequency but in M2 Nano you were not.

If that is your definition of a bug, it stinks. If a user is reporting an issue with a 0.8 beta version cf. functionality of a 0.7 version, it is from a user perspective a bug (regardless of the developer intent) and unless there is a good reason for developers not to treat it as a bug then they should do so.
I have no issues with labels showing device-specific information, but that does not invalidate the issue / requirement as reported by me as a user - labels still need to be consistent and show information generally useful to a reasonable proportion of the user base. (And since I am literally doing more testing and reporting more issues than all other users put together, I would hope that my needs carry some extra weight.

Crash in Execute Window on clear of Merge Passes checkbox:
MeerK40t crash log. Version: 0.8.0025 Beta17 on Windows: Python 3.7.9: AMD64 - wxPython: 4.1.1 msw (phoenix) wxWidgets 3.1.5
Traceback (most recent call last):
  File "meerk40t\kernel\kernel.py", line 1742, in schedule_run
  File "meerk40t\kernel\kernel.py", line 1957, in process_queue
  File "meerk40t\kernel\kernel.py", line 1927, in _process_signal_queue
  File "meerk40t\gui\choicepropertypanel.py", line 845, in listen_to_myself
RuntimeError: wrapped C/C++ object of type CheckBox has been deleted

Hm. Mine isn't crashing there. The bug says that a signal is issued with the listen_to_myself listener still attached. But, if execute window is opened it wouldn't. Let me see. Maybe shut it down and reopen it. -- yeah, that did it.
The lifecycle is there. It won't do it the first time. You need to open, close, and reopen Execute Window then change the state of Merge Passes.

Great catch. It's a little harder to invoke than just uncheck a box though.
It would actually plant that bomb any time. So if you loaded Execute Job, then dechecked merge passes in LaserPane it should also crash there.

Issues when attempting to use 0.8.0B17 for production use.

[ ] Execute / Simulation of selected operations does not include default special ops.
[ ] Simulation - scene not updated when you move the playback slider - but does update if you do e.g. a zoom scene - so an update signal may be missing. (May be a consequence of previous crash - will restart and retest.)
[ ] Simulation - playback speed of e.g. 5000% does not seem to go faster.
[ ] Execute - Copy button stays as Copy after copy rather than becoming Blob. Ops get duplicated. (May be a consequence of previous crash.)
[ ] Execute - Automated finish beep occurs before burn has actually finished. Interrupt popup happens before previous ops have actually finished burning.
[ ] Close MK after burn has completed gives "The device is actively sending data. Really quit?".
[ ] Red background behind scene when burn is happening is missing.

[ ] Each burn is now shown as a single item in spooler even with special ops and no merge ops. I can see this is a good thing when e.g. you are running several lasers in parallel and you want to see each job as a single line.  This was a VERY useful ability in 0.7 and I miss it already in 0.8. Can we please have the ability to select a burn in the spooler and open a window showing individual "operations" bearing in mind merges etc. and e.g. cancel later ops?
[ ] Execute - multiple operation passes not burned
[ ] Spooler - Estimate column shows elapsed time not estimated total time.
[ ] Spooler - Name column should show project name or similar (for multiple projects to multiple lasers simultaneously) - number of items should be a separate column.
[ ] Spooler - Device column needed (for multiple simultaneous lasers).
[ ]

The crash kills the scheduler so anything that gets scheduled is somewhat likely to not work, also a lot of console commands break after a crash. Crashes break a the kernel basically which doesn't easily recover. The Bee

Is it possible for the crash handler to reset the kernel so that things all work - or will this e.g. break burns in progress? How about fixing things that do not effect a burn in progress?

No more testing from me now until all essential issues above have been fixed in beta 18.

Closed because beta18 is available.


github-actions-runner: do not use static names but create them based on parameters

No longer necessary. We shift to the operator.


Please add Windows 11 home edition.

Hi, My customer use Remote desktop at Windows 10 Home Edition.
They want to upgrade to Windows 11, but Windows Home Edition is not supported remote desktop.
I think it's probably a miss the words, so please add it.

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

ID: f10e667a-7b08-b25f-5c0a-b776b7f11cb4
Version Independent ID: ae4936a7-f5a3-b3ad-170c-7aba109435db
Content: Supported configurations for the Windows Desktop client
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/windowsdesktop-supported-config.md
Product: windows-server
Technology: remote-desktop-services
GitHub Login: @Heidilohr
Microsoft Alias: helohr

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


(maint) Bump Leatherman version to get fixes

(maint) Fixup AppVeyor to remove sh.exe from path
(maint) Fix compilation on Windows with GCC 5.2.0

:+1: The changes made in leatherman associated with this bump seem reasonable to me
For reference, new commits are:
$ git log --no-merges --oneline  b3616ef7bf5fb96ec8633b0060b1e28b058f73a9..6511f28746fa1809bde322496d687637682ee0fd
15bfdc6 (maint) Fixup AppVeyor to remove sh.exe from path
2606f5f (maint) Fix compilation on Windows with GCC 5.2.0


vendor.js is too big to use on mobile

Resources:
Before submitting an issue, please consult our troubleshooting guide (http://ionicframework.com/docs/troubleshooting/) and developer resources (http://ionicframework.com/docs/developer-resources/)
Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

hey guys,
thanks for build amazing framework ionic,
but the vendor.js is too big to use on mobile device,
it's more than 400kb without gzip

Hello! Thank you for opening an issue with us!
This is a large part of the reason that with Ionic v4 we are moving to using web components, as well as built in lazy loading and code splitting. This will help to make that bundle smaller.
Note that it will still initially be on the larger size because of the use of Angular. However, that is also becoming more optimized over time, and eventually, you should be able to use Ionic with lighter weight frameworks such as VueJS or Preact.
Thank you for using Ionic

@keithdmoore
amazing,
wish ur guys goal,
by the way i think angular is good enough

@keithdmoore
I read about Stencil sourcecode.
do u mean ionic v4 will base on it.
and v3 can't compatible with v4?
like angular.js & angular 2?

@yozman - yes, Ionic v4 will use Stencil. In fact, that is why Stencil is being created.
As for compatibility, expect some breaking changes. Ionic v4 will not just be a drop in replacement of v3. There will be some breaking changes where the code will need to change in order to use it, but I would not expect anything like the change from AngularJS to Angular.

@keithdmoore
Will Ionic v4 use angular 5 too?
Should we expect a change to the routing system, maybe an adaptation of angular's routing system.
The current ionic routing system works well for mobile but has some snags for PWAs.
it will be good if we could get a tentative timeline for its release too