Added support for sorting

Adds support for sorting on top-level properties of IQueryable.
Supported in this PR:
public class Foo {
     public string Role {get;set;}
     public int Bars {get;set;}
}

{
  foos(order_by: { role: ASC,  bars: DESC}}) {
    role
    bars
  }
}

Not supported in this PR:
public class Foo {
     public string Role {get;set;}
     public Bar Bars {get;set;}
}

public class Bar {
     public string Baz {get;set;}
     public int Blub {get;set;}
}

{
  foos(order_by: { role: ASC,  bar: {baz: DESC}}) {
    bar {
      baz
    }
  }
}

Addresses #923

@gmiserez  I will go through the code more thoroughly tonight ... but overall it looks very good!


[Event Request] Table 5093 - OnFinishWizardAfterInsertEntry

Hi, we need event in function FinishWizard in table 5093
in Following place:
    procedure FinishWizard()
    var
        OppEntry: Record "Opportunity Entry";
    begin
        UpdateEstimates;
        OppEntry := Rec;
        InsertEntry(OppEntry, "Cancel Old To Do", false);
New->   OnFinishWizardAfterInsertEntry(OppEntry);
        Delete;
    end;

Publisher will be:
LOCAL [IntegrationEvent] OnFinishWizardAfterInsertEntry(OpportunityEntry : Record "Opportunity Entry")

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


ETH-like utils generator

Here we introduce a generator of common ETH-like utils which need to have their own representation in chain-specific packages. That generator should help to avoid code duplication and will make future maintenance easier. Specifically, those tools are now managed by the generator:

resubscribe.go
 error_resolver.go
logging_wrapper.go
rate_limiter.go

Let's use the gen directory layout for the generated code. This layout is even mentioned in the directory structure section of keep-common README.

Let's use the gen directory layout for the generated code. This layout is even mentioned in the directory structure section of keep-common README.

Done in https://github.com/keep-network/keep-common/commit/7597045701348109e0b84a2dbd59545666c9ba99

In another galaxy, 8 months later...
We see that Celo and Ethereum are diverging. One example is https://github.com/keep-network/keep-common/pull/84 where Ethereum changed the transaction pricing mechanism and this was not propagated to Celo. Also, Celo plans to add a gateway fee to transaction parameters that will not be available in Ethereum.
I think holding off with the work on this PR was a good choice and I vote for closing it without merging. What do you say @lukasz-zimnoch?

In another galaxy, 8 months later...
We see that Celo and Ethereum are diverging. One example is #84 where Ethereum changed the transaction pricing mechanism and this was not propagated to Celo. Also, Celo plans to add a gateway fee to transaction parameters that will not be available in Ethereum.
I think holding off with the work on this PR was a good choice and I vote for closing it without merging. What do you say @lukasz-zimnoch?

Yep, I agree!


Adds details for episodes 315, 316, and 317

Hey @xandrade here are the details for three more headed backwards in time. :)

Amazing, great contrition @mikeckennedy

Thanks @xandrade


gui events don't propagate to parent control nodes

Godot version: 3.0.2
Issue description:
From the docs, regarding event propagation:
"Second, it will try to feed the input to the GUI, and see if any control can receive it. If so, the Control will be called via the virtual function Control._gui_input() and the signal “input_event” will be emitted (this function is re-implementable by script by inheriting from it). If the control wants to “consume” the event, it will call Control.accept_event() and the event will not spread any more. Events that are not consumed will propagate up, to Control’s ancestors."
from http://docs.godotengine.org/en/3.0/tutorials/inputs/inputevent.html?highlight=input events
I interpreted this to mean that if a Control subclass node does not handle the event, the parent of that node would then be able to accept that event through it's own _gui_input method.  However, that doesn't seem to be the case.  Only the control that has focus will get the _gui_input call.
Am I misunderstanding what that line means?

Are you sure the Mouse/Filter property on the children controls are not set to Stop?

That was the issue.  This was confusing because MouseFilter's name and description both imply that it's only for Mouse button events, when I was trying to have a keyboard event propagated.  But turning MouseFilter to Pass did fix the issue.

Hmm yeah. That is either bad naming or a bug depending on the original intention.

I've tried replicating the issue. I couldn't get the event to go to the parent even if I set the MouseFilter to Pass
Here is example project:
keyboardMouseStop.zip
Not sure if it is intended behavior or not.

For some reason I couldn't get your project to open, but I must have been mistaken earlier and must have been printing a mouse event; I can't get it to propagate keyboard events either.
It still seems like a bug or bad documentation.  If it's just bad documentation I would want to have a similar "KeyboardFilter" feature so there is an easy way of intercepting keyboard inputs.

I guess it is because my project file was created with a master branch (3.1) build and you are using a 3.0x build

I think I've found this problem in the current stable 3.2.2.
From my understanding of the docs, if I set a Control mouse filter to pass then I need to explicitly call accept_event in _gui_input or the event will propagate to its parent, but this is not happening in my project.
Is my understanding wrong or is there actually a bug in input propagation?

This is  also true for Godot 3.3 stable.
See this test game:
testgame.zip
This is the order of the nodes:

(They're all stacked over another and consuming the whole scene)
I start the scene and press the mouse button and this is the only output:

b received input

Using _input instead of _gui_input works as expected:

b received input
ab received input
aab received input
aaa received input
aa received input
a received input

I have the same issue in 3.4.4 – children set to IGNORE, parent control doesn't receive _gui_input.

I believe that this behavior still persist in v4.1.1

I also have this issue, I am using the following code to check if there is an action outside my control to then close a menu. I set "has_point" to be global so I can receive mouse event from outside my component to check if there is a click elsewhere.
func _gui_input(event):
	var is_correct_event = event is InputEventScreenTouch || event is InputEventScreenDrag || event is InputEventMouseMotion
	if "position" in event && Rect2(Vector2(0,0), size).has_point(event.position):
		if is_correct_event:

			close_timer.stop()
			show_quick_inventory("open")

			var scrollbar = $PanelContainer/ScrollContainer.get_h_scroll_bar()
			scrollbar.value += event.relative.x
	else:
		if is_correct_event:
			close_timer.start(0.1)

func _has_point(vec: Vector2):
	return true

I am surprised it hasn't more comments as I guess detecting if a touch event is outside a control is quite common ? (for example to close a hand of cards in a card game, or a popup)


Application Insights: Add Analytics Items API

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-node

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

@ravbhatnagar new Swagger

@yoramsinger - Are these existing APIs in the service that you are adding to swagger now?

Yes. Existing API.

Signing off from ARM since there are APIs in prod for a while

Automation for azure-libraries-for-java

I updated per your review. Please verify, thanks.

@yoramsinger please address the outstanding issues; this PR will otherwise be closed due to inactivity (but can reopened at any time, once you are ready to move forward)

I think I fixed the mentioned issues. Hope it works this time.

@yoramsinger Tiny bug: ItemTypeParameter still has a PascalCased default value of None 🙂

@AutorestCI regenerate azure-sdk-for-go


Computing Relative Wealth Index Zonal Statistics

What does this PR do?
This PR uses RWI data spanning from 2016 to 2022 and computes the zonal statistics (using geowrangler) of Lacuna Cities (by barangay). Selected statistics are:

Mean
Median
Min
Max
Standard Deviation

Where should the reviewers start?
01-compute-RWI-zonal-statistics.ipynb - Notebook used to compute the zonal statistics with RWI data from 2016 to 2022 as the input data and Lacuna City Barangay Administrative Boundary Data as Zonal Data.
Screenshots

thanks for the contribution @tm-tan-zablan. aside from the line comments, to make it consistent with the other repo edits, kindly remove the folder rwi and put this directly under notebooks\tm.


Webhook is valid but Pogramm says its not

i did a test to the weebhook from different websites but on your pogramm its not valid

im getting a syntax error when trying the stealer on my self saying invalid token


Distribute config xml outside of framework/hadoop binary

From user JohnO, "The myriad-config-default.yml needs to be moved outside the bundled jar so we can update our clusters without rebuilding."

It maybe related to #54.  Moreover, In the comment of the issue, klucar proposed that myriad would be able to read configuration from more flexible data sources than config file only.


Add run build action

With https://github.com/cfpb/consumerfinance.gov/pull/7732 and https://github.com/cfpb/consumerfinance.gov/pull/7735
Behavior:

pulls in the frontend build artifact and unzips it
uses the centos image produced by the builder with the repo files mapped in a volume
runs create.py inside this container via _build.sh
pushes the results to S3.

Superseded by https://github.com/cfpb/consumerfinance.gov/pull/7960


move top level functions as method on the Client struct

Is your feature request related to a problem? Please describe.
The way the main-level functions are organized prevents the reuse of an HTTP client and thus forces the creation of a new TCP connection for every function call.
Here is the list of the functions I talking about:
func GetInfo
func GetVersion
func GetDependencies
func GetProject
func GetAdvisory
func GetQuery

Mainly all the exported functions in this file https://github.com/edoardottt/depsdev/blob/main/pkg/depsdev/api.go
Describe the solution you'd like
These functions should be converted to methods on a struct that encapsulates the Client struct
This way, the HTTP client would be kept between requests, which in turn would allow to re-use the TCP connection.
Additional context
I've run a very simple benchmark to highlight the potential gain of this change.
I've benchmarked the GetInfo and GetVersion API calls.
Here are the results:
goos: darwin
goarch: arm64
pkg: github.com/edoardottt/depsdev/pkg/depsdev
             │ functions.perf │             methods.perf             │
             │     sec/op     │    sec/op     vs base                │
GetInfo-8       143.66m ±  6%   78.47m ± 15%  -45.38% (p=0.000 n=10)
GetVersion-8    119.86m ± 16%   53.91m ±  6%  -55.02% (p=0.000 n=10)
geomean          131.2m         65.04m        -50.43%

The code of the benchmark can be found at https://github.com/zaibon/depsdev/blob/reuse-http-client/pkg/depsdev/api_test.go

Amazing @zaibon !
Cool suggestion. Waiting for the PR :)

Great, I'll start to work on that then 👍

Done! #31


net/http: server.ListenAndServe should honour SetKeepAlivesEnabled

Calling http.ListenAndServe will do a net.Listen(...) and then wrap that TCPListener in a tcpKeepAliveListener before calling Serve to handle any incoming connections.
This is clear from the docs on ListenAndServe, which states that:

Accepted connections are configured to enable TCP keep-alives.

But, http.Server also provides the handy SetKeepAlivesEnabled method. So, it would seem to me that calling SetKeepAlivesEnabled(false) and then ListenAndServe() would do the obvious thing and disable keep-alives on that server's connections.
... which it seems to do — sending Connection: close, and likely closing it off after writing the response — however...
As far as I can tell the tcpKeepAliveListener will still always call SetKeepAlive(true) and SetKeepAlivePeriod(3 * time.Minute) on each new connection, triggering a bunch of syscall.SetsockoptInt(fd.sysfd, syscall.SOL_SOCKET, syscall.SO_KEEPALIVE, boolint(keepalive)) magic each time.
Which can be avoided, especially if I'm disabling keep-alives before I even start serving my traffic.
So at the very least, this could be as simple as something like:
diff --git a/src/net/http/server.go b/src/net/http/server.go
index 89574a8..dd21955 100644
--- a/src/net/http/server.go
+++ b/src/net/http/server.go
@@ -2216,7 +2216,10 @@ func (srv *Server) ListenAndServe() error {
 	if err != nil {
 		return err
 	}
-	return srv.Serve(tcpKeepAliveListener{ln.(*net.TCPListener)})
+	if srv.doKeepAlives() {
+		return srv.Serve(tcpKeepAliveListener{ln.(*net.TCPListener)})
+	}
+	return srv.Serve(ln)
 }

 var testHookServerServe func(*Server, net.Listener) // used if non-nil

