specify that CoerceUnsized should ignore PhantomData fields

Otherwise you can't have a Smaht<T>(*const T, PhantomData<T>), which is problematic for drop check, which requires this annotation if Smaht owns the referrent (Box, Rc, Vec, etc..).
See https://github.com/rust-lang/rust/issues/26905 for some discussion.
You could arguably ignore all zero-sized fields, but it's currently hard for the compiler to do so (size is largely a trans notion, to my knowledge). Also maybe you actually want a generic ZST field that has some meaning beyond PhantomData?
A potential workaround to this problem would be Smaht(*const u8, PhantomData<T>) but this is entering a fairly degenerate state for the semantics of raw pointers. Although we're already in a bit of a degenerate state in that all Smaht pointers desire some level of mutable access, but are forced to use *const to acquire the correct variance. We paper over that mess via Unique -- we could consider just embracing that and making Unique the only proper way to build Smaht pointers.
Of course Unique is incorrect for Rc, but I plan on adding a Shared equivalent for Rc anyway which could have similar magicks.

+1
Alternatively we could allow this for all ZSTs, and just implement for PhantomData for now. Special casing PhantomData is pretty much an implementation detail.
Could we approximate zero-sized == no non-zero-sized fields? Presumably we'd need to wait for the drop flag to disappear. Then just assert that we were right in trans.

:+1:

I agree with @nrc that the text shouldn't be specific to PhantomData just because of the current implementation issues. But otherwise :+1: from me.

I guess the question is whether unsizing two fields is forbidden for semantic or implementation (offsets fall over with multiple unsized fields) reasons -- if it's semantic then PhantomData is the only type we really know doesn't matter. If it's implementation then presumably all ZSTs should be fine.

if we're going to patch in place, can we make a section at the end entitled something like "Updates since being accepted" and record the changes

@nikomatsakis done.

So, regarding the specialization to PhantomData, it's unclear to me why we even need to limit this to ZST. IIRC, we could permit coercion of any number of fields (whether ZST or not) within the smart pointer type. I presume that the current limitation is there to ease implementation (and be conservative).
@eddyb thoughts?

@nikomatsakis I would suggest limiting this as much as possible, otherwise it might allow writing transmute in safe code.
Not sure if this is what you're referring to, but removing the requirement that exactly one field coerces via unsizing would only require an impl<T, U> CoerceUnsized<PhantomData<U>> for PhantomData<T> (and more impls could be added for each ZST that needs it).
However, I don't think we should allow those 0-fields-unsizing coercions to actually happen in expressions (except that if the impl exists, it can't be guarded against it being used in coercions due to generics).

@eddyb can you explain what you mean by "allow writing transmute in safe code"? (e.g., maybe give an example) Perhaps I have to refresh my memory of just which traits we're talking about here.

@nikomatsakis The transmute remark was about the worst case, where arbitrary types are converted without an actual coercion, e.g:
struct Foo<P, T>(Box<P>, T);
impl<P: Unsize<Q>, T, U> CoerceUnsized<Foo<Q, U>> for Foo<P, T> {}

Though I doubt you had that in mind.
Then there's odd usecases of ZSTs, such as those contravariant over a lifetime acquired from borrowing some resource (a "token" type).
But generic ZSTs are either PhantomData or have ZST fields, and in the latter case a CoerceUnsized impl for the ZST itself would be required anyway (and would not hold due to having no actual unsizing happening).
So whether you ignore all struct S<...>; or just PhantomData, it doesn't matter, only PhantomData can have that form.

We discussed this today in the language subteam meeting. The conclusion was that while many of us believe we can generalize DST coercions further (read: @nikomatsakis does, at least), we should not let the perfect be the enemy of the good. Therefore, we should take this RFC, and consider further generalizing the rules another time.

Hear ye, hear ye. This RFC is entering final comment period.

Ah, @Gankro, can you amend the RFC to include as an "unresolved question" the extent to which coercion can be generalized to simply permit multiple fields to be coerced?

@nikomatsakis done

Huzzah. The lang subteam has decided to accept this RFC. Tracking issue https://github.com/rust-lang/rust/issues/28239.


To learn: check microservices status and update chatbot status

Errbot has polling method, and MongoDB has ping command.


Heap size calculation for envoy

Title: Heap size calculation for envoy
Description:
We use envoy v1.26.5. We deploy it on the Kubernetes. We configured overload_manager. As I wrote in https://github.com/envoyproxy/envoy/issues/28948 we had an issue with heap constantly growing.
We are trying to calculate some dependency between envoy heap size and some metrics like: routes amount, amount of clusters for those routes, amount of requests per second.
For example, we have
10 clusters,
for each cluster we have a route:
                      routes:
                        - match:
                            prefix: /
                          request_headers_to_add:
                            - header:
                                key: "X-Forwarded-Host"
                                value: "my-host"
                            - header:
                                key: "X-Forwarded-Port"
                                value: "%DOWNSTREAM_LOCAL_PORT%"
                            - header:
                                key: "X-Forwarded-Proto"
                                value: "%PROTOCOL%"
                          route:
                            host_rewrite_literal: my-host-rewrite:8080
                            cluster: my-host-rewrite||my-host-rewrite||8080
                            timeout: 120s

For 300 requests per second envoy memory ~ 100 mb.
Can you help with that?

is 300 rps  for 100 mb is a normal value?
what parameters/metrics influence heap size?
is there any configuration to minimize/decrease heap size? for example, turn off statistics or other features...

It highly varies on what all features are used in Envoy and the concurrency. 100MB for 300 RPS seems high. Decreasing the number of stats (via prefix exclusion/inclusion), reducing xDS are all potential ways to improve memory efficiency.
You can try to capture a heap dump and see where all memory went. It could also be caused by a spike and memory is not returned to OS.
CC @KBaichoo

You can try to capture a heap dump and see where all memory went

+1 I think this will provide great insight for your question.

You can also check /memory admin endpoint? It reports the internal stats about the memory use which would help you see if how TCMalloc is slicing the memory for the application.

pprof gave me the following picture for envoy heap:

this is /memory admin endpoint output:
{
 "allocated": "11780240",
 "heap_size": "115343360",
 "pageheap_unmapped": "13443072",
 "pageheap_free": "35733504",
 "total_thread_cache": "21652464",
 "total_physical_bytes": "106693326"
}

The pprof isn't too useful due to it being not symbolized -- a debug build would allow us insights there.
Regardless, the numbers from the memory endpoint suggest to me that lots of memory is available and free but
tcmalloc background release also needs to be configured for it to be able to give back memory in the background (vs when triggering overload action).
https://google.github.io/tcmalloc/tuning.html#memory-releasing
Default: https://github.com/google/tcmalloc/blob/7714d76c9e1dec7a0e5889da2228694c7ec10180/tcmalloc/parameters.cc#L159
AFAIK we haven't exposed setting the background release rate https://github.com/google/tcmalloc/blob/7714d76c9e1dec7a0e5889da2228694c7ec10180/tcmalloc/parameters.h#L40 and should consider doing so.

I run envoy:debug-v1.26.5 and get the following image from pprof:

Please, plan to merge https://github.com/envoyproxy/envoy/pull/30353.


Syncing session state across browsers when using sessionStorage

Core Library
MSAL.js v2 (@azure/msal-browser)
Core Library Version
2.14.1
Wrapper Library
MSAL React (@azure/msal-react)
Wrapper Library Version
None
Description
Hi,
If a user has duplicated a tab in Chrome whilst the cache location is set to sessionStorage, we have noticed that logging a user out of one tab does not log them out of the duplicated tab.
We followed this guide and are able to make it work when the cache location is set to localStorage.
Is it just not possible for sessionStorage?
Thanks
MSAL Configuration
No response
Relevant Code Snippets
No response
Identity Provider
Azure B2C Custom Policy
Source
External (Customer)

@talhatfl It is not possible, unfortunately, as storage events for sessionStorage are only fired for the multiple frames of the same website within the same tab, but not across different tabs.

thanks for the update @jasonnutter


fix(dao) target dao and endpoints

Summary
fix(dao) iteration to pass options to row to entity
Iterator had issue that it did not pass options to when it called
dao:row_to_entity.
E.g. http :8001/upstreams/upstream/targets returned:
{
    "data": [
        {
            "created_at":<PHONE_NUMBER>.497,
            "id": "06f8e88f-f874-44ab-a6bf-989e58a505ba",
            "target": "<IP_ADDRESS>:8000",
            "upstream": {
                "id": "5f3a5828-7e09-4d80-b899-41b0111510cd"
            },
            "weight": 100
        }
    ],
    "next": null
}

Instead of:
{
    "data": [
        {
            "created_at":<PHONE_NUMBER>.497,
            "id": "06f8e88f-f874-44ab-a6bf-989e58a505ba",
            "tags": null,
            "target": "<IP_ADDRESS>:8000",
            "upstream": {
                "id": "5f3a5828-7e09-4d80-b899-41b0111510cd"
            },
            "weight": 100
        }
    ],
    "next": null
}

On Admin API (see the missing tags=null).
fix(dao) targets to pass options on custom dao methods
Targets did not pass the options on custom dao calls, this
commit fixes that.
fix(dao) implement upserts as synonym to inserts on targets
Targets gave 500 error on:
http PUT :8001/upstreams/example.com/targets/<IP_ADDRESS>

This commit fixes that.
fix(api) upstream targets endpoints fixes

disallow (405) PATCH on /upstreams/:upstreams/targets/:targets
make GET to do proper pre-checks on /upstreams/:upstreams/targets/:targets
make PUT to do proper pre-checks on /upstreams/:upstreams/targets/:targets

is this for a specific bug or just something you noticed while in the area?

Yes, I just noticed it on my own.


use DOMContentLoaded instead of load for scopePollingDetectionStrategy

Currently the detection mechanism waits for the load event to check for the wallets. This can take a lot of time incase a resource such as an image required by the page keeps loading for a while. This prevents the user from getting the correct UI, does not portray correct state of installed/loadable wallets.
We can use DOMContentLoaded instead of load  to performDetection once the DOM structure is loaded.
Current implementation:
function scopePollingDetectionStrategy(performDetection: () => boolean): void {
    if (typeof window === 'undefined' || typeof document === 'undefined') return;
    if (document.readyState === 'complete') {
        performDetection();
        return;
    }
    function listener() {
        window.removeEventListener('load', listener);
        performDetection();
    }
    window.addEventListener('load', listener);
}

Suggested Change:
function scopePollingDetectionStrategy(performDetection: () => boolean): void {
    if (typeof window === 'undefined' || typeof document === 'undefined') return;
    if (document.readyState === 'complete') {
        performDetection();
        return;
    }
    function listener() {
        window.removeEventListener('DOMContentLoaded', listener);
        performDetection();
    }
    window.addEventListener('DOMContentLoaded', listener);
}

Found #215 already exists, closing.

#215 is just a proposal -- most of the work is on wallets to implement. If that's the only reason to close this, I'd suggest reopening and let's see if it makes sense to PR.

Related: https://github.com/solana-labs/wallet-adapter/pull/327
@sjchmiela + @vpontis wdyt?

Great suggestion, @ArpitAgrawal321! I'm trying to load this all back into my head to figure out why I didn't do this in the first place.
I think maybe it's because the check for document.readyState was equivalent to listening to DOMContentLoaded in most cases, because usually the wallet code by definition doesn't execute until after DOMContentLoaded has fired. This might not be true for all apps, so this could very well be worth adding.
I dug around and found some of my early exploration into this with Phantom: https://jsfiddle.net/tb8gy7fc/1/
In any case, we should definitely:

