[BUG] 'No catch for tag: emacs-version-changed' error from doom update

error update with straight branch.
➜  .emacs.d (straight) ✗ doom update                                                                                (straight|…
> Checking core autoloads file
  ✓ Skipping core autoloads, they are up-to-date
> Scanning for outdated packages (this may take a while)...
  .
  .

Output:
/dev/null

--------------------------------------------------

There was an unexpected error:
  No catch for tag (no-catch)
Wrong type argument: listp, "No catch for tag: emacs-version-changed, nil"
➜  .emacs.d (straight) ✗

That happens when straight is compiled with a different version of emacs than the one you're running it on. Please update to the latest commit of straight and run doom build -fto rebuildstraight.el` and necessary packages.

That happens when straight.el is compiled with a different version of Emacs than the one you're running it on. Please update to the latest commit of straight and run bin/doom build -f to rebuild straight.el and necessary packages.

error with  bin/doom build -f
dbus[26964]: Dynamic session lookup supported but failed: launchd did not provide a socket path, verify that org.freedesktop.dbus-session.plist is loaded!
  → Building org-gcal → Building alert...done
  → Building org-gcal...
  → Building org-gcal...done
  → Building elfeed...
  → Building elfeed...done
--------------------------------------------------

There was an unexpected error:
  error (error)
  Invalid version syntax: ‘fatal: No names found, cannot describe anything.’ (must start with a number)

Run the command again with the -d (or --debug) option to enable debug
mode and, hopefully, generate a stack trace. If you decide to file a bug
report, please include it!

Emacs outputs to standard error, so you’ll need to redirect stderr to
stdout to pipe this to a file or clipboard!

  e.g. doom -d install 2>&1 | clipboard-program
Error in Doom Emacs core: error, "Invalid version syntax: ‘fatal: No names found, cannot describe anything.’ (must start with a number)"

error with doom update 2>&1 | clipcopy
Building orgit...
  Update them? (y or n) y

  > Updating "melpa"
  > Updating "emacsmirror-mirror"
  > Updating "smartparens"
  > Updating "protobuf-mode"
  > Updating "org-drill"
  > Updating "ox-hugo"
  > Updating "easy-hugo"
  > Updating "treemacs"
  > Updating "pyim"
  > Updating "counsel"
  > Updating "doom-themes"
  > Updating "doom-modeline"
  > Updating "evil"
  > Updating "docker"
  > Updating "ein"
  > Updating "lsp-mode"
  > Updating "magit"
  > Updating "forge"
  > Updating "ox-reveal"
  > Updating "lsp-java"
  > Updating "cargo"
  > Updating "go-mode"
  > Updating "cmake-mode"
  > (Re)building packages...

  → Building smartparens...
  → Building smartparens → Building dash...
  → Building smartparens → Building dash...done
  → Building smartparens → Building cl-lib...
  → Building smartparens → Building cl-lib...done
  → Building smartparens...
  → Building smartparens...done
  → Building protobuf-mode...
  → Building protobuf-mode...done
  → Building org-drill...
  → Building org-drill → Building seq...
  → Building org-drill → Building seq...done
  → Building org-drill → Building org...

  → Building org-drill → Building org...done
  → Building org-drill → Building persist...
  → Building org-drill → Building persist...done
  → Building org-drill...
  → Building org-drill...done
  → Building ox-hugo...
  → Building ox-hugo...done
  → Building easy-hugo...
  → Building easy-hugo → Building popup...
  → Building easy-hugo → Building popup...done
  → Building easy-hugo → Building request...
  → Building easy-hugo → Building request...done
  → Building easy-hugo...
  → Building easy-hugo...done
  → Building pyim...
  → Building pyim → Building async...
  → Building pyim → Building async...done
  → Building pyim → Building pyim-basedict...
  → Building pyim → Building pyim-basedict...done
  → Building pyim...
  → Building pyim...done
  → Building ivy...
  → Building ivy...done
  → Building counsel...
  → Building counsel → Building swiper...
  → Building counsel → Building swiper...done
  → Building counsel...
  → Building counsel...done
  → Building doom-themes...
  → Building doom-themes...done
  → Building doom-modeline...
  → Building doom-modeline → Building all-the-icons...
  → Building doom-modeline → Building all-the-icons → Building memoize...
  → Building doom-modeline → Building all-the-icons → Building memoize...done
  → Building doom-modeline → Building all-the-icons...
  → Building doom-modeline → Building all-the-icons...done
  → Building doom-modeline → Building shrink-path...
  → Building doom-modeline → Building shrink-path → Building s...
  → Building doom-modeline → Building shrink-path → Building s...done
  → Building doom-modeline → Building shrink-path → Building f...
  → Building doom-modeline → Building shrink-path → Building f...done
  → Building doom-modeline → Building shrink-path...
  → Building doom-modeline → Building shrink-path...done
  → Building doom-modeline...
  → Building doom-modeline...done
  → Building treemacs...
  → Building treemacs → Building ace-window...
  → Building treemacs → Building ace-window → Building avy...
  → Building treemacs → Building ace-window → Building avy...done
  → Building treemacs → Building ace-window...
  → Building treemacs → Building ace-window...done
  → Building treemacs → Building pfuture...
  → Building treemacs → Building pfuture...done
  → Building treemacs → Building hydra...
  → Building treemacs → Building hydra → Building lv...
  → Building treemacs → Building hydra → Building lv...done
  → Building treemacs → Building hydra...
  → Building treemacs → Building hydra...done
  → Building treemacs → Building ht...
  → Building treemacs → Building ht...done
  → Building treemacs...
  → Building treemacs...done
  → Building evil...
  → Building evil → Building undo-tree...
  → Building evil → Building undo-tree...done
  → Building evil → Building goto-chg...
  → Building evil → Building goto-chg...done
  → Building evil...
  → Building evil...done
  → Building docker...
  → Building docker → Building docker-tramp...
  → Building docker → Building docker-tramp...done
  → Building docker → Building magit-popup...
  → Building docker → Building magit-popup...done
  → Building docker → Building tablist...
  → Building docker → Building tablist...done
  → Building docker → Building json-mode...
  → Building docker → Building json-mode → Building json-reformat...
  → Building docker → Building json-mode → Building json-reformat...done
  → Building docker → Building json-mode → Building json-snatcher...
  → Building docker → Building json-mode → Building json-snatcher...done
  → Building docker → Building json-mode...
  → Building docker → Building json-mode...done
  → Building docker...
  → Building docker...done
  → Building ein...
  → Building ein → Building websocket...
  → Building ein → Building websocket...done
  → Building ein → Building auto-complete...
  → Building ein → Building auto-complete...done
  → Building ein → Building deferred...
  → Building ein → Building deferred...done
  → Building ein → Building polymode...
  → Building ein → Building polymode...done
  → Building ein → Building markdown-mode...
  → Building ein → Building markdown-mode...done
  → Building ein → Building skewer-mode...
  → Building ein → Building skewer-mode → Building simple-httpd...
  → Building ein → Building skewer-mode → Building simple-httpd...done
  → Building ein → Building skewer-mode → Building js2-mode...
  → Building ein → Building skewer-mode → Building js2-mode...done
  → Building ein → Building skewer-mode...
  → Building ein → Building skewer-mode...done
  → Building ein...
  → Building ein...done
  → Building lsp-mode...
  → Building lsp-mode → Building dash-functional...
Output:

--------------------------------------------------

There was an unexpected error:
  Symbol's value as variable is void (void-variable)
Wrong type argument: listp, "Symbol’s value as variable is void: ein:notebook-mode"

@shuxiao9058 Are you still experiencing this issue on the latest commit of the straight branch? A few improvements were made to doom update which should address your issues (including the elfeed issue).

@hlissner Thanks for your work, it works fine now.


Relist behaviour differs between ServiceBroker and ClusterServiceBroker

Bug Report
What happened:
For controller manager I set broker-relist-interval to 1minute.
My definitions for ClusterServiceBroker and ServiceBroker are simple, I specify only broker URL. In logs I see the following error:
E0828 07:11:05.871077       1 controller_servicebroker.go:117] ServiceBroker "default/ns-broker" v37: Unable to process because RelistBehavior is set to Duration with a nil RelistDuration value

(every 5 minute which is also strange).
In the case of ClusterServiceBroker there is the following line:
duration := defaultRelistInterval