And doing the same on ListenAndServeTLS.
I haven't looked how feasible it would be to have the tcpKeepAliveListener itself stop setting socket keep-alives once the server setting is changed, but I guess that would be even better?
Am I missing something? Let me know what you think.
What version of Go are you using (go version)?
go version go1.7.4 darwin/amd64
What operating system and processor architecture are you using (go env)?
GOHOSTARCH="amd64"
GOHOSTOS="darwin"

What did you do?
package main

import "net/http"

func main() {
	s := &http.Server{Addr: ":8080"}
	s.SetKeepAlivesEnabled(false)
	s.ListenAndServe()
}

/cc @bradfitz

You're conflating two different types of keep-alives. The docs distinguish between the two by saying "TCP keep-alives" vs "HTTP keep-alives".
The method you cite is about TCP keep-alives:

SetKeepAlivesEnabled controls whether HTTP keep-alives are enabled.

It has no control over TCP keep-alives as designed.
You can pass in your own TCP listener if you want to modify the default ListenAndServe behavior.

Merry Chrismas! 🎄
Ah okay, I understand. Is there any benefit to enabling TCP keep-alives when we know beforehand that no HTTP keep-alives will be performed?

Thanks for your issue.
I'm using the default ListenAndServe function, and it will prints like "http: Accept error: accept tcp [::]:8077: accept4: too many open files; retrying in 1s". I checked the code after, I'm thinking if the error is because of the "tc.SetKeepAlivePeriod(3 * time.Minute)".


Chef Infra Server search(solr) uses supported/secure software(es), resolving open CVEs

Replaces solr with Elasticsearch to remove unsupported solr package that contains security vulnerabilities.
Operationally brings Chef Infra Server users toward migrating to the Automate Infra API (automate uses elasticsearch)
Aha! Link: https://chef.aha.io/epics/CHEF-E-2

Done!


Fix handling of Map.ZoomLevel when animating changes

The behavior of how map animation works changed with the SDK 26 update. Technically the test was correct because with minSdk 7 and no targetSdk, Robolectric would simulate SDK 7 for testing. Now, it simulates SDK 26. Inside osmdroid, the animation behavior changes at SDK 11 (Honeycomb), and this breaks the test. This update fixes the behavior of how we report the ZoomLevel from the map so that it always returns the ZoomLevel for after the animation.
Change-Id: If652e6759e95e47884a1324f8dfd203d8b15e65a

@jisqyv @SusanRatiLane @halatmit Note that I'm submitting this against master even though it affects ucr. Ideally we should merge this next while master and ucr are in sync so that it fixes the failing test before we start merging other changes.

This needs more detail on the bug and how to reproduce it.

There isn't really a bug here per-se. The problem is that the method to get the current zoom level of the map changes behavior from pre Honeycomb to post Honeycomb. Prior to Honeycomb, it always reported the final zoom level during an animation, whereas post Honeycomb it reports the zoom level prior to the transition. This breaks the unit test, which was written on the former assumption. Thankfully, the get zoom method takes a parameter that allows us to always get the target zoom level during an animation, so this change does that and fixes the unit tests.


Add brackets to fix parse errors on some runtimes

