🐛 [Bug]: scrolling performance issue on overview page

Version
latest
node-version
latest
Link to minimal reproduction
open https://opentiny.design/tiny-ng/overview
open chrome dev tools, select rendering, select Pain flashing

Then scroll the doc, as you can see all items on the page was repainted.
Step to reproduce
open https://opentiny.design/tiny-ng/overview
open chrome dev tools, select rendering, select Pain flashing

Then scroll the doc, as you can see all items on the page was repainted.
What is expected
Since the overview page is pretty simple from functional perspective, page FPS should keep as high as your monitor 60/90/144.
What is actually happening
But in fact, it has dropped ~20 FPS in each scroll. if you click the Frame rendering stats as well on the rendering tab of chrome dev tools. And you can see everything inside the .n-layout-scroll-container was repaint due to each scrolling.
No scrolling (~141 fps)

Scrolling  (~126 fps)

I suggest add css
will-change: transform;

in .n-layout-scroll-container
Any additional comments (optional)
I use a game monitor as default fresh rate is 144 hz.
So when people are using high fresh rate screen like game monitor or premium phone, they would be much easier to detect the scrolling performance issue of the page.
Pretty sure we should take care of the use cases like this in documentation web site. For details you can see
https://dev.opera.com/articles/css-will-change-property/
If you do not tell chrome to render the element using GPU then it will have a long repaint phase in the middle.

Thank you very much for your feedback. This problem is very important and we are trying to fix it.

thx for advise  ~  we have fixed it~


⚠ Bofrost.es has degraded performance

In 777200c, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1774 ms

Resolved: Bofrost.es performance has improved in a91b49f after 6 minutes.


Fix and simplify SIMD tests

The tests for SSE and AVX must only be done if the correct compiler
flags were used.
Signed-off-by: Stefan Weil<EMAIL_ADDRESS>

This should fix issue #1113. The dummy implementations for SSE and AVX on Android are now removed, so we can be sure that they are never called.

Why do you need -DAVX when you already have __AVX__?

__AVX__ is set automatically by the compiler if the code is compiled with AVX support.
simdetect.cpp must work on hosts without AVX, too, so that code cannot be complied with AVX support. That means a macro which says that AVX is included (in another compilation unit!) is needed. I could have taken __AVX__ as well, but macros starting with underscores are reserved for the system and the compiler. That's why I decided to use AVX.


Add option to specify output file.

No merge or comment on this one?

Of course I'll merge it, thanks a lot btw!! I am just really busy during the week normally, I'll review it and commit it this Friday & probably release a new version this weekend or early next week!
Thank you very much for contributing. I hope you find HastyScribe useful -- and feel free to open additional issues also for new suggestions or feedback!

No worries, just wanted to make sure that there wasn't an issue with it. HastyScribe is great, it's simple and it works!


⚠ Bofrost.es has degraded performance

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

HTTP code: 200
Response time: 1609 ms

Resolved: Bofrost.es performance has improved in bc255cf after 6 minutes.


QA searches

Building without height (Overpass?)
Building without bin (Overpass?)
Old construction (Overpass?)
Double bins (Overpass?
Old temp bins

Buildings without height: https://overpass-turbo.eu/s/1pQP
Buildings without BIN: https://overpass-turbo.eu/s/1pQR
Buildings with a BIN but no height: https://overpass-turbo.eu/s/1pQS
Old construction sites that haven't been touched in more than 3 years, which seems to be the sweet spot: https://overpass-turbo.eu/s/1pQU

I'm thinking about how we could structure this. My first thought is a new page for QA. I envision that page having a drop-down list of the searches. Then maybe a map and table of relevant details for the query results. I don't know if we can run the bin search on all the results or if we need to give the user the option to look them up one by one or in groups.
For running the queries I think Overpass could handle some citywide but others might not work on that large of an area. I think the bottleneck might also be displaying the data for a citywide query.


coverflow mobile safari css rendering issues

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 4.2.2.

Platform/Target and Browser Versions: IOS mobile safari 11.3
parameters:
const params = {
centeredSlides: true,
effect: 'coverflow',
coverflowEffect: {
rotate: 50,
stretch: 0,
depth: 100,
modifier: 1,
slideShadows: true
},
height: tableHeight,
initialSlide: this.props.centerAt || Math.floor(this.props.cards.length / 2),
navigation: {
nextEl: '.swiper-button-next',
prevEl: '.swiper-button-prev'
},
grabCursor: true,
slidesPerView: 'auto',
simulateTouch: true,
spaceBetween: -30,
roundLengths: true
// rebuildOnUpdate: true
}

What you did
I use 3d cover flow to swipe between a few slides.
Expected Behavior
It should work the same as a the demos on the site.
Actual Behavior
Between 4.1.6 and 4.2.2 the coverflow effect got broken. I've been locked to 4.1.6 ever since my testers started bugging the swiper carousels on mobile safari. (other browsers are still ok) The cards appear offset and rotate incorrectly on swipe.

Live example or fiddle with the issue would be great as i can't replicate it

I started working on the repro, based on your own examples. But then it did repro for me on the device I have here, when used on: http://idangero.us/swiper/demos/240-effect-coverflow.html
Do you have any ios test devices? It's not happening in a desktop safari.

sometimes it works. Load timing issue? resize happening? Tried switching to landscape and refreshing the page?

this demo doesn't use latest swiper version

it's broken in versions greater than 4.1.6. This demo uses a version greater than 4.1.6

I am asking the same, can you please provide live example or fiddle with the issue using latest 4.2.5+ version?

I would like to. Please note that I observed the broken behavior on your own demo; so making another demo that reproduces the same is not my favorite task of the day. I suspect it's a test environment issue with respect to resize or initialization timing, based on my observation that it's not always broken, but one can usually refresh until achieving a broken state.


Add to homebrew

Hey! Awesome script... Would it be possible to add it to Homebrew for easier installation? Thanks!

Sounds like an excellent idea, @piersolenski !
I'm not that familiar with creating a new Homebrew "package" though ... Is it something you think you can help with?

I've not done it before either, but I could look into it when I have time! Looks like 3.0.0 was just released to fix Big Sur incompatibility too!


Update Composer dependencies (2022-04-15-00-04)

Visual regression test passed!
View the visual regression test report


ifm3d-camera 0.17.0 unavailable via ifm-robotics_ubuntu_xenial_amd64 apt repo

Describe the bug
While attempting to install ifm3d-tools from ifm-robotics_ubuntu_xenial_amd64 (the no-ROS option), I had an error collecting all of the dependencies. I was able to trace it back to the fact that ifm3d-camera 0.17.0 is missing from that repository. I was able to install ifm3d-camera 0.17.0 from ifm-robotics_ubuntu_xenial_amd64_ros and follow with ifm3d-tools, but I can't be sure that I haven't installed an incompatible package.
To Reproduce
(On a machine which has never installed ifm3d-*)

Add the apt repository to sources:
sudo sh -c 'echo "deb [arch=amd64] https://nexus.ifm.com/repository/ifm-robotics_ubuntu_xenial_amd64 xenial main" > /etc/apt/sources.list.d/ifm-robotics.list'
Add the apt key:
sudo apt-key adv --keyserver hkp://ha.pool.sks-keyservers.net:80 --recv-key 8AB59D3A2BD7B692
Install ifm3d-tools:
sudo apt-get install ifm3d-tools

Expected behavior
ifm3d-tools 0.17.0 is installed.
Output/Screenshots
$ sudo apt-get install ifm3d-tools
Reading package lists... Done
Building dependency tree
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 ifm3d-tools : Depends: ifm3d-camera (= 0.17.0) but it is not going to be installed
               Depends: ifm3d-image (= 0.17.0) but it is not going to be installed
               Depends: ifm3d-swupdater (= 0.17.0) but it is not going to be installed
               Depends: ifm3d-framegrabber (= 0.17.0) but it is not going to be installed
E: Unable to correct problems, you have held broken packages.

$ apt list | grep ifm3d

WARNING: apt does not have a stable CLI interface. Use with caution in scripts.

ifm3d-camera/xenial 0.15.1 amd64
ifm3d-framegrabber/xenial 0.17.0 amd64
ifm3d-image/xenial 0.17.0 amd64
ifm3d-opencv/xenial 0.17.0 amd64
ifm3d-pcicclient/xenial 0.17.0 amd64
ifm3d-pcl-viewer/xenial 0.1.0 amd64
ifm3d-python/xenial 0.17.0 amd64
ifm3d-python3/xenial 0.17.0 amd64
ifm3d-swupdater/xenial 0.17.0 amd64
ifm3d-tools/xenial 0.17.0 amd64

Here's my workaround:
$ wget https://nexus.ifm.com/repository/ifm-robotics_ubuntu_xenial_amd64_ros/pool/i/ifm3d-camera/ifm3d-camera_0.17.0_amd64.deb
$ sudo dpkg -i ifm3d-camera_0.17.0_amd64.deb
$ sudo apt-get install ifm3d-tools

Configuration/Environment (please complete the following information):

OS: Ubuntu 16.04.6
ifm3d version: 0.17.0
using ROS: no

Yes it looks like I botched something. My apologies; I'll have this corrected today.

Thanks for the detailed report and workaround -- I've re-posted the 0.17.0 binary to the apt repository. Confirmed that I could reproduce before but not after. Should be fixed now.


Class instead of ingressclassName

Section ClusterIssuer Resource
The code is wrong, it should be like this (class, instead of ingressclassName)
`
#!/bin/bash
kubectl apply -f - <<EOF
apiVersion: cert-manager.io/v1
kind: ClusterIssuer
metadata:
name: letsencrypt-staging
spec:
acme:
email<EMAIL_ADDRESS>server: https://acme-staging-v02.api.letsencrypt.org/directory
privateKeySecretRef:
name: example-issuer-account-key
solvers:
- http01:
ingress:
class: azure/application-gateway
EOF
`

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

ID: f53838fc-1654-fc18-7e45-7d556193f77e
Version Independent ID: 4f92bd5b-4b91-b03d-4f1d-8c31d45af0ef
Content: Use LetsEncrypt.org certificates with Application Gateway
Content Source: articles/application-gateway/ingress-controller-letsencrypt-certificate-application-gateway.md
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

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

@psavary
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.
@greg-lindsay
Can you please check and add your comments on this doc update request as applicable.


Fix opacity typo in opacity-visibility doc

Hi, just found a little typo

Thank you, @BabkinAleksandr :100:


kpt can't reference images by sha

Expected behavior
It would be wonderful if kpt could understand image references via SHA
gcr.io/yakima-eap/generate-folders@sha256:3434a5299f8fcb2c2ade9975e56ca5a622427b9d5a9a971640765e830fb90a0e
Actual behavior
kpt1 fn render .
Kptfile is invalid:
Field: `pipeline.mutators[0].image`
Value: "gcr.io/yakima-eap/generate-folders@sha256:3434a5299f8fcb2c2ade9975e56ca5a622427b9d5a9a971640765e830fb90a0e"
Reason: function name "gcr.io/yakima-eap/generate-folders@sha256:3434a5299f8fcb2c2ade9975e56ca5a622427b9d5a9a971640765e830fb90a0e" is invalid

Kptfile:
apiVersion: kpt.dev/v1alpha2
kind: Kptfile
metadata:
  name: simple-hierarchy
info:
  description: Resource hierarchy blueprint for GCP
pipeline:
  mutators:
    - image: gcr.io/yakima-eap/generate-folders@sha256:3434a5299f8fcb2c2ade9975e56ca5a622427b9d5a9a971640765e830fb90a0e

Strangely, with kpt fn eval --image XYZ it seems possible to reference image via sha:
docker run gcr.io/yakima-eap/generate-folders@sha256:3434a5299f8fcb2c2ade9975e56ca5a622427b9d5a9a971640765e830fb90a0e
apiVersion: v1
kind: ResourceList
metadata:
  name: output
items: []

Information
% kpt version
1.0.0-alpha.10
Steps to reproduce the behavior

Thanks for submitting this.
@natasha41575

We should relax the validation function for the function image to support this use-case. I think we have taken the validation too far :)
Also, it will be good to make the validation consistent across fn eval and fn render.