(https://github.com/kubernetes-incubator/service-catalog/blob/master/pkg/controller/controller_clusterservicebroker.go#L127)
and it is prepopulated from ControllerManagerConfiguration.ServiceBrokerRelistInterval.
In case of ServiceBroker this line is missing. Is it on purpose?
As a result ServiceBrokers probably won't be relisted at all.
What you expected to happen:
ServiceBroker and ClusterServiceBroker relist behaviour is the same
Environment:

Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.0", GitCommit:"91e7b4fd31fcd3d5f436da26c980becec37ceefe", GitTreeState:"clean", BuildDate:"2018-06-27T22:29:25Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.0", GitCommit:"fc32d2f3698e36b93322a3465f63a14e9f0eaead", GitTreeState:"clean", BuildDate:"2018-03-26T16:44:10Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

service-catalog version: 0.1.30
Cloud provider or hardware configuration: minikube
Do you have api aggregation enabled?

Do you see the configmap in kube-system?
Does it have all the necessary fields?

kubectl get cm -n kube-system extension-apiserver-authentication -o yaml and look for requestheader-XXX fields

Install tools:

Did you use helm? What were the helm arguments? Did you --set any extra values?
helm install svc-cat/catalog  --name catalog --namespace catalog

Are you trying to use ALPHA features? Did you enable them?

I missed it in #2125.
Needs a similar change as in https://github.com/kubernetes-incubator/service-catalog/commit/ea84fa59baf8899b85f76354804fc98f9894b220#diff-1aa1085b3dd6f459f3f1e5d2ec2ec35b
/help-wanted

/assign @luksa


4310 - Update item quantity validation error message

Resolves #4310

Updates validation error message when adding an item quantity
Updates related unit tests and system specs

Screenshots

Looks good to me. @cielf did you want to kick the tires?

Looks good to me -- i noticed a different issue, but it's out of scope of this change.

Let's merge after the release, though.


How to send html email?

Hello Graham,
I tried to send html email by adding html code such as  to the email body, but after received it's shown as plain text.
Is there a way to send html emails? Thanks!

I've updated the code to treat emails as HTML by default.  I've added a new parameter to the CreateSesMessage function although this has a default value of "true" so your code should not need any changes to use the new HTML support.
Note:  will not appear to show anything in the email as there's no text between the >< tags.

Thanks Graham, will try it later.


shrink mnist accuracy test and make it run on nosetests / cpu

OK, I confirm that on cirrascale, I get the same drop in accuracy if I use CUDA_VISIBLE_DEVICES=-1. So

It seems like tensorflow is using GPU on cirrascale even when CUDA_VISIBLE_DEVICES is an empty string
It seems like this is likely to be an issue with tensorflow CPU accuracy not matching tensorflow GPU accuracy, rather than anything we do in cleverhans. Somewhat hard to tell if it's clearly a "bug" or just catastrophic changes in compounding rounding error.


Update {{ImageURL}} for Pocketcasts streams to retrieve "https://" links (currently retrieving http://)

Guten abend @marcjulianschwarz,
Problem

With Pocketcast imports, the {{ImageURL}} placeholder is currently retrieving http:// image links.
When transcluded ![image](http://static.pocketcasts.com/...), these image links are broken on Obsidian mobile (but they do load on Obsidian desktop!).
If I manually change the link to https:// the image transclusions load properly in Obsidian preview.

What I tried

I tested with Google Podcasts and Spotify services, and Podcast Note retrieves https:// links.
I didn't test any other of your supported services.

Requested solution

is it possible for you to instruct Podcast Note to retrieve the {{ImageURL}} placeholder as https:// links.

Vielen Dank!
Tom

Guten Abend @DryIce1 ;)
Thank you for pointing out this issue.
Do I understand it correctly that http links don't work on the mobile version but they do work with the desktop version?

Yes, that is my impression. The Pocketcasts images load if I change them to https.
Also if I change a Spotify https image to http, it doesn't load.

Thank you.
Fixed with 1.3.0


make handler return type interfaces

This PR adds a Reponser interface, which enables us to construct any form of custom type and turn them into responses accepted by goapi-gen.
The goal of this proposal is to shorten the error handling code, and clean up ugly multi-line errors made when constructing a non-standard/specified response body.
The example provided with the petstore may not be the best because of it's short nature, but we can easily save dozens of lines of code, and standardize better with such an API.

After some more reading and code reviewing, I figured I could just reuse render's Renderer interface for it, thus changing very little, yet achieving the desired functionality.


lime display air output is missing -swf-lib for native extensions

...which causes a failure when vshaxe tries to build a cache, see https://github.com/vshaxe/vshaxe/issues/270.

To reproduce, start with an empty OpenFL project and:

add import lv.hansagames.extensions.settings.SettingsANE; to Main.hx

add  this to project.xml:
<dependency name="lv.hansagames.extensions.settings.SettingsANE" path="ext/SettingsANE.ane" if="air" />

create an ext directory and add this file to it

run lime build air - this should succeed

run lime display air - this is missing -swf-lib ext/SettingsANE.ane

open the project in VSCode - building the cache fails:

Ah, so this is located in hxp now. I guess issues with the tools should preferably be opened there then?


Requesting support as a non-experienced user.

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Thank you for your contribution! We will review the pull request and get back to you soon.

This seems to be a spam.


Ruby Concurrent Jscall failure

When ruby code use Jscall, if the Ruby use multi-thread concurrent, the Jscall will be failed.

# define a function in js
Jscall.exec <<-CODE
    async function test(x)
    {
        await new Promise(r => setTimeout(r, 1000));
        return x
    }
CODE

# concurrent call in js
Jscall.exec <<-CODE
    async function parallel() {
        let p = []
        for (var i=1;i<=20;i++) {
            p.push(test(i))
        }
        return await Promise.all(p)
    }
CODE
ret1 = Jscall.parallel()

puts ret1.to_s
# output [1,2,3,4,5 ... , 20], ok

# non-concurrent call in ruby
ret2 = (1..20).to_a.map do |x|
    Jscall.test(x)
end
puts ret2.to_s
# output [1,2,3,4,5 ... , 20], ok

# concurrent call in ruby
ret3 = (1..20).to_a.map do |x|
    Thread.new do
        Jscall.test(x)
    end
end.map(&:value)

puts ret3.to_s
# except [1,2,3,4,5 ... , 20], but error```

Yes, that's true.  The implementation of Jscall supports only single-threaded code in Ruby.

Is there any possibility that we could support multi-threaded code from Ruby call (As the example I showed)
My thought was like this:  in the send_command / send_reply data send from Ruby via PipeToJs, we add a session_id field to identify the session (The session_id is stay the same for multiple send_command / send_reply back-forth in one Ruby initiate Jscall.exec)
When the Ruby side Pipe receive the data in one thread (there could multiple threads initiate Jscall.exec with different session), the thread only fetch data in Pipe who has the same session_id.  (Like a Messaging Queue, receiver only subscribe the specific topic)
For the JS side Pipe, it can process all session_id data in the same main_loop.  But the process should be non-block via JS concurrency.
The Pipe between JS & Ruby is the shared resource.  So multiple session can use the Pipe together with mutex and topic subscribe.


Update README

Update the README to better reflect how the plugin works.  See for example https://github.com/jmcook1186/carbon-intensity-plugin

I'd like to contribute to the README plz.

I'd like to contribute to the README plz.

Thanks! I've added you to the the list of collaborators so you'll be able to open a branch and then a pull request.  We appreciate the help!


Ensure only Karma tests run on win32, move items from lib/setup to setup

In regular win32 environments, redis-server won't be available and launching browser-test-server in the background doesn't behave correctly, so we only run Karma tests. In Windows Subsystem for Linux, launching browsers isn't straightforward and PhantomJS currently crashes (see https://github.com/mbland/wsl-websocket-crash-demo), so we run all tests except for Karma tests.
Now that there's no longer more than one setup script, and there's very little in scripts/lib/setup to begin with, it seems better to just reabsorb URLP_PLATFORM and urlp.npm_install into script/setup. That said, the urlp.npm_install implementation may make sense to include in a go-script-bash module for the reasons stated in its function comments, namely that file streams are managed differently on Windows, and npm can be pretty noisy anyway.

Coverage remained the same at 97.742% when pulling 0ffb0c2b0defc4f73094b564819a0aed45fd0a19 on setup-tweaks into c117b7e037bc7625a0edaa7a50d77e76b96e0f99 on master.


doc: added uprobe info

Closes: #878
Preview here.
Added doc support for uprobe for user to get overview of what it is and how to use with tracingPolicy in the concepts section

Thank you @mtardy  😊

@mtardy mind giving this an ACK if your happy with the updates?

@mtardy mind giving this an ACK if your happy with the updates?

yes, please give me some time to understand something on our uprobes support to document it correctly and we'll merge this.

Thank you for improving this @mtardy  😊


Adding a favicon to the website

Will add website logo as favicon

Please assign me.

Please assign me to this @QAZIMAAZARSHAD

@aakankshabhende do you want to work on it .... ? or i assign it to @Rithwik07

I want to work on it
On Sat, Jul 10, 2021, 11:45 PM Satyam Dengre @.***>
wrote:

@aakankshabhende https://github.com/aakankshabhende do you want to work
on it .... ? or i assign it to @Rithwik07 https://github.com/Rithwik07
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/QAZIMAAZARSHAD/Apni-Dukaan/issues/55#issuecomment-877680677,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APCR2BBMWKKKGAWNGJIZVSLTXCE3DANCNFSM5AEME73A
.

I too wanted to work on it.
On Sat, Jul 10, 2021, 11:46 PM Aakanksha Bhende @.***>
wrote:

I want to work on it
On Sat, Jul 10, 2021, 11:45 PM Satyam Dengre @.***>
wrote:

@aakankshabhende https://github.com/aakankshabhende do you want to
work
on it .... ? or i assign it to @Rithwik07 https://github.com/Rithwik07
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/QAZIMAAZARSHAD/Apni-Dukaan/issues/55#issuecomment-877680677
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/APCR2BBMWKKKGAWNGJIZVSLTXCE3DANCNFSM5AEME73A
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/QAZIMAAZARSHAD/Apni-Dukaan/issues/55#issuecomment-877680848,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQBIPIPIPXVB4QQ2U6QRCVDTXCE75ANCNFSM5AEME73A
.

Brother, this is the first time iam working on a project.
I find everything strange.
But fevicon one , I knew it .
It would be a good start for me , if I get this opportunity.
Thank you.
On Sat, Jul 10, 2021, 11:47 PM Rithwik Shettie @.***>
wrote:

I too wanted to work on it.
On Sat, Jul 10, 2021, 11:46 PM Aakanksha Bhende @.***>
wrote:

I want to work on it
On Sat, Jul 10, 2021, 11:45 PM Satyam Dengre @.***>
wrote:

@aakankshabhende https://github.com/aakankshabhende do you want to
work
on it .... ? or i assign it to @Rithwik07 <https://github.com/Rithwik07
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/QAZIMAAZARSHAD/Apni-Dukaan/issues/55#issuecomment-877680677
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/APCR2BBMWKKKGAWNGJIZVSLTXCE3DANCNFSM5AEME73A
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/QAZIMAAZARSHAD/Apni-Dukaan/issues/55#issuecomment-877680848,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQBIPIPIPXVB4QQ2U6QRCVDTXCE75ANCNFSM5AEME73A
.

@aakankshabhende, @Rithwik07 this issue is already open.
Before opening any issue make sure that no other similar issue is open.

Brother, this is the first time iam working on a project. I find everything strange. But fevicon one , I knew it . It would be a good start for me , if I get this opportunity. Thank you. On Sat, Jul 10, 2021, 11:47 PM Rithwik Shettie @.> wrote:
…
I too wanted to work on it. On Sat, Jul 10, 2021, 11:46 PM Aakanksha Bhende @.> wrote: > I want to work on it > > On Sat, Jul 10, 2021, 11:45 PM Satyam Dengre @.***> > wrote: > > > @aakankshabhende https://github.com/aakankshabhende do you want to > work > > on it .... ? or i assign it to @Rithwik07 <https://github.com/Rithwik07 > > > > > > — > > You are receiving this because you were mentioned. > > Reply to this email directly, view it on GitHub > > < > #55 (comment) > >, > > or unsubscribe > > < > https://github.com/notifications/unsubscribe-auth/APCR2BBMWKKKGAWNGJIZVSLTXCE3DANCNFSM5AEME73A > > > > . > > > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <#55 (comment)>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/AQBIPIPIPXVB4QQ2U6QRCVDTXCE75ANCNFSM5AEME73A > . >

That issue has already been assigned to others, look for other issues.


Improve the codegen diff bot

Motivation and Context
This updates the codegen diff bot GitHub Action to generate an HTML diff and upload it to a CloudFront fronted S3 bucket so that the diffs can live longer and so that we don't need to push branches to create the diffs anymore.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The last diff posted above represents what the diffing looks like with this PR.

I'm going to tear down the old stack in order to rename things. The old diff comments will become broken links. Will also play around with splitting out the SDK vs. the server codegen diffs into separate files, adding whitespace/no-whitespace diffing, and modifying the bot comment to say if a diff has any changes in it.


Yubico.NativeShims doesn't build as expected with AOT

When building an executable the native AOT compiler, Yubico.NativeShims.dll is output next to the executable instead of being trimmed away. There are also no compiler warnings about dependencies having trimming issues, which is odd. But I suspect it has to do with native shims being pre-built/native or shared vs static, it's not IL. It probably needs to be IL or static, or maybe static with a p/invoke wrapper class, in order for trimming to work (not sure haven't done one of these myself before).
Expected output
single executable with no libraries around it
What's actually happening
Yubico.NativeShims.dll in the output dir.
Steps to reproduce

dotnet new console
dotnet add package Yubico.YubiKey
Add <PublishAot>true</PublishAot> to the project file
dotnet publish -o ./out

Sample with expected output
Instead of adding the YubiKey reference add one to Newtonsoft.Json
You should get a ton of warnings about trimming, but the output is as expected absent of any Newtonsoft libs.
Why do I want this?
if you take a peek at my latest builds: https://github.com/SheepReaper/yk-csr-generator/releases/tag/v1.0.0 I'm building 80MB assemblies because of the runtime bits included. Granted, if I didn't do single file, it would be way smaller, but then the user needs to have a runtime installed. I just want to build a single executable with no external dependencies, so I'm fine with the bloat.
However, in testing AOT builds, I was able to achieve a 1.5KB binary plus the Shim lib next to it with almost 3KB. Which is super nice, but we're back at the original problem of external dependencies.
Additional references

native static library: https://github.com/dotnet/samples/blob/main/core/nativeaot/NativeLibrary/README.md
aot compatible library limitations: https://learn.microsoft.com/en-us/aspnet/core/fundamentals/aot/native-aot-tutorial?view=aspnetcore-8.0&tabs=netcore-cli#libraries-and-native-aot
declaring your libs to be aot compatible enables the aot-compatibility analyzers (compiler warnings): https://learn.microsoft.com/en-us/dotnet/core/deploying/native-aot/?tabs=net8plus%2Cwindows#aot-compatibility-analyzers
preparing libs for AOT: https://aka.ms/dotnet-illink/libraries

I don't know enough about the Shims lib to determine what about it is preventing AOT from working nicely, but worst case it's hitting one of the many limitations of AOT compatibility, which might be too much work to overcome. In which case, I'll be sticking to my thicc binaries...

Phew - clearly there has been a lot of advancements on the NativeAOT front that I need to catch up on!
We have some of our own internal motivations to get the SDK more NativeAOT friendly - however I'm not sure what the timeline on this will be. The SDK was just not designed with NativeAOT in mind. We architected it back in the .NET Core 3.1 era when Microsoft still treated the AOT runtime as a science experiment and had zero commitment to the project.
I think our goals do align here. I'm just not sure that I can commit to a timeline. Is this a showstopper for your project? Or something that you would simply like for us to improve over time? Left to our own devices, we would likely bake in NativeAOT support over the next 6 months or so. But again, that's not a commitment. The trickiest part being that we need to maintain .NET Framework 4.7.2+ compatibility, which means we're stuck on .NET Standard 2.0. I've noticed some of the new source generators that help with AOT target .NET Standard 2.0 (such as the JSON one), but others like the LibraryImport/PInvoke one do not.
I really appreciate the links. Your reports have been super enlightening and helpful to increase my own knowledge 😄
As an aside to this issue: If you know of any links or sample projects that would serve as a better structure for our NativeShims project, I would love to learn. I've never been happy with how that project has worked out. Since one of the goals of the SDK has been to be usable "out of the box" - that's meant we've needed to (NuGet) package our own native dependencies. We've taken a lot of ideas from how the dotnet/runtime project itself does it (at least how they did things 3 years ago) but without taking a dependency on their very complicated build system. It seems to work for 90% of our consumers, but there always seems to be special cases (package.config, PowerShell, etc.) that have just been difficult if not impossible to figure out.
I'm not sure we'll be able to use the DirectPInvoke stuff, though that looks amazing. I'm still reading through your resources, so maybe I'll find more insights in there. Remember that whatever we use, our SDK needs to be consumable/compatible with .NET Framework applications. I have no problem taking a build dependency on the .NET 8 SDK, though.
Any additional insights would be greatly appreciated!

Definitely not a show-stopper for me, my little tool has no adoption as of yet. I did make a typo though, my assemblies are only 20MB. modern .NET has me sold on the native cross-platform capabilities, but I hear the nay-sayers screaming about binary size, and (insert argument invented in the 90s about memory measured in Kilobytes). AOT just seems like the thing that, if it works, why not?
So yeah, no hurry. It's never as simple as specifying multiple target frameworks. If I knew how to do it, I'd share, but I don't do much direct native development these days.


Detecting audio/video call in rocket chat iOS app

Currently in my iOS app I'm using rocketchat integrated with Jitsi so as to enable audio and video calls.
currently when I send a call through the functionality openAudioOrVideoCall the other end user when receives this call, there is no way to detect whether this is an audio or video call so it mutes/unmutes the camera. Is there a way to get this working in my app ?.
Any help would be appreciated, best regards :)