This might look like a pointless thing BUT not having this in the generated
bundle can cause parse errors on certain javascript runtimes.
Using nashorn for instance, we get
the following error:
jdk.nashorn.internal.runtime.ParserException:
#7:16:9:10 Expected ; but found function
/******/ 	function hotDownloadUpdateChunk(chunkId) { // eslint-disable-line ...
^
I'm afraid than writing a test case would be difficult since we have to use a JVM to reproduce.
However I can provide a repro if you guys are interested.
Since it is such a small change and I'm obviously not breaking anything, I hope you will trust me on this!

Sorry about this pull request.
I just realised that webpack hot reloading was relying heavily on the browser to work.
Silly me. There is no point running it in nashorn.

Sorry about commenting such an old issue, but we are actually in need of that change.  Can you explain it further? Why shouldn't it be implemented?


feat: make api as module

Description
Changes proposed in this pull request:

make api reusable as module from other projects

Related issue(s)

https://github.com/kyma-project/template-operator/issues/102

/test pull-build-template-operator


Disable Azure support for BTP

Description
Changes proposed in this pull request:

Disable Azure support for BTP

Related issue(s)

#66

/hold

/unhold


Hey,
Check out the quick start guide located:
https://games-on-whales.github.io/wolf/stable/user/quickstart.html
There are multiple example and also additional info regarding environment variables


can not full screen!!

中文用户请注意：请尽量用英文描述你的 issue，这样能够让尽可能多的人帮到你。
If you want to report a bug, please provide the following information:

The steps to reproduce.
A minimal demo of the problem via https://jsfiddle.net or http://codepen.io/pen if possible.
Which versions of DPlayer, and which browser / OS are affected by this issue?

Get out

garbage

@satakwwl 考你个英语阅读理解
If you want to report a bug, please provide the following information:
The steps to reproduce.
A minimal demo of the problem via https://jsfiddle.net or http://codepen.io/pen if possible.
Which versions of DPlayer, and which browser / OS are affected by this issue?
阅读以上文字并且回答
1.为什么过气网红很生气
2.为什么issue得不到解答

undirectlookable /_\


Compare with bpmnmatrix.github.io

https://bpmnmatrix.github.io/ looks pretty neat.
And the code very simple --> https://github.com/bpmnmatrix/bpmnmatrix.github.io
What can we use from there for our framework? Maybe some HTML and CSS magic?

Won't be included because this style can only be applied if we know exactly how many columns we will have but cannot be used for a more general solution with an arbitrary amount of columns.


benches: remove / fix dependency on rusty-loader binary

Fix the following line (we don't ship a binary of the loader anymore):
https://github.com/hermit-os/uhyve/blob/b32272418bdc50c6c9d8b745a9cc3c19723dec9b/benches/complete_binary/mod.rs#L65-L74
See: https://github.com/hermit-os/uhyve/commit/b1b5594e9a6310097fbebf4864f53d1960cbc6fb
Originally posted by @n0toose in https://github.com/hermit-os/uhyve/issues/797#issuecomment-2489002263

Seems like the loader was deleted and the dependency on hermit-loader-x86_64 was introduced in that commit. This seems accidental.
Ping @jounathaen and @mkroening.

Removing the loader was not an accident, but I did not notice these benches (or did not care at the time). I think we should change the path to $CARGO_MANIFEST_DIR/hermit-loader-x86_64 and have the user of these benchmarks place a loader binary there, which is a one-liner btw: gh release download --repo hermit-os/loader --pattern hermit-loader-x86_64.

Why is the loader in uhyve necessary anyway?

Are you sure this is a good idea? I can think of the following tasks that doing this would implicitly entail in the long run:

We should specifically add that file to .gitignore.
Optimally, we would let the user supply a custom loader using a CLI parameter for debugging and (offline) development reasons.

Given for how long this issue went unnoticed, the solutions to that problem might not be worth the effort - but it would make sense for longer-term maintainability and if the benchmarks are to be used more seriously.

It is a benchmark in the end and not a user application.

That's fair enough, I'll try to work on it - I'll just try to look for the hermit loader and only require the gh CLI if hermit-loader-x86_64 is missing - so as to let the user/developer just manually fetch the file instead, and not download the same file repeatedly. If the gh CLI is also missing, I'll mention both options.
This might cause problems if the hermit-loader-x86_64 is too old, but, then again, the gh release download would only work if --clobber is included, too. We could make the solution more elegant by comparing our binary with the release's, but that's for later.

After a quick discussion with Jonathan, I shared an approach without gh download so as to speed things up: https://github.com/hermit-os/uhyve/pull/797
I'll open a new issue and consider revisiting that feature after the current PR is merged.


Support Context Pinecone

https://github.com/tmc/langchaingo/blob/d44ea4436ae92fc5b957e4b6fac8be0b5fd5f1ba/vectorstores/pinecone/pinecone.go#L159
..\..\go\pkg\mod\github.com\tmc\langchaingo@v0.1.11\vectorstores\pinecone\pinecone.go:159:3: cannot use &ctx (value of type *context.Context) as context.Context value in argument to indexConn.QueryByVectorValues: *context.Context does not implement context.Context (type *context.Context is pointer to interface, not interface)

how I can reproduce it?


Incorrect set of runtime-endpoint parameters in kube-up for test clusters

What happened:
I checked on one of env YingH used in her tests, from the runtime’s perspective, this is not right. There could be other potential parameter issues so we might need to take a further look.
From runtime’s perspective, this is due to we take the same parameter from the Admin cluster which suppose only setup the cluster to orchestrate the performance clusters. Hence the docker shim is used.
For our kube-up, its goal is to set up a real Arktos cluster for uses, so the Kubelet commandline parameters should match with or Arktos-up ( or at least the Kubemark cluster’s settings ).
@Sonya Li,  most likely an issue in our kube-up tool and we should consider fix.
ybai2016@ying-scaleout-rp-1-minion-group-fxmh:~$
ybai2016@ying-scaleout-rp-1-minion-group-fxmh:~$ ps -ef | grep kubelet
root        6733       1  2 17:10 ?        00:00:09 /home/kubernetes/bin/kubelet --v=4 --cloud-provider=gce --max-pods=110 --experimental-mounter-path=/home/kubernetes/containerized_mounter/mounter --experimental-check-node-capabilities-before-mount=true --cert-dir=/var/lib/kubelet/pki/ --dynamic-config-dir=/var/lib/kubelet/dynamic-config --bootstrap-kubeconfig=/var/lib/kubelet/bootstrap-kubeconfig --tenant-server-kubeconfig=/etc/srv/kubernetes/tp-kubeconfigs/tp-1-kubeconfig,/etc/srv/kubernetes/tp-kubeconfigs/tp-2-kubeconfig --kubeconfig=/var/lib/kubelet/kubeconfig --cni-bin-dir=/opt/cni/bin --network-plugin=cni --non-masquerade-cidr=<IP_ADDRESS>/0 --volume-plugin-dir=/home/kubernetes/flexvolume --container-runtime-endpoint=unix:///var/run/dockershim.sock --config /home/kubernetes/kubelet-config.yaml --resolv-conf=/run/systemd/resolve/resolv.conf
root        6866       1  0 17:10 ?        00:00:00 bash /home/kubernetes/bin/health-monitor.sh kubelet
ybai2016   10869   10755  0 17:17 pts/0    00:00:00 grep --color=auto kubelet
ybai2016@ying-scaleout-rp-1-minion-group-fxmh:~$

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Arktos version (use kubectl version):
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Network plugin and version (if this is a network-related bug):
Others:

if you are talking --container-runtime-endpoint=unix:///var/run/dockershim.sock, yes, for mizar, we are using docker as runtime.
For all default kube-up, we are using containerd as default runtime and kubelet config is like below:
# ps -ef | grep kubelet
root        2583       1  3 02:15 ?        00:00:30 /home/kubernetes/bin/kubelet --v=2 --runtime-cgroups=/system.slice/containerd.service --cloud-provider=gce --max-pods=110 --experimental-mounter-path=/home/kubernetes/containerized_mounter/mounter --experimental-check-node-capabilities-before-mount=true --cert-dir=/var/lib/kubelet/pki/ --dynamic-config-dir=/var/lib/kubelet/dynamic-config --register-with-taints=node-role.kubernetes.io/master=:NoSchedule --kubeconfig=/var/lib/kubelet/bootstrap-kubeconfig --register-schedulable=false --cni-bin-dir=/home/kubernetes/bin --network-plugin=cni --non-masquerade-cidr=<IP_ADDRESS>/0 --volume-plugin-dir=/home/kubernetes/flexvolume --container-runtime=remote --container-runtime-endpoint=unix:///run/containerd/containerd.sock --config /home/kubernetes/kubelet-config.yaml --resolv-conf=/run/systemd/resolve/resolv.conf

the container-runtime-endpoint should be
--container-runtime=remote --container-runtime-endpoint=containerRuntime,container,/run/containerd/containerd.sock;vmRuntime,vm,/run/virtlet.sock

the kube-up script in 130, we should be able to start scaleout cluster for user to try Arktos scaleout architecture, the VM should be supported too, also containerD is what Arktos claimed container runtime which we should use.
the arktos-up.sh start Mizar successfully so funcationality wise, Mizar should be fine with contaienrD. otherwise, arktos-up.sh will fail or not be able to orchestrate test containers with Mizar.

Change title to reflect the concern of this setting. looks like we have two different settings in testing:

locally with Arktos-up.sh with mizar,  it uses containers as runtime
with the kube-up tool to setup scaleout clusters, it is hardcoded dockershim as container runtime.

which means all the test scenarios with containerd as runtime which Arktos has claimed as default since Arktos project started.
since we cannot control user which one to use -- nowadays, most will be OOF dockershim -- missing full test pass with containerd is not ideal.


Integer Overflow of BinaryStatistics#isSmallerThan()

"(min.length() + max.length()) < size" didn't handle integer overflow
https://github.com/apache/parquet-mr/blob/master/parquet-column/src/main/java/org/apache/parquet/column/statistics/BinaryStatistics.java#L103
Reporter: Xinli Shang / @shangxinli
Assignee: Xinli Shang / @shangxinli
PRs and other links:

GitHub Pull Request #704
GitHub Pull Request #721
GitHub Pull Request #722

Note: This issue was originally created as PARQUET-1690. Please see the migration documentation for further details.

Gabor Szadovszky / @gszadovszky:
Removing 1.11.0 target because this is an improvement only and not a regression since 1.10.0.

Alexey Diomin:
@gszadovszky  could you review the latest version of pr https://github.com/apache/parquet-mr/pull/722
or do we need to resubmit pr (we can add unit tests, but it will require at least 1G+ on disk for ci, don't know what the limits for your ci workers)?
> Removing 1.11.0 target because this is an improvement only and not a regression since 1.10.0.

it's incorrect, the ticket was created with the incorrect Type
what we see in real life: # try to write large blobs with byte arrays

have overflowed value in function "do we need or not to write min/max statistics"

writing it in final file + have some incorrect offsets

have corrupted files and exceptions on read java.lang.RuntimeException: corrupted file: the footer index is not within the file:<PHONE_NUMBER>3
instead, file with 1 recond in 1GB we have at least = 1GB for data + metadata (1GB for min + 1GB for max + some other data)
file with 1gb blob, in reality, consume 5GB+ on disk and can't be read

reproducible example for spark
https://gist.github.com/REASY/98d80827d53e290d53a93f2bc3b76ab0

Xinli Shang / @shangxinli:
It is a quite long time ago. I don't remember. Yeah, it would be great to start off a new PR.

Gabor Szadovszky / @gszadovszky:
[~humanoid], I don't know/remember the background of this issue and the closed PRs. I think it would be the best to start over with a new PR.
<EMAIL_ADDRESS>do you remember why the last PR was closed and not reviewed/submitted?


How to run validation when click the finish button?

How to run validation when click the finish button?

@MrMaxim22 thank you for using our product. Since we've been doing just the interface, we recommend you to check the official documentation on validation here: http://vinceg.github.io/twitter-bootstrap-wizard/ and this example here: http://vinceg.github.io/twitter-bootstrap-wizard/examples/validation.html
Hope this helps.
Best,
Alex

Just change the finish button to type submit.

How can I give custom validation to inputs on each tabs?

Exactly, change the button "Finish":
 <input type='button' for  <input type='submit'


[BUG]:TestRunID not picking up the value

Describe the bug
Hi there, not sure if someone else experienced this, but I am hardcoding a string value as the runId and nothing comes through the dashboard. It's still showing TestRun#build. Am I missing something or doing something wrong ?
To Reproduce
Steps to reproduce the behavior:

Step 1: Replace the .env variablle on the runId value wrapped in the conectOptions with some string.
Step 2: Run the tests

Expected behavior
TestRun should have the name assigned in the config file when lookin at the dashboard
 connectOptions: {
      wsEndpoint: `${process.env.PLAYWRIGHT_SERVICE_URL}?cap=${JSON.stringify({
        // Can be 'linux' or 'windows'.
        os,
        runId: `TestingExample - ${new Date().toISOString()}`
      })}`,
      timeout: 30000,
      headers: {
        'x-mpt-access-key': process.env.PLAYWRIGHT_SERVICE_ACCESS_TOKEN!
      },
      // Allow service to access the localhost.
      exposeNetwork: '<loopback>'
    }

Dashboard test run name:
TestRun#build
AB#1907781

Hi @imartinflores I just tried the above fixture change and i can see changed runId showing up in activityLog

I want to point one diferent issue in above snippet that if you put runId value directly then every individual session will show up as different entry in activityLog dashboard, which may not be inteneded as you like to see summary of entire run. hence, here is what i would suggest
change line https://github.com/microsoft/playwright-testing-service/blob/main/samples/get-started/playwright.service.config.ts#L25 to
// Name the test run if it's not named yet.
process.env.PLAYWRIGHT_SERVICE_RUN_ID = process.env.PLAYWRIGHT_SERVICE_RUN_ID || `TestingExample - ${new Date().toISOString()}`

and no other change in fixture.

hey @puagarwa that is what we've done initially but it is not being updated in the activityLog. We hardcoded the value directly in the runId to see if that makes a change but nothing either..
Is there any other place where I should look into ? should I enable logs ?
Thanks

lace where I should look into ? should I enable logs

Logs wont help much here unless you add your own logs to print the value.
check these things

runId is set to runId: process.env.PLAYWRIGHT_SERVICE_RUN_ID in connectOptions
set this before connectOptions process.env.PLAYWRIGHT_SERVICE_RUN_ID = process.env.PLAYWRIGHT_SERVICE_RUN_ID || TestingExample - ${new Date().toISOString()}
make sure you are using right config file, probably print some console log in config file.

@puagarwa  I have added a console.log, I can see this being printed:
"Landers_Build - 2023-10-25T13:25:23.721Z
However in the activityLog, it seems the value is being truncated:
TestRun#3.721Z (Matches with the last part of the build name we used)

To rule out truncate issue, you can actually put some random string because timestamp could be same for other reasons as well.
You could enable DEBUG=pw:channel:command* logs, and search for log like this
pw:channel:command     wsEndpoint: 'wss://eastasia.api.playwright-int.io/api/authorize/connectSession?cap={"os":"windows","runId":"TestingExample - 2023-10-25T13:46:03.845Z"}',
this will confirm value being sent from client side.
PS: Dont enable this log in CI since this print accessKey as well.

HI @imartinflores We dont have any such truncate logic, i have tested different values and all are working properly.
It seems like you are always seeing the same value "TestRun#Build" , can you change your value to "AnyRandomString" and see what do you see.
Another way would be you try our sample suite with different value of runId as i mentioned earlier to validate it.

@puagarwa we can close it - I've tried it again on my own and I can see the name coming in full now. On my colleague machine, we still see it truncated, so we will dig into. Thanks for the help!

Glad to hear, it resolved for you. Feel free to open it if you have again or another issue.

@imartinflores Actually, we do have a bug which truncate anything after _ from runId string, We will fix it asap.

Ahh alright, cool, makes sense. Thanks for the update!
On Thu, 26 Oct 2023, 15:44 Puneet Agarwal, @.***> wrote:

@imartinflores https://github.com/imartinflores Actually, we do have a
bug which truncate anything after _ from runId string, We will fix it asap.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/playwright-testing-service/issues/63#issuecomment-1781275343,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEBZGZMR57L5C3T2J73GVELYBJZOHAVCNFSM6AAAAAA6PFMUFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBRGI3TKMZUGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

@imartinflores This is fixed now.


Use dataclass for the App classes

Almost done putting these objects on the stub

Oh you might be running into the synchronicity issue with generating type stubs for field() defaults that defeated me :(

(it's not really anything specifically to do with field(), just annotations for types with nontrivial reprs.


Forking courtesy call, close after reading.

Hi, I'm in the process of creating a similar project to yours but I have a few different priorities (creating open, machine-readable data and eventually ascending to P2P heaven). For this reason I'm forking your project and this is my courtesy call.
https://gitlab.com/willemmali/awesome-awesome
I wish you the best of luck with your project. I hope we might be able to consolidate our efforts in the future, or coexist peacefully.

Cool. Have fun :) 🦄🌈

It might be nice to make a central data-store with all the data, metadata, and so forth.  Then various things can be generated for human readable for specific purposes or machine readable or so forth.

I'll be looking at making a decentralized data-store with the data and
metadata, but first I'm doing a conversion from Markdown to YAML.
I could notify you once I've done that?
On Thu, May 11, 2017 at 4:44 PM, OvermindDL1<EMAIL_ADDRESS>wrote:

It might be nice to make a central data-store with all the data, metadata,
and so forth. Then various things can be generated for human readable for
specific purposes or machine readable or so forth.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sindresorhus/awesome/issues/958#issuecomment-300811229,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEePFyiWL2c_bpvbWQlzmJoFzztB_m05ks5r4x7kgaJpZM4NX-cq
.


Add ability to send context information with conversations to proxy server

e.g. where the user is, in which block, which plugin etc.


Searching for publication year ranges broken

Searching for publication year like >2016 is broken. Results in a NumberFormatException
Caused by: java.lang.NumberFormatException: For input string: ">"
        at java.base/java.lang.NumberFormatException.forInputString(NumberFormatException.java:65) ~[na:na]
        at java.base/java.lang.Integer.parseInt(Integer.java:652) ~[na:na]
        at java.base/java.lang.Integer.parseInt(Integer.java:770) ~[na:na]
        at ch.difty.scipamato.core.entity.search.IntegerSearchTerm.<init>(IntegerSearchTerm.java:108) ~[scipamato-core-entity-1.1.6.jar!/:1.1.6]
        at ch.difty.scipamato.core.entity.search.IntegerSearchTerm.<init>(IntegerSearchTerm.java:54) ~[scipamato-core-entity-1.1.6.jar!/:1.1.6]
        at ch.difty.scipamato.core.entity.search.IntegerSearchTerm.<init>(IntegerSearchTerm.java:50) ~[scipamato-core-entity-1.1.6.jar!/:1.1.6]
        at ch.difty.scipamato.core.entity.search.SearchTerm.newIntegerSearchTerm(SearchTerm.java:85) ~[scipamato-core-entity-1.1.6.jar!/:1.1.6]
        at ch.difty.scipamato.core.entity.search.SearchCondition.setIntegerValue(SearchCondition.java:475) ~[scipamato-core-entity-1.1.6.jar!/:1.1.6]
        at ch.difty.scipamato.core.entity.search.SearchCondition.setPublicationYear(SearchCondition.java:216) ~[scipamato-core-entity-1.1.6.jar!/:1.1.6]
        at jdk.internal.reflect.GeneratedMethodAccessor516.invoke(Unknown Source) ~[na:na]
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[na:na]
        at java.base/java.lang.reflect.Method.invoke(Method.java:566) ~[na:na]
        at org.apache.wicket.core.util.lang.PropertyResolver$MethodGetAndSet.setValue(PropertyResolver.java:999) ~[wicket-core-8.3.0.jar!/:8.3.0]
        ... 116 common frames omitted

Quick problem analysis:
The PaperPanel processes each character as it is typed. The IntegerSearchTerm thus first receives a > and cannot parse it, as it expects > with a total length of > 1 and falls into parsing the > as integer -> exception.
Two options I need to decide on:

Enhance the IntegerSearchTerm to be able to deal with incomplete search term specification
Disable the character by character processing for searches and only process complete search terms

The latter is definitely more efficient for the server and also more concise, as boiler-plate for dealing with an incomplete search term can be omitted. However, the user risks losing partially specified searches if she closes the browser or has a blue screen. Currently, the search is saved up to the point of the incident.

Workaround for now:

Enter the year first 2017 (-> valid search term)
Add the range operator as prefix in the end >2017 (still valid search term)

I limited the scope of this ticket for IntegerSearchTerms to handle incomplete input. I'm currently not touching the character by character processing.

Also fixed searching by ID (PSQLException: ERROR: column reference "id" is ambiguous).


Message when open terminal

"The program 'docker' is currently not installed.  You can install it by typing:
sudo apt-get install docker"
Disabled plugins:
-docker
Disabled completions:
-packer
How to remove this message ?

Can you post the following please?

Your ~/.bash_profile and/or ~/.bashrc.
The output of bash-it show plugins

Does the message go away when you comment out the line that starts Bash-it in either ~/.bash_profile or ~/.bashrc (probably the latter, since you seem to be running Linux)?
Are you running the latest master version of Bash-it, or an earlier version?

~/.bash_profile not found
cat ~/.bashrc
#!/usr/bin/env bash

# Path to the bash it configuration
export BASH_IT=$HOME/.bash_it

# Lock and Load a custom theme file
# location /.bash_it/themes/
export BASH_IT_THEME='bobby'

# Your place for hosting Git repos. I use this for private repos.
export<EMAIL_ADDRESS>
# Don't check mail when opening terminal.
unset MAILCHECK

# Change this to your console based IRC client of choice.
export IRC_CLIENT='irssi'

# Set this to the command you use for todo.txt-cli
export TODO="t"

# Set this to false to turn off version control status checking within the prompt for all themes
export SCM_CHECK=true

# Set vcprompt executable path for scm advance info in prompt (demula theme)
# https://github.com/xvzf/vcprompt
#export VCPROMPT_EXECUTABLE=~/.vcprompt/bin/vcprompt

# Load Bash It
source $BASH_IT/bash_it.sh

bash-it show plugins
Plugin              Enabled?  Description
base                  [x]     miscellaneous tools
battery               [x]     display info about your battery charge level
browser               [x]     render commandline output in your browser
chruby-auto           [ ]     load chruby + auto-switching (from /usr/local/share/chruby)
chruby                [ ]     load chruby                  (from /usr/local/share/chruby)
dirs                  [x]     directory stack navigation
docker                [ ]     Helpers to get Docker setup correctly for boot2docker and to more easily work with Docker
extract               [x]     one command to extract them all...
fasd                  [ ]     navigate "frecently" used files and directories
fzf                   [x]     load fzf, if you are using it
gh                    [x]     load gh, if you are using it.
gif                   [x]     gif helper functions
git                   [x]     git helper functions
hg                    [x]     hg helper functions
history               [x]
java                  [ ]     Java and JAR helper functions
javascript            [x]     download jquery files into current project
jekyll                [x]     manage your jekyll site
latex                 [x]     use mactex
nginx                 [x]     manage your nginx service
node                  [x]     Node.js helper functions
nvm                   [x]     node version manager, as a bash function
osx                   [x]     osx-specific functions
pipsi                 [x]     load pipsi, if you are using it
postgres              [ ]     postgres helper functions
proxy                 [x]     Proxy Tools
pyenv                 [ ]     load pyenv, if you are using it
python                [ ]     alias "http" to SimpleHTTPServer
rbenv                 [x]     load rbenv, if you are using it
ruby                  [x]     ruby and rubygems specific functions and settings
rvm                   [x]     load rvm, if you are using it
ssh                   [x]     ssh helper functions
subversion            [x]     svn helper functions
textmate              [x]     set textmate as a default editor
tmuxinator            [x]     sources tmuxinator script if available
tmux                  [x]     make sure that tmux is launched in 256 color mode
todo                  [x]
virtualenv            [x]     virtualenvwrapper helper functions
_xterm                [ ]     automatically set your xterm title with host and location info
z_autoenv             [x]     source into environment when cding to directories
z                     [ ]     maintains a jump-list of the directories you actually use
                              z is DEPRECATED, use fasd instead

Does the message go away when you comment out the line that starts Bash-it in either ~/.bash_profile or ~/.bashrc (probably the latter, since you seem to be running Linux)?

Message not output

Are you running the latest master version of Bash-it, or an earlier version?

branch master. How to see version ?

Do you have anything in the ~/.bash_it/custom folder? Any scripts?
You can also run bash -x ./bash_it.sh in the ~/.bash_it folder. This will produce a lot of output, but it should show where docker is called. You can create a gist with the output and link to it here.

Do you have anything in the ~/.bash_it/custom folder? Any scripts?

No. Only empty example.bash
./bash_it.sh
/home/max/.bash_it/aliases/enabled/docker.aliases.bash: line 6: dklcid: command not found
/home/max/.bash_it/plugins/enabled/history.plugin.bash: line 3: bind: warning: line editing not enabled
/home/max/.bash_it/plugins/enabled/history.plugin.bash: line 4: bind: warning: line editing not enabled

thanks!!!
bash-it disable alias docker

Ah, it looks like the docker alias dklcip calls the alias dklcid, and that seems to be executed when the alias is defined. Sounds a bit dodgy to me...
Please close the issue if you feel that this is resolved.


Add validation for xnames in BootparametersPost and enhance tests

This adds validation of xnames for bss using the Cray-HPE xname library and adds tests to validate the behavior I believe is requested in #51

Tested this on our cluster:
$ curl -H "Authorization: Bearer $ACCESS_TOKEN" https://foobar.openchami.cluster:8443/boot/v1/bootparameters -v -d '{"hosts": ["test1.local", "test2.local"], "macs": ["00:40:a6:82:f6:c5","a4:bf:01:3e:c0:a2","00:40:a6:82:f7:0c"], "nids": [1, 2, 3], "params": "test", "kernel": "test", "initrd": "test", "cloud-init": {"user-data": {"foo": "bar"},"meta-data": {"foo": "bar"}}}'
[...]
< HTTP/2 400
< content-type: application/problem+json
< date: Mon, 02 Dec 2024 21:00:06 GMT
< content-length: 109
<
{"type":"about:blank","title":"Bad Request","detail":"Bad Request: invalid xname: test1.local","status":400}
* Connection #0 to host foobar.openchami.cluster left intact

$ curl -H "Authorization: Bearer $ACCESS_TOKEN" https://foobar.openchami.cluster:8443/boot/v1/bootparameters -v
[...]
< HTTP/2 200
< content-type: application/json; charset=UTF-8
< date: Mon, 02 Dec 2024 21:01:09 GMT
< content-length: 5
<
null
* Connection #0 to host foobar.openchami.cluster left intact

$ curl -H "Authorization: Bearer $ACCESS_TOKEN" https://foobar.openchami.cluster:8443/boot/v1/bootparameters -v -d '{"hosts": ["x0c0s2b0n0", "x0c0s3b0n0"], "macs": ["00:40:a6:82:f6:c5","a4:bf:01:3e:c0:a2","00:40:a6:82:f7:0c"], "nids": [1, 2, 3], "params": "test", "kernel": "test", "initrd": "test", "cloud-init": {"user-data": {"foo": "bar"},"meta-data": {"foo": "bar"}}}'
[...]
< HTTP/2 201
< bss-referral-token: 2c6fd5db-8224-4ae7-ad8d-d692f54f5801
< content-type: application/json; charset=UTF-8
< date: Mon, 02 Dec 2024 21:03:07 GMT
< content-length: 0
<
* Connection #0 to host foobar.openchami.cluster left intact

$ curl -H "Authorization: Bearer $ACCESS_TOKEN" https://foobar.openchami.cluster:8443/boot/v1/bootparameters -v
[...]
< HTTP/2 200
< content-type: application/json; charset=UTF-8
< date: Mon, 02 Dec 2024 21:04:03 GMT
< content-length: 246
<
[{"hosts":["x0c0s2b0n0","x0c0s3b0n0"],"params":"test","kernel":"test","initrd":"test","cloud-init":{"meta-data":null,"user-data":null,"phone-home":{"pub_key_dsa":"","pub_key_rsa":"","pub_key_ecdsa":"","instance_id":"","hostname":"","fqdn":""}}}]
* Connection #0 to host foobar.openchami.cluster left intact


Implement Corpora API to read projects from database

Instead of hardcoding the datasets, implement the GET /v1/project API to read existing projects from the database which will eventually be used to populate the frontend of corpora.cziscience.com.
For more information, see the high level Corpora architecture documentation, API Design section here.
Tasks:

[ ] Implement endpoint to query DB
[ ] Unit test

@calvinnhieu To break down this ticket further and provide estimates.

Updated and estimated

@Bento007 this should be in the Now pipeline right?


Switched to FnvHashMap and removed nightly only features to fix failing tests

Tests needed to be updated from std::collections::HashMap to fnv::FnvHashMap to match the Registry constructor
Nightly only features needed to be removed for stable branches to pass in CI

@matt-keibler awesome, thanks.
I wanted to do the same thing this weekend, so the effort is much appreciated!


Added  option to connect to Ollama

Fixes #29
This pull request introduces integration with Ollama for enhanced functionality in Storybook Genie. It includes:

Added environment variable placeholders for both OpenAI (OPENAI_API_KEY) and Ollama (OLLAMA_HOST) to the .env file for flexibility between models.

Updated key project files, including index.js, client.js, converter.js, and models.js, to accommodate Ollama integration alongside OpenAI.

Minor modifications to package.json and package-lock.json for dependency adjustments related to the integration.

I tried to test your implementation and I get an error after I select Ollama from the menu

@eduardconstantin

used this api but still getting same error

@eduardconstantin  used this api but still getting same error

In the models file you need to use fetch to call the api, not exec


Export DocumentSet to bibtex

Currently, there is no way to export results after loading them into memory. On possibility would be to add the ability to export a DocumentSet to a bibtex file.

Hopefully this workaround will help.
import bibtexparser

def export_documentset_to_bibtex(document_set, output_filename="exported_file.bib"):
    """
    Exports entries from a litstudy DocumentSet to a .bib file.

    Parameters:
        document_set (DocumentSet): A litstudy DocumentSet containing documents to export.
        output_filename (str): The name of the output .bib file (default: "exported_file.bib").
    """
    # Convert DocumentSet entries to bibtexparser format
    bibtex_entries = []
    for entry in document_set:
        bibtex_entry = entry.entry
        bibtex_entries.append(bibtex_entry)

    # Create a BibDatabase object and set entries
    bib_database = bibtexparser.bibdatabase.BibDatabase()
    bib_database.entries = bibtex_entries

    # Write entries to the output BibTeX file
    with open(output_filename, "w") as bibfile:
        bibtexparser.dump(bib_database, bibfile)

    print(f"BibTeX file exported successfully as '{output_filename}'")


Added parameter offset, fixed scrolling of multiple forground elements.

Hi!
I am using enllax.js in a project. I have two sections, which are in the foreground. I had troubles with the second element, especially on OSX in Safari.
I managed to get it working with my fixes.
I also added a parameter offset to take into account a static navigation at the top of the browser window.
With kind regards
Hannes Dorn

Thanks hannesdorn, for your support and contribution. :+1:


Organization of the Op-Design Repository(Designfree week)-2

-Organization of GitHub OctopOS Prime Design repository.
-Organizing the 2D and 3D visuals of our Octopos Prime project.
-Indicating what are the tasks performed.

I have uploaded my 3D projects that I designed for the Web team of our Octopos prime project and our swimming animation project that we worked on as a team to GitHub together with the (Render and Blender) files.


[CI] Set up a nightly pipeline to test with dev versions of RAPIDS

Currently, XGBoost is tested with latest stable version of RAPIDS. As a result, the latest XGBoost may not be fully compatible with an upcoming version of RAPIDS. By the time RAPIDS releases a new version, it may end up packaging a version of XGBoost that's not fully compatible with the new RAPIDS version. See https://github.com/dmlc/xgboost/pull/10175 for an example.
We should set up a CI pipeline that tests XGBoost with development version of RAPIDS on a nightly basis.

Not entirely sure if we want to have nightly packages in the CI, we have very limited resources for handling sudden CI changes, maybe we need integration tests that are run before release instead of unit tests in CI?

issues can take time to resolve

I think it's better if we can find issues earlier rather than wait until the release.


Add advanced repository management

i used a resident for 2 weeks, and by running /newmaps, the pgm rescans it repo for changes. i would like to see this implemented in cardinal since i use cardinal a lot for making and testing maps.

@kroest Can't you use /recycle or /cycle?

It's pretty annoying stop and start again the server to load a new map. Would be useful.

a /errors command would also be apreciated for xml testing

@Pablete1234 You can use online XML checkers to parse the XML before you load the map, or just keep a repo with one working map and the one that is being tested. Be sure to enable display-map-load-errors in config.yml

sung matt. /errors is not for wrongly phrased xml's, it's also for example if you type a item name incorrectly, then the whole kit doesn't work, if you have some misspelled stuff that can be detected ,etc

@Pablete1234 that would require a huge amount more effort, not something we are going to do right now

@CaptainElliott although when writing new features making it fail-safe is a good idea

@twiznwazin All the modules would have to be rewritten and you could add all the Exceptions to an array list.

@dentmaged Not entirely, although the plan is to rewrite most modules in the coming weeks. I am going to do an overhaul of the module system, and Elly planned to do some much needed refractoring and cleanup.

Fixed in 1.9


Decide how to handle Dart plugin registration for direct implementation dependencies

Dart plugin registration uses the implementing-package resolution system described in the federated plugin design doc to figure out which plugins to run Dart registration for. At a high level the rules are:

If there's a direct dependency from the app on an implementation package, use that (to allow overriding a default implementation).
Otherwise, use the default implementation.

This works well for normal federated plugin usage. We recently ran into an edge case, which is that sometimes a package might depend only on a specific platform implementation. For example: shared_preference_windows depends directly on path_provider_windows (but not path_provider), because we need the functionality to implement shared_preferences_windows, but we wouldn't want to cause, say, the Android and iOS apps of anything using shared_preferences to unnecessarily include the path_provider code for those platforms. In this case:

the application has no dependency on the implementation package, and
there is no default implementation (because the concept of "default implementation" is defined by the app-facing package that we aren't using)
so Dart registration doesn't run for path_provider_windows. This is fine if we use PathProviderWindows directly (which we do), but breaks if we try to use PathProviderPlatform.instance (which is the equivalent of what the PR above did) for reasons that are probably not at all clear to anyone hit by it.

We have a few options here:

Keep the current behavior of not running registration
Run registration for implementation packages whose app-facing packages are not in the transitive dependency list. The question then becomes what to do in the edge case where two different implementations are in the transitive dependency list:

Pick one of them arbitrarily.
Error out and require the application that is ultimately the transitive consumer of the two to pick one to depend on.

I'm leaning toward thinking we should do some form of 2 since I would expect the case where two different packages pick two different implementation packages for the same platform to be very rare, so 2 should give surprising behavior less often.

I just ran into a variant of this, which I feel is slightly more confusing than the rules set out above. Consider the following packages:

my_app (application)
my_auth (pure dart library)
flutter_appauth (pub.dev implementation plugin)
flutter_appauth_platfrom_interface (pub.dev platform interface package)
flutter_appauth_desktop (my extension to add/prototype windows and linux support)

The idea would be to have all appauth plugins be an implementation detail of my_auth and for my_app to depend on that library. To that end, I'd like to have a way to make sure that the non-endorsed flutter_appauth_desktop plugin is registered. The current language on docs.flutter.dev on that is (emphasis mine):

A developer can still use your implementation, but must manually add the plugin to the app’s pubspec file. So, the developer must include both the foobar dependency and the foobar_windows dependency in order to achieve full functionality.
— https://docs.flutter.dev/development/packages-and-plugins/developing-packages#non-endorsed-federated-plugin

Personally, I missed that this really only refers to direct dependencies of a top-level application and packages depending on (non-endorsed) plugin implementations are not supported. This could probably be emphasized more, but there's something else:
As this issue confirms, this only applies to the dart portion of plugins. The flutter_appauth_platform_interface package happens implement and use a platform implementation based on MethodChannel as the default instance. If I add a flutter.plugin.platforms.windows.pluginClass value to the pubspec, this is registered and usable.
Only when I try to use dartPluginClass is that not picked up, unless the plugin is added to the app pubspec. This difference in behavior seems confusing and unintentional, and it's unfortunate because I do have a pure dart implementation for the package and don't really want to drop down to C++ if I can avoid it.
So implementing 2. as proposed would make things even more confusing for non-endorsed plugins: If I transitively depend only on flutter_appauth_desktop, the dartPluginClass should start working on those platforms as long as I don't also depend on flutter_appauth.
Some actionable ideas, that might help directing developers in the right direction:

With the current behavior: Warn (or error) if a plugin sets flutter.plugin.implements, but is not endorsed and is not a direct dependency of the application. Right now, these are partially registered as independent native plugins.
Warn if the app-facing package in implements is not in the dependency tree at all. Currently, I can set implements: foobar in pubspec and nothing complains.
If the intent is not to register non-endorsed plugins, consider restricting the ability of non-implements native plugins to addMessageListener on channels created by other plugins. This seems to be difficult to implement in a backwards-compatible way, because the current APIs to create a MessageChannel don't necessarily require using a Registrar or other means of linking the channel registration to the current plugin.
If that is not the intent, consider lifting the non-endorsement restriction and register all dartPluginClasses in the dependency tree unconditionally, as is done for native pluginClass and only error/pick one when there are multiple colliding implementations in the dependency tree.

This difference in behavior seems confusing and unintentional

It is unintentional in that we haven't yet extended the resolution system back to the existing native registration system. That is the intent, at which point the behaviors will be consistent.

If that is not the intent, consider lifting the non-endorsement restriction and register all dartPluginClasses in the dependency tree unconditionally, as is done for native pluginClass and only error/pick one when there are multiple colliding implementations in the dependency tree.

This seems reasonable; I agree that the use case you described should work as you expected that it would, and I don't see any issues with enabling that approach.
I suspect that the focus on direct dependencies (other than for conflict resolution) in the original design document was just an oversight.


Add Corvus CF from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of micha, to add Corvus CF to CKAN.
Mod details:
name = /mod/1405/Corvus%20CF
author = micha
abstract = A 2-Kerbal Command Pod for the early game, originally by OrionKermin
license = CC-BY-SA
Homepage = http://forum.kerbalspaceprogram.com/index.php?showtopic=108993
description = Corvus is a small parts pack based around a 1.25m two seat capsule.
It has been made to be a "Kerbalized" interpretation of Gemini.
The pack consists of

the command capsule
an appropriately themed parachute/ docking port combo
a service module
a small in-line monopropellant engine
a heatshield
a decoupler

The pod also features a complete IVA.
Please direct questions about this pull request to micha.

I redid this already


argo lint panics on a certain invalid workflow

Summary
What happened/what you expected to happen?
argo lint workflow.yaml panics.
This is not a blocker for me. Just stumbled upon it and decided to report.
What version of Argo Workflows are you running?
v3.3.0-rc2
Diagnostics
Either a workflow that reproduces the bug, or paste you whole workflow YAML, including status, something like:
apiVersion: argoproj.io/v1alpha1
kind: Workflow
spec:
  entrypoint: root
  templates:
    - name: template
      inputs:
        parameters:
          - name: data
      container:
        name: main
        image: alpine
    - name: root
      inputs:
        parameters:
          - name: anything_param
      dag:
        tasks:
          - name: task
            template: template
            arguments:
              parameters:
                - name: data
                  # There is a bug here. I still keep forgetting which syntax is right >_<
                  valueFrom:
                    parameter: "{{inputs.parameters.anything_param}}"
  arguments:
    parameters:
      - name: anything_param
        value: anything_param

panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x0 pc=0x19c6f3b]

goroutine 1 [running]:
github.com/argoproj/argo-workflows/v3/workflow/validate.validateDAGTaskArgumentDependency({{0xc000189080, 0x1, 0x4}, {0x0, 0x0, 0x0}}, {0x478a408, 0x0, 0x23ec660})
        /home/runner/work/argo-workflows/argo-workflows/workflow/validate/validate.go:1332 +0x3bb
github.com/argoproj/argo-workflows/v3/workflow/validate.(*templateValidationCtx).validateDAG(0xc00012bb40, 0xc00042dd40, 0x2526839, 0xc000180900)
        /home/runner/work/argo-workflows/argo-workflows/workflow/validate/validate.go:1311 +0xce5
github.com/argoproj/argo-workflows/v3/workflow/validate.(*templateValidationCtx).validateTemplate(0xc00012bb40, 0xc000180240, 0xc00012e9c0, {0x32dd130, 0xc00042dbf0})
        /home/runner/work/argo-workflows/argo-workflows/workflow/validate/validate.go:399 +0xf3c
github.com/argoproj/argo-workflows/v3/workflow/validate.(*templateValidationCtx).validateTemplateHolder(0x21628c0, {0x32ec2c0, 0xc00012e9c0}, 0xf, {0x32dd130, 0xc00042dbf0})
        /home/runner/work/argo-workflows/argo-workflows/workflow/validate/validate.go:486 +0x2e9
github.com/argoproj/argo-workflows/v3/workflow/validate.ValidateWorkflow({0x32bb620, 0xc000b86f50}, {0x32bb600, 0xc000b86f70}, 0xc000264900, {0x1, {0x0, 0x0}, 0x0, 0x0, ...})
        /home/runner/work/argo-workflows/argo-workflows/workflow/validate/validate.go:208 +0xfd6
github.com/argoproj/argo-workflows/v3/server/workflow.(*workflowServer).LintWorkflow(0xc000990b40, {0x330f6b0, 0xc0008bbe90}, 0xc00012bb00)
        /home/runner/work/argo-workflows/argo-workflows/server/workflow/workflow_server.go:504 +0x1d5
        /home/runner/work/argo-workflows/argo-workflows/pkg/apiclient/argo-kube-workflow-service-client.go:92 +0x2b
github.com/argoproj/argo-workflows/v3/pkg/apiclient.(*errorTranslatingWorkflowServiceClient).LintWorkflow(0x250efd0, {0x330f6b0, 0xc0008bbe90}, 0xc000264900, {0x0, 0x400, 0x32bb100})
        /home/runner/work/argo-workflows/argo-workflows/pkg/apiclient/error-translating-workflow-service-client.go:85 +0x33
github.com/argoproj/argo-workflows/v3/cmd/argo/lint.lintData({0x330f6b0, 0xc0008bbe90}, {0xc00005e090, 0x8c}, {0xc00083ec00, 0x2b0, 0x400}, 0xc000a27be8)
        /home/runner/work/argo-workflows/argo-workflows/cmd/argo/lint/lint.go:224 +0x883
        /home/runner/work/argo-workflows/argo-workflows/cmd/argo/lint/lint.go:155 +0x2aa
path/filepath.walk({0xc00005e090, 0x8c}, {0x3333620, 0xc000483c70}, 0xc0009dda80)
        /opt/hostedtoolcache/go/1.17.6/x64/src/path/filepath/path.go:418 +0x125
path/filepath.Walk({0xc00005e090, 0x8c}, 0xc0009dda80)
        /opt/hostedtoolcache/go/1.17.6/x64/src/path/filepath/path.go:505 +0x6c
github.com/argoproj/argo-workflows/v3/cmd/argo/lint.Lint({0x330f6b0, 0xc0008bbe90}, 0xc000a27be8)
        /home/runner/work/argo-workflows/argo-workflows/cmd/argo/lint/lint.go:124 +0x1eb
github.com/argoproj/argo-workflows/v3/cmd/argo/lint.RunLint({0x330f6b0, 0xc0008bbe90}, {0x3332660, 0xc0005b5d20}, {0xc0000d2600, 0x4, 0x4}, {0x250bd5c, 0x6}, 0x0, ...)
        /home/runner/work/argo-workflows/argo-workflows/cmd/argo/lint/lint.go:98 +0x29f
github.com/argoproj/argo-workflows/v3/cmd/argo/commands.NewLintCommand.func1(0xc000833680, {0xc00094e4d0, 0x1, 0x1})
        /home/runner/work/argo-workflows/argo-workflows/cmd/argo/commands/lint.go:52 +0x325
github.com/spf13/cobra.(*Command).execute(0xc000833680, {0xc00094e4a0, 0x1, 0x1})
        /home/runner/go/pkg/mod/github.com/spf13/cobra@v1.2.1/command.go:860 +0x5f8
github.com/spf13/cobra.(*Command).ExecuteC(0xc000832c80)
        /home/runner/go/pkg/mod/github.com/spf13/cobra@v1.2.1/command.go:974 +0x3bc
github.com/spf13/cobra.(*Command).Execute(...)
        /home/runner/go/pkg/mod/github.com/spf13/cobra@v1.2.1/command.go:902
main.main()
        /home/runner/work/argo-workflows/argo-workflows/cmd/argo/main.go:14 +0x1e

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

Thanks for reporting. I am fixing the panic in https://github.com/argoproj/argo-workflows/pull/7706.
The problem with the YAML is that you are passing parameter values incorrectly in the DAG task. It should be:
      dag:
        tasks:
          - name: task
            template: template
            arguments:
              parameters:
                - name: data
                  value: "{{inputs.parameters.anything_param}}"


setuptools using CC for compiling C++ files

When compiling an extension like
ext_modules = [
    Extension(
        "_foobar",
        [
           # ...
        ],
        language="C++",
    )
]

setuptools appears to use the CC environment variable when compiling the project. E.g.,
CC=clang++ pip3 install . --user --verbose

will switch to clang++. Since this is a C++ project (as specified in the extension), I would have expected CXX to have an effect, and not CC.

Hi @nschloe I think you simply need to write c++ instead of C++ (lowercase c instead of uppercase C). Let me know if this fixes your problem.

Dear @jaraco and @pganssle it seems that no checks are implemented on the language variable (I can set it to language='jadshad' and the installation process would still work). If you point me in the right direction (where to look in the repo), I would be happy to try to tackle this issue. Thank you.
-Filippo

I think you simply need to write c++ instead of C++ (lowercase c instead of uppercase C). Let me know if this fixes your problem.

It does not.

Did you remove the build directory first?

Yes.

I've just bumped into this issue because I had a similar problem. I am not a maintainer for setuptools. However, your one-liner answer are not really helping me in possibly helping you. Maybe if you are willing to give more details I could help.

I think I've given all the necessary details in the original post. Also, after having tried your suggestion and seeing it fail, I'm wondering what makes it work on your machine.

What about the OS? Are you compiling on OS X? Could you provide the full setup.py file? What is the output of pip install with the verbose flag on?

What about the OS?

Linux.

Are you compiling on OS X?

No.

Could you provide the full setup.py file?

I see this happening on all of my projects that involve C++ code, e.g., https://github.com/nschloe/accupy.

What is the output of pip install with the verbose flag on?

CXX="clang++" pip3 install . --user --verbose

Created temporary directory: /tmp/pip-ephem-wheel-cache-cr9ij94h
Created temporary directory: /tmp/pip-req-tracker-pl67s0va
Created requirements tracker '/tmp/pip-req-tracker-pl67s0va'
Created temporary directory: /tmp/pip-install-ko80ezgm
Processing /home/nschloe/rcs/accupy
  Created temporary directory: /tmp/pip-req-build-k7bjh5uk
  Added file:///home/nschloe/rcs/accupy to build tracker '/tmp/pip-req-tracker-pl67s0va'
    Running setup.py (path:/tmp/pip-req-build-k7bjh5uk/setup.py) egg_info for package from file:///home/nschloe/rcs/accupy
    Running command python setup.py egg_info
    running egg_info
    creating /tmp/pip-req-build-k7bjh5uk/pip-egg-info/accupy.egg-info
    writing /tmp/pip-req-build-k7bjh5uk/pip-egg-info/accupy.egg-info/PKG-INFO
    writing dependency_links to /tmp/pip-req-build-k7bjh5uk/pip-egg-info/accupy.egg-info/dependency_links.txt
    writing requirements to /tmp/pip-req-build-k7bjh5uk/pip-egg-info/accupy.egg-info/requires.txt
    writing top-level names to /tmp/pip-req-build-k7bjh5uk/pip-egg-info/accupy.egg-info/top_level.txt
    writing manifest file '/tmp/pip-req-build-k7bjh5uk/pip-egg-info/accupy.egg-info/SOURCES.txt'
    reading manifest file '/tmp/pip-req-build-k7bjh5uk/pip-egg-info/accupy.egg-info/SOURCES.txt'
    reading manifest template 'MANIFEST.in'
    writing manifest file '/tmp/pip-req-build-k7bjh5uk/pip-egg-info/accupy.egg-info/SOURCES.txt'
  Source in /tmp/pip-req-build-k7bjh5uk has version 0.2.0, which satisfies requirement accupy==0.2.0 from file:///home/nschloe/rcs/accupy
  Removed accupy==0.2.0 from file:///home/nschloe/rcs/accupy from build tracker '/tmp/pip-req-tracker-pl67s0va'
Requirement already satisfied: mpmath in /home/nschloe/.local/lib/python3.8/site-packages (from accupy==0.2.0) (1.1.0)
Requirement already satisfied: numpy in /home/nschloe/.local/lib/python3.8/site-packages (from accupy==0.2.0) (1.17.3)
Requirement already satisfied: pybind11>=2.2 in /usr/lib/python3/dist-packages (from accupy==0.2.0) (2.3.dev0)
Requirement already satisfied: pyfma in /home/nschloe/.local/lib/python3.8/site-packages (from accupy==0.2.0) (0.1.1)
Building wheels for collected packages: accupy
  Created temporary directory: /tmp/pip-wheel-vj_mtrg1
  Building wheel for accupy (setup.py) ...   Destination directory: /tmp/pip-wheel-vj_mtrg1
  Running command /usr/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-req-build-k7bjh5uk/setup.py'"'"'; __file__='"'"'/tmp/pip-req-build-k7bjh5uk/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' bdist_wheel -d /tmp/pip-wheel-vj_mtrg1 --python-tag cp38
  running bdist_wheel
  running build
  running build_py
  creating build
  creating build/lib.linux-x86_64-3.8
  creating build/lib.linux-x86_64-3.8/accupy
  copying accupy/dot.py -> build/lib.linux-x86_64-3.8/accupy
  copying accupy/ill_cond.py -> build/lib.linux-x86_64-3.8/accupy
  copying accupy/sums.py -> build/lib.linux-x86_64-3.8/accupy
  copying accupy/__about__.py -> build/lib.linux-x86_64-3.8/accupy
  copying accupy/__init__.py -> build/lib.linux-x86_64-3.8/accupy
  running build_ext
  building '_accupy' extension
  creating build/temp.linux-x86_64-3.8
  creating build/temp.linux-x86_64-3.8/src
  x86_64-linux-gnu-gcc -pthread -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -fwrapv -O2 -g -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -fPIC -I/usr/include/eigen3/ -I/usr/local/include/python3.8 -I/home/nschloe/.local/include/python3.8 -I/usr/include/python3.8 -c src/pybind11.cpp -o build/temp.linux-x86_64-3.8/src/pybind11.o
[...]

Note x86_64-linux-gnu-gcc.

I am on Ubuntu 18.04. I've clone the repo, installed the additional Eigen library, and finally installed the package with pip install --verbose -e .. It all went well and I am able to import the package and use it. I am not setting CXX. Here the output:
Created temporary directory: /tmp/pip-ephem-wheel-cache-c7mj9ozl
Created temporary directory: /tmp/pip-req-tracker-lqlcyskl
Created requirements tracker '/tmp/pip-req-tracker-lqlcyskl'
Created temporary directory: /tmp/pip-install-lak1c9yk
Obtaining file:///work/bfilippo/test_to_cancel/accupy
  Added file:///work/bfilippo/test_to_cancel/accupy to build tracker '/tmp/pip-req-tracker-lqlcyskl'
    Running setup.py (path:/work/bfilippo/test_to_cancel/accupy/setup.py) egg_info for package from file:///work/bfilippo/test_to_cancel/accupy
    Running command python setup.py egg_info
    running egg_info
    creating accupy.egg-info
    writing accupy.egg-info/PKG-INFO
    writing dependency_links to accupy.egg-info/dependency_links.txt
    writing requirements to accupy.egg-info/requires.txt
    writing top-level names to accupy.egg-info/top_level.txt
    writing manifest file 'accupy.egg-info/SOURCES.txt'
    reading manifest file 'accupy.egg-info/SOURCES.txt'
    reading manifest template 'MANIFEST.in'
    writing manifest file 'accupy.egg-info/SOURCES.txt'
  Source in /work/bfilippo/test_to_cancel/accupy has version 0.2.0, which satisfies requirement accupy==0.2.0 from file:///work/bfilippo/test_to_cancel/accupy
  Removed accupy==0.2.0 from file:///work/bfilippo/test_to_cancel/accupy from build tracker '/tmp/pip-req-tracker-lqlcyskl'
Requirement already satisfied: mpmath in /work/bfilippo/miniconda3/envs/cervino/lib/python3.7/site-packages (from accupy==0.2.0) (1.1.0)
Requirement already satisfied: numpy in /work/bfilippo/miniconda3/envs/cervino/lib/python3.7/site-packages (from accupy==0.2.0) (1.17.2)
Requirement already satisfied: pybind11>=2.2 in /work/bfilippo/miniconda3/envs/cervino/lib/python3.7/site-packages (from accupy==0.2.0) (2.4.2)
Requirement already satisfied: pyfma in /work/bfilippo/miniconda3/envs/cervino/lib/python3.7/site-packages (from accupy==0.2.0) (0.1.1)
Installing collected packages: accupy
  Found existing installation: accupy 0.2.0
    Uninstalling accupy-0.2.0:
      Created temporary directory: /tmp/pip-uninstall-0d9fgf3_
      Removing file or directory /work/bfilippo/miniconda3/envs/cervino/lib/python3.7/site-packages/accupy.egg-link
      Removing pth entries from /work/bfilippo/miniconda3/envs/cervino/lib/python3.7/site-packages/easy-install.pth:
      Removing entry: /work/bfilippo/test_to_cancel/accupy
      Successfully uninstalled accupy-0.2.0
  Running setup.py develop for accupy
    Running command /work/bfilippo/miniconda3/envs/cervino/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/work/bfilippo/test_to_cancel/accupy/setup.py'"'"'; __file__='"'"'/work/bfilippo/test_to_cancel/accupy/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' develop --no-deps
    running develop
    running egg_info
    writing accupy.egg-info/PKG-INFO
    writing dependency_links to accupy.egg-info/dependency_links.txt
    writing requirements to accupy.egg-info/requires.txt
    writing top-level names to accupy.egg-info/top_level.txt
    reading manifest file 'accupy.egg-info/SOURCES.txt'
    reading manifest template 'MANIFEST.in'
    writing manifest file 'accupy.egg-info/SOURCES.txt'
    running build_ext
    building '_accupy' extension
    creating build
    creating build/temp.linux-x86_64-3.7
    creating build/temp.linux-x86_64-3.7/src
    gcc -pthread -B /work/bfilippo/miniconda3/envs/cervino/compiler_compat -Wl,--sysroot=/ -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fPIC -I/usr/include/eigen3/ -I/work/bfilippo/miniconda3/envs/cervino/include -I/work/bfilippo/miniconda3/envs/cervino/include -I/work/bfilippo/miniconda3/envs/cervino/include/python3.7m -c src/pybind11.cpp -o build/temp.linux-x86_64-3.7/src/pybind11.o
    cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++
    creating build/lib.linux-x86_64-3.7
    g++ -pthread -shared -B /work/bfilippo/miniconda3/envs/cervino/compiler_compat -L/work/bfilippo/miniconda3/envs/cervino/lib -Wl,-rpath=/work/bfilippo/miniconda3/envs/cervino/lib -Wl,--no-as-needed -Wl,--sysroot=/ build/temp.linux-x86_64-3.7/src/pybind11.o -o build/lib.linux-x86_64-3.7/_accupy.cpython-37m-x86_64-linux-gnu.so
    copying build/lib.linux-x86_64-3.7/_accupy.cpython-37m-x86_64-linux-gnu.so ->
    Creating /work/bfilippo/miniconda3/envs/cervino/lib/python3.7/site-packages/accupy.egg-link (link to .)
    Adding accupy 0.2.0 to easy-install.pth file

    Installed /work/bfilippo/test_to_cancel/accupy
Successfully installed accupy
Cleaning up...
Removed build tracker '/tmp/pip-req-tracker-lqlcyskl'

You can see that it invoked g++ towards the end.

Okay. So this bug report is about how setuptools respects CC, but not CXX, right? I can't see right now how your build tests that.

Aha, yes. setuptools seems to compile src/pybind11.cpp using CC, and the linking happens with CXX. Same here, and the the variables are actually respected.
I'm wondering why src/pybind11.cpp actually compiles with a C compiler.

Dear @jaraco and @pganssle it seems that no checks are implemented on the language variable (I can set it to language='jadshad' and the installation process would still work). If you point me in the right direction (where to look in the repo), I would be happy to try to tackle this issue. Thank you.

Can you make a separate issue for this, @filippo82? I don't want the discussion to be lost in the shuffle of this ticket investigation. I think we need to discuss whether it is feasible to be stricter about the specified language and if so what that would look like.

@pganssle will do!

@nschloe I did some research and it seems that

Distuitls use always the compiler used for compilation of Python (link)
and GCC can actually compile code just fine as long as it links the c++ standard library (link)
From what I see, the process actually works (as I was able to compile accupy) and even setting CC=g++ would lead to the same result. These are just my thoughts.

So is there something that projects need to configure in order to get C++ files compiled with a C++ compiler, or is that simply not possible in setuptools right now, just as it is not possible in distutils? If it's just not possible, can it please be made possible? (Maybe the patches in the distutils bug report are a starting point?) It is imperative that users (and package management systems acting on their behalf) be able to tell setuptools which C++ compiler to use to compile C++ files and what additional CXXFLAGS to set when doing so.

FYI Nix maintains a setuptools patch here: https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/python-modules/setuptools/setuptools-distutils-C%2B%2B.patch


fix: base project configurations

Need to comment on the Github action that added test coverage on PR because was generating a very hard error and to don't get stuck on this I disabled it by now.


injector.Injector.create_object on pydantic-settings BaseSettings does not without factory function in Injector

As stated in this closed bug at pydantic-settings, i cannot achieve an initialization through injector on python 3.8 and 3.9 (works ok on 3.10+) without a factory method.
The issue was turned down by the pydantic-settings team.
The error message

TypeError: unsupported operand type(s) for |: 'type' and 'NoneType'

Why ?
Pydantic BaseSettings base class uses PEP-604 (str | None) annotation instead of PEP-484 (Union, Optional).
It seems that Pydantic BaseSettings class don’t like to be initialized through cls.__new__(cls).
This works
class Configuration(Module):
    def configure(self, binder: Binder):
        binder.bind(BaseSettings, scope=SingletonScope, to=lambda: BaseSettings())

This does not
class Configuration(Module):
    def configure(self, binder: Binder):
        binder.bind(BaseSettings, scope=SingletonScope)

Reproduced issue
https://github.com/Guibod/pydantic-settings-bug-298

Thanks for reporting! Could you please post a stack trace or similar to aid anyone that would be willing to try to solve this?

Hey @Guibod, sorry for your bad experience here.
From a quick look it looks like a pydantic-settings problem – using Python 3.10+ constructs on older Python versions – and I'm not sure trying to find a way to hack around that is the right thing to do.

At least the issue is documented, as well as a work-around. So you don’t have to be sorry about it.
It was discarded by pydantic-settings team, so if you cannot achieve a solution on your own, i’m cool with it.
You can close or leave the ticket open. It’s your choice.


Diff, CE check, SPID, etc

Codecov Report
Attention: 89 lines in your changes are missing coverage. Please review.

Comparison is base (b00a18e) 78.73% compared to head (ca0334b) 85.44%.
 Consider uploading reports for the commit 97684aa to get more accurate results

Files
Patch %
Lines

R/utils.R
0.00%
34 Missing :warning:

R/check_ce_missing_month.R
0.00%
26 Missing :warning:

R/run_all_checks.R
0.00%
15 Missing :warning:

R/check_tr_dup.R
72.72%
3 Missing :warning:

R/check_ae_aeacn_ds_disctx_covid.R
84.61%
2 Missing :warning:

R/check_ae_fatal.R
85.71%
2 Missing :warning:

R/check_ss_ssdtc_dead_ds.R
75.00%
2 Missing :warning:

R/check_ae_aeacnoth_ds_stddisc_covid.R
80.00%
1 Missing :warning:

R/check_ae_aedthdtc_ds_death.R
94.73%
1 Missing :warning:

R/check_ae_aeout_aeendtc_nonfatal.R
90.00%
1 Missing :warning:

... and 2 more

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #287      +/-   ##
==========================================
+ Coverage   78.73%   85.44%   +6.70%
==========================================
  Files         111      113       +2
  Lines        2676     2796     +120
==========================================
+ Hits         2107     2389     +282
+ Misses        569      407     -162

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add how much is my xVADAER worth to Staking

Display how much is my xVADAER worth in similar way as in claimable now etc.

Added in 477ebb62ee3f1f26891d9e88071371c0abe7e6b8 as aggregated balance indicator.


1349 campaign impact goals

fixes #1349

I merged this in too quickly and wanted to say thanks @colhountech for contributing and thanks for taking part in our project!  Every contribution counts and please let us know how we can help you contribute more in the future.  Btw, it looks like a commit snuck in from another PR into yours.  I am guessing it had to do with merging changes form a PR since you had checked out the system.
There may be something in this guide and its linked resources that is helpful to support the workflow with git we are using and @dpaquette and @stevejgordon would be happy to help with any questions.
Thanks again

Thanks @tonysurma for the guidance. I'll go through your workflow see if I can get aligned better.


Migrating to Fire Squid fam

Squidy will be turning-off v5 archives, so we should move to FireSquid ones
Some of them have already been migrated, and some of them are being migrated

[x] click-001-rc0
[x] snekk-dev
[ ] snekk-003 -> gonna be off
[x] snekk-004-rc0
[x] rubick-005
[x] rubick-006
https://github.com/kodadot/nft-gallery/issues/3813

Ballpark eta of turning it off @ 23/8 -> 6-8 weeks -> end of September would be best to have it migrated

For a comprehensive guide on migrating to latest version, you can check our dedicated document page:
https://docs.subsquid.io/migrate/migrate-to-fire-squid

Meanwhile done


Related to issue 72 - feel free to close.

I can't reply to you, because you are closing the issues (and comments there are forbidden), prior to having a chance to explain anything. Feel free to close this as well, but I just want to make my point. Then it is your decision how to proceed (obviously, you already taken it).
This is related to: https://github.com/googleads/videojs-ima/issues/72

No, I don't complain against DFP/DFP SB.
But you are not trying to understand the issue and only find reason to close it.
With other VAST solutions for other players like Flowplayer and JW Player, you can run mid,post-rolls and overlays with DFP/DFP SB. As their architecture allows you to call different VAST tags on different positions during the video. However this plugin only allows single position, which is presumed (by the plugin) to be pre-roll.
The only solution to have mid,post-rolls and overlays is to have VMAP. VMAP is not widely supported.

The goal of this project is to provide a simple way to integrate videojs and the IMA SDK. The IMA SDK already supports midrolls and other scheduling via VMAP, so adding that kind of functionality to the plugin would just create duplicate code and duplicate ways of playing a midroll.
Additionally, VMAP is a much better solution as it allows your ad trafficker to change the ad positions and rules all in one place, rather than having to set up some extra CMS or release a new version of your site to change things.

I see what you're saying now, and @tensafefrogs's answer is the path we've decided to take. If you're looking for mid- and post-roll support, we recommend VMAP responses as they are industry-standard and well supported by the plugin.


Remove pod-plugin and scikit-learn from official docker image

Why are the changes needed?
The flytekit pod plugin has been deprecated for a while and scikit used to be part of the getting started guide (emphasis on used to be).
What changes were proposed in this pull request?
Remove pod plugin and scikit-learn packages from the flytekit image.

How was this patch tested?

Setup process
Screenshots
Check all the applicable boxes

[ ] I updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] All commits are signed-off.

Related PRs

Docs link

As a side effect this removed the kubernetes Python package.
Could we please add kubernetes again?
Otherwise we now have to build custom images just to be able to use pod templates.


Please add/update [smol]

Please read our crate policy. Any issue opened for a new or updated crate version will be closed with a reference to the same policy.
Hi,
Any possibility to add async runtime support in addition to tokio?  For example, smol ?
Thank you!

Please read our crate policy. Any issue opened for a new or updated crate version will be closed with a reference to the same policy.


[Bug] Problem using Monaco 'wordWrapColumn', editor scrolls outside of view

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[X] Not reproducible in the monaco editor playground

Additional info
I don't know if the cause is using it in a Webcomponent, or some other setting. Also it only is broken in Chrome (and derivates). It works in Safari and Firefox.
I also don't know if it is a Monaco Error, a error of me or a Chrome Bug. Maybe someone can help where we need to look (for example who is responsible for this scrolling).
I already also created a Chrome Bug: https://bugs.chromium.org/p/chromium/issues/detail?id=1416178
Reproduction Steps
Much more simple example is now below:
https://github.com/microsoft/monaco-editor/issues/3587#issuecomment-1441530602
has nothing to do with webcomponents nor my designer. It's a problem when "wordWrapColumn" is set and the line is to long.
1.) Go to (the link is only so big, it contains the content of the designer): https://node-projects.github.io/web-component-designer-demo/index.html?html=

2.) Switch to Code
3.) Go to the End of Line 4
4.) Enterer some characters
See also this Video: https://watch.screencastify.com/v/tnSRGDiFCvSDwALTzio4
Actual (Problematic) Behavior
Editor is moved outside of the screen via Scroll Left
Expected Behavior
Editor should work as expected
Additional Context
No response

