UWP unit tests - Unable to install Windows app package in 30 sec.

I am getting this error when running UWP unit tests from VSTS build:
2016-09-26T02:42:20.7330642Z Executing the powershell script: c:\agent\tasks\VSTest\1.0.59\VSTest.ps1
2016-09-26T02:42:23.8892543Z ##[debug]Calling Invoke-VSTest for all test assemblies
2016-09-26T02:42:24.0767505Z Working folder: D:\agent\_work\2\s
2016-09-26T02:42:24.0767505Z Executing C:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\IDE\CommonExtensions\Microsoft\TestWindow\vstest.console.exe "D:\agent\_work\2\s\src\App.Mobility\Test Projects\App.Mobility.Tests\AppPackages\App.Mobility.Tests_2.42.10414.0_x86_Test\App.Mobility.Tests_2.42.10414.0_x86.appx"  /logger:trx
2016-09-26T02:42:24.5142422Z Microsoft (R) Test Execution Command Line Tool Version 14.0.25420.1
2016-09-26T02:42:24.5142422Z Copyright (c) Microsoft Corporation.  All rights reserved.
2016-09-26T02:42:24.7642377Z Starting test execution, please wait...
2016-09-26T02:42:24.7798623Z Warning: Using Isolation mode to run unit tests for Windows Store apps. Use the /InIsolation parameter to suppress this warning.
2016-09-26T02:42:58.6307248Z ##[error]Error: Installation of package 'D:\agent\_work\2\s\src\App.Mobility\Test Projects\App.Mobility.Tests\AppPackages\App.Mobility.Tests_2.42.10414.0_x86_Test\App.Mobility.Tests_2.42.10414.0_x86.appx' failed with Error: (0x5B4) Operation timed out. Unable to install Windows app package in 30 sec..
2016-09-26T02:42:58.6307248Z ##[error]For more details look into Event Viewer under Applications and Services Logs -> Microsoft -> Windows -> AppXDeployment-Server -> Microsoft-Windows-AppXDeploymentServer/Operational.
2016-09-26T02:42:58.6307248Z ##[error]
2016-09-26T02:42:58.7088442Z ##[error]System.Exception: VSTest Test Run failed with exit code: 1
2016-09-26T02:42:58.7400932Z ##[warning]No results found to publish.

How can I get this test task to successfully pass?

This can happen for multiple reasons. There is a little known configuration variable that you can use to increase this timeout via the runsettings file
<?xml version="1.0" encoding="utf-8"?>
<RunSettings>
  <!-- Configurations that affect the Test Framework -->
  <RunConfiguration>
    <!-- [x86] | x64
      - You can also change it from menu Test, Test Settings, Default Processor Architecture -->
    <TargetPlatform>x64</TargetPlatform>
  </RunConfiguration>

  <!-- Adapter Specific sections -->

  <!-- MSTest adapter -->
  <MSStoreTest>
    <AppxInstallTimeout>120000</AppxInstallTimeout>
  </MSStoreTest>
</RunSettings>

So little known that there isn't a single result on Google!
Unfortunately the problem went away by itself a couple of days ago (maybe Azure sorted out a storage perf issue?) but I'll add a runsettings file anyway.

@JamesNK Your question just helped it becoming more known :)
For reference. From what I have read/seen the timeout can occur due to machine perf so its generally not a 100% repro unless you have a ton of binaries that are actually causing the delay in getting the app installed.


Remove cloud-init from the Openstack image configuration

cloud-init is replaced by modules used by our EC2 image (Openstack metadata service provides an EC2 API).
Tests are added: check if ssh keys and configuration.nix are applied
novaImage is renamed to openstackImage

This is mainly to reduce the size of the image (from 2.2GB to 1.3GB). Also, I think we generally don't care about cloud-init features since we have a configuration.nix file.
Unfortunately, one of the two tests doesn't work in the sandbox because it needs Internet. This is also the case of EC2 tests... I hope to fix them later (help is welcome on that).
I also test this image on an Openstack cloud. It is working as expected.
Note this also fixes the current Nova image which is currently broken.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg test ec2-nixops

The ec2 tests failure is not related to these patch (same error without my patches).
ping @flokli :)

@edolstra from your reaction on the PR, is there a way you can test this?


[WIP] Spherical Terrain

A WIP implementation of spherical terrain.
This will be in a prototype stage for a couple of weeks.

Both planar and spherical terrains use the same implementation now. You can enable the latter using the cargo "spherical" feature.
I have also experimented with an early attempt at mapping a cube map to the planet.

What terrain data do you use in the spherical terrain example? I'm always getting a "No such file or directory" error and adding the six 32K tiles also seems to be the wrong approach.
Amazing work so far! 🙌

@spectrachrome
Thank you. This PR is quite a big overhaul and constantly changing. I would not depend on it so far. I hope to get the plugin in a usable state again by early next year.
I have not added the large textures to git, to avoid bloating the repo.
I will make a smaller dataset for testing, once the rest is cleaned up.


Table token bug

Task
TR #115
Why?
An invalid token was being used.
What Changed

[X] Fix token

Sanity Check

[X] Have you updated any usage of changed tokens?
[X] Have you updated the docs with any component changes?

[X] Have you tried building the css?
[X] Have you tried building storybook?
Do you need to update the package version?

 Table Bug


Use PVC source names wherever missing

This fixes issues arising due to the name mapping of PVC objects.

This PR actually fixed two BZs that were filed after it was merged, hence the missing BZ in the title. For posterity:
https://bugzilla.redhat.com/show_bug.cgi?id=1996627
https://bugzilla.redhat.com/show_bug.cgi?id=1997051


OCR Keyword extraction

Changes

Add (simple) keyword extractor (ocr.NewKeywordExtractor(...))
Add fromWorker grpc NewKeywords function
Add Table Keywords (streamId, text, language, valid)
Extract keywords after streamEnd, vodUpload (after silence detection)

Follow-Ups

Integrate Keywords in Search Results
Filter more keywords from Extractor results

Privacy statement
Um den Funktionsumfang von TUM-Live zu erweitern, werden Aufzeichnungen automatisiert und maschinell verarbeitet.

New Problem
The current approach (store every keyword in an entry) is super slow. The full-text-search-query takes a lot of time. Will need to find another way to store keywords and potentially store less of them.

What does super slow mean in terms of milliseconds?

What does super slow mean in terms of milliseconds?

SELECT streams.*,
       (match(streams.name) against('correctness*' in boolean mode) +
        match(streams.description) against('correctness*' in boolean mode) +
        match(c.message) against('correctness*' in boolean mode) +
        match(vs.description) against('correctness*' in boolean mode) +
        match(kw.text) against('correctness')) as 'relevance'
FROM `streams`
         left join video_sections vs on vs.stream_id = streams.id
         left join chats c on c.stream_id = streams.id
         left join tumlive.keywords kw
                   on kw.stream_id = streams.id
WHERE ((match(streams.name) against('correctness*' in boolean mode) OR
        match(streams.description) against('correctness*' in boolean mode) OR
        match(c.message) against('correctness*' in boolean mode) OR
        match(vs.description) against(53 in boolean mode) OR
        match(kw.text) against('correctness')) AND streams.course_id = 53 AND streams.recording = 1)
  AND `streams`.`deleted_at` IS NULL
GROUP BY `streams`.`id`
ORDER BY relevance

This query takes 29s(!!!) on my machine.


chore: upgrade charts

updated chart cdf/tekton-pipeline from 0.27.2 to 0.29.0
updated chart jxgh/jx-build-controller from 0.3.9 to 0.3.11
updated chart jxgh/jx-preview from 0.0.194 to 0.0.195
updated chart jxgh/jxboot-helmfile-resources from 1.1.38 to 1.1.47

/close


Test Problem

I train my model[Webface] in parms of s=30, m=0.35. The Result in lfw is 98.53%. I try to change parms but it get worse result. Is the parms is the best in your tests?   Can data strength help me improve the effect?   thanks for your advice

@zys1994 i also get worse result in lfw，i want to know how many iter have you train on ResNet?

@happynear hi, have u trained the ResNet using VGG2 dataset? i only get 98.5 on LFW,i want to know how many iter have you train on ResNet using VGG2 dataset?


w2layout.onShow isn't firing?

hello vitman,
sorry to trouble you again but i don't appear to be able to get w2layout.onShow to fire with either an onShow object property or by using the after-object-creation method, as per:
http://w2ui.com/web/docs/w2layout.onShow
is something broken with it atm?
edit: some more information:
i'm showing the layout using w2layout.content after the page has been loaded, triggered by the user clicking a button.

onShow only fires if you use layout.show(...) and layout.hide(...) methods. It is not effected by layout.content(...)

vman, i don't mean to be a pain, but i think the fundamental problem is there isn't a callback option for parent.content()
in essence i was just hoping to be able to keep many child layouts loaded and swap them in/out of a parent layout without having to resort to parent.load() in order to reload/repopulate the child layouts.
but i'll try and hack a workaround, so you may close this again!

update: ok i worked around this by using a setTimeout immediately after calling parent.content()

I have content loaded with angular via async request, and I want to attach behavior to an element inside it's content. How can I know when element is created, meaning when content has been rendered inside panel?
Thanks
$('#layout').w2layout({
                name: 'name',
                panels: [
                    {
                        type: 'top',
                        hidden: false,
                        resizeable: false,
                        content: $compile('

')($scope),
                        size: 32,
                        ...
                        }
                    },
});


Add info how to build modular kas

These changes should be merged with the next release of meta-zarhus as they're currently staged in the meta-zarhus develop branch

I'm not sure why it is not merged? @artur-rs ?

I'm not sure why it is not merged? @artur-rs ?

Check PR description, these changes are for next meta-zarhus release.


Highlight changes for folders

A file that was changed has its name colored in the files tree. If a folder contains a file with changes, it should be colored as well.

Actually I was about to create a separate issue but this one is very relevant. The specific problem I kept facing with the local editor is it doesn't see changes to any document unless changes were staged.
My situation is as follows: A developer who has a Gitbook documentation within his project folder. This setup allows me to easily make few changes from inside the IDE so I don't have to launch Gitbook editor or go to the site. There are other niceties such as when renaming something it can be done all together etc. The problem arises once I want to dive in more into documenting and I launch the Gitbook editor forgetting to stage every freaking change. Many times I lost all the changes I made.
I actually have ditched the Gitbook editor for that reason for a long time now and instead use Atom. With a couple of great markdown extensions it's pretty decent alternative to the Gitbook editor.
Aside from this, and this is just my 0.02 cents, If you guys make a dedicated Gitbook extension for Atom, you might not need to recreate the wheel, I mean the editor part of it.

@sahal2080 Support of files edited outside the GitBook Editor is tricky but we are working on it 👷
Somebody created an Atom plugin for editing GitBooks: https://github.com/cthos/atom-gitbook

@sahal2080 I've posted an issue for this: #101


Not enough information parsed from response

Hey, I tried to use your bundle to create an oauth2 connection with SymfonyConnect and found that I can't link accounts with email, because ResourceOwner doesn't parse this info from the OAuth response. Is it possible to fix it? Or probably there is a workaround?

@sadikoff feel free to fix it in a pull request, I will be happy to merge it ;)
You can check how to get the email here https://github.com/symfonycorp/connect/blob/master/src/Api/Parser/VndComSymfonyConnectXmlParser.php#L173