Do what you've suggested.
Only set up the DOMContentLoaded trigger if the document.readyState === 'loading' (ie. otherwise we're past the point of DOMContentLoaded having fired already)
Keep the load trigger, in case the wallet still isn't ready on DOMContentLoaded.

Hey from Magic Eden - we did a patch package to use DOMContentLoaded instead of load since our pages are image dependent. Happy to implement @steveluscher's suggestion to push out the optimization

@safaiyeh nice! And yes please, that would be great!

cool opened here! https://github.com/solana-labs/wallet-adapter/pull/346

Closed by #346!


Added #include for compilation to succeed

Upstream fix: https://github.com/Azure/azure-sdk-for-cpp/pull/5766


Build css from scss sources in client ui build

Description

This PR integrates the compiling of the scss sources in the build of the client ui module, so the css files can be removed from the sourcecode.
I also moved the a folder which contains all the frontend resources from the root directory to the client a folder, because they get only used in this module.

Furthermore, I readded the empty custom.css file which was missing due to the changes in the refactoring if the styles. this regression was introduced in PR #3328

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] New feature (non-breaking change which adds functionality)
[X] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

How Has This Been Tested?
Deployed this to our test environment and tested functionality.

There are many conflicts, will this break UI building/usage and introduce a new dependency (npm etc.)?

I will split the bugfix and the build changes in two PRs.


Proxy Protocol support in kube-vip

Is your feature request related to a problem? Please describe.
https://github.com/k3s-io/k3s/discussions/2997#discussioncomment-449838
https://github.com/moby/moby/issues/39465
https://github.com/moby/moby/issues/25526
https://github.com/traefik/traefik-helm-chart/issues/380
Getting real client-ip is a very big issue that comes all over the place. The only real way to solve this in a scalable, standards compliant way is to inject Proxy Protocol headers
Describe the solution you'd like
Proxy protocol is a fully open standard and supported by most known network servers:

https://www.haproxy.org/download/1.8/doc/proxy-protocol.txt
https://docs.aws.amazon.com/elasticloadbalancing/latest/classic/enable-proxy-protocol.html
https://cloud.google.com/load-balancing/docs/tcp/setting-up-tcp#proxy-protocol
https://docs.openshift.com/container-platform/3.9/install_config/router/proxy_protocol.html
https://docs.traefik.io/configuration/entrypoints/#proxyprotocol
https://docs.nginx.com/nginx/admin-guide/load-balancer/using-proxy-protocol/
https://aws.amazon.com/about-aws/whats-new/2013/07/30/elastic-load-balancing-now-supports-proxy-protocol/
https://avinetworks.com/docs/16.3/proxy-protocol-support/
https://blog.digitalocean.com/load-balancers-now-support-proxy-protocol/
https://www.brightbox.com/blog/2019/03/12/proxy-protocol-load-balancer-api/
https://kubernetes.github.io/ingress-nginx/user-guide/nginx-configuration/configmap/#use-proxy-protocol

kube-vip should inject proxy protocol v2 headers into the requests.

Kube-vip is not a proxy / load balancer - it's a control-plane component that provides VIPs. As it doesn't sit in the data path, I don't believe there's any scope for it to do anything with Proxy Protocol.

Closing as stale, please re-open if there is a need in the future.

@exfly kube-vip is still just Layer 4 (TCP/UDP) and relies on Kube-proxy rules for the service implementation. I don‘t get how it could inject L7 details.


Mips64 rust stub

Write in description, that you close this issue https://github.com/IvanArkhipov1999/Martos/issues/56


Minor issue in the documentation

There is some code in the documentation which probably does not belong there in the paragraph on:
https://github.com/harc/ohm/blob/master/doc/syntax-reference.md#terminology

Thanks for pointing this out. Should be fixed now...apparently GitHub changed the way they render unknown script types in their Markdown.


Question

Based on the demo it does not seem to allow you to filter based on zip code. Is it possible to add this?

You can put a zip code in the location, it'll only get homes in those zipcodes.


fix Live chat full screen mode, change page and style

@gauravsinhaweb now live chat look like this:

There is a Chat bot float button that I think is not on the best position in this page. What do you think?
Chat bot component exists on several pages. We can change it (move a little bit upper) for all of them, or just for this page.
If you think that it should be change only for live chat, I think it'll be better to create a new ticket( I can take it on me).

@Neniya, Nice Work 💯