Hopefully someone can help me, to find the root of that problem.
Maybe look at the chrome Bug, I already tried to change all the scroll methods, so the debugger is hit, but this did not happen.

I can verify the bug, but without a reproducible sample we cannot help you unfortunately.
Please try to recreate the bug in the monaco editor playground.

How should i recreate in Playground? Maybe it is related if the Monaco Editor is wrapped in a webcomponent?

You can also use web components in the playground

@hediet got it reproduced in monaco playground:
https://microsoft.github.io/monaco-editor/playground.html?source=v0.36.0-dev.20230223#XQAAAALXDQAAAAAAAABBqQkHQ5NjdYX3AXhNewrSKZslyM_G2I4bTkHQYW1lB1MplP_B-iGH6S_hAKEQYDCGeVLBbiLOVOAth4exIuI9h-Ns6ao7jNlU0Gr79ecXwsA86mR8U2CnxDf7leW3ZDXDm-bjXCQtmMMIVymXIFkHrT0W3_FLpZ9ih6AeqOUh3Eco9f9AQGrtI9LmRPFtOGqs8kKRDKg0m6rLrfJYjacWmR-mt90zKJAfk-PeSUtHmuokep3BBJRm8HQDyAK1S56TqNTD8KFZBuM8XQOojSeFcGn4KAKFfuAu5fMb_DYwPiJkXjsH6gyitp9julM0nfXIlixAL8vhfjZQWflYEEwx3GkOcAEspWXtzqVXfP9z5UoEofg0xzaiyG5trX1BuzJygArVk1El8ocJ0vnztpJ2-KMMj1pVRUUJuHwd48ZZVKyMu3Y9bpmNV7R2tlOi0U6yWTliO8RNULW9u4NM2Kicc2_1StetRT-Ac1bnzC4lbOvAHVe4q2BLtjBGOBsAU2M9BWh3AQkiWlU6IJMTEbGlLoZum-dkiHpmdOns3f1kwCU3KIpJ-5CcO3AfsSaB9cHid3QnSqqMgO0kY5Nu5RKhVnofOvlwMEJfXCTudSGt0lwEVgMGjeTaYXkeEAZXygAOh4jDV6F1W7dg6Kc8XtMCXmouNJoEXJ1hLMkEtoqzq_4UwnHDo2MlrfR9ktaZNyDkNpkLM2lhzIXtC_-jWgSaA8VqaGvbJHFl269rJOB1HhY1KNDVJVuCVJR-hCLnSUpTQhjGnIWMAg8WWw6Vrc3AFt-Azi5dDm2FpWhXRNYgYC4Sujvjg-7kiPWT_Rq6mpQH5WOrbadBXHyhxo_yjfhtWBJNaEIsxhRpDYpB5G-ENXSFb5uccxcIwL3-bhRgz093SMX3uq30jgKLG5v5sEwOfBdOK_AUTbPSVnfzUtX4qDa1QmXNs3a7-oFmRA2QgCo3e446WqpA2q1teeK8jS0nkt08VvXRfk-Qo4E89P-s6jQgYg4wq0wUgX3gKDOvFZq-vl15Lk3AJJtlMP0YuTneN-TP56D598qQIZEnxzPvQQEnMt58cxdJTKS6Tim3-nwo7Jb5UVX8c7YCW6i4xoOwZwQ81jykRj-YQELPkQb6HpndnCR4uNFMz-clf1aoLl1ILKS-mLSUXaYbsw-_pri6NEedQnGhUwZTsRWdYEGNJdEaJCV2w9c8RBAtl38w32Rxvd9SU3cpckILySDlKw72U07of0WgJ1osaWcmoindLPY0rep0MMGWQSCkwtAByLjkd_BgBQWfl8EvJ8tcALn8GZM8BJocQoDUM9aAbGJpkANNpb2FLb-oAGSJHS4bmBvcBGNHkcHlj5ZXMU0T4S4WyuKUufB0yBksarkjDgs-VpUD7m8KHWydNbqa8Q1G1KUJ65B8b9aSftWLqtXxmQidgWaeFPTQ_y8p_1uIRL-qvfNMNxAYBEnaBHEQIMIqoK2N9uvi45TrF_BR78n0bRrCDfj-Lxn0DO049cO7h5rqoGmSvlwB1G8H_A2gWlhYvFv2uTILQ4btftSLyC5kmFL8vKaDwWHzhm7v5zWXHs90jYCexLLcC3NV-EM_8lUt2vQVzM5Nqi46WWPLS9aOep9cvRBZGYD7eciG_tG-c0MV7pj_8kx2DWTM4bbvujuFxy1Wscx2tsDs7C2kbhOpw5goUFDq_KI3iTh-o_8IycJqg-az4H3VPZ8Jh4-XTAJF8eWjej6zDzEgYyyrdIX-c9t-