It's android repository, IOS related issue should be submitted to https://github.com/RocketChat/Rocket.Chat.iOS


NO MERGE - Upstream 155

@dsamojlenko can you give this a review - if we do merge it should be before the next set.


Support for positional arguments

I have a command that can be invoked like so:
my_command --foo=bar "first_arg" "second_arg"

(first_arg and second_arg are arbitrary strings)
My parser definition looks like this:
parser_definition() {
  setup ARGS
  param FOO --foo
}
eval "$(getoptions parser_definition __parse) exit 1"
__parse "$@"

echo "$FOO"

That all works great, but then I need to do something like this for the remaining positional arguments:
parse_args() {
  local var_names=("$@")
  local i=0
  for arg in $ARGS; do
    echo export ${var_names[$i]}="$arg"
    i=$((i + 1))
  done
}

eval "$(parse_args "FIRST" "SECOND")"

echo $FIRST         # outputs "first_arg"
echo $SECOND    # outputs "second_arg"

Do you think it makes sense to add a new kind of argument to the parser_definition where I can define the positional arguments too? Maybe something along these lines:
parser_definition() {
  setup ARGS
  param FOO --foo
  arg MY_ARG
  arg ANOTHER_ARG
}

I imagine that this could filter $@ to elements that do not begin with a - or --, and then $MY_ARG is assigned the value of the first arg, $ANOTHER_ARG is assigned the value of the second arg, and so forth.
There are probably other considerations for this library, but as a starting point, does this idea make sense and/or would you be interested in a PR to add something like that?

It should be able to be written as follows
parser_definition() {
  setup ARGS
  param FOO --foo
}
eval "$(getoptions parser_definition __parse) exit 1"
__parse "$@"
eval "set -- $ARGS" # 👈

echo "$FOO"

FIRST="$1" SECOND="$2"

echo $FIRST         # outputs "first_arg"
echo $SECOND    # outputs "second_arg"

It could be simplified a bit more and written as follows
parser_definition() {
  setup ARGS
  param FOO --foo
}
eval "$(getoptions parser_definition) exit 1"
echo "$FOO"

FIRST="$1" SECOND="$2"

echo $FIRST         # outputs "first_arg"
echo $SECOND    # outputs "second_arg"

Perhaps documentation needs to be improved.

Right. I'm saying that it would be nicer if the positional args could be defined in the same parser definition function (and have the positional args loaded into the right vars), rather than doing extra stuff after calling __parse

Understood. I'm asserting that the args vs options vs flags vs params argument is a distinction without a difference. Ultimately, all flags/params/options/args are going to be in $@, so why not have a simple mechanism for taking the remaining arguments and sticking them into the desired variables?

As @kkerce says, the thought that the treatment of (non-option) arguments is out of scope is one reason. But, I do not think it is that far out of scope. However, In my opinion, arg is unnecessary. Because it can be implemented in one line without arg.
Although getoptions is designed to be embedded in shell scripts as a library, some people are not comfortable with the inclusion of long option parsing code. So I prefer to keep my implementation of getoptions as small as possible.
About Terms: I follow the POSIX terminology (in principle) and consider flags and options to be the same thing and flags to be historical usage.