https://github.com/GoogleContainerTools/kpt/issues/2020


[Feature] Expose outside click event on DialogContent

Describe the feature
Currently, the default behavior for clicking outside the DialogContent is to close the dialog.
In some cases, however, one might want to prevent this behavior, e.g. when the contents inside the dialog have changed and we want to notify the user of unsaved changes before closing.
The original radix-ui forwards the onPointerDownOutside and onInteractionOutside allowing for them to be overridden.
Consider adding the same option for this project in order to align the APIs/features of both libraries.
Additional information

[ ] I intend to submit a PR for this feature.
[ ] I have already implemented and/or tested this feature.

DialogContent does support those behaviour by preventing the default closing action.
I see that the docs is missing this piece of info!
Can you try using the following code and let me know if it works! We definitely need to update the docs!
   <DialogContent
        @pointer-down-outside.prevent
        @interact-outside.prevent
        @escapeKeyDown="handleEvent"
      >

Awesome! It works as expected :)
Thanks

Glad to hear that! Thanks for reporting the issue! 😁


Cannot apply for collective

I am trying to apply for a collective for github.com/fabiolb/fabio with 4300 starts which is a github org which I own and for which I'm the sole maintainer. When I log in as @magiconair it doesn't show up. What am I missing?
URL: github.com/fabiolb/fabio
Logged in as: magiconair
Error message if any: Can't find repository with more than 100 stars
Expected result: find fabiolb/fabio
Browser: any

Hi @magiconair, you need to OAuth the organization, since this repo is under the org. It's usually on the bottom left of the GitHub auth screen. If it doesn't show you the screen again, you can go in settings and take remove Open Collective's OAuth token and then screen will show again.
If it still doesn't work, could you please share a screenshot of the OAuth screen?

How do I do this since I only ever get to see this screen: https://opencollective.com/opensource/apply
and from there I get to the one I've pasted before. There is no page where I could choose the org or my account.

Visit: https://github.com/settings/applications and revoke Open Collective app. It should then show you the GitHub OAuth screen again.

Nope. Same issue. I get to authorize the access but it still won't list the fabio repo or org.

When I setup an OAuth app in the fabiolb org with the details from #209 then I'm at least logged in but didn't get much farther.

Hmm can you try visiting https://github.com/organizations/fabiolb/settings/oauth_application_policy and see if it's restricted there?
Otherwise, we just changed github permissions today and I wonder if that's what's causing it: https://github.com/opencollective/opencollective/issues/355

@asood123 any idea?

I have the exact same issue - the repository in the org with > 100 stars cannot be seen.

Ok I'm going to revert the scope change and see if this fixes it for you.

@magiconair and @dstpierre: Would you please try again? You should automatically see the Github oauth screen (since scope is increasing). If you don't, please revoke Open Collective permissions at: https://github.com/settings/applications.

@asood123 yep, that's better.

same here thanks @asood123


IPv6 Network

Extended support to perform CRUD operations on IPv6 Network.
Added a function to allocate an IPv6 network.

One more thing. Please add negative test cases, only positive tests are not enough.

@anagha-infoblox  let us close this PR since an internal development is being done by you and will raise a new PR once that is tested.


Remote Development using SSH - riscv64 arch Support

Hello,
I try remote develop to Unmatched board using SSH, but it reports:
Unsupported architecture: riscv64
[16:15:51.924] Unsupported architecture
[16:15:51.925] Resolver error: Error: The remote host's architecture is not supported
at Function.ServerInstallError (c:\Users\vimer.vscode\extensions\ms-vscode-remote.remote-ssh-0.78.0\out\extension.js:1:585975)
at p (c:\Users\vimer.vscode\extensions\ms-vscode-remote.remote-ssh-0.78.0\out\extension.js:1:579264)
at Object.t.handleInstallOutput (c:\Users\vimer.vscode\extensions\ms-vscode-remote.remote-ssh-0.78.0\out\extension.js:1:583109)
at Object.t.tryInstall (c:\Users\vimer.vscode\extensions\ms-vscode-remote.remote-ssh-0.78.0\out\extension.js:1:681924)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at async c:\Users\vimer.vscode\extensions\ms-vscode-remote.remote-ssh-0.78.0\out\extension.js:1:644564
at async Object.t.withShowDetailsEvent (c:\Users\vimer.vscode\extensions\ms-vscode-remote.remote-ssh-0.78.0\out\extension.js:1:647909)
at async Object.t.resolve (c:\Users\vimer.vscode\extensions\ms-vscode-remote.remote-ssh-0.78.0\out\extension.js:1:645643)
at async c:\Users\vimer.vscode\extensions\ms-vscode-remote.remote-ssh-0.78.0\out\extension.js:1:722950
Is there plan to support the feature or I can do somethong for it?

Probably belongs in the respective repository of remote development extension: https://github.com/microsoft/vscode-remote-release

Ok,
The request has been there:
https://github.com/microsoft/vscode-remote-release/issues/4802
thank you.


[WIP] Add GPU selection

DO NOT MERGE! :stuck_out_tongue_closed_eyes:
This is as far as I got tonight. No tested, and I'm pretty sure it's not done, but I can think anymore.
Just sending it out so you know i'm working on it, and so you can tell me if you want to do things differently :]

Yes, this comes close to what I've thought about (and started working on). I'm not sure about how I'll handle the selection, but having a new command line argument seems to be the easiest way, so I guess I'll go with that one ;)

and started working on