If I remove the " wordWrap: 'wordWrapColumn'," it does work

Simplified sample once more:
https://microsoft.github.io/monaco-editor/playground.html?source=v0.36.0-dev.20230223#XQAAAAKbBwAAAAAAAABBqQkHQ5NjdYX3AXhNewrSKZslyM_G2I4bTkHQYW1lB1MplP_B-iGH6S_hAKEQYDCGeVLBbiLOVOAth4exIuI9h-Ns6ao7jNlU0Gr79ecXwsA86mR8U2CnxDf7leW3ZDXDm-bjXCQtmMMIVymXIFkHrT0W3_FLpZ9ih6AeqOUh3Eco9f9AQGrtI9LmRPFtOGqs8jMxl-KTde0KkgLnu5nd_mDMR87RY4JZFkWqO0C8SQvYUQtSrv2cXFsvxAZwUzA3Y9SHmREjGC4ujGDDzoLFUTkfO31D4yMkPml-oc_Je1ZrcmIHTDHA3f-IKpNG3Wcwczy64dDZ4uNku7VK7C4MfnHa4WJsffes67jnn29HI5p6eUD0Ku8pBW7_8InHZJGM_a-rdH4SNMwQsaNG2ECTtAU2tJhr0QPhOCjKFlSvI1-B1EowZfaDrGDu0I8ePfOXfMDyVoCNEbTmTnsd5-yK3TDagpphixjYMeO6R0p2Y-HmTTXlHsr_R_h81LfYJ0bkmDifG2IR9xqJ7XWflN7848sbyxSN_teZWnbklVwqldvecgA1i5avEVWMf7xkahQOOZAufZLqkikkWdqPxiUpglWyV9iLt8mdssSnrulGkw0WpFkWSTpDilkYG0S7wm3LSL-Vs6cmfisYMKy1gUD5Iuvwm7YjgU3oNDKQg8JYMs-KzQ_aAzf9enUPhvx6GocupjyByM0hhnNfUXHLS87PhYdAr2D7WQdI3xyVbSnJpY0Yj8U0ycECAzOe489lR1vzeL70LKZNKE8eI70rYY3i3zpUF-3xp6KRj1kZ5aTEp1Hr5xStRdqDYZGPcqqB1SSu-pzHy8xfLJ33Gjb81NkmV50c74ezMVDdjAPJc2eTyC4KMUzyY7fuBmjGA2AezzGryQYVMx_s75qzyYht7ZhrzAasPEcpftu_loplhp4ZdXWx9Ckydudl4JktcWG9D-tj9NFVSe0DZi_ijj9U1x2QDeTCuJ8KgFtVoGZNOYvg4qhNPzJcDFa4hHQJaBwRwMngU6fvAUUxEY0hRyTFUPL7WYm8sHIPRqZh0Jh3xTM4XpUfnjRw-fnvZjBfuMN22K5SkTWFT4x3rX90j0WvuGlCq_sMBpg