are known as "options" (or, historically, "flags").

However, the term “operand” does not seem to be common. Therefore, I use “argument” instead of “operand” and the meaning of “argument” has become ambiguous. “argument” may be used to mean all arguments including options, or it may mean ”operand".


Hardware constrain for Stage-2

Hi @CiaoHe ,
Thanks for sharing great work, I was trying to recreate the results, you mention in paper,  You have mention in paper they you have been doing "Full fine tuning on stage-2 ".   I have tried to full fine-tune model on stage-2 on my 4 A100 40GB  GPUs, but its seems they are not enough, can you provide what exact hardware requirements for training full fine-tuning the model ?
Also there are about 6.7B trainable parameters in stage-2, are we training all the parameters ? or train a subset of Parameters ?
Moreover have you tried to train Stage-2 on LoRA ? what are your thoughts on train stage-2 and stage-3 on LoRA ?

In stage-2, we trained through full-finetuning on 8 A6000 GPUs (48GB). So for 4xA100 40GB is quite challenging.
We've tried train using LoRA and did some ablation (refer to Figure 6 or Appendix C.1). If you don't need to maintain the model's ability on other general tasks (like reasoning, and knowledge evaluation), we suggest full-tuning.

Hi @CiaoHe ,
Thanks for your quick responses, I tried to find technique to full fine tuning of model, but It seems we might need to quantize the model, can you suggest  your opinion on selecting precision  ?

We trained with bf16.


FEAT: support BlueLM from vivo

hf: https://huggingface.co/vivo-ai
modelscope: https://modelscope.cn/organization/vivo-ai

bluelm是已经支持了吗？

mark


Can't verify second domain's Return Path?

I've setup my first domain and postal is working fine. Today I wanted to add another domain.
Here is the domain setup.
SPF and DKIM are verified fine. But Return Path still can't get verified.

I've made CNAME record as setup suggests

This:

Anyone help?

It looks like your DNS record isn't publicly available so you should discuss it with GoDaddy who currently handle the DNS.

Ah, I thought so. I'm gonna contact Godaddy support and will let you know.

Hmm It should be just psrp not psrp.nmma.co in DNS settings. It worked, thanks.


能不能单独扩展一下WX api然后放出来 不要求全部但是常用的能多扩展一下么  不是所有人都能整明白多态

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

...


TO-TABLE on the table-range-mixin does not allow for the transformation control functions

This is because this method won't call transform, instead working with the iterator directly. This is intentional, as transform will call value predicates, where those predicates are already called before. Adding control blocks to the method body should solve this issue.

OK, this has been resolved, as much as feasable.

OK, this has been resolved, as much as feasable.


UG fixes

Update sample data
Include annotated Ui image
Include terms in the glossary and FAQ

Codecov Report

Merging #212 (8476e6e) into master (9c99c0a) will decrease coverage by 0.16%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #212      +/-   ##
============================================
- Coverage     71.61%   71.44%   -0.17%
  Complexity      714      714
============================================
  Files           114      114
  Lines          2332     2336       +4
  Branches        275      275
============================================
- Hits           1670     1669       -1
- Misses          567      572       +5
  Partials         95       95

Impacted Files
Coverage Δ