Are far are you? We might as well merge our work (since you said it's the same idea, so it should be compatible).
The question is then: do I merge your changes, or do you merge mine?  ie.: who finishes the work?
I guess I should've asked/told you before starting to work on it, to avoid doing the same work twice. :grin:

Forgot to ask: which of these two do you prefer?
On invalid/missing user selection, if there's more than one device:
select first device
print an error and abort

Select the first device. Device name is displayed in the title bar anyway.

Are far are you? We might as well merge our work (since you said it's the same idea, so it should be compatible).

Almost finished, but my spare time is very limited right now, so I don't know if I'll be able to tackle this before the weekend.

OK, letting you finish then.
Feel free to use whatever code I wrote that might be useful to you. You can close this issue once you're done :]

Better late than never. Just implemented this now that the command line argument handling has been reworked and is working on linux too. The example can now list all available Vulkan capable GPUs (-listgpus) and you can specify the GPU (index) to use via -g (or -gpu).


Integrating postcss

How would one use postcss instead of sass?

By changing the webpack loader.  That's the short answer.

I think I got it working, see the comment at #158.

@RanzQ Yes, that works! I too had trouble with cssnext...

Shall we close? @erikras


Move Json out of the standard library

Today we have a Json module in the builtins that should not exist there longer-term. We'd like to move the JSON module to user space.
Removing Json is not as simple as deleting the file because many compiler-internal tests rely on the existence of the module for testing abilities.
So, the first step of this issue will be creating a synthetic encoder/decoder implementation that those tests can use inside of the compiler itself (this can just be a raw string that implement a Roc encoder/decoder faithfully), and updating all tests that depend on Json to use that module instead.

To clarify @ayazhafiz, do we need an opaque type that lives in crates/compiler/builtins/roc, it fully implements the Encode and Decode abilities, and replaces Json in the relevant tests in crates/compiler/test_gen/src/gen_abilities.rs and crates/compiler/test_mono/src/tests.rs?
Should this type be a builtin or can it live somewhere else? I assume we don't want it to show up in docs generation or accidentally shadow another module.

As discussed in private communication, this module need not be in the builtins. A string representation used in the tests will do. m

Can I take this ticket? I've got a PR ready #6845


CI/PR labeling improvements - Automatically generate and publish API docs in CI

Work related to first point in issue #1316

[ ] Automatically generate and publish API docs in CI

This code was written and reviewed by GitStart Community. Growing great engineers, one PR at a time.

@gitstart Thanks for the PR! Unfortunately we're doing massive refactoring in this repo and won't be able to merge this.


Hair Segmentation on web app

Hello.
Currently, I am trying to build web app that uses many mediapipe solutions, since it is mobile-friendly.
My issue is that I cannot figure out how to use hair segmentation solution in web app(using js).
As far as I know, hair segmentation does not provide js solution. It only has web demo.
How can I achieve what I want?
Hair segmentation would have js solution?
Somewhere, I read that it was converted to wasm for the web demo.
Then, where can I get that wasm file and make use in web app?
Thanks.

Hi @probator1059  Present we are not supporting Hair segmentation in Javascript. See here.  May be you can share your exact need so that i can check with the team and let you know if there are any workaround, But not sure this will be addressable as per your requirement.

@probator1059,
Marking it as a Feature Request. Thanks!

so will the hair segmentation come out for js?


SYNPY-966 Update tests

Hello @kimyen! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file tests/unit/internal/test_cache.py:

Line 444:121: E501 line too long (122 > 120 characters)


Call to get suggestions

Input.js should call a REST endpoint to get the autocomplete suggestions once the entered string is longer than 3 characters.

done


Debug console: be able to disable automatic scrolling to end

VSCode Version: Version 1.15.0 (1.15.0)
OS Version: Darwin ecvaawplun5.local 16.7.0 Darwin Kernel Version 16.7.0: Thu Jun 15 17:36:27 PDT 2017; root:xnu-3789.70.16~2/RELEASE_X86_64 x86_64

Steps to Reproduce:

Run a debug session that generates a lot of output.
The debug console will automatically scroll to the end.

Reproduces without extensions: No

This has been tested with https://github.com/rubyide/vscode-ruby, but I would guess the same happens with other languages as well. When the debug console receives lots of output (say 10s or 100s of lines), it automatically scrolls to the bottom.
For me, this is annoying since the errors I am most interested in are most likely at the beginning of the output. Means I have to scroll up manually every time.
Could we make this an option? (Auto-scroll on/off)

VS code supports "smart scroll", that means automatic scrolling only occurs if you are at the very end of the debug console. If you scroll at least one line up, automatic scrolling should stop (see this feature request https://github.com/Microsoft/vscode/issues/10486)
@isidorn I could not find "smart scroll" in the doc. We should add this to the doc.
In addition we could introduce a button that shows the state of "smart scroll" and can be used to control the behaviour directly.

Thanks for the reply @weinand!

VS code supports "smart scroll", that means automatic scrolling only occurs if you are at the very end of the debug console. If you scroll at least one line up, automatic scrolling should stop (see this feature request #10486)

For me, this is not what happens. Every time I launch a task, the debug console is cleared, and it scrolls down to the bottom.
So, to make things clear: this is not dealing with a continuous debugging session, but more referring to the fact that "every time a task is launched, the debug console is cleared and gets scrolled all the way to the bottom unconditionally". Sorry for not being clear about that initially.

Smart scroll is only supported for the output, not the debug console.

Smart scroll works to some extent in the "terminal" tab, until it must hit some buffer limitations of sort then you can't stop it to go to the bottom anymore (Linux)

"Smart scroll" actually works now in the debug console. Thus closing this
My program
setInterval(() => {
    var count = Math.random() * 50;
    var message = Date.now() + 'new log message: ';
    for (var i = 0; i < count; i++) {
        message += ' ' + Math.random();
    }
    console.log(message);
}, 200);

I have verified that when I am scrolled all the way to the end that the debug console will follow the latest output, however when I am scrolled not at the end that the output will be locked as expected.
Please note related issue that I am investingating https://github.com/microsoft/vscode/issues/107704


(short issue description)

Describe the feature
We want to create a new hook for packages/react-hooks called useInternal
Use Case
We need to have a react way to use the setInternal function
Proposed Solution
We want to create a new hook for packages/react-hooks with the following header:
const useInterval = (callback: Function, delay?: number | null) => { . . . };

Other Information

Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

Version used

Environment details (OS name and version, etc.)

@pauchena will take care of this!


pkgsLLVM.mc: explicitly set PERL= variable

Otherwise build fails as:
mc-x86_64-unknown-linux-gnu> bash: ../../../src/man2hlp/man2hlp: /usr/bin/perl: bad interpreter: No such file or directory

Motivation for this change
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/767

Could we now remove perl from buildInputs?

Could we now remove perl from buildInputs?

Sounds good. Removed.


User model and schema update

Hello @oyeb. Here are the changes I made.

I refactored the changeset functions and I wrote tests for the user schema

I made a migration to ensure the password_hash field is never null.

I wasn't clear as to which field you wanted to change to have type :utc_datetime. However since the       timestamps were the only field in the snitch_users table to have dates I assumed you meant those. I made another migration to reflect that.

I made a migration that renamed the is_admin? field in the snitch_users table to is_admin

I created the user model with tests.

I found an issue in QueryHelper.  In the private function commit_if_valid it was returning the following on error, {:error, changeset.errors}, which violates the return value spec of                          {:ok, Ecto.Schema.t()} | {:error, Ecto.Changeset.t()}.  I had to make this change for my tests in Model.User to pass.

I assumed, since you moved the address association to another issue, that you would want the improvements to user_with_address in factory.ex taken care of in that issue as well.

It seems that the changes to password_hash never being null is causing various tests to fail.

glad you caught me before i went to sleep :smile: .  Working on the changes now

It seems the tests in CardPaymentMethodsTest does not like that password_hash cannot be null

I just pulled your branch and all tests passed on my local (including CardPaymentTest). The Travis build indicates that your changes were rejected by mix format --check-formatted:

The following files were not formatted:

apps/snitch_core/lib/core/data/model/user.ex
apps/snitch_core/lib/core/data/schema/user.ex
apps/snitch_core/test/data/model/user_test.exs
apps/snitch_core/test/data/schema/user_test.exs

Just run mix format in the project root.

It worked for the most part.  Not sure what this other failure is.  It only happened in one of the runs it seems.

Yes, the build for Elixir 1.5.3 passed because we don't check if the files are formatted in that build (because mix format was added to Elixir in 1.6.0).
The other build (on Elixir 1.6.2) failed because some committed files are not formatted. For more details check our Contribution Guide.
This is simple to resolve, from the repo root directory issue this command:
$ cd path/to/repo/root
$ mix format

This will auto-format the files according to our .formatter.exs. Now, just stage the formatted files and push a new commit. That should do it! :crossed_fingers:

Finally :smile:

Thanks a lot @Euklidian-Space for your excellent PR, I :heart: that you squashed your commits for us!

No problem @oyeb!  This was fun!


Removed unnessary encoding & used consistent quotes

To save bytes & improve legibility, spaces don't need to be encoded and double quotes (") can be converted to single quotes (').
For this input:
div {
    @include angled-edge('outside bottom','lower right', red);
}

The result is 17 bytes smaller after minification & gzipping:
Old:
div::after {
  background-image: url("data:image/svg+xml,%3Csvg%20xmlns=%27http://www.w3.org/2000/svg%27%20height=%22100%22%20width=%222800%22%20fill=%22rgb(255,0,0)%22%20fill-opacity=%221%22%3E%3Cpolygon%20points=%220,0%202800,0%200,100%22%3E%3C/polygon%3E%3C/svg%3E");
}

New:
div::after {
  background-image: url("data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/svg' height='100' width='2800' fill='rgb(255,0,0)' fill-opacity='1'%3E%3Cpolygon points='0,0 2800,0 0,100'%3E%3C/polygon%3E%3C/svg%3E");
}

Looks good to me. Thanks for the contribution!


[AMBARI-24050] Oozie service check failed during EU on unsecure clust…

…er due to presence of "ELService" configs (dsen)
What changes were proposed in this pull request?
Added configs removal during upgrade
Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2624/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2626/
Test PASSed.

Just tested the patch on a cluster, everything works as expected.

@d0zen1: Only 2.6 -> 3.0 upgrade should be affected by this change.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2664/
Test PASSed.


Use documented jQuery API

Passing a string to the $.fn.html method works in the latest version
of the jQuery library, but this signature is neither documented [1] nor
tested [2]. This code does not work with the latest version of the
Cheerio library, and because it is non-standard, it is unlikely to be
implemented in a future version.
Use $.fn.empty and $.fn.append to implement the desired behavior in
a way that is compatable with Cheerio and unlikely to be removed in a
future release of jQuery.
[1] http://api.jquery.com/html/
[2] https://github.com/jquery/jquery/blob/2380028ec4a6a77401b867a51de26a3cb8e8d311/test/unit/manipulation.js

@tbranyen This doesn't make us ready for the Cheerio upgrade, but it's a start!

Awesome, totally knew this and even have a blog post about it... for shame.  Thanks for the PR!

My pleasure


Update OAW package hash for 2306 release

This PR updates the package hash for OAW 2306 released package.

#sign-off


Scrutiny

App dont gather metrics.
Scrutiny
Current version: 0.3.13-6
supervisor-2022.01.0
core-2021.12.10
Home Assistant OS 7.2
System i386 Nuc, Disk mSATA SSD
App log:
time="2022-01-31T19:43:21+01:00" level=info msg="Collecting smartctl results for sda\n" type=metrics
time="2022-01-31T19:43:21+01:00" level=info msg="Executing command: smartctl -x -j /dev/sda" type=metrics
time="2022-01-31T19:43:21+01:00" level=error msg="smartctl returned an error code (2) while processing sda\n" type=metrics
time="2022-01-31T19:43:21+01:00" level=error msg="smartctl could not open device" type=metrics
time="2022-01-31T19:43:21+01:00" level=info msg="Publishing smartctl results for \n" type=metrics
time="2022-01-31T19:43:21+01:00" level=error msg="An error occurred while updating device data from smartctl metrics WHERE conditions required"
time="2022-01-31T19:43:21+01:00" level=error msg="<IP_ADDRESS> - db21ed7f-scrutiny [31/Jan/2022:19:43:21 +0100] "POST /api/device//smart" 500 17 "" "Go-http-client/1.1" (5ms)" clientIP=<IP_ADDRESS> hostname=db21ed7f-scrutiny latency=5 method=POST path=/api/device//smart referer= respLength=17 statusCode=500 userAgent=Go-http-client/1.1
2022/01/31 19:43:21 /app/scrutiny/webapp/backend/pkg/web/handler/upload_device_metrics.go:31 WHERE conditions required
[0.213ms] [rows:0] UPDATE devices SET created_at="2022-01-31 18:07:34.087",updated_at="2022-01-31 19:43:21.131",device_name="sda",device_type="scsi"
time="2022-01-31T19:43:21+01:00" level=info msg="Main: Completed" type=metrics
[19:43:21] INFO: Local Devices Added
Supervisor log:
22-01-31 18:07:27 ERROR (SyncWorker_1) [supervisor.docker.addon] Add-on Scrutiny try to access to blocked device sda!
22-01-31 18:07:27 ERROR (SyncWorker_1) [supervisor.docker.addon] Add-on Scrutiny try to access to blocked device sda!
22-01-31 18:07:27 ERROR (SyncWorker_1) [supervisor.docker.addon] Add-on Scrutiny try to access to blocked device sda1!

Hi, thanks. Are you sure your disk supports SMART?

not shure but i will boot from a stick with ubuntu with smartctl installed, tomorrow, come back with result an in case of "yes" the full log, thnx

Hi again, i am lost an your idea is right, no smart support from my disk, so i close.


update and rename NextMillennium

changed name company

Confirmed this is already what the adapter is named. Thanks for the fix. Merging.


Automatically shut down test cluster resources after configurable TTL

@codysoyland commented on Fri Dec 16 2016

@codysoyland commented on Tue Dec 20 2016
This is proving to be more difficult than anticipated. AWS has a blog post that shows a method of doing this: https://aws.amazon.com/blogs/devops/scheduling-automatic-deletion-of-application-environments/
But the AWS method requires use of an EC2 instance with the cfn tools installed (such as Amazon Linux) and I would like to avoid having extra software installed on the pilosa machines. I would like to use Lambda to schedule the deletion, but I haven't found a clear way to schedule a lambda job.
I will Icebox this for now and we can discuss options in the future.

edited the title to reflect that this could apply to any of our infrastructure testing code - we have various stuff for Azure and AWS now


Add custom querier to wasm simapp image

We use the wasm simapp image in the IBC Eureka Solidity implementation with the Union Ethereum Light Client (08-wasm). For that particular light client to work, it needs a custom querier registered with 08-wasm.
The goal here is to add that simple custom querier to be available for 08-wasm light clients in the wasm simapp image.
This is required for https://github.com/cosmos/solidity-ibc-eureka/issues/94

Closed in #7559


Updates needed for KNative 0.7

/kind bug
Knative 0.7 has changed the route status to have URL and not Domain. We need to change our status update to use this otherwise samples are broken.
We should specify we need Knative 0.6 for current release.
Environment:

Istio Version:
Knative Version: 0.7
KFServing Version: 0.1

Related to #190

/area project-health

/assign

Is this still valid (as the project is now using 0.8)?

Yes it is no longer relevant

/close


Ranged Skill Damage and Ranged Skill Damage

Ranged Skill Damage may be Marksman Skill Damage
Melee Skill Damage may be CutThroat Skill Damage
Right?

I don't know what you mean. Ranged and Melee skill damage applies to all classes, not just rogues.


model-specific route-independent logic

In so many cases we want to run some logic that is model specific and route independent. This logic should be run regardless of "how" the records were retrieved (like through a populate or directly, or via findOne, or findLastName...etc).
Ideally I would love to have expansion of the "model" layer to include things like model policies and model controllers. But a simpler direct solution is to support a beforeRetrieve and an afterRetrieve callbacks in a similar fashion to other life-cycle callbacks. This will save us from the toJSON override hack that pops up every now and then (e.g. #2176 and https://github.com/balderdashy/waterline/issues/532).
Not sure if this should go here or in waterline though!

same as https://github.com/balderdashy/waterline/pull/525 ??

Thanks, somehow it skipped my eyes!
Yes this is same. However, after reading that thread I sort of agree with @particlebanana that waterline should be kept out of such logic, which means this is something that goes into sails. The right place for such logic is the controller. The problem is that we do have route controllers and policies, but we don't have model controllers and policies. We need some logic to be tightly coupled to Models and not Routes. This logic could be something that controls the access to some data, does some decoration and/or some preprocessing, or calls external systems before handing it off. Regardless, the most important point is that this logic is tightly coupled to data and independent of the route... putting it in route controllers is repetitive and dangerous. In my case it also depends on other stuff in the req, like the session owner, and some params that where injected by the RoutePolicies... etc,  so it should have access to the req.
I created a proof of concept sails hook that would run model policies after model.find and before handing the results off.
@mphasize and @aclave1 am really keen to hear your opinion on this since you seem to have run into a similar situation as mine!


returning undefined instead of null

Not a big deal but since null is a valid JSON type wouldn't it be more consistent to return undefined on invalid JSON? Although unlikely, JSON.parse("null") is a valid JSON expression that would return null. Undefined does not exist in JSON and would truly mean that the parsing failed.

Agreed, it has been something I've been meaning to do for a while. Thanks for opening the issue.

shouldn't this issue be closed? this module returns undefined ever since the last commit.


gpg: Can't check signature: No public key

Setup:
    steps:
      - name: Checkout
        uses: actions/checkout@v2
      - name: Run driftctl
        uses<EMAIL_ADDRESS>        with:
          version: 0.9.0

Output:
gpg: directory '/github/home/.gnupg' created
gpg: keybox '/github/home/.gnupg/pubring.kbx' created
gpg: keyserver receive failed: No name
Downloading driftctl:0.9.0
No installed versions of driftctl matched '0.9.0', let's install it
gpg: Signature made Tue May 25 13:54:13 2021 UTC
gpg:                using EDDSA key 277666005A7F01D484F6376DACC776A79C824EBD
gpg:                issuer<EMAIL_ADDRESS>gpg: Can't check signature: No public key
PGP signature rejected!
Installation of version 0.9.0 failed
Failed to get version from dctlenv-version-name

Hey @tuniemela, I've just tried a new workflow with your setup and it worked as expected:
gpg: directory '/github/home/.gnupg' created
gpg: keybox '/github/home/.gnupg/pubring.kbx' created
gpg: /github/home/.gnupg/trustdb.gpg: trustdb created
gpg: key ACC776A79C824EBD: public key "Cloudskiff<EMAIL_ADDRESS>imported
gpg: Total number processed: 1
Downloading driftctl:0.9.0
gpg:               imported: 1
No installed versions of driftctl matched '0.9.0', let's install it
gpg: Signature made Tue May 25 13:54:13 2021 UTC
gpg:                using EDDSA key 277666005A7F01D484F6376DACC776A79C824EBD
gpg:                issuer<EMAIL_ADDRESS>gpg: Good signature from "Cloudskiff<EMAIL_ADDRESS>[unknown]
gpg: WARNING: This key is not certified with a trusted signature!
gpg:          There is no indication that the signature belongs to the owner.
Primary key fingerprint: 2776 6600 5A7F 01D4 84F6  376D ACC7 76A7 9C82 4EBD
Switching version to v0.9.0
Switching completed

The import of the key failed on your run, can you try again ?

Hi,
Thanks for the quick response. It works now! This is still there but maybe
we can live with that: gpg: WARNING: This key is not certified with a
trusted signature!
-Tuomo
On Mon, 14 Jun 2021, 13:27 William BEUIL, @.***> wrote:

Hey @tuniemela https://github.com/tuniemela, I've just tried a new
workflow with your setup and it worked as expected:
gpg: directory '/github/home/.gnupg' created
gpg: keybox '/github/home/.gnupg/pubring.kbx' created
gpg: /github/home/.gnupg/trustdb.gpg: trustdb created
gpg: key ACC776A79C824EBD: public key "Cloudskiff @.>" imported
gpg: Total number processed: 1
Downloading driftctl:0.9.0
gpg:               imported: 1
No installed versions of driftctl matched '0.9.0', let's install it
gpg: Signature made Tue May 25 13:54:13 2021 UTC
gpg:                using EDDSA key 277666005A7F01D484F6376DACC776A79C824EBD
gpg:                issuer @."
gpg: Good signature from "Cloudskiff @.***>" [unknown]
gpg: WARNING: This key is not certified with a trusted signature!
gpg:          There is no indication that the signature belongs to the owner.
Primary key fingerprint: 2776 6600 5A7F 01D4 84F6  376D ACC7 76A7 9C82 4EBD
Switching version to v0.9.0
Switching completed
The import of the key failed on your run, can you try again ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cloudskiff/driftctl-action/issues/3#issuecomment-860576616,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKC77OIBGRQDMY7NSVV6ZDTTSXKQJANCNFSM46UR5JHA
.

Yep, it's not important since it's in a CI environment.


Invalid block checksum

After compressing a block and decompressing it with blockChecksum enabled, node-lz4 bombs:
    const decoder = lz4.createDecoderStream({
      });
      const encoder = lz4.createEncoderStream({
        blockChecksum: true,
        //debug: true,
      })
      const buffers = []
      encoder.on('data', data => decoder.write(data))
      encoder.on('end', () => decoder.end())
      decoder.on('data', data => buffers.push(data))
      decoder.on('end', () => {
        console.log('Received %s', Buffer.concat(buffers))
      })
      encoder.end('panchi pinchi pinchi panchi punchi\n')

results in:
Invalid block checksum:<PHONE_NUMBER> vs<PHONE_NUMBER> for {"type":"Buffer","data":[153,112,97,110,99,104,105,32,112,105,7,0,3,21,0,96,117,110,99,104,105,10]} @26

After investigating in the guts of lib/encoder_stream.js here it appears that it computes the checksum on a buffer that contains the compressed block, and some padding: instead of only 9970616e636869207069070003150060756e6368690a it hashes 9970616e636869207069070003150060756e6368690a0000000000000000000000000000000000000000000000000000000000, which yields the error.

I got a similar error, maybe it's related to this, here is the exact error message "Invalid stream descriptor checksum @6".
Sometimes error happens sometimes not.
I consume the chunks concatenate them and decompress them at the end of consumption.
It happens only for larger .json documents.
Here is the exact code
const getObject = async (key, bucketName = AWS_TRIP_BUCKET_NAME) => {
  const { Body: object } = await s3.send(
    new GetObjectCommand({
      Bucket: bucketName,
      Key: key,
    }),
  )
  let data
  object.on('data', (chunk) => (data = !data ? chunk : (data += chunk)))

  return new Promise<Buffer>((resolve) => {
    object.on('end', () => {
      resolve(Buffer.from(data))
    })
  })
}

compression
const compressObject = (object: Record<string, unknown>) => {
  const buff = lz4.encode(Buffer.from(JSON.stringify(object)), {
    highCompression: true,
  })
  return buff
}

decompression after getObject
const getPlainObject = async <T = any>(key: string, options?: GetPlainObjectOptions) => {
  const { bucketName, decompress = false } = options || {}
  const object = await getObject(key, bucketName)

  return JSON.parse(decompress ? lz4.decode(object).toString() : object.toString()) as T
}


Improve Guide fonts and Style

The technical documents fonts are very small and the style is not user friendly.
This is not the issue only to EF related docs.
http://docs.efproject.net/en/latest/platforms/aspnetcore/existing-db.html

cc @danroth27 to comment on this -  the styles we use are used in several Microsoft sites and aren't specific to EF.

We'll be doing a style update when we move to docfx.


[Question]  Is there any specific reason for using deepspeed zero2 for pretraining and zero3 for SFT?

Question
Do zero2 perform better than zero3 in pretraining? Thanks@haotian-liu

@DietDietDiet Could you figure out the reason for this?


Syntax highlight for postcss

Please, add the option to set the style lang to postcss, pcss and sss (SugarSS).

I know that there is still no syntax highlighter for sugarss #11

The syntax highlighting from the pull request doesn't seem to be working for me. And I have the PostCSS syntax installed.

As you can see, it's messing up the rest of the syntax after the end style tag. Not too pleasant.

its has yet to be released.  you need to clone the package manually.

when I set the lang to postcss, I get an error in vue-loader trying to load a module.
...is not a PostCSS plugin

What's the point if PostCSS not working with vue-loader?

I believe postcss is now supported in the extension. Closing this issue.


stdenv/mkDerivation include dot folders in globs

Description of changes
unpackFile copies folders into the working directory and the second part of unpackPhase then checks whether anything was indeed copied.
It does so by using for i in * (a glob).
By default globs in bash do not include items with names that start with a dot ("hidden files").
However, sometimes derivation sources maybe in "hidden" folders and when they are, nix-build fails.
Bash allows their inclusion with the dotglob shell option, which is what this commit does.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Closes #208615

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2447

Did you tests this by building something like nix or the eval-lib-tests or run ofborg eval locally?

This will change the hashes of fetchers and various other breakage


[question] e-mail responses blocked on google group

Hello!
I replied to a thread on the google-group and got a bounce saying that e-mail postings were not permitted to it.
The group<EMAIL_ADDRESS>does not allow posting through email.

If this is a conscious decision, could this please be revisited? It's a lot easier to manage all our mailing lists through e-mail than it is by visiting their web-interfaces individually to reply.
Cheers!

Thanks for pointing this out!  I have changed the settings to allow posts by email.

Thanks very much @joewgraham .


cleanup / reduce errors in the ctests

Description
We need to tighten up some of our testing. Floating point exception trapping has been disabled for many tests, and the tolerances have gotten quite large on other tests.

Fix all possible floating point exceptions, so that SIGFPE can be re-enabled for all tests that do not use a model advance
Reduce the tolerances required. (fix https://github.com/JCSDA-internal/soca/issues/216, and/or use a pregenerated ensemble for the ensemble related tests)
Make sure tolerances are set so that all tests pass on Orion/Discover

test tolerances were tightened up in a previous PR by using precomputed ensemble members.  closing


make coursier download dependency sources

The compilation of the branch fails :(
[info] Compiling 1 Scala source to /Users/sme/meel/out/contrib/bsp/compile/dest/classes ...
[error] /Users/sme/meel/contrib/bsp/src/mill/contrib/bsp/MillBuildServer.scala:161:55: variable definition needs type because 'sources' is used as a named argument in its body.
[error]                                   Agg.empty[PathRef]) ++
[error]

I just test and compile success? Strange...

Well, seems like so. Commit 3fdf0629 is failing when running sh ci/publish-local.sh

My fault, sorry about that bug...

I can't tell if this PR is good to merge. It would be great, if someone who understands the issue and tried this PR could comment/review. I'd be glad to merge after positive feedback.

Oh, I know, I may need help from @samvel1024 to review.

Regardless of who reviews this, it’s going to need a better PR description

Currently I think this PR is just a hot-fix to contrib/bsp, there still need something to do in the Intellij.
@lihaoyi So sorry for that, is this PR description OK?

@sequencer no problem, that description is fine. Doesn't need to be super fancy, just needs to be something that something that an ignorant future maintainer can glance at and see what's happening. e.g. with the description you have now, I already have a ton more context in what the PR is doing than I did last time I looked.
I can't sign off on it though, since I know nothing about BSP. Hopefully someone who knows something about either Coursier (@alexarchambault!) or the BSP can chime in and let us know if it looks sensible!

That seems more related to BSP than coursier, right? This fetches source JARs instead of standard JARs, around the lines this PR touches.

I tend to merge, once the travis-ci tests are green. Could you please rebase on current master to trigger a new build on travis?


Typescript types error in version 7.0.1

Typescript types have errors from the launch of the version 7.0.1.
Detailed description
In the latest version of carbon-components-react the typescript types are included which have errors which prevent the project from compiling.
Errors
lib/components/Accordion/Accordion.Skeleton.d.ts

lib/components/UIShell/HeaderGlobalAction.d.ts

Hi @httpsOmkar! 👋 Sorry about this, looking into fixing it now!


FEATURE/CHALLENGE- Nav example with sub-level elements, how to recreate?

Not really an issue, but more of a request/challenge. I recently found this amazing example of a priority navigation and was wondering how it might be recreated in "basic" HTML/CSS/JS instead of Angular:
https://www.regionalaustraliabank.com.au/
Any thoughts?

This is something you could do purely with CSS based on the classes priorityNav toggles.
Good luck!


sun.* source code not included in src.zip

I have downloaded this binary:
https://github.com/AdoptOpenJDK/openjdk8-releases/releases/download/jdk8u144-b01/OpenJDK8_x64_Win_jdk8u144-b01.exe
Once installed, it appears sun.* source code is not included in src.zip.
As at least the class I'm interested in (src/share/classes/sun/util/locale/provider/NumberFormatProviderImpl.java) is licensed under GPLv2, coud you please include in src.zip all the source code we can find in OpenJDK repositories?

@neomatrix369 any ideas about this?

@gdams No idea why this would be the case, we are archiving the image folder as it it produced by the build process , see src.zip in https://ci.adoptopenjdk.net/job/openjdk8_build_x86-64_windows/ws/openjdk/build/windows-x86_64-normal-server-release/images/jdk8u162-b01/.

@don-vip Can you please point us to OpenJDK Windows builds (produced anywhere else) in which the src.zip does contain the file you are looking for?

? There is not. I'm asking to include the missing files. If they were already included, I wouldn't have created the ticket :)

@don-vip Thanks for clarifying.
It would mean changing the build script, have a look at http://jdk.java.net/java-se-ri/8 and then the http://download.java.net/openjdk/jdk8u40/ri/openjdk-8u40-src-b25-10_feb_2015.zip, which does not contain the file you are talking about either.
@gdams @karianna what do you think about the request? Our build script will fork away from the standard OpenJDK one if we implement this, is this desirable for the medium/long run?

@neomatrix369 Worth asking upstream OpenJDK folks why this isn't included (for jdk8u).   I can't see any reason not to include it if it's GPLv2 with Classpath and not a closed Sun/Oracle class.

FYI it has been true 'forever' that src.zip does not contain all the library source code.  That zip file is produced to assist development tools display relevant source code.  IMHO it should contain less source than it already does.
Now that we are all connected, better still we should simply include coordinates to the git repo sources that produced the build so that dev tools can reach on-line the entire code from which the binaries are produced if they so choose.

@tellison Good point, we could provide the link to the tag in Github in the download.

This issue was moved to AdoptOpenJDK/openjdk-build#476


:bug: 400 when modifying a user with a space in username

To reproduce :

Create a user with a space in its username "example "
Modify it via a PUT request
You should get a 400 "Path id exaple doesn't match with body id example "

This username should be accepted and the request go well instead

This is a bug on ui


Current coverage is 97.57%

Merging #233 into master will increase coverage by +0.01% as of 994aeac

@@            master    #233   diff @@
======================================
  Files           26      26
  Stmts          369     371     +2
  Branches         0       0
  Methods         81      81
======================================
+ Hit            360     362     +2
  Partial          0       0
  Missed           9       9

Review entire Coverage Diff as of 994aeac

Powered by Codecov. Updated on successful CI builds.


Norton antivirus says this is a virus

When I download the launcher zip and extract the files into a folder Norton antivirus deletes the exe and says that it was a virus. why does it say this, how do I fix this, and if it is a virus, how do I remove the virus from the file?

It is open source there is little chance that it could be a virus.
Check this


Update project-squared to cisco-spark

Project Squared has been changed to Cisco Spark. Its cask has been updated accordingly.
[Source]

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


why parameter:send.reconnect default value is false?

why parameter:send.reconnect default value = false?
For what reason？

This param control a client do connect() when send a request.
The default ExchangeClient is HeaderExchangeClient which doesn't need to connect when send request, so the default SEND_RECONNECT_KEY is false to match the HeaderExchangeClient.
If u choose LazyConnectExchangeClient, the constructor like:
this.url = url.addParameter(Constants.SEND_RECONNECT_KEY, Boolean.TRUE.toString());
The SEND_RECONNECT_KEY is always true because LazyConnectExchangeClient will init client and connect to server when send request.
&READY-TO-CLOSE&


Increase fluentd container memory resource limit

Fluentd containers on the master nodes are getting
OOM killed. This change increases the allowed
memory from 500Mi to 1000Mi to attempt to fix this

This change has been applied by running terraform apply in the components folder.


Question about integration with DataMapPlot

Great work with this package, I'm just starting to experiment with it. Very Exciting!
Just wondering about plugging the clustered data into DataMapPlot. Will UMAP (or other) still be required to reduce higher dim vectors down to 2D to supply to data_map_coords separately? Or can evoc supply that too? Just thinking if evoc is doing some of what UMAP does anyway, is there some efficiency by not recalculating the dimension reduction separately? Or is it better for the user to have discrete control over the coordinates for the visualization?
Thanks

I guess one further comment to this, it would be ideal to have the visual placement of the points to align as close as possible to how the points group together within the clusters. So if the following...
UMAP(
    n_neighbors=15,
    n_components=2,
    min_dist=0.0,
    metric='cosine'
)

...is going to be as close as possible visually to whatever is happening inside this...
EVoC(
    n_neighbors = 15,
)

... then great. But if EVoC is grouping things different to how UMAP would, then it might be useful to also have coordinates coming out of EVoC.

In practice you will want a separate UMAP run for feeding into DataMapPlot unfortunately. EVoC has a very custom approach to the effective dimension reduction step, and the results will be quite bad for visualization purposes (but work very well for clustering purposes). In general the clustering should align very well with UMAP results (with occasional stray points here and there), especially with the parameter choices you have above (although you can likely vary min_dist to something larger without much loss).
If you really need very good alignment with the UMAP result is is likely best to actually do your clustering on the UMAP output (and not with EVoC, which will do very weird things to that).

No worries, thanks!


[emacs][Linux] Typing Ctrl-S twice fast results in the browser Save Page dialog coming up.

Does not occur w/ Mac since Ctrl-S doesn't have a browser-native binding.

Still occurs if you type Ctrl-S twice, click in the editor, and then type Ctrl-S. (tested on http://codemirror.net/demo/emacs.html )

I can't reproduce it anymore. Do you see an stack trace in your dev tools when it happens?

hmm, you're right. It looks fine now.


testWidgets hangs if resources created in setUp

The following test hangs until the global 30s timeout. The scenario is a chain of StreamControllers, with a Completer that is resolved when the final StreamController receives its event. However, IFF the controllers are initialized in setUp, and IFF the test method is testWidgets (this behavior is not seen with the test/test package's test() method), then the last output below will be "pre-wait".
import 'dart:async';
import 'package:flutter_test/flutter_test.dart';

late StreamController<int> controller;
late StreamController<int> controller2;

void main() {
  setUp(() {
    controller = StreamController<int>();
    controller2 = StreamController<int>();

    controller.stream.listen(controller2.add);
  });

  testWidgets(
    'Await StreamControllers initialized in setUp',
    (tester) async {
      final completer = Completer<void>();
      controller2.stream.listen((_) {
        print('pre-complete');
        completer.complete();
        print('post-complete');
      });

      controller.add(0);

      print('pre-wait');
      await completer.future;
      print('post-wait');

      expect(true, true);
    },
  );
}

Output:
pre-wait
pre-complete
post-complete
TimeoutException after 0:00:30.000000: Test timed out after 30 seconds.
dart:isolate  _RawReceivePortImpl._handleMessage

✖
Exited (1)

cc @felangel @slightfoot

I think this maybe a variation of the "run widget tests async" faq, for which the answer is to wrap with runAsync:
testWidgets(
    'Await StreamControllers initialized in setUp',
    (tester) async {
      await tester.runAsync(() async {
        final completer = Completer<void>();
        controller2.stream.listen((_) {
          print('pre-complete');
          completer.complete();
          print('post-complete');
        });

        controller.add(0);

        print('pre-wait');
        await completer.future;
        print('post-wait');

        expect(true, true);
      });
    },
  );

courtesy of  @zoechi : https://stackoverflow.com/a/52177385/85472

Hello @craiglabenz. Is the solution @maks posted a viable one? If so, could this be documented better?

Yes it does fix the issue, @exaby73. Thanks, @maks! We can close for now, but I'm going to take a personal AI to understand this better and get it documented.

Can we reopen this.. as I don't believe this is the solution exactly. It requires more discussion.

What are you thinking, @slightfoot? Are you aware of a different scenario this doesn't solve?


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 3021 ms

Resolved: Bofrost.at performance has improved in 644f71d.


[PW-5047] Handle IdentifyShopper, ChallengeShopper, Pending and RedirectShopper with the same createFromAction

Summary
Payment methods that returned Pending and a QR code action (like bcmc_mobile/Payconiq) successfully created the component but were not mounting it so that the code could be shown. Here these result codes are being combined with the handling of IdentifyShopper, ChallengeShopper. The component will take care of the appropriate action.
Tested scenarios
Happy and unhappy flows for bcmc_mobile/Payconiq, 3DS1, 3DS2, iDeal.
Fixed issue:  PW-5047

@acampos1916 , we already installed this PR. When the modal of the handlePaymentDataCreateFromAction opens. There is no possibility to cancel that flow. There is no cancel or close button in the modal. Is the wanted functionality? At this point clients are stuck and try to use the browser back button which clears the basked session. When using redirects the client can cancel their payment by using the 'go back to merchant'-button.

@acampos1916 , we already installed this PR. When the modal of the handlePaymentDataCreateFromAction opens. There is no possibility to cancel that flow. There is no cancel or close button in the modal. Is the intended functionality? At this point clients are stuck and try to use the browser back button which clears the basked session. When using redirects the client can cancel their payment by using the 'go back to merchant'-button.

Hey @wannevancamp, this PR hasn't changed that modal, only reuses it for different result codes. Was this not the behavior before deploying this PR?

@acampos1916 , we already installed this PR. When the modal of the handlePaymentDataCreateFromAction opens. There is no possibility to cancel that flow. There is no cancel or close button in the modal. Is the intended functionality? At this point clients are stuck and try to use the browser back button which clears the basked session. When using redirects the client can cancel their payment by using the 'go back to merchant'-button.

Hey @wannevancamp, this PR hasn't changed that modal, only reuses it for different result codes. Was this not the behavior before deploying this PR?

@acampos1916 , you are right. This PR hasn't changed the modal. When testing the integration we saw an issue with the modal. Will create a new ticket where we can discuss this behaviour.

@wannevancamp sounds good! Thanks. Ideally we would listen for the modal close event and handle it as a regular payment failure. A new ticket would be great as it isn't exactly related to this PR 👍


Newbie here and getting the following errors when trying to compile

          Newbie here and getting the following errors when trying to compile

In file included from C:\Users\rober\AppData\Local\Temp.arduinoIDE-unsaved202406-12528-1d9cd3.mz2qk\gpioviewer\gpioviewer.ino:34:
C:\Users\rober\OneDrive\Documents\Arduino\libraries\GPIOViewer\src/gpio_viewer.h:11:10: fatal error: ESPAsyncWebServer.h: No such file or directory
11 | #include <ESPAsyncWebServer.h>
|          ^~~~~~~~~~~~~~~~~~~~~
compilation terminated.
exit status 1
Compilation error: exit status 1
Originally posted by @rescalle83 in https://github.com/thelastoutpostworkshop/gpio_viewer/issues/58#issuecomment-1879755572

@rescalle83 did you include the  ESPAsyncWebServer ?

Yes

From: thelastoutpostworkshop @.>
Sent: Saturday, January 6, 2024 12:03:15 PM
To: thelastoutpostworkshop/gpio_viewer @.>
Cc: rescalle83 @.>; Mention @.>
Subject: Re: [thelastoutpostworkshop/gpio_viewer] Newbie here and getting the following errors when trying to compile (Issue #60)
@rescalle83https://github.com/rescalle83 did you include the ESPAsyncWebServer ?
—
Reply to this email directly, view it on GitHubhttps://github.com/thelastoutpostworkshop/gpio_viewer/issues/60#issuecomment-1879769926, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALY7EJINXHNGSHGP3FHP4NDYNGGWHAVCNFSM6AAAAABBPWQW7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZZG43DSOJSGY.
You are receiving this because you were mentioned.Message ID: @.***>

The library ESPAsyncWebServer is not installed, recheck.
Try some examples that comes with the ESPAsyncWebServer.

I have installed and uninstalled several times but will try again. I can see the folder and files in libraries

From: thelastoutpostworkshop @.>
Sent: Saturday, January 6, 2024 2:42:31 PM
To: thelastoutpostworkshop/gpio_viewer @.>
Cc: rescalle83 @.>; Mention @.>
Subject: Re: [thelastoutpostworkshop/gpio_viewer] Newbie here and getting the following errors when trying to compile (Issue #60)
The library ESPAsyncWebServer is not installed, recheck.
Try some examples that comes with the ESPAsyncWebServer.
—
Reply to this email directly, view it on GitHubhttps://github.com/thelastoutpostworkshop/gpio_viewer/issues/60#issuecomment-1879823108, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALY7EJPLCOHH4XVG3TP4IF3YNGZLPAVCNFSM6AAAAABBPWQW7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZZHAZDGMJQHA.
You are receiving this because you were mentioned.Message ID: @.***>

ESPAsyncWebServer works great from one of the examples like simple server. But when I try and use the GPIOViewer example, I get the error below. I have tried this on multiple machines and not sure what the issue is.
In file included from C:\Users\rober\AppData\Local\Temp.arduinoIDE-unsaved202406-5396-y313rb.c4qme\gpioviewer\gpioviewer.ino:8:
c:\Users\rober\OneDrive\Documents\Arduino\libraries\GPIOViewer\src/gpio_viewer.h:11:10: fatal error: ESPAsyncWebServer.h: No such file or directory
11 | #include <ESPAsyncWebServer.h>
  |          ^~~~~~~~~~~~~~~~~~~~~

compilation terminated.
exit status 1
Compilation error: exit status 1
From: thelastoutpostworkshop @.>
Sent: Saturday, January 6, 2024 2:43 PM
To: thelastoutpostworkshop/gpio_viewer @.>
Cc: rescalle83 @.>; Mention @.>
Subject: Re: [thelastoutpostworkshop/gpio_viewer] Newbie here and getting the following errors when trying to compile (Issue #60)
The library ESPAsyncWebServer is not installed, recheck.
Try some examples that comes with the ESPAsyncWebServer.
—
Reply to this email directly, view it on GitHub https://github.com/thelastoutpostworkshop/gpio_viewer/issues/60#issuecomment-1879823108 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ALY7EJPLCOHH4XVG3TP4IF3YNGZLPAVCNFSM6AAAAABBPWQW7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZZHAZDGMJQHA .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/ALY7EJNXNRRBOM2FE34AS7DYNGZLPA5CNFSM6AAAAABBPWQW7WWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTQBPJQI.gif Message ID: @.*** @.***> >

@rescalle83 you may have problems in your Arduino IDE setup.  If your are getting this message :
c:\Users\rober\OneDrive\Documents\Arduino\libraries\GPIOViewer\src/gpio_viewer.h:11:10: fatal error: ESPAsyncWebServer.h: No such file or directory
It is because the ESPAsyncWebServer cannot be found in the project where you are using the GPIOViewer library.

Not sure what different settings I can use. I can compile everything except GPIO. I even changed from DEV board to stable and loaded 2.0.14 and still have the same issue.
I can uninstall again and try to install it again. Not sure what I am doing wrong … odd
From: thelastoutpostworkshop @.>
Sent: Saturday, January 6, 2024 4:37 PM
To: thelastoutpostworkshop/gpio_viewer @.>
Cc: rescalle83 @.>; Mention @.>
Subject: Re: [thelastoutpostworkshop/gpio_viewer] Newbie here and getting the following errors when trying to compile (Issue #60)
@rescalle83 https://github.com/rescalle83  you may have problems in your Arduino IDE setup. If your are getting this message :
c:\Users\rober\OneDrive\Documents\Arduino\libraries\GPIOViewer\src/gpio_viewer.h:11:10: fatal error: ESPAsyncWebServer.h: No such file or directory
It is because the ESPAsyncWebServer cannot be found in the project where you are using the GPIOViewer library.
—
Reply to this email directly, view it on GitHub https://github.com/thelastoutpostworkshop/gpio_viewer/issues/60#issuecomment-1879861702 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ALY7EJO6WTL4TJEUYKGDC4TYNHGW7AVCNFSM6AAAAABBPWQW7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZZHA3DCNZQGI .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/ALY7EJOG2ZQ2RG7NDEHVF4DYNHGW7A5CNFSM6AAAAABBPWQW7WWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTQBRU4M.gif Message ID: @.*** @.***> >

I have an error related to that:
Installed the FreeRTOS Library but didn't work
Arduino\libraries\GPIOViewer\src/gpio_viewer.h:17:10: fatal error: FreeRTOS.h: No such file or directory
17 | #include <FreeRTOS.h>
|          ^~~~~~~~~~~~
compilation terminated.
exit status 1
Compilation error: exit status 1

See the troubleshooting section : https://github.com/thelastoutpostworkshop/gpio_viewer#troubleshooting

Thanks. Followed this and no issues compiling without GPIOViewer. Once I add and try to compile, I get the same error. It may be I am using the C6 devkit. I have tried everything and the only variable that prevents compile is GPIOViewer. I will try again at some point when C6 is supported.
Thanks.

From: thelastoutpostworkshop @.>
Sent: Sunday, January 7, 2024 3:06:56 PM
To: thelastoutpostworkshop/gpio_viewer @.>
Cc: rescalle83 @.>; Mention @.>
Subject: Re: [thelastoutpostworkshop/gpio_viewer] Newbie here and getting the following errors when trying to compile (Issue #60)
See the troubleshooting section : https://github.com/thelastoutpostworkshop/gpio_viewer#troubleshooting
—
Reply to this email directly, view it on GitHubhttps://github.com/thelastoutpostworkshop/gpio_viewer/issues/60#issuecomment-1880171946, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALY7EJNCRTCTNOA347S4IJTYNME7BAVCNFSM6AAAAABBPWQW7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBQGE3TCOJUGY.
You are receiving this because you were mentioned.Message ID: @.***>

Please add print screen of your boards version and boards selected, like this:

From: thelastoutpostworkshop @.>
Sent: Monday, January 8, 2024 2:45 PM
To: thelastoutpostworkshop/gpio_viewer @.>
Cc: rescalle83 @.>; Mention @.>
Subject: Re: [thelastoutpostworkshop/gpio_viewer] Newbie here and getting the following errors when trying to compile (Issue #60)
Please add print screen of your boards version and boards selected, like this:
image.png (view on web) https://github.com/thelastoutpostworkshop/gpio_viewer/assets/127604345/917c17ea-d1ec-4981-bd77-1249a9fa401b
image.png (view on web) https://github.com/thelastoutpostworkshop/gpio_viewer/assets/127604345/fff6f672-94e7-4068-afad-f0035d42eba2
—
Reply to this email directly, view it on GitHub https://github.com/thelastoutpostworkshop/gpio_viewer/issues/60#issuecomment-1881793390 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ALY7EJOM2SOYOWCOASNVLCTYNRLFVAVCNFSM6AAAAABBPWQW7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBRG44TGMZZGA .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/ALY7EJNULTKPCKNPXNY5EITYNRLFVA5CNFSM6AAAAABBPWQW7WWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTQFHRW4.gif Message ID: @.*** @.***> >

I dot not see your print screens, maybe they do not show up when your answer by email.  Go to github and add them in this issue.

I have tested it only on stable release 2.011 - 2.014

Thanks. Hopefully there will be support for C6 and the latest board package. I will continue to watch for future support.
The board I need support for is the ESP32-C6-DevKitC-1
From: thelastoutpostworkshop @.>
Sent: Monday, January 8, 2024 3:50 PM
To: thelastoutpostworkshop/gpio_viewer @.>
Cc: rescalle83 @.>; Mention @.>
Subject: Re: [thelastoutpostworkshop/gpio_viewer] Newbie here and getting the following errors when trying to compile (Issue #60)
I have tested it only on stable release 2.011 - 2.014
—
Reply to this email directly, view it on GitHub https://github.com/thelastoutpostworkshop/gpio_viewer/issues/60#issuecomment-1881876074 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ALY7EJOOLN73U27MFZMOWBTYNRSZBAVCNFSM6AAAAABBPWQW7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBRHA3TMMBXGQ .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/ALY7EJL7UXPRYWOM2ZTI5Z3YNRSZBA5CNFSM6AAAAABBPWQW7WWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTQFMTGU.gif Message ID: @.*** @.***> >


Config validations

Adding some validations

Sorry, it was done by mistake

Sorry, it was done by mistake


Patch for /trunk/MoreLinq/DistinctBy.cs #72

While your implementation is logically equivalent, it's less helpful than the current one in the sense that it causes an extra allocation of a closure for lambda passed to the Where.


Config keys should keep consistent with k8s

Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[x] User Experience
[ ] Developer Infrastructure

@hzxuzhonghu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e-dashboard_istio
8d7183ea1568ec354a2ddb94294ec8e2590063fd
link
/test e2e-dashboard_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e-dashboard_istio
8d7183ea1568ec354a2ddb94294ec8e2590063fd
link
/test e2e-dashboard_istio

integ-pilot-k8s-tests_istio
8d7183ea1568ec354a2ddb94294ec8e2590063fd
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e-dashboard_istio
8d7183ea1568ec354a2ddb94294ec8e2590063fd
link
/test e2e-dashboard_istio

integ-pilot-k8s-tests_istio
8d7183ea1568ec354a2ddb94294ec8e2590063fd
link
/test integ-pilot-k8s-tests_istio

e2e-simpleTests-distroless_istio
8d7183ea1568ec354a2ddb94294ec8e2590063fd
link
/test e2e-simpleTests-distroless_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@hzxuzhonghu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e-simpleTests_istio
8d7183ea1568ec354a2ddb94294ec8e2590063fd
link
/test e2e-simpleTests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@hzxuzhonghu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
8d7183ea1568ec354a2ddb94294ec8e2590063fd
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
8d7183ea1568ec354a2ddb94294ec8e2590063fd
link
/test lint_istio

integ-galley-k8s-tests_istio
8d7183ea1568ec354a2ddb94294ec8e2590063fd
link
/test integ-galley-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@hzxuzhonghu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
8d7183ea1568ec354a2ddb94294ec8e2590063fd
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
8d7183ea1568ec354a2ddb94294ec8e2590063fd
link
/test integ-security-k8s-tests_istio

gencheck_istio
dd79bcfa0e6d6a171ef143b3682a68fdb11c38c1
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
8d7183ea1568ec354a2ddb94294ec8e2590063fd
link
/test integ-security-k8s-tests_istio

gencheck_istio
dd79bcfa0e6d6a171ef143b3682a68fdb11c38c1
link
/test gencheck_istio

unit-tests_istio
dd79bcfa0e6d6a171ef143b3682a68fdb11c38c1
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
8d7183ea1568ec354a2ddb94294ec8e2590063fd
link
/test integ-security-k8s-tests_istio

gencheck_istio
dd79bcfa0e6d6a171ef143b3682a68fdb11c38c1
link
/test gencheck_istio

unit-tests_istio
dd79bcfa0e6d6a171ef143b3682a68fdb11c38c1
link
/test unit-tests_istio

lint_istio
dd79bcfa0e6d6a171ef143b3682a68fdb11c38c1
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

unit-tests_istio
dd79bcfa0e6d6a171ef143b3682a68fdb11c38c1
link
/test unit-tests_istio

lint_istio
dd79bcfa0e6d6a171ef143b3682a68fdb11c38c1
link
/test lint_istio

gencheck_istio
e8b46cb65fd05ee3412847c4991ad5ffb9b9dbc3
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@hzxuzhonghu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_istio
e8b46cb65fd05ee3412847c4991ad5ffb9b9dbc3
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Dots in Routes giving 404

When you have the code:
Route::get('attachments/{file}', function($file){
   return Reponse::json($file);
});

And you will use the following url:
http://website.com/attachments/mydocument.pdf, it returns a 404.
Because of the dot in the filename. I wanna have the dot as well in the route, that is not possible now for some reason.

hi when i remove dot and percentage symbol from url parameter like (http://localhost:8000/activate/%242y%.sroG) to (http://localhost:8000/activate/242ysroG) then its work and Route accepted, if url contains these two characters then Route do not accepted that url and give 404 error in Laravel 5.3. Why its perform that behavior


175538989 add support for converting parametrized gates

This is a (hopefully) final step in our efforts to implement the conversion of arbitrary Orquestra gates to PyQuil gates.
At this stage, it should be possible to convert an Orquestra gate to PyQuil gate even if it contains symbolic parameters. The symbolic parameters are also collected, i.e. appropriate variable declarations are also added to the corresponding PyQuil program in which context's the conversion is happening.

@simonwa7

One question I have is if we convert from pyquil -> zapata -> pyquil or zapata -> pyquil -> zapata, do we retain all the same information? If so, we should try to test these two cases.

We don't have pyquil -> zapata conversion yet, but I can already tell you that this won't work in some cases when modifiers are used. The reason is, that in our gates we optimize modifiers so e.g. X(0).dagger or X(0).dagger.dagger is just X(0) because X is hermitian, but this is not the case in PyQuil (which makes sense, such optimizations are probably left to Quil compiler).

The only other thing I would add is we should add a simple test case where we pass in an empty circuit and try to do the conversion and make sure everything works as expected.

Good idea, I added one in 680e84b.


Add support for requirements.in (from pip-tools)

pip-tools provides a tool called pip-compile that takes a
requirements.in file with the same format used by requirements.txt to produce a "lock file"
(the usual requirements.txt file).
This commits enables the pip-requirements syntax for the aforementioned files.

Thanks for hte PR, @lvillani ! We are in code freeze for our next release, but we will look at this after we open the repo back up to feature work.

FYI @lvillani the PR looks good, but I need to be in a position to add a news entry before I can merge (unless you want to add a news entry yourself 😁 ).

Hey @brettcannon, sure I can add the news entry myself. I'm unfamiliar with the process, do I have to create a file called "961.md" in the "1 Enhancements" directory?

@lvillani https://github.com/Microsoft/vscode-python/blob/master/news/README.md should explain what to do (and if it doesn't please let me know as you're the first external contributor to do this 😁 ).

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: lvillani sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@brettcannon I have added this file and rebase the branch against master. I can also squash all commits together if you want.

Thanks, @lvillani !


The ground truth of Test_026 too

Can you send me one tif format annotation?I will be very grateful to you. My mail<EMAIL_ADDRESS>

@Nibbles55 Hi, I've sent you the file through my email


Various autoscale fixes

1. Reference the right fields in the trigger inside the new relic critical and recovery rule
https://gist.github.com/Kami/2d424ba856d1799809d5
Rule was referencing trigger.long_description field, but it looks like it should be trigger.alert.long_description since the top level trigger instance object doesn't have this attribute. Same issue exists for trigger.alert_policy_name - should be trigger.alert.alert_policy_name.
Here is a trigger instance in question:
{'alert': {u'account_name': u'fryman.io',
           u'alert_policy_name': u'Default server alert policy',
           u'alert_url': u'https://rpm.newrelic.com/accounts/880740/incidents/11717078',
           u'created_at': u'2015-02-03T01:07:00Z',
           u'long_description': u'Alert opened: CPU > 80%',
           u'message': u'CPU > 80%',
           u'server_events': [{u'created_at': u'2015-02-03T01:07:00Z',
                               u'message': u'CPU > 60%',
                               u'server': u'glaswegians-clops'},
                              {u'created_at': u'2015-02-03T01:07:00Z',
                               u'message': u'CPU > 80%',
                               u'server': u'glaswegians-clops'}],
           u'servers': [u'glaswegians-clops'],
           u'severity': u'critical',
           u'short_description': u'New alert on glaswegians-clops'},
 'header': {'Accept': u'*/*',
            'Content-Length': u'844',
            'Content-Type': u'application/x-www-form-urlencoded',
            'Host': u'localhost:10001',
            'User-Agent': u'curl/7.35.0'}}

And looking at the sensor, the fix is indeed correct since all the attributes are included inside the alert dict https://github.com/StackStorm/st2contrib/blob/master/packs/newrelic/sensors/newrelic_application_sensor.py#L168
2. Use the correct action parameter name inside the new relic critical and recovery alert rule
https://gist.github.com/Kami/a0bbfeb8d37993c8fd8b
autoscale.{crit,recovery_aler} action expects attribute application and not application_name as an input. See https://github.com/StackStorm/st2incubator/blob/master/packs/autoscale/actions/crit_alert.yaml#L8.
3. Fix typo in crit_alert.yaml workflow
Exception: https://gist.github.com/Kami/1c559790ee33cfa8a930
Fix in: d40e2dae0cd109b52a5b52a508aa5e4ad54449b1
4. Fix max_bound check in the scaling governor
98df9ce582ca467cb4bcd1b313f493f3b73abce2
5. Fix trigger name
Exception: https://gist.github.com/Kami/cd5b79563607c53aa74d
Fix: 637cdda69112d3037543898be730c93483828255
6. Fix a bug with adding a node to the loadbalancer and dns record creation failing
Sometimes those two tasks would fail because we picked up IPv6 and not IPv4 address (order in public_ips attribute is not guaranteed).
Fix: e0a1c8d8ee78f03599e78ca86861bf4d637fb8c1

:+1:


dateFormat as with previous versions

Hi,
I've read info about format/localization - multiple times - still can't figure out how to obtain results similar to those (v.5.0-beta):
<owl-date-time
...
[dateFormat]="'DD.MM.YYYY'"
...

...
<owl-date-time
...
[dateFormat]="'YYYY/MM/DD'"
...

Could you please show an example with two diffrent date formats? Or maybe it can be change dynamically?

There are examples on the demo pages


how to add custom role that can assign spacific roles only to users not all the available roles

the case that i have is
i want to create a role ex- admin
and this admin have the ability to add users and assign two roles only ex- viewer , publisher to the added users
how to do something like this ??
superset version 1.4.1
Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Closing this since
• it's remained inactive for over a year
• it's more feature/how-to request than bug report.
• it's regarding a no-longer-supported version of Superset.
If you wish to keep digging into this, I'd suggest you bring it to Superset Slack or take it up in a Q&A thread in our Github Discussions area.


Make cluster presence detection an optional feature

What would you like to be added:
I would like to add the ability to flag certain clusters as persistent, meaning that Lens should not report these clusters as being invalid when they can not be reached.
Why is this needed:
I do support many clusters. Some are private cloud, others are in public cloud. Some require a VPN connection, others a proxy to be reachable.
Currently Lens is checking if these clusters are reachable which they are not always. If a cluster can not be reached a warning pops up with the recommendation to fix the .kube/config file.
Environment you are Lens application on:

Kubernetes distribution: AKS, Rancher, Minikube, KinD, K3S
Desktop OS: macOS 11.2.3 with Lens 4.1.4

Hi, what do you mean by "cannot be reached"? If Lens cannot connect to a cluster it will not report the cluster as invalid. In Lens 4.1.4 we added some kubeconfig validation, which checks if the expected fields are present (context, cluster, user, etc). Is this what is happening? Does your context get removed from the kubeconfig (by something other than Lens) when it becomes unreachable?

That is exactly what is happening. I was not aware of the details. But yes, the kubeconfig file is modified automatically.
But these clusters often are added again as well so it would be nice if they stay in the group in which they were placed.

Ok, I think this is not what we expected in terms of cluster workflow and how clusters become unreachable. Are you essentially deleting a cluster and creating a new one with the same name? Can you make it so that the cluster is not removed from the kubeconfig?
FYI if you cancel the invalid kubeconfig notification, the cluster won't be removed from Lens. However, to see the cluster again in Lens, once the cluster is restored in the kubeconfig, you must restart Lens. The cluster will then appear again in the UI. I know this is not the best UX, we could possibly support it on reload as well.

I understand that it is not a common way of working. But at the moment I do have 26 clusters in my kubeconfig.
And I do love Lens to work with them, it's the best I've seen so far.
I guess I'll have to restart Lens every now and then when the kubeconfig changes, not really a big deal, but it would have been nice if I didn't get the notification that a cluster has been removed.
Maybe that's a better request to submit. To be able to silently accept that clusters have been removed...
Thanks for your time on this issue.


Set transparency to get heatmap to display

Fixes heatmap sample
Removed Read Setting service sample - Per Caleb's suggestion

LGTM


upload package to artifact and OSS nightly

Try to search whether there's some action like oss upload.

OK


Qingwei/expose destroy

I did a publish since you've merge the previous PR.
This PR changes only PlainChart thus is safe, I just change the api a bit as Amin need to destroy chart manually

Coverage decreased (-0.06%) to 36.433% when pulling 5ebffc069fd40eca6b4a35db971b0926db7edbeb on qingweibinary:qingwei/expose_destroy into 9904fb33a1c870447abda0806f9aa497f3d8ef7d on binary-com:master.


phpExtensions.uv: init at 0.3.0

Description of changes
Adds the uv extension for php. Here are some links:

https://github.com/amphp/ext-uv
https://pecl.php.net/package/uv

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build phpExtensions.uv

I did it myself in GitHub ;)
Thanks for your contribution, keep them coming!


Adding layer alpha values not elegant

pix.alphas .= max.(pix.alphas, layer.alphas)
This means that if we add 2 0.5 alpha layers, the final alpha is also 0.5, when maybe it should be something like 0.75?

This has been kinda fixed by #64 as a design change. Now each individual @fum will deal with the incoming color separately


Apply additional rspec settings

I like these changes 👍

Not familiar with any of these settings. Is there more rationale or description on what they do?

I googled each of them:
RSpec/DescribeClass enforces using the class name in a describe block
# bad
describe 'Do something' do
end

# good
describe TestedClass do
end

RSpec/DescribedClass enforces using the described_class variable instead of the class:
# bad
describe MyClass do
  subject { MyClass.do_something }
end

# good
describe MyClass do
  subject { described_class.do_something }
end

RSpec/MessageExpectation enforces using either expect or allow for stubs:
# bad
expect(foo).to receive(:bar)

# good
allow(foo).to receive(:bar)

RSpec/MultipleExpectations enforces rules about having multiple expect calls in an it block
RSpec/NestedGroups enforces rules about how many context or describe blocks can be nested inside of others.
All of these cases are situational, and it should be left to the developer to determine the best way to write their tests.

Thanks! Based on what you've put I agree on the changes. My only request would be to better document in .rubocop.yml what they're doing and why we're disabling them. The documentation that is there depends on knowing what they mean and the commit doesn't go into any detail so git blame isn't any help.

And this PR will be hard to find once it's been merged.

100% agree. @albus522, can you amend the commit and provide more documentation, or add more documentation in the file?

@kopchickm it looks like you already did. You could just move it into the file


Inform users how to join SQLite private beta

it would be good to do some pre-deployment validation to help give users information to sign-up for SQLite private beta. We could bake this into spin cloud deploy as @macolso suggests:
NoOps SQL DB is currently in private beta. Please visit our developer documentation for instructions on how to request access: https://developer.fermyon.com/cloud/noops-sql-db

This would also be helpful for the spin cloud sqlite subcommands, which apparently will currently just fail with:
❯ spin cloud sqlite list
Error: Problem listing databases

Caused by:
    Exception of type 'Nimbus.Application.Exceptions.ForbiddenAccessException' was thrown.

Is this now moot?

Yes, we should close this


batch-correction/integration

[ ] unserialize with integrated datasets ?

[ ] UI cleanup

[ ] validation

[ ] error catching

[ ] better messages

Default shouhld be MNN correction as soon as multiple batches are selected.
If only a single batch is selected, the correction panel should only have one choice - none.

Only show the name text boxes as soon as multiple batches are requested.