@qdequippe Woohoo! Nice! Just wanted to check if you still support it =) I'll make a PR soon 🍺


Introducing Transforms

Issue #, if available:
Description of changes:
Introducing transforms feature to index management - https://github.com/opensearch-project/index-management/blob/main/docs/transform-rfc.md
Most of the code was 1 person reviewed during development
CheckList:
[ X] Commits are signed per the DCO using --signoff
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #17 (1d2c8ea) into main (72018d4) will increase coverage by 0.78%.
The diff coverage is 81.25%.

@@             Coverage Diff              @@
##               main      #17      +/-   ##
============================================
+ Coverage     77.31%   78.10%   +0.78%
- Complexity     1557     1856     +299
============================================
  Files           201      255      +54
  Lines          8588     9962    +1374
  Branches       1326     1506     +180
============================================
+ Hits           6640     7781    +1141
- Misses         1250     1366     +116
- Partials        698      815     +117

Impacted Files
Coverage Δ
Complexity Δ

...pensearch/indexmanagement/IndexManagementRunner.kt
55.55% <0.00%> (+5.55%)
4.00 <0.00> (+1.00)

...arch/indexmanagement/rollup/RollupMapperService.kt
58.87% <ø> (ø)
23.00 <0.00> (ø)

...ch/indexmanagement/rollup/RollupMetadataService.kt
75.24% <ø> (ø)
34.00 <0.00> (ø)

...management/rollup/interceptor/RollupInterceptor.kt
68.96% <ø> (ø)
31.00 <0.00> (ø)

...ensearch/indexmanagement/rollup/model/ISMRollup.kt
93.04% <ø> (ø)
27.00 <0.00> (ø)

.../opensearch/indexmanagement/rollup/model/Rollup.kt
89.17% <ø> (ø)
53.00 <0.00> (ø)

...ansform/exceptions/TransformValidationException.kt
0.00% <0.00%> (ø)
0.00 <0.00> (?)

...ensearch/indexmanagement/util/ScheduledJobUtils.kt
26.92% <26.92%> (ø)
0.00 <0.00> (?)

...opensearch/indexmanagement/rollup/RollupIndexer.kt
58.92% <50.00%> (-2.09%)
8.00 <0.00> (ø)

...form/action/index/TransportIndexTransformAction.kt
54.00% <54.00%> (ø)
3.00 <3.00> (?)

... and 121 more

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 72018d4...1d2c8ea. Read the comment docs.


Protobuf messages registration conflicts

tss-lib points to the github.com/golang/protobuf v1.3.2 dependency in the go.mod file. However, according to Go modules version selection policy all versions of protobuf >= v1.3.2 and < v2 can be picked if present in the module list.
This can be a problem because tss-lib protobuf messages are registered twice which causes a registration conflict. The first registration occurs in the generated *.pb.go files and then the second registration is performed in messages.go files.
Example for the ecdsa/keygen package:

First registration: https://github.com/binance-chain/tss-lib/blob/abd66f888438ead112891bc3d74d04d09b94446f/ecdsa/keygen/ecdsa-keygen.pb.go#L235
Second registration: https://github.com/binance-chain/tss-lib/blob/abd66f888438ead112891bc3d74d04d09b94446f/ecdsa/keygen/messages.go#L35

The github.com/golang/protobuf v1.3.2 doesn't complain but the newer github.com/golang/protobuf v1.4.0 displays a warning:
2020/08/05 14:43:02 WARNING: proto: message KGRound1Message is already registered
A future release will panic on registration conflicts. See:
https://developers.google.com/protocol-buffers/docs/reference/go/faq#namespace-conflict

hi @lukasz-zimnoch , thank you for reporting,
can you suggest ways to fix this issue?

Thoughts on fixing this by simply removed from messages.go? If the generated pb.go file takes care of registration, the explicit manual registration in messages.go seems like it should be unnecessary. The init function is used in both cases, and both thepb.go file and messages.go are in the same package which means both registrations run at the same time during package initialization. That should mean the messages.go init function is redundant. I believe this should be true even on earlier versions that may not warn on duplicate registration.

Hey @omershlo,
We've noticed that when both ecdsa and eddsa is used depending on the intialization order, protobuf can't handle both runtime because of name clashes.
I've opened a PR to fix that. As a side effect the warnings have been cleared as well.
Let me know what you think.

This PR also fixed the issue.
See go.mod
ecdsa/signing/messages.go
ecdsa/keygen/ecdsa-keygen.pb.go
and other related part.


Pass dockercfg path straight to docker-py's login

This doesn't expose the password in the logs, as docker daemon
sends used creds as a response to login.

Coverage decreased (-0.5%) to 87.212% when pulling 8446e8038fbcec3b1f7fe66e6949dc4a8c5436b3 on vrutkovs:pulp-sync-auth-no-pass into 67c682756a5f8b99d38fae005cbf86be234ed503 on projectatomic:master.

LGTM

Coverage decreased (-0.2%) to 87.5% when pulling 888d7ac5960db89c8a1d1625a42a595b718a1688 on vrutkovs:pulp-sync-auth-no-pass into 67c682756a5f8b99d38fae005cbf86be234ed503 on projectatomic:master.


Metadata: Automate release note generation

Description
Add a new section to the team's PR template that includes a 'Proposed Release Notes' section. Use that content to automatically generate release notes. The Node team has a similar process that can be drawn upon for inspiration. For this to be successful, we will also have to flow this new flow into team processes.
Acceptance Criteria
Create a new PR template and automate the generation of release notes.

Hey team! Please add your planning poker estimate with Zenhub @fallwith @kaylareopelle @tannalynn


Support Sublime-Text-style selection transpose