Has nothing to do with webcomponents, also breaks in this sample:
https://microsoft.github.io/monaco-editor/playground.html?source=v0.36.0-dev.20230223#XQAAAALSAwAAAAAAAABBqQkHQ5NjdMjwa-jY7SIQ9S7DNlzs5W-mwj0fe1ZCDRFc9ws9XQE0SJE1jc2VKxhaLFIw9vEWSxW3yscw407USTLdlLqG8408kR6oz8Ayz-6RxBOlazrLF2usiAGPdsdtEEquciiMk1G5Sd0Fc9zUGvG3J5NFhRgKjsIl53PAN3XaYUe0o3P5x6lyLTEji-4ejFqzHWKy30Rk9LEkx9XrccrfkriFj298Luidab5sF_MvuL1LpMVAhAEoj7rVQWisneuszB7rvgkX5pHUCj-qVpjfDlFsl9065e-dChGRX07tpawKcX4jfHbQPZd5dw_-W0EnWw6Peye302JfU_IWUjgdJgq4rK9CE1IfvZZdkSmaP-v8GOCU22zxp5nu4DjkWIrnnnJ5jj5Q8iF66YFhNMzS5XGJiosgxfj1v5vG8vricNKexNC4DTq63m1UoITyVOFLDf4eKWKQmCkokzWFb3GD1ArHI00XJ1TP14CrWbQ6lbC0n9vk_sRwPFQIgy5KOS1c-5buz8kXlUH_cAZFVxIqqTWrM2kPGQVE6HQ4aIgPGxdtUIrdwo9WHd-mmvhlesYXh75qeyfBXf6OWIY