.../java/seedu/address/model/util/SampleDataUtil.java
15.38% <0.00%> (-7.35%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


params: drop evidence.MaxAgeNumBlocks in the next major release

After PBTS is enabled by default for all chains, we should be able to remove evidence.MaxAgeNumBlocks (see https://github.com/tendermint/tendermint/issues/2653#issuecomment-438524834). That's because a timewarp attack (+1/3 cabal modifying block's time) will no longer be possible.

My understanding is that PBTS will not be enabled by default.
If enable is not enforced then I believe you might still need to keep evidence.MaxAgeNumBlocks around, or do a big refactoring of the evidence codebase logic to check if PBTS is enabled or disabled (or when it will be enabled) and then use MaxAgeNumBlocks in the logic or not.
But since PBTS can be enabled for a future height via FeatureParams.PbtsEnableHeight I guess this just complicates the logic for evidence validation.

But block times can still be arbitrarily long, due to failures, asynchrony, etc.
So we probably still need to have the requirement of both time interval ("real" time) and blocks committed ("logic" time).

But block times can still be arbitrarily long, due to failures, asynchrony, etc.
So we probably still need to have the requirement of both time interval ("real" time) and blocks committed ("logic" time).

You mean 1/3+ cabal will still be able to delay block commit for arbitrary time. Therefore we still need MaxAgeNumBlocks?


chore: add promise-function-async rule

Description
Add promise-function-async rule

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test


Discrepancy between Amplify CLI deployment using local push and Amplify CI/CD deployment for serverless container api

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v18.18.0
Amplify CLI Version
12.10.1
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
no
Describe the bug
Resource access permissions not updated for container based server less REST API when deploying it through Amplify CI/CD but permissions updated when deploying through amplify push from local system.
Expected behavior
Resource access permissions should be updated when deploying  container based server less REST API using Amplify CI/CD
Reproduction steps

amplify init
enable container based deployment using  amplify configure project
amplify add api (Add container based serverless api)
amplify add storage
amplify add function
amplify update api
Do you want to access other resources in this project from your api? Yes
? Select the categories you want this function to have access to.

◉ function
◉ storage
❯◉ testfunction
◉ testfunction2
◉ restserverlessapia6c7abf2
❯◉ create
◉ read
◉ update
◉ delete
You can access the following resource attributes as environment variables from your Lambda function
FUNCTION_RESTSERVERLESSAPIA6C7ABF2_NAME
FUNCTION_TESTFUNCTION2_NAME
FUNCTION_TESTFUNCTION_NAME
STORAGE_TEST6_ARN
STORAGE_TEST6_NAME
STORAGE_TEST6_STREAMARN
STORAGE_USERS2_ARN
STORAGE_USERS2_NAME
STORAGE_USERS2_STREAMARN
STORAGE_USERS_ARN
STORAGE_USERS_NAME
STORAGE_USERS_STREAMARN
permissions updated in "containerapi-cloudformation-template.json" file.
amplify push -y
permissions added in "TaskDefinitionTaskRoleDefaultPolicy282E8624"
update in GitHub or Code Commit repo and trigger Amplify build
permission lost in "TaskDefinitionTaskRoleDefaultPolicy282E8624"
There is no any permission lost issue when we do deployment from local system using amplify push but permissions lost when trigger Amplify build using CI/CD
Project Identifier
No response
Log output

# Put your logs below this line

Additional information
Workaround : custom-policies.json
Tested deployment using CI/CD after adding permissions in custom-policies.json file and it's added to TaskDefinitionTaskRoleFD4-idoWvD9uZrLv with customer inline policy name CustomExecutionPolicyForContainer.
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Since the issue occurs only in the hosting CI/CD environment and not via local push, we believe hosting team would be able to look into the relevant logs and help you better. Closing this since it is being tracked internally and assigned to relevant team.

I see that the permissions are updated in amplify-meta.json file but not correctly updated in the backend-config.json file. amplify-meta.json doesn't get committed to the repo, so when the deployment happens through CI, it has no idea about the policy changes.
As a workaround, for now, manually copy the value of api > [API_NAME] > categoryPolicies from the amplify-meta.json to backend-config.json (same path: api > [API_NAME] > categoryPolicies), save the file and commit to the repo. Wait for the deployment to complete and verify the policy.


vendor:publish failed with Nothing to publish for tag []. and more problems with Intervention

Hi,
i have installed the newest version of intervention for Laravel 5.2. After I have inserted the necessary lines in config/app.php and started the command php artisan vendor:publish --provider="Intervention\Image\ImageServiceProviderLaravel5" the console shows me Nothing to publish for tag []. Whats wrong with that?
Then I have copied manually the file image.php in the config folder. Is that ok or is it important to register the script with the publish command?
It is not possible to configure the driver in this file. Intervention uses permanently the gd driver. But I have changed the line to 'driver' => 'imagick'. How can I use imagick without inserting Image::configure(array('driver' => 'imagick')); in my controllers?
Why I cant use the image object of Intervention in my controllers? I have set Illuminate\View\ViewServiceProvider::class, under providers and 'Image'      => Intervention\Image\Facades\Image::class, under aliases and refreshed the Laravel autoloader. But the only way to use Image is to define it by use Intervention\Image\ImageManagerStatic as Image;
I don´t see anything of this under the installation instructions: http://image.intervention.io/getting_started/installation

I had the same problem. Try to run:
php artisan config:clear

after that:
php artisan vendor:publish --provider="Intervention\Image\ImageServiceProviderLaravel5"


Add support for automatically building docker images as part of CI

@FedericoCeratto mentioned it would be useful if we were building docker images for services like ooni/api, but possibly also ooni/explorer as part of the CI process.
This would allow us to deploy things faster.

This was done for API. I am closing this and suggest we rather open new issues for the other components as needed.


Fixed #32322 -- Fixed autocomplete widget wrapping.

Ticket https://code.djangoproject.com/ticket/32322

buildbot, test on selenium.

buildbot, test on selenium.

@grabov Thanks for this patch :+1: Welcome aboard :boat:

@grabov Thanks for this patch :+1: Welcome aboard :boat:


Will JetPack 4.4 be supported?

@ccerhan you find any way to setup on jetpack4.4?

@ccerhan you find any way to setup on jetpack4.4?

@ccerhan its working fine on jetpack 4.4

@ccerhan its working fine on jetpack 4.4


Ensure global-ray is uninstalled before uninstalling package

This PR adds the global-ray uninstall command to be performed before global-ray is uninstalled via composer on the users system.
This is to ensure their PHP.ini is updated and will not generate an exception due to missing the global-ray-loader.php include post-uninstall.

A nice, I didn't know there was a hook for this 👍


Question of the paper——The forward pass may forget one rescaling？

On page 4 of flashattention2, is the 4th equation of O(1) has the element $$e^{S(1)-m(1)}$$
but the last equation just becomes the $$e^{S(1)-m}$$
This happened on page 5, too.
From the former paper flashattention1, it has a rescaling of $$e^{m(1)-m} $$ in the equation of O(2) and O(1).
Is there a clerical error？ （Overall, the two papers really attract me!!!）

Hi @Wu-Fisher, I think it is not an error.
m_new = (m_old, m_temporary), the m(1) is m_old and m is m_new here.
Correct me if I am wrong. Thanks

@jundaf2 yep >😂 m_new = (m_old, m_temporary), the m(1) is m_old and m is m_new here.
But the O(2) 's equation just uses m to replace m(1) (flashattention2 page4 around O(1) ).
From the flashattention1, O(2) = diag(l1/l2)^(-1)e(m-m1)O(1)....

@jundaf2 I am not sure what the sentence "the first term in the last line of page 2" means. (I didn't find it in both two papers)
I can't find which equation on page 4 shows that O1 has been scaled.
Of course, it is right if O1 has been scaled.

Sorry, I mean page 4. Maybe it's better to let Dao to confirm.

@jundaf2 Thx🙂,  I think it was just a clerical error or a lack of rigor. Equations on Page 6 are right as you can see diag(m(1)-m(2))O1.

My thought of the right equation is:
$$diag({l^{(2)}}/{l^{(1)}})^{-1}·e^{m(1)-m(2)}·O(1)$$

My thought of the right equation should be:
diag(l(2)/l(1))−1·em(1)−m(2)·O(1)
I also think the right format of equation shoud be :

$$ O^{(2)} = \text{diag}\left(\frac{l^{(1)}}{l^{(2)}} \right)  e^{m^{(1)}-m^{(2)}}O^{(1)} + \tilde{P}^{(2)} V^{(2)} $$
as mentioned in https://github.com/Dao-AILab/flash-attention/issues/897#issue-2202994515


docs: fix url in delegated-routing example's README

IPFS-Install-Guide link in README.md cannot accessed because of wrong https cert.
Signed-off-by: InGee Kim<EMAIL_ADDRESS>

Thank you!


Getting invalid scope error

I am following the tutorial posted on https://developer.okta.com/blog/2017/04/17/angular-authentication-with-oidc . I followed all the steps all the way to where we click the Login button and sign-in with one of the people that’s configured in our Okta application. I am getting an error  when I click on "Login" button. It changes the link on address bar to http://localhost:4200/home#state=wgwMkOVYzOCdbMHD6D2QtuMMHjcDhhHQv7Xl9qrp&error=invalid_scope&error_description=One+or+more+scopes+are+not+configured+for+the+authorization+server+resource.
And on console it logs the following :
ERROR Error: Uncaught (in promise): TypeError: Cannot read property 'nonceStateSeparator' of null
TypeError: Cannot read property 'nonceStateSeparator' of null
    at OAuthService.push../node_modules/angular-oauth2-oidc/esm5/angular-oauth2-oidc.js.OAuthService.tryLogin

By looking at the error I thought something might be wrong with how I am setting up oauthService in app.component. But everything looks ok to me over there as well. Below is copy of my app.component ts file.
import { Component } from '@angular/core';
import { OAuthService, JwksValidationHandler } from 'angular-oauth2-oidc';

@Component({
  selector: 'app-root',
  templateUrl: './app.component.html',
  styleUrls: ['./app.component.css']
})
export class AppComponent {
  title = 'app';
  constructor(private oauthService: OAuthService) {
    this.oauthService.redirectUri = window.location.origin;
    this.oauthService.clientId = '0oafim33hmykBg49e0n9';
    this.oauthService.scope =<EMAIL_ADDRESS>    this.oauthService.issuer = 'https://dev-548729.oktapreview.com/oauth2/default';
    this.oauthService.tokenValidationHandler = new JwksValidationHandler();

    // Load Discovery Document and then try to login the user
    this.oauthService.loadDiscoveryDocumentAndTryLogin();
  }

}

Can someone tell me what might be going on ? I am very new to this.

The scope should be the following:
this.oauthService.scope = 'openid profile email';

Can you try changing to that and see if it works for you?

Thanks for the prompt response Matt. After doing your change when I click on Login button I actually get redirected to Okta login page. But when I click sign in and go back into my Angular application I dont see the expected results. It doesn't pull my name back and also Login button  text does not change to Logout as it should according to the tutorial.
I also get the same message on console which I was getting earlier.
core.js:1542 ERROR Error: Uncaught (in promise): TypeError: Cannot read property 'nonceStateSeparator' of null
TypeError: Cannot read property 'nonceStateSeparator' of null
    at OAuthService.push../node_modules/angular-oauth2-oidc/esm5/angular-oauth2-oidc.js.OAuthService.tryLogin (angular-oauth2-oidc.js:1068)
    at angular-oauth2-oidc.js:294

this is how my auth.guard.service ts look like:
import { Injectable } from '@angular/core';
import { ActivatedRouteSnapshot, CanActivate, Router, RouterStateSnapshot } from '@angular/router';
import { OAuthService } from 'angular-oauth2-oidc';

@Injectable({
  providedIn: 'root'
})
export class AuthGuard implements CanActivate {

  constructor(private oauthService: OAuthService, private router: Router) {}

  canActivate(route: ActivatedRouteSnapshot, state: RouterStateSnapshot): boolean {
    if (this.oauthService.hasValidIdToken()) {
      return true;
    }

    this.router.navigate(['/home']);
    return false;
  }
}

And this is how my home component looks like
import { Component } from '@angular/core';
import { OAuthService } from 'angular-oauth2-oidc';

@Component({
  template: `

Welcome, {{givenName}}!

      <button (click)="logout()">Logout</button>

Search

      <button (click)="login()">Login</button>

`
})
export class HomeComponent {

  constructor(private oauthService: OAuthService) {
  }

  login() {
    this.oauthService.initImplicitFlow();
  }

  logout() {
    this.oauthService.logOut();
  }

  get givenName() {
    const claims = this.oauthService.getIdentityClaims();
    if (!claims) {
      return null;
    }
    return claims['name'];
  }
}

What version of angular-oauth2-oidc are you using? I’ve only tested with v3.1.4.

I am using  "angular-oauth2-oidc": "^4.0.2" version. I will try to go down that version and give it another try.

Wow angular-oauth2-oidc version was the issue! After downgrading to 3.1.4 version it worked like we expected. Thanks for all your help!


error 10053

Your os

windows 10 pro version:1803
The version of electron-ssr

v0.2.3
What's the problem

error 10053
App log here

2018-07-27 23:56:11 INFO     local.py:50 local start with protocol[auth_aes128_md5] password [xxxxx] method [chacha20] obfs [tls1.2_ticket_auth] obfs_param [itunes.apple.com/cn/app/464]
2018-07-27 23:56:11 INFO     local.py:54 starting local at <IP_ADDRESS>:1080
2018-07-27 23:56:11 INFO     asyncdns.py:324 dns server: [('<IP_ADDRESS>', 53), ('<IP_ADDRESS>', 53)]
2018-07-27 23:56:11 INFO     util.py:85 loading libcrypto from C:\WINDOWS\system32\libcrypto.dll
2018-07-27 23:56:17 ERROR    shell.py:50 [Errno 10053] An established connection was aborted by the software in your host machine
2018-07-27 23:56:17 ERROR    shell.py:50 [Errno 10053] An established connection was aborted by the software in your host machine
2018-07-27 23:56:17 ERROR    shell.py:50 [Errno 10053] An established connection was aborted by the software in your host machine
2018-07-27 23:56:17 ERROR    shell.py:50 [Errno 10053] An established connection was aborted by the software in your host machine
2018-07-27 23:56:17 ERROR    shell.py:50 [Errno 10053] An established connection was aborted by the software in your host machine
2018-07-27 23:56:17 ERROR    shell.py:50 [Errno 10053] An established connection was aborted by the software in your host machine
2018-07-27 23:56:17 ERROR    shell.py:50 [Errno 10053] An established connection was aborted by the

ssr-python的原因


Update grpc-httpjson-transcoding to 2021-02-23

Signed-off-by: Wayne Zhang<EMAIL_ADDRESS>Commit Message:
Updated to the grpc_httpjson_transcoding with a fix to
https://github.com/grpc-ecosystem/grpc-httpjson-transcoding/issues/49
Risk Level: None
Testing: unit-tested
Docs Changes:  N/A
Release Notes: N/A
Platform Specific Features: N/A

/lgtm deps


add integration tests for capture command

Currently we have integration tests tests/integration.txtar for these command:

analyze
hunt
build

The capture command is not verified, so we should add some commands to ensure its integrity at every run.
We could use the test binary generated by the previous command harpoon analyze and get the syscalls from a single function.

This has been fixed by #58.
I added several integration tests trough testscript.
Please take a look here: https://github.com/alegrey91/harpoon/tree/main/tests


Update docker version to v24.0.5

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area cmd

/area pkg

/area docs

/area tests

What this PR does / why we need it:
This PR fixes a bug due to a breaking change in golang 1.20.6 / 1.19.11 which prevents "driverkit docker" to run successfully.
The build fails with the following error:
DEBU starting container                            image="driverkit-builder:latest"
DEBU context canceled
FATA exiting                                       error="http: invalid Host header"

The respective docker issue is https://github.com/docker/cli/issues/4437
Updating to docker version v24.0.5 causes the build to fail due to a change to the ContainerStop method. This PR changes the code to use the updated semantis.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
No

Fix `driverkit docker` when compiled with golang 1.20.6 / 1.19.11.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: marwinski
Once this PR has been reviewed and has the lgtm label, please assign leodido for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @marwinski! It looks like this is your first PR to falcosecurity/driverkit 🎉

Hi! Thanks for this contribution!
I think this is already part of https://github.com/falcosecurity/driverkit/pull/286, can you take a look?

/hold

LGTM label has been added.  Git tree hash: f41e38d0ee40664db8fe689f8a6678b546c06058

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: EXONER4TED, marwinski
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [EXONER4TED]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Ah, just realized this is not passing tests and needs dco sign off.

New master includes this change! I think this can be closed @marwinski

@FedeDP: Closed this PR.

In response to this:

/close
This is solved by latest master. We are gonna release new driverkit version soonish!

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


feat: handle particles unlit shader and alpha transparent textures

Support exporting the particles unlit shader, and add some hacky way to ensure that alpha transparent textures are still transparent when imported at runtime.
ED-1310

A small thing I've discovered while testing - it doesn't concern this PR directly, but after exporting the Small Pet Racing project, the sun is beyond the camera's far clipping plane, so it's often culled away and doesn't display well. Seems like the camera settings aren't exported to glTF. Can I check if you also encounter the same problem? I'll open a ticket if so.

@ngsp94 oh yeah, I get the same thing but I kept forgetting to either fix it or create a ticket for that 😅

@ngsp94 oh yeah, I get the same thing with the clipping distance on the sun but I kept forgetting to either fix it or create a ticket for that sweat_smile

OK then, please do. I do think sometimes we want different camera settings for a game (eg FPS view from character with specific culling mask), vs looking at a scene, so I'm not too sure how we could resolve that. Could possibly set the Live Preview camera's default culling planes to further distances to prevent clipping in any kind of case.

OK then, please do. I do think sometimes we want different camera settings for a game (eg FPS view from character with specific culling mask), vs looking at a scene, so I'm not too sure how we could resolve that. Could possibly set the Live Preview camera's default clipping planes to further distances to prevent clipping in any kind of case.

Yeah I think setting the default could be a workaround, not sure about the best way to fix this either. I created a ticket at the top of the backlog for now, feel free to add any details to it.


python3.pkgs.deploykit: 1.0.2 -> 1.1.0

Diff: https://github.com/numtide/deploykit/compare/1.0.2...1.1.0
Changelog: https://github.com/numtide/deploykit/releases/tag/1.1.0
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg build python3.pkgs.deploykit


livebook.space gets caught by adblockers

It seems a rule blocking livebook.space has managed to find it's way onto easylist, resulting in both uBlock Origin and Adguard (and perhaps others) blocking iframes from loading over https.
I get that that's not really the responsibility of the livebook devs but I thought you'd want to know.

Thanks @dpatterbee! I installed uBlock Origin here and it seems it still works fine. Do you see this issue happening on your machine? It also seems like adding a www. would be enough to bypass it?

Nevermind, you are right. I guess it was still downloading the list when I tried.


Zeek v4.1 compatibility

Plugin updates for Zeek v4.1.x compatibility

Standardized some of the btest files and regenerated baseline log files so that the output is the same in v4.0.x and v4.1.x.

@sethhall, ping? This looks good to me and still works all the way up to current Zeek master.


contrib: script to send selected text to ad_repl

This script sends text that has been selected (added to dot) to the ad_repl buffer for execution.

Thanks for this @davcam! Would you mind adding a quick comment to the top of the script outlining the intended usage? From the conversations we've had elsewhere I think I understand what it is you are doing with this but it would be nice to have that context in the script for anyone else who is looking through the contrib directory and wondering what is available.

@sminez hope the comment fits the bill

Perfect! Thanks 🙂


Reduce the repository and homepage warnings when not present in package.json

There are use cases where a user doesn't want to publish their theme (such as if private or just testing something). The warnings that are emitted I feel are a tiny bit over-kill (especially when first getting started with the example repo):
22:34:40 [WARN] [my-theme] Add a 'homepage' to your theme's 'package.json'!     Astro uses this value to populate the integrations page https://astro.build/integrations/

        "homepage": "https://github.com/UserName/theme-playground",

22:34:40 [WARN] [my-theme] Add a 'repository' to your theme's 'package.json'!   Astro uses this value to populate the integrations page https://astro.build/integrations/

        "repository": {
            "type": "git",
            "url": "https://github.com/UserName/my-theme",
            "directory": "package"
        },

If the intention is to use this package for preparing a user to submit their theme to the Astro integration page then maybe a validate command could be added? This could check if everything is correct and also give a developer any additional context.

Thanks for opening an issue on this! I agree, these warnings can feel like overkill, especially for this use case. I haven't documented this yet, but I tried to get around this by adding the ability to silence these warning if the package.json has "private": true or if the log option is set to false. What do you think about this? This does not solve the issue with the template warnings being scary, but the template is meant for packages. Maybe we could fill these with properties with default values in the template, so the warnings don't scare anyone.
Context for these warnings: The main reason I want to support these warning is that the integrations page is populated automatically. This means It is possible for user's to publish their package with undefined fields, which makes it harder for people to discover the theme on the integrations page. The goal of these warnings are to make sure that themes authored with ATP follow best practices and are discoverable by the Astro.

Ahh, awesome! I just tried the "private": true trick and that's exactly what I expected. Possibly just adding a sentence to the log output telling the developer they can do the same would fix it I think. And those who read are annoyed my console messages are maybe those who actually read the messages there and would notice that hint if so ;)

Good idea! 📋 I also plan on writing some docs to cover these options/use cases

If anyone is interested in contributing, this is a great first issue!
Here is the code for these warnings: https://github.com/astrolicious/astro-theme-provider/blob/main/package/src/utils/package.ts#L38
If any of these warnings are triggered, we should also warn that they can be silenced with "private": true inside the package.json


Discard Status before publishing in provider

Discard the Status of the Gateway and xRoute resources before saving them in the watchable layer. This reduces the memory consumption within watchable / lesser things to cache. Its safe to discard the status since it is always recomputed by the gateway-api layer, and only written back into the K8s API Server if something, other than the LastTransitionTime has changed.

Fixes: https://github.com/envoyproxy/gateway/issues/1370

/retest


Run android-build-freerdp.sh error on Ubuntu 14.04

Hi, All
when i run ./scripts/android-build-freerdp.sh --ndk <ANDROID_NDK> --sdk <ANDROID_SDK>
i can build openh264.so openssl.so libcrypto.so in jniLibs/armeabi
but when run Build and install the library , will show msgs
[RUN] cd /home/user/WorkPrj/FreeRDP-master
[RUN] mkdir -p ./scripts/../build/freerdp-build/armeabi
[RUN] cd ./scripts/../build/freerdp-build/armeabi
[RUN] export ANDROID_NDK=/home/user/NDK/android-ndk-r11c
[RUN] cmake -DANDROID_NDK=/home/user/NDK/android-ndk-r11c -DANDROID_NATIVE_API_LEVEL=android-12 -DCMAKE_TOOLCHAIN_FILE=./scripts/../cmake/AndroidToolchain.cmake -DCMAKE_INSTALL_PREFIX=./scripts/../client/Android/Studio/freeRDPCore/src/main/jniLibs -DCMAKE_BUILD_TYPE=Debug -DFREERDP_EXTERNAL_PATH=./scripts/../client/Android/Studio/freeRDPCore/src/main/jniLibs -DWITH_OPENH264=ON -DANDROID_ABI=armeabi ./scripts/..
CMake Error: The source directory "/home/user/WorkPrj/FreeRDP-master/build/freerdp-build/armeabi" does not appear to contain CMakeLists.txt.
Specify --help for usage, or press the help button on the CMake GUI.
[ERROR] cmake -DANDROID_NDK=/home/user/NDK/android-ndk-r11c -DANDROID_NATIVE_API_LEVEL=android-12 -DCMAKE_TOOLCHAIN_FILE=./scripts/../cmake/AndroidToolchain.cmake -DCMAKE_INSTALL_PREFIX=./scripts/../client/Android/Studio/freeRDPCore/src/main/jniLibs -DCMAKE_BUILD_TYPE=Debug -DFREERDP_EXTERNAL_PATH=./scripts/../client/Android/Studio/freeRDPCore/src/main/jniLibs -DWITH_OPENH264=ON -DANDROID_ABI=armeabi ./scripts/.. retured 1
so i try FreeRDP-master.zip unzip to "/home/user/WorkPrj/FreeRDP-master/build/freerdp-build/armeabi and run again , will show below msg
[RUN] cd /home/user/WorkPrj/FreeRDP-master
[RUN] mkdir -p ./scripts/../build/freerdp-build/armeabi
[RUN] cd ./scripts/../build/freerdp-build/armeabi
[RUN] export ANDROID_NDK=/home/user/NDK/android-ndk-r11c
[RUN] cmake -DANDROID_NDK=/home/user/NDK/android-ndk-r11c -DANDROID_NATIVE_API_LEVEL=android-12 -DCMAKE_TOOLCHAIN_FILE=./scripts/../cmake/AndroidToolchain.cmake -DCMAKE_INSTALL_PREFIX=./scripts/../client/Android/Studio/freeRDPCore/src/main/jniLibs -DCMAKE_BUILD_TYPE=Debug -DFREERDP_EXTERNAL_PATH=./scripts/../client/Android/Studio/freeRDPCore/src/main/jniLibs -DWITH_OPENH264=ON -DANDROID_ABI=armeabi ./scripts/..
-- Could NOT find PkgConfig (missing:  PKG_CONFIG_EXECUTABLE)
-- FreeRDP GIT
-- Git Revision n/a
-- Performing Test Wno-unused-result
-- Performing Test Wno-unused-result - Success
-- Performing Test Wno-unused-but-set-variable
-- Performing Test Wno-unused-but-set-variable - Success
-- Performing Test Wno-deprecated-declarations
-- Performing Test Wno-deprecated-declarations - Success
-- Performing Test Wno-deprecated-declarationsCXX
-- Performing Test Wno-deprecated-declarationsCXX - Success
-- GCC default symbol visibility: hidden
-- Performing Test Wimplicit-function-declaration
-- Performing Test Wimplicit-function-declaration - Success
-- Performing Test Wredundant-decls
-- Performing Test Wredundant-decls - Success
-- Looking for include file fcntl.h
-- Looking for include file fcntl.h - found
-- Looking for include file unistd.h
-- Looking for include file unistd.h - found
-- Looking for include file execinfo.h
-- Looking for include file execinfo.h - not found
-- Looking for include file inttypes.h
-- Looking for include file inttypes.h - found
-- Looking for include file sys/modem.h
-- Looking for include file sys/modem.h - not found
-- Looking for include file sys/filio.h
-- Looking for include file sys/filio.h - not found
-- Looking for include file sys/sockio.h
-- Looking for include file sys/sockio.h - not found
-- Looking for include file sys/strtio.h
-- Looking for include file sys/strtio.h - not found
-- Looking for include file sys/select.h
-- Looking for include file sys/select.h - found
-- Looking for include file syslog.h
-- Looking for include file syslog.h - found
-- Performing Test HAVE_TM_GMTOFF
-- Performing Test HAVE_TM_GMTOFF - Success
-- Looking for pthread_mutex_timedlock in pthread
-- Looking for pthread_mutex_timedlock in pthread - not found
-- Looking for include file sys/eventfd.h
-- Looking for include file sys/eventfd.h - found
-- Looking for include file sys/eventfd.h
-- Looking for include file sys/eventfd.h - found
-- Looking for eventfd_read
-- Looking for eventfd_read - found
-- Looking for include file sys/timerfd.h
-- Looking for include file sys/timerfd.h - not found
-- Looking for include file poll.h
-- Looking for include file poll.h - found
-- Looking for ceill
-- Looking for ceill - found
-- Skipping disabled feature X11 for X11 (X11 client and server)
-- Skipping disabled feature Wayland for Wayland (Wayland client)
-- Skipping disabled feature DirectFB for DirectFB (DirectFB client)
-- Finding required feature ZLIB for compression (data compression)
-- Found ZLIB: /home/user/NDK/android-ndk-r11c/platforms/android-12/arch-arm/usr/lib/libz.so (found version "1.2.3")
-- Finding required feature OpenSSL for cryptography (encryption, certificate validation, hashing functions)
CMake Error at /usr/share/cmake-2.8/Modules/FindPackageHandleStandardArgs.cmake:108 (message):
Could NOT find OpenSSL, try to set the path to OpenSSL root folder in the
system variable OPENSSL_ROOT_DIR (missing: OPENSSL_LIBRARIES
OPENSSL_INCLUDE_DIR)
Call Stack (most recent call first):
/usr/share/cmake-2.8/Modules/FindPackageHandleStandardArgs.cmake:315 (_FPHSA_FAILURE_MESSAGE)
cmake/FindOpenSSL.cmake:317 (find_package_handle_standard_args)
cmake/FindFeature.cmake:16 (find_package)
CMakeLists.txt:674 (find_feature)
-- Configuring incomplete, errors occurred!
See also "/home/user/WorkPrj/FreeRDP-master/build/freerdp-build/armeabi/CMakeFiles/CMakeOutput.log".
See also "/home/user/WorkPrj/FreeRDP-master/build/freerdp-build/armeabi/CMakeFiles/CMakeError.log".
[ERROR] cmake -DANDROID_NDK=/home/user/NDK/android-ndk-r11c -DANDROID_NATIVE_API_LEVEL=android-12 -DCMAKE_TOOLCHAIN_FILE=./scripts/../cmake/AndroidToolchain.cmake -DCMAKE_INSTALL_PREFIX=./scripts/../client/Android/Studio/freeRDPCore/src/main/jniLibs -DCMAKE_BUILD_TYPE=Debug -DFREERDP_EXTERNAL_PATH=./scripts/../client/Android/Studio/freeRDPCore/src/main/jniLibs -DWITH_OPENH264=ON -DANDROID_ABI=armeabi ./scripts/.. retured 1
could anyone help me point how to finish to build a success freerdp android library , thank a lot

@BrianChangchien Just tested on a fresh 14.04 installation:

You need realpath, cmake and git installed
Just clone git clone https://github.com/freerdp/freerdp
Run ./scripts/android-build-freerdp.sh --ndk <path to your NDK> --sdk <path to your SDK>
Continue with android studio

So best retry with a clean workspace.
If you still have issues, please use the FreeRDP IRC or mailing list for support, as this is obviously not a but.

Hello , akallabeth
Thanks your response
Could you show me how to set realpath, or how to edit a success cmd about
SCRIPT_PATH=$(realpath "$SCRIPT_PATH") plz.......
2016-05-11 16:00 GMT+08:00 akallabeth<EMAIL_ADDRESS>
@BrianChangchien https://github.com/BrianChangchien Just tested on a
fresh 14.04 installation:

You need realpath, cmake and git installed
Just clone git clone https://github.com/freerdp/freerdp
Run ./scripts/android-build-freerdp.sh --ndk
--sdk
Continue with android studio

So best retry with a clean workspace.
If you still have issues, please use the FreeRDP IRC or mailing list for
support, as this is obviously not a but.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/FreeRDP/FreeRDP/issues/3326#issuecomment-218389190

apt-get install realpath

Yes,thx
2016年5月11日 下午8:57於 "akallabeth"<EMAIL_ADDRESS>
apt-get install realpath
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/FreeRDP/FreeRDP/issues/3326#issuecomment-218450841

Hi akallabeth
I can build aFreeRD.apk to install to android phone, thx your great help
but I meet connect failed issue now
I prepare a Win7 PC and open RDP protocal , the Win7 PC can remoted by
another PC
then I use aFreeRDP to  setConnectionInfo() and  connect
it will response event OnConnectionFailure(inst) ,How do i log what problem
to connectfailure , could you tell me how to find problem , thx your great
help !!
2016-05-11 22:20 GMT+08:00 Brian Chang-Chien<EMAIL_ADDRESS>
Yes,thx
2016年5月11日 下午8:57於 "akallabeth"<EMAIL_ADDRESS>
apt-get install realpath
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/FreeRDP/FreeRDP/issues/3326#issuecomment-218450841

@BrianChangchien
i have the same issue with you, however, i apt-install realpath successfully and clone FreeRDP, run script ./scripts/android-build-freerdp.sh --ndk <ANDROID_NDK> --sdk <ANDROID_SDK> ，this issue still happens.
my develop environment configuration
OS: Ubuntu 14.04
Git version: 1.9.1
JDK version: 1.8.0_111
cmake version: <IP_ADDRESS>

the following is the error log
-- Could NOT find PkgConfig (missing:  PKG_CONFIG_EXECUTABLE)
-- FreeRDP GIT
-- Git Revision 2a6dbab
-- Performing Test Wno-unused-result
-- Performing Test Wno-unused-result - Failed
-- Performing Test Wno-unused-but-set-variable
-- Performing Test Wno-unused-but-set-variable - Failed
-- Performing Test Wno-deprecated-declarations
-- Performing Test Wno-deprecated-declarations - Failed
-- Performing Test Wno-deprecated-declarationsCXX
-- Performing Test Wno-deprecated-declarationsCXX - Failed
-- GCC default symbol visibility: hidden
-- Performing Test Wimplicit-function-declaration
-- Performing Test Wimplicit-function-declaration - Failed
-- Performing Test Wredundant-decls
-- Performing Test Wredundant-decls - Failed
-- Looking for include file fcntl.h
-- Looking for include file fcntl.h - not found
-- Looking for include file unistd.h
-- Looking for include file unistd.h - not found
-- Looking for include file execinfo.h
-- Looking for include file execinfo.h - not found
-- Looking for include file inttypes.h
-- Looking for include file inttypes.h - not found
-- Looking for include file sys/modem.h
-- Looking for include file sys/modem.h - not found
-- Looking for include file sys/filio.h
-- Looking for include file sys/filio.h - not found
-- Looking for include file sys/sockio.h
-- Looking for include file sys/sockio.h - not found
-- Looking for include file sys/strtio.h
-- Looking for include file sys/strtio.h - not found
-- Looking for include file sys/select.h
-- Looking for include file sys/select.h - not found
-- Looking for include file syslog.h
-- Looking for include file syslog.h - not found
-- Performing Test HAVE_TM_GMTOFF
-- Performing Test HAVE_TM_GMTOFF - Failed
-- Looking for include file pthread.h
-- Looking for include file pthread.h - not found
CMake Error at /usr/share/cmake-2.8/Modules/FindPackageHandleStandardArgs.cmake:108 (message):
  Could NOT find Threads (missing: Threads_FOUND)
Call Stack (most recent call first):
  /usr/share/cmake-2.8/Modules/FindPackageHandleStandardArgs.cmake:315 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-2.8/Modules/FindThreads.cmake:166 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:533 (find_package)

-- Configuring incomplete, errors occurred!
See also "/home/parallels/Android/workspace/FreeRDP/build/freerdp-build/armeabi/CMakeFiles/CMakeOutput.log".
See also "/home/parallels/Android/workspace/FreeRDP/build/freerdp-build/armeabi/CMakeFiles/CMakeError.log".
[ERROR] cmake -DANDROID_NDK=/home/parallels/Android/android-ndk-r13 -DANDROID_NATIVE_API_LEVEL=android-21 -DCMAKE_TOOLCHAIN_FILE=/home/parallels/Android/workspace/FreeRDP/scripts/../cmake/AndroidToolchain.cmake -DCMAKE_INSTALL_PREFIX=/home/parallels/Android/workspace/FreeRDP/scripts/../client/Android/Studio/freeRDPCore/src/main/jniLibs -DCMAKE_BUILD_TYPE=Debug -DFREERDP_EXTERNAL_PATH=/home/parallels/Android/workspace/FreeRDP/scripts/../client/Android/Studio/freeRDPCore/src/main/jniLibs -DANDROID_ABI=armeabi -DCMAKE_INSTALL_PREFIX=/home/parallels/Android/workspace/FreeRDP/scripts/../client/Android/Studio/freeRDPCore/src/main/jniLibs -DCMAKE_INSTALL_LIBDIR=armeabi /home/parallels/Android/workspace/FreeRDP/scripts/.. retured 1

could you tell me what reason is?

I'm also having this problem on Ubuntu:
-- Looking for include file pthread.h
-- Looking for include file pthread.h - not found
CMake Error at /usr/share/cmake-2.8/Modules/FindPackageHandleStandardArgs.cmake:108 (message):
Could NOT find Threads (missing: Threads_FOUND)
Can't seem to figure out what the problem is.

I get a similar error on  Ubuntu and I'm not able to build:
-- Could NOT find PkgConfig (missing:  PKG_CONFIG_EXECUTABLE)
-- FreeRDP GIT
-- Git Revision 05f6dac
-- GCC default symbol visibility: hidden
CMake Error at /usr/share/cmake-3.2/Modules/FindPackageHandleStandardArgs.cmake:138 (message):
Could NOT find Threads (missing: Threads_FOUND)
Call Stack (most recent call first):
/usr/share/cmake-3.2/Modules/FindPackageHandleStandardArgs.cmake:374 (_FPHSA_FAILURE_MESSAGE)
/usr/share/cmake-3.2/Modules/FindThreads.cmake:204 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
CMakeLists.txt:540 (find_package)


added a new technical question

Please check if this PR fulfills these

[x] The commit message is descriptive of changes
[x] You have tried at least one local test (this includes viewing the changes on your local fork)
[x] Docs have been added / updated as needed (for bug fixes / features)

What kind of change does this PR introduce?

[ ] Typo / Doc Update
[ ] Bug Fix
[x] Feature / New Item

What is the new behavior (if this is a feature change)?

[x] Added Interview Technical Question
[ ] Added Interview Whiteboard Question
[ ] Corrected existing items
[ ] Other

Other information:

Added a new and interesting question as I thought that might have good chance of being asked in a interview

added another question

added another question

Looks good!


 It seems like it didn't. We didn't modify the workspace creation code. The HEAD request we use to check it looks to be just broken, because HEAD returns a 200 on nonexistent workspaces:
16:35:54-0500 yagody $ http localhost:8001/workspaces/noexist
HTTP/1.1 404 Not Found

{
    "message": "Not found"
}

16:35:59-0500 yagody $ http HEAD localhost:8001/workspaces/noexist
HTTP/1.1 200 OK

This is... questionable behavior from the admin API, though it may not technically be a violation of RFC 7231, which is moot on whether a HEAD MUST return the same status as a GET (in fact it says nothing about status at all, and the closest thing it has is comment on the headers which only SHOULD match those of a GET).
So the addition of client.Workspaces.Exists() to go-kong (which relies on the generic client.exists(), which always uses HEAD) never satisfied its intended purpose. I don't think we had the old version of this code in 2.x, but https://github.com/Kong/kubernetes-ingress-controller/blame/next/cli/ingress-controller/util.go#L36 broke 1.x's ensure workspace feature.
IMO we file a bug upstream with Kong Enterprise asking them to make HEAD /workspaces/<workspace> return a 404 if a GET for the same would and ignore backwards compatibility with broken Enterprise versions. Ensuring workspaces isn't critically important IMO, and if you want it, you can upgrade your Kong version.
Approved on the basis that this PR does fix what it should. The stuff that's still broken was always broken and isn't in scope #1353. Being able to access non-default workspaces (but not being able to create them) is more useful than not being able to access non-default workspaces (and still not being able to create them).

However, the CLA check seems to be stuck, so I cannot merge this :(

However, the CLA check seems to be stuck, so I cannot merge this :(

Did not see an error in the CI ? Do I miss anything ?

However, the CLA check seems to be stuck, so I cannot merge this :(

Did not see an error in the CI ? Do I miss anything ?

It seems the licensing check was successful after an update of next... we're going to need to do something about that CI task eventually... :thinking:


Search on any criteria and sort by 'Recently added' or 'Recently updated' and they both produce the same order of results

Tested on staging: https://s112p01-find-npd-data-s112p01-staging.azurewebsites.net/categories
Browser: Chrome
OS: Windows 10
Steps to reproduce

On the 'Home' page input 'Age' into the search field
Select 'Recently added' in the 'Sort By' drop down
You will be presented with the results that are in recently added order
Now select 'Recently Updated' in the 'Sort By' drop down
Notice that the order has not changed and are still displayed in the same order. I have tested this with multiple search results and both options produce the same results

Another question is: ''Recently added'' is the default return - is this actually it, or is it relevance (relevance as decided by our search rules?).


DRILL-8485: HashJoinPOP memory leak is caused by an oom exception whe…

…n read data from InputStream
DRILL-8485: HashJoinPOP memory leak is caused by an oom exception when read data from InputStream
Description
it is similar to DRILL-8484
exception info
Caused by: org.apache.drill.exec.exception.OutOfMemoryException: Unable to allocate buffer of size 16384 (rounded from 15364) due to memory limit (41943040). Current allocation: 4337664
        at org.apache.drill.exec.memory.BaseAllocator.buffer(BaseAllocator.java:241)
        at org.apache.drill.exec.memory.BaseAllocator.buffer(BaseAllocator.java:216)
        at org.apache.drill.exec.memory.BaseAllocator.read(BaseAllocator.java:856)

leak  info
  Allocator(frag:5:1) 5000000/1000000/27824128/40041943040 (res/actual/peak/limit)
      child allocators: 1
        Allocator(op:5:1:1:HashJoinPOP) 1000000/16384/22822912/41943040 (res/actual/peak/limit)
          child allocators: 0
          ledgers: 2
            ledger[442780] allocator: op:5:1:1:HashJoinPOP), isOwning: true, size: 8192, references: 2, life:<PHONE_NUMBER>491..0, allocatorManager: [390894, life:<PHONE_NUMBER>180..0] holds 4 buffers.
                DrillBuf[458469], udle: [390895 1024..8192]
         event log for: DrillBuf[458469]

Documentation
(Please describe user-visible changes similar to what should appear in the Drill documentation.)
Testing
The testing method for drill-8485 is the similar as for DRILL-8484. we can throw exception  in the method readVectors

LGTM +1 Thanks @shfshihuafeng for all these memory leak fixes.

I am honored to get your approved.


Reverting Fix/1177 full reset rsh3g2a only

Due to increasing errors reported from clients after an update came from this branch I'm reverting commits here.
Internal discussion - https://ably-real-time.slack.com/archives/C0178QGC093/p1642761003017000

FYI, @paddybyers

Going into ably-flutter version 1.2.11 via https://github.com/ably/ably-flutter/pull/305


Error about satacprocess function in SCATE

Hi,
I followed the R codes in the user manual PDF file. I got the error about the parameter 'libsizefilter' in the function satacprocess. I don't whether to keep it or not, since both situations led to errors. I have already installed the latest version of SCATE. Do you know how to fix it?

Best,
Xi

Hi Xi,
Sorry, there is a typo in this function. I have fixed the typo. Please update the package and try it out again. Thanks!


Refactor Scheduler

This change makes the Scheduler adopt a more object-oriented style.  The
different node types get their own classes and inherit from a base
class.  That base class requires implementation of a couple interfaces
allowing for a consistent pattern of evaluating and accepting/rejecting
offers.
A significant amount of logic is removed from the Scheduler class itself
and placed in the Node class hierarchy.  Additional refactoring work is
still needed espcially in the HdfsNode class, but this change produces
a significant restructure of the Scheduler.
It's a large change, so as a suggestion one might want to approach the code like this.

Look at resourceOffers in the HdfsScheduler.  You'll see the creation of the different node types as well as their launching through the nodeLauncher object.

Look at the nodeLauncher to see the consistent pattern of offer evaluation and accepting/rejecting offers.

Look at the JournalNode, NameNode, DataNode, and HdfsNode classes.  The majority of the logic which was in the HdfsScheduler class is now here.  In particular each of the concrete (non-abstract) classes implements the two required interfaces for the offer evaluation and node launching behavior.  The tryToLaunchXNode functions have had their logic placed in relevant XNode classes.   There is still some logic in the HdfsNode base class which shouldn't be there.  Sometimes it refers to a particular kind of node, and this logic should be moved to the specific Node classes in future refactors.

Testing: As usual I tested on DCOS and Mesos clusters.  Automated tests on DCOS all passed.  Killing all types of Nodes in conjunction with the Scheduler and singly returned to a healthy state.

Great clean up! I think we can mark one of Ken's clean up tasks marked when this one is done. Will review this ASAP and @Kensipe can take a look as well if he'd like

Moved PR to https://github.com/mesosphere/hdfs/pull/195


LOG4J2-2032 Curly braces in parameters should not be treated as placeholders

https://issues.apache.org/jira/browse/LOG4J2-2032
When logging event parameter contains two consecutive opening and
closing curly braces - for example {} in the middle of parameter,
they should not be treated as a placeholder.

Travis build fails with
$ ./mvnw --toolchains=./.travis-toolchains.xml install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
Error: JAVA_HOME is not defined correctly.
  We cannot execute /usr/lib/jvm/java-7-oracle/bin/java
The command "./mvnw --toolchains=./.travis-toolchains.xml install -DskipTests=true -Dmaven.javadoc.skip=true -B -V" failed and exited with 1 during .

Definitely not related to changes I did.

Thank you for your contribution @kshchepanovskyi! I think this is fixed by a very similar commit for a slightly different permutation of this issue here.
I've committed your test and fix for windows.

Thanks!