@Neniya , I'm getting some error in deployment, can you fix it?
Error:
15:39:05.400  	Cloning github.com/gauravsinhaweb/EduSmart (Branch: live-chat-full-screen, Commit: 80afe53)
15:39:16.470  	Cloning completed: 11.070s
15:39:16.514  	Analyzing source code...
15:39:17.840  	Installing build runtime...
15:39:19.972  	Build runtime installed: 2.132s
15:39:22.492  	Looking up build cache...
15:39:31.671  	Build cache downloaded [180.71 MB]: 8945.617ms
15:39:32.961  	Detected package.json
15:39:32.962  	Installing dependencies...
15:39:33.280  	yarn install v1.22.11
15:39:33.389  	warning package-lock.json found. Your project contains lock files generated by tools other than Yarn. It is advised not to mix package managers in order to avoid resolution inconsistencies caused by unsynchronized lock files. To clear this warning, remove package-lock.json.
15:39:33.394  	[1/4] Resolving packages...
15:39:34.196  	warning react-scripts > webpack-dev-server > chokidar ><EMAIL_ADDRESS>fsevents 1 will break on node v14+ and could be using insecure binaries. Upgrade to fsevents 2.
15:39:34.406  	[2/4] Fetching packages...
15:40:04.359  	info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
15:40:04.360  	info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
15:40:04.360  	info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
15:40:04.360  	info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
15:40:04.364  	[3/4] Linking dependencies...
15:40:04.368  	warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>15:40:04.375  	warning "react-scripts > @typescript-eslint/eslint-plugin ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| >= 3.2.0-dev || >= 3.3.0-dev || >= 3.4.0-dev || >= 3.5.0-dev || >= 3.6.0-dev || >= 3.6.0-beta || >= 3.7.0-dev || >= 3.7.0-beta".
15:40:04.379  	warning "react-scripts > @svgr/webpack > @babel/preset-env ><EMAIL_ADDRESS>has incorrect peer dependency "<EMAIL_ADDRESS>15:40:04.392  	warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>15:40:04.392  	warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>15:40:04.392  	warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>15:40:04.392  	warning "react-simple-chatbot ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>15:40:04.392  	warning "react-simple-chatbot ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>15:40:04.393  	warning " ><EMAIL_ADDRESS>has unmet peer dependency "react-is@>= 16.8.0".
15:40:19.372  	[4/4] Building fresh packages...
15:40:20.908  	success Saved lockfile.
15:40:20.914  	Done in 47.64s.
15:40:21.529  	><EMAIL_ADDRESS>build /vercel/path0/client
15:40:21.529  	> craco build
15:40:25.169  	Creating an optimized production build...
15:41:06.845  	Failed to compile.
15:41:06.845  	src/pages/Chatroom.js
15:41:06.846  	Syntax error: Identifier 'room' has already been declared. (82:12)
15:41:06.846  	  80 |     const [showChat, setShowChat] = useState(false);
15:41:06.846  	  81 |
15:41:06.846  	> 82 |     const [room, setRoom] = useState("");
15:41:06.846  	     |            ^
15:41:06.846  	  83 |     const history = useHistory();
15:41:06.846  	  84 |     const joinRoomstudent = () => {
15:41:06.846  	  85 |         console.log("sds");
15:41:06.905  	npm ERR! code ELIFECYCLE
15:41:06.905  	npm ERR! errno 1
15:41:06.908  	npm ERR<EMAIL_ADDRESS>build: craco build
15:41:06.909  	npm ERR! Exit status 1
15:41:06.909  	npm ERR!
15:41:06.909  	npm ERR! Failed at the<EMAIL_ADDRESS>build script.
15:41:06.909  	npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
15:41:06.915  	npm ERR!


default repo should be ctan

@rultor release, tag is 0.10.1

@rultor release, tag is 0.10.1

@yegor256 Done! FYI, the full log is here (took me 2min)


Update interface returnErr

description
Returning 'nil' for interfaces in returnErr
More information in issue: #28

@nicolaa5 Good catch. Thanks for the PR!

Well done, this looks awesome!
I'm going to merge this PR, thank you very much for the contribution!


CGP-2 Support EPICS area under Git

We need the ADE to be able to release modules into the epics area.

Coverage remained the same at 90.463% when pulling 1cb012e63a755f9ef4a2d048e735e6bc5a13807e on CGP-2-epics into addb362152181857df748cda95552a978489d1ea on master.

I don't think I took enough care over your requirement. I've pushed a commit that should allow building on top of an existing module directory. Contrary to other modules, the epics release script now creates a $_module directory and checks $_version out in it, as opposed to cloning into a directory called $_module/$_version.

Yes, that looks more appropriate in the epics.sh script. It's all a bit convoluted, so please test it somewhere (not in prod) with a few different R<IP_ADDRESS>dlsNN tags in there to make sure it does the right thing...

I tested these changes building into /scratch. The releases stack as expected, updating the git repo then calling make && make clean && make install.

OK, try a test-release on the RHEL7 build server next. Using -r 7 -tT

Seems to have built correctly into
/dls_sw/work/etc/build/test/build_20180208-100139_uxj42447_epics_base_3-14-12-7/R<IP_ADDRESS>/base


Markdown: Syntax highlighting not displaying correctly

I've been writing markdown files in VScode recently, and I'm enjoying very much.
I'm having an issue with the preview display:
I can't get get syntax highlighting for code to work. In markdown view, the syntax highlighting does work, but in preview it just shows up as plain text in a gray box.  (I am currently specifically looking for VB)

Here is a screenshot of what I am seeing. You can see that on the left side, I am getting syntax highlighting for Python, but not for VBnet, and on the right side, I am not getting any syntax highlighting.

Just looking into this bug a little, it looks like some languages (css, php, html) are correctly syntax highlighted in the preview, while most others are not:

Syntax highlighting in the preview and the editor also do not always match. For example, php gets highlighted in the preview but not the editor, and python gets highlighted in the editor but not the preview.

Yes, we have some inconsistency. Plan to fix this when markdown gets moved to its own extension.

One of those inconsistencies is PHP only working when there is an open tag. It's not required for markdown highlighting:
// Does not have "<?php" tag.
if (1 + 2) {
  $x = 3;
}

I'm having this issue as well with Python inside Markdown code blocks. It highlights correctly in the editor, but in the preview it doesn't highlight at all.

Please reopen if you still see this with our new markdown extension we ship in June.

Verified using simple python and php examples.

Is there something different I have to do to "switch-on" for c#. Seems to work for Java but not c#

Running 1.3.0-insider with the c# extension 1.0.3-rc2
I have .NET Core RC2 Preview1 and .NET Core 1.0.0 RTM Preview 2 installed if that helps.

Works for me after upgrade to 1.3 (tested on Typescript and Html)! 👍
One question, though: is there a way to change the default color scheme? As can be seen above, the theme is not the same in MD editor (or VS Code generally) and in the preview and honestly... the default theme in preview is too much Skittles for me 😒


remove an old travis CI script

I do hope we can still get redox in the CI and make the support more official at some point.


fix for responsive resolutions, fix some crashes and warnings, and code cleaning

Merged, thank you so much for contributing and being patient with this!

No prob, kudos again for your work on this repo.


Visual Studio Code fails to open on macOS Big Sur 11.7.10: Error with OpenGL

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.91v
OS Version: Big Sur 11.7.10

I couldn’t get my Mac to open Visual Studio Code. In the console, I receive this error when trying to open it from the application icon:
Jul  5 13:22:57 TEST1s-MacBook com.apple.xpc.launchd[1]: Coalition Cache Hit: app<application.com.microsoft.VSCode.12926083832.12926083838(501)> [1561]
Jul  5 13:23:00 TEST1s-MacBook Code Helper (GPU)[2804]: getattrlist failed for /System/Library/Extensions/AppleIntelBDWGraphicsGLDriver.bundle/Contents/MacOS/AppleIntelBDWGraphicsGLDriver: #2: No such file or directory
Jul  5 13:23:00 TEST1s-MacBook Code Helper (GPU)[2804]: getattrlist failed for /System/Library/Frameworks/OpenGL.framework/Resources//GLRendererFloat.bundle/GLRendererFloat: #2: No such file or directory
Jul  5 13:23:01 TEST1s-MacBook com.apple.xpc.launchd[1] (application.com.microsoft.VSCode.12926083832.12926083838[2800]): Service exited due to SIGSEGV

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.91.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I have the same exact issue.

VS Code Version: 1.91.0
OS Version: Big Sur 11.7.10

Jul  6 01:48:45 Mac-One com.apple.xpc.launchd[1]: Coalition Cache Hit: app<application.com.microsoft.VSCode.33702760.33702766(501)> [720]
Jul  6 01:48:46 Mac-One Code Helper (GPU)[1057]: getattrlist failed for /System/Library/Extensions/AppleIntelHD5000GraphicsGLDriver.bundle/Contents/MacOS/AppleIntelHD5000GraphicsGLDriver: #2: No such file or directory
Jul  6 01:48:46 Mac-One Code Helper (GPU)[1057]: getattrlist failed for /System/Library/Frameworks/OpenGL.framework/Resources//GLRendererFloat.bundle/GLRendererFloat: #2: No such file or directory
Jul  6 01:48:46 Mac-One com.apple.xpc.launchd[1] (application.com.microsoft.VSCode.33702760.33702766[1054]): Service exited due to SIGSEGV

Probably https://github.com/microsoft/vscode/issues/219893

same exact thing here

same thing here. i am having the same issue as ghesshu when i try to open VSC. i also downloaded the latest version and nothing changed.

me too , i also removed the latest version and downloaded the previous version. I worked fine.

I have the same problem. I removed the old version and downloaded a new one, but the result is still the same. When I open VSCode, it appears for just a second and then crashes without any error messages.

Same here. When trying to start VSCode, the icon appears on the dock and fails to open without errors. The issue started yesterday. I was able to make VSCode work yesterday by restoring the key VSCode files/folders using a time machine backup but today same problem again.

I've been experiencing this issue since Thursday/Friday (or whenever the latest version was rolled out).
OS: MacOS Big Sur 11.7.10
VS Code: 1.91.0
As detailed in this issue 219893
I've uninstalled 1.91 and installed the previous stable version 1.90.2 and it runs perfectly.
Also gone to settings Update: Mode and set it to "none" to prevent VSC auto updating to 1.91

I did removed the latest one and downgraded to 1.90.2 too. This fixed the issue for now.
The reason seems to be related to the location of the GPU drivers (.dylib, bundle). Somehow, VScode looks into different directories.

/duplicate https://github.com/microsoft/vscode/issues/219893

Same issue I had yesterday. i just downgraded to Version: 1.90.2 (Universal) and it fixed the issue.

I'm experiencing the same issue too.
MacOS Big Sur 11.7.10
VS Code: 1.91.0

Big Sur only works for 1.90.2

@ClarenceC have you actually tried 1.91.1 on Big Sur? It is supposed to have fixed the issue that broke 1.91.0 on that platform. If 1.91.1 still fails for you please open a new issue with full details and link to it from here.


fix: ensure atlas has execute permission

Close, the permission already set by archive step:
https://github.com/ariga/terraform-provider-atlas/blob/87ccfe064c08c458425da5602a8b35dc76e00e05/.goreleaser.yml#L35-L43


Calling array2d_array with call fail with Error: Invalid number of arguments

Describe the bug
Calling array2d_array with call (see below for reproduction) fail with the following error but works just file if called with meta-class functions  i.e contract.array2d_array
/Users/mac/Workspace/abiwan-tutorial/node_modules/starknet/dist/index.js:3186
      throw Error(
            ^

Error: Invalid number of arguments, expected 1 arguments, but got 2
    at CallData.validate (/Users/mac/Workspace/abiwan-tutorial/node_modules/starknet/dist/index.js:3186:13)
    at /Users/mac/Workspace/abiwan-tutorial/node_modules/starknet/dist/index.js:6185:23
    at getCalldata (/Users/mac/Workspace/abiwan-tutorial/node_modules/starknet/dist/index.js:6104:10)
    at Contract.call (/Users/mac/Workspace/abiwan-tutorial/node_modules/starknet/dist/index.js:6183:22)
    at main (/Users/mac/Workspace/abiwan-tutorial/dist/example.js:14:68)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

To Reproduce
import { Provider, Contract, constants } from "starknet";

async function main() {
  const provider = new Provider({
    sequencer: { network: constants.NetworkName.SN_GOERLI2 },
  });

  const testAddress =
    "0x01c6b2a993b335cabc12dd5fa39875f456f07f23b10dcbdf6529667a322bbda4";
  const { abi: testAbi } = await provider.getClassAt(testAddress);

  if (testAbi === undefined) {
    throw new Error("No ABI.");
  }

  const myTestContract = new Contract(testAbi, testAddress, provider);

  console.log(
    "array2d_array meta-class =",
    await myTestContract.call("array2d_array", [
      [1n, 2n],
      [3n, 4n],
    ])
  );
}

main();

Expected behavior
The function call should not fail
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Browser & version [e.g. chrome, safari, webworker]: No brower, I'm running the script from terminal
Node version [e.g. 16.0.1]: v18.9.0
Starknet.js version: I'm on branch 634-contract-class-types, commit 1ec347a6ba02dbf6194f7208e09791cc49c2e77b
Network [devnet, testnet]: testnet2

Additional context
Add any other context about the problem here.

This command :
 await myTestContract.call("array2d_array", [
      [1n, 2n],
      [3n, 4n],
    ])
    ```
    Is not an array of array.
It's 2 parameters ; each one of type array.
Have a look here : https://www.starknetjs.com/docs/guides/interact#light-and-fast-call

call's second parameter is ArgsOrCalldata, which is an Array of raw Arguments or compiled Calldata.
So the usage would be:
await myTestContract.call("method", [argument]);

In this case
await myTestContract.call("array2d_array", [
  [[1n, 2n],[3n, 4n]], // one argument that is 2d array
]);


Starting from scratch and running into 411 error on 'refreshToken' step

Hello All,
I'm setting up this Homebridge plugin for the first time and have hit a snag at the 'refreshToken' step. I've been trying what I believe is the correct command, but I keep encountering the following error:
<title>Error 411 (Length Required)!!1</title>
I'm happy to provide more details about my setup if needed but didn’t want to overload the post with too much information upfront.
Any guidance or suggestions would be greatly appreciated!
Thank you!

Using a different format I was able to get around the above error but now seeing the below:
{   "error": "invalid_grant",   "error_description": "Bad Request" }

curl -X POST "https://www.googleapis.com/oauth2/v4/token" \ -H "Content-Type: application/x-www-form-urlencoded" \ -d "client_id=<VALUE>" \ -d "client_secret=<VALUE>" \ -d "code=<VALUE>" \ -d "grant_type=authorization_code" \ -d "redirect_uri=https://www.google.com"


should k8s.WaitForReadyOrDone pass for an resource with no conditions ?

The current intended behaviour for an object that isn't ready or succeeded is to cycle through the list of conditions to determine if k8s.WaitForReadyOrDone should pass. If the object has no conditions, the pass succeeds.
I ran into a case where I expected things to fail and was surprised when everything passed :)
Steps

Ran setup for discovery e2e tests, however the discovery controller was borked.

controller
2021/03/16 02:30:24 Registering 3 clients
controller
2021/03/16 02:30:24 Registering 3 informer factories
controller
2021/03/16 02:30:24 Registering 2 informers
controller
2021/03/16 02:30:24 Registering 1 controllers
controller
{"severity":"INFO","timestamp":"2021-03-16T02:30:24.7761488Z","caller":"logging/config.go:116","message":"Successfully created the logger."}
controller
{"severity":"INFO","timestamp":"2021-03-16T02:30:24.7762763Z","caller":"logging/config.go:117","message":"Logging level set to: info"}
controller
{"severity":"INFO","timestamp":"2021-03-16T02:30:24.7764096Z","logger":"controller","caller":"profiling/server.go:64","message":"Profiling enabled: false","commit":"b713233"}
controller
{"severity":"EMERGENCY","timestamp":"2021-03-16T02:30:24.7789131Z","logger":"controller","caller":"sharedmain/main.go:287","message":"Version check failed","commit":"b713233","error":"kubernetes version \"1.16.9\" is not compatible, need at least \"1.17.0-\" (this can be overridden with the env var \"KUBERNETES_MIN_VERSION\")","stacktrace":"knative.dev/pkg/injection/sharedmain.CheckK8sClientMinimumVersionOrDie\n\tknative.dev/pkg@v0.0.0-20210311075326-c326b70b833d/injection/sharedmain/main.go:287\nknative.dev/pkg/injection/sharedmain.MainWithConfig\n\tknative.dev/pkg@v0.0.0-20210311075326-c326b70b833d/injection/sharedmain/main.go:184\nknative.dev/pkg/injection/sharedmain.MainWithContext\n\tknative.dev/pkg@v0.0.0-20210311075326-c326b70b833d/injection/sharedmain/main.go:142\nknative.dev/pkg/injection/sharedmain.Main\n\tknative.dev/pkg@v0.0.0-20210311075326-c326b70b833d/injection/sharedmain/main.go:116\nmain.main\n\tknative.dev/discovery/cmd/controller/main.go:28\nruntime.main\n\truntime/proc.go:225"}

Ran the e2e tests, expecting a failure

$ ./test/kind/run-tests.sh
Starting End-to-End tests for Discovery API
2021/03/15 21:47:53 PWD:  /Users/svohra/workspace/discovery/test/e2e
2021/03/15 21:47:53 FILENAME:  /Users/svohra/workspace/discovery/test/e2e/config/smoke/smoke.go
{"level":"info","ts":1615862873.912114,"logger":"fallback","caller":"injection/injection.go:61","msg":"Starting informers..."}
level All, state Any

=== RUN   TestSmoke
=== PAUSE TestSmoke
=== RUN   TestKoPublish

// The following could be used to bypass the image generation process.

import "knative.dev/reconciler-test/pkg/environment"

func init() {
	environment.WithImages(map[string]string{

	})
}

--- PASS: TestKoPublish (0.00s)
=== CONT  TestSmoke
=== RUN   TestSmoke/Setup/install_a_simple_ClusterDuckType
2021/03/15 21:47:54 PWD:  /Users/svohra/workspace/discovery/test/e2e
2021/03/15 21:47:54 FILENAME:  /Users/svohra/workspace/discovery/test/e2e/config/smoke/smoke.go
2021/03/15 21:47:54 new files in /var/folders/mf/9ld4g_fn6kz01xylk1p65vgm0000gp/T/processed_yaml263288979
I0315 21:47:54.976746   69820 manifest.go:62] Reading YAML filepath: /var/folders/mf/9ld4g_fn6kz01xylk1p65vgm0000gp/T/processed_yaml263288979 recursive: false
2021/03/15 21:47:54 Created:
2021/03/15 21:47:54 {ClusterDuckType  minimums.smoke.knative.dev  discovery.knative.dev/v1alpha1  }
=== RUN   TestSmoke/Assert/[Alpha/MUST]for_a_single_ClusterDuckType_goes_ready
--- PASS: TestSmoke (1.09s)
    --- PASS: TestSmoke/Setup/install_a_simple_ClusterDuckType (0.01s)
    --- PASS: TestSmoke/Assert/[Alpha/MUST]for_a_single_ClusterDuckType_goes_ready (0.01s)
PASS
ok  	knative.dev/discovery/test/e2e	1.771s

CDT

$ k get cducks
NAME                         SHORT NAME   DUCKS   READY   REASON
minimums.smoke.knative.dev   Minimum

$ k get cducks/minimums.smoke.knative.dev -oyaml
apiVersion: discovery.knative.dev/v1alpha1
kind: ClusterDuckType
metadata:
  creationTimestamp: "2021-03-16T01:52:50Z"
  generation: 1
  name: minimums.smoke.knative.dev
  resourceVersion: "9432"
  selfLink: /apis/discovery.knative.dev/v1alpha1/clusterducktypes/minimums.smoke.knative.dev
  uid: 6261ccfe-22cd-4c06-bb20-29f01524f380
spec:
  group: smoke.knative.dev
  names:
    name: Minimum
    plural: minimums
    singular: minimum
  selectors:
    - labelSelector: smoke.knative.dev/minimum=true
  versions:
    - name: v1
      refs:
        - group: apiextensions.k8s.io
          kind: CustomResourceDefinition
          version: v1

Should the k8s.WaitForReadyOrDone fail if there are no conditions on the object like in this case ?

Yes. We just hit this too. Fix inbound.


Add separate font settings for the terminal

After the implementation of #240, proportional fonts seem to be working well for the main edit area.
However, since the same font setting is used for the terminal that still looks wrong. We need to have the ability to specify a separate font for the terminal area, probably with fallback to the main editor font if none is specified (most people won't bother with a separate font because they are not weird like me, wanting to program in proportional fonts :-)

Claimed.

This is implemented in this branch https://github.com/lapce/lapce/tree/new_theme

This is fixed already


win10 32位系统 基于64位处理器使用-win的包时报错

Expected Behavior
正常安装
Actual Behavior

Steps to Reproduce
在32位win10系统，基于64位处理器的环境安装自动选择32位和64位的安装包
Operating System and Browser
win10专业版32位，基于64位处理器

I will check for it.

Please take the trouble.

This issue seems to have been fixed, at least I can't reproduce it at the moment.


FATAL ERROR: Couldn't open "sun50i-h616-biqu.dtb": No such file or directory

OS image source:
https://mangopi.org/mqquad   under the "FW&SDK" section
Debian IMG with XFCE from the MEGA link: https://mega.nz/file/7L5w3CyD#ClAPLD9zHpwiIeM9JarotuU44G66ELNpPulKMwVnyiM
Issue:
According to the other issue reported here by VGCH, I2C works using your guide. So that only means that I am doing something wrong.
After running the command sudo apt-get install sunxi-tools device-tree-compiler rpi.gpio-common
and going to the directory cd /boot/dtb/allwinner/ , I found that I did not have the sun50i-h616-biqu.dts and sun50i-h616-biqu.dtb files that are needed to run the next command dtc -I dts -O dtb -f sun50i-h616-biqu.dts -o sun50i-h616-biqu.dtb  .
I saw that you provided the sun50i-h616-biqu.dts file, so I created a new file with the same name under the  cd /boot/dtb/allwinner/ directory and copy/pasted the file contents over to it. Although I still needed the sun50i-h616-biqu.dtb file to run dtc -I dts -O dtb -f sun50i-h616-biqu.dts -o sun50i-h616-biqu.dtb .
Without the appropriate files, I only get the following error: "FATAL ERROR: Couldn't open "sun50i-h616-biqu.dtb": No such file or directory"
Can you explain what I might be doing wrong or what I am missing?

You are using the official system image, this image uses a configuration file with the following name sun50i-h616-orangepi-zero2.dtb.
Take the following steps to solve your problem:
1 sudo git clone https://github.com/mamin27/H616-mangopi.git
2 cd H616-mangopi
3 sudo dtc -I dts -O dtb -f sun50i-h616-biqu.dts -o sun50i-h616-biqu.dtb
If necessary, back up your original file sun50i-h616-orangepi-zero2.dtb
4 sudo cp -i sun50i-h616-biqu.dtb /boot/dtb/allwinner/sun50i-h616-orangepi-zero2.dtb
5  sudo shutdown -r now


[Snyk] Upgrade angular-clipboard from 1.6.2 to 1.7.0

Snyk has created this PR to upgrade angular-clipboard from 1.6.2 to 1.7.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.

The recommended version was released on 6 years ago.

Release notes

Package name: angular-clipboard

1.7.0 - 2019-04-08v1.7.0

1.6.2 - 2017-10-011.6.2

    from angular-clipboard GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


failed to start pipeline if "input" was missing from pipeline descriptor

fixed with bug #199

start pipeline without input for algorithm

running a stored pipeline that didnt had input
returns an error: Cannot read property '0' of undefined.
{
"jobId":"addmult:8ab65ce1-3c50-4e7e-9e8e-7fca4b9f3669.addmult",
"timestamp":"2018-08-08T07:12:27.015Z",
"pipeline":"addmult",
"status":"failed",
"error":"Error: Cannot read property '0' of null",
"timeTook":1.756
}


RC Spot Check: States of Matter 1.1.0-rc.2

@KatieWoe and @ariel-phet, States of Matter 1.1.0-rc.2 is ready for RC testing. Testing on 1.1.0-rc.1 was completed, several issues have been addressed, and this is the follow up test.  For the most part, a full test on all platforms is not necessary, just a verification of the issues and spot testing on key platforms.  See the list of issues below for guidance on what should be tested.
Please document any issues found in https://github.com/phetsims/states-of-matter/issues and link to this issue.
Assigning @ariel-phet for prioritization.

General RC Test

What to Test
Versus the last RC, the only changes are related to the issues listed below, so that can be the focus of the testing.  Having said that, this is a very popular sim, so enough testing should be done to make sure that this particular release is stable enough to replace the currently published version.

Issues to Verify

[ ] Neon and Argon on Interaction short when stringTest=long
[ ] Why did the HearterCoolerNode get smaller?
[ ] Thermometer resets tilted on states screen

These issues should have the "status:fixed-pending-testing" label. Check these issues off and change their status to fixed-awaiting-deploy if resolved. Otherwise, post a comment in the issue saying that it wasn't fixed.

Link(s)

Simulation
iFrame
XHTML
Legends of Learning Harness

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

List looks good to me.

Current top testing priority

QA is done.

Assigning @ariel-phet since @jbphet is gone right now.

Thanks for the testing, I'll follow up on the issues.  Closing.


mobile: hide keyboard on scroll in chat scroller

Fixes LAND-921 by creating the ref in the parent component for chat channels, DMs, multi DMs, and then passing the ref into the scroller and input in each case and tracking whether or not we're scrolling to determine when we should blur the input.
https://github.com/tloncorp/landscape-apps/assets/1221094/57441dba-5ded-4fa9-8293-f5dd0cdf2abb

code looks good, but is this working for you guys on device? Doesn't seem to be for me

code looks good, but is this working for you guys on device? Doesn't seem to be for me

I tested in the simulator. I can test on my Android device in the morning, though.

Also not seeing this on-device:
https://github.com/tloncorp/landscape-apps/assets/748181/c651d261-bc7a-427a-aed8-db760ac00e3e

@latter-bolden @jamesacklin can you guys try now?

This works for me now! Just some conflicts and we're g2g.


Resolve reconnect issue

Fixed reconnect issue and updated safe_app to pre_42

Coverage decreased (-0.4%) to 83.725% when pulling 07f6f7bb125d1a852b419eea67e537fb3c711dc9 on shankar2105:reconnect_fix into 3689a26ce9c11a6c8f9593dca3a20351c409004f on maidsafe:master.

Coverage decreased (-0.9%) to 83.235% when pulling 55a96fd7044d011139d41c66878e09068c25cc67 on shankar2105:reconnect_fix into 3689a26ce9c11a6c8f9593dca3a20351c409004f on maidsafe:master.

Coverage decreased (-0.9%) to 83.235% when pulling 55a96fd7044d011139d41c66878e09068c25cc67 on shankar2105:reconnect_fix into 3689a26ce9c11a6c8f9593dca3a20351c409004f on maidsafe:master.


Add ability to create Linux perf event and metric json

Add a script to create the Linux perf event and metric json. When the event and metric data is updated running this script will create json files for the Linux perf tool that can be mailed to the Linux kernel mailing list. A single script is used so, for example, metrics can verify events in the formula against those in the json files.

I used the new script to generate the core/uncore/metric JSON files, I  diff the files, and they are almost the same except for the following :
1)Removed "CollectPEBSRecord"/ "Counter"/ "PEBScounters"/ "Speculative"
These items are not used by both old jevents.c and new jevents.py, so currently removing them should be ok, but we may use these items in the future.
2)Sort the events.
3) the following uncore events for BDX/IVT/BDW-DE/HSX/ are missed by new scripts
"EventName": "UNC_H_ADDR_OPC_MATCH.FILT",
"EventName": "UNC_H_ADDR_OPC_MATCH.ADDR",
"EventName": "UNC_H_ADDR_OPC_MATCH.OPC",
"EventName": "UNC_H_ADDR_OPC_MATCH.AD",
"EventName": "UNC_H_ADDR_OPC_MATCH.BL",
"EventName": "UNC_H_ADDR_OPC_MATCH.AK",
4)   the following uncore event for BDX/IVT/BDE-DE/JKT/HSX is missed by new scripts
"EventName": "UNC_H_ADDR_OPC_MATCH.FILT",
5)  the following uncore event for ICX/SNR/BDX/IVT/BDW-DE/JKT/HSX is missed by new scripts
"EventName": "UNC_I_SNOOP_RESP.HIT_ES"
6) Because the events are sorted, it is hard to diff, I manually do it, so may miss something else.

Thanks for looking! On reviewing the points above I don't find any issues to address in the code. A more detailed explanation is below.

the following uncore events for BDX/IVT/BDW-DE/HSX/ are missed by new scripts
"EventName": "UNC_H_ADDR_OPC_MATCH.FILT",
"EventName": "UNC_H_ADDR_OPC_MATCH.ADDR",
"EventName": "UNC_H_ADDR_OPC_MATCH.OPC",
"EventName": "UNC_H_ADDR_OPC_MATCH.AD",
"EventName": "UNC_H_ADDR_OPC_MATCH.BL",
"EventName": "UNC_H_ADDR_OPC_MATCH.AK",

If you look at the events they have filters:
{
"Unit": "HA",
"EventCode": "0x20",
"UMask": "0x3",
"EventName": "UNC_H_ADDR_OPC_MATCH.FILT",
"BriefDescription": "QPI Address/Opcode Match; Address & Opcode Match",
"PublicDescription": "UNC_H_ADDR_OPC_MATCH.FILT",
"Counter": "0,1,2,3",
"MSRValue": "0x00",
"ELLC": "0",
"Filter": "HA_AddrMatch0[31:6], HA_AddrMatch1[13:0], HA_OpcodeMatch[5:0]",
"ExtSel": "0"
},
It is expected even in the old code that these are events are dropped:
https://github.com/intel/event-converter-for-linux-perf/blob/master/uncore_csv_json.py#L244
This behavior was discussed with Kan Liang on LKM.

the following uncore event for BDX/IVT/BDE-DE/JKT/HSX is missed by new scripts
"EventName": "UNC_H_ADDR_OPC_MATCH.FILT",

This is the same as above.

the following uncore event for ICX/SNR/BDX/IVT/BDW-DE/JKT/HSX is missed by new scripts
"EventName": "UNC_I_SNOOP_RESP.HIT_ES"

I don't see this. For example, in perf/icelakex/uncore-other.json:
{
    "BriefDescription": "Responses to snoops of any type that hit E or S line in the IIO cache",
    "EventCode": "0x12",
    "EventName": "UNC_I_SNOOP_RESP.ALL_HIT_ES",
    "PerPkg": "1",
    "PublicDescription": "Responses to snoops of any type (code, data, invalidate) that hit E or S line in the IIO cache",
    "UMask": "0x74",
    "Unit": "IRP"
},

Because the events are sorted, it is hard to diff, I manually do it, so may miss something else.

Agreed. You can add a sort here:
https://github.com/intel/event-converter-for-linux-perf/blob/master/uncore_csv_json.py#L280
like:
events : list[Dict[str, str]] = sorted(list(iter), key=lambda event: event['EventName'])
to reduce this.

+Kan
On 11/17/2022 1:14 AM, Ian Rogers wrote:

Thanks for looking! On reviewing the points above I don't find any
issues to address in the code. A more detailed explanation is below.
 3. the following uncore events for BDX/IVT/BDW-DE/HSX/ are missed
    by new scripts
    "EventName": "UNC_H_ADDR_OPC_MATCH.FILT",
    "EventName": "UNC_H_ADDR_OPC_MATCH.ADDR",
    "EventName": "UNC_H_ADDR_OPC_MATCH.OPC",
    "EventName": "UNC_H_ADDR_OPC_MATCH.AD",
    "EventName": "UNC_H_ADDR_OPC_MATCH.BL",
    "EventName": "UNC_H_ADDR_OPC_MATCH.AK",

If you look at the events they have filters:
{
"Unit": "HA",
"EventCode": "0x20",
"UMask": "0x3",
"EventName": "UNC_H_ADDR_OPC_MATCH.FILT",
"BriefDescription": "QPI Address/Opcode Match; Address & Opcode Match",
"PublicDescription": "UNC_H_ADDR_OPC_MATCH.FILT",
"Counter": "0,1,2,3",
"MSRValue": "0x00",
"ELLC": "0",
"Filter": "HA_AddrMatch0[31:6], HA_AddrMatch1[13:0], HA_OpcodeMatch[5:0]",
"ExtSel": "0"
},
It is expected even in the old code that these are events are dropped:
https://github.com/intel/event-converter-for-linux-perf/blob/master/uncore_csv_json.py#L244
This behavior was discussed with Kan Liang on LKM.

Add Kan to confirm if IVT/BDW-DE/HSX also needs to remove these events.
In the following LKM thread, Kan only checks the BDX  for the events
with three Filters.
  drop_event_filter_start = [
                "ha_addrmatch",
                "ha_opcodematch",
                "irpfilter",
            ]
@.***/

 4. the following uncore event for BDX/IVT/BDE-DE/JKT/HSX is
    missed by new scripts
    "EventName": "UNC_H_ADDR_OPC_MATCH.FILT",

This is the same as above.
 5. the following uncore event for ICX/SNR/BDX/IVT/BDW-DE/JKT/HSX
    is missed by new scripts
    "EventName": "UNC_I_SNOOP_RESP.HIT_ES"

I don't see this. For example, in perf/icelakex/uncore-other.json:

I double check, it has no problem.

|{ "BriefDescription": "Responses to snoops of any type that hit E or
S line in the IIO cache", "EventCode": "0x12", "EventName":
"UNC_I_SNOOP_RESP.ALL_HIT_ES", "PerPkg": "1", "PublicDescription":
"Responses to snoops of any type (code, data, invalidate) that hit E
or S line in the IIO cache", "UMask": "0x74", "Unit": "IRP" }, |
 6. Because the events are sorted, it is hard to diff, I manually
    do it, so may miss something else.

Agreed. You can add a sort here:
https://github.com/intel/event-converter-for-linux-perf/blob/master/uncore_csv_json.py#L280
like:
events : list[Dict[str, str]] = sorted(list(iter), key=lambda event:
event['EventName'])
to reduce this.

Thanks, I will do that.

—
Reply to this email directly, view it on GitHub
https://github.com/intel/perfmon/pull/27#issuecomment-1317372042, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AVHPWBGMSK7W3XNMVXQA3TDWIUJATANCNFSM6AAAAAARZPVG2M.
You are receiving this because you commented.Message ID:
@.***>

--
Zhengjun Xing

On 2022-11-17 3:30 a.m., Xing, Zhengjun wrote:

+Kan
On 11/17/2022 1:14 AM, Ian Rogers wrote:

Thanks for looking! On reviewing the points above I don't find any
issues to address in the code. A more detailed explanation is below.
3. the following uncore events for BDX/IVT/BDW-DE/HSX/ are missed
        by new scripts
        "EventName": "UNC_H_ADDR_OPC_MATCH.FILT",
        "EventName": "UNC_H_ADDR_OPC_MATCH.ADDR",
        "EventName": "UNC_H_ADDR_OPC_MATCH.OPC",
        "EventName": "UNC_H_ADDR_OPC_MATCH.AD",
        "EventName": "UNC_H_ADDR_OPC_MATCH.BL",
        "EventName": "UNC_H_ADDR_OPC_MATCH.AK",
If you look at the events they have filters:
{
"Unit": "HA",
"EventCode": "0x20",
"UMask": "0x3",
"EventName": "UNC_H_ADDR_OPC_MATCH.FILT",
"BriefDescription": "QPI Address/Opcode Match; Address & Opcode Match",
"PublicDescription": "UNC_H_ADDR_OPC_MATCH.FILT",
"Counter": "0,1,2,3",
"MSRValue": "0x00",
"ELLC": "0",
"Filter": "HA_AddrMatch0[31:6], HA_AddrMatch1[13:0],
HA_OpcodeMatch[5:0]",
"ExtSel": "0"
},
It is expected even in the old code that these are events are dropped:
https://github.com/intel/event-converter-for-linux-perf/blob/master/uncore_csv_json.py#L244
This behavior was discussed with Kan Liang on LKM.

Add Kan to confirm if IVT/BDW-DE/HSX also needs to remove these events.

We never the filters on any platforms. I think we can remove them.
Thanks,
Kan

In the following LKM thread, Kan only checks the BDX  for the events
with three Filters.
  drop_event_filter_start = [
                "ha_addrmatch",
                "ha_opcodematch",
                "irpfilter",
            ]
@.***/

4. the following uncore event for BDX/IVT/BDE-DE/JKT/HSX is
        missed by new scripts
        "EventName": "UNC_H_ADDR_OPC_MATCH.FILT",
This is the same as above.
5. the following uncore event for ICX/SNR/BDX/IVT/BDW-DE/JKT/HSX
        is missed by new scripts
        "EventName": "UNC_I_SNOOP_RESP.HIT_ES"
I don't see this. For example, in perf/icelakex/uncore-other.json:

I double check, it has no problem.

|{ "BriefDescription": "Responses to snoops of any type that hit E or
S line in the IIO cache", "EventCode": "0x12", "EventName":
"UNC_I_SNOOP_RESP.ALL_HIT_ES", "PerPkg": "1", "PublicDescription":
"Responses to snoops of any type (code, data, invalidate) that hit E
or S line in the IIO cache", "UMask": "0x74", "Unit": "IRP" }, |
6. Because the events are sorted, it is hard to diff, I manually
        do it, so may miss something else.
Agreed. You can add a sort here:
https://github.com/intel/event-converter-for-linux-perf/blob/master/uncore_csv_json.py#L280
like:
events : list[Dict[str, str]] = sorted(list(iter), key=lambda event:
event['EventName'])
to reduce this.

Thanks, I will do that.

—
Reply to this email directly, view it on GitHub
https://github.com/intel/perfmon/pull/27#issuecomment-1317372042, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AVHPWBGMSK7W3XNMVXQA3TDWIUJATANCNFSM6AAAAAARZPVG2M.
You are receiving this because you commented.Message ID:
@.***>

I checked all the platforms, and no other core/uncore events were missed when converted by the new scripts.
But for metrics, I find the following issues, the uncore events should not be "cpu@" (mostly in "MEM_Parallel_Reads")
SPR:
old : "MetricExpr": "UNC_CHA_TOR_OCCUPANCY.IA_MISS_DRD / cha@UNC_CHA_TOR_OCCUPANCY.IA_MISS_DRD\,thresh\=1@",
new: "MetricExpr": "UNC_CHA_TOR_OCCUPANCY.IA_MISS_DRD / cpu@UNC_CHA_TOR_OCCUPANCY.IA_MISS_DRD\,cmask\=1@",
SKL:
old:  "MetricExpr": "arb@event\=0x80\,umask\=0x2@ / arb@event\=0x80\,umask\=0x2\,cmask\=1@",
new:  "MetricExpr": "UNC_ARB_TRK_OCCUPANCY.DATA_READ / cpu@UNC_ARB_TRK_OCCUPANCY.DATA_READ\,cmask\=1@",
SKX:
old:  "MetricExpr": "cha@event\=0x36\,umask\=0x21\,config\=0x40433@ / cha@event\=0x36\,umask\=0x21\,config\=0x40433\,thresh\=1@",
new: "MetricExpr": "UNC_CHA_TOR_OCCUPANCY.IA_MISS_DRD / cpu@UNC_CHA_TOR_OCCUPANCY.IA_MISS_DRD\,cmask\=1@",
HSX:
old :"MetricExpr": "cbox@event\=0x36\,umask\=0x3\,filter_opc\=0x182@ / cbox@event\=0x36\,umask\=0x3\,filter_opc\=0x182\,thresh\=1@",
new: "MetricExpr": "UNC_C_TOR_OCCUPANCY.MISS_OPCODE@filter_opc\=0x182@ / UNC_C_TOR_OCCUPANCY.MISS_OPCODE@cmask\=1\,filter_opc\=0x182@",
CLX:
old:    "MetricExpr": "cha@event\=0x36\,umask\=0x21\,config\=0x40433@ / cha@event\=0x36\,umask\=0x21\,config\=0x40433\,thresh\=1@",
new:        "MetricExpr": "UNC_CHA_TOR_OCCUPANCY.IA_MISS_DRD / cpu@UNC_CHA_TOR_OCCUPANCY.IA_MISS_DRD\,cmask\=1@",

So this is wrong and something that can be fixed. The old logic is here:
https://github.com/intel/event-converter-for-linux-perf/blob/master/extract-tma-metrics.py#L91
You will see that the replacement is tagged as being only for SKL, however, you have seen that it is actually applying in the old code on SPR, SKL, SKX, HSX and CLX. The replacement is happening so that the<EMAIL_ADDRESS>expansion of the c1 doesn't happen for an uncore event. We can do the same fix in the new code here:
https://github.com/captain5050/perfmon-metrics/blob/main/scripts/create_perf_json.py#L680
However, the new code has to be explicit with a rewrite for every architecture it applies to (the 5 listed above). I'll patch the original CLs, but it is also possible to fix issues like this once the code is merged, which would unblock the EBS_Mode work that's needed on the Valkyrie metrics.

I added the rewrites such as for SKL here:
https://github.com/captain5050/perfmon-metrics/blob/main/scripts/create_perf_json.py#L741
and verified the CPU expansion wasn't applying to other uncore events. Note, I did use the raw event encoding like the legacy code as the named event is clearer and less error prone. PTAL and let me know if there are any other outstanding issues.

I tried the updated scripts, CLX/ICX/SPR/SKL/SKX "cpu@" issue disappeared.
But the following HSX issue still existed ("thresh" change to "cmask"  for "MEM_Parallel_Reads")
HSX:
old :"MetricExpr": "cbox@event=0x36,umask=0x3,filter_opc=0x182@ / cbox@event=0x36,umask=0x3,filter_opc=0x182,thresh=1@",
new: "MetricExpr": "UNC_C_TOR_OCCUPANCY.MISS_OPCODE@filter_opc=0x182@ / UNC_C_TOR_OCCUPANCY.MISS_OPCODE@cmask=1,filter_opc=0x182@",


Suggestion: Tab view - vertical tabs

Hi,
is it possible change the tab view for vertical tabs (on the left/right side)?
Thank you.

Hi,
not supported directly now. However, besides TOP, the BOTTOM position is also supported. Similarly it LEFT and RIGHT can be added.
Would like to implement it and send a PR?

Hi, sorry for the late response, I hadn't lot of free time. I tried change your API for tabs and now, I can use it in right and left. There is lot of changes and I don't know, if it is ok for Pull Request now. It is changed mainly for my usage. If you want see it, it is on my fork on branch ZAVAVOV this commit: https://github.com/zavovi/lvgl/commit/effc5f8700dd68c0754e266d459109659fbcb76c

The default style couldn't replaced by theme.

Anyway please send a PR. Probably the default transparent style needs to be reverted but it's nota  big deal.

Hi a sent the PR. Here is some example of my usage of yours lib: https://www.facebook.com/zavavov/videos/829343017414636/

Thank you for the PR!
Nice project! Is it a commercial or a hobby product? Anyway, it'd be great to add it to LittlevGL references.

It is commercial, but only small pieces. I am making it at home. It is for my command station: http://www.zavavov.cz/en/tcs-en/ and you could add it into your references, after it will be done and I create web page for it. Thank you for your support :-)
About changes in PR, I will make it, when I will have more time for test it.

It is commercial, but only small pieces. I am making it at home. It is for my command station: http://www.zavavov.cz/en/tcs-en/ and you could add it into your references, after it will be done and I create web page for it

Awesome! I think it would be better if you added the project and wrote a few lines. You can highlight the features better and add the most relevant image. It's not a problem if are not fully ready with it, you can update the references page later.

About changes in PR, I will make it, when I will have more time for test it..

Okay, thank you. Then I close this issue. Let's continue in the PR.


dependencies: bump dependencies to comply with Invenio 3.2

Changes:

invenio-search
invenio-db
invenio-records-files

Ready for merge if all good.

I will bump the version of invenio-records-permissions so that https://github.com/inveniosoftware/invenio-rdm-records/pull/41 can pass too.


Fix type stability tests in reverse mode

At the moment, the JET tests in reverse mode are breaking due to runtime dispatch. This is presumably happening because Zygote-computed pullbacks introduce type instability.
The solution may be to write conditions with explicit rrules.

Investigation is tricky because of https://github.com/JuliaDebug/Cthulhu.jl/issues/478

I think this is due to inherent type instabilities in AbstractDifferentiation.jl

Type stability tests are still failing sometimes, even though I switched to DifferentiationInterface in #135. It's hard to guarantee that the (dense or lazy) jacobian will be type-stable


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1403 ms

Resolved: Bofrost.at performance has improved in f17649c after 16 minutes.


Cannot find MsDeploy.exe while using WinRM - IIS web app deployement

When the WinRm - IIS web app deployment task been triggered, i got the issue as below.
##[error]Microsoft.PowerShell.Commands.WriteErrorException: Deployment on one or more machines failed. System.Exception: Cannot find MsDeploy.exe location. Verify MsDeploy.exe is installed on  and try operation again.
I just wondering do i need to install web deploy tool on target machine first?  Since i can't find any instruction on the document. (https://marketplace.visualstudio.com/items?itemName=ms-vscs-rm.iiswebapp)
If not, could you help me to figure out why the issue happen.

Install MSDeploy?

@huier23 : You are right. You will need to install web deploy tool on target machine.


ST ins't oppening in floating mode

I've set my i3 to call st in floating mode when I press mod+Enter.
But it isn't working since I installed xresources path to st and added some things to my .Xresources file.

For some reason after I applied the Xresources path to st the terminal name become capital(new name is "St".
A good tip is to use xprop to show the window info(including WM_CLASS).


Teeny tiny change in readme

For Laravel 5.x users  (PHP 5.5.x ), it will be easier to grab it without change :)

Thanks!

Thanks to you for correcting nicely the first PR :smile:


Data races in cli/cli.go

Issue: manifoldco/promptui/issues/210
Maybe use raw bufio instead

Old cli implementation d9b3517744906c40d2b5ad4c91ae46fd0cb715ef

no info in the linked issue
reverted integration - 1728a6212ac77de6865ed610f8ccbde6431d6110


[question] - -e Docker Variables available

Unraid uses a pretty cool UI (Community Applications - which are basically docker containers deployed via the UI).
It allows the specification of the port, the volumes etc, but also some environment variables.
Do you surface up -e Network_Key for example? It would be useful to specify that (and maybe any other useful variables) for a user to carry out a one click install.
Most valuable reason for this is the automatic updating of the docker containers.
I've kicked off a Community App request here

Hi @jaburges ! I could make the support for that but why supply network key from env var instead of provide it from the UI? For security purposes?

in UnRaid UI you can add these items so the Z2M starts with the right key

@jaburges I understand this but you can also provide it from the UI, this is why I'm asking the pro/cons by add it using env vars

I think the preferred way for Unraid users is to specify everything possible in the UI with -e / ENV variables.
If it's a lot of work, I would not bother. If it's easy it will open your solution up to more Unraid users.

Configuration are a lot, create an env var for each one would be a lot of work and I sincerly still don't understand the advantage of this. If you simply run z2m with Unraid then you go to localhost:8091 and configure it from the UI why would you need to specify things from env?
I would use env vars for the port of the server (there is already an env var for that) and other server specific configuration. Also adding the controller key would be easy but it can be configured from the ui so I'm not sure if it's useful

in terms of security the qt-openzwave docker has the option to add the network_key as an ENV.
i get your point, and you maybe able to just set up an Unraid template (for updating without the ENV variables)
i'll close this

Open an issue for that, it's a very easy feature to add I will make it once I come back from holidays :)

@jaburges just submitted a feat for this


Issue with "sail tinker" command and "edit" function in Laravel Sail environment

Sail Version
v1.23.0
Laravel Version
v10.11.0
PHP Version
8.2.7
Operating System
macOS
OS Version
13.4.1
Description
I am encountering an issue when using Laravel Sail, specifically with the sail tinker command. When attempting to use the edit command to input a large amount of text, I receive the error message sh: 1: nano: not found.
This issue has been consistently reproducible in both Ubuntu 22.04 and MacOS environments.
I have tried installing nano, modifying the psysh/config.php file, and using update-alternatives --config editor to try to resolve the problem, but to no avail.
Related Issue:
Additionally, after running sail shell and then executing php artisan tinker, I often encounter an stty: 'standard input': Input/output error message when attempting to terminate tinker with Ctrl+C. I'm not sure if this is related to the above problem.
I would appreciate any advice or insight into these issues. Thank you.
Steps To Reproduce
sail tinker then run edit

Thank you for your issue. However, I don't think this is specific to Sail. This doesn't seems like a core feature/usage of Sail so please try a support channel instead:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC


[Iris] Physics and terrain loading issues

Is it a mod compatibility issue?
Yes
Minecraft version
1.20
Describe the issue
Iris compatibility issue. Just having Iris installed is enough, enabling it is not needed. Tested with Quilt, may test with Fabric soon as well.
When the world is saved while looking at a portal, terrain isn't loaded on the next world load - only skybox is visible and the only possible action is looking around.
It also has a tendency to drop frames and stop all entity physics after some time interacting in proximity of the portal - mobs don't move and player can only look around.

Oddly this seems to be resolved when switching driver from mesa to zink. Might actually be an issue related to either driver or device - it was tested on a Steam Deck loaded from SD card, I don't recall if it ever happened on my laptop as well, I can't reproduce that on it now and it might be because of its much faster loading times on SSD. Closing for now

Reopening because it started happening again and after updating to MC 1.20.1 and ImmPtl 3.1.0 I noticed Entity Culling incompatibility note. Since deleting EC the issue doesn't seem to happen anymore. So just wanted to ask if it's what the EC incompatibility could be about? I've had a lot of hit and miss guesses and false positives (like Iris and MemoryLeakFix), so just wanted to make sure. Thank you!

Same quastion.


feat(cost-analysis): add show-favorite-only in cost-analysis LNB

To Reviewers

[ ] Skip (style, chore, ci ONLY)
[ ] Not that difficult

Type of Change

[x] New feature
[ ] Bug fixes
[ ] Feature improvement
[ ] Refactor
[ ] Others (performance improvement, CI/CD, etc.)

Affects to

[ ] Packages

[ ] core-lib
[ ] mirinae
[ ] etc

[x] Apps

[ ] storybook
[x] web

Checklist

Did you check the lint and type?
Did you document the changes?

Changes in mirinae should be reflected in storybook.

Did you test the app after the package changes?

Description
SSIA

Thanks! I apply your reviews and refactor them!


Update output-data-widgets.md

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues

fixed typo

Thanks!


Revisit XMVectorPow

Both the ARM-NEON and SSE implementation of XMVectorPow just invoke the standard scalar powf method. It should be optimized at some point.

Should be fixable with Intel's SVML library using _mm_pow_ps.


Module works in development, but not in production (expo)

Hi there,
we've added this module to our project and it works well when we try it in our test clients. It does unfortunately not log any errors in production. After further debugging it appears that it doesn't even try to talk to the Amplitude API when logging events.
Anyone else seeing a similar issue?
Cheers,
Sascha

Hi @justin-fiedler  (Sascha and I work together).

We are using v2.10.0
Both
We have just started using the Amplitude SDK for mobile

We aren't actually able to identify any "error" as such, even when tailing logs in iOS and Android. But we know that events should be logged and zero are logged from any mobile devices.
We are using Expo with EAS builds.
We can see events from mobile devices if we create a developmentClient build and run it on a simulator for example...

I have things set up so I can see Android logs of my app, but I'm not finding anything right now on why it wouldn't send an event. Is there a way to turn on debug logging so that all interactions with Amplitude are logged to give us some confidence in what is/isn't working?

Hello again. We've identified our issue. It was our fault, something got mixed up with the release channels and our updates were not flowing through to prod (doh!). Apologies for any time wasted.
It may still be a good idea to expose the logging level to the RN client though.


Fix local variable depth in rescue clauses

Local variables assigned in rescue clauses are accessible outside the rescue's scope:
begin
  require "foo"
rescue LoadError
  my_var = "uh oh!"
end

my_var  # => "uh oh!"

For begin and else clauses, MRI does not push a stack frame and so locals can be assigned at depth 0. rescue clauses in contrast push a stack frame, so locals should be assigned at depth 1.
This PR pushes a transparent scope while parsing rescue clauses in order to increase the depth by 1.

In general the approach is sound, I see that you're correctly pushing the transparent scope.
I'm curious why you need to change the handling of the depth though. Shouldn't that just work as in once the transparent scope is in place?

@kddnewton you're absolutely right, the depth is already correctly calculated. I got rid of the other code 😄

rescue clauses in contrast [on MRI] push a stack frame, so locals should be assigned at depth 1.

This is actually only true on CRuby, and not on JRuby and TruffleRuby (also not on CRuby 1.8 FWIW):
~ ∴ ruby -ve 'begin; raise; rescue; puts caller_locations(0); end'
ruby 3.2.2 (2023-03-30 revision e51014f9c0) [x86_64-linux]
-e:1:in `rescue in <main>'
-e:1:in `<main>'

~ ∴ ruby -ve 'begin; raise; rescue; puts caller_locations(0); end'
truffleruby 24.0.0-dev-bb05e0bb, like ruby 3.2.2, GraalVM CE Native [x86_64-linux]
-e:1:in `<main>'

~ ∴ ruby -ve 'begin; raise; rescue; puts caller_locations(0); end'
jruby <IP_ADDRESS> (3.1.4) 2023-10-18 9c9bfd3dc7 OpenJDK 64-Bit Server VM 17.0.8+7 on 17.0.8+7 +jit [x86_64-linux]
-e:1:in `<main>'

I would really like to be able to trust the depth from Prism on TruffleRuby.
So I think either we need an option for this (but it probably doesn't make sense to set this except for using Prism as a Ruby engine parser) or a template-time env var to configure it (like PRISM_SERIALIZE_ONLY_SEMANTICS_FIELDS).
How about PRISM_PUSH_SCOPE_FOR_RESCUE (defaulting to true == CRuby behavior)?
In terms of implementation, because prism.c is not generated, you can use the trick to define in the header like https://github.com/ruby/prism/blob/e84c4662ed54f7be9528d8d9785836d6255b3ff5/templates/include/prism/ast.h.erb#L198

Wow.  You learn something everyday and I didn't notice this worked this way for 20+ years.  This will indeed break JRuby and TR short-term since it is not just a parser change.  I guess the motivation for this was to increase backtrace visibility?
I think we can come up with a way of subtracting the depth (which is what happens with for loop variables already) to maintain our current behavior.  This is somewhat icky since we have to recursively figure out how many rescues are between us and where the scope is really defined.   Since our system knows for is special we just count number of for scopes to substract.  We have no scopes for rescue (and I guess ensure?) so this will probably be some counter?  Not sure if other interwoven elements will make this more difficult.
As I see it there are two problems:

Fixing the behavior so all impls have the same backtrace.  I assume this backtrace behavior is motivation?  If not someone should give more info.  I have only just seen this but it feels like a non-trivial task
Having two depths means one serialization format will not be usable by all impls.  This is a longer term goal but being able to use the same blob means we can all share a universal pre-compiled source

I think @eregon workaround gives us time to figure out 1) but even if we decided not to impl 1) then we still need 2) in the future.  So I think any suggestion like this should only be temporary.

Okay, so local variable depth is a whole confusing thing so I just added a doc for it. It's here: https://github.com/ruby/prism/blob/main/docs/local_variable_depth.md.
I think we have a shocking number of differences here that we will need to account for. I don't particularly want to deviate from CRuby, but it looks like we already do in quite a few places. In this case, I think we should clearly define our own concept of "depth" (since it's clearly not shared between implementations) and then make sure our documentation is super clear. CRuby is going to use our depth variable, but it's going to have to adjust it according to its own needs. JRuby/TruffleRuby will have to do the same. Hopefully we can converge around the most common cases and document the edge cases.
With this in mind, I think we shouldn't increase the scope depth for rescues, simply because it's easier to add to the number of scopes than it is to subtract. We'll handle this explicitly in the CRuby compiler.

@kddnewton you also need to include BEGIN as well to that doc.

That document looks great.
I commented on some clearer wording regarding for in https://github.com/ruby/prism/commit/ed8f5682cadc74486f1f6f2c87b54fa967ad6659#r133270393.
I think Prism, as it seems to be already the case, should only create new scopes when they need to be there semantically (i.e a separate set of variables only accessible in that scope and nested scopes).
IOW avoid artificial extra (semantically unnecessary) scopes which only exist on some Ruby implementations and are implementation details.

Thanks for weighing in, everyone! Sounds like the compiler is the right place to make depth adjustments 👍 Thanks for that very nice doc too @kddnewton 😄


[Enhancement] Use log4j to control hudi logs

What type of PR is this：

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

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

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

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

Bugfix cherry-pick branch check:

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

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

run starrocks_admit_test

@Mergify rebase

@mergify rebase


Optimistic concurrency control by NES and NeventStore+SQL

Hi there,
I've noticed that when the stream is commited, NES performs a read first to detect concurrency:
using (IEventStream eventStream = this._eventStore.OpenStream(bucketId, id, version, int.MaxValue))
    {
        EventStoreAdapter.Logger.Debug("Opened stream has StreamRevision {0}", (object) eventStream.StreamRevision);
        if (version != eventStream.StreamRevision && Transaction.Current != (Transaction) null)
        {
            throw new ConflictingCommandException

Still there's a possibility for concurrency issue this way (result of OpenStream becomes stale immediately). I am using NEventStore with SQL persistence and the consistency of event stream is guaranteed by the database (PK on bucket, streamId and version). So the OpenStream looks like a redundant action in that case for me.
Please comment on that, thank you!

@Lees-Oz
Before we can do a commit, we will open then stream
using (var stream = _eventStore.OpenStream(bucketId, id, version, int.MaxValue))
The version here says, which the minimum version of the stream should be. So when you attend to open version 2, maybe the opened stream version will be 3, but you expected to be 2, because you made your business decisions based on the state of the version 2. Therefore is the check there. Do you agree?

Not completely, because between the moment you do the check and the moment you persist stream there's a possibility that version 3 will be added by concurrent thread. So that check cannot guarantee the event stream consistency in the storage. In case of SQL storage, SQL only can guarantee that (with PK mechanism). Then I get that exception:
NES.ConflictingCommandException: Cannot insert duplicate key row in object 'dbo.Commits' with unique index 'IX_Commits_CommitSequence'. The duplicate key value is (default, EE855B7AC9741EB9306D59FC5102297DEB0EC678, 3). ---> NEventStore.ConcurrencyException: Cannot insert duplicate key row in object 'dbo.Commits' with unique index 'IX_Commits_CommitSequence'. The duplicate key value is (default, EE855B7AC9741EB9306D59FC5102297DEB0EC678, 3). ---> NEventStore.Persistence.Sql.UniqueKeyViolationException: Cannot insert duplicate key row in object 'dbo.Commits' with unique index 'IX_Commits_CommitSequence'. The duplicate key value is (default, EE855B7AC9741EB9306D59FC5102297DEB0EC678, 3). ---> System.Data.SqlClient.SqlException: Cannot insert duplicate key row in object 'dbo.Commits' with unique index 'IX_Commits_CommitSequence'. The duplicate key value is (default, EE855B7AC9741EB9306D59FC5102297DEB0EC678, 3)

If this check in NES detects the concurrency conflict, I get same exception type, with a different message and stacktrace though:
NES.ConflictingCommandException: EventSource a1d896f6-11ff-470e-a815-a65500cf078a has the version 17 and the stream has version 19

Those 2 exceptions mean same and it's not clear for me why distinguish them, especially if that costs another OpenStream operation.

You are right. Let's check some examples:
Example 1:
we open stream 2
we have our aggregate object populated
we close stream 2
job is done, so lets write
we open stream 2
another thread writes stream 3
we write stream 3
exception by database because of duplicate key
This is totally fine, our additional check not needed.
Example 2:
we open stream 2
we have our aggregate object populated
we close stream 2
job is done, so lets write
another thread writes stream 3
we open stream 3
we write stream 4
Here we have no database exception, because we have no conflict.
In second example we made our business decision based on stream 2 and not stream 3, therefore the check. Do you agree now?

Probably I know where the confusion comes from. You have to open the stream(s) anyway when you Commit unit of work. Because if you use the one you've opened before "job is done", and write it with incremented version, db exception should do the trick. Am I correct?

Correct. If NES would keep the stream opened during the whole UnitOfWork, then we would get the db exception like expected by you previously.
Actually during the read NES opens and closes the stream. Then, when NES is going to write, it opens the stream again and then it writes and closes the stream.

What is the reason of not keeping it open?

Did not analyze what consequences this would have. What are the drawbacks of the actual solution etc.
Any suggestions and pull requests are welcome.


Relations search/filter(tabular-table?)

Currently it has become quite impossible to find specific relations. E.g. you want to know who was in Budapest in 1885:
https://pmb.acdh.oeaw.ac.at/apis/entities/entity/place/43471/detail
you can try sorting and then go through page after page. But the sorting is not 100% reliable
so it would be a relevant upgrade if the tables in pmb could be represented as tabular-tables with the possibility to enter "1885" in the filter on top

this won't work (easy enough for me), but I want to recreate the relation list/detail views (as in the old application)


Idea: provide a way to customize generated migration and entity naming strategy

Is your feature request related to a problem? Please describe.
Currently, migrations are generated with upper case (just like migration class name).
This can cause problems in some systems; and is a controversed strategy (not going into details here; trying not to be polemic)
Describe the solution you'd like
It would be a good thing to have the ability to customize the file-naming-strategy; just like config allows namingStrategy: UnderscoreNamingStrategy (for table names I assume; just discovered this).
Maybe :
{
    entityFileNamingStrategy: KebabNamingStrategy,
    migrations: {
      migrationFileNamingStrategy: KebabNamingStrategy
    }
}

Which would generate migration-20200402211502.ts instead of Migration20200402211502.ts
EDIT: Maybe along with some prefix / suffix customization; to be able to add things like .model.ts automatically for entities.
Describe alternatives you've considered

Accepting the fact to have a single CamelCased files directory in my codebase...
Renaming each migration each time I generate one, and tell my developers to do so

Additional context
To expand on kebab versus camel. (still avoiding debate, just to say some have reasons to prefer kebab-everywhere, and that it's a decent use-case)
try this
aFile.js
const a = {
  b: 'foo'
}
module.exports = a

another-one.js
const aa = require('./aFile')
const ab = require('./afile')

console.log(aa === ab)

Now retry this on windows, mac, and ubuntu.
It might give different results.
Also :

Another argument https://nodejs.org/api/modules.html#modules_folders_as_modules
Another one https://google.github.io/styleguide/jsguide.html#file-name

We can have a customizable callback to generate the name, why not, just keep in mind that those file names are used for ordering.
Btw the example with windows issues and LC vs UC is kinda invalid, this is about migrations, they will be in a specific folder, all the content is generated, even if future ORM version would change the case, you still have the timestamp suffix.
But what I can understand is that you simply want to have consistent file names across your app, so again, why not :]

EDIT: Maybe along with some prefix / suffix customization; to be able to add things like .model.ts automatically for entities.

Do not mix this with NamingStrategy, that is more complex than just camelCase/PascalCase/snake_case, so it won't fit here - but the customizable callback for migration file name will be just fine. Also note that you can have any entity suffix, including .model.ts, by default. And this is actually something you can customize already, via the NamingStrategy:
https://mikro-orm.io/docs/defining-entities#entity-file-names
https://mikro-orm.io/docs/naming-strategy/#namingstrategygetclassnamefile-string-separator-string-string

@B4nan ,
Callbacks sounds good. About ordering, I'm pretty used to umzug through sequelize-cli and even an own cross-db implementation; the best I can think of is, maybe, timestamp-first (and maybe .migration.ts suffix); but this is all opinions. Upvote for the callback.
I'm pretty much starting with TS, and mikro-orm; and I have to say I'm seduced :)

Hum; well about entities and NamingStrategy, I hadn't read that much the API. It's pretty great.
Thinking of it... you argue

Do not mix this with NamingStrategy, that is more complex than just camelCase/PascalCase/snake_case, so it won't fit here

As far as I can read, this is about Class/prop-to-db-Table/col and the reverse operation.
I think the migrationOption API can be improved in the same direction. I mean, there is (apparently) a need to have a code-to-filename strategy; and there is already a simple filename-to-code strategy (with pattern option). Am I wrong ? :D

Closing as implemented in v4 via https://github.com/mikro-orm/mikro-orm/commit/bdc60be9c7bfd9de45805687738266f1871c9564 (will be part of 4.0.0-alpha.4), see #527 more more updates on that.


Decouple queue package from pipeline.

This PR decouples the queue package from pipeline by moving queue.PipelineSource to pipeline.QueueStore.
This is part of a broader effort to provide a clearer delineation between the "in memory" application coordination code and the persisted layers (see also #209).

You can see this further separation emerging in the modgraph.  It renders handler/cache in a weird spot, but just ignore that :P


State node tests

Codecov Report

Merging #417 into master will increase coverage by 1.12%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master     #417      +/-   ##
==========================================
+ Coverage      81%   82.13%   +1.12%
==========================================
  Files          72       81       +9
  Lines        2522     2703     +181
  Branches       78       79       +1
==========================================
+ Hits         2043     2220     +177
- Misses        479      483       +4

Impacted Files
Coverage Δ

...src/main/scala/com/ing/baker/baas/state/Main.scala
0% <0%> (ø)

...scala/com/ing/baker/baas/state/StateNodeHttp.scala
94.64% <100%> (ø)

.../baker/baas/state/ServiceDiscoveryKubernetes.scala
93.33% <100%> (ø)

...er/baas/state/EventListenersServiceDiscovery.scala
100% <100%> (ø)

...aker/baas/state/InteractionsServiceDiscovery.scala
88.46% <100%> (ø)

...baker/baas/akka/RemoteBakerEventListenerHttp.scala
90.9% <0%> (ø)

.../ing/baker/baas/akka/RemoteEventListenerHttp.scala
76.92% <0%> (ø)

.../com/ing/baker/baas/state/ClusterHealthCheck.scala
0% <0%> (ø)

... and 10 more

Continue to review full report at Codecov.

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


Where will you go??

Hi
Where will you go when CodePlex is shutdown?
Regards
Dennis
Originally posted: 2017-04-28T01:34:11

https://github.com/dahall/taskscheduler
There is an old version there that I posted to hold the URL, but the code is not current and I haven't switched over to use it as the source repository. I'm still looking for a place to move the discussions. That seems to be an important community tool so I'd like to perpetuate it.
Originally posted: 2017-04-28T07:55:04


Stick clockpicker to input-group container

This pull request makes that clockpicker is appended to input-group container. It sticks clockpicker to the input field regardless of changing clockpicker position on the page. Currently clockpicker positioning does not work properly when is placed inside bootstrap modal that has a lot of content and requires scrolling.
Recently:

Pull request fix:

@jalbrzym hey this is not working for me. Can you please try to put it in JSFiddle? Thanks


Need more detail on the payload being sent and examples on transformations on the payload

https://www.rudderstack.com/docs/destinations/webhooks/setup-guide/
I'm dying to try to change the payload to a simple payload:
[{
"version"    : 1,
"type"       : "track",
"userId"     : "019mr8mf4r",
"event"      : "Purchased an Item",
"properties" : {
"revenue"        : "39.95",
"shippingMethod" : "2-day"
},
"timestamp" : "2012-12-02T00:30:08.276Z"
}]'
But the webhook payload has always a specific format:
{
    "body": {
        "XML": {},
        "FORM": {},
        "JSON": {
                    "data": {
                        "premium": false,
                        "question_id": 103597,
                        "answers_count": 0
                    },
                    "name": "Question visited",
                    "external_id": "0f288e49-fbd6-4d65-a7af-ecadfdd225da"
                }
        },
        "JSON_ARRAY": {}
    },

What?

Hi @asennoussi - thanks for raising the issue and apologies for a delayed response - we'll get back to you ASAP.


Has anyone created a d.ts file for this control?

It's a really nice control... would love to use it in an Aurelia project, but I'm having a hell of a time getting the module to import cleanly into the Typescript. Having a d.ts file to go along with it would be great.

No but we are always looking for volunteers. Please feel free to provide a PR.


Protect main branch

Created ruleset for protecting main branch here. @liammulh will you take a look to see if there is anything you recommend to be changed?

Looks good.


Update node tag and configuration input

Closes # .
Pre-review checklist

[ ] All code has been formatted using our config (make format)
[ ] Any new API features or modification of existing behavior is covered as defined in our test plan
[ ] The changelog has been updated under the ## Unreleased header, using the appropriate sub-headings (### Added, ### Removed, ### Fixed), and the links to the appropriate issues/PRs have been included

We will have to update it when there's a proper release


Remote API documentation doesn't mention that the "Content-Length" field is necessary in POST requests

Description of problem:
Neither the documentation itself nor the examples in the Remote API documentation (specifically for v1.20, https://docs.docker.com/reference/api/docker_remote_api_v1.20/ ) mention that the "Content-Length" field is necessary in all POST requests that have a body.
The value in this field is not exactly important, at least as it is larger or equal to the actual body size.
docker version:
Client:
Version:      1.8.2
API version:  1.20
Go version:   go1.4.2
Git commit:   0a8c2e3
Built:        Thu Sep 10 19:19:00 UTC 2015
OS/Arch:      linux/amd64
Server:
Version:      1.8.2
API version:  1.20
Go version:   go1.4.2
Git commit:   0a8c2e3
Built:        Thu Sep 10 19:19:00 UTC 2015
OS/Arch:      linux/amd64
docker info:
Containers: 98
Images: 9
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 211
Dirperm1 Supported: false
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.13.0-63-generic
Operating System: Ubuntu 14.04.3 LTS
CPUs: 4
Total Memory: 7.664 GiB
Name: sirius.mydomain.test
ID: OKAA:4QF6:6QJ4:R4SE:IWFG:YJLY:PFFW:YTEK:URCB:MTCV:QCOY:VP3Y
WARNING: No swap limit support
uname -a:
Linux sirius.mydomain.test 3.13.0-63-generic #103-Ubuntu SMP Fri Aug 14 21:42:59 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
Environment details (AWS, VirtualBox, physical, etc.):
Physical
How reproducible:
Always
Steps to Reproduce:

Execute the following command line in the terminal
$ echo -e "POST /containers/4d50a1065a88/copy HTTP/1.1\r\nContent-Type: application/json\r\n\r\n{\r\n"Resource": "test.txt"\r\n}\r\n" | sudo nc -U /var/run/docker.sock

Actual Results:
HTTP/1.1 500 Internal Server Error
Content-Type: text/plain; charset=utf-8
Server: Docker/1.8.2 (linux)
Date: Wed, 23 Sep 2015 08:35:54 GMT
Content-Length: 4
EOF
HTTP/1.1 400 Bad Request
Expected Results:
HTTP/1.1 500 Internal Server Error
Content-Type: text/plain; charset=utf-8
Server: Docker/1.8.2 (linux)
Date: Wed, 23 Sep 2015 08:39:53 GMT
Content-Length: 59
Could not find the file test.txt in container 4d50a1065a88
Additional info:
The expected result is printed with this command line:
$ echo -e "POST /containers/4d50a1065a88/copy HTTP/1.1\r\nContent-Type: application/json\r\nContent-Length: 1000\r\n\r\n{\r\n"Resource": "test.txt"\r\n}\r\n" | sudo nc -U /var/run/docker.sock

That's because this is part of the http spec.
https://tools.ietf.org/html/rfc2616#section-4.4

For compatibility with HTTP/1.0 applications, HTTP/1.1 requests
containing a message-body MUST include a valid Content-Length header
field unless the server is known to be HTTP/1.1 compliant. If a
request contains a message-body and a Content-Length is not given,
the server SHOULD respond with 400 (bad request) if it cannot
determine the length of the message, or with 411 (length required) if
it wishes to insist on receiving a valid Content-Length.

As @phemmer said, it's part of the HTTP spec.
Closing.

What phemmer wrote does not relate to this doc bug report.
The examples are wrong. They don't work. It's not possible to copy&paste them and get the expected results.
Besides of this, the RemoteAPI documentation SHOULD point to the http spec. It could also mention that - in opposition to the http spec - the value of Content-Length may be larger than the actual length.

@hpollinger oh you're right, we should update documentation to make the copy/paste !! My bad :sweat:

@hpollinger would you be interested in contributing, and opening a pull request with the changes you think are needed? The Markdown files used to generate the API documentation can be found in this repository; https://github.com/docker/docker/tree/master/docs/reference/api

#dibs


Refactor ticket_* tests into unit tests for staff.py

Currently the tests for possible actions on tickets are rudimentary and dispersed:

test_ticket_submission.py tests creation of tickets, but leaves many sections in staff.py untouched.
I recently added test_ticket_actions to test for e.g. deletion, update and creation

Test coverage for staff.py and related ticket functionality should be extended. For this we should refactor these two files into proper unit tests for the staff.py views, with the goal of increasing the test coverage.
This is a little matter of taste, but one could possibly avoid the django.test.Client() but use instead constructed request instances to do proper unit testing of all view functions (and their branches).

I completely agree with you on all points.

When that happens, its definitely time to change the way we do it :-)


[JENKINS-7767] Performance report JMeter not possible to set threshold >0% but <1%

[JENKINS-7767] Changed performance test thresholds to allow 0% and using negative values to indcate no threshold used.
Note: Existing jobs using the performance plug-in might start failing if the threshold is set to 0% (disabled) since it will be enabled after this patch and fail if any error occurs. So the threshold must be changed to -1 to still be disabled after applying the patch.

Hi,
Any idea if and when this is going to be integrated into a new release? I've tried to build the plugin myself but when I install the new version, all the performance related project settings disappear and I've not been able yet to identify the problem.
Kind regards,
M

I'm also waiting on this to be included in a new release. We have thousands of tests and want to fail the build if any of the tests fail. Currently I'll see a .001% failure yet the build is still successful.


Debian packaging

Working on it, see Debian bug#795101

Amazing, thanks @hillu

Thanks, Hilko!  This is really exciting!

Curious, what the status of this?
Thanks,
Alex Wang,

The main problem with getting stenographer into Debian is usage of github.com/golang/leveldb, in particular an issue with failing tests (https://github.com/golang/leveldb/issues/25) that caused that package to removed from Debian, see Debian bug #799472.
I don't know enough about LevelDB to fix this.

BTW, this is done for quite some time now. There's a package for stenographer in Debian buster.


notify the media and contact information that if the "identifier" is changed to the same as the default identifier, there will be a bug.

已修复


execvp(3) failed.: No such file or directory

I get this error if I run bundler or other gems. I'm guessing it's related to rbenv used for ruby version control?
But if I open CP through terminal it all runs great.
Thanks

I guess so too. You need to add the command path to working directory in CP settings page.


[Question] how to include custom attributes in Eloquent model of search results

Does anyone know how to include custom attributes from the index into the model when returning search results?
I've added custom fields in the index using toSearchableArray and these are showing up when returning the raw results. However, when I use the ->get(), the custom fields are not included in the model objects (as they are not in the underlying database table), and I can't seem to find out how to include them.

Two options:
Eloquent accessor

Two options:
Eloquent accessor

PS. You can implement your own class by binding it in the ioc. This can be done in AppServiceProvider::register():
        $this->app->bind(
            'Matchish\ScoutElasticSearch\ElasticSearch\HitsIteratorAggregate',
            'App\Elasticsearch\EloquentHitsIteratorAggregate'
        );

PS. You can implement your own class by binding it in the ioc. This can be done in AppServiceProvider::register():
        $this->app->bind(
            'Matchish\ScoutElasticSearch\ElasticSearch\HitsIteratorAggregate',
            'App\Elasticsearch\EloquentHitsIteratorAggregate'
        );

Great, thanks for the extensive reply @ametad! I've been able to add the required fields, thanks!

Great, thanks for the extensive reply @ametad! I've been able to add the required fields, thanks!


Axis1.3 iss

What Does This Do
Motivation
Additional Notes
增加 axis1.4 支持

 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.You have signed the CLA already but the status is still pending? Let us recheck it.