If you reset the "scrollLeft" of the element with class "overflow-guard", display is correct again

@hediet
Is this enough now to reproduce the bug?

The bug seems to be the following focus call scrolling into view the current cursor location:
setSelectionRange(h, f, S) {
                const p = this.c;
                let m = null;
                const b = N.getShadowRoot(p);
                b ? m = b.activeElement : m = document.activeElement;
                const L = m === p
                  , k = p.selectionStart
                  , I = p.selectionEnd;
                if (L && k === f && I === S) {
                    w.isFirefox && window.parent !== window && p.focus();
                    return
                }
                if (L) {
                    this.setIgnoreSelectionChangeTime("setSelectionRange"),
                    p.setSelectionRange(f, S),
                    w.isFirefox && window.parent !== window && p.focus();
                    return
                }
                try {
                    const M = N.saveParentsScrollTop(p);
                    this.setIgnoreSelectionChangeTime("setSelectionRange"),
                    p.focus(),  // <--- This focus call scrolls the input caret into view
                    p.setSelectionRange(f, S),
                    N.restoreParentsScrollTop(p, M)
                } catch {}
            }

This seems to be because the textarea itself is extremely wide. I was able to fix the bug by applying width: 0 !important to the following textarea:
<textarea data-mprt="6" class="inputarea monaco-mouse-cursor-text" wrap="on" autocorrect="off" autocapitalize="off" autocomplete="off" spellcheck="false" aria-label="Editor content;Press Alt+F1 for Accessibility Options." tabindex="0" role="textbox" aria-roledescription="editor" aria-multiline="true" aria-haspopup="false" aria-autocomplete="both" style="tab-size: 28.9062px; font-family: Menlo, Monaco, "Courier New", monospace; font-weight: normal; font-size: 12px; font-feature-settings: "liga" 0, "calt" 0; font-variation-settings: normal; line-height: 18px; letter-spacing: 0px; top: 0px; left: 0px; height: 1px; width: 0px !important;"></textarea>

After that, focusing the text area no longer requires scrolling to the right in order to scroll the text caret into view.

@flackr thanks for investigating! Why is this related to wrapping though?

as I mentioned here: https://github.com/microsoft/monaco-editor/issues/3587#issuecomment-1441530602
it also happens in safari (in a little bit different way)

Right so the browser difference seem to be resulting from different text wrapping in the height: 0  element. if you make the element also width: 0 there will never be a scroll to the right which should prevent the issue here.
I'm not sure whether the text wrapping differences are bugs or just due to different selected fonts and other metrics. If you want to pursue this could you make a simple test page of the textarea contents and selection in cases where you see different behavior?

Blaming points to https://github.com/microsoft/vscode/compare/da1406166702eb790c2629f4c0ced8eaa503cc88...2936f3cab8468cc5f4139a23b462313dcb652c9d , very likely introduced by https://github.com/microsoft/vscode/pull/163229

@alexdima will it be fixed?

I think this can be fixed by using overflow: clip instead of overflow: hidden on the .overflow-guard element? overflow: clip doesn't support any kind of programmatic scrolling, so the p.focus() call that @flackr linked to above won't scroll the element.
Here's a copy of the repro link above, but with a hacky overflow: clip fix
Assuming I'm not missing something, is this something the VSCode team could prioritize? This is a pretty frustrating UX bug with a seemingly trivial fix!

@zelliott If I open your demo with the overflow clip fix, for me it still does not work correctly.
If I click inside of the long row, the cursor position is wrong, but okay, the display works.


simple GUI weight matrix viewer. Useful for images in particular.

Added as part of the 1.1.2 release via the HistCallback class and nvis.  More info available here: http://neon.nervanasys.com/docs/latest/tools.html#layer-histogram-visualization


chore: fix cdn module test

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Save a copy of session data to DraftQuestion

What problem does this pull request solve?
This work follow on from:

https://github.com/alphagov/forms-admin/pull/669
https://github.com/alphagov/forms-admin/pull/658
https://github.com/alphagov/forms-admin/pull/659

This PR is fills a stop gap for us between completely switching from sessions to activerecord.

Makes each Page form object require an instance of DraftQuestion
For each submit method in the form object we basically saving the same data to draft_question instance without validations. The reason for not running validations is because we will run validation against a completed DraftQuestion before creating/updating forms-api page.

Theres a lot of file changes but most are repetitive and best to review commit by commit.
Trello card: https://trello.com/c/RGsOE1bl/1074-switch-all-the-page-form-objects-over-to-use-draftquestion-instead-of-session
Things to consider when reviewing

Ensure that you consider the wider context.
Does it work when run on your machine?
Is it clear what the code is doing?
Do the commit messages explain why the changes were made?
Are there all the unit tests needed?
Has all relevant documentation been updated?

Really nice work @aliuk2012 ! TYSM for putting in so much work on this.