(Way back in December, @rebornix said in #3776 that this was a good idea to contribute to core, but I never really got around to it until now...)
 But if at least one selection is non-empty, transpose will instead transpose the contents of the selection, replacing the contents of selection 2 with selection 1, selection 3 with selection 2, and so on until selection 1 is replaced with the contents of the last selection.
This allows quick swaps, for instance, you can double-click on one word, Opt+double-click on another word, and press Ctrl+T to swap them.
VS Code currently has two Transpose actions:

editor.action.transposeLetters
"Transpose Letters" (bound to ^t by default)
editor.action.transpose
"Transpose characters around the cursor" (not bound to anything by default)

These are very subtly different: transposeLetters does nothing at the end of a line, while transpose transposes the last character to the other side of the line break (in a clone of Sublime Text's behavior).
(Note that neither of these matches the emacs/macOS behavior, which is to transpose the last two characters of the line.)
This behavior has been added to transpose rather than transposeLetters, because transpose seems to both be a less specific name, and because it was designed for Sublime compatibility.

@Zarel,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@Zarel, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

@rebornix Incidentally, I had to rewrite this functionality from scratch compared to my extension, because apparently core commands use a different API involving executeCommands and ICommand[], rather than the vscode.TextEditorEdit system that extensions seem to use. Am I missing something?

@Zarel thanks for your PR, it looks great! Just left some tiny comments, once they are addressed, we can have this merged.

@rebornix I'm having trouble with the test now. I'm getting this error message when I try to install dependencies:
node-pre-gyp ERR! Tried to download(404): https://node-inspector.s3.amazonaws.com/profiler/v5.6.5/electron-v1.6-darwin-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(electron-v1.6 ABI) (falling back to source compile with node-gyp)
  CXX(target) Release/obj.target/profiler/src/profiler.o
  CXX(target) Release/obj.target/profiler/src/cpu_profiler.o
  CXX(target) Release/obj.target/profiler/src/cpu_profile.o
  CXX(target) Release/obj.target/profiler/src/cpu_profile_node.o
  CXX(target) Release/obj.target/profiler/src/heap_profiler.o
../src/heap_profiler.cc:35:18: warning: 'TryCatch' is deprecated
      [-Wdeprecated-declarations]
        TryCatch try_catch;
                 ^
/Users/zarel/.electron-gyp/.node-gyp/iojs-1.6.6/deps/v8/include/v8.h:7711:40: note:
      'TryCatch' has been explicitly marked deprecated here
  V8_DEPRECATED("Use isolate version", TryCatch());
                                       ^
1 warning generated.
  CXX(target) Release/obj.target/profiler/src/heap_snapshot.o
../src/heap_snapshot.cc:46:46: error: no member named 'GetHiddenValue' in
      'v8::Object'
      info.GetReturnValue().Set(info.This()->GetHiddenValue(__root));
                                ~~~~~~~~~~~  ^
../src/heap_snapshot.cc:50:20: error: no member named 'SetHiddenValue' in
      'v8::Object'
      info.This()->SetHiddenValue(__root, _root);
      ~~~~~~~~~~~  ^
2 errors generated.
make: *** [Release/obj.target/profiler/src/heap_snapshot.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:125:13)
gyp ERR! stack     at ChildProcess.emit (events.js:213:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:197:12)
gyp ERR!

@Tyriar can you help on this, I don't have clue what went wrong here.

I think this might be to do with node 7.7.0 as I've seen this too in node-pty https://github.com/Tyriar/node-pty/issues/44
@Zarel did you use ./scripts/npm.sh install (or .bat on Windows) to install the dependencies? A simple npm install will not work.

@Tyriar Oh, I forgot I was on Node 7.7.0 to test something. I'm on Node 8.2.0 and I'm getting a different error:
><EMAIL_ADDRESS>install /Users/zarel/Workspace/vscode-test/node_modules/electron-mksnapshot
> node ./download-mksnapshot.js

/Users/zarel/Workspace/vscode-test/node_modules/extract-zip/index.js:35
      zipfile.readEntry()
              ^

TypeError: zipfile.readEntry is not a function
    at /Users/zarel/Workspace/vscode-test/node_modules/extract-zip/index.js:35:15
    at /Users/zarel/Workspace/vscode-test/node_modules/yauzl/index.js:31:7
    at /Users/zarel/Workspace/vscode-test/node_modules/yauzl/index.js:96:14
    at /Users/zarel/Workspace/vscode-test/node_modules/yauzl/index.js:342:5
    at /Users/zarel/Workspace/vscode-test/node_modules/fd-slicer/index.js:32:7
    at FSReqWrap.wrapper [as oncomplete] (fs.js:664:17)

Yes, I'm using ./scripts/npm.sh install. I have successfully built and tested VS Code in the past; I did in fact confirm that the test I added ran successfully when I opened this pull request, after all.

@Zarel never seen that one, I'm not sure what the status of building with node@8/npm@5 is. I know<EMAIL_ADDRESS>works, maybe you could try that to rule out node being an issue?

<EMAIL_ADDRESS>worked. I guess I'll test on that for now.

Any update on this? Looks like it's done or almost done, just not merged.

I got weird build errors that no one knew how to fix, so this sort of just got ignored.
In the meantime, my VS Code extension does the same thing:
https://marketplace.visualstudio.com/items?itemName=Zarel.sublime-commands

Is it possible to just bundle an extension into VS Code? That might be easier than trying to figure this out.

After having it sit here for a couple of years, I currently prefer that we keep this feature in an extension, like https://marketplace.visualstudio.com/items?itemName=Zarel.sublime-commands, @Zarel sorry for going back and forth on this, thank you again for your contribution, kudos to you ❤ .


OCPCLOUD-2010: Remove feature gate for external platform

This changes moves the external platform out of the feature gate:

Remove TechPreviewNoUpgrade on CloudControllerManager field
Move feature externalCloudProviderExternal to default feature set
Move integration tests from techpreview to stable
Update CEL on ExternalPlatformStatus to make it immutable and add
infrastructure tests for it. It was not implemented in the previous
change: https://github.com/openshift/enhancements/blob/master/enhancements/cloud-integration/infrastructure-external-platform-type.md?plain=1#L591-L592

/cc @elmiko
I'm opening a PR against CCO to test this change

Can we get either some CI signal or QE ack that this is tested and ready for promotion before we merge?

Can we get either some CI signal or QE ack that this is tested and ready for promotion before we merge?

Hey @JoelSpeed - here are some semi-automated tests:

Custom release created based on 4.14.0-ec.3 (registry.build05.ci.openshift.org/ci-ln-6yfxjtt/release:latest)

build 4.14.0-ec.3,openshift/cluster-config-operator#331

Cluster installed (UPI) in OCI with platform external without feature gate config
CCM manifests are added in install time with proper taints to initialize the node
installation finished successfully

$ oc get clusterversion
NAME      VERSION                                                AVAILABLE   PROGRESSING   SINCE   STATUS
version   4.14.0-0.test-2023-07-18-182945-ci-ln-6yfxjtt-latest   True        False         54m     Cluster version is 4.14.0-0.test-2023-07-18-182945-ci-ln-6yfxjtt-latest

nodes

$ oc get nodes
NAME                               STATUS   ROLES                  AGE   VERSION
master-01.priv.ocp.oraclevcn.com   Ready    control-plane,master   74m   v1.27.3+e8b13aa
master-02.priv.ocp.oraclevcn.com   Ready    control-plane,master   72m   v1.27.3+e8b13aa
master-03.priv.ocp.oraclevcn.com   Ready    control-plane,master   72m   v1.27.3+e8b13aa
worker-01.priv.ocp.oraclevcn.com   Ready    worker                 61m   v1.27.3+e8b13aa
worker-02.priv.ocp.oraclevcn.com   Ready    worker                 61m   v1.27.3+e8b13aa
worker-03.priv.ocp.oraclevcn.com   Ready    tests,worker           62m   v1.27.3+e8b13aa

infra object:

  spec:
    cloudConfig:
      name: ""
    platformSpec:
      external:
        platformName: oci
      type: External
  status:
    apiServerInternalURI: https://api-int.opct-ci.$basedomain:6443
    apiServerURL: https://api.opct-ci.$basedomain:6443
    controlPlaneTopology: HighlyAvailable
    etcdDiscoveryDomain: ""
    infrastructureName: opct-ci-5mmqk
    infrastructureTopology: HighlyAvailable
    platform: External
    platformStatus:
      external:
        cloudControllerManager:
          state: External
      type: External

install-config cm

data:
  install-config: |
    additionalTrustBundlePolicy: Proxyonly
    apiVersion: v1
    baseDomain: $basedomain
    compute:
    - architecture: amd64
      hyperthreading: Enabled
      name: worker
      platform: {}
      replicas: 3
    controlPlane:
      architecture: amd64
      hyperthreading: Enabled
      name: master
      platform: {}
      replicas: 3
    metadata:
      creationTimestamp: null
      name: opct-ci
    networking:
      clusterNetwork:
      - cidr: <IP_ADDRESS>/14
        hostPrefix: 23
      machineNetwork:
      - cidr: <IP_ADDRESS>/16
      networkType: OVNKubernetes
      serviceNetwork:
      - <IP_ADDRESS>/16
    platform:
      none: {}
    publish: External
    pullSecret: ""
    sshKey: ...
kind: ConfigMap

Kubelet and KCM flag

$ for NODE in $(oc get nodes -o jsonpath='{.items[*].metadata.name}'); do echo -e "\n---> $NODE:"; oc debug node/$NODE -- chroot /host /bin/bash -c "echo '>> Kubelet:' ; ps aux |grep /usr/bin/kubelet | head -n 1 | tr ' ' '\n' | grep cloud-provider; echo '>> kube-controller-manager:'; ps aux |grep 'kube-controller-manager --openshift-config' | head -n1 | tr ' ' '\n' | grep cloud-provider " 2>/dev/null; done

---> master-01.priv.ocp.oraclevcn.com:
>> Kubelet:
--cloud-provider=external
>> kube-controller-manager:
--cloud-provider=external

---> master-02.priv.ocp.oraclevcn.com:
>> Kubelet:
--cloud-provider=external
>> kube-controller-manager:
--cloud-provider=external

---> master-03.priv.ocp.oraclevcn.com:
>> Kubelet:
--cloud-provider=external
>> kube-controller-manager:
--cloud-provider=external

---> worker-01.priv.ocp.oraclevcn.com:
>> Kubelet:
--cloud-provider=external

---> worker-02.priv.ocp.oraclevcn.com:
>> Kubelet:
--cloud-provider=external

---> worker-03.priv.ocp.oraclevcn.com:
>> Kubelet:
--cloud-provider=external

Additionally, conformance tests (o/k suites) are running right now with OPCT in CI.
Anything else to collect?
cc @elmiko @adriengentil


Feature: adding new driver for Agilent signal generator E8267C

added all basic commands.
why no one likes 💃 :(
@giulioungaretti @jenshnielsen

@jenshnielsen @giulioungaretti fixed the numpy functions


Grappler graph optimizations may add nondeterministic behavior

System information

OS windows 10
tensorflow-gpu==2.5.1
python=3.7
CUDA/cuDNN version: 11.2
GPU model and memory: 471.68-desktop-win10-win11-64bit-international-dch-whql

Got non-determinism on GPU.I found out it's the problem of Grappler graph optimizations,then i set this option false,but the train is more slower than cpu.
options = {'implementation_selector':False}
tf.config.optimizer.set_experimental_options(options)
https://github.com/tensorflow/community/blob/master/rfcs/20210119-determinism.md#grappler-changes
I found out this document may help me to fix this problem ,but don't know how to disabled timeout with a RewriteConfig option on tensorflow-gpu==2.5.1.

@jackyesf,
Can you share a simple reproducible code to expedite the trouble-shooting process? Thanks!


Previously used Lsat for webview

Opening this issue as we would like to be able to check if a relay user already has an lsat with the authorization for a specific endpoint
building off of this PR
https://github.com/stakwork/sphinx-mac/pull/12
will need to use the deleteLsat and getLsat endpoints on relay and we will also need to update how we save the lsat as we are not saving or sending the route paths just yet

hey @kevkevinpal I'm taking a look at this. I am able to get back the LSAT upon loading the webview and also decode its amount.
In my experiment I:

Authorized a budget of 100 sats
Got my LSAT from the server
Close the 2nd brain wndow
Re-open it
Call the /active_lsat endpoint
Decode the payment_request attached to it

When I do that, I see a request for what appears to be 10 sats. Is that correct? It seems that the budget itself is simply stored in persistent memory and that the LSAT just contains an invoice for the next request.
Can you explain the goal of this ticket more? Is the goal to delete an LSAT if it already exists at launch?

Hey @kevkevinpal I'm keen to do this but I'm not really sure how I would actually determine if the LSAT entitles them to second brain or whatever the credential is. Could you expand on that so I could do it?


fix fd in_http plugin keepalive_timeout option

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #965
Does this PR introduced a user-facing change?

None

Additional documentation, usage docs, etc.:

Leave the variable KeepLiveTimeout as it is to avoid user change. It may be better to rename it to KeepaliveTimeout


cmd/compile: = and := errors are unnecessarily different

% cat x.go
package p

func f() (int, int, int)

func g() {
	x, y := f()
	var a, b int
	a, b = f()
	_ = x
	_ = y
	_ = a
	_ = b
}
% go tool compile x.go
x.go:6:10: cannot initialize 2 variables with 3 values
x.go:8:9: cannot assign 3 values to 2 variables
% go1.17 tool compile x.go
x.go:6:7: assignment mismatch: 2 variables but f returns 3 values
x.go:8:7: assignment mismatch: 2 variables but f returns 3 values
%

In the old type checker, we worked to make sure that we always say the number on the left hand side of the assignment before the number on the right hand side. Originally the text was literally “2 = 3”, but that was a bit cryptic. Even so, is helps to avoid reversing the numbers entirely.
We should therefore reword the current x.go:8 message so that 2 appears before 3.
It would also help, I think, to restore the name of the function returning multiple values. That may not be clear on a complex line. The character positioning helps, of course, but it requires a tool to interpret. If you're looking at a list of errors, it helps more to see 'f' mentioned in every error that's about f.
Here's a real-world example that illustrates the usability difference better than the trivial case above:
% go test net/url
# net/url [net/url.test]
net/url/url.go:924:16: cannot assign 3 values to 2 variables
net/url/url.go:932:17: cannot initialize 2 variables with 3 values
net/url/url.go:1005:21: cannot assign 3 values to 2 variables
net/url/url_test.go:2068:16: cannot assign 3 values to 2 variables
FAIL	net/url [build failed]
FAIL
% go test -gcflags=-G=0 net/url
# net/url [net/url.test]
net/url/url.go:924:14: assignment mismatch: 2 variables but strings.Cut returns 3 values
net/url/url.go:932:14: assignment mismatch: 2 variables but strings.Cut returns 3 values
net/url/url.go:1005:19: assignment mismatch: 2 variables but strings.Cut returns 3 values
net/url/url_test.go:2068:14: assignment mismatch: 2 variables but strings.Cut returns 3 values
FAIL	net/url [build failed]
FAIL
%

/cc @griesemer @findleyr

Here's a nearby mini-bug:
net/http/client_test.go:434:23: cannot initialize 3 variables with 1 values

Note 'values' instead of 'value'. (This was a, b, c := f() for a single-result f.)


Tweak text removing "Don't want to"

Prerequisites checklist

[x] I have read the contributing guidelines.

What is the purpose of this pull request?
[x] Documentation update
[ ] Bug fix
[ ] New feature
[ ] Other, please explain:
What changes did you make? (Give an overview)
Changed the text on the home page to remove "Don't want to attend".
Is there anything you'd like reviewers to focus on?
This is a cool idea. It kicked off a lively discussion in our Slack group organizing the Bangkok.rb ruby meetup about offline vs online meetups. We don't want to discourage people from attending in-person meetups, so we think it good to emphasise people who cannot attend (e.g. no meetup in city, times of IRL meetup is inconvenient) rather than those who do not want to attend (it's intimidating, i'm scared of coronavirus) - it's tough enough to get people to attend as it is :) Let this not be seen as a criticism of the idea itself which i think is excellent!
Screenshots

Thanks! Can you run prettier pls?
yarn fix

done

@all-contributors please add @matthewmayer for content


Restrict which metadata fields are sent to outputs

The following outputs currently support enriching messages sent with metadata:

kafka
aws_s3
amqp_0_9
redis_hash
redis_streams
aws_sqs
gcp_pubsub
http_client

However, sometimes there are metadata fields we'd like to use for interpolation but not add to the resulting message sent over the output. In order to solve this we have three options:

Add a "private tier" to metadata given a certain key prefix (_ is a common prefix for private variable names), private metadata can be explicitly referenced but will always be excluded from outputs
Add a field include_meta_pattern which is a metadata key pattern to include in the output messages
Add a field exclude_meta_pattern which is a metadata key pattern to exclude in the output messages

Options 2 or 3 are "safer" in that they're capable of being implemented in a backwards compatible way, but also add another config field to all of the above outputs. Option 1 is passive and (I think) intuitive but could trip some users up that aren't expecting it, and in the unlikely case where a user wants to send metadata keys with our chosen prefix there wouldn't be a work around to solving it.
I'm currently leaning towards option 3.

Maybe having both 2 and 3 mutually exclusive would be a good solution: if we'd specify the include we'd just keep the ones matching the pattern if by the contrary we'd specify exclude we'd keep all metadata unless it'd match the pattern.

Starting off with just exclude_prefixes, which is a static list of key prefixes to ignore: https://github.com/Jeffail/benthos/commit/a4d9dfc23b6fcc13c28f6d45a4ea94eb6e28ca3e, we can expand that later if/when it doesn't satisfy a particular use case.
@dmoeff, we could potentially do things that way as messages carry a Context which we could add arbitrary values to. However, I'm keen to try and solve this without adding a new "tier" of metadata as I think it might be difficult to document the differences and it'd mean a whole new suite of tooling would need to be built out.
If this solution turns out to be confusing or awkward to deal with then we can revisit but I think this should be good enough for now.


support ALT+DPAD special handling

in the Android system, ALT+DPAD_LEFT is treated as HOME, ALT+DPAD_RIGHT as END, ALT+DPAD_UP as PAGE_UP and ALT+DPAD_DOWN as .. (surprise!) PAGE_DOWN. This is hardcoded in frameworks/base/core/java/android/text/method/BaseMovementMethod.java
    protected boolean handleMovementKey(TextView widget, Spannable buffer,
            int keyCode, int movementMetaState, KeyEvent event) {
        switch (keyCode) {
            case KeyEvent.KEYCODE_DPAD_LEFT:
                if (KeyEvent.metaStateHasNoModifiers(movementMetaState)) {
                    return left(widget, buffer);
                } else if (KeyEvent.metaStateHasModifiers(movementMetaState,
                        KeyEvent.META_CTRL_ON)) { // this works
                    return leftWord(widget, buffer);
                } else if (KeyEvent.metaStateHasModifiers(movementMetaState,
                        KeyEvent.META_ALT_ON)) { // <========!
                    return lineStart(widget, buffer);
                }
                break;

It would be truely great if ATE could act in the same manner.. :smiley:

Ok after having another look at it, here's some mockup code how to do it, referring to a new keyboard option "Translate Alt+DPAD":
    if (mTranslateAltDpad) {
        mKeyMap.put(KEYMOD_ALT | KEYCODE_DPAD_LEFT, "\033[1~");  // HOME
        mKeyMap.put(KEYMOD_ALT | KEYCODE_DPAD_RIGHT, "\033[4~"); // END
        mKeyMap.put(KEYMOD_ALT | KEYCODE_DPAD_UP, "\033[5~");    // PAGE_UP
        mKeyMap.put(KEYMOD_ALT | KEYCODE_DPAD_DOWN, "\033[6~");  // PAGE_DOWN
    else {
        mKeyMap.put(KEYMOD_ALT | KEYCODE_DPAD_LEFT, "\033[1;3D");
        mKeyMap.put(KEYMOD_ALT | KEYCODE_DPAD_RIGHT, "\032[1;3C");
        mKeyMap.put(KEYMOD_ALT | KEYCODE_DPAD_UP, "\033[1;3A");
        mKeyMap.put(KEYMOD_ALT | KEYCODE_DPAD_DOWN, "\033[1;3B");
    }


test: validate relationships after action changes

Didn't fix the bug.
I just wrote a simple failing test.
In create_with_default_comment action, I am setting default value of comment text attribute.
However, because of the order of validations, this form cannot be submitted.

The fix for this is complex. The way that forms provide intelligent errors up front is by "pre-validating" the inputs. If you want to handle that error you're going to need to add an error_handler, or when submitting use force?: true to force submission even though a child form is invalid.

generated?: true option can be a workaround for this? @zachdaniel

On the destination action (of the related resource), you could add allow_nil_input [:the, :action], would potentially be one way to solve it.


Webmentions modal should close when [esc] is pressed

Example URL: http://lewisnyman.co.uk/blog/indie-web-camp-2015/#webmentions

That means I have to use JS now :P booo


Allow env name to be alphanumeric

This PR is intended to support alphanumeric (not only letters) in ENV_SHORT_NAME and make 3 characters longer. In Azure, it allows alphanumeric in the naming of Storage Account, Blob, Container, Queue, Table and etc:
-Storage Account	3-24	lowercase 	alphanumeric

Blob Name	        1-1024	case-sensitive	any url char
Container Name	3-63	lowercase	alphanumeric and dash
Queue Name	3-63	lowercase	alphanumeric and dash
Table Name	        3-63	case-insensitive	alphanumeric

Moreover, ENV_NAME is used as Azure Resource Group Name.
it might container number of versioning (e.g. rg-v100).

Hello!  Closing this issue for now as it's been a few weeks, please re-open if this is still a concern.  Thanks!


Mac build_gallery failed, unable to find sdk 'iphoneos14.0'

https://ci.chromium.org/ui/p/flutter/builders/prod/Mac build_gallery/960/overview
    error: unable to find sdk 'iphoneos14.0' (in target 'video_player' from project 'Pods')
    error: unable to find sdk 'iphoneos14.0' (in target 'url_launcher' from project 'Pods')
    error: unable to find sdk 'iphoneos14.0' (in target 'Pods-Runner' from project 'Pods')
    error: unable to find sdk 'iphoneos14.0' (in target 'Reachability' from project 'Pods')
    error: unable to find sdk 'iphoneos14.0' (in target 'Flutter' from project 'Pods')
    error: unable to find sdk 'iphoneos14.0' (in target 'device_info' from project 'Pods')
    error: unable to find sdk 'iphoneos14.0' (in target 'Runner' from project 'Runner')
    error: unable to find sdk 'iphoneos14.0' (in target 'connectivity' from project 'Pods')
    error: unable to find sdk 'iphoneos14.0' (in target 'integration_test' from project 'Pods')

It seems like the tests is getting its properties from the system installation of xcode (new version), and not the sandbox version (which is older and does not contain the sdk)

It seems like the tests is getting its properties from the system installation of xcode (new version), and not the sandbox version (which is older and does not contain the sdk)

Another: https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket.appspot.com/8858373866854169456/+/steps/prepare_environment/0/steps/Build_gallery/0/steps/build_gallery/0/stdout
Did something change with this test?

Another: https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket.appspot.com/8858373866854169456/+/steps/prepare_environment/0/steps/Build_gallery/0/steps/build_gallery/0/stdout
Did something change with this test?

Also why is it passing on some bots and not others?  It passed on a re-run.

Also why is it passing on some bots and not others?  It passed on a re-run.

This is keeping the tree closed, it's failed 3 times in a row on the last commit.

This is keeping the tree closed, it's failed 3 times in a row on the last commit.

Also interesting it only seems to be failing on Mac build_gallery, not the many other thats are building iOS apps like Mac build_tests.

Also interesting it only seems to be failing on Mac build_gallery, not the many other thats are building iOS apps like Mac build_tests.

I temporarily turned these tests off so it doesn't keep the tree closed https://github.com/flutter/flutter/pull/73728

I temporarily turned these tests off so it doesn't keep the tree closed https://github.com/flutter/flutter/pull/73728

@keyonghan This isn't the same manifestation of Swift being missing, but maybe mac_build_gallery also needs the updated Xcode version? https://github.com/flutter/infra/pull/325

@keyonghan This isn't the same manifestation of Swift being missing, but maybe mac_build_gallery also needs the updated Xcode version? https://github.com/flutter/infra/pull/325

Eh nevermind that was a devicelab config, this is happening on luci.

Eh nevermind that was a devicelab config, this is happening on luci.


typography

Hi,
I get typography not supported error when I use -t serif option while changing the theme
Thanks

same question

https://github.com/transcranial/jupyter-themer/issues/33#issuecomment-417864100


Multi-directory PR creation with Terraform

Is there an existing issue for this?

[x] I have searched the existing issues

Package ecosystem
terraform
Package manager version
No response
Language version
No response
Manifest location and content before the Dependabot update
No response
dependabot.yml content
No response
Updated dependency
No response
What you expected to see, versus what you actually saw
https://github.com/dependabot/dependabot-core/issues/7547#issuecomment-2085316877
Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
No response

Closed and added to Multi-directory support bug with go, terraform, docker #9664


quick fix™: clean command triggers

I can't test this rn and made it in github editor (probably should have cloned it lol) but it looks right. Uses await Utils.clean(trigger) to clean command triggers and content.

Tested this on my test bot since brxxn was not able to. This is how it's looking at the moment: https://i.imgur.com/1XlhKwN.png Just needs a little bit of clean-up, but it appears to be fixed now and nothing exploded.

@Kitty-Therapist do i need to make changes?

@brxxn nah, you're fine as far that I can tell. If AE wants to make a change or request it, he will.

while yes the escape mentions would be nice to use, i'm just gona deploy this as a quick fix until i fix it properly due to how much this can be abused


Refactor Travis config.

r? @jamesmunns
(rust_highfive has picked a reviewer for you, use r? to override)

bors try

Added.

bors r+


this.getCustomData(...).forEach is not a function

"react-native-snap-carousel": "^3.1.0",

Got the same error on passing array of images from props having only one element. was working fine with static data.
TypeError: TypeError: this._getCustomData(...).forEach is not a function

This error is located at:
    in Carousel (at Slider.js:86)
    in RCTView (at View.js:35)
    in View (at Slider.js:85)
    in Slider (at ProductDetails.js:103)
    in RCTView (at View.js:35)
    in View (at ScrollView.js:1007)
    in RCTScrollView (at ScrollView.js:1147)
    in ScrollView (at ProductDetails.js:102)
    in RCTView (at View.js:35)
    in View (at ProductDetails.js:101)
    in ProductDetails (at SceneView.js:9)
    in SceneView (at StackViewLayout.tsx:899)
    in RCTView (at View.js:35)
    in View (at createAnimatedComponent.js:151)
    in AnimatedComponent (at StackViewCard.tsx:93)
    in RCTView (at View.js:35)
    in View (at createAnimatedComponent.js:151)
    in AnimatedComponent (at screens.native.js:71)
    in Screen (at StackViewCard.tsx:80)
    in Card (at createPointerEventsContainer.tsx:95)
    in Container (at StackViewLayout.tsx:971)
    in RCTView (at View.js:35)
    in View (at screens.native.js:101)
    in ScreenContainer (at StackViewLayout.tsx:383)
    in RCTView (at View.js:35)
    in View (at createAnimatedComponent.js:151)
    in AnimatedComponent (at StackViewLayout.tsx:379)
    in PanGestureHandler (at StackViewLayout.tsx:372)
    in StackViewLayout (at withOrientation.js:30)
    in withOrientation (at StackView.tsx:103)
    in RCTView (at View.js:35)
    in View (at Transitioner.tsx:267)
    in Transitioner (at StackView.tsx:40)
    in StackView (at createNavigator.js:61)
    in Navigator (at createKeyboardAwareNavigator.js:12)
    in KeyboardAwareNavigator (at SceneView.js:9)
    in SceneView (at createTabNavigator.js:39)
    in RCTView (at View.js:35)
    in View (at ResourceSavingScene.js:37)
    in RCTView (at View.js:35)
    in View (at ResourceSavingScene.js:26)
    in ResourceSavingScene (at createBottomTabNavigator.js:121)
    in RCTView (at View.js:35)
    in View (at screens.native.js:101)
    in ScreenContainer (at createBottomTabNavigator.js:111)
    in RCTView (at View.js:35)
    in View (at createBottomTabNavigator.js:110)
    in TabNavigationView (at createTabNavigator.js:197)
    in NavigationView (at createNavigator.js:61)
    in Navigator (at SceneView.js:9)
    in SceneView (at SwitchView.js:12)
    in SwitchView (at createNavigator.js:61)
    in Navigator (at createAppContainer.js:429)
    in NavigationContainer (at App.js:102)
    in RCTView (at View.js:35)
    in View (at createAnimatedComponent.js:151)
    in AnimatedComponent (at AnimatedSplash.js:117)
    in RCTView (at View.js:35)
    in View (at AnimatedSplash.js:113)
    in RCTView (at View.js:35)
    in View (at AnimatedSplash.js:110)
    in AnimatedSplash (at App.js:95)
    in App (at renderApplication.js:40)
    in RCTView (at View.js:35)
    in View (at AppContainer.js:98)
    in RCTView (at View.js:35)
    in View (at AppContainer.js:115)
    in AppContainer (at renderApplication.js:39)

This error is located at:
    in NavigationContainer (at App.js:102)
    in RCTView (at View.js:35)
    in View (at createAnimatedComponent.js:151)
    in AnimatedComponent (at AnimatedSplash.js:117)
    in RCTView (at View.js:35)
    in View (at AnimatedSplash.js:113)
    in RCTView (at View.js:35)
    in View (at AnimatedSplash.js:110)
    in AnimatedSplash (at App.js:95)
    in App (at renderApplication.js:40)
    in RCTView (at View.js:35)
    in View (at AppContainer.js:98)
    in RCTView (at View.js:35)
    in View (at AppContainer.js:115)
    in AppContainer (at renderApplication.js:39)


[libuv] add libuv for linux

I add support build for linux but I was not able to check on windows.
Can someone check if it still compile on windows ?
And on osx pls ?

It seem to be good

Please merge! 😸

Thanks for the PR and sorry for the delay! This is great to see!


"Open project" location in SqlProj for VSCode should be current workspace

Currently, the VS Code version of SqlProj opens to a different default folder (in my case, it's a source folder for the extension, so probably last-used or similar).
If a workspace is already opened, the target folder should be the workspace root.  If not, perhaps the user home directory.

Also occurring in "Select project location" for "Generate Project from OpenAPI/Swagger".  Either not possible with current VS Code APIs, or calls all need updating.


Incorrect offset used to mask GetKeyState in _sapp_win32_mods

_SOKOL_PRIVATE uint32_t _sapp_win32_mods(void) {
    uint32_t mods = 0;
    if (GetKeyState(VK_SHIFT) & (1<<31)) {
        mods |= SAPP_MODIFIER_SHIFT;
    }
...

This was segfaulting (EXCEPTION_ILLEGAL_INSTRUCTION) in the latest version of Zig in debug mode. Looks like the GetKeyState result is a short and it is being masked with a 32 bit number: https://docs.microsoft.com/en-us/windows/win32/api/winuser/nf-winuser-getkeystate
Changing it each of them to (1<<15) fixed it to me and seems to be consistent with some other usages of the function: https://stackoverflow.com/a/6332025

Yep, definitely a bug, thanks for catching that. I guess it works in C code because the 16-bit return value might be extended to a signed 32-bit value. Fix incoming.


Allow config parameter in the gnews.get_full_article()

I using GNews get_full_article() function to extract the top_image from the Article. However, when I run this on my production server it throws me the below error:
ERROR: Article download() failed with HTTPSConnectionPool(host='indianexpress.com', port=443): Max retries exceeded with url: /article/idea-exchange/gautam-gambhir-idea-exchange-first-challenge-mcd-polls-change-narrative-bjp-doesnt-do-anything-8158944/ (Caused by ProxyError('Cannot connect to proxy.', OSError('Tunnel connection failed: 403 Forbidden'))) on URL https://indianexpress.com/article/idea-exchange/gautam-gambhir-idea-exchange-first-challenge-mcd-polls-change-narrative-bjp-doesnt-do-anything-8158944/
I searched through Google and ended up with this solution:
from newspaper import Article
from newspaper import Config

user_agent = 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:78.0) Gecko/20100101 Firefox/78.0'
config = Config()

config.browser_user_agent = user_agent

url = "https://www.chicagotribune.com/nation-world/ct-florida-school-shooter-nikolas-cruz-20180217-story.html"

page = Article(url, config=config)

page.download()
page.parse()
print(page.text)

As per the code above, I need to mention the user agent and get it assigned to config.browser_user_agent to prevent the server from getting banned. However, if I want to use gnews.get_full_article() I am not able to specify the config parameter inside. Is there any provision to mention this parameter? Am I missing something?

https://github.com/ranahaani/GNews/projects/2#card-86371640


Added custom scrollbar

Related Issue
Added a custom scrollbar to the webpage to improve the overall design.

Closes: #391

Describe the changes you've made
The basic browser scrollbar is changed to a custom theme-based scrollbar to improve the UI/UX of the web page.
Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update.
[ ] This change requires a documentation update

How Has This Been Tested?
I have setup the website on my local environment and then run it and tested the changes.
Checklist

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented on my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Screenshots (if applicable)

Original
Updated

Code of Conduct

[x] I agree to follow this project's Code of Conduct

I have checked, and the code works on most modern browsers. However, it's always a good practice to write code that ensures cross-browser compatibility, as it helps maintain consistent rendering and functionality across different browsers.
I have attached the sample code for your reference. Please review it and let me know your thoughts. Additionally, I recommend testing the code on the respective browsers to ensure it works as expected.
/* Firefox */
* {
  scrollbar-width: thin;
  scrollbar-color: #8034e4 #E638EAFF;
}

/* Internet Explorer and Microsoft Edge */
*::-ms-scrollbar {
  width: 16px;
}

*::-ms-scrollbar-track {
  background: transparent;
}

*::-ms-scrollbar-thumb {
  background: linear-gradient(#8034e4, #E638EAFF);
}

Hello! This is a great advice! 💯

Hello @abhinav-m22 :wave: Can you add @Anmol-Baranwal suggestion so that we merge your PR ?

Hey @abhinav-m22, do you need help? (:

Actually I was a bit inactive since last few days so I couldn't reply... I'll implement the changes till tomorrow and let you know for any help
Thanks!!

Hey guys @mkubdev @Mustafiz04 @vinzvinci @Anmol-Baranwal I've made the required changes!
Please have a look now

Please check now


Inconsistency in Steam overlay and bugging out friend list

Patched using Adwaita v3.1
OS: Windows 11
Steam is on Stable Channel

The theme overall looks very good and consistent

But when it comes to the overlay and the friendlist, it looks buggy and seems like the overlay did not get applied

Overlay is not styled yet, it should look as it does in the default Steam theme excluding some widgets which were already styled elsewhere (like avatars and friends window on your screenshot).

There's not enough space for the window title when window is so narrow. Also window control buttons take up a lot of space.
I guess most of the buttons should be moved to a separate toolbar below the window title if there's not enough space, I'm not yet sure how hard it would be to implement.
For now you can make the friends window wider, dock chats to the friends list in Steam settings, set window control buttons to use GNOME or macOS style, or just hide some of the buttons.


CCO-412: jump azureWorkloadIdentity to the head of the line.

The azureWorkloadIdentity feature needs to be enabled by default in 4.14. However, lots of changes have happened in the API prior to the relevant PR merging that we do not want in 4.14. This PR re-orders all of those merge commits such that the azureWorkloadIdentity PR is at the head of the line. As a result, this change can now be brought into cluster-config-operator without bringing int he other things.

/assign @JoelSpeed

This might not be the right approach.


Username available?

Would you be happy to let me use the username "Embyr" instead? I am pretty active on here mostly in private repos but I'm standardizing all my stuff on the embyr.sh domain and all my handles to "Embyr " where I can.
I'm happy to compensate you for it.

I apologize it has taken me so long to reply and thank you for your interest but I cannot give up the name.
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Wednesday, May 26th, 2021 at 10:21 AM, Hannah Crawford @.***> wrote:

Would you be happy to let me use the username "Embyr" instead? I am pretty active on here mostly in private repos but I'm standardizing all my stuff on the embyr.sh domain and all my handles to "Embyr " where I can.
I'm happy to compensate you for it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Kandji app docs

Purpose of this pull request
This pull request is to create Kandji apps docs.
Select the type of change

[ ] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions, updating sections
[x] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)
Bronze App - Kandji: Public doc generation

@JV0812, I'm merging this PR at the request of @ankitgoelcmu.


why quantize opt-1.3b or llama 7b with W8A8 config loss is nan?

the quant command is
python main.py --model llama-7B --epochs 20 --calib_dataset c4 --nsamples 128 --batch_size 1 --seed 0 --eval_ppl --wbits 4 --abits 8 --lwc --let --output_dir quant_models/llama-7b-w4a8 --save_dir fake_models/llama-7b-w4a8

@ChenMnZ

Training with 8bit quantization would cause NAN loss in mixer-precision training.

Training with 8-bit quantization leads to NAN loss in mixed-precision training.
Please refer to the updated code; we've implemented full-precision training for quantization bits greater than 8, enabling successful W4A8 training.

Training with 8-bit quantization leads to NAN loss in mixed-precision training. Please refer to the updated code; we've implemented full-precision training for quantization bits greater than 8, enabling successful W4A8 training.

thx a lot.

Training with 8-bit quantization leads to NAN loss in mixed-precision training.

“Training with 8-bit quantization leads to NAN loss in mixed-precision training.”
why is this happening?
@ChenMnZ

@brisker
During the backward of mixed-precision training, gradient is calculated through float16 format, which has a narrower representation scope.
For the 8-bit quantization, the step size of quantization is 2^8-1. Such large step size during backward would lead overflow for float16.

@ChenMnZ
here:   https://github.com/OpenGVLab/OmniQuant/blob/main/quantize/omniquant.py#L192

why are shifts not used in Llama model equivalent transformation?

@ChenMnZ here: https://github.com/OpenGVLab/OmniQuant/blob/main/quantize/omniquant.py#L192

why are shifts not used in Llama model equivalent transformation?

Also intrigued by the rationale behind this question.

The original LLaMa models do not contain bias parameters, and we find that shift operation can not bring benefit to weight-only quantization. Therefore, we only activate shifts equivalent transformation in the weight-activation quantization for LLaMa models.
As for the initialization, the initialization of shifts is crucial for OPT but negligible for LLaMa, so we simple init it as 1 for LLaMa.

我们试了一下4A4W的LLAMA2，为什么即使在8bit以下，如果打开半精度训练，损失依然是nan？另外，当我们使用全精度训练时，wikitext2的ppl是11.，而不是论文中的5.，我们想确认一下现在代码的版本是否可以正确运行LLAMA2，感谢！

@Xingrun-Xing
我刚跑了一遍LLaMa-2-7b w4a16，现在的代码版本可以正确运行LLAMA2，你可以提供你的训练指令以进行进一步查看。

@Xingrun-Xing 我刚跑了一遍LLaMa-2-7b w4a16，现在的代码版本可以正确运行LLAMA2，你可以提供你的训练指令以进行进一步查看。

感谢回复 我刚才重新拉现在的代码又跑了一遍，用的和你一样的命令，结果已经正常了。另外有一个小问题，我们的跑的结果略有差异，可能的原因是什么呢，请问你的transformers版本是多少

你好，发现另有一个问题，我们试了一下/script下的各个脚本，发现W6A6不能正常训练 LLAMA2，其他脚本都正常

@Xingrun-Xing

transformer version 4.36.0。结果差异可以尝试下载训练好的量化参数进行测试，排查是否环境存在问题。若是环境没问题，训练好的量化参数应该能得到和论文中差不多的结果。
W6A6遇到NAN尝试加入--deactive_amp应该可解决

W6A6遇到NAN尝试加入--deactive_amp应该可解决

May I ask if you know why torch.amp does not work with W6A6? Thank you for the great work.

Dear sir, when using w6a6.sh, I have encountered the same problem that the norm is nan, after using "--deactive_amp"，in QuantLinear'forward(), the line of  "out = self.fwd_func(input, weight, bias, **self.fwd_kwargs)" would cause "mat1 and mat2 must have the same dtype, but got Float and Half". However, after I change both weight and input into float32, it would cause "self and mat2 must have the same dtype, but got Half and Float", when I change them both into float16, it would cause "self and mat2 must have the same dtype, but got Float and Half". Using autocast before the line woudn't cause this problem ,but the norm would still be nan. Should I change other code after using the "--deactive_amp"?Thank you @ChenMnZ

@Wotoosh Do you use the packages with recommend version?
Additionally, recommending you to use our new algorithm, PrefixQuant, which is stronger and without NAN problem.

some versions are higher than the least-needed, but thanks, your work is really strong


Revise Event Scheduling support

There are roughly two main tasks when it comes to revising the EventScheduler support within Axon Framework.
Firstly, the EventScheduler API should comply with the new Unit of Work (as introduced in #2953) and follow a similar layering solution as performed for the Command Bus in pull request #3003.
This part of the issue should be regarded as a must, as without these changes, Event Scheduling would no longer fit within the design of Axon Framework 5.
Secondly, we want to reimagine the placement of event scheduling logic.
Right now, the implementation schedules events in a storage layer separate from the Event Store, as is done through Quartz, JobRunr, and db-scheduler.
The only solution that combines the concerns of event storage/publication and scheduled event storage/publication is Axon Server.
Thus, the reimagining comes in on this front: shouldn't we make scheduled event storage a first-class citizen of every Event Store?
Although important to discuss within the development of Axon Framework 5 due to the breaking changes this would introduce, this reimagining is not a must for the time being.

We need to add new scheduling methods overloads in order to pass Metadata along with payload, in order to schedule without using static factory methods for messages.


Backtesting Idea/Question

Hey everyone,
I'm not the strongest Python person yet but I wanted to know if this could be added to the code.

In the screenshot, I have --simstartdate 2020-01-01. It goes between 1/1/2020 - 1/13/2020 due to the 300 max internal.
My question is, is there any way to instead have the date period go from 1/1/2020 - Today? Have the system calculate how many 300 intervals are needed and do the entire time period. I have some ideas for back testing and like how this visually is displayed and would like to do the testing this way if possible. I like that it shows where the buy and sell areas are on each line and then if you are positive or negative on that position. I also want to see how the smart switching would act in different time periods during the back testing.
I know Mike has written a Backtesting Trading Strategies article on how to play out different ideas but I'm hoping this idea could be implementing to give everyone options.

this works when using the following on the command line:
python3 pycryptobot.py --sim fast-sample simstartdate 2020-01-01 simenddate now
for long periods such as that it can take a little while for the bot to start analysis as it has to gather lots of data.
Binance may not return any data for anything over 6 months old but coinbase will.

Hey,
Your example is giving me this error.
pycryptobot.py --market BTC-USD --sim fast-sample --autorestart --simstartdate 2020-01-01 --simenddate now

I have not made any changes to the source code. Am I doing something wrong here?
If I put the command line code like you did in your example then I get this. It does not do what I want.


INT-2957: manage attribute id

INT-2957
issue: https://github.com/tinymce/tinymce-webcomponent/issues/32

@spocke I added the changelog entry, have a look at it


Two errors on first run

Parent of RectTransform is being set with parent property. Consider using the SetParent method instead, with the worldPositionStays argument set to false. This will retain local orientation and scale rather than world orientation and scale, which can prevent common UI scaling issues.
UnityEngine.Transform:set_parent(Transform)
DebugPlusNS.LogDrawer:Start() (at Assets/DebugPlus/Scripts/LogDrawer.cs:25)
Parent of RectTransform is being set with parent property. Consider using the SetParent method instead, with the worldPositionStays argument set to false. This will retain local orientation and scale rather than world orientation and scale, which can prevent common UI scaling issues.
UnityEngine.Transform:set_parent(Transform)
DebugPlusNS.LogDrawer:Start() (at Assets/DebugPlus/Scripts/LogDrawer.cs:42)
first error only once and the other whiule running, many times

Thanks. Just downed it. Shall use and revert!
On Fri, 15 May 2020 at 01:03, Benoît<EMAIL_ADDRESS>wrote:

Closed #8 https://github.com/benoit-dumas/DebugPlus/issues/8 via 9b8dbcb
https://github.com/benoit-dumas/DebugPlus/commit/9b8dbcb723b8b9cdcf5cfcd861e553617fc6f110
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/benoit-dumas/DebugPlus/issues/8#event-3338116571, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDNKRSI6F5VC3PYBC5S6CTRRRBRNANCNFSM4MSQPQZQ
.

--
Kalasuri Diliup Gabadamudalige
https://dahamgatalu.wordpress.com/
http://soft.diliupg.com/
http://www.diliupg.com

This e-mail is confidential. It may also be legally privileged. If you are
not the intended recipient or have received it in error, please delete it
and all copies from your system and notify the sender immediately by return
e-mail. Any unauthorized reading, reproducing, printing or further
dissemination of this e-mail or its contents is strictly prohibited and may
be unlawful. Internet communications cannot be guaranteed to be timely,
secure, error or virus-free. The sender does not accept liability for any
errors or omissions.


Show the notification

Is there a way to show the notification on the screen,
and not only on the Notification Center?

What would you like to achieve? Do you have an example (screenshot)?


Fixed position of Google Analytics script and added a property default

Google Analytics wants its script be positioned inside the  tag. Moreover, the GOOGLE_ANALYTICS_UNIVERSAL_PROPERTY is by default "auto" as given by the GA configuration page.

Hi!
First of all, thanks for your pull request! I appreciate the contribution, but I cannot accept this PR at the moment. It's with heavy heart that I have decided that I can no longer maintain this theme due to lack of time and changing interests. To makes sure the theme lives on however and not goes to waste, I've donated it to the Pelican community. It can now be found in the official pelican-themes repository. I'd suggest you re-submit your pull request there.
Sorry for the inconvenience!


Release on galaxy?

Hi,
I was trying to set up a requirements.txt file for ansible roles and found that I either suck at searching or this role is not uploaded on galaxy.ansible.com. Is there any reason not to do so? If not, could someone register it there?
Thanks in advance!

+1 to this. I attempted to add to ansible galaxy myself but it requires the Readme to be renamed to README https://github.com/ICTO/ansible-jenkins/pull/26

ping, anyone willing to make that happen?

This role is now available on ansible galaxy, closing this issue
https://galaxy.ansible.com/list#/roles/4269

Many thanks!!


ARC: west: mdb runner: fix folder where MDB is run

Fix folder where mdb is run by changing it to the build folder explicitly. Otherwise mdb will store the .sc.project folder in the place where twister is launched - so .sc.project folder will be shared across the multiple twister runs and across multiple processes in one twister launch which causes unexpected MDB behavior.
For that we add **kwargs argument to popen_ignore_int method so we can pass a cwd argument to the Popen constructor.

@evgeniy-paltsev Can you please create an Issue for this PR.


🐛 [Bug]: Tree树形控件的current-change事件传递的节点状态信息错误

Version
3.10.1
Vue Version
3.2.31
Link to minimal reproduction

Step to reproduce
勾选"一级 2"节点，画面显示已勾选，但从currentNode里获取到的状态还是false。
What is expected
No response
What is actually happening
No response
Any additional comments (optional)
No response

您好，感谢你的意见，我们内部评估下~~


Add test for memory leaks

Running the test against PHP 7.1 with debug enabled to check for memory leaks.

The php71-debug image is missing composer. Working on it.


Input freeze on heavy output

Windows Terminal version
1.16.10261.0
Windows build number
10.0.19045.0
Other Software
No response
Steps to reproduce
Produce heavy output (e.g. big compile job, cat on very long file).
Expected Behavior
Responsive Windows Terminal even when heavy output is in place.
Actual Behavior
Windows Terminal becomes unresponsive while there is a heavy output. For example, I am unable to even abort a longer compile job that produces heavy output, i.e. CTRL+{C,D,...} don't work anymore and I have to wait for the output to stop.
Another example is Linux' tmux. If I have a split window open and start compiling on one side and working on the other, as soon as the output starts my session hangs and I am unable to interact with the terminal (till the end of compilation). I tried the same steps using PowerShell, where it works fine.

Thanks for the report! This is a part of a broader class of issues that we're tracking over in #6258. Marking as a /dup of #6258.


PHP 8 support

https://github.com/felixkiss/uniquewith-validator is still php 7 bound, maybe drop the dependency as it seems stale.
Sadly like this your fantastic lib isn't compatible with php 8.

felixkiss/uniquewith-validator is not required by this package, in fact none of our packages require it.
We're using a forked version rinvex/tmp-felixkiss-uniquewith-validator for all latest releases which is compatible with PHP v8.
Make sure you're trying latest release of the package, and let me know if you still believe there's a compatibility issue.

felixkiss/uniquewith-validator is not required by this package, in fact none of our packages require it.
We're using a forked version rinvex/tmp-felixkiss-uniquewith-validator for all latest releases which is compatible with PHP v8.
Make sure you're trying latest release of the package, and let me know if you still believe there's a compatibility issue.

I see, thanks for taking the time. I've went down my composer requirements rabbit hole and found the issue being that you locked the required php version to v8.0.0 here : https://github.com/rinvex/tmp-watson-validating/blob/master/composer.json#L17 you probably want this to be more open to future php 8 versions.
But i'm on 8.0.1 by now system wide.
created a PR: https://github.com/rinvex/tmp-watson-validating/pull/1

I see, thanks for taking the time. I've went down my composer requirements rabbit hole and found the issue being that you locked the required php version to v8.0.0 here : https://github.com/rinvex/tmp-watson-validating/blob/master/composer.json#L17 you probably want this to be more open to future php 8 versions.
But i'm on 8.0.1 by now system wide.
created a PR: https://github.com/rinvex/tmp-watson-validating/pull/1

Thank you for catching this, PR merged 👍

Thank you for catching this, PR merged 👍


New PR to test SignCLA at 2018-02-22 13:41:10.96136268 -0800 PST m=+6.315430770

Body of PR

unsigned-fail-staging: test succeeded


Use overmind if available

Use overmind instead of foreman, if available
Overmind is a process manager (like foreman) for Procfile-based
applications. The advantage is that it allows to use debuggers like
pry or binding.irb.
(Maybe this should be a PR against Rails?)

Use overmind instead of foreman, if available

Meaning, we'd check to see if Overmind is available on the user's system?
Currently, Rails generates ./bin/dev for applications using jsbundling-rails.
We could probably modify that file to check for the existence of Overmind.

@stevepolitodesign yes! I forgot to attach the code for it:
#!/usr/bin/env sh

# Magic happens here
if command -v overmind &> /dev/null
then
  overmind start -f Procfile.dev "$@"
  exit $?
fi

if ! gem list foreman -i --silent; then
  echo "Installing foreman..."
  gem install foreman
fi

exec foreman start -f Procfile.dev "$@"


flutter test does not understand concurrency

Possible fix: Mimic https://github.com/flutter/flutter/pull/115160
Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
flutter test
Expected results
Tests should be executed one-by-one, since we specify concurrency=1 in dart_test.yaml
Actual results
They are executed parallelly.

flutter test
00:02 +0: /Users/tom/RefCode/bug/test/c_test.dart: hello this is test
c_test is running (i=0)
00:02 +0: /Users/tom/RefCode/bug/test/b_test.dart: hello this is test
b_test is running (i=0)
00:02 +0: /Users/tom/RefCode/bug/test/a_test.dart: hello this is test
a_test is running (i=0)
00:03 +0: /Users/tom/RefCode/bug/test/c_test.dart: hello this is test
c_test is running (i=1)
00:03 +0: /Users/tom/RefCode/bug/test/b_test.dart: hello this is test
b_test is running (i=1)
00:03 +0: /Users/tom/RefCode/bug/test/a_test.dart: hello this is test
a_test is running (i=1)
00:03 +0: /Users/tom/RefCode/bug/test/c_test.dart: hello this is test
c_test is running (i=2)
00:03 +0: /Users/tom/RefCode/bug/test/b_test.dart: hello this is test
b_test is running (i=2)
00:03 +0: /Users/tom/RefCode/bug/test/a_test.dart: hello this is test
a_test is running (i=2)
00:04 +0: /Users/tom/RefCode/bug/test/c_test.dart: hello this is test
c_test is running (i=3)
00:04 +0: /Users/tom/RefCode/bug/test/b_test.dart: hello this is test
b_test is running (i=3)
00:04 +0: /Users/tom/RefCode/bug/test/a_test.dart: hello this is test
a_test is running (i=3)
00:04 +0: /Users/tom/RefCode/bug/test/c_test.dart: hello this is test
c_test is running (i=4)
00:04 +0: /Users/tom/RefCode/bug/test/b_test.dart: hello this is test
b_test is running (i=4)
00:04 +0: /Users/tom/RefCode/bug/test/a_test.dart: hello this is test
a_test is running (i=4)
00:05 +0: /Users/tom/RefCode/bug/test/c_test.dart: hello this is test
c_test is running (i=5)
00:05 +0: /Users/tom/RefCode/bug/test/b_test.dart: hello this is test
b_test is running (i=5)
00:05 +0: /Users/tom/RefCode/bug/test/a_test.dart: hello this is test
a_test is running (i=5)
00:05 +0: /Users/tom/RefCode/bug/test/c_test.dart: hello this is test
c_test is running (i=6)
00:05 +0: /Users/tom/RefCode/bug/test/b_test.dart: hello this is test
b_test is running (i=6)
00:05 +0: /Users/tom/RefCode/bug/test/a_test.dart: hello this is test
a_test is running (i=6)
00:06 +0: /Users/tom/RefCode/bug/test/c_test.dart: hello this is test
c_test is running (i=7)
00:06 +0: /Users/tom/RefCode/bug/test/b_test.dart: hello this is test
b_test is running (i=7)
00:06 +0: /Users/tom/RefCode/bug/test/a_test.dart: hello this is test
a_test is running (i=7)
00:06 +0: /Users/tom/RefCode/bug/test/c_test.dart: hello this is test
c_test is running (i=8)
00:06 +0: /Users/tom/RefCode/bug/test/b_test.dart: hello this is test
b_test is running (i=8)
00:06 +0: /Users/tom/RefCode/bug/test/a_test.dart: hello this is test
a_test is running (i=8)
00:07 +0: /Users/tom/RefCode/bug/test/c_test.dart: hello this is test
c_test is running (i=9)
00:07 +0: /Users/tom/RefCode/bug/test/b_test.dart: hello this is test
b_test is running (i=9)
00:07 +0: /Users/tom/RefCode/bug/test/a_test.dart: hello this is test
a_test is running (i=9)
00:07 +3: All tests passed!

Code sample
Code sample
dart_test.yaml
concurrency: 1

test/a_test.dart
import 'dart:io';
import 'package:flutter_test/flutter_test.dart';
void main() {
  test('hello this is test', () {
    for (var i = 0; i < 10; ++i) {
      print('a_test is running (i=$i)');
      sleep(const Duration(milliseconds: 500));
    }
  });
}

test/b_test.dart
same, just change the name in "print", i.e.
import 'dart:io';
import 'package:flutter_test/flutter_test.dart';
void main() {
  test('hello this is test', () {
    for (var i = 0; i < 10; ++i) {
      print('b_test is running (i=$i)');
      sleep(const Duration(milliseconds: 500));
    }
  });
}

test/c_test.dart
similar to b_test.dart
import 'dart:io';
import 'package:flutter_test/flutter_test.dart';
void main() {
  test('hello this is test', () {
    for (var i = 0; i < 10; ++i) {
      print('c_test is running (i=$i)');
      sleep(const Duration(milliseconds: 500));
    }
  });
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

Thanks for the report. Seeing the same behavior as reported.

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.7.12, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.7.12 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4d9e56e694 (3 days ago), 2023-04-17 21:47:46 -0400
    • Engine revision 1a65d409c7
    • Dart version 2.19.6
    • DevTools version 2.20.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.10.0-19.0.pre.28, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.10.0-19.0.pre.28 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 707261fa05 (40 minutes ago), 2023-05-03 00:08:20 -0400
    • Engine revision 228da31894
    • Dart version 3.1.0 (build 3.1.0-66.0.dev)
    • DevTools version 2.23.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.


Corda Enterprise option for Float in separate cluster

Description
As a developer I want to create an option for enabling Firewall (bridge and float) and DMZ in separate cluster.
https://docs.corda.net/docs/corda-enterprise/4.4/node/corda-firewall-component.html#embedded-developer-node-node--artemis--internal-bridge-no-float-no-dmz
https://docs.corda.net/docs/corda-enterprise/4.4/node/corda-firewall-component.html#dmz-ready-node--bridge--float
Acceptance Criteria

Firewall enable should create network Bridge and Float
Firewall disable should create simple self container node with no firewall
DMZ enable should create DMZ in separate cluster.

This needs to be architected first and then relevant issues needs to be created

This issue has been done with the PR https://github.com/hyperledger-labs/blockchain-automation-framework/pull/1202

This issue has been done with the PR https://github.com/hyperledger-labs/blockchain-automation-framework/pull/1202


fix(logger): remove slashes before path

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[BUG]Resume copying will copy the same file twice or more.

Why are you showing your pirated movie?


Remove the discovery TTL consul health checks

We are misusing the Consul Health Check functionality to register a bunch of TTL checks which are not actually needed.
We can supersede these with a single trento-agent health check reporting the status of the agent itself (instead of the result of the HA Configuration Checks, which is going to be run by the Runner component instead).
Related to #89

@stefanotorresi what should the Trento Health check depend on?

@stefanotorresi what should the Trento Health check depend on?

Well, my idea would be that if any of the discovery loops fails, than this single check fails with some output about the specific error that occurred.
IOW I would like all the Consul health checks that we currently have for the discovery loops consolidated under a single one.
That said, if you have other ideas on how to self-check the Trento Agent, I'm all ears!

@stefanotorresi I have a PR ready for this, just holding it after the runner branch merge, i basically did what you just described but changed a little bit when the discoveries return errors (e.g. no error if nothing is discovered, just a negative result)


Fix issue breaking cudagraphs

@ngimel figured this one out.
The errors we were seeing from cudagraphs capture were coming from cuStreamGetCtx which is not allowed while a stream is capturing.
It appears the result of cuStreamGetCtx() isn't even used, so I believe it can just be removed.

@ptillet this still looks a bit sus, I don't see where a per-device cache of compiled functions is kept, and it looks like if someone compiled a function for device 0 during warm-up, and then attempted to run it on device 1, it would hit all the caches (key would match, function would be non-null), but then it would error out (I might be missing something in the code).

Hmmm, my understanding is that CUDA modules are loaded on a per context basis, so we shouldn't need to cache them at a lower level than that. But now indeed the code isn't caching the compiled function with respect to the current cuContext, and that is certainly a problem

Yeah cuda modules are loaded on a per-context basis, but since current context it not part of the cache key, we'll get spurious matches and errors.

Yeah, I agree. I initially did that, but it became dead code when I switched from C++ to C to lower compilation time and increase portability. Now I'm working on re-factoring the JIT to decouple the kernel entry point from the cubin (so that different kernels with the same signature can share the same .so entry point). I will also re-incorporate context-dependent key to resolve the problem that you mentioned.


Shortcut for %preview --limit to show all rows

Currently --limit is default to 200 rows. I tried to use --limit -1 to show all rows (vtools convention) but it shows all but the last row. --limit -2 will show all but the last 2 rows, etc. Is there a shorthand to show all rows? Shall we make behavior of negative --limit well defined?

The parameter is basically sent to DataFrame.head and the behavior of negative index is undefined. I have adjusted the behavior of --limit with negative index to show all records.


Add Check view Page

When this is mounted a modal is shown. This modal have an icon/image, a title, content and a button:

Mobile
Desktop

Notice:

Modal have a blueish overlay over content
Doesn't have cancel/close actions. Seems like overlay should close it

After clicking button in modal you will see a Back button, some Title and subtitle, an Image, then a Form which have a Title, an input and a button, like this:

Mobile
Desktop

Notice:

Layout elements (lets call it BLUE and BG) changes a lot. BLUE loss the border-radius and BG changes color! Also title of the form changes color
Image changes align from start to end in her div
Back button gains a Text (why not have it across all sizes?
This should show input errors (ticket doesn't exists on DB, format not allowed, others TBD)
This should show error toast for server errors (server unavailable, server error), others TBD) or Client High Severity errors.
Should Button be disabled if no Ticket given?
Should button have a sending state?
What happens when Ticket state changes to destination_reached or delivery_confirmed? Changes the view/show modal?

Clicking Button should redirects to Map view with ticket as param

What do you think transforms the enunciated to task? Like this:

[ ] When this is mounted a modal is shown. This modal have an icon/image, a title, content and a button:

Mobile	Desktop

Notice:

Modal have a blueish overlay over content
Doesn't have cancel/close actions. Seems like overlay should close it

[ ] After clicking button in modal you will see a Back button, some Title and subtitle, an Image, then a Form which have a Title, an input and a button, like this:

Mobile	Desktop

Notice:

Layout elements (lets call it BLUE and BG) changes a lot. BLUE loss the border-radius and BG changes color! Also title of the form changes color
Image changes align from start to end in her div
Back button gains a Text (why not have it across all sizes?
This should show input errors (ticket doesn't exists on DB, format not allowed, others TBD)
This should show error toast for server errors (server unavailable, server error), others TBD) or Client High Severity errors.
Should Button be disabled if no Ticket given?
Should button have a sending state?
What happens when Ticket state changes to destination_reached or delivery_confirmed? Changes the view/show modal?

[ ] Clicking Button should redirects to Map view with ticket as param

Thanks @Katsu08


tsv-summarize: doesn't support windows line-ending

When aggregating the last column, the prebuilt binary for linux (v1.1.15) returns the following error:

' when converting from type const(char)[] to type doubleUnexpected '
File: dummy.tsv Line: 2

I've used tsv-append, tsv-uniq and tsv-select, and each works just fine. After updating line-endings to unix, tsv-summarize works.

Thanks for the report and for trying the tools.
The only tool explicitly supporting Windows line-endings on Linux is csv2tsv. You are correct, tsv-append works fine, and a couple other tools as well, but it's more accidental than by design.
What's going on is that the tools are using D standard library functions for reading lines, these functions assume unix line endings on unix platforms. If the file has Windows line ending (a \r\n pair), the line is left with an extraneous \r character at the end of the last field. If this extraneous \r interfers with processing, the tool doesn't work.
In the case of tsv-summarize and tsv-filter, if they try to interpret that the last field as numeric value, the conversion will fail. However, even if they don't perform a conversion, the tools are necessarily working correctly. tsv-select for example, isn't really preserving Windows line endings.
As an example:
$ # This outputs the Windows line endings
$ echo $'AA\tXX\tYY\r\nBB\tXX\tYY\r' | grep $'YY\r'
AA	XX	YY
BB	XX	YY
$ # Select the last field.
$ echo $'AA\tXX\tYY\r\nBB\tXX\tYY\r' | tsv-select -f 3
YY
YY
$ # Grep shows the Windows line ending
$ echo $'AA\tXX\tYY\r\nBB\tXX\tYY\r' | tsv-select -f 3 | grep $'YY\r'
YY
YY
$ # Select the second field
$ echo $'AA\tXX\tYY\r\nBB\tXX\tYY\r' | tsv-select -f 2
XX
XX
$ # Grep shows its not a Windows line ending.
$ echo $'AA\tXX\tYY\r\nBB\tXX\tYY\r' | tsv-select -f 2 | grep $'XX\r'

I'm not inclined to add support for Windows line endings on Unix platforms. The dos2unix tool is a good tool for this and fits the pipeline approach being used by the tools.
However, something is going wrong with the error message formatting, and the error message should identify a Windows line ending as a likely problem. The documentation for the tools should also discuss line endings. I'll have to look into both of these.
Regarding csv2tsv - This tool explicitly supports Windows line-endings because they are commonly used in many programs that generate CSV files.

The badly formatted error message is due to the \r character being included in the error message. I'll have to fix it.

Thanks for your explanation and the great tools.
As you said, I used dos2unix to convert the line endings, so this issue wasn't deterrent to my work. Updating the documentation and especially the error message is more than enough to solve this issue.

Current plan: On Unix builds, check for Windows/DOS line endings when processing the first line of a file. That should handle most cases prior to ever hitting the error message. Regarding the poor error message format: There's an open D bug for it: https://issues.dlang.org/show_bug.cgi?id=17708

Addressed by PR #103, merged to master. Will be included in the next release.


Frozen string failure in output

Summary
A random run ended up with:
can't modify frozen String (RuntimeError)
/project/path/vendor/bundle/ruby/2.3.0/gems/cucumber-3.1.0/lib/cucumber/formatter/html.rb:571:in `snippet'
/project/path/vendor/bundle/ruby/2.3.0/gems/cucumber-3.1.0/lib/cucumber/formatter/html.rb:293:in `extra_failure_content'
...

Possible Solution
This error doesn't seem to make any sense.

if the string is appended there, the converter is NullConverter
in that case highlighted == raw_code
raw_code comes from snippet_for
snippet_for returns an array

Unless something, somewhere overrides the behaviour, this can't happen 🤔But this project doesn't override anything in related classes.
Steps to Reproduce (for bugs)
Unfortunately the ordering is random and I can't reproduce it.
Your Environment

Version used: 3.1.0
Operating System and version: Ubuntu trusty, ruby 2.3
Link to your project:

No, I don't have the Syntax gem

@viraptor I've created a PR with the intention of preparing a new release in the next few weeks.
In the meantime, I would recommend either temporarily monkey-patching Cucumber::Formatter::Html.snippet as per https://github.com/cucumber/cucumber-ruby/pull/1287 or installing the Syntax gem. Once we've released a new version of cucumber-ruby, you will be able to remove the monkey patch or uninstall the Syntax gem. :)


[FLINK-28941][Runtime/Checkpointing] Add concurrent checkpoint support in Operator Coordinator

What is the purpose of the change
#20275 has improved the checkpoint behavior of OperatorCoordinator, guaranteeing the distributed consistency of operator events sent from the coordinator to its subtasks. However, while the previous implementation only requires that there is no concurrent "triggering phase" of checkpoints, the implementation in #20275 requires that before the previous checkpoint is fully completed or aborted, the next checkpoint should not be started. This has limited the usage of coordinators in cases when savepoints are triggered or the maximum allowed concurrent checkpoints is larger than 1.
This PR solves the problem above by adding concurrent checkpoint support in the coordinator's implementation.
Brief change log

The subtask gateway can cache events for multiple ongoing checkpoints.

Verifying this change
This change added tests and can be verified as follows:

Added integration tests to verify that coordinator can work correctly without message loss when the concurrent checkpoint is enforced.

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, Kubernetes/Yarn, ZooKeeper: (yes)

This PR affects the behavior of the coordinator in case of concurrent checkpoints, as described above.

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